Last modified: 2014-04-17 15:33:43 UTC
Do we really need to reinvent the wheel? Thought not.
Because two seconds on Terminal are easier than five minutes peering at your Database Manager Of Choice's query window. Nobody likes messing around with the db more than can be helped, hence the handy install script. What's the big deal?
I think the OP is referring to the fact that it could use the Updater classes (and log the update as completed) via update.php.
(In reply to comment #1) > Because two seconds on Terminal are easier than five minutes peering at your > Database Manager Of Choice's query window. Nobody likes messing around with the > db more than can be helped, hence the handy install script. What's the big > deal? That has nothing to do with it... Reedy is talking about why its rolling its own method instead of using the defacto standard that uses the update.php magic.
At the time the script was written, I think update.php and friends sucked and didn't really support this stuff too well.
(In reply to comment #3) > (In reply to comment #1) > > Because two seconds on Terminal are easier than five minutes peering at your > > Database Manager Of Choice's query window. Nobody likes messing around with the > > db more than can be helped, hence the handy install script. What's the big > > deal? > > That has nothing to do with it... Reedy is talking about why its rolling its > own method instead of using the defacto standard that uses the update.php > magic. Ah, I misunderstood. I thought his gripe was with that the extension makes the db changes with a script, rather than posting the query stuff on the MWwiki extension page - for pasting into a query window. I know a lot of people who run their own sites don't have shell access and have to find workarounds.
I've changed the summary to clarify that we probably want to kill off that installer script. IIRC there *are* already updater hooks in which should take care of setup when maintenance/update.php is run. Should check to confirm there's nothing else that installer does that's necessary and go ahead and drop it out and update documentation to indicate using the regular updater channels.
*** Bug 32456 has been marked as a duplicate of this bug. ***