Item5489: Tables can no longer be rendered in text area form fields

This did not work in Cairo either but actually worked in 4.1

In 4.2 it is broken again.

4.1: http://merlin.lavrsen.dk/twiki41/bin/view/Motion/ConfigOptionAutoBrightness

4.2: http://merlin.lavrsen.dk/twiki42/bin/view/Motion/ConfigOptionAutoBrightness

Note that formfield works but in the form itself the table is shown as raw text.

I checked the two topics and they are identical so it saves OK. It is the rendering that goes wrong.

-- TWiki:Main/KennethLavrsen - 31 Mar 2008

Note: Item1133 is related but not the same bug.

This one is a bug because this was made to work in 4.1 and then broke again. So one would think it is possible to fix. I do not expect the ETP and TP to work in textarea fields but simple tables should be rendered as tables like in my example does in 4.1

-- TWiki:Main.KennethLavrsen - 31 Mar 2008

This is a bug taht needs to be fixed, but I do not consider it a release blocker for 4.2.1. Normal priority recommend.

-- TWiki:Main.PeterThoeny - 01 Apr 2008

Formfield values are protected from further rendering because otherwise it's easy to include TWiki syntax (and even HTML) in form values that breaks the enclosing table, and even the topic.

The solution may be to explicitly run the formfield value through TWiki rendering before protecting it. See TWiki::Form::FieldDefinition::renderForDisplay e.g. by inserting the line: $value = $this->{session}->renderer->getRenderedVersion($value);

Confirmed. However I'm not convinced it is possible to fix, as it is a cross-scripting issue to allow a formfield value to include embedded HTML.

-- CC - 01 May 2008

Thanks for the analysis

If this is not a nobrainer regex fix then we lower this to normal. I do not want to risk injecting a new bug or maybe even a new security issue on a rush fix for 4.2.1.

-- KJL - 01 May 2008