anmolbabu has posted comments on this change.

Change subject: webadmin: Fix for NPE in remove brick status dialog
......................................................................


Patch Set 2: Code-Review+1

(2 comments)

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeRebalanceStatusModel.java
Line 140:                 showStatus(rebalanceEntity);
Line 141:             }
Line 142:         }), volumeEntity.getClusterId(), volumeEntity.getId());
Line 143:     }
Line 144: 
Done
Line 145:     public GlusterVolumeEntity getEntity() {
Line 146:         return entity;
Line 147:     }
Line 148: 


Line 163:     }
Line 164: 
Line 165:     public void setStatusAvailable(boolean isStatusAvailable) {
Line 166:         this.isStatusAvailable = isStatusAvailable;
Line 167:         if (isStatusAvailable == true) {
Done
Line 168:             onPropertyChanged(new 
PropertyChangedEventArgs("IS_STATUS_APPLICABLE"));//$NON-NLS-1$
Line 169:         }
Line 170:     }
Line 171: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibfe886c240a6d55798a1ad021c58dc874b8cff12
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Ramesh N <[email protected]>
Gerrit-Reviewer: anmolbabu <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to