diff options
| author | Dave Lawrence <dlawrence@mozilla.com> | 2013-02-18 08:02:36 +0100 |
|---|---|---|
| committer | Dave Lawrence <dlawrence@mozilla.com> | 2013-02-18 08:02:36 +0100 |
| commit | 51ac7d7bc2b71b0575d374a7e7f312778be19614 (patch) | |
| tree | 366cc2e206ccfaabbd4f4b7300be729bfdb2f77b /extensions/MozProjectReview/Extension.pm | |
| parent | 744a02114ab467888f3cc82689d9b3d0cd6ba9f8 (diff) | |
| download | bugzilla-51ac7d7bc2b71b0575d374a7e7f312778be19614.tar.gz bugzilla-51ac7d7bc2b71b0575d374a7e7f312778be19614.tar.xz | |
Bug 831770 - Project Kickoff Form: Legal section - Change to "Business Objective"
Diffstat (limited to 'extensions/MozProjectReview/Extension.pm')
| -rw-r--r-- | extensions/MozProjectReview/Extension.pm | 43 |
1 files changed, 20 insertions, 23 deletions
diff --git a/extensions/MozProjectReview/Extension.pm b/extensions/MozProjectReview/Extension.pm index 821a3619e..02621ff7d 100644 --- a/extensions/MozProjectReview/Extension.pm +++ b/extensions/MozProjectReview/Extension.pm @@ -56,34 +56,31 @@ sub post_bug_after_creation { if ($params->{new_or_change} eq 'New') { $do_legal = 1; - $do_privacy_policy = 1; - } - elsif ($params->{new_or_change} eq 'Existing') { - $do_legal = 1; } - if ($params->{separate_party} eq 'Yes' - && $params->{relationship_type} ne 'Hardware Purchase') - { - $do_legal = 1; - } + if ($params->{separate_party} eq 'Yes') { + if ($params->{relationship_type} ne 'Hardware Purchase') { + $do_legal = 1; + } - if ($params->{data_access} eq 'Yes') { - $do_privacy_policy = 1; - $do_sec_review = 1; - } + if ($params->{data_access} eq 'Yes') { + $do_privacy_policy = 1; + $do_legal = 1; + $do_sec_review = 1; + } - if ($params->{data_access} eq 'Yes' - && $params->{'privacy_policy_vendor_user_data'} eq 'Yes') - { - $do_privacy_vendor = 1; - } + if ($params->{data_access} eq 'Yes' + && $params->{'privacy_policy_vendor_user_data'} eq 'Yes') + { + $do_privacy_vendor = 1; + } - if ($params->{vendor_cost} eq '> $25,000' - || ($params->{vendor_cost} eq '<= $25,000' - && $params->{po_needed} eq 'Yes')) - { - $do_finance = 1; + if ($params->{vendor_cost} eq '> $25,000' + || ($params->{vendor_cost} eq '<= $25,000' + && $params->{po_needed} eq 'Yes')) + { + $do_finance = 1; + } } my ($sec_review_bug, $legal_bug, $finance_bug, $privacy_vendor_bug, |