r90738 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r90737‎ | r90738 | r90739 >
Date:21:36, 24 June 2011
Author:bawolff
Status:ok
Tags:
Comment:
follow-up r90737 - also normalize 'createtalk' for good measure.

Note, atm not such a big deal as there are no direct uses of 'createtalk' permission
but good to check for potential future bugs.
Modified paths:
  • /trunk/extensions/TitleBlacklist/TitleBlacklist.hooks.php (modified) (history)

Diff [purge]

Index: trunk/extensions/TitleBlacklist/TitleBlacklist.hooks.php
@@ -29,7 +29,7 @@
3030 # Some places check createpage, while others check create.
3131 # As it stands, upload does createpage, but normalize both
3232 # to the same action, to stop future similar bugs.
33 - if( $action === 'createpage' ) {
 33+ if( $action === 'createpage' || $action === 'createtalk' ) {
3434 $action = 'create';
3535 }
3636 if( $action == 'create' || $action == 'edit' || $action == 'upload' ) {

Follow-up revisions

RevisionCommit summaryAuthorDate
r92265MFT r90737, r90738 to 1.17wmf1, REL1_18reedy18:20, 15 July 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r90737(bug 28466) Make titleblack list consider 'createpage' action to be same as '...bawolff21:25, 24 June 2011

Status & tagging log