[ic] [interchange] Add numeric ordercheck

Mike Heins mike at perusion.com
Thu Mar 25 17:14:21 UTC 2010


Quoting Ton Verhagen (tverhagen at alamerce.nl):
> 
> On Mar 25, 2010, at 17:19 PM, Jon Jensen wrote:
> 
> > On Thu, 25 Mar 2010, Mike Heins wrote:
> > 
> >> Hmm. This is no definition of numeric I know of.
> >> 
> >> If it is for a SQL purpose, perhaps. I don't know enough to check that. I will tell you MySQL and Postgres both accept +1, 1. and .1 just fine, as does Perl.
> > 
> > Yes, we considered those but decided to make the check restrictive at least to start with. We can loosen it up if needed.
> > 
> > The idea was that if validating a numeric entry from a customer, 1. is almost certainly missing something, +1 is not likely intentional, and .1 we were on the fence about -- some cases it might be intentional, other times a mistake.
> > 
> >> I think perhaps I would prefer:
> >> 
> >> 	use Scalar::Util qw/looks_like_number/;
> > 
> > I could see using that. Ton, what do you think?
> 
> 
> The purpose for which we wanted this check was to check on user
> input (mainly backend) for amounts, like prices etc. Price amounts
> normally look like 'd.dd' or '-d.dd'. Having prices like '+1.25'
> input into a table would not make much sense to us. Same for .1 and
> 1. type of input.
>
> Maybe we should rename the current numeric order check to reflect it's
> purpose. Or an other idea would be to have a kind of a 'price/amount
> picture' order check, where the format given should be matched.

How about using "currency" as the name? That would make me happy.

-- 
Mike Heins
Perusion -- Expert Interchange Consulting    http://www.perusion.com/
phone +1.765.328.4479  <mike at perusion.com>

The tenor's voice is spoilt by affectation,
 And for the bass, the beast can only bellow;
In fact, he had no singing education,
 An ignorant, noteless, timeless, tuneless fellow. -- Lord Byron



More information about the interchange-users mailing list