The weird _copy fallback should be removed

The move method in plugin class calls rename, if rename fails it calls copy and then ldap->rmdir_recursive. This copy method calls copy, which contains weird code calling directly ldap* methods and using a plugin::rebind method which seems wrong (the array key used are the wrong ones).

We should remove this whole fallback mechanism, if ldap_rename fails the rename should fail and the admin needs to fix the LDAP server.

(from redmine: issue id 4652, created on 2016-03-24, closed on 2016-04-04)

• Changesets:
  • Revision 3e4d2e0c by Côme Chilliet on 2016-03-24T10:50:16.000Z:

Fixes #4652 Remove the fallback mechanism of rename

• Revision 9ca02056 by Côme Chilliet on 2016-03-24T10:51:26.000Z:

Fixes #4652 Remove the fallback mechanism of rename

• Custom Fields:
  • Bug in version: 1.0.10

This can be closed as soon as we are sure the selenium tests are still passing.

(from redmine: written on 2016-04-04)

By Côme Chilliet on 2017-09-02T15:27:43 (imported from GitLab)

Close issue

(from redmine: written on 2016-04-04)

By Jonathan Swaelens on 2017-09-02T15:27:43 (imported from GitLab)

closed

By Jonathan Swaelens on 2017-09-02T15:27:43 (imported from GitLab)

added Removed label

By bmortier on 2018-10-05T09:59:49 (imported from GitLab)