From 42e7bbfa6271c99fff3fa484733a504f4bc19c20 Mon Sep 17 00:00:00 2001 From: David Lawrence Date: Fri, 13 Feb 2015 19:05:07 +0000 Subject: Bug 1108816: Project kickoff form, changes to privacy review --- extensions/MozProjectReview/Extension.pm | 62 +++-------- ...ent-moz-project-review-data-compliance.txt.tmpl | 28 +++++ .../bug/create/create-moz-project-review.html.tmpl | 121 +++++++++++++++------ .../MozProjectReview/web/js/moz_project_review.js | 27 ++--- 4 files changed, 137 insertions(+), 101 deletions(-) create mode 100644 extensions/MozProjectReview/template/en/default/bug/create/comment-moz-project-review-data-compliance.txt.tmpl (limited to 'extensions/MozProjectReview') diff --git a/extensions/MozProjectReview/Extension.pm b/extensions/MozProjectReview/Extension.pm index 215927146..a5f4e0b8c 100644 --- a/extensions/MozProjectReview/Extension.pm +++ b/extensions/MozProjectReview/Extension.pm @@ -46,13 +46,11 @@ sub post_bug_after_creation { 'legal_cc' => { 'type' => 'multi' } }); - my ($do_sec_review, $do_legal, $do_finance, $do_privacy_vendor, - $do_privacy_tech, $do_privacy_policy); + my ($do_sec_review, $do_legal, $do_finance, $do_data_compliance); if ($params->{'mozilla_data'} eq 'Yes') { $do_legal = 1; - $do_privacy_policy = 1; - $do_privacy_tech = 1; + $do_data_compliance = 1; $do_sec_review = 1; } @@ -62,7 +60,7 @@ sub post_bug_after_creation { } if ($params->{'data_access'} eq 'Yes') { - $do_privacy_policy = 1; + $do_data_compliance = 1; $do_legal = 1; $do_sec_review = 1; } @@ -70,7 +68,7 @@ sub post_bug_after_creation { if ($params->{'data_access'} eq 'Yes' && $params->{'privacy_policy_vendor_user_data'} eq 'Yes') { - $do_privacy_vendor = 1; + $do_data_compliance = 1; } if ($params->{'vendor_cost'} eq '> $25,000' @@ -81,9 +79,8 @@ sub post_bug_after_creation { } } - my ($sec_review_bug, $legal_bug, $finance_bug, $privacy_vendor_bug, - $privacy_tech_bug, $privacy_policy_bug, $error, @dep_comment, - @dep_errors, @send_mail); + my ($sec_review_bug, $legal_bug, $finance_bug, $data_compliance_bug, + $error, @dep_comment, @dep_errors, @send_mail); # Common parameters always passed to _file_child_bug # bug_data and template_suffix will be different for each bug @@ -161,55 +158,22 @@ sub post_bug_after_creation { _file_child_bug($child_params); } - if ($do_privacy_tech) { + if ($do_data_compliance) { $child_params->{'bug_data'} = { - short_desc => 'Privacy-Technical Review: ' . $bug->short_desc, - product => 'mozilla.org', - component => 'Security Assurance: Review Request', - bug_severity => 'normal', - priority => '--', - keywords => 'privacy-review-needed', - groups => [ 'mozilla-employee-confidential' ], - op_sys => 'All', - rep_platform => 'All', - version => 'other', - blocked => $bug->bug_id, - }; - $child_params->{'template_suffix'} = 'privacy-tech'; - _file_child_bug($child_params); - } - - if ($do_privacy_policy) { - $child_params->{'bug_data'} = { - short_desc => 'Privacy-Policy Review: ' . $bug->short_desc, - product => 'Privacy', - component => 'Product Review', - bug_severity => 'normal', - priority => '--', - groups => [ 'mozilla-employee-confidential' ], - op_sys => 'All', - rep_platform => 'All', - version => 'unspecified', - blocked => $bug->bug_id, - }; - $child_params->{'template_suffix'} = 'privacy-policy'; - _file_child_bug($child_params); - } - - if ($do_privacy_vendor) { - $child_params->{'bug_data'} = { - short_desc => 'Privacy / Vendor Review: ' . $bug->short_desc, - product => 'Privacy', - component => 'Vendor Review', + short_desc => 'Data Compliance Review: ' . $bug->short_desc, + product => 'Data Compliance', + component => 'General', bug_severity => 'normal', priority => '--', + bug_severity => $params->{data_comp_urgency}, groups => [ 'mozilla-employee-confidential' ], op_sys => 'All', rep_platform => 'All', version => 'unspecified', + cf_due_date => $params->{release_date}, blocked => $bug->bug_id, }; - $child_params->{'template_suffix'} = 'privacy-vendor'; + $child_params->{'template_suffix'} = 'data-compliance'; _file_child_bug($child_params); } diff --git a/extensions/MozProjectReview/template/en/default/bug/create/comment-moz-project-review-data-compliance.txt.tmpl b/extensions/MozProjectReview/template/en/default/bug/create/comment-moz-project-review-data-compliance.txt.tmpl new file mode 100644 index 000000000..889543c0c --- /dev/null +++ b/extensions/MozProjectReview/template/en/default/bug/create/comment-moz-project-review-data-compliance.txt.tmpl @@ -0,0 +1,28 @@ +[%# This Source Code Form is subject to the terms of the Mozilla Public + # License, v. 2.0. If a copy of the MPL was not distributed with this + # file, You can obtain one at http://mozilla.org/MPL/2.0/. + # + # This Source Code Form is "Incompatible With Secondary Licenses", as + # defined by the Mozilla Public License, v. 2.0. + #%] + +[% USE Bugzilla %] +[% cgi = Bugzilla.cgi %] + +> Request Type +[%+ cgi.param('data_comp_request_type') %] + +> Area +[%+ cgi.param('data_comp_area') %] + +> Description +[%+ cgi.param('data_comp_desc') %] + +> Does this request involve a change in data collection or handling? +[%+ cgi.param('data_comp_handling_change') %] + +> Does this request involve a change in to the business unit's data practices? +[%+ cgi.param('data_comp_practice_change') %] + +> Other Information +[%+ cgi.param('data_comp_other') || "None" %] diff --git a/extensions/MozProjectReview/template/en/default/bug/create/create-moz-project-review.html.tmpl b/extensions/MozProjectReview/template/en/default/bug/create/create-moz-project-review.html.tmpl index 0dd3da5a8..d3a2a7253 100644 --- a/extensions/MozProjectReview/template/en/default/bug/create/create-moz-project-review.html.tmpl +++ b/extensions/MozProjectReview/template/en/default/bug/create/create-moz-project-review.html.tmpl @@ -323,60 +323,109 @@ -
-
Privacy (Policy/Project)
+
+
Data Compliance
+ +
+ Type of request + + + +
-