Active
Project:
Configuration Management
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
22 Jun 2012 at 17:49 UTC
Updated:
6 Dec 2012 at 20:25 UTC
Jump to comment: Most recent file
Comments
Comment #1
helior commentedComment #2
dragonwize commentedThe gotcha here that needs more thinking is if configuration module is being used correctly then the config files and dir will be non-writable by drupal in production. This is why we also have a download option to backport config from prod in cases where DB changes were made there and need to be backported.
The correct way would be that the changes would not be made on live and only made on dev then committed and pushed to live. However, as much as we try, those situations will happen with the vast general public and even ourselves if we are not careful or get into a situation.
I think it might be good enough to just make a check that the dir is writeable, maybe even on the module level not just uninstall as this info is good in other situations. That way it is possible to uninstall the module without removing the config and instead of the user getting PHP errors they should just get a notice from us saying we were unable to remove the files.
Comment #3
dagmar