Last modified: 2006-09-25 03:04:10 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]],
or, alternatively, using definition list syntax:
; [[:Image:Bad image.jpg]] : [[Page to allow it on]], [[Another page]],
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.
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.
The ImagePage.php code should ideally link the image name to the image
description page. This is, however, a very low priority issue.
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.
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).
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
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.
I wasn't aware of this patch so I just wrote my own feature that does the same