Last modified: 2013-07-01 19:58:39 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 T52129, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 50129 - VisualEditor: Transclusions not properly round-tripped
VisualEditor: Transclusions not properly round-tripped
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Data Model (Other open bugs)
unspecified
All All
: Highest normal
: VE-deploy-2013-07-04
Assigned To: Ed Sanders
https://en.wikipedia.org/wiki/Chlorop...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-24 20:43 UTC by Gabriel Wicke
Modified: 2013-07-01 19:58 UTC (History)
4 users (show)

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


Attachments

Description Gabriel Wicke 2013-06-24 20:43:42 UTC
When doing a simple test edit on https://en.wikipedia.org/wiki/Chloroplast?veaction=edit and then previewing the diff, several transclusions of the form {{expand section|..}} show up with dirty diffs. This is very likely a DOM corruption in the VE that disables selective serialization.
Comment 2 Gabriel Wicke 2013-06-25 01:19:27 UTC
The nowiki escaping in Schuylar_Oordt is Parsoid bug 50144. It would normally be hidden with selective serialization.
Comment 3 James Forrester 2013-06-25 02:37:51 UTC

*** This bug has been marked as a duplicate of bug 50070 ***
Comment 4 Gabriel Wicke 2013-06-27 00:11:54 UTC
At least one {{expand section|..}} transclusion still dirty-diffs.

This will soon not show up as a diff any more because we improved our native serialization, but it seems that the VE still dirties the transclusion, which needs to be fixed.
Comment 5 Roan Kattouw 2013-06-27 00:15:54 UTC
(In reply to comment #4)
> At least one {{expand section|..}} transclusion still dirty-diffs.
> 
> This will soon not show up as a diff any more because we improved our native
> serialization, but it seems that the VE still dirties the transclusion, which
> needs to be fixed.
It doesn't seem to be dirtied by VE directly. On [[Chloroplast]], I get a clean diff if I don't make any changes, but a dirty diff if I make any change at all. This leads me to suspect a selser / DOMDiffer bug.

Will try to produce a minimal test case and investigate from there.
Comment 6 Gabriel Wicke 2013-06-27 06:36:04 UTC
Another example where an unmodified template was dirtied:

https://en.wikipedia.org/w/index.php?title=User%3AEdgepedia%2FVE%2FGNoSR&diff=561782383&oldid=561781680

Since our DOM diff is so simple I have a lot of faith in it. Did you diff the template DOM fragment after making an unrelated change?
Comment 8 Roan Kattouw 2013-07-01 17:58:13 UTC
(In reply to comment #7)
> Another case:
> https://en.wikipedia.org/w/index.
> php?title=Dub_Jones_%28American_football%29&curid=5240085&diff=561834911&oldi
> d=561833425
WFM when I tried to reproduce at http://en.wikipedia.org/wiki/User:Catrope/Dub_Jones?veaction=edit . VE's sanity check says the DOM is clean. I believe these failures are due to cached content or some sort of bug in Parsoid/selser.
Comment 9 Gabriel Wicke 2013-07-01 18:42:41 UTC
I don't see the Dub Jones issue any more, so that appears to be fixed.

There was also an independent Parsoid issue that resulted in incomplete DSR on transclusions, which is now also fixed. 

If you don't see this issue any more then this bug can be closed as fixed.

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


Navigation
Links