Item1242: EditTablePlugin cannot move rows correctly when we have footerrows

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, distro:c84977b72e9e 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