Last modified: 2012-09-16 03:52:21 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 T42151, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 40151 - Numbers passed to $ variables in MediaWiki:Categorytree-member-num must not be formatted but raw
Numbers passed to $ variables in MediaWiki:Categorytree-member-num must not b...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
CategoryTree (Other open bugs)
unspecified
All All
: Unprioritized major (vote)
: ---
Assigned To: Nobody - You can work on this!
: code-update-regression, i18n
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-10 23:18 UTC by Danny B.
Modified: 2012-09-16 03:52 UTC (History)
6 users (show)

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


Attachments

Description Danny B. 2012-09-10 23:18:08 UTC
Numbers passed to $ variables in MediaWiki:Categorytree-member-num must not be formatted but raw. If parser functions are used in such message, then they crash if the number is bigger than 999 and formatted with space as thousand separator.

Seems to me as a regression because I never noticed such issue before.
Comment 1 Siebrand Mazeland 2012-09-11 22:26:09 UTC
Having the numbers properly internationalised is the issue that was resolved. Using parserfunctions on MediaWiki:Categorytree-member-num was not an explicitely supported use case. I updated the component to ParserFunctions, as that one is apparently borking.

The quick fix is to probably provide an additional (unformatted and unused) parameter to 'categorytree-member-num' so the use case mentioned above can be salvaged.

I'll have a look if there are additional parameters that would need this in CategoryTree.

The issue summary should probably be updated to indicate where the root cause for the observed behaviour lies (in that ParserFunctions does not handle localised numbers).
Comment 2 Siebrand Mazeland 2012-09-11 22:31:57 UTC
Patch available in Gerrit change #23476.
Comment 3 Niklas Laxström 2012-09-12 00:32:10 UTC
{{FORMATNUM:$1|R}} also exists but that wouldn't keep BC.
Comment 4 db [inactive,noenotif] 2012-09-14 19:43:01 UTC
See bug 22798
and see also bug 29357, delete MediaWiki:Categorytree-member-num and have a look, if parser functions needed anymore for your case

This is a categorytree bug, the problem that Extension ParserFunctions does not handle localized numbers should be another bug (or maybe is)

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


Navigation
Links