From 503f732b1c64e4c950f747a07dc6cdd056a520ff Mon Sep 17 00:00:00 2001
From: "mkanat%bugzilla.org" <>
Date: Mon, 26 Jan 2009 21:57:28 +0000
Subject: Bug 474997: Make update_see_also actually return the right thing
 Patch by Max Kanat-Alexander <mkanat@bugzilla.org> r=dkl, a=mkanat

---
 Bugzilla/WebService/Bug.pm | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

(limited to 'Bugzilla/WebService')

diff --git a/Bugzilla/WebService/Bug.pm b/Bugzilla/WebService/Bug.pm
index 21645af3d..ffeab2ce1 100755
--- a/Bugzilla/WebService/Bug.pm
+++ b/Bugzilla/WebService/Bug.pm
@@ -375,14 +375,14 @@ sub update_see_also {
     foreach my $bug (@bugs) {
         my $change = $bug->update();
         if (my $see_also = $change->{see_also}) {
-            $changes{$bug->id} = {
+            $changes{$bug->id}->{see_also} = {
                 removed => [split(', ', $see_also->[0])],
                 added   => [split(', ', $see_also->[1])],
             };
         }
         else {
             # We still want a changes entry, for API consistency.
-            $changes{$bug->id} = { added => [], removed => [] };
+            $changes{$bug->id}->{see_also} = { added => [], removed => [] };
         }
 
         Bugzilla::BugMail::Send($bug->id, { changer => $user->login });
@@ -1372,12 +1372,16 @@ bug ids 1 and 2:
  {
    changes => {
        1 => {
-           added   => (an array of bug URLs),
-           removed => (an array of bug URLs),
+           see_also => {
+               added   => (an array of bug URLs),
+               removed => (an array of bug URLs),
+           }
        },
        2 => {
-           added   => (an array of bug URLs),
-           removed => (an array of bug URLs),
+           see_also => {
+               added   => (an array of bug URLs),
+               removed => (an array of bug URLs),
+           }
        }
    }
  }
-- 
cgit v1.2.3-24-g4f1b