From 6479b425011ed90a7e6ef46b827f1e0e84f5cb94 Mon Sep 17 00:00:00 2001 From: River Tarnell Date: Thu, 8 May 2008 16:06:21 +0000 Subject: [PATCH] on SJS Web Server with action paths, 'action' is always set. this caused WatchlistEditor::getMode() to ignore $par when deciding if the user is editing the watchlist, so it would always consider the action to be view. fix by giving precedence to $par over action. --- includes/WatchlistEditor.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/includes/WatchlistEditor.php b/includes/WatchlistEditor.php index 172cf4bcb3..6a39acb60f 100644 --- a/includes/WatchlistEditor.php +++ b/includes/WatchlistEditor.php @@ -431,7 +431,8 @@ class WatchlistEditor { * @return int */ public static function getMode( $request, $par ) { - $mode = strtolower( $request->getVal( 'action', $par ) ); + if (($mode = $par) === "") + $mode = strtolower( $request->getVal( 'action', "" ) ); switch( $mode ) { case 'clear': return self::EDIT_CLEAR; -- 2.20.1