Last modified: 2011-01-22 20:25: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 T25588, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 23588 - Transclusion of file or category pages may produce unexpected results if post-expand include size limit is exceeded
Transclusion of file or category pages may produce unexpected results if post...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
1.17.x
All All
: Normal minor (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 26869 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-18 23:32 UTC by Ilmari Karonen
Modified: 2011-01-22 20:25 UTC (History)
0 users

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


Attachments

Description Ilmari Karonen 2010-05-18 23:32:09 UTC
When the post-expand include size limit is exceeded while transcluding a page, Parser::braceSubstitution() will replace the transclusion with a link to the page in question.  However, no colon is prepended to the page name, leading to unexpected behavior if the page being transcluded is a file or a category page (or an interlanguage link, but I don't think there's any sensible reason to try transcluding those).

While such pages are rarely transcluded, it does occasionally happen; in particular, this behavior recently triggered a rather nasty bug in the ImageAnnotator script on Commons: see http://commons.wikimedia.org/w/index.php?title=Commons:Village_pump&oldid=39469993#Many_megabytes_of_unwanted_downloads

Incidentally, the code also doesn't prepend the template namespace to titles with no namespace given, which means that such links will incorrectly point to the main namespace instead.

I have a fix for this bug that I plan to commit shortly, but I felt I should file this bug first so that I can refer to it in the comments instead of having to explain the issue in detail there.
Comment 1 Ilmari Karonen 2010-05-19 00:04:25 UTC
Committed fix in r66634.
Comment 2 Umherirrender 2011-01-22 20:24:49 UTC
*** Bug 26869 has been marked as a duplicate of this bug. ***

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


Navigation
Links