Last modified: 2010-12-12 18:35:38 UTC
There is malformed html in the 'label' tags for the minor edit and watchlist checkboxes on the Edit page. This causes rendering errors if served as XHTML. The malformed code: <label for='wpMinoredit' id='mw-editpage-minoredit'Mark this as a minor edit [i]>This is a minor edit</label> <label for='wpWatchthis' id='mw-editpage-watch'Add this page to your watchlist [w]>Watch this page</label></div> This occurs on lines 2355 and 2368 of EditPage.php; the following is put into the label tag: $skin->titleAttrib( 'minoredit', 'withaccess' ) ... and I have no idea why. The diff below removes this and fixes the bug. Index: includes/EditPage.php =================================================================== --- includes/EditPage.php (revision 78212) +++ includes/EditPage.php (working copy) @@ -2355,7 +2355,7 @@ ); $checkboxes['minor'] = Xml::check( 'wpMinoredit', $checked['minor'], $attribs ) . - " <label for='wpMinoredit' id='mw-editpage-minoredit'" . $skin->titleAttrib( 'minoredit', 'withaccess' ) . ">{$minorLabel}</label>"; + " <label for='wpMinoredit' id='mw-editpage-minoredit'>{$minorLabel}</label>"; } $watchLabel = wfMsgExt( 'watchthis', array( 'parseinline' ) ); @@ -2368,7 +2368,7 @@ ); $checkboxes['watch'] = Xml::check( 'wpWatchthis', $checked['watch'], $attribs ) . - " <label for='wpWatchthis' id='mw-editpage-watch'" . $skin->titleAttrib( 'watch', 'withaccess' ) . ">{$watchLabel}</label>"; + " <label for='wpWatchthis' id='mw-editpage-watch'>{$watchLabel}</label>"; } wfRunHooks( 'EditPageBeforeEditChecks', array( &$this, &$checkboxes, &$tabindex ) ); return $checkboxes;
Tracing to r75763.
r78264 Thanks for the report! :)