Aaron Schulz has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/55017


Change subject: Call loadPageData() as needed in Title::moveToInternal.
......................................................................

Call loadPageData() as needed in Title::moveToInternal.

* This avoids use of a slave for loading the page ID to do
  the updates using $newpage. That bug prevented page moves
  by using the old 0 ID and throwing an exception.

bug : 45594
Change-Id: Iea3259dce6840e3f2959d98a20177acd60433b64
---
M includes/Title.php
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/17/55017/1

diff --git a/includes/Title.php b/includes/Title.php
index e81023a..84848eb 100644
--- a/includes/Title.php
+++ b/includes/Title.php
@@ -3838,6 +3838,7 @@
 
                $this->resetArticleID( 0 );
                $nt->resetArticleID( $oldid );
+               $newpage->loadPageData( WikiPage::READ_LOCKING ); // bug 46397
 
                $newpage->updateRevisionOn( $dbw, $nullRevision );
 

-- 
To view, visit https://gerrit.wikimedia.org/r/55017
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iea3259dce6840e3f2959d98a20177acd60433b64
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>

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

Reply via email to