fix(userinfo) Sonar fix: merge if statement with enclosing one

issue #6054

fix(userinfo) Sonar fix: merge if statement with enclosing one
issue #6054
acef132f · Côme Chilliet · 2c16686c · acef132f
Unverified Commit acef132f authored 5 years ago by Côme Chilliet
Hide whitespace changes
Inline Side-by-side

Showing

with 3 additions and 5 deletions
+3 -5
--- a/include/class_userinfo.inc
+++ b/include/class_userinfo.inc
@@ -222,11 +222,9 @@ class userinfo
          continue;
        }
-        if (!empty($ACLRule['userfilter'])) {
+        if (!empty($ACLRule['userfilter']) && !$ldap->object_match_filter($this->dn, $ACLRule['userfilter'])) {
-          /* Check if we match the user filter */
+          /* We do not match the user filter */
-          if (!$ldap->object_match_filter($this->dn, $ACLRule['userfilter'])) {
+          continue;
-            continue;
-          }
        }
        if (!empty($ACLRule['targetfilter'])) {