Skip to main content

Bug Tracker

Side navigation

#2424 closed bug (fixed)

Opened February 27, 2008 01:24PM UTC

Closed March 20, 2008 12:15PM UTC

Last modified March 15, 2012 02:05AM UTC

[validate] custom rule dont work with maxlength property.

Reported by: vinipitta Owned by: joern
Priority: major Milestone: 1.2.3
Component: plugin Version: 1.2.1
Keywords: validate Cc:
Blocked by: Blocking:

PLUGIN: Validate

VERSION: 1.2.1

A custom rule don't work if the ''maxlength'' property of the input field is defined. I created an example that illustrate the bug. There are two input text fields and both are validated by the same rule. The first have the maxlength property defined. The custom rule show a alert message with the field name. The first field will be checked only when there is some text in the field because in other way, the maxlength rule will return a ''dependecy-mismatch'' and the check method will abort the execution. The second field will be aways checked because it has not the maxlength property. This execution flow can be easy checked using the firebug plugin of Firefox to debug the js execution. See the check method starting in the 411 line when the rules for the element are retrieved.

Attachments (1)
  • test.html (1.6 KB) - added by vinipitta February 27, 2008 01:30PM UTC.

    Showcase of the bug.

Change History (5)

Changed February 29, 2008 01:50PM UTC by joern comment:1

owner: → joern

Changed March 11, 2008 09:12AM UTC by gregers comment:2

I think this might be related to the bug I reported last week:

If a field has multiple tests, and one of the tests returns "dependency-mismatch", it will stop further testing of this field and return without any value. I believe it should continue to do the other tests, and if none of them fail, it can return without any value. This is fixed in my patch.

Changed March 19, 2008 06:48PM UTC by joern comment:3

resolution: → duplicate
status: newclosed

Duplicate of #2473.

Changed March 20, 2008 11:27AM UTC by joern comment:4

resolution: duplicate
status: closedreopened

I take that back. The issue described in #2473 is similar, but not the same.

Changed March 20, 2008 12:15PM UTC by joern comment:5

resolution: → fixed
status: reopenedclosed

Fixed in [5098].