Move <warnings> and <query-continue> to result's top and optimize

[lhc/web/wiklou.git] / thumb.php
diff --git a/thumb.php b/thumb.php

index f1d5341..ede30e6 100644 (file)
--- a/thumb.php
+++ b/thumb.php
@@ -170,11 +170,11 @@ function wfStreamThumb( array $params ) {
  
         // Check the source file storage path
         if ( !$img->exists() ) {
-               wfThumbError( 404, 'The source file for the specified thumbnail does not exist.' );
+               wfThumbError( 404, "The source file '$fileName' does not exist." );
                 wfProfileOut( __METHOD__ );
                 return;
         } elseif ( $img->getPath() === false ) {
-               wfThumbError( 500, 'The source file is not locally accessible.' );
+               wfThumbError( 500, "The source file '$fileName' is not locally accessible." );
                 wfProfileOut( __METHOD__ );
                 return;
         }
@@ -215,16 +215,16 @@ function wfStreamThumb( array $params ) {
         // Check that the zone relative path matches up so squid caches won't pick
         // up thumbs that would not be purged on source file deletion (bug 34231).
         if ( isset( $params['rel404'] ) ) { // thumbnail was handled via 404
-               if ( urldecode( $params['rel404'] ) === $img->getThumbRel( $thumbName ) ) {
+               if ( rawurldecode( $params['rel404'] ) === $img->getThumbRel( $thumbName ) ) {
                         // Request for the canonical thumbnail name
-               } elseif ( urldecode( $params['rel404'] ) === $img->getThumbRel( $thumbName2 ) ) {
+               } elseif ( rawurldecode( $params['rel404'] ) === $img->getThumbRel( $thumbName2 ) ) {
                         // Request for the "long" thumbnail name; redirect to canonical name
                         $response = RequestContext::getMain()->getRequest()->response();
                         $response->header( "HTTP/1.1 301 " . HttpStatus::getMessage( 301 ) );
                         $response->header( 'Location: ' .
                                 wfExpandUrl( $img->getThumbUrl( $thumbName ), PROTO_CURRENT ) );
                         $response->header( 'Expires: ' .
-                               gmdate( 'D, d M Y H:i:s', time() + 7*86400 ) . ' GMT' );
+                               gmdate( 'D, d M Y H:i:s', time() + 7 * 86400 ) . ' GMT' );
                         if ( $wgVaryOnXFP ) {
                                 $varyHeader[] = 'X-Forwarded-Proto';
                         }
@@ -234,7 +234,9 @@ function wfStreamThumb( array $params ) {
                         wfProfileOut( __METHOD__ );
                         return;
                 } else {
-                       wfThumbError( 404, 'The given path of the specified thumbnail is incorrect.' );
+                       wfThumbError( 404, "The given path of the specified thumbnail is incorrect;
+                               expected '" . $img->getThumbRel( $thumbName ) . "' but got '" .
+                               rawurldecode( $params['rel404'] ) . "'." );
                         wfProfileOut( __METHOD__ );
                         return;
                 }
@@ -362,7 +364,7 @@ function wfThumbError( $status, $msg ) {
                 $hostname = htmlspecialchars( wfHostname() );
                 $debug = "<!-- $url -->\n<!-- $hostname -->\n";
         } else {
-               $debug = "";
+               $debug = '';
         }
         echo <<<EOT
  <html><head><title>Error generating thumbnail</title></head>