Recent changes to bugs

bug in CheckMask function

Anonymous — Thu, 27 Sep 2001 12:26:32 -0000

If you are using custom error messages, the mask test
never returns an error.

need to set the error variable to true (error = true;)
if the mask.test statement (line 118 of
CheckMask_Javascript.cfm) returns false. The current
error=true statement is currently only set when
message_type is not custom - see line 126.

Looks like the error = true; statement should be below
the '}' on the line below!

JS Error when submitting radio buttons.

Trevor Wise — Mon, 02 Jul 2001 20:06:58 -0000

The script returns an error stating that 'type is not
an object' apparently coming from
the "CheckRequired_javascript.cfm" page when radio
buttons are used and validated with CF_CheckForm.

This problem only arises when used with the javascript
validation. The server side validation in Cold Fusion
works just fine.

Even easier to user engine

Eric André Gravel — Wed, 28 Mar 2001 15:08:40 -0000

Alright, thinking a little further on the idea of using
structures (to replace the need to remember that [0]
is for the field name and [1] is the description and
[...]), I've come up with an idea that John had once
mentionned to me. Make use of nested custom tag.

<CF_CheckForm_FORM name="..." method="..." ...>
<CF_CheckForm_INPUT name="..." .../>
...
</CF_CheckForm_FORM>

When the page is processed and get to CF_CheckForm_FORM
it would check for a validation="yes" attribute. If it
finds one then it creates a javascript function to
validate this form. It would also initialize arrays
of structures for all the validation methods.

When it executes a form field type tag such as
CF_CheckForm_INPUT it would store the tag's attribute
into the appropriate array/structure.

When the end tag for the custom tag form is executed
it would output all the relevant information (arrays
into the javascript validation method for that form)
into the javascript section of the page head tag.

Simplified Data Array

Eric André Gravel — Tue, 27 Mar 2001 15:00:30 -0000

It would be simpler for developers to code the tag's
definition data array using structures instead of
array index ro represent a field name or a error
message type.

IE:

StructInsert( requiredCheck, "fieldType", "text" );

is easier to code then

requiredCheck[i][3] = "text";

It's easier in the sense that you know what that array
element is for and it allows you to insert any new
fields without worrying how it'll look at that
position.

Too many array parameters too pass to CheckForm()

Eric André Gravel — Fri, 23 Feb 2001 14:53:10 -0000

For each validation routine you add you also add a new parameter to the CheckForm() list of in parameters. You might not use all of them so it would preferable to provide the means to give only the arrays you need checked and it figures out which ones to call.

To do this we would have an in parameter that contains all the arrays in a 2 dimension array. Dimension 1 would be the array name expected by CheckForm and the second dimension would be the array that would end up being passed to the validation routine.

Minimum Value Length Check

Eric André Gravel — Wed, 14 Feb 2001 20:25:42 -0000

Had a validation for minimum value checks.

note potential problems with mixed-case form field names

Anonymous — Wed, 31 Jan 2001 18:59:37 -0000

I discovered that I had problems getting length and checkbox validation to work in Netscape 4.6 until I converted my form field names to all lowercase instead of mixed case. I am not sure why this was a problem, but I know that WDDX converts variable names to all lower-case when deserializing to JavaScript because ColdFusion is case-insensitive, so it may be the culprit (but then I would expect it to show up in Netscape 6 and IE5 as well, which wasn't the case). You might want to make note of it in the docs.

typo in variable name causes error

Anonymous — Wed, 31 Jan 2001 18:58:18 -0000

CheckLength_coldfusion.cfm: line 72, changed Required -> Length
(it looks like this is just a typo; it caused an error until I fixed it)

add words "at least" to clarify meaning of error message

Anonymous — Wed, 31 Jan 2001 18:54:52 -0000

CheckForm_lang_EN.cfm: line 153, add "At least" because it is possible to check more than the minimum number of checkboxes.

change "then" to "than"

Anonymous — Wed, 31 Jan 2001 18:45:48 -0000

The proper word in a few sentences in the english version is "than" (meaning "in comparison with") instead of "then" (meaning "at that time")

CheckForm_lang_EN.cfm: line 172, then -> than
CheckForm_lang_EN.cfm: line 69, then -> than