search

backdrop-contrib / backdrop_upgrade_status

Checks to see if the installed modules on a Drupal 7 site are ready for upgrade to Backdrop CMS.
1 stars 5 forks source link

This module sometimes makes dangerous / misleading suggestions #28

Closed indigoxela closed 1 year ago

indigoxela commented 2 years ago

From our Zulip chat:

the amazing backdrop-upgrade report says: disable and uninstall module Variable in my drupal 7. Is that a mistake? Lots of other modules with "leave enabled" depend on it.

The depending module(s) are i18n (mostly) - but in an update hook i18n grabs data from the variable_store table to not lose them.

So suggesting to uninstall the Variable module on D7 already, can lead to data loss.

jenlampton commented 1 year ago

Yes, if the variable module deletes anything when uninstalled, that would be dangerous. I'll update the recommendation to leave enabled. Having data in that table in backdrop is also not a risk. It's definitely safer to leave it than loose something!