From 4ac48f08be400b915e88ba25249a04a8a2747fd7 Mon Sep 17 00:00:00 2001
From: Max Kanat-Alexander <mkanat@bugzilla.org>
Date: Fri, 26 Feb 2010 23:53:50 -0800
Subject: Bug 547852: Make the JSON-RPC interface return the correct error
 format to JSON-RPC 1.0 clients. r=dkl, a=mkanat

---
 Bugzilla/Error.pm                     |  4 +++-
 Bugzilla/WebService/Server/JSONRPC.pm | 13 ++++++++++---
 2 files changed, 13 insertions(+), 4 deletions(-)

(limited to 'Bugzilla')

diff --git a/Bugzilla/Error.pm b/Bugzilla/Error.pm
index 8e3710129..98e7d1d2f 100644
--- a/Bugzilla/Error.pm
+++ b/Bugzilla/Error.pm
@@ -124,7 +124,9 @@ sub _throw_error {
                 # higher than 999, but we do this to avoid conflicts with
                 # the internal JSON::RPC error codes.
                 $server->raise_error(code    => 100000 + $code,
-                                     message => $message);
+                                     message => $message,
+                                     id      => $server->{_bz_request_id},
+                                     version => $server->version);
                 # We die with no message. JSON::RPC checks raise_error before
                 # it checks $@, so it returns the proper error.
                 die;
diff --git a/Bugzilla/WebService/Server/JSONRPC.pm b/Bugzilla/WebService/Server/JSONRPC.pm
index 35eda1af1..f929b28ac 100644
--- a/Bugzilla/WebService/Server/JSONRPC.pm
+++ b/Bugzilla/WebService/Server/JSONRPC.pm
@@ -35,8 +35,6 @@ sub new {
     Bugzilla->_json_server($self);
     $self->dispatch(WS_DISPATCH);
     $self->return_die_message(1);
-    # Default to JSON-RPC 1.0
-    $self->version(0);
     return $self;
 }
 
@@ -95,6 +93,15 @@ sub datetime_format {
     return $iso_datetime;
 }
 
+
+# Store the ID of the current call, because Bugzilla::Error will need it.
+sub _handle {
+    my $self = shift;
+    my ($obj) = @_;
+    $self->{_bz_request_id} = $obj->{id};
+    return $self->SUPER::_handle(@_);
+}
+
 # Make all error messages returned by JSON::RPC go into the 100000
 # range, and bring down all our errors into the normal range.
 sub _error {
@@ -116,7 +123,7 @@ sub _error {
 
     # We want to always send the JSON-RPC 1.1 error format, although
     # If we're not in JSON-RPC 1.1, we don't need the silly "name" parameter.
-    if (!$self->version) {
+    if (!$self->version or $self->version ne '1.1') {
         my $object = $self->json->decode($json);
         my $message = $object->{error};
         # Just assure that future versions of JSON::RPC don't change the
-- 
cgit v1.2.3-24-g4f1b