summaryrefslogtreecommitdiff
path: root/httemplate
diff options
context:
space:
mode:
authorIvan Kohler <ivan@freeside.biz>2015-01-07 14:47:53 -0800
committerIvan Kohler <ivan@freeside.biz>2015-01-07 14:47:53 -0800
commit1bed0bfbc92a01a86069a947ee016750c334e4d0 (patch)
tree5a2f8c7ee8cb3a57ceb74a18a2a085252581cb59 /httemplate
parent0f12bb525e362f0eb97085079c3046c4bf2f6e64 (diff)
fix one time charge costs, RT#31429, RT#32894
Diffstat (limited to 'httemplate')
-rw-r--r--httemplate/edit/process/quick-charge.cgi8
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*$/ ) {