765423 - Update security group to mailing list mappings

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Reed Loden [:reed]]

Attachment: patch - v1

As per discussion with dveditz, updating the security group --> mailing list mappings, including adding support for more than one list.

Comment 1

[:glob ✱]

Comment on attachment 633743 [details] [diff] [review]
patch - v1

untested patch:

Global symbol "$group_to_cc_map" requires explicit package name at ./extensions/BMO/Extension.pm line 344.
Global symbol "$group_to_cc_map" requires explicit package name at ./extensions/BMO/Extension.pm line 345.
BEGIN not safe after errors--compilation aborted at ./extensions/BMO/Extension.pm line 755.

should be:

    if (exists $group_to_cc_map{$group}) {
        foreach my $login (@{ $group_to_cc_map{$group} }) {
            my $id = login_to_id($login);
            $recipients->{$id}->{+REL_CC} = Bugzilla::BugMail::BIT_DIRECT();
        }
    }

Comment 2

[Reed Loden [:reed]]

Attachment: patch - v2 (untested)

still untested.

Comment 3

[:glob ✱]

Comment on attachment 634282 [details] [diff] [review]
patch - v2 (untested)

r=glob

Comment 4

[Reed Loden [:reed]]

Committing to: bzr+ssh://bzr.mozilla.org/bmo/4.0/
modified extensions/BMO/Extension.pm
modified extensions/BMO/lib/Data.pm
Committed revision 8217.

Committing to: bzr+ssh://bzr.mozilla.org/bmo/4.2/
modified extensions/BMO/Extension.pm
modified extensions/BMO/lib/Data.pm
Committed revision 8209.