Last modified: 2011-05-17 13:17:26 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 T31012, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 29012 - sep parameter in list result format ignored
sep parameter in list result format ignored
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Semantic MediaWiki (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Markus Krötzsch
:
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2011-05-16 15:28 UTC by planetenxin
Modified: 2011-05-17 13:17 UTC (History)
2 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description planetenxin 2011-05-16 15:28:05 UTC 
The sep parameter in result format 'list' is ignored. List always uses the default separator ','.

It seems to be a bug in SMW_QP_List.php around line 228:
 $result .= ', ';

The above line should be changed as follows:
 $result .= $listsep.' ';

Testcase:
{{#arraymap:{{{Param|}}}|;|x|[[Param::x| ]]|;}}
Comment 1 Jeroen De Dauw 2011-05-17 13:17:26 UTC 
Ah, that's where the bug was hiding... :)

Applied your fix in r88302, thanks for debugging and reporting!
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links