From 0dd1a446c856e5d9bc260d01c1f7662364a0f3ea Mon Sep 17 00:00:00 2001 From: Tim Starling Date: Tue, 12 Nov 2013 14:57:22 +1100 Subject: [PATCH] thumb.php: map b/c params prior to attempting redirection This should fix most of the "No width specified" exceptions. I guess most of this code will be rewritten to fix bug 56926, superseding this patch, but I thought this might be a useful stopgap. Bug: 56924 Change-Id: I5bcaaafc81e1e2a8a33819fa64790d5509dc6aa6 --- thumb.php | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/thumb.php b/thumb.php index 913adc18ec..35ab9f27e6 100644 --- a/thumb.php +++ b/thumb.php @@ -107,6 +107,15 @@ function wfStreamThumb( array $params ) { $fileName = isset( $params['f'] ) ? $params['f'] : ''; + // Backwards compatibility parameters + if ( isset( $params['w'] ) ) { + $params['width'] = $params['w']; + unset( $params['w'] ); + } + if ( isset( $params['p'] ) ) { + $params['page'] = $params['p']; + } + // Is this a thumb of an archived file? $isOld = ( isset( $params['archived'] ) && $params['archived'] ); unset( $params['archived'] ); // handlers don't care @@ -235,14 +244,6 @@ function wfStreamThumb( array $params ) { } } - // Backwards compatibility parameters - if ( isset( $params['w'] ) ) { - $params['width'] = $params['w']; - unset( $params['w'] ); - } - if ( isset( $params['p'] ) ) { - $params['page'] = $params['p']; - } unset( $params['r'] ); // ignore 'r' because we unconditionally pass File::RENDER unset( $params['f'] ); // We're done with 'f' parameter. -- 2.20.1