Change in ovirt-engine[master]: frontend: add an option of default timezone

tjelinek at redhat.com tjelinek at redhat.com
Mon Mar 25 07:49:18 UTC 2013


Tomas Jelinek has submitted this change and it was merged.

Change subject: frontend: add an option of default timezone
......................................................................


frontend: add an option of default timezone

This patch adds an empty entry in the timezones list presented in the
UI, which represents the selection of the default timezone that is
defined in the system.

This new empty entry is now selected by default when creating new VM,
and when editing a VM that no timezone was defined for it.

This patch fix bug 922609: before this patch the first timezone was
selected by default when opening the edit dialog of VM with no timezone
defined and therefore when pressing 'ok' the system detected a change in
the timezone field and tried to update it - even when it wasn't possible.
from now on, the empty entry will be selected by default in that case so
no change will be detected unless the user actually changed the timezone
field.

Change-Id: I7f8c80ae783e88740ee907d9ff52c0bd6c40734c
Bug-Url: https://bugzilla.redhat.com/922609
Signed-off-by: Arik Hadas <ahadas at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
2 files changed, 13 insertions(+), 27 deletions(-)

Approvals:
  Tomas Jelinek: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/13197
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7f8c80ae783e88740ee907d9ff52c0bd6c40734c
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list