Daniel Erez has posted comments on this change.
Change subject: webadmin: expand all expands selected node
......................................................................
Patch Set 1: (2 inline comments)
....................................................
File
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/tree/SystemTree.java
Line 122: }
Line 123:
Line 124: private void collapseTree(TreeNode node) {
Line 125: TreeNode parent = node.getParent();
Line 126: if(parent == null) {
If you're using Eclipse, just enable 'Format edited lines' on
'Window->Preferences->Java Editor->Save Actions'.
Line 127: node.setChildOpen(0, false);
Line 128: } else {
Line 129: parent.setChildOpen(node.getIndex(), false);
Line 130: }
Line 135: if(node == null) {
Line 136: return null;
Line 137: }
Line 138:
Line 139: int i = 0;
Still can be converted to a 'for' loop with early return which might be
clearer, but fair enough..
Line 140: while(result == null && i < node.getChildCount()) {
Line 141: if(null != model && model.equals(node.getChildValue(i))) {
Line 142: result = node.setChildOpen(i, true);
Line 143: break;
--
To view, visit http://gerrit.ovirt.org/10913
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I110efdff51be6e5545a1de304bdce657ec5cda88
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <[email protected]>
Gerrit-Reviewer: Alexander Wels <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
