diff options
| author | Ivan Kohler <ivan@freeside.biz> | 2015-01-07 14:47:53 -0800 |
|---|---|---|
| committer | Ivan Kohler <ivan@freeside.biz> | 2015-01-07 14:47:53 -0800 |
| commit | 1bed0bfbc92a01a86069a947ee016750c334e4d0 (patch) | |
| tree | 5a2f8c7ee8cb3a57ceb74a18a2a085252581cb59 /httemplate/edit | |
| parent | 0f12bb525e362f0eb97085079c3046c4bf2f6e64 (diff) | |
fix one time charge costs, RT#31429, RT#32894
Diffstat (limited to 'httemplate/edit')
| -rw-r--r-- | httemplate/edit/process/quick-charge.cgi | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/httemplate/edit/process/quick-charge.cgi b/httemplate/edit/process/quick-charge.cgi index e87be6272..6de746ea4 100644 --- a/httemplate/edit/process/quick-charge.cgi +++ b/httemplate/edit/process/quick-charge.cgi @@ -83,9 +83,11 @@ if ( $param->{'pkgnum'} =~ /^(\d+)$/ ) { #modifying an existing one-time charge my $amount = $1; my $setup_cost = ''; - $param->{"setup_cost"} =~ /^\s*(\d*(?:\.?\d{1,2}))\s*$/ - or $error .= "Illegal setup_cost " . $param->{"setup_cost"} . " "; - my $setup_cost = $1; + if ( $param->{setup_cost} =~ /\S/ ) { + $param->{setup_cost} =~ /^\s*(\d*(?:\.?\d{1,2}))\s*$/ + or $error .= "Illegal setup_cost " . $param->{setup_cost} . " "; + $setup_cost = $1; + } my $quantity = 1; if ( $cgi->param('quantity') =~ /^\s*(\d+)\s*$/ ) { |