Page MenuHomePhabricator

Moving a page and its subpages to a subpage yields an extra move
Closed, ResolvedPublic

Description

Author: mike.lifeguard+bugs

Description:
"One" has been moved to "One/test"

  • The page One/test has been moved to One/test/test.
  • The page One/two has been moved to One/test/two.

So "One" is moved, then moved again because it is a subpage. So this needs to check subpages /before/ moving anything so that if creating subpages is part of the moving process they are not moved more than once. Here we have created a double redirect by moving One -> One/test -> One/test/test erroneously.


Version: 1.13.x
Severity: normal

Details

Reference
bz14368

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:12 PM
bzimport set Reference to bz14368.
bzimport added a subscriber: Unknown Object (MLST).

Fixed in r35762; same problem as bug 14258. The list of additional pages was being pulled from a slave server, which was not up to date, and ended up including the original page, so it tried to move it twice.