Required for:
#1397960: Combine storm attribute module into the storm module
#1392484: Separate Storm Knowledgebase into separate project
This issue will move the installation of specific attributes into the modules that they are used.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | storm--attributes-db-v11.patch | 69.39 KB | juliangb |
| #12 | storm--attributes-db-v10.patch | 69.01 KB | juliangb |
| #11 | storm--attributes-db-v09.patch | 67.74 KB | juliangb |
| #9 | storm--attributes-db-v6.patch | 66.75 KB | juliangb |
| #8 | storm--attributes-db-v4.patch | 66.73 KB | juliangb |
Comments
Comment #1
juliangb commentedThis patch just handles the Storm knowledge base domain, to test.
I'll add to it later for the other domains.
Comment #2
juliangb commentedThis patch needs testing.
It moves all installation of attributes into the modules that use them, and the installation of the actual
stormattributetable into the storm module.It seemed more logical to move the installation of the actual table at the same time as moving the installation of the attributes.
Comment #4
juliangb commentedRemoved whitespace errors.
Comment #6
juliangb commentedCorrected syntax.
Comment #8
juliangb commentedThe exceptions were caused by no attributes being present, as in the tests only the storm attribute module is installed. This should fix.
Comment #9
juliangb commentedThis version of the patch fixes a problem in stormticket when no updates are pending.
Also refixes the whitespace errors that have found their way in again.
Comment #10
juliangb commentedHas to be rerolled.
Comment #11
juliangb commentedReroll to account for changes from last patches committed.
Comment #12
juliangb commentedAdded attribute uninstalls.
Comment #13
juliangb commentedAdded descriptions to the database update functions (display when updating via drush).
Comment #14
juliangb commentedTested on new install, upgrading from 6.x-2.x-dev and from 6.x-1.37.
Committed to 2.x.