You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@syncope.apache.org by il...@apache.org on 2022/02/06 07:56:12 UTC
[syncope] branch 2_1_X updated: [SYNCOPE-1663] pgjsonb was missing
This is an automated email from the ASF dual-hosted git repository.
ilgrosso pushed a commit to branch 2_1_X
in repository https://gitbox.apache.org/repos/asf/syncope.git
The following commit(s) were added to refs/heads/2_1_X by this push:
new 0211426 [SYNCOPE-1663] pgjsonb was missing
0211426 is described below
commit 0211426a5b5c46adbfdb3e6beb7253f99eeaabe7
Author: Francesco Chicchiriccò <il...@apache.org>
AuthorDate: Sun Feb 6 08:55:55 2022 +0100
[SYNCOPE-1663] pgjsonb was missing
---
.../persistence/jpa/dao/PGJPAJSONAnySearchDAO.java | 42 ++++------------------
1 file changed, 6 insertions(+), 36 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 c22771c..9a14d7e 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
@@ -225,12 +225,7 @@ public class PGJPAJSONAnySearchDAO extends JPAAnySearchDAO {
final List<Object> parameters,
final SearchSupport svs) {
- Pair<PlainSchema, PlainAttrValue> checked;
- try {
- checked = check(cond, svs.anyTypeKind);
- } catch (IllegalArgumentException e) {
- return ALWAYS_FALSE_ASSERTION;
- }
+ Pair<PlainSchema, PlainAttrValue> checked = check(cond, svs.anyTypeKind);
StringBuilder query = new StringBuilder();
@@ -406,12 +401,7 @@ public class PGJPAJSONAnySearchDAO extends JPAAnySearchDAO {
final List<Object> parameters,
final SearchSupport svs) {
- Realm realm;
- try {
- realm = check(cond);
- } catch (IllegalArgumentException e) {
- return ALWAYS_FALSE_ASSERTION;
- }
+ Realm realm = check(cond);
StringBuilder query = new StringBuilder().append('(');
if (cond.isFromGroup()) {
@@ -437,12 +427,7 @@ public class PGJPAJSONAnySearchDAO extends JPAAnySearchDAO {
final List<Object> parameters,
final SearchSupport svs) {
- String memberKey;
- try {
- memberKey = check(cond);
- } catch (IllegalArgumentException e) {
- return ALWAYS_FALSE_ASSERTION;
- }
+ String memberKey = check(cond);
StringBuilder query = new StringBuilder().append('(');
@@ -508,12 +493,7 @@ public class PGJPAJSONAnySearchDAO extends JPAAnySearchDAO {
final List<Object> parameters,
final SearchSupport svs) {
- String rightAnyObjectKey;
- try {
- rightAnyObjectKey = check(cond);
- } catch (IllegalArgumentException e) {
- return ALWAYS_FALSE_ASSERTION;
- }
+ String rightAnyObjectKey = check(cond);
StringBuilder query = new StringBuilder().append('(');
@@ -540,12 +520,7 @@ public class PGJPAJSONAnySearchDAO extends JPAAnySearchDAO {
final List<Object> parameters,
final SearchSupport svs) {
- List<String> groupKeys;
- try {
- groupKeys = check(cond);
- } catch (IllegalArgumentException e) {
- return ALWAYS_FALSE_ASSERTION;
- }
+ List<String> groupKeys = check(cond);
String where = groupKeys.stream().
map(key -> "group_id=?" + setParameter(parameters, key)).
@@ -597,12 +572,7 @@ public class PGJPAJSONAnySearchDAO extends JPAAnySearchDAO {
cond.setExpression(realm.getKey());
}
- Triple<PlainSchema, PlainAttrValue, AnyCond> checked;
- try {
- checked = check(cond, svs.anyTypeKind);
- } catch (IllegalArgumentException e) {
- return ALWAYS_FALSE_ASSERTION;
- }
+ Triple<PlainSchema, PlainAttrValue, AnyCond> checked = check(cond, svs.anyTypeKind);
StringBuilder query = new StringBuilder();