Last modified: 2007-03-17 19:22:24 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T10582, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 8582 - Imagemap: Space after imagesize prevent thumbnailing
Imagemap: Space after imagesize prevent thumbnailing
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
1.10.x
All All
: Normal normal with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
http://de.wikipedia.org/wiki/Benutzer...
: patch, patch-reviewed
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-01-11 17:39 UTC by Raimond Spekking
Modified: 2007-03-17 19:22 UTC (History)
1 user (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Raimond Spekking 2007-01-11 17:39:59 UTC
A space after the imagesize prevent thumbnailing of the image, see
[[de:Benutzer:Raymond/imagemap_bug]]
Comment 1 Greg Sabino Mullane 2007-01-11 18:21:59 UTC
I think we should be able do trim the options list in makeImage, but I'm not
100% positive it won't affect other things, so I'll not commit, but just post
the patch here for another developer to look over. It should fix your problem,
however:

Index: Parser.php
===================================================================
--- Parser.php  (revision 19123)
+++ Parser.php  (working copy)
@@ -4373,7 +4373,7 @@
                #  * center             center the image
                #  * framed             Keep original image size, no magnify-button.

-               $part = explode( '|', $options);
+               $part = array_map( 'trim', explode( '|', $options) );

                $mwThumb  =& MagicWord::get( 'img_thumbnail' );
                $mwManualThumb =& MagicWord::get( 'img_manualthumb' );
Comment 2 Greg Sabino Mullane 2007-03-17 19:20:18 UTC
Seems okay after sitting on it awhile, applied in r20540

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links