Last modified: 2011-03-13 18:06:16 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 T11770, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 9770 - Allow WikiTables in ParserFunctions without using transclusions
Allow WikiTables in ParserFunctions without using transclusions
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
unspecified
All All
: Lowest enhancement with 1 vote (vote)
: ---
Assigned To: Daniel Friesen
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-05-03 01:37 UTC by Daniel Friesen
Modified: 2011-03-13 18:06 UTC (History)
2 users (show)

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


Attachments
Patch for includes/Parser.php (3.62 KB, patch)
2007-05-03 01:43 UTC, Daniel Friesen
Details

Description Daniel Friesen 2007-05-03 01:37:37 UTC
Normally there are 2 ways to make a table which is affected by ParserFunctions
like #if:

Using HTML Tables:
<table>
<tr>
<td>Something</td>{{#if:{{{1|}}}|<nowiki>/
<td>Something else</td>| }}
</tr>
</table>

Or using the template {{!}} which would contain the | character:
{|
|-
| Something{{#if:{{{1|}}}|<nowiki>/
{{!}} Something else| }}
|}

But DynamicPageList2 introduced the idea of getting around the ParserFunctions'
absolute control over the | by using ¦ instead. We should be able to use this
syntax to allow WikiTables to work with ParserFunctions. Technically this is the
most readable and most efficient method of adding ParserFunctions to tables:
{|
|-
| Something{{#if:{{{1|}}}|<nowiki>/
¦ Something else| }}
|}
Comment 1 Daniel Friesen 2007-05-03 01:43:21 UTC
Created attachment 3567 [details]
Patch for includes/Parser.php

This patch was made for the current 1.10a HEAD trunk, MediaWiki was updated
before the patch was applied to the file.

The patch was made through only adding extra conditions and a few restricted
replacements so there should be no room for glitches such as ¦'s in the page
content being rendered as |'s. Also, only one explode was replaced with a
preg_split so there shouldn't be any sort of higher server load.
Comment 2 Rob Church 2007-05-19 19:34:43 UTC
This adds another horrible hack to workaround shortcomings of the parser, and furthermore complicates table markup which a lot of users object to.

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


Navigation
Links