Page MenuHomePhabricator

Cite: Improve compatibility between Parsoid's port and the PHP extension
Open, NormalPublic

Description

  • Several error checks that the PHP Cite extension does are not performed in Parsoid's cite impl.
    • nested ref tags are erroneous (T104662)
    • if group is set on <references>, one should be set on all refs defined in references-body
    • only one group param, and one name/follow param.
    • For ref in references body,
      • name must be present
      • content must be present
      • group should be used in main body
      • name should be used in main body
    • <ref /> is meaningless
    • nested references tags are erroneous
  • Parsoid does not support the follow arg on ref tags

Version: unspecified
Severity: normal

Details

Reference
bz49538

Event Timeline

bzimport raised the priority of this task from to Normal.Nov 22 2014, 1:47 AM
bzimport added a project: Parsoid.
bzimport set Reference to bz49538.
ssastry created this task.Jun 13 2013, 5:41 PM

[Parsoid component reorg by merging JS/General and General. See bug 50685 for more information. Filter bugmail on this comment. parsoidreorg20130704]

Per http://www.mediawiki.org/wiki/Extension:Cite/Cite.php#Merging_two_texts_into_a_single_reference_.28wikisource_issue.29

"<ref name"foo">blablabla</ref> yamamamam <ref follow="foo"><br/>chawachawa</ref>"

should create a single ref with both texts in it

We should also probably try to run upstream's mediawiki/extensions/Cite/citeParserTests.txt.

Note also that bug 2700 seems to have the PST run on the body of the <ref> and <references> tags.

Adding here for visibility: The current patch at https://gerrit.wikimedia.org/r/170936 implements a way to style references and cites using CSS, while leaving a default HTML rendering for older browsers. The only problem right now is how to disable the CSS that makes the compatibility content invisible in browsers that don't support CSS counters. Any help will be appreciated.

ssastry moved this task from In Progress to Backlog on the Parsoid board.Mar 3 2015, 9:09 PM
He7d3r added a subscriber: He7d3r.Jul 20 2015, 12:57 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 20 2015, 12:57 PM
Aklapper removed marcoil as the assignee of this task.Sep 22 2015, 3:38 PM
ssastry updated the task description. (Show Details)May 23 2016, 3:08 PM
ssastry moved this task from Backlog to Read Views on the Parsoid board.Jan 11 2018, 9:47 PM
Reedy edited projects, added Parsoid-Read-Views; removed Parsoid.Sep 17 2018, 7:25 PM