Last modified: 2009-02-21 08:56:56 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T19300, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 17300 - Redirects don't work after page move (regression)
Redirects don't work after page move (regression)
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Redirects (Other open bugs)
1.15.x
All All
: Normal normal with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
http://en.wikipedia.org/wiki/Ang_Baba...
:
: 17276 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-02-01 11:41 UTC by P.Copp
Modified: 2009-02-21 08:56 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description P.Copp 2009-02-01 11:41:50 UTC
This is maybe related to bug 17276: When moving a page over a redirect, the new created redirect (at the old title) isn't working, probably stored with page_is_redirect = 0 (see url).

Apparently the behavior has been introduced with last scap.
Comment 1 Ryan Schmidt 2009-02-01 23:53:23 UTC
cannot duplicate on my test wiki running the latest svn rev. It seems wikimedia scapped to r46424, and I did some cleanup of the new redirect code I introduced in r46502. It appears that the changed redirect code from r46502 does not have this regression.

If this regression has indeed been fixed in that later revision, the page will likely remain broken until it is re-moved (moving the original article back and forth should do the trick).
Comment 2 Tim Starling 2009-02-10 02:16:39 UTC
Fixed with one-line patch against r46424.

Index: includes/Article.php
===================================================================
--- includes/Article.php	(revision 46424)
+++ includes/Article.php	(working copy)
@@ -1274,7 +1274,7 @@
 		wfProfileIn( __METHOD__ );
 
 		$text = $revision->getText();
-		$rt = Title::newFromRedirect( $text );
+		$rt = Title::newFromRedirect( $text, false, true );
 
 		$conditions = array( 'page_id' => $this->getId() );
 		if( !is_null( $lastRevision ) ) {
Comment 3 Tim Starling 2009-02-10 02:47:56 UTC
Running fixup script.
Comment 4 Niklas Laxström 2009-02-21 08:56:56 UTC
*** Bug 17276 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links