MID-8005 Fixed "stringIgnoreCase" matching rule without namespace doe…

…s not work (cherry picked from commit ec8acff)
Evolveum · Jul 6, 2022 · 4ac85de · 4ac85de
1 parent 33e10af
commit 4ac85de
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/repo/repo-sqale/src/test/java/com/evolveum/midpoint/repo/sqale/func/SqaleRepoSearchTest.java b/repo/repo-sqale/src/test/java/com/evolveum/midpoint/repo/sqale/func/SqaleRepoSearchTest.java
@@ -9,8 +9,7 @@
 import static org.assertj.core.api.Assertions.*;
 import static org.testng.Assert.*;
 
-import static com.evolveum.midpoint.prism.PrismConstants.T_OBJECT_REFERENCE;
-import static com.evolveum.midpoint.prism.PrismConstants.T_PARENT;
+import static com.evolveum.midpoint.prism.PrismConstants.*;
 import static com.evolveum.midpoint.prism.xml.XmlTypeConverter.createXMLGregorianCalendar;
 import static com.evolveum.midpoint.schema.constants.SchemaConstants.ORG_DEFAULT;
 import static com.evolveum.midpoint.util.MiscUtil.asXMLGregorianCalendar;
@@ -191,6 +190,7 @@ public void initObjects() throws Exception {
                         .modifyTimestamp(asXMLGregorianCalendar(2L)))
                 .subtype("workerA")
                 .subtype("workerC")
+                .employeeNumber("user1")
                 .policySituation("situationA")
                 .policySituation("situationC")
                 .activation(new ActivationType(prismContext)
@@ -2383,5 +2383,15 @@ public void test971IsDescendant() throws Exception {
         expect("isDescendant returns false for reverse relationship");
         assertFalse(repositoryService.isDescendant(org11, org112Oid));
     }
+
+    @Test(description = "MID-8005")
+    public void test991SearchObjectWithStringIgnoreCaseWithoutNamespace()
+            throws SchemaException {
+        searchUsersTest("with string item matching ignore-case comparison",
+                f -> f.item(UserType.F_EMPLOYEE_NUMBER).contains("USer1")
+                        // Use QName without namespace
+                        .matching(new QName(STRING_IGNORE_CASE_MATCHING_RULE_NAME.getLocalPart())),
+                user1Oid);
+    }
     // endregion
 }
diff --git a/...main/java/com/evolveum/midpoint/repo/sqlbase/filtering/item/ItemValueFilterProcessor.java b/...main/java/com/evolveum/midpoint/repo/sqlbase/filtering/item/ItemValueFilterProcessor.java
@@ -76,9 +76,11 @@ protected FilterOperation operation(ValueFilter<?, ?> filter) throws QueryExcept
         throw new QueryException("Can't translate filter '" + filter + "' to operation.");
     }
 
+    private static final String STRING_IGNORE_CASE = STRING_IGNORE_CASE_MATCHING_RULE_NAME.getLocalPart();
+
     protected boolean isIgnoreCaseFilter(ValueFilter<?, ?> filter) {
         return filter.getMatchingRule() != null
-                && filter.getMatchingRule().equals(STRING_IGNORE_CASE_MATCHING_RULE_NAME);
+                && filter.getMatchingRule().getLocalPart().equals(STRING_IGNORE_CASE);
     }
 
     @NotNull