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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chx’s picture

I am fine with that -- another naming convention would be upgrade vs update but sure as hell it'd be too confusing.

webchick’s picture

Status: Active » Reviewed & tested by the community

Works 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.

dww’s picture

Project: Drupal.org site moderators » Drupal core
Version: » 7.x-dev
Component: Textual improvements » update system
Status: Reviewed & tested by the community » Needs review
FileSize
842 bytes

In 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...

dww’s picture

Status: Needs review » Reviewed & tested by the community

Happened again: #1006302-1: We can't install projects packed as zip
Can we please do this?

Thanks,
-Derek

webchick’s picture

Component: update system » database update system
Status: Reviewed & tested by the community » Fixed

Committed to HEAD, renamed component and moved it under "database system" so it makes sense alphabetically.

dww’s picture

Yay, thank you!!!

aspilicious’s picture

Lol my mistake has been usefull after all :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.