Set Date to null if character entered is non alpha numeric instead of 31 December 1899

[davidpatters0n]

The purpose of this pull request is to address the potential affect of entering a non alpha-numeric which sets the date back to 31 December 1899 as shown
This change simply matches against any single non alpha-numeric character and returns null. Example shown

[ahmetabdi]

👍

[AuspeXeu]

@davidpatters0n Could you explain the regex to me briefly, please? Thanks!

[davidpatters0n]

@AuspeXeu the regex will match characters that are numeric and followed by a hyphen followed by another number more than 1 or more times.

[AuspeXeu]

@davidpatters0n So this is format specific as it requires the user to use hyphens. Is that correct?

[davidpatters0n]

Correct. As shown in the above gif in the description you'll see that entering a hyphen in this instance will cause the date to be set 31 December 1899

[AuspeXeu]

But wouldn't it make sense to have it tolerate different date formats? Maybe even rely on the format that is currently set for the component?

[davidpatters0n]

That is correct. As shown below, i ran the regex against a series of different date formats and it seems to comply.

!(/^(\d+-?|\d+\/?|.)+\d.*/).test('2015-04-05')
false
!(/^(\d+-?|\d+\/?|.)+\d.*/).test('05-04-2015')
false
!(/^(\d+-?|\d+\/?|.)+\d.*/).test('05/04/2015')
false
!(/^(\d+-?|\d+\/?|.)+\d.*/).test('2015/04/05')
false
!(/^(\d+-?|\d+\/?|.)+\d.*/).test('-')
true
!(/^(\d+-?|\d+\/?|.)+\d.*/).test('?')
true
!(/^(\d+-?|\d+\/?|.)+\d.*/).test('^')
true

[davidpatters0n]

And I also think maybe it would be a good idea for it to rely based on the format of the what is currently set for the component.

[AuspeXeu]

Couldn't we use the format setting to generate a regex from that, what do you think?

[SoLoHiC]

👍

[gf3]

👍

[davidpatters0n]

@AuspeXeu - The tests on Travis are going to continue to fail because of this PR: #571 - As mentioned in the comment for that PR i have opened a separate PR that addresses this too.

WRT to your comment:

Couldn't we use the format setting to generate a regex from that, what do you think?

I didn't need to do this because I made use of: nonpunctuation instead which works just as well.