Page MenuHomePhabricator

Store the page_id of the redirect page in log_params when a page is moved without suppressing the redirect
Open, MediumPublic

Description

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.


See Also:
T59084: Store the page_id of the moved page in log_page

Details

Reference
bz68930

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:40 AM
bzimport set Reference to bz68930.

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

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)?

Good Q. How about "moved_pageid"?

(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?

Change 150969 had a related patch set uploaded by leucosticte:
Make log_params contain null revision rev_id

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

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

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

  • Bug 69837 has been marked as a duplicate of this bug. ***

A new patch is needed, since this is one is obsolete after gerrit 157872.

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

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

Krenair subscribed.