Bug 8278: build the structure with to_json instead of manually.
authorJonathan Druart <jonathan.druart@biblibre.com>
Mon, 22 Apr 2013 14:19:01 +0000 (16:19 +0200)
committerJared Camins-Esakov <jcamins@cpbibliography.com>
Wed, 8 May 2013 13:49:55 +0000 (09:49 -0400)
Test plan:
Same as previously.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>

cataloguing/ysearch.pl

index 9c69939..49dfa39 100755 (executable)
@@ -29,6 +29,7 @@ use CGI;
 use C4::Context;
 use C4::Charset;
 use C4::Auth qw/check_cookie_auth/;
+use JSON qw/ to_json /;
 
 my $input = new CGI;
 my $query = $input->param('term');
@@ -54,14 +55,9 @@ $sql .= qq( ORDER BY $field);
 my $sth = $dbh->prepare($sql);
 $sth->execute("$query%", "% $query%", "%-$query%");
 
-print "[";
-my $i = 0;
+my $a = [];
 while ( my $rec = $sth->fetchrow_hashref ) {
-    if($i > 0){ print ","; }
-    print "{\"fieldvalue\":\"" . nsb_clean($rec->{$field}) . "\"" .
-          "}";
-    $i++;
+    push @$a, { fieldvalue => nsb_clean($rec->{$field}) };
 }
-print "]";
-
 
+print to_json($a);