Reverting r16861; incompatible change to message texts, breaks a lot of toggle displa...

[lhc/web/wiklou.git] / includes / ImageGallery.php

diff --git a/includes/ImageGallery.php b/includes/ImageGallery.php
index 001305f..7ff456b 100644 (file)
--- a/includes/ImageGallery.php
+++ b/includes/ImageGallery.php
@@ -55,7 +55,7 @@ class ImageGallery
         *
         * @param $skin Skin object
         */
-       function useSkin( &$skin ) {
+       function useSkin( $skin ) {
                $this->mSkin =& $skin;
        }
        
@@ -82,6 +82,7 @@ class ImageGallery
         */
        function add( $image, $html='' ) {
                $this->mImages[] = array( &$image, $html );
+               wfDebug( "ImageGallery::add " . $image->getName() . "\n" );
        }
 
        /**
@@ -133,9 +134,9 @@ class ImageGallery
         *
         */
        function toHTML() {
-               global $wgLang, $wgIgnoreImageErrors;
+               global $wgLang, $wgIgnoreImageErrors, $wgGenerateThumbnailOnParse;
 
-               $sk =& $this->getSkin();
+               $sk = $this->getSkin();
 
                $s = '<table class="gallery" cellspacing="0" cellpadding="0">';
                if( $this->mCaption )
@@ -149,18 +150,23 @@ class ImageGallery
                        $name = $img->getName();
                        $nt = $img->getTitle();
 
-                       # If we're dealing with a non-image, or a blacklisted image,
-                       # spit out the name and be done with it
-                       if( $nt->getNamespace() != NS_IMAGE
-                               || ( $this->mParsing && wfIsBadImage( $nt->getDBkey() ) ) ) {
-                               $s .=
-                                       (($i%4==0) ? "<tr>\n" : '') .
-                                       '<td><div class="gallerybox" style="height: 152px;">' .
-                                       htmlspecialchars( $nt->getText() ) . '</div></td>' .  
-                                       (($i%4==3) ? "</tr>\n" : '');
-                               $i++;
-
-                               continue;
+                       if( $nt->getNamespace() != NS_IMAGE ) {
+                               # We're dealing with a non-image, spit out the name and be done with it.
+                               $thumbhtml = '<div style="height: 152px;">' . htmlspecialchars( $nt->getText() ) . '</div>';
+                       }
+                       else if( $this->mParsing && wfIsBadImage( $nt->getDBkey() ) ) {
+                               # The image is blacklisted, just show it as a text link.
+                               $thumbhtml = '<div style="height: 152px;">'
+                                       . $sk->makeKnownLinkObj( $nt, htmlspecialchars( $nt->getText() ) ) . '</div>';
+                       } else if( !( $thumb = $img->getThumbnail( 120, 120, $wgGenerateThumbnailOnParse ) ) ) {
+                               # Error generating thumbnail.
+                               $thumbhtml = '<div style="height: 152px;">'
+                                       . htmlspecialchars( $img->getLastError() ) . '</div>';
+                       }
+                       else {
+                               $vpad = floor( ( 150 - $thumb->height ) /2 ) - 2;
+                               $thumbhtml = '<div class="thumb" style="padding: ' . $vpad . 'px 0;">'
+                                       . $sk->makeKnownLinkObj( $nt, $thumb->toHtml() ) . '</div>';
                        }
 
                        //TODO
@@ -182,28 +188,14 @@ class ImageGallery
                                $sk->makeKnownLinkObj( $nt, htmlspecialchars( $wgLang->truncate( $nt->getText(), 20, '...' ) ) ) . "<br />\n" :
                                '' ;
 
+                       # ATTENTION: The newline after <div class="gallerytext"> is needed to accommodate htmltidy which
+                       # in version 4.8.6 generated crackpot html in its absence, see:
+                       # http://bugzilla.wikimedia.org/show_bug.cgi?id=1765 -Ævar
+
                        $s .= ($i%4==0) ? '<tr>' : '';
-                       $thumb = $img->getThumbnail( 120, 120 );
-                       if ( $thumb ) {
-                               $vpad = floor( ( 150 - $thumb->height ) /2 ) - 2;
-                               $s .= '<td><div class="gallerybox">' . '<div class="thumb" style="padding: ' . $vpad . 'px 0;">';
-
-                               # ATTENTION: The newline after <div class="gallerytext"> is needed to accommodate htmltidy which
-                               # in version 4.8.6 generated crackpot html in its absence, see:
-                               # http://bugzilla.wikimedia.org/show_bug.cgi?id=1765 -Ævar
-                               $s .= $sk->makeKnownLinkObj( $nt, $thumb->toHtml() ) . '</div><div class="gallerytext">' . "\n" .
-                                       $textlink . $text . $nb .
-                                       '</div>';
-                               $s .= "</div></td>\n";
-                       } else {
-                               # Error during thumbnail generation
-                               $s .= '<td><div class="gallerybox" style="height: 152px;">' .
-                                       #htmlspecialchars( $nt->getText() ) . "<br />\n" .
-                                       htmlspecialchars( $img->getLastError() ) .
-                                       "</div><div class=\"gallerytext\">\n" .
-                                       $textlink . $text . $nb .
-                                       "</div></td>\n";
-                       }
+                       $s .= '<td><div class="gallerybox">' . $thumbhtml
+                               . '<div class="gallerytext">' . "\n" . $textlink . $text . $nb
+                               . "</div></div></td>\n";
                        $s .= ($i%4==3) ? '</tr>' : '';
                        $i++;
                }