Lior Vernia has uploaded a new change for review.
Change subject: webadmin: Provider popups refresh tab immediately
......................................................................
webadmin: Provider popups refresh tab immediately
Altered the models to refresh the source tab/subtab upon return of the
performed actions.
Change-Id: I46f1669a8c45b4f690c332126a7e59183681ede3
Signed-off-by: Lior Vernia <lver...@redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/RemoveProvidersModel.java
3 files changed, 34 insertions(+), 10 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/60/15860/1
diff --git
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java
index 1913fb7..ecd44cf 100644
---
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java
+++
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/DiscoverNetworksModel.java
@@ -19,7 +19,10 @@
import org.ovirt.engine.ui.uicommonweb.dataprovider.AsyncDataProvider;
import org.ovirt.engine.ui.uicommonweb.models.ListModel;
import org.ovirt.engine.ui.uicommonweb.models.Model;
+import org.ovirt.engine.ui.uicommonweb.models.SearchableListModel;
import org.ovirt.engine.ui.uicompat.ConstantsManager;
+import org.ovirt.engine.ui.uicompat.FrontendActionAsyncResult;
+import org.ovirt.engine.ui.uicompat.IFrontendActionAsyncCallback;
@SuppressWarnings("deprecation")
public class DiscoverNetworksModel extends Model {
@@ -27,7 +30,7 @@
private static final String CMD_DISCOVER = "OnDiscover"; //$NON-NLS-1$
private static final String CMD_CANCEL = "Cancel"; //$NON-NLS-1$
- private final ListModel sourceListModel;
+ private final SearchableListModel sourceListModel;
private final Provider provider;
private ListModel dataCenters;
@@ -49,7 +52,7 @@
this.networkList = networkList;
}
- public DiscoverNetworksModel(ListModel sourceListModel, Provider provider)
{
+ public DiscoverNetworksModel(SearchableListModel sourceListModel, Provider
provider) {
this.sourceListModel = sourceListModel;
this.provider = provider;
@@ -133,7 +136,13 @@
}
}
- Frontend.RunMultipleAction(VdcActionType.AddNetwork,
mulipleActionParameters);
+ Frontend.RunMultipleActions(VdcActionType.AddNetwork,
mulipleActionParameters, new IFrontendActionAsyncCallback() {
+
+ @Override
+ public void executed(FrontendActionAsyncResult result) {
+ sourceListModel.getSearchCommand().execute();
+ }
+ });
cancel();
}
diff --git
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java
index 8d3ddde..4f095af 100644
---
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java
+++
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/ProviderModel.java
@@ -22,6 +22,7 @@
import org.ovirt.engine.ui.uicommonweb.models.EntityModel;
import org.ovirt.engine.ui.uicommonweb.models.ListModel;
import org.ovirt.engine.ui.uicommonweb.models.Model;
+import org.ovirt.engine.ui.uicommonweb.models.SearchableListModel;
import org.ovirt.engine.ui.uicommonweb.validation.IValidation;
import org.ovirt.engine.ui.uicommonweb.validation.NotEmptyValidation;
import org.ovirt.engine.ui.uicommonweb.validation.UrlValidation;
@@ -42,7 +43,7 @@
private static final String CMD_CANCEL_IMPORT = "CancelImport";
//$NON-NLS-1$
private static final String EMPTY_ERROR_MESSAGE = ""; //$NON-NLS-1$
- protected final ListModel sourceListModel;
+ protected final SearchableListModel sourceListModel;
private final VdcActionType action;
protected final Provider provider;
@@ -110,7 +111,7 @@
return testResult;
}
- public ProviderModel(ListModel sourceListModel, VdcActionType action,
Provider provider) {
+ public ProviderModel(SearchableListModel sourceListModel, VdcActionType
action, Provider provider) {
this.sourceListModel = sourceListModel;
this.action = action;
this.provider = provider;
@@ -210,7 +211,13 @@
protected void actualSave() {
flush();
- Frontend.RunAction(action, new ProviderParameters(provider));
+ Frontend.RunAction(action, new ProviderParameters(provider), new
IFrontendActionAsyncCallback() {
+
+ @Override
+ public void executed(FrontendActionAsyncResult result) {
+ sourceListModel.getSearchCommand().execute();
+ }
+ });
cancel();
}
diff --git
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/RemoveProvidersModel.java
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/RemoveProvidersModel.java
index 8cc164f..9c6068b 100644
---
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/RemoveProvidersModel.java
+++
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/providers/RemoveProvidersModel.java
@@ -11,8 +11,10 @@
import org.ovirt.engine.ui.frontend.Frontend;
import org.ovirt.engine.ui.uicommonweb.UICommand;
import org.ovirt.engine.ui.uicommonweb.models.ConfirmationModel;
-import org.ovirt.engine.ui.uicommonweb.models.ListModel;
+import org.ovirt.engine.ui.uicommonweb.models.SearchableListModel;
import org.ovirt.engine.ui.uicompat.ConstantsManager;
+import org.ovirt.engine.ui.uicompat.FrontendActionAsyncResult;
+import org.ovirt.engine.ui.uicompat.IFrontendActionAsyncCallback;
@SuppressWarnings("deprecation")
public class RemoveProvidersModel extends ConfirmationModel {
@@ -20,11 +22,11 @@
private static final String CMD_REMOVE = "OnRemove"; //$NON-NLS-1$
private static final String CMD_CANCEL = "Cancel"; //$NON-NLS-1$
- private final ListModel sourceListModel;
+ private final SearchableListModel sourceListModel;
private final List<Provider> providers;
@SuppressWarnings("unchecked")
- public RemoveProvidersModel(ListModel sourceListModel) {
+ public RemoveProvidersModel(SearchableListModel sourceListModel) {
this.sourceListModel = sourceListModel;
providers = (List<Provider>) sourceListModel.getSelectedItems();
@@ -58,7 +60,13 @@
parameterList.add(new ProviderParameters(provider));
}
- Frontend.RunMultipleAction(VdcActionType.RemoveProvider,
parameterList);
+ Frontend.RunMultipleActions(VdcActionType.RemoveProvider,
parameterList, new IFrontendActionAsyncCallback() {
+
+ @Override
+ public void executed(FrontendActionAsyncResult result) {
+ sourceListModel.getSearchCommand().execute();
+ }
+ });
cancel();
}
--
To view, visit http://gerrit.ovirt.org/15860
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I46f1669a8c45b4f690c332126a7e59183681ede3
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lver...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
