Last modified: 2013-03-19 21:34:06 UTC
Example https://en.wikipedia.org/w/index.php?title=2012_in_film&oldid=521959220 this probably goes wrong because the cell that needs exploding is a TH instead of a TD.
For the records: Issue brought up in https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=522004773#Javascript_screwing_up_tables
Fixed by Ibe4cc7e9.
(In reply to comment #2) > Fixed by Ibe4cc7e9. No, it wasn't. That fixes a different bug. A minimal test case for this bug seems to be the following table {| class="wikitable sortable" |- ! foo !! bar !! baz !! n |- !rowspan=2| foo |rowspan=2| bar | baz || 1 |- | baz || 2 |} When the rowspans are exploded, both rows should have "foo" in the first cell, "bar" in the second, and "baz" in the third. But currently the second row will get "baz" in the second cell and "bar" in the third. Gerrit change #34104 should fix it, plus a number of similar problems when row or colspans are used in creative ways.
Brad: Could you add some potential reviewers in https://gerrit.wikimedia.org/r/#/c/34104/ ?
Done. I seldom know who exactly to add, so I usually forget.
https://gerrit.wikimedia.org/r/34104 (Gerrit Change Icb674f7eece053435ca9525d45709579df14cc74) | Code-Review: Reviewed:+2 Patch Set 8: Code-Review+2 OK, it can be merged, then. I would do it myself, except that the unit test file has a fairly large merge conflict. [by com.google.gerrit.server.events.AccountAttribute@54cf85f]
https://gerrit.wikimedia.org/r/34104 (Gerrit Change Icb674f7eece053435ca9525d45709579df14cc74) | Patch Set 8: -Verified Starting gate-and-submit jobs. http://integration.mediawiki.org/zuul/status [by com.google.gerrit.server.events.AccountAttribute@ac7e4af]
https://gerrit.wikimedia.org/r/34104 (Gerrit Change Icb674f7eece053435ca9525d45709579df14cc74) | Code-Review: Verified:-1 Patch Set 8: Verified-1 Build failed This change was unable to be automatically merged with the current state of the repository. Please rebase your change and upload a new patchset. [by com.google.gerrit.server.events.AccountAttribute@f5c901c]
This is merged in -wmf12 and -wmf12 is in the process of being deployed across all WMF sites (should be done by end of next week). Closing.