r104066 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r104065‎ | r104066 | r104067 >
Date:19:18, 23 November 2011
Author:reedy
Status:resolved (Comments)
Tags:
Comment:
MW_SUPPORTS_EDITFILTERMERGED has been in since r27416

Kill the alternate EditFilter hook usage (underlying code still used, probably wants refactoring at somepoint)

Simplify a boolean return
Modified paths:
  • /trunk/extensions/ConfirmEdit/Captcha.php (modified) (history)
  • /trunk/extensions/ConfirmEdit/ConfirmEdit.php (modified) (history)
  • /trunk/extensions/ConfirmEdit/ConfirmEditHooks.php (modified) (history)

Diff [purge]

Index: trunk/extensions/ConfirmEdit/Captcha.php
@@ -645,11 +645,7 @@
646646 */
647647 private function doConfirmEdit( $editPage, $newtext, $section, $merged = false ) {
648648 if ( $this->shouldCheck( $editPage, $newtext, $section, $merged ) ) {
649 - if ( $this->passCaptcha() ) {
650 - return true;
651 - } else {
652 - return false;
653 - }
 649+ return $this->passCaptcha();
654650 } else {
655651 wfDebug( "ConfirmEdit: no need to show captcha.\n" );
656652 return true;
@@ -686,7 +682,6 @@
687683 return $this->confirmEdit( $editPage, $newtext, false, true );
688684 }
689685
690 -
691686 function confirmEditAPI( $editPage, $newtext, &$resultArr ) {
692687 if ( !$this->doConfirmEdit( $editPage, $newtext, false, false ) ) {
693688 $this->addCaptchaAPI( $resultArr );
Index: trunk/extensions/ConfirmEdit/ConfirmEdit.php
@@ -185,11 +185,7 @@
186186 $wgExtensionMessagesFiles['ConfirmEdit'] = "$wgConfirmEditIP/ConfirmEdit.i18n.php";
187187 $wgExtensionAliasesFiles['ConfirmEdit'] = "$wgConfirmEditIP/ConfirmEdit.alias.php";
188188
189 -if ( defined( 'MW_SUPPORTS_EDITFILTERMERGED' ) ) {
190 - $wgHooks['EditFilterMerged'][] = 'ConfirmEditHooks::confirmEditMerged';
191 -} else {
192 - $wgHooks['EditFilter'][] = 'ConfirmEditHooks::confirmEdit';
193 -}
 189+$wgHooks['EditFilterMerged'][] = 'ConfirmEditHooks::confirmEditMerged';
194190 $wgHooks['UserCreateForm'][] = 'ConfirmEditHooks::injectUserCreate';
195191 $wgHooks['AbortNewAccount'][] = 'ConfirmEditHooks::confirmUserCreate';
196192 $wgHooks['LoginAuthenticateAudit'][] = 'ConfirmEditHooks::triggerUserLogin';
Index: trunk/extensions/ConfirmEdit/ConfirmEditHooks.php
@@ -17,10 +17,6 @@
1818 return $wgCaptcha;
1919 }
2020
21 - static function confirmEdit( $editPage, $newtext, $section ) {
22 - return self::getInstance()->confirmEdit( $editPage, $newtext, $section );
23 - }
24 -
2521 static function confirmEditMerged( $editPage, $newtext ) {
2622 return self::getInstance()->confirmEditMerged( $editPage, $newtext );
2723 }

Follow-up revisions

RevisionCommit summaryAuthorDate
r104083Fix hook copy paste fail...reedy20:37, 23 November 2011

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r27416Added EditFilterMerged hook: like EditFilter but uses the text after section ...tstarling07:30, 12 November 2007

Comments

#Comment by Raymond (talk | contribs)   20:57, 23 November 2011
mediawiki-bw_: /w/api.php Exception from line 245 of /www/w/includes/Hooks.php: Detected bug in an extension! Hook ConfirmEditHooks::APIEditBeforeSave failed to return a value; should return true to continue hook processing or false to abort.
#Comment by Reedy (talk | contribs)   21:11, 23 November 2011

Was already fixed in r104083

Status & tagging log