Change in ovirt-engine[master]: webadmin: fix unable to save priority (#840006)

gchaplik at redhat.com gchaplik at redhat.com
Sun Aug 5 08:21:50 UTC 2012


Gilad Chaplik has submitted this change and it was merged.

Change subject: webadmin: fix unable to save priority (#840006)
......................................................................


webadmin: fix unable to save priority (#840006)

https://bugzilla.redhat.com/840006

When a user try to create new vm or edit an exsisting one, the priority
is not saved.

The max priority value and the VM priority value where bothe cached to
the same key. once the vm priority was 1 - the max priority was 1 and
the 3 integer options for choosing priority where all equal 1.

Change-Id: I863fa7a00c3208d6137bd982f9702fe850d42bed
Signed-off-by: Ofri Masad <omasad at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
3 files changed, 78 insertions(+), 65 deletions(-)

Approvals:
  Gilad Chaplik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I863fa7a00c3208d6137bd982f9702fe850d42bed
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: ofri masad <omasad at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: ofri masad <omasad at redhat.com>



More information about the Engine-commits mailing list