Change in ovirt-engine[master]: userportal, webadmin: Blank template has active action button...

gchaplik at redhat.com gchaplik at redhat.com
Thu Jul 19 10:18:26 UTC 2012


Gilad Chaplik has submitted this change and it was merged.

Change subject: userportal,webadmin: Blank template has active action buttons (#840856)
......................................................................


userportal,webadmin: Blank template has active action buttons (#840856)

https://bugzilla.redhat.com/840856

Have at least one non-blank template. Select first the non-blank,
than the blank. The action buttons will be enabled.

Fixed by enriching the login of
TemplateListModel.isBlankTemplateSelected() which now checks
all the selected templates (getSelectedItems()) and not only
the first selected (getSelectedItem()).

This logic is common for both webadmin and userportal,
so this fix fixes the same problem in userportal as well.

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

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3959752d734c01d6d1b20afeacbecaa59f2b6608
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez 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