Change in ovirt-engine[master]: webadmin: New pool contained template provisioning (#839046)

gchaplik at redhat.com gchaplik at redhat.com
Tue Jul 24 11:46:40 UTC 2012


Gilad Chaplik has submitted this change and it was merged.

Change subject: webadmin: New pool contained template provisioning (#839046)
......................................................................


webadmin: New pool contained template provisioning (#839046)

https://bugzilla.redhat.com/839046

If there are no templates in the environment, than:
Pools->New->Resource Allocation
It contains the Template Provisioning.
The template provisioning makes actually no sense for the
pool dialog, so it never should be there.

It was there, because the logic which disabled it was in the
Template_SelectedItemChanged method which is called when the
selected template changes. The problem is, that when there is
no template in the system at all, this method gets never called.

Fixed by moving this logic into the Initialize method.

Change-Id: I5ade45b807f35161511e352ebd0ffb818730fcee
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java
1 file changed, 3 insertions(+), 1 deletion(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5ade45b807f35161511e352ebd0ffb818730fcee
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list