[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-06-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

ssas...@wikimedia.org changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-06-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

--- Comment #9 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 140015 merged by jenkins-bot:
(Bug 52618) Suppress nowikis for table WT strings outside tables

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-06-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

--- Comment #8 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 140015 had a related patch set uploaded by Subramanya Sastry:
(Bug 52618) Suppress nowikis for table WT strings outside tables

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-06-16 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

Gerrit Notification Bot gerritad...@wikimedia.org changed:

   What|Removed |Added

 Status|RESOLVED|PATCH_TO_REVIEW
 Resolution|FIXED   |---

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-04-17 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

ssas...@wikimedia.org changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

--- Comment #4 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 114897 merged by jenkins-bot:
Bug 52618: Rescue stripped tds outside of table context

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

--- Comment #5 from Gabriel Wicke gwi...@wikimedia.org ---
TODO from the commit summary:

- Avoid nowikification on round-trip (even with selser)
- Avoid paragraph splitting by moving the paragraph wrapper to the DOM (major
project)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

Gabriel Wicke gwi...@wikimedia.org changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |NEW

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

--- Comment #6 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 115436 had a related patch set uploaded by GWicke:
Bug 52618: Avoid nowikification of td/tr/th syntax outside of tables

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

Gerrit Notification Bot gerritad...@wikimedia.org changed:

   What|Removed |Added

 Status|NEW |PATCH_TO_REVIEW

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

--- Comment #7 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 115436 merged by jenkins-bot:
Bug 52618: Avoid nowikification of td/tr/th syntax outside of tables

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

Gabriel Wicke gwi...@wikimedia.org changed:

   What|Removed |Added

   Priority|Normal  |High

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

Gerrit Notification Bot gerritad...@wikimedia.org changed:

   What|Removed |Added

 Status|NEW |PATCH_TO_REVIEW

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2014-02-21 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

--- Comment #3 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 114897 had a related patch set uploaded by GWicke:
WIP Bug 52618: Rescue stripped tds outside of table context

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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2013-12-03 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

Gabriel Wicke gwi...@wikimedia.org changed:

   What|Removed |Added

   Priority|Unprioritized   |Normal

--- Comment #2 from Gabriel Wicke gwi...@wikimedia.org ---
The issue here is that we are tokenizing this to a td token, which is then
dropped by the treebuilder when that does not end up inside a table. We should
be able to detect this on the DOM based on shadow info. When detected, we can
re-insert the original pipe so that it is not lost.

That might not yet avoid the nowiki insertion, but would at least preserve
the content. It is also possible that selser avoids the nowiki.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 52618] Bad wikitext lines starting with | (not in a table context) getting the pipes removed and replaced by nowiki /nowiki on RT

2013-10-28 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=52618

James Forrester jforres...@wikimedia.org changed:

   What|Removed |Added

 CC||ssas...@wikimedia.org
  Component|General |General
   Assignee|jforres...@wikimedia.org|gwi...@wikimedia.org
Product|VisualEditor|Parsoid
Summary|VisualEditor: nowiki-ing|Bad wikitext lines starting
   |and removing content when   |with |  (not in a table
   |it encounters incorrect |context) getting the pipes
   |markup  |removed and replaced by
   ||nowiki /nowiki on RT

--- Comment #1 from James Forrester jforres...@wikimedia.org ---
This looks to be a Parsoid bug, though frankly the behaviour of the system
when given such broken input is somewhat undefined.

Minimum test case:

Foo
| Bar

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l