Lior Vernia has uploaded a new change for review.

Change subject: webadmin: Fixed event listener assignment (Bugzilla)
......................................................................

webadmin: Fixed event listener assignment (Bugzilla)

Assign a newly-created listener to the right listener member. This is
a response to bugzilla warning UWF_UNWRITTEN_FIELD.

Change-Id: I35bb5fb094168e44bd77f4847067aa94b7c1fe22
Signed-off-by: Lior Vernia <lver...@redhat.com>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/HorizontalSplitTable.java
1 file changed, 7 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/42/21342/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/HorizontalSplitTable.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/HorizontalSplitTable.java
index 6cbc469..9500101b 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/HorizontalSplitTable.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/HorizontalSplitTable.java
@@ -162,6 +162,7 @@
         if (listener != null) {
             
table.asEditor().flush().getItemsChangedEvent().removeListener(listener);
         }
+
         listener = new IEventListener() {
 
             @Override
@@ -169,7 +170,13 @@
                 getSelectionModel(topTableIsEdited).clear();
             }
         };
+        if (topTableIsEdited) {
+            topItemsChangedListener = listener;
+        } else {
+            bottomItemsChangedListener = listener;
+        }
         model.getItemsChangedEvent().addListener(listener);
+
         table.asEditor().edit(model);
     }
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I35bb5fb094168e44bd77f4847067aa94b7c1fe22
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lver...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to