Foswiki on GitHub is open for business! Next release meeting: Monday September 29, 1300Z

Item1242: EditTablePlugin cannot move rows correctly when we have footerrows

Priority: CurrentState: AppliesTo: Component: WaitingFor:
Urgent Closed Extension EditTablePlugin  
While working with my test cases I tried to move a row up or down.

I had not expected errors here since this is mainly javascript but there is a problem I just found.

When you have a footerrow defined in the TABLE tag and you move rows up and down there is a problem in the math so when saved it is the wrong rows that are moved up and down.

Use the simple TestCaseTablesMixedTagOrder test case as an example. Edit and move up a row. Or add a row and then move it up. You end up with a mess with the footer moved up in the middle.

-- KennethLavrsen - 11 Mar 2009

Checking in updated test cases that describes the steps to reproduce this bug.

The checkins I do it not a fix.

-- KennethLavrsen - 11 Mar 2009

I have been looking into this for the whole evening, and still don't grasp what is going on.

  • Too bad you didn't tell me you were looking into it... I crashed my evening because I said I gave up, and you said you didn't look into it. So I looked deeper and deeper... -- OlivierRaginel - 12 Mar 2009

TablePlugin does not write the correct thead html in edit mode (and needs to be fixed, as well as it assumes there is always 1 header row...) but that is not the root cause.

  • Exactly. And you're saying this needs to be fixed, but this is fixed. This was the purpose of my commit from last night, Foswikirev:3018 which fixes this, but as you mentioned, doesn't solve the issue. -- OlivierRaginel - 12 Mar 2009

This same bug occurs with TWiki plugin version 4.9.

-- ArthurClemens - 11 Mar 2009

My analysis so far:

First test without headerrows and footerrows

Let's say we have a table:
header
a
b
c
footer

The plugin sees it like this:
row content etrow_id
1 header 1
2 a 2
3 b 3
4 c 4
5 footer 5

  • Are you sure about this? According to the JavaScript code, is assumes the etrow_id from header is first, and from footer is second, therefore I'm very surprised by your 5 there for footer. -- OlivierRaginel - 12 Mar 2009

With javascript interface, swap rows so that it looks like:
header
b
c
a
footer

Now parsing rows:
row content etrow_id
1 header 1
2 a 3
3 b 4
4 c 2
5 footer 5

Now the same thing with headerrows and footerrows:

row content etrow_id
1 header 1
2 a 2
3 b 4
4 c 5
5 footer 3

-- ArthurClemens - 12 Mar 2009

Fixed with Item1268.

-- ArthurClemens - 15 Mar 2009

ItemTemplate edit

Summary EditTablePlugin cannot move rows correctly when we have footerrows
ReportedBy KennethLavrsen
Codebase trunk
SVN Range Foswiki-1.0.0, Thu, 08 Jan 2009, build 1878
AppliesTo Extension
Component EditTablePlugin
Priority Urgent
CurrentState Closed
WaitingFor
Checkins Foswikirev:2964 Foswikirev:2965 Foswikirev:3018 Foswikirev:3026 Foswikirev:3027
TargetRelease patch
ReleasedIn 1.0.4
Topic revision: r13 - 19 Mar 2009, KennethLavrsen
 
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. see CopyrightStatement. Creative Commons License