CustomFields - suggestion for new validation sub

[SrNupsen]

I have a suggestion for a small validation sub to be included in the next version of CustomFields. The purpose is to check for "a valid file name".

Perhaps the most handy would be to check that the file uploaded only contains a-z/A-Z, underscores and 0-9.

This way I won't have to be concerned about my users getting into trouble when posting pictures etc.

SrNupsen

[bozoka45]

This might not work because my regexp stinks, but I'm fairly certain I'm on the right path. You can simply use a regexp that checks for those values, and those values only.

Code: Select all

/^([\w|\.])*?$/


Test it out, but it should work for alphanumerics only.

[Parahead]

If the regexp works, then all you have to do is incorporate it in a validation sub and place it in cruser.pl and it will be available to choose from the validation dropdown. Check out the addon docs for an example of how to create such a validation sub.

[faithless]

Code: Select all

/^([\w|\.])*?$/

\. matches for the actual . (dot) sign.
. matches for any characters at all, including non-alphanumerics.
In your pattern, \w matches for alpha characters, so in order to catch underscores and numerics, you probably need to add \d and _ into the [] brackets.

* match zero or more any character.
? match zero or one character.
In this case, since it's a user name, we probably want to match one or more. Hence + would probably be more suitable.

Somthing like the one below might just do the trick:

Code: Select all

/^([\w|\_|\d]+)$/

Additional Readings:
http://free.beyonddream.com/CGITutorial/pattern.html

[bozoka45]

So is this page wrong about regexp?

\w Match "word" character (alphanumeric plus "_")

[SrNupsen]

If the regexp works, then all you have to do is incorporate it in a validation sub and place it in cruser.pl and it will be available to choose from the validation dropdown. Check out the addon docs for an example of how to create such a validation sub.

I know - my suggestion was to include it with the next release

SrNupsen

[faithless]

So is this page wrong about regexp?

\w Match "word" character (alphanumeric plus "_")

Woops. my bad. haha. It's correct.
I'm always under the impression that \w consists of a-z and A-Z only.
In that case, I guess it can be shortened to

Code: Select all

/^([\w|\_]+)$/

[Parahead]

If the regexp works, then all you have to do is incorporate it in a validation sub and place it in cruser.pl and it will be available to choose from the validation dropdown. Check out the addon docs for an example of how to create such a validation sub.

I know - my suggestion was to include it with the next release

Okidoki, wasn't sure so just wanted to point it out. Anyway, this thread is bookmarked now...