[syncope] branch master updated: [SYNCOPE-1609] Cleanup

[ilgrosso]

This is an automated email from the ASF dual-hosted git repository.

ilgrosso pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/syncope.git


The following commit(s) were added to refs/heads/master by this push:
     new 419894f  [SYNCOPE-1609] Cleanup
419894f is described below

commit 419894f3c2b123650fe43b56378b7f02c6d7db3f
Author: Francesco Chicchiriccò <ilgro...@apache.org>
AuthorDate: Tue Jan 12 14:16:45 2021 +0100

    [SYNCOPE-1609] Cleanup
---
 .../core/persistence/jpa/dao/PGJPAJSONAnySearchDAO.java      | 12 +++++-------
 .../core/persistence/jpa/dao/AbstractAnySearchDAO.java       |  4 ++--
 2 files changed, 7 insertions(+), 9 deletions(-)

diff --git 
a/core/persistence-jpa-json/src/main/java/org/apache/syncope/core/persistence/jpa/dao/PGJPAJSONAnySearchDAO.java
 
b/core/persistence-jpa-json/src/main/java/org/apache/syncope/core/persistence/jpa/dao/PGJPAJSONAnySearchDAO.java
index 3fc86cc..59623db 100644
--- 
a/core/persistence-jpa-json/src/main/java/org/apache/syncope/core/persistence/jpa/dao/PGJPAJSONAnySearchDAO.java
+++ 
b/core/persistence-jpa-json/src/main/java/org/apache/syncope/core/persistence/jpa/dao/PGJPAJSONAnySearchDAO.java
@@ -160,7 +160,7 @@ public class PGJPAJSONAnySearchDAO extends 
AbstractJPAJSONAnySearchDAO {
                     if (schema.getType() == AttrSchemaType.String || 
schema.getType() == AttrSchemaType.Enum) {
                         
query.append("jsonb_path_exists(").append(schema.getKey()).append(", '$[*] ? ").
                                 append("(@.").append(key).append(" like_regex 
\"").
-                                append(value.replaceAll("%", ".*")).
+                                append(value.replace("%", ".*")).
                                 append("\"").
                                 append(lower ? " flag \"i\"" : 
"").append(")')");
                     } else {
@@ -417,9 +417,8 @@ public class PGJPAJSONAnySearchDAO extends 
AbstractJPAJSONAnySearchDAO {
 
         StringBuilder query = new StringBuilder("(");
         if (cond.isFromGroup()) {
-            realmDAO.findDescendants(realm).forEach(current -> {
-                query.append("realm_id=?").append(setParameter(parameters, 
current.getKey())).append(" OR ");
-            });
+            realmDAO.findDescendants(realm).forEach(current -> 
query.append("realm_id=?").
+                    append(setParameter(parameters, 
current.getKey())).append(" OR "));
             query.setLength(query.length() - 4);
         } else {
             for (Realm current = realm; current.getParent() != null; current = 
current.getParent()) {
@@ -711,10 +710,9 @@ public class PGJPAJSONAnySearchDAO extends 
AbstractJPAJSONAnySearchDAO {
     protected StringBuilder buildOrderBy(final OrderBySupport obs) {
         StringBuilder orderBy = new StringBuilder();
 
-        obs.items.forEach(item -> {
-            orderBy.append(item.orderBy).append(',');
-        });
         if (!obs.items.isEmpty()) {
+            obs.items.forEach(item -> 
orderBy.append(item.orderBy).append(','));
+
             orderBy.insert(0, " ORDER BY ");
             orderBy.deleteCharAt(orderBy.length() - 1);
         }
diff --git 
a/core/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/AbstractAnySearchDAO.java
 
b/core/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/AbstractAnySearchDAO.java
index 479efe4..1748778 100644
--- 
a/core/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/AbstractAnySearchDAO.java
+++ 
b/core/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/AbstractAnySearchDAO.java
@@ -241,10 +241,10 @@ public abstract class AbstractAnySearchDAO extends 
AbstractDAO<Any<?>> implement
         }
 
         List<String> matching = cond.getGroup().indexOf('%') == -1
-                ? 
Optional.ofNullable(groupDAO.findKey(cond.getGroup())).map(List::of).orElseGet(()
 -> List.of())
+                ? 
Optional.ofNullable(groupDAO.findKey(cond.getGroup())).map(List::of).orElseGet(List::of)
                 : groupDAO.findKeysByNamePattern(cond.getGroup());
         if (matching.isEmpty()) {
-            LOG.error("Could not find group(s) for '" + cond.getGroup() + 
'\'');
+            LOG.error("Could not find group(s) for '{}'", cond.getGroup());
             throw new IllegalArgumentException();
         }