From 09df26371ec2c33f692eadcb9ed90a0cd0c31c0e Mon Sep 17 00:00:00 2001 From: Krinkle Date: Sun, 8 Jan 2012 22:15:23 +0000 Subject: [PATCH] [HistoryAction] Revert r108341 as this breaks revision delete. This construction never looked cool, but looks like this fix is not the right way, so reverting it. This form is used for two purposes: -- Submitting diff/oldid to (implied) action=view -- Submitting ids-array to action=revisiondelete That makes it complicated. Also for IE6/IE7 compatibility, there is no way to do this from the HTML itself (i.e. using would work but leaves no option to have a custom i18n label. and works but IE7 submits "I18N LABEL" as value for "action" instead of "revision delete". Will re-fix bug 33587 in a few minutes with a "progressive-enhancement" solution from the front-end instead. --- includes/actions/HistoryAction.php | 1 + 1 file changed, 1 insertion(+) diff --git a/includes/actions/HistoryAction.php b/includes/actions/HistoryAction.php index 8a34aa4ced..aa1ec45c70 100644 --- a/includes/actions/HistoryAction.php +++ b/includes/actions/HistoryAction.php @@ -407,6 +407,7 @@ class HistoryPager extends ReverseChronologicalPager { $s = Html::openElement( 'form', array( 'action' => $wgScript, 'id' => 'mw-history-compare' ) ) . "\n"; $s .= Html::hidden( 'title', $this->getTitle()->getPrefixedDbKey() ) . "\n"; + $s .= Html::hidden( 'action', 'historysubmit' ) . "\n"; $s .= '
' . $this->submitButton( $this->msg( 'compareselectedversions' )->text(), array( 'class' => 'historysubmit' ) ) . "\n"; -- 2.20.1