Item5163: SpreadSheetPlugin causes table to misrender an empty row
Current State: Closed
Released In: 1.1.0
Target Release: minor
The last row is not rendered as empty row.
- 21 Dec 2007
I just tested on Cairo and on 4.1, same issue. That is, this is a bug that existed for a long time but was discovered just now.
Reprioritizing to normal per decision at TWiki:Codev/FreetownReleaseMeeting2008x01x07
- 07 Jan 2008
isn't a problem with EditTablePlugin
at all but has to do with the rendering of 'empty' cells? -- Just guessing.
- 08 Jan 2008
No, the TML spec is clear. An empty cell means merge cell with cell on the left.
Without the %CALC the empty cell is not rendered.
With the %CALC the empty cell seems to loose its right bar and becomes a bar.
it does not happen with a table with two columns. It does not happen if the cell contains any text or a space. It only happens when you have a table where the last row is a merged single cell. It does not happen if the last row is a table with data in a column and then an empty merged cell. I can only trigger the bug when it is a single column table where the last row is empty. Why would you have an empty row which is not displayed.
So no need to hurry fixing this. It is a very exotic bug.
- 08 Jan 2008
Peter Thoeny fixed this with the following patch:
--- twiki/trunk/SpreadSheetPlugin/lib/TWiki/Plugins/SpreadSheetPlugin/Calc.pm 2010-06-24 05:55:47 UTC (rev 19121)
+++ twiki/trunk/SpreadSheetPlugin/lib/TWiki/Plugins/SpreadSheetPlugin/Calc.pm 2010-06-26 01:28:48 UTC (rev 19122)
@@ -106,6 +106,7 @@
$line =~ s/^(\s*\|)(.*)\|\s*$/$2/o;
$before = $1;
@row = split( /\|/o, $line, -1 );
+ $row = '' unless( @row );
push( @tableMatrix, [ @row ] );
$line = "$before";
- 26 Jun 2010