Page MenuHomePhabricator

collision between PageForms, VEForAll and parsoid - pipes in tables.
Open, Needs TriagePublic

Description

Page Forms can't save tables defined by pipes (|), so instead using pipe comment ( {{!}} ).
When field is VEditor, the content parsed by parsoid which returns tables with pipes.
So you can't resave table.

Steps to Reproduce:

  • Create form uses VEForAll
  • Add Table to field using VEForAll
  • Save
  • Try to save again

Event Timeline

Anysite created this task.May 17 2020, 5:32 AM
Restricted Application added subscribers: Liuxinyu970226, Aklapper. · View Herald TranscriptMay 17 2020, 5:32 AM
Anysite updated the task description. (Show Details)May 17 2020, 5:33 AM

Change 555469 had a related patch set uploaded (by Anysite; owner: Anysite):
[mediawiki/extensions/VEForAll@master] Fix collision collision between PageForms, VEForAll and parsoid - pipes in tables.

https://gerrit.wikimedia.org/r/555469

Change 596852 had a related patch set uploaded (by Anysite; owner: Anysite):
[mediawiki/extensions/PageForms@master] Fix collision between PageForms, VEForAll and parsoid - pipes in tables

https://gerrit.wikimedia.org/r/596852

Clarification: the suggested patch have two parts: one in VEForAll and one in PageForms, this parts working together.