Last modified: 2006-09-25 03:04:10 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 T7985, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 5985 - Allow images in the bad image to be used on specific pages
Allow images in the bad image to be used on specific pages
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
File management (Other open bugs)
1.7.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-05-16 17:23 UTC by Ilmari Karonen
Modified: 2006-09-25 03:04 UTC (History)
0 users

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


Attachments
Patch to allow exceptions to to bad image list (2.61 KB, patch)
2006-05-16 17:26 UTC, Ilmari Karonen
Details
New patch against revision 14260 (4.69 KB, patch)
2006-05-16 23:19 UTC, Ilmari Karonen
Details
Updated patch: link to image page in galleries, don't show bad images in categories (7.06 KB, patch)
2006-05-17 00:20 UTC, Ilmari Karonen
Details
Patch to allow exceptions to to bad image list (simplified, against svn HEAD) (4.74 KB, patch)
2006-06-29 20:25 UTC, Ilmari Karonen
Details
Patch to allow exceptions to to bad image list (simplified, depends on bugs 6490 and 6491) (5.35 KB, patch)
2006-06-29 20:27 UTC, Ilmari Karonen
Details

Description Ilmari Karonen 2006-05-16 17:23:56 UTC
The MediaWiki:Bad_image_list feature would be much more useful if the images
listed in it could be allowed on specific pages.  I'd propose the following
syntax for specifying such exceptions:

 * [[:Image:Bad image.jpg]] : [[Page to allow it on]], [[Another page]],
[[Namespace:Third page]]

or, alternatively, using definition list syntax:

 ; [[:Image:Bad image.jpg]] : [[Page to allow it on]], [[Another page]],
[[Namespace:Third page]]
Comment 1 Ilmari Karonen 2006-05-16 17:26:57 UTC
Created attachment 1754 [details]
Patch to allow exceptions to to bad image list

This patch implements the syntax proposed above.  Both variants are supported. 
Exceptions may also be listed on a separate line (or lines), prefixed with a
colon, just like in normal MediaWiki definition lists.
Comment 2 Ilmari Karonen 2006-05-16 23:19:06 UTC
Created attachment 1760 [details]
New patch against revision 14260

I've made a new patch that works with the ImageGallery.php fixes committed by
robchurch.  I make no comment on whether mParsing is (or was) a descriptive
name for the property I've co-opted -- nonetheless, I've left it as is for now.
Comment 3 Ilmari Karonen 2006-05-16 23:23:07 UTC
The ImagePage.php code should ideally link the image name to the image
description page.  This is, however, a very low priority issue.
Comment 4 Ilmari Karonen 2006-05-17 00:20:35 UTC
Created attachment 1762 [details]
Updated patch: link to image page in galleries, don't show bad images in categories

This is an update of the previous patch with better gallery rendering,
including a link to the image description page.  This involved a significant
reorganization of ImageGallery::toHTML(), which, however, was IMO in need of
one anyway.  Also, this patch contains a one-line addition to CategoryPage.php
that makes the bad image list also affect galleries.
Comment 5 Ilmari Karonen 2006-06-29 20:25:57 UTC
Created attachment 2025 [details]
Patch to allow exceptions to to bad image list (simplified, against svn HEAD)

This is a simplified patch that does not include unrelated cleanup to
ImageGallery.php (bug 6490) and does not extend the scope of the bad image list
to galleries (bug 6491).
Comment 6 Ilmari Karonen 2006-06-29 20:27:50 UTC
Created attachment 2026 [details]
Patch to allow exceptions to to bad image list (simplified, depends on bugs 6490 and 6491)

Alternative patch that may be applied on top of the patches for bug 6490 and
bug 6491.
Comment 7 Ilmari Karonen 2006-06-29 20:30:55 UTC
Note that the two patches above differ from the previous ones in that the syntax
for the exceptions is stricter: lines must begin with "*" (not ";") and all
exceptions must be listed on the same line as the image.  However, the colon
between the image and the exceptions is no longer required.
Comment 8 Tim Starling 2006-09-25 03:04:10 UTC
I wasn't aware of this patch so I just wrote my own feature that does the same
thing.

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


Navigation
Links