https://www.mediawiki.org/wiki/Special:Code/MediaWiki/112935

Revision: 112935
Author:   aaron
Date:     2012-03-03 10:47:37 +0000 (Sat, 03 Mar 2012)
Log Message:
-----------
Optimized onTitleMoveComplete() to avoid master DB queries

Modified Paths:
--------------
    trunk/extensions/FlaggedRevs/backend/FlaggedRevs.hooks.php

Modified: trunk/extensions/FlaggedRevs/backend/FlaggedRevs.hooks.php
===================================================================
--- trunk/extensions/FlaggedRevs/backend/FlaggedRevs.hooks.php  2012-03-03 
10:41:11 UTC (rev 112934)
+++ trunk/extensions/FlaggedRevs/backend/FlaggedRevs.hooks.php  2012-03-03 
10:47:37 UTC (rev 112935)
@@ -57,27 +57,27 @@
 
        /**
         * (a) Update flaggedrevs page/tracking tables
-        * (b) Autoreview pages moved into content NS
+        * (b) Autoreview pages moved into reviewable namespaces (bug 19379)
         */
        public static function onTitleMoveComplete(
                Title $otitle, Title $ntitle, $user, $pageId
        ) {
-               $fa = FlaggableWikiPage::getTitleInstance( $ntitle );
-               $fa->loadPageData( 'fromdbmaster' );
-               // Re-validate NS/config (new title may not be reviewable)
-               if ( $fa->isReviewable() ) {
-                       // Moved from non-reviewable to reviewable NS?
-                       // Auto-review such edits like new pages...
-                       if ( !FlaggedRevs::inReviewNamespace( $otitle )
-                               && FlaggedRevs::autoReviewNewPages()
-                               && $ntitle->userCan( 'autoreview' ) )
-                       {
+               if ( !FlaggedRevs::inReviewNamespace( $otitle )
+                       && FlaggedRevs::inReviewNamespace( $ntitle )
+                       && FlaggedRevs::autoReviewNewPages() 
+               ) {
+                       $fa = FlaggableWikiPage::getTitleInstance( $ntitle );
+                       $fa->loadPageData( 'fromdbmaster' );
+                       // Re-validate NS/config (new title may not be 
reviewable)
+                       if ( $fa->isReviewable() && $ntitle->userCan( 
'autoreview' ) ) {
+                               // Auto-review such edits like new pages...
                                $rev = Revision::newFromTitle( $ntitle );
                                if ( $rev ) { // sanity
                                        FlaggedRevs::autoReviewEdit( $fa, 
$user, $rev );
                                }
                        }
                }
+
                # Update page and tracking tables and clear cache
                FlaggedRevs::stableVersionUpdates( $otitle );
                FlaggedRevs::HTMLCacheUpdates( $otitle );


_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs

Reply via email to