Variable name typo, but again I'm guessing this function isn't used if no-one has...

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

index d4e561e..c325d07 100644 (file)
--- a/thumb.php
+++ b/thumb.php
@@ -17,26 +17,26 @@ require_once( 'Image.php' );
  require_once( 'StreamFile.php' );
  
  // Get input parameters
+$fileName = isset( $_REQUEST['f'] ) ? $_REQUEST['f'] : '';
+$width = isset( $_REQUEST['w'] ) ? intval( $_REQUEST['w'] ) : 0;
+$page = isset( $_REQUEST['p'] ) ? intval( $_REQUEST['p'] ) : null;
  
  if ( get_magic_quotes_gpc() ) {
-       $fileName = stripslashes( $_REQUEST['f'] );
-       $width = stripslashes( $_REQUEST['w'] );
-} else {
-       $fileName = $_REQUEST['f'];
-       $width = $_REQUEST['w'];
+       $fileName = stripslashes( $fileName );
  }
  
  $pre_render= isset($_REQUEST['r']) && $_REQUEST['r']!="0";
  
  // Some basic input validation
-
-$width = intval( $width );
  $fileName = strtr( $fileName, '\\/', '__' );
  
  // Work out paths, carefully avoiding constructing an Image object because that won't work yet
  
  $imagePath = wfImageDir( $fileName ) . '/' . $fileName;
  $thumbName = "{$width}px-$fileName";
+if ( ! is_null( $page ) ) {
+       $thumbName = 'page' . $page . '-' . $thumbName;
+}
  if ( $pre_render ) {
         $thumbName .= '.png';
  }
@@ -54,9 +54,17 @@ require_once( 'Setup.php' );
  wfProfileIn( 'thumb.php-render' );
  
  $img = Image::newFromName( $fileName );
-if ( $img ) {
-       $thumb = $img->renderThumb( $width, false );
-} else {
+try {
+       if ( $img ) {
+               if ( ! is_null( $page ) ) {
+                       $img->selectPage( $page );
+               }
+               $thumb = $img->renderThumb( $width, false );
+       } else {
+               $thumb = false;
+       }
+} catch( Exception $ex ) {
+       // Tried to select a page on a non-paged file?
         $thumb = false;
  }
  
@@ -65,12 +73,15 @@ if ( $thumb && $thumb->path ) {
  } else {
         $badtitle = wfMsg( 'badtitle' );
         $badtitletext = wfMsg( 'badtitletext' );
+       header( 'Cache-Control: no-cache' );
+       header( 'Content-Type: text/html' );
         echo "<html><head>
         <title>$badtitle</title>
         <body>
  <h1>$badtitle</h1>
  <p>$badtitletext</p>
-</body></html>";
+</body></html>
+";
  }
  
  wfProfileOut( 'thumb.php-render' );