mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools
synced 2024-11-28 10:11:45 +00:00
ffd680ee7f
The issue occurred when replying to a comment consisting of multiple list items, starting with a <dt> (instead of the expected <dd>), so that the comment is considered to be unindented. Modifier tried to add the reply directly inside the list (<dl>) rather than inside the last list item (<dt>), which caused it to be confused about indentation levels and try to un-indent more times than there were indentations. The simplest solution, given the existing code, is to add the reply outside the list instead, in a new list. This results in a "list gap" (<dl><dt>...</dt><dd>...</dd></dl><dl><dd>...</dd></dl>), but I think it's acceptable for this rare case. There are separate tests cases for old Parser and for Parsoid HTML, because they parse the original wikitext differently (with the old Parser producing HTML with a list gap too). Bug: T279445 Change-Id: Ie0ee960e7090cf051ee547b480c980e9530eda51
157 lines
5.4 KiB
JSON
157 lines
5.4 KiB
JSON
[
|
|
{
|
|
"name": "plwiki oldparser",
|
|
"dom": "cases/pl-big-oldparser/pl-big-oldparser.html",
|
|
"expected": "../cases/pl-big-oldparser/pl-big-oldparser.json",
|
|
"config": "../data/plwiki-config.json",
|
|
"data": "../data/plwiki-data.json"
|
|
},
|
|
{
|
|
"name": "plwiki parsoid",
|
|
"dom": "cases/pl-big-parsoid/pl-big-parsoid.html",
|
|
"expected": "../cases/pl-big-parsoid/pl-big-parsoid.json",
|
|
"config": "../data/plwiki-config.json",
|
|
"data": "../data/plwiki-data.json"
|
|
},
|
|
{
|
|
"name": "enwiki oldparser",
|
|
"dom": "cases/en-big-oldparser/en-big-oldparser.html",
|
|
"expected": "../cases/en-big-oldparser/en-big-oldparser.json",
|
|
"config": "../data/enwiki-config.json",
|
|
"data": "../data/enwiki-data.json"
|
|
},
|
|
{
|
|
"name": "enwiki parsoid",
|
|
"dom": "cases/en-big-parsoid/en-big-parsoid.html",
|
|
"expected": "../cases/en-big-parsoid/en-big-parsoid.json",
|
|
"config": "../data/enwiki-config.json",
|
|
"data": "../data/enwiki-data.json"
|
|
},
|
|
{
|
|
"name": "ckbwiki oldparser",
|
|
"dom": "cases/ckb-big-oldparser/ckb-big-oldparser.html",
|
|
"expected": "../cases/ckb-big-oldparser/ckb-big-oldparser.json",
|
|
"config": "../data/ckbwiki-config.json",
|
|
"data": "../data/ckbwiki-data.json"
|
|
},
|
|
{
|
|
"name": "ckbwiki parsoid",
|
|
"dom": "cases/ckb-big-parsoid/ckb-big-parsoid.html",
|
|
"expected": "../cases/ckb-big-parsoid/ckb-big-parsoid.json",
|
|
"config": "../data/ckbwiki-config.json",
|
|
"data": "../data/ckbwiki-data.json"
|
|
},
|
|
{
|
|
"name": "arwiki no-paragraph oldparser",
|
|
"dom": "cases/ar-no-paragraph-oldparser/ar-no-paragraph-oldparser.html",
|
|
"expected": "../cases/ar-no-paragraph-oldparser/ar-no-paragraph-oldparser.json",
|
|
"config": "../data/arwiki-config.json",
|
|
"data": "../data/arwiki-data.json"
|
|
},
|
|
{
|
|
"name": "arwiki no-paragraph parsoid",
|
|
"dom": "cases/ar-no-paragraph-parsoid/ar-no-paragraph-parsoid.html",
|
|
"expected": "../cases/ar-no-paragraph-parsoid/ar-no-paragraph-parsoid.json",
|
|
"config": "../data/arwiki-config.json",
|
|
"data": "../data/arwiki-data.json"
|
|
},
|
|
{
|
|
"name": "arwiki nbsp-timezone oldparser",
|
|
"dom": "cases/ar-nbsp-timezone-oldparser/ar-nbsp-timezone-oldparser.html",
|
|
"expected": "../cases/ar-nbsp-timezone-oldparser/ar-nbsp-timezone-oldparser.json",
|
|
"config": "../data/arwiki-config.json",
|
|
"data": "../data/arwiki-data.json"
|
|
},
|
|
{
|
|
"name": "arwiki nbsp-timezone parsoid",
|
|
"dom": "cases/ar-nbsp-timezone-parsoid/ar-nbsp-timezone-parsoid.html",
|
|
"expected": "../cases/ar-nbsp-timezone-parsoid/ar-nbsp-timezone-parsoid.json",
|
|
"config": "../data/arwiki-config.json",
|
|
"data": "../data/arwiki-data.json"
|
|
},
|
|
{
|
|
"name": "frwiki fr-unsigned oldparser",
|
|
"dom": "cases/fr-unsigned-oldparser/fr-unsigned-oldparser.html",
|
|
"expected": "../cases/fr-unsigned-oldparser/fr-unsigned-oldparser.json",
|
|
"config": "../data/frwiki-config.json",
|
|
"data": "../data/frwiki-data.json"
|
|
},
|
|
{
|
|
"name": "frwiki fr-unsigned parsoid",
|
|
"dom": "cases/fr-unsigned-parsoid/fr-unsigned-parsoid.html",
|
|
"expected": "../cases/fr-unsigned-parsoid/fr-unsigned-parsoid.json",
|
|
"config": "../data/frwiki-config.json",
|
|
"data": "../data/frwiki-data.json"
|
|
},
|
|
{
|
|
"name": "itwiki it-unsigned oldparser",
|
|
"dom": "cases/it-unsigned-oldparser/it-unsigned-oldparser.html",
|
|
"expected": "../cases/it-unsigned-oldparser/it-unsigned-oldparser.json",
|
|
"config": "../data/itwiki-config.json",
|
|
"data": "../data/itwiki-data.json"
|
|
},
|
|
{
|
|
"name": "itwiki it-unsigned parsoid",
|
|
"dom": "cases/it-unsigned-parsoid/it-unsigned-parsoid.html",
|
|
"expected": "../cases/it-unsigned-parsoid/it-unsigned-parsoid.json",
|
|
"config": "../data/itwiki-config.json",
|
|
"data": "../data/itwiki-data.json"
|
|
},
|
|
{
|
|
"name": "srwiki sr-ec variant",
|
|
"dom": "cases/sr-ec/sr-ec.html",
|
|
"expected": "../cases/sr-ec/sr-ec.json",
|
|
"config": "../data/srwiki-config.json",
|
|
"data": "../data/srwiki-data.json"
|
|
},
|
|
{
|
|
"name": "srwiki sr-el variant",
|
|
"dom": "cases/sr-el/sr-el.html",
|
|
"expected": "../cases/sr-el/sr-el.json",
|
|
"config": "../data/srwiki-config.json",
|
|
"data": "../data/srwiki-data.json"
|
|
},
|
|
{
|
|
"name": "Accidental dt tags (old parser)",
|
|
"dom": "cases/dt-tags-oldparser/dt-tags-oldparser.html",
|
|
"expected": "../cases/dt-tags-oldparser/dt-tags-oldparser.json",
|
|
"config": "../data/enwiki-config.json",
|
|
"data": "../data/enwiki-data.json"
|
|
},
|
|
{
|
|
"name": "Accidental dt tags (Parsoid)",
|
|
"dom": "cases/dt-tags-parsoid/dt-tags-parsoid.html",
|
|
"expected": "../cases/dt-tags-parsoid/dt-tags-parsoid.json",
|
|
"config": "../data/enwiki-config.json",
|
|
"data": "../data/enwiki-data.json"
|
|
},
|
|
{
|
|
"name": "No heading",
|
|
"dom": "cases/no-heading/no-heading.html",
|
|
"expected": "../cases/no-heading/no-heading.json",
|
|
"config": "../data/enwiki-config.json",
|
|
"data": "../data/enwiki-data.json"
|
|
},
|
|
{
|
|
"name": "Manually added signature with LRM",
|
|
"dom": "cases/lrm-signature/lrm-signature.html",
|
|
"expected": "../cases/lrm-signature/lrm-signature.json",
|
|
"config": "../data/nlwiki-config.json",
|
|
"data": "../data/nlwiki-data.json"
|
|
},
|
|
{
|
|
"name": "Link using fallback 8-bit encoding (invalid UTF-8)",
|
|
"dom": "cases/fallback-encoding-link/fallback-encoding-link.html",
|
|
"expected": "../cases/fallback-encoding-link/fallback-encoding-link.json",
|
|
"config": "../data/huwiki-config.json",
|
|
"data": "../data/huwiki-data.json"
|
|
},
|
|
{
|
|
"name": "Signatures in funny places",
|
|
"dom": "cases/signatures-funny/signatures-funny.html",
|
|
"expected": "../cases/signatures-funny/signatures-funny.json",
|
|
"config": "../data/enwiki-config.json",
|
|
"data": "../data/enwiki-data.json"
|
|
}
|
|
]
|