Introduce deleteRow() method for TemplateProcessor

[simivar]

Introduce deleteRow() method for TemplateProcessor method that works fine with cell's spanning multiple rows.

[FBnil]

Checked the code and it is superseeded by #1147
Which also contains deleteRow() and a test.

@simivar We are rivals in code! :)

Your test document does not contain a cell containing multiple rows (double as high as the previous one containing the <w:vMerge w:val="restart"/>).

+----------+----------+
| ${cell1} | ${cell2} |
+          +----------+
|          | ${cell3} |
+----------+----------+

where you select to delete ${cell1} and ${cell2} and ${cell3} should be gone too, but are not.
I suspect that the CloneRow() never worked as intended. I am fixing it in my branch.

[FBnil]

Ok, in LibreOffice (5.4.1 and 5.4.2)
The Rowspan is <w:vMerge w:val="continue"/>, and does not match cloneRow()'s ('#<w:vMerge w:val="continue" />#', $tmpXmlRow)
So the easy fix is adding a question for the space:
('#<w:vMerge w:val="continue" ?/>#', $tmpXmlRow) // <w:vMerge w:val="continue"/>

Microsoft has spaces:
<w:vMerge /> <w:vMerge w:val="restart" />

LibreOffice (docx) does not:
<w:vMerge/> <w:vMerge w:val="restart"/>

Sources:
http:https://officeopenxml.com/WPtableGrid.php
https://msdn.microsoft.com/en-us/library/office/ff951689(v=office.14).aspx

[FBnil]

I also found another thing that should be tested:

+---+---+---+
| A |   |   |
|   |   +---+
|   | B |   |
|   +---+---+
|   |   |   |
+---+---+---+

edit: this works as expected.

@simivar: Why not join me? I'm building a new type of TemplateProcessor that allows sub classes to do lots of neat stuff, like making a table the normal way with addTable()->addRow()->addCell() and inject that into the template with replaceTable() or appendRow().

[manukieli]

I the code working with the version 0.16.0
It's a very good idea.