Item9786: configure fails on FastReport rows with '{' and '}' in field content

pencil
Priority: Urgent
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Engine
Component:
Branches:
Reported By: KennethLavrsen
Waiting For: Main.KennethLavrsen
Last Change By: KennethLavrsen
I just attempted to run configure and run "Install and update extensions"

And I get

Software error:

Use of uninitialized value in substitution (s///) at /var/www/foswiki11/lib/Foswiki/Configure/Dependency.pm line 409.

So that blew the release again.

-- KennethLavrsen - 04 Oct 2010

I have root caused it as far as it is the Compatibility field in JHotDrawPlugin that contains the characters { and } (configure settings so it is a normal thing to put) that causes this field to be truncated and configure then fails to pick up the right value for the version.

-- KennethLavrsen - 04 Oct 2010

I changed the regex to anchor on \n{.....}\n instead of just {.....}. I couldn't think of a way to use FilterPlugin or the new 1.1 ENCODE macro to force { and } to be encoded as HTML entities.

The problem is that passing arbitrary content to macros requires a well-formed (escaped double-quotes) parameter string.

So I hope this gets us out of trouble and we can release.

In the next minor/major release, we should definitely get FastReport emitting proper JSON, which has a well-formed syntax which can deal with all these issues (and the QUERY macro knows how to emit well-formed JSON).

-- PaulHarvey - 04 Oct 2010

The we would need one FastReport for 1.1.1+ and the old for 1.0.X.

There are people that will be on 1.0.X the next 1-2 years. Even more.

I even see Beijing sites still on the Internet

-- KennethLavrsen - 04 Oct 2010

Closing

-- KennethLavrsen - 04 Oct 2010
 

ItemTemplate edit

Summary configure fails on FastReport rows with '{' and '}' in field content
ReportedBy KennethLavrsen
Codebase 1.1.0, trunk
SVN Range
AppliesTo Engine
Component
Priority Urgent
CurrentState Closed
WaitingFor KennethLavrsen
Checkins distro:f4bdbf06f570 distro:44122dc2bcf1
TargetRelease minor
ReleasedIn 1.1.0
Topic revision: r5 - 04 Oct 2010, 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