Ever since the update.module went into core as "update.module" (instead of update_status, etc), there's all sorts of confusion about "update system" vs. "update.module". I don't have the bandwidth to closely follow the "update system" queue, so I frequently miss update.module issues that are incorrectly posted there. It's worse now that we have the Update manager, and there are parts of update manager (like authorize.php) that aren't part of update.module itself -- people appear to be even more prone to call such issues part of the "update system" (which makes perfect sense just given the names of the components). I've completely lost track of how many times I've had to move issues from the "update system" to "update.module" or vice versa. However, having to do it twice in the same issue was the straw that broke my back. I can't stand it anymore.
Can we please rename "update system" to "database update system" to attempt to disambiguate WTF that component is really for?
Thanks,
-Derek
Comment | File | Size | Author |
---|---|---|---|
#3 | 990366-3.MAINTAINERS-database-update-system.patch | 842 bytes | dww |
Comments
Comment #1
chx CreditAttribution: chx commentedI am fine with that -- another naming convention would be upgrade vs update but sure as hell it'd be too confusing.
Comment #2
webchickWorks for me. I suggested "update.php" instead thinking that might be even clearer, but dww pointed out that the database update system also includes a fair number of other things (system.install, includes/updater.inc) that go along with it, so it's easier to keep them as part of the same thing.
My guess is that everything's going to still come in through "update.module", since that's the closest thing to "update.php" but at least this will let the Update Manager folks vet each issue before moving it away, as opposed to them being hidden from view completely as they are now.
Comment #3
dwwIn IRC webchick pointed out that MAINTAINERS.txt should be renamed to match. So here's a core patch first. Once this lands, I'll do the deed on d.o itself...
Comment #4
dwwHappened again: #1006302-1: We can't install projects packed as zip
Can we please do this?
Thanks,
-Derek
Comment #5
webchickCommitted to HEAD, renamed component and moved it under "database system" so it makes sense alphabetically.
Comment #6
dwwYay, thank you!!!
Comment #7
aspilicious CreditAttribution: aspilicious commentedLol my mistake has been usefull after all :)