Merge pull request #671 from harshanL/master

Fix for Advanced Search Issue #670

components/device-mgt/org.wso2.carbon.device.mgt.core/src/main/java/org/wso2/carbon/device/mgt/core/search/mgt/impl/QueryBuilderImpl.java

@@ -35,6 +35,7 @@ public class QueryBuilderImpl implements QueryBuilder {
 
     private static final Log log = LogFactory.getLog(QueryBuilderImpl.class);
     private final String WILDCARD_OPERATOR = "%";
+    private final String OR_OPERATOR = "OR";
     private String current_username;
     private boolean isDeviceAdminUser;
 
@@ -131,7 +132,7 @@ public class QueryBuilderImpl implements QueryBuilder {
             for (Condition con : conditions) {
                 if (Utils.checkDeviceDetailsColumns(con.getKey())) {
                     if (con.operator.equals(WILDCARD_OPERATOR)) {
-                        querySuffix = querySuffix + " OR DD." + Utils.getDeviceDetailsColumnNames().get(con.getKey())
+                        querySuffix = querySuffix + " AND DD." + Utils.getDeviceDetailsColumnNames().get(con.getKey())
                                 + " LIKE  ? ";
                         ValueType type = new ValueType();
                         type.setColumnType(ValueType.columnType.STRING);
@@ -215,6 +216,11 @@ public class QueryBuilderImpl implements QueryBuilder {
         } catch (Exception e) {
             throw new InvalidOperatorException("Error occurred while building the sql", e);
         }
+        if (!querySuffix.isEmpty()) {
+            //Replacing the first OR operator as it's unnecessary
+            querySuffix = querySuffix.replaceFirst(OR_OPERATOR, "");
+            querySuffix = " AND (" + querySuffix + ")";
+        }
         return querySuffix;
     }