search

backdrop-contrib / simplesamlphp_auth

Support SAML for authentication of users. The module will auto-provision user accounts and dynamically assign roles in Backdrop CMS if you want it to.
https://backdropcms.org/project/simplesamlphp_auth
GNU General Public License v2.0
2 stars 3 forks source link

Remove the correct variables in `simplesamlphp_auth_update_1000()` #8

Closed argiepiano closed 1 year ago

argiepiano commented 1 year ago

All the update_variable_del() calls in simplesamlphp_auth_update_1000() pass the wrong variable names to the delete function. For example, you want to delete simplesamlphp_auth_activate, not activate (which doesn't exist).

This means that all those D7 variables are kept in the database even after the upgrade.

laryn commented 1 year ago

👍 I noticed this, too, when looking at another issue.