r54225 - Code Review

From MediaWiki.org
Jump to: navigation, search
Repository:MediaWiki
Revision:< r54224 | r54225 (on ViewVC) | r54226 >
Date:19:35, 2 August 2009
Author:demon
Status:resolved (Comments)
Tags:
Comment:Merge maintenance-work branch (now with less errors!):
* Docs have been updated to indicate the standard on how to write maintenance scripts (MW.org docs will follow) Have ported vast majority of maintenance scripts to new format. Remaining ones (mostly FiveUpgrade-related) are a bit more tricky. commandLine.inc is untouched for now. Many have gotten code-style updates as well. Deleted .inc files were only used by their .php counterparts, and have been merged into single files.
* (bug 11867) Lock error on redirect table when running orphans.php
* (bug 16322) Allow maintenance scripts to accept DB user/pass over input or params
* (bug 18566) Maintenance script to un/protect pages
* initStats overhaul, now uses class SiteStatsInit. Also fixes bug 18930
Modified paths:

Follow-up revisions

RevisionCommit summaryAuthorDate
r54583

Fix apparent regression in r54225

werdna14:44, 7 August 2009
r54839

Revert eval.php changes from r54653, r54312, r54225 (maintenance-work branch mer...

tstarling03:44, 12 August 2009
r57807

Readd line break which was lost in r54225

ialex08:58, 16 October 2009
r58049

fix regression introduced by r54225 - special page skipping was ignored by maint...

midom09:37, 23 October 2009
r58050

upwards merge from deployment-58049, fix regression introduced by r54225 - speci...

midom09:40, 23 October 2009
r68513

Follow up r54225. DumpRenderer::handleRevision needs to be public in order to be...

platonides12:44, 24 June 2010
r71288

Followup r54225

reedy11:32, 19 August 2010

Comments

#Comment by ^demon (Talk | contribs)   21:31, 2 August 2009

Followup revs r54226, r54227, r54228, r54229, r54230, r54232, r54233, r54235, r54236, r54237.

#Comment by ^demon (Talk | contribs)   21:56, 2 August 2009

Also r54238, r54239, r54240.

#Comment by Brion VIBBER (Talk | contribs)   02:33, 3 August 2009

There's some real nasty bugs here; updaters are totally broken, attempting to include files that don't exist.

require_once( 'populateCategory.inc' ); require_once( 'populateParentId.inc' ); require_once( 'populateLogSearch.inc' );

That's just on first look... :(

#Comment by ^demon (Talk | contribs)   02:46, 3 August 2009

populateCategory.inc was still there, restored parentId and LogSearch in r54253

#Comment by Simetrical (Talk | contribs)   17:03, 10 August 2009

When running generateSitemap.php on near-trunk: 

Notice: Use of undefined constant NS_MAIN - assumed 'NS_MAIN' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_TALK - assumed 'NS_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_USER - assumed 'NS_USER' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_USER_TALK - assumed 'NS_USER_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_PROJECT - assumed 'NS_PROJECT' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_PROJECT_TALK - assumed 'NS_PROJECT_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_FILE - assumed 'NS_FILE' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_FILE_TALK - assumed 'NS_FILE_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_MEDIAWIKI - assumed 'NS_MEDIAWIKI' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_MEDIAWIKI_TALK - assumed 'NS_MEDIAWIKI_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_TEMPLATE - assumed 'NS_TEMPLATE' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_TEMPLATE_TALK - assumed 'NS_TEMPLATE_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_HELP - assumed 'NS_HELP' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_HELP_TALK - assumed 'NS_HELP_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_CATEGORY - assumed 'NS_CATEGORY' in /var/www/w/maintenance/doMaintenance.php on line 41

Notice: Use of undefined constant NS_CATEGORY_TALK - assumed 'NS_CATEGORY_TALK' in /var/www/w/maintenance/doMaintenance.php on line 41
#Comment by ^demon (Talk | contribs)   17:11, 10 August 2009

Resolved in r54728.

#Comment by Brion VIBBER (Talk | contribs)   23:17, 19 August 2009

We seem to have sorted out the bugs by now. :)

#Comment by Tim Starling (Talk | contribs)   06:45, 4 September 2009

doMaintenance.php is a hair's breadth away from being an arbitrary script execution vulnerability. Does that not count as a bug?

#Comment by Tim Starling (Talk | contribs)   22:58, 4 September 2009

Fixed in r55807.

#Comment by Midom (Talk | contribs)   09:43, 23 October 2009

Skipping of disabled cached querypages wasn't functional, fixed in r58050

#Comment by Platonides (Talk | contribs)   18:34, 25 July 2010

What's the point of that $DP used in loadWikimediaSettings() ?

I only see it assigned there, and another time $DP = $IP; in wikimedia CommonSettings.php No more usages.

#Comment by ^demon (Talk | contribs)   18:07, 26 July 2010

It was in the old commandLine.inc. I only kept it for back-compat. Didn't think it was really used though. If a shell can confirm it isn't actually used anywhere, we can kill it.

Status & tagging log

Retrieved from "http://www.mediawiki.org/wiki/Special:Code/MediaWiki/54225"
Personal tools
Namespaces
Variants
Views
Actions
Site
Support
Download
Development
Communication
Toolbox