Item4878: 4.2 IF not complete / crashes

looks like the 4.2.0 IF is not ready for release.

• ok
• broken
• ok - THESE CRASHed TWIKI
• ok - THESE CRASHed TWIKI
• broken
• ok
• ok
• broken
• broken
• ok
• ok
• ok
• ok
• ok

it also seems that topics like CurrentState in the released version of the BugsContrib (which work on 4.1.2 etc) are broken, crashing twiki.

I'm trying to craft some way of detecting and fixing 4.0 and 4.1 backwards compatibilitied in TWikiApps, and this is

The broken bullet points, and the extra BR's are inserted erroniously by wysiwyg.

-- TWiki:Main/SvenDowideit - 23 Oct 2007

Most of the above test cases fail due to wrong syntax, according to the code. I fixed the crashes and clarified the documentation to be in line with the code. I am not sure if CDot wants to make work something like

%IF{"defined FOO" then="yes" else="no"}%

instead of

%IF{"defined 'FOO'" then="yes" else="no"}%

that is demand the variable name to be quoted. If so then TWiki::Query::Node::_getField() has to be fixed accordingly. Alas, I can't forsee the implications of such a change in this central function. So I just fixed the docu.

The defined predicate has been fixed also to work for SYSTEMWEB and the like. These are "functionTags" and not url, preference or session parameters. As a sideeffect, you can now check for more function tags, e.g. those registered by plugins.

-- TWiki:Main.MichaelDaum - 24 Oct 2007

trouble is, some of the examples are lifted from released versions of BugsContrib, and so work in 4.0 and 4.1.

-- TWiki:Main.SvenDowideit - 24 Oct 2007

Okay, let's see. isweb and istopic are new in 4.2. So we are left with defined and the question if variable names must be quoted or not.

%IF{"defined MAINWEB" then="ok" else="broken"}%

never worked, as even in 4.0 and 4.1 MAINWEB has been a function tag. That's fixed now.

And yes:

%IF{"defined SYSTEMWEB" then="ok" else="broken"}%

and

%IF{"defined 'SYSTEMWEB'" then="ok" else="broken"}%

both worked just fine before...

-- TWiki:Main.MichaelDaum - 24 Oct 2007

The tests were up the crrek, and giving false positives. Should be fixed now but COULD SOMEONE PLEASE MERGE TO MAIN!

Not without a brain splits, now that you tackled the issue in TWikiRelease04x02 and I fixed it in MAIN, both different. Please, let's resolve issues in one place - MAIN - completely, and then afterwards merge to the release branch.

Thanks,

CC

No, CDot, first tell me if

%IF{"defined SYSTEMWEB" then="ok" else="broken"}%

is supposed to work. I'll merge when the code is fixed.

-- TWiki:Main.MichaelDaum - 24 Oct 2007

red comments added

-- TWiki:Main.MichaelDaum - 24 Oct 2007

I was going to merge Crawfords changes to MAIN when I read this.

Please get the branches in sync asap.

-- TWiki:Main.KennethLavrsen - 24 Oct 2007

Sorry, I'm on the wrong end of a dialup link and merging to both branches is likely to cause errors (tortoise svn is pretty bad for doing merges). Since the priority if for 4.2, I need to make sure I fix it there. Yes, that code is supposed to work, and does work on the 4.2.0 branch (it is covered by a unit testcase)

-- TWiki:Main.CrawfordCurrie - 24 Oct 2007

Okay I will take care of the merge.

-- TWiki:Main.MichaelDaum - 24 Oct 2007

Many thanks, Michael. I dropped this to Normal priority, as it is no longer a release blocker.

CC

Done.

MD