Merge branch 'cherry-pick-29ca9876' into '1.3.1-fixes'

Merge branch 'stricter-ldap-error-check' into '1.3.1-fixes'

See merge request fusiondirectory/fd!649

include/class_ldap.inc

@@ -881,7 +881,7 @@ class LDAP
    */
   function success()
   {
-    return preg_match('/Success/i', $this->error);
+    return (trim($this->error) === 'Success');
   }
 
   /*!

[bmortier]

SonarQube analysis reported 1 issue

• 1 minor

Note: The following issues were found on lines that were not modified in the commit. Because these issues can't be reported as line comments, they are summarized here:

1. Rename class "tabs_configInLdap" to match the regular expression ^[a-zA-Z][a-zA-Z0-9]*$.

By Ghost User on 2019-07-29T12:17:08 (imported from GitLab)

[bmortier]

SonarQube analysis reported 1 issue

• 1 minor

Note: The following issues were found on lines that were not modified in the commit. Because these issues can't be reported as line comments, they are summarized here:

1. Rename class "tabs_configInLdap" to match the regular expression ^[a-zA-Z][a-zA-Z0-9]*$.

By Ghost User on 2019-08-28T09:08:27 (imported from GitLab)

[bmortier]

SonarQube analysis reported 1 issue

• 1 minor

Note: The following issues were found on lines that were not modified in the commit. Because these issues can't be reported as line comments, they are summarized here:

1. Rename class "tabs_configInLdap" to match the regular expression ^[a-zA-Z][a-zA-Z0-9]*$.

By Ghost User on 2019-08-28T12:30:16 (imported from GitLab)