In case of LDAP errors during a deletion, a PHP error is triggered

Description

ldap_remove returns a string instead of an array in case of LDAP errors

PHP error: "array_merge(): Argument #2 is not an array" Trace[1]: fonction array_merge Fichier: /usr/share/fusiondirectory/include/simpleplugin/class_simpleTabs.inc (Ligne 427)

FusionDirectory Version

1.3

Steps to Reproduce

Not sure, you need a delete to go wrong because of an LDAP error, hard to reproduce.

Expected behavior:

Pop-up with error

Actual behavior:

No pop-up, PHP error instead

created branch 5757-ldap_remove-returns-a-string-instead-of-an-array-in-case-of-ldap-errors

By Côme Chilliet on 2018-02-01T10:22:59 (imported from GitLab)

mentioned in merge request !115

By Côme Chilliet on 2018-02-01T10:23:00 (imported from GitLab)

mentioned in commit b0480315

By Côme Chilliet on 2018-02-01T10:24:07 (imported from GitLab)

added fixes label

By Côme Chilliet on 2018-02-01T10:25:07 (imported from GitLab)

added 15m of time spent at 2018-02-01

By Côme Chilliet on 2018-02-01T10:25:27 (imported from GitLab)

mentioned in merge request !117

By Côme Chilliet on 2018-02-01T11:15:19 (imported from GitLab)

added fixes-merged and removed fixes labels

By Côme Chilliet on 2018-02-01T11:16:58 (imported from GitLab)

changed milestone to %FusionDirectory 1.2.1

By Côme Chilliet on 2018-02-01T11:17:00 (imported from GitLab)

closed

By Côme Chilliet on 2018-02-01T13:00:05 (imported from GitLab)

added Fixed label

By Côme Chilliet on 2018-02-01T13:00:25 (imported from GitLab)

changed title from ldap_remove returns a string instead of an array in case of LDAP errors to In case of LDAP errors during a deletion, a PHP error is triggered

By Côme Chilliet on 2018-02-01T13:01:40 (imported from GitLab)

changed the description

By Côme Chilliet on 2018-02-01T13:01:40 (imported from GitLab)

removed Bugs label

By bmortier on 2018-04-05T09:55:19 (imported from GitLab)