The Media module currently contains a number of variables which actually belong to its submodules. These variables should be moved to their respective submodules for consistency, performance and code readability.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | migrate-submodule-variables-2147221-1.patch | 13.05 KB | devin carlson |
Comments
Comment #1
devin carlson commentedA patch which adds an update function to move variables to submodules as appropriate, implements
hook_uninstall()in all relevant submodules to remove the variables on uninstall and accommodates the minor changes caused by the variable shuffle.Comment #2
devin carlson commentedTested #1 a fresh Media install with all submodules enabled to verify that nothing was broken and on an existing Media install to verify that the update function successfully migrated each of the variables.
Committed #1 to Media 7.x-2.x.