| Index: trunk/phase3/includes/api/ApiMove.php |
| — | — | @@ -107,7 +107,6 @@ |
| 108 | 108 | $r['subpages'] = $this->moveSubpages($fromTitle, $toTitle, |
| 109 | 109 | $params['reason'], $params['noredirect']); |
| 110 | 110 | $this->getResult()->setIndexedTagName($r['subpages'], 'subpage'); |
| 111 | | - // TODO: Should we move talk subpages if moving the talk page failed? |
| 112 | 111 | if($params['movetalk']) |
| 113 | 112 | { |
| 114 | 113 | $r['subpages-talk'] = $this->moveSubpages($fromTalk, $toTalk, |
| Index: trunk/phase3/includes/specials/SpecialMovepage.php |
| — | — | @@ -378,6 +378,8 @@ |
| 379 | 379 | # would mean that you couldn't move them back in one operation, which |
| 380 | 380 | # is bad. FIXME: A specific error message should be given in this |
| 381 | 381 | # case. |
| | 382 | + |
| | 383 | + // FIXME: Use Title::moveSubpages() here |
| 382 | 384 | $dbr = wfGetDB( DB_MASTER ); |
| 383 | 385 | if( $this->moveSubpages && ( |
| 384 | 386 | MWNamespace::hasSubpages( $nt->getNamespace() ) || ( |