Change in ovirt-engine[master]: webadmin, userportal: Fix action table element IDs

vszocs at redhat.com vszocs at redhat.com
Tue May 7 13:17:21 UTC 2013


Vojtech Szocs has submitted this change and it was merged.

Change subject: webadmin,userportal: Fix action table element IDs
......................................................................


webadmin,userportal: Fix action table element IDs

This patch fixes a problem in AbstractActionTable which uses
two table widgets internally (optional header + main table).

Since both above mentioned table widgets are now derived from
ColumnResizeCellTable, and sice column definition objects are
shared between both tables, we must not set element IDs for
the optional header table.

Change-Id: I318338a8e92b3967b4ce40ac288add846f0a1444
Bug-Url: https://bugzilla.redhat.com/960519
Signed-off-by: Vojtech Szocs <vszocs at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/AbstractActionTable.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/ElementIdCellTable.java
2 files changed, 7 insertions(+), 6 deletions(-)

Approvals:
  Vojtech Szocs: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I318338a8e92b3967b4ce40ac288add846f0a1444
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>



More information about the Engine-commits mailing list