% echo "<table><tr><td>x</td><td align="center" data-parsoid='{"stx":"row"}'></td><td data-parsoid='{"stx":"row"}'>y</td></tr></table>" | parse.js --html2wt
{|
|x|| align="center" |||y
|}There should have been a <nowiki/> between the tds so that output looked like |x|| align="center" |<nowiki/>||y
The incorrect html2wt Parsoid output now parses as:
<table><tr> <td>x</td> <td>align="center"</td> <td>y</td></tr></table>
Oh hmm .. bad test case.
[subbu@earth:~/work/wmf/mediawiki] echo "<table><tr><td>x</td><td data-parsoid='{"stx":"row"}'></td><td data-parsoid='{"stx":"row"}'>y</td></tr></table>" | parse.js --html2wt | parse.js --normalize=parsoid
<table>
<tbody>
<tr>
<td>x</td>
<td></td>
<td>y</td>
</tr>
</tbody>
</table>
[subbu@earth:~/work/wmf/mediawiki] echo "<table><tr><td>x</td><td data-parsoid='{"stx":"row"}'></td><td data-parsoid='{"stx":"row"}'>y</td></tr></table>" | parse.js --html2wt | php maintenance/parse.php
parse.php: warning: reading wikitext from STDIN. Press CTRL+D to parse.
<div class="mw-parser-output"><table>
<tr>
<td>x</td>
<td></td>
<td>y
</td></tr></table>
</div>I got the parse ambiguity flipped around. Requires td attributes and empty content. Fixed test case.
Change 433087 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] WIP: Serialize empty <td>s as <nowiki/> in certain scenarios
Change 433087 abandoned by Subramanya Sastry:
Serialize empty <td>s as <nowiki/> in certain scenarios
Reason:
I was not thinking straight. Instead of all this nowiki crap, I should simply emit a space there.
Change 433103 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Empty td and th cells are serialized with a single whitespace char
I decided to eliminate the nowikis and instead a whitespace char in all empty cells which is a much better approach.
Change 433103 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Empty td and th cells are serialized with a single whitespace char