Javor, thanks for the patch. It seems effective. I have a question
though.

Why do we need to create a new Bugzilla::Bug() ? Couldn't we simply
issue:

Bugzilla::BugMail::Send( $bugid, { changer => $username } );

Can you please try to find the least amount of code that will send the
email ?


On Mon, 2009-03-09 at 18:40 +0200, Javor Nikolov wrote:
> +#
> +# Send notification
> +sub integration_send_notification {
> +    my $self = shift;
> +    my ( $bugid, $username ) = ( @_ );
> +
> +    my $changer = new Bugzilla::User( { name => $username } );
> +    Bugzilla->set_user( $changer );
> +    my $bug = new Bugzilla::Bug( $bugid );
> +    Bugzilla::BugMail::Send( $bug->bug_id, { changer =>
> $bug->reporter->login } );
> +
> +    return 0;
> +}
> + 

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
scmbug-users mailing list
[email protected]
http://lists.mkgnu.net/cgi-bin/mailman/listinfo/scmbug-users

Reply via email to