LDAP *: Uninstall Issues [#1898960]

Comment #1

johnbarclay CreditAttribution: johnbarclay commented 28 January 2013 at 04:47

Title:

uninstall doesn't work for ldap_servers - reinstall doesn't create ldap_servers table

» LDAP Servers: uninstall doesn't work for ldap_servers - reinstall doesn't create ldap_servers table

What does the system database table say in the ldap_server record in the "schema_version" column?

If you manually want to reinstall, be sure to remove all the ldap_* records in the system table.

Log in or register to post comments

Comment #2

johnbarclay CreditAttribution: johnbarclay commented 10 February 2013 at 19:18

Version:	7.x-2.0-beta1	» 7.x-2.x-dev
Assigned:	Unassigned	» johnbarclay
Priority:	Normal	» Major

I'm not sure about the downgrade path, but the uninstall needs to remove everything.

Log in or register to post comments

Comment #3

johnbarclay CreditAttribution: johnbarclay commented 27 February 2013 at 13:45

Title:	LDAP Servers: uninstall doesn't work for ldap_servers - reinstall doesn't create ldap_servers table	» LDAP *: Uninstall Issues
Priority:	Major	» Normal

Currently the uninstall code should remove all ldap_* tables, all ldap_* variables, and all ldap fields. I just committd a patch to get rid of the last ldap_user variables. The only data it should leave in $user->data[] data which will need a cron/batch process to remove. I'm leaving this issue open as a catch all for uninstall issues until they are all worked through. My experience has been that uninstall code is unreliable to test with simpletest coverage.

Log in or register to post comments

Comment #4

CodyS CreditAttribution: CodyS commented 5 March 2013 at 15:12

I am using 7.x-2.0-beta3 and I disabled all LDAP modules and uninstalled them.

But the LDAP modules still appear in my module list disabled and no longer show up in my Uninstall list.

Log in or register to post comments

Comment #5

johnbarclay CreditAttribution: johnbarclay commented 5 March 2013 at 15:44

This thread is for 7.x-2.x-dev

Log in or register to post comments

Comment #6

johnbarclay CreditAttribution: johnbarclay commented 1 April 2013 at 04:07

Priority:

Normal

» Major

Log in or register to post comments

Comment #7

johnbarclay CreditAttribution: johnbarclay commented 3 April 2013 at 01:30

Status:

Active

» Needs review

@#4, I can't reproduce this. Is there a dependency or were the fields in ldap_user not removed?

The most recent commit covers removal of $user->data['ldap_authorizations'] data on uninstall. This is the last of the code for uninstalling ldap_* completely. Please test.

All tables, variables, and $user->data related to ldap should be removed on uninstall.