Greg Sheremeta has posted comments on this change.

Change subject: userportal, webadmin: introduce AssetProvider
......................................................................


Patch Set 7: Verified+1

(1 comment)

https://gerrit.ovirt.org/#/c/38724/7/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotCreatePopupView.java
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotCreatePopupView.java:

Line 146:     private final static ApplicationConstants constants = 
AssetProvider.getConstants();
Line 147: 
Line 148:     @Inject
Line 149:     public GlusterVolumeSnapshotCreatePopupView(EventBus eventBus,
Line 150:             ApplicationResources resources, ApplicationConstants 
constants,
I missed this one. Fixed in next patchset.
Line 151:             ApplicationTemplates templates) {
Line 152:         super(eventBus);
Line 153:         initEditors();
Line 154:         initWidget(ViewUiBinder.uiBinder.createAndBindUi(this));


-- 
To view, visit https://gerrit.ovirt.org/38724
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifafe5a16068452dac46208c6f06b05c113403443
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Sheremeta <[email protected]>
Gerrit-Reviewer: Alexander Wels <[email protected]>
Gerrit-Reviewer: Greg Sheremeta <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: [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