| Index: trunk/phase3/includes/SpecialMovepage.php |
| — | — | @@ -363,7 +363,14 @@ |
| 364 | 364 | ); |
| 365 | 365 | # The following line is an atrocious hack. Kill it with fire. |
| 366 | 366 | $newNs = $nt->getNamespace() + ($oldPage->getNamespace() & 1); |
| 367 | | - $newPage = Title::makeTitle( $newNs, $newPageName ); |
| | 367 | + # Bug 14385: we need makeTitleSafe because the new page names may |
| | 368 | + # be longer than 255 characters. |
| | 369 | + $newPage = Title::makeTitleSafe( $newNs, $newPageName ); |
| | 370 | + if( !$newPage ) { |
| | 371 | + $oldLink = $skin->makeKnownLinkObj( $oldPage ); |
| | 372 | + $extraOutput []= wfMsgHtml( 'movepage-page-unmoved', $oldLink, Title::makeName( $newNs, $newPageName ) ); |
| | 373 | + continue; |
| | 374 | + } |
| 368 | 375 | |
| 369 | 376 | # This was copy-pasted from Renameuser, bleh. |
| 370 | 377 | if ( $newPage->exists() && !$oldPage->isValidMoveTarget( $newPage ) ) { |
| Index: trunk/phase3/RELEASE-NOTES |
| — | — | @@ -323,6 +323,7 @@ |
| 324 | 324 | * (bug 11659) Urldecode image names in galleries |
| 325 | 325 | * (bug 14258, 14368) Fix for subpage renames in replication environments |
| 326 | 326 | * (bug 14367) Failed block no longer adds phantom watchlist entry |
| | 327 | +* (bug 14385) "Move subpages" option no longer tries to move to invalid titles |
| 327 | 328 | |
| 328 | 329 | |
| 329 | 330 | === API changes in 1.13 === |