Last modified: 2014-09-15 00:45:57 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 T70930, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 68930 - Store the page_id of the redirect page in log_params when a page is moved without suppressing the redirect
Store the page_id of the redirect page in log_params when a page is moved wit...
Status: PATCH_TO_REVIEW
Product: MediaWiki
Classification: Unclassified
Database (Other open bugs)
1.24rc
All All
: Normal normal (vote)
: ---
Assigned To: Nathan Larson
:
: 69837 (view as bug list)
Depends on:
Blocks: 68311
  Show dependency treegraph
 
Reported: 2014-07-31 17:02 UTC by Nathan Larson
Modified: 2014-09-15 00:45 UTC (History)
3 users (show)

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


Attachments

Description Nathan Larson 2014-07-31 17:02:47 UTC
Store the page_id of the moved page in log_params. Currently, as noted in bug 57084, log_page contains the page_id of the redirect that is created when a page is moved, or it contains 0 when no redirect is created.
Comment 1 Gerrit Notification Bot 2014-07-31 17:10:54 UTC
Change 150864 had a related patch set uploaded by leucosticte:
Store the page_id of the moved page in log_params Currently, as noted in bug 57084, log_page contains the page_id of the redirect that is created when a page is moved, or it contains 0 when no redirect is created. This change stores the page_id of the mov

https://gerrit.wikimedia.org/r/150864
Comment 2 Nathan Larson 2014-07-31 17:49:25 UTC
Do we want to call it something less ambiguous than "6::pageid" (since there are two page ids involved when there's a move with a redirect creation)?
Comment 3 Aaron Halfaker 2014-07-31 23:14:20 UTC
Good Q.  How about "moved_pageid"?
Comment 4 Nathan Larson 2014-07-31 23:34:26 UTC
(In reply to Aaron Halfaker from comment #3)
> Good Q.  How about "moved_pageid"?

Is it allowable to have an underscore, or should it be movedpageid?
Comment 5 Gerrit Notification Bot 2014-08-01 02:30:58 UTC
Change 150969 had a related patch set uploaded by leucosticte:
Make log_params contain null revision rev_id

https://gerrit.wikimedia.org/r/150969
Comment 6 Gerrit Notification Bot 2014-08-01 19:16:22 UTC
Change 150969 had a related patch set uploaded by leucosticte:
Make log_params contain null revision rev_id and moved page ID

https://gerrit.wikimedia.org/r/150969
Comment 7 Gerrit Notification Bot 2014-08-02 14:07:50 UTC
Change 150864 abandoned by leucosticte:
Store the page_id of the moved page in log_params

Reason:
Superseded by https://gerrit.wikimedia.org/r/#/c/150969/

https://gerrit.wikimedia.org/r/150864
Comment 8 Brad Jorsch 2014-08-21 15:11:24 UTC
*** Bug 69837 has been marked as a duplicate of this bug. ***
Comment 9 Nathan Larson 2014-09-14 22:41:18 UTC
A new patch is needed, since this is one is obsolete after Gerrit change #157872.
Comment 10 Gerrit Notification Bot 2014-09-15 00:39:58 UTC
Change 160368 had a related patch set uploaded by leucosticte:
action=logevents now provides redirpageid for page move events

https://gerrit.wikimedia.org/r/160368
Comment 11 Gerrit Notification Bot 2014-09-15 00:45:57 UTC
Change 150969 abandoned by leucosticte:
Make log_params contain null revision rev_id and moved page ID

Reason:
Superseded by Gerrit change #160368

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

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


Navigation
Links