Page MenuHomePhabricator

CX2: Dirty markup in coordinate template
Closed, DeclinedPublic

Description

I published test article in CX2 production. It published without any issue but shows dirty markup.

See: https://gu.wikipedia.org/w/index.php?title=%E0%AA%A6%E0%AA%97%E0%AA%A1%E0%AB%80_%E0%AA%9A%E0%AA%BE%E0%AA%B2&type=revision&diff=577066&oldid=577065&diffmode=source

While adapting {{Coord|18.977129|N|72.829131|E|display=title}} template, CX2 shows it like:

<span id="coordinates">[[Geographic coordinate system|Coordinates]]: <span class="plainlinks nourlexpansion">[//tools.wmflabs.org/geohack/geohack.php?pagename=Dagdi_Chawl&params=18.977129_N_72.829131_E_ <span class="geo-nondefault"><span class="geo-dms" title="Maps, aerial photos, and other data for this location"><span class="latitude">18°58′38″N</span> <span class="longitude">72°49′45″E</span></span></span><span class="geo-multi-punct">&#xFEFF; / &#xFEFF;</span><span class="geo-default"><span class="geo-dec" title="Maps, aerial photos, and other data for this location">18.977129°N 72.829131°E</span><span style="display:none">&#xFEFF; / <span class="geo">18.977129; 72.829131</span></span></span>]</span></span>

This is same issue in CX1.

Event Timeline

Pginer-WMF triaged this task as Medium priority.May 25 2018, 1:40 PM
Pginer-WMF moved this task from Needs Triage to CX2 on the ContentTranslation board.

It seems that this is related to issues with template adaptation. In those cases we should never generate rich HTML text. This may be worth considering as we work on the approaches for templates that cannot be adapted (T192271)

Vvjjkkii renamed this task from CX2: Dirty markup in coordinate template to gacaaaaaaa.Jul 1 2018, 1:09 AM
Vvjjkkii raised the priority of this task from Medium to High.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed a subscriber: Aklapper.
CommunityTechBot renamed this task from gacaaaaaaa to CX2: Dirty markup in coordinate template.Jul 2 2018, 4:24 AM
CommunityTechBot lowered the priority of this task from High to Medium.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot added a subscriber: Aklapper.

Since we are filtering out unsupported templates including this, we can close this bug as of now. If it appears again, we will file new bug.