Bug 304642: Bugzilla::Flag::GetTarget() should be called internally by Bugzilla::Flag::process() - Patch by Frédéric Buclin <LpSolit@gmail.com> r/a=myk

1 files changed, 2 insertions, 4 deletions

diff --git a/attachment.cgi b/attachment.cgi
index ea04e2c19..fba504690 100755
--- a/attachment.cgi
+++ b/attachment.cgi
@@ -1054,8 +1054,7 @@ sub insert
   }   
   
   # Create flags.
-  my $target = Bugzilla::Flag::GetTarget(undef, $attachid);
-  Bugzilla::Flag::process($target, $timestamp, $cgi);
+  Bugzilla::Flag::process($bugid, $attachid, $timestamp, $cgi);
    
   # Define the variables and functions that will be passed to the UI template.
   $vars->{'mailrecipients'} =  { 'changer' => Bugzilla->user->login,
@@ -1203,8 +1202,7 @@ sub update
   # to attachments so that we can delete pending requests if the user
   # is obsoleting this attachment without deleting any requests
   # the user submits at the same time.
-  my $target = Bugzilla::Flag::GetTarget(undef, $attach_id);
-  Bugzilla::Flag::process($target, $timestamp, $cgi);
+  Bugzilla::Flag::process($bugid, $attach_id, $timestamp, $cgi);
 
   # Update the attachment record in the database.
   SendSQL("UPDATE  attachments