Oved Ourfali has uploaded a new change for review.

Change subject: backend+ui: search filter not working on Foreman Provider
......................................................................

backend+ui: search filter not working on Foreman Provider

Added empty constructor to the parameters, setting them using setters,
to solve some weird serialization issue with the parameters class.

Change-Id: I54110b5bda9a96b0b4a77256f6dbe2e07b1cb80b
Bug-Url: https://bugzilla.redhat.com/988012
Signed-off-by: Oved Ourfali <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetHostListFromExternalProviderParameters.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
2 files changed, 9 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/90/17290/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetHostListFromExternalProviderParameters.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetHostListFromExternalProviderParameters.java
index 8bbcf32..62d2d8f 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetHostListFromExternalProviderParameters.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/GetHostListFromExternalProviderParameters.java
@@ -9,6 +9,10 @@
     private boolean filterOutExistingHosts;
     private String searchFilter;
 
+    public GetHostListFromExternalProviderParameters() {
+        super();
+    }
+
     public GetHostListFromExternalProviderParameters(Guid providerId, boolean 
filterOutExistingHosts, String searchFilter) {
         super();
         this.providerId = providerId;
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
index 2e34b00..9a812ec 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java
@@ -2645,8 +2645,12 @@
                 return source;
             }
         };
+        GetHostListFromExternalProviderParameters params = new 
GetHostListFromExternalProviderParameters();
+        params.setFilterOutExistingHosts(filterOutExistingHosts);
+        params.setProviderId(providerId);
+        params.setSearchFilter(searchFilter);
         Frontend.RunQuery(VdcQueryType.GetHostListFromExternalProvider,
-                new GetHostListFromExternalProviderParameters(providerId, 
filterOutExistingHosts, searchFilter),
+                params,
                 aQuery);
     }
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I54110b5bda9a96b0b4a77256f6dbe2e07b1cb80b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to