Item1184: CALC inside table is not working

pencil
Priority: Urgent
Current State: Closed
Released In: 1.0.4
Target Release: patch
Applies To: Engine
Component:
Branches:
Reported By: Foswiki:Main.KennethLavrsen
Waiting For:
Last Change By: KennethLavrsen
SpreadSheetPlugin CALC does not work if it is on the same topic as an EDITTABLE tag.

The interaction between the two seems borked.

Item Description Qty Reason Unit Price Total Price
Hej Ho
4
50000
200000
Jah No
2
Hello
150000
300000
 
 
 
0
0
Total
 
 
 
500000

Conversion Specifier Description Before Description After
%a
italic bold script=
==bold script=
bold italic
Keep as is
%A
The full weekday name according to the current locale.
Keep as is
dsfdsa
dsDSd
FDSF
with br /
*bold*
_italic_
=script=
Rendered?
with BR macro
*bold*
_italic_
=script=
Rendered?

And inside normal text, with br / :

*bold*
_italic_
=script=

and with BR macro:

*bold*
_italic_
=script=

This bug makes me stop 1.0.3 release.

-- KennethLavrsen - 02 Mar 2009

Editing the tables to add TML formatting, so I can see what it's supposed to be doing.

-- OlivierRaginel - 02 Mar 2009

Moving the Foswiki::Plugins::EditTablePlugin::Core::prepareForView from beforeCommonTagsHandler to commonTagsHandler fixes this problem but re-introduces the huge problem that macros in a table are saved as their value instead of staying a macro.

I have inserted %WIKINAME% in the table above as an example. Edit and save should end up saving %WIKINAME% and not the value of %WIKINAME.

-- KennethLavrsen - 02 Mar 2009

Fixed: in handleTmlInViewMode variables are expanded now.

-- ArthurClemens - 02 Mar 2009

Thanks Arthur for fixing this so quickly.

And thanks also to Olivier for trying hard.

I will give it a good testing today at work.

-- KennethLavrsen - 03 Mar 2009

Fixed the unit test.

-- ArthurClemens - 03 Mar 2009

Removed my comments which were wrong.

Re-opening the bug so I can merge the re-factoring I did while analyzing this bug. I hope Arthur you like the way I re-factored, but as you implemented some of the things I did, I hope you will.

Also, fixed one bug (if you clicked "Cancel" with assert on, it would crash, as FSA no longer terminates upon redirects. (and merged this into Release01x00)

Arthur set this to "waiting for release", so I left it that way, even though I think it should be closed, as this was introduced between 1.0.0 and the next release.

-- OlivierRaginel - 03 Mar 2009
Topic revision: r24 - 19 Mar 2009, KennethLavrsen
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy