Forum OpenACS Q&A: Re: Guidance on groups ui

Collapse
9: Re: Guidance on groups ui (response to 1)
Posted by Jeff Davis on
Making that one line change will make the code broken the way it was broken since it was written. The right fix is to put
the . back in and change the regexps in the if tag and
elsewhere to make them consistent. That is what I have done on HEAD.

The reason x.y.z needs to parse to x(y.z) is because if you
create array variables in forms you need the formerror key and some other things to be able to parse this
(when element.id is group.group_name)

<if \@mailto:formerrer.@element.id@\@ nil>
to
if {[exists_and_not_null formerror(group.group_name)]}
If the regexps are inconsistent the nil op in if
does not recognize it as a variable and the regular
adp variable regexp picks it up and tries to localize it.

see this commit for details.