From 26f83ae708f54be6bb1eb5fff8a8b1745eaaea00 Mon Sep 17 00:00:00 2001
From: Chris Cormack <crc@liblime.com>
Date: Tue, 8 Jan 2008 19:41:21 -0600
Subject: [PATCH] Killing more replace statements, bug 1546

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 admin/cities.pl | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/admin/cities.pl b/admin/cities.pl
index ffda794cee..fe26caaa25 100755
--- a/admin/cities.pl
+++ b/admin/cities.pl
@@ -88,16 +88,16 @@ if ($op eq 'add_form') {
  	my $sth;
 	
 	if ($input->param('cityid') ){
-		$sth=$dbh->prepare("replace cities (cityid,city_name,city_zipcode) values (?,?,?) ");
-		$sth->execute(map { $input->param($_) } ('cityid','city_name','city_zipcode'));
+		$sth=$dbh->prepare("UPDATE cities SET city_name=?,city_zipcode=? WHERE cityid=?");
+		$sth->execute($input->param('city_name'),$input->param('city_zipcode'),$input->param('cityid'));
 	
 	}
 	else{	
-		$sth=$dbh->prepare("replace cities (city_name,city_zipcode) values (?,?)");
-		$sth->execute(map { $input->param($_) } ('city_name','city_zipcode'));
+		$sth=$dbh->prepare("INSERT INTO cities (city_name,city_zipcode) values (?,?)");
+		$sth->execute($input->param('city_name'),$input->param('city_zipcode'));
 	}
 	$sth->finish;
-	print "Content-Type: text/html\n\n<META HTTP-EQUIV=Refresh CONTENT=\"0; URL=cities.pl\"></html>";
+	print $input->redirect("/cgi-bin/koha/admin/cities.pl");
 	exit;
 # END $OP eq ADD_VALIDATE
 ################## DELETE_CONFIRM ##################################
@@ -160,4 +160,4 @@ if ($op eq 'add_form') {
 
 
 } #---- END $OP eq DEFAULT
-output_html_with_http_headers $input, $cookie, $template->output;
\ No newline at end of file
+output_html_with_http_headers $input, $cookie, $template->output;
-- 
2.39.5