Form validation issues

[miniskulljob]

Hi.

The new form validation shorthand has some issues. Check out this JSFiddle. In it, I have used the two methods: the conventional one, and the shorthand. Conventional works fine, but the shorthand always returns a success, even if the field is empty.
Another issue is that both methods call two times the result function, onSuccess or onFailure.

[jlukic]

Yeah... I'm not sure why this obvious case isnt working.. grr...

[maturanomx]

I made a PR that I believe solve this issue[*]. You can see the result here: https://jsbin.com/xahide/3/edit?html,console,output

[*]: Although I don't know what I'm doing 😅

[miniskulljob]

Yeah, seems like it works, but it isn't using the official repos from semantic.org, maybe there's something different...
And, in your jsbin, the result function is called two times too.

[jlukic]

I'll try to take a look at this soon.

[maturanomx]

@miniskulljob

but it isn't using the official repos from semantic.org

That's right. Is using the version of my fork, just for test the patch.

And, in your jsbin, the result function is called two times too

It seems that something has changed at this time. Also your JSFiddle change; both cases doesn't works, now generate the error TypeError: range.indexOf is not a function (confirm please)

This PR solve this new error... I keep watching

Edit

I'm confused. I don't see any change in next neither master.

Really your JSFiddle change the behavior your describe here or I didn't understand?. This is what I see:

[federicotarantino]

Hi,
related to this issue, i suggest implementing range logic also on "decimal" rule.
In my project I create a "range" rule that work with integer and decimal.

[jlukic]

I believe this was fixed but mistagged. If I'm wrong here please let me know.