Last modified: 2014-11-17 14:34:31 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 T75121, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 73121 - Parsoid internal error with mw:Extension/references
Parsoid internal error with mw:Extension/references
Status: NEW
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Parsoid Team
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-07 07:17 UTC by Niklas Laxström
Modified: 2014-11-17 14:34 UTC (History)
5 users (show)

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


Attachments

Description Niklas Laxström 2014-11-07 07:17:14 UTC
Steps to reproduce:
1) Go to http://parsoid-lb.eqiad.wikimedia.org/_html/
2) Use <ol typeof="mw:Extension/references"></ol>
3) Submit

Expect results: No error
Actual results: Error 503 Service Unavailable
Service Unavailable
Guru Meditation:
XID: 640881439

This breaks content translation publishing on some pages, see bug 73119
Comment 1 ssastry 2014-11-07 16:12:18 UTC
We actually require an about id and a data-mw for extension content. I just noticed there is a minor gap in our spec for the references extension (that section show the data-mw for the references block in <ol>). However, all transclusions and extensions do show data-mw.

I'll fix this.

But, we would need this for it to serialize properly.

<ol class="references" typeof="mw:Extension/references" about="#mwt2"  data-mw='{"name":"references","attrs":{}}'></ol>

We should perhaps not throw a 503 and instead ignore this and generate an error message perhaps. We'll discuss that internally.

We could also perhaps tweak our serializer to infer empty attributes from the typeof and generate a self-closing empty tag. To be investigated.

But, if you your content translation engine to publish the data-mw and about attributes, it should work.

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


Navigation
Links