X-Git-Url: http://git.freeside.biz/gitweb/?p=freeside.git;a=blobdiff_plain;f=FS%2FFS%2FRecord.pm;h=b476df2c0cd2f0509f28188d22a4059829ea4b3a;hp=2c745b4137a599ac31749ed340a4a2a5b40ee081;hb=f5266a4d07d116efd732f433d0f4f3a47b143a7d;hpb=6ef34dda51afba96d8dc6c4dd72427c3d4003945 diff --git a/FS/FS/Record.pm b/FS/FS/Record.pm index 2c745b413..b476df2c0 100644 --- a/FS/FS/Record.pm +++ b/FS/FS/Record.pm @@ -813,10 +813,17 @@ Check/untaint zip codes. =cut sub ut_zip { - my( $self, $field ) = @_; - $self->getfield($field) =~ /^\s*(\w[\w\-\s]{2,8}\w)\s*$/ - or return "Illegal (zip) $field: ". $self->getfield($field); - $self->setfield($field,$1); + my( $self, $field, $country ) = @_; + if ( $country eq 'US' ) { + $self->getfield($field) =~ /\s*(\d{5}(\-\d{4})?)\s*$/ + or return "Illegal (zip) $field for country $country: ". + $self->getfield($field); + $self->setfield($field,$1); + } else { + $self->getfield($field) =~ /^\s*(\w[\w\-\s]{2,8}\w)\s*$/ + or return "Illegal (zip) $field: ". $self->getfield($field); + $self->setfield($field,$1); + } ''; } @@ -848,13 +855,30 @@ Untaints arbitrary data. Be careful. =cut sub ut_anything { - my($self,$field)=@_; + my( $self, $field ) = @_; $self->getfield($field) =~ /^(.*)$/s or return "Illegal $field: ". $self->getfield($field); $self->setfield($field,$1); ''; } +=item ut_enum COLUMN CHOICES_ARRAYREF + +Check/untaint a column, supplying all possible choices, like the "enum" type. + +=cut + +sub ut_enum { + my( $self, $field, $choices ) = @_; + foreach my $choice ( @$choices ) { + if ( $self->getfield($field) eq $choice ) { + $self->setfield($choice); + return ''; + } + } + return "Illegal (enum) field $field: ". $self->getfield($field); +} + =item fields [ TABLE ] This can be used as both a subroutine and a method call. It returns a list @@ -969,7 +993,7 @@ sub DESTROY { return; } =head1 VERSION -$Id: Record.pm,v 1.25 2001-08-21 09:34:13 ivan Exp $ +$Id: Record.pm,v 1.27 2001-09-11 00:08:18 ivan Exp $ =head1 BUGS