Last modified: 2013-08-05 19:02:53 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 T54305, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52305 - Parsing issue on a few of pl.wp's infoboxes
Parsing issue on a few of pl.wp's infoboxes
Status: RESOLVED INVALID
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: ssastry
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-30 23:46 UTC by Bartosz Dziewoński
Modified: 2013-08-05 19:02 UTC (History)
2 users (show)

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


Attachments

Description Bartosz Dziewoński 2013-07-30 23:46:14 UTC
http://parsoid.wmflabs.org/pl/Darmsztadt – |}}|- is visible before the infobox. I didn't investigate why yet.

According to [1] this affects {{Związek chemiczny infobox}}, {{Pierwiastek infobox}} and {{Polimer infobox}} (infoboxes for chemical compound, chemical element and polymer).

[1] https://pl.wikipedia.org/w/index.php?title=Wikipedia:Kawiarenka/Kwestie_techniczne&curid=481333&diff=37172467&oldid=37171659
Comment 1 ssastry 2013-08-01 00:11:36 UTC
I just checked http://pl.wikipedia.org/w/index.php?title=Szablon:Pierwiastek_infobox&action=edit based on output for Darmsztadt.

There is a bug in the template at line 264 (copied below):

}}|}}{{infobox projekt|c

The "|}}" is excess text which shows up verbatim in parsoid output.  This is based on PHP preprocessor expanding this template.  I suppose the php parser cleans that up somehow when generating HTML.  In any case, "|}}" on that line could be deleted.

I can run the other templates through the PHP preprocessor as well and look at the output.  Will do tomorrow.
Comment 2 ssastry 2013-08-01 16:46:54 UTC
The other issue is that lines with categories that start with spaces is wrapped in indent-pre whereas it shouldn't be.  I created a new bug for it (bug 52405).

So, once you fix the template on plwp and I fix bug 52405 (and get it deployed), these issues will be taken care of.
Comment 3 Bartosz Dziewoński 2013-08-05 19:02:53 UTC
Thanks. I fixed the template. Marking INVALID.

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


Navigation
Links