Bugzilla – Attachment 27620 Details for
Bug 12114
Reports web service JSON output has an encoding problem
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12114: Fix encoding problem with reports JSON web service (staff)
0002-PASSED-QA-Bug-12114-Fix-encoding-problem-with-report.patch (text/plain), 1.47 KB, created by
Kyle M Hall (khall)
on 2014-04-25 13:00:30 UTC
(
hide
)
Description:
Bug 12114: Fix encoding problem with reports JSON web service (staff)
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2014-04-25 13:00:30 UTC
Size:
1.47 KB
patch
obsolete
>From 24898b2837f6be0dfa1833e700291e6867a2491d Mon Sep 17 00:00:00 2001 >From: Katrin Fischer <Katrin.Fischer.83@web.de> >Date: Mon, 21 Apr 2014 00:07:31 +0200 >Subject: [PATCH 2/3] [PASSED QA] Bug 12114: Fix encoding problem with reports JSON web service (staff) >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit > >Repeat tests from first patch, but this time, access the report >with your staff client base URL. > ><Staff BaseURL>/cgi-bin/koha/svc/report?id=<reportid> > >Based on paste from Galen Charlton. > >http://bugs.koha-community.org/show_bug.cgi?id=9915 > >Followed test plan. Result as expected. >Signed-off-by: Marc Véron <veron@veron.ch> > >Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> >--- > svc/report | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > >diff --git a/svc/report b/svc/report >index 0cff7dc..839a91b 100755 >--- a/svc/report >+++ b/svc/report >@@ -22,7 +22,7 @@ use Modern::Perl; > > use C4::Auth; > use C4::Reports::Guided; >-use JSON; >+use JSON::XS; > use CGI; > > use Koha::Cache; >@@ -66,14 +66,14 @@ unless ($json_text) { > else { > $lines = $sth->fetchall_arrayref; > } >- $json_text = to_json($lines); >+ $json_text = encode_json($lines); > > if ($cache_active) { > $cache->set_in_cache( $cache_key, $json_text, $report_rec->{cache_expiry} ); > } > } > else { >- $json_text = to_json($errors); >+ $json_text = encode_json($errors); > } > } > >-- >1.7.2.5 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 12114
:
27333
|
27334
|
27360
|
27361
|
27619
| 27620 |
27621