- $s .= '<form action="' . $wgTitle->escapeLocalURL( '-' ) . '" method="get">';
- $prefixedkey = htmlspecialchars($wgTitle->getPrefixedDbKey());
-
- // The following line is SUPPOSED to have double-quotes around the
- // $prefixedkey variable, because htmlspecialchars() doesn't escape
- // single-quotes.
- //
- // On at least two occasions people have changed it to single-quotes,
- // which creates invalid HTML and incorrect display of the resulting
- // link.
- //
- // Please do not break this a third time. Thank you for your kind
- // consideration and cooperation.
- //
- $s .= "<input type='hidden' name='title' value=\"{$prefixedkey}\" />\n";
-
- $s .= $this->submitButton();
+ $s .= Xml::openElement( 'form', array( 'action' => $wgScript, 'id' => 'mw-history-compare' ) );
+ $s .= Xml::hidden( 'title', $wgTitle->getPrefixedDbKey() );
+ if( $wgEnableHtmlDiff ) {
+ $s .= $this->submitButton( wfMsg( 'visualcomparison'),
+ array(
+ 'name' => 'htmldiff',
+ 'class' => 'historysubmit',
+ 'accesskey' => wfMsg( 'accesskey-visualcomparison' ),
+ 'title' => wfMsg( 'tooltip-compareselectedversions' ),
+ )
+ );
+ $s .= $this->submitButton( wfMsg( 'wikicodecomparison'),
+ array(
+ 'class' => 'historysubmit',
+ 'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
+ 'title' => wfMsg( 'tooltip-compareselectedversions' ),
+ )
+ );
+ } else {
+ $s .= $this->submitButton( wfMsg( 'compareselectedversions'),
+ array(
+ 'class' => 'historysubmit',
+ 'accesskey' => wfMsg( 'accesskey-compareselectedversions' ),
+ 'title' => wfMsg( 'tooltip-compareselectedversions' ),
+ )
+ );
+ }