Daniel Erez has posted comments on this change.
Change subject: userportal,webadmin: Frontend refactor
......................................................................
Patch Set 6: Looks good to me, but someone else must approve
(3 inline comments)
....................................................
File
frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java
Line 161: }
Line 162: failureEventHandler(errorMessage);
Line 163: }
Line 164:
Line 165: static UIConstants getConstants() {
why not saving constants as a static variable instead?
Line 166: if (constants == null) {
Line 167: constants = ConstantsManager.getInstance().getConstants();
Line 168: }
Line 169: return constants;
Line 386: (ArrayList<VdcQueryParametersBase>) queryParamsList,
Line 387: new AsyncCallback<ArrayList<VdcQueryReturnValue>>() {
Line 388: @Override
Line 389: public void onFailure(final Throwable caught) {
Line 390: try {
1. what could throw an exception?
2. don't we want to catch and log it?
Line 391: if (ignoreFailure(caught)) {
Line 392: return;
Line 393: }
Line 394: logger.log(Level.SEVERE, "Failed to execute
RunPublicQuery: " + caught, caught); //$NON-NLS-1$
....................................................
File
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/FrontendEventsHandlerImpl.java
Line 48
Line 49
Line 50
Line 51
Line 52
don't we want to support multiple faults any more?
--
To view, visit http://gerrit.ovirt.org/15399
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I3dae5a455ad16660b97cddafba53b53d3b4196f1
Gerrit-PatchSet: 6
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: Greg Sheremeta <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
