Alexander Wels has posted comments on this change.
Change subject: userportal,webadmin: Frontend refactor
......................................................................
Patch Set 6: (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() {
Exactly, since that would cause GWT.create to be called, which would not allow
me to create unit tests. This way I can setup some mock constants before making
calls.
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 {
No actually I need to make sure that raiseQueryCompleteEvent is fired, even in
the case of ignoreFailure(caught) being true (which immediately returns).
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
I traced all the invocations and concluded that faults NEVER got populated, I
am assuming that some time in the past it used to get populated, but at this
point in time, it doesn't getting populated at all. So I removed it as it
served no purpose.
--
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
