Bug 835212 - Add additional comment on parent bug listing the bug ids and summaries for child bug reports

1 files changed, 14 insertions, 2 deletions

diff --git a/extensions/MozProjectReview/Extension.pm b/extensions/MozProjectReview/Extension.pm
index c7b6bdd36..83d351b62 100644
--- a/extensions/MozProjectReview/Extension.pm
+++ b/extensions/MozProjectReview/Extension.pm
@@ -82,7 +82,8 @@ sub post_bug_after_creation {
     }
 
     my ($sec_review_bug, $legal_bug, $finance_bug, $privacy_vendor_bug,
-        $data_safety_bug, $privacy_tech_bug, $privacy_policy_bug);
+        $data_safety_bug, $privacy_tech_bug, $privacy_policy_bug,
+        @dep_bug_comment);
 
     eval {
         if ($do_sec_review) {
@@ -99,6 +100,7 @@ sub post_bug_after_creation {
                 blocked      => $bug->bug_id,
             };
             $sec_review_bug = _file_child_bug($bug, $vars, 'sec-review', $bug_data);
+            push(@dep_bug_comment, "Bug " . $sec_review_bug->id . " - " . $sec_review_bug->short_desc);
         }
 
         if ($do_legal) {
@@ -132,7 +134,7 @@ sub post_bug_after_creation {
                 cc           => $params->{'legal_cc'},
             };
             $legal_bug = _file_child_bug($bug, $vars, 'legal', $bug_data);
-
+            push(@dep_bug_comment, "Bug " . $legal_bug->id . " - " . $legal_bug->short_desc);
         }
 
         if ($do_finance) {
@@ -149,6 +151,7 @@ sub post_bug_after_creation {
                 blocked      => $bug->bug_id,
             };
             $finance_bug = _file_child_bug($bug, $vars, 'finance', $bug_data);
+            push(@dep_bug_comment, "Bug " . $finance_bug->id . " - " . $finance_bug->short_desc);
         }
 
         if ($do_data_safety) {
@@ -165,6 +168,7 @@ sub post_bug_after_creation {
                 blocked      => $bug->bug_id,
             };
             $data_safety_bug = _file_child_bug($bug, $vars, 'data-safety', $bug_data);
+            push(@dep_bug_comment, "Bug " . $data_safety_bug->id . " - " . $data_safety_bug->short_desc);
         }
 
         if ($do_privacy_tech) {
@@ -182,6 +186,7 @@ sub post_bug_after_creation {
                 blocked      => $bug->bug_id,
             };
             $privacy_tech_bug = _file_child_bug($bug, $vars, 'privacy-tech', $bug_data);
+            push(@dep_bug_comment, "Bug " . $privacy_tech_bug->id . " - " . $privacy_tech_bug->short_desc);
         }
 
         if ($do_privacy_policy) {
@@ -198,6 +203,7 @@ sub post_bug_after_creation {
                 blocked      => $bug->bug_id,
             };
             $privacy_policy_bug = _file_child_bug($bug, $vars, 'privacy-policy', $bug_data);
+            push(@dep_bug_comment, "Bug " . $privacy_policy_bug->id . " - " . $privacy_policy_bug->short_desc);
         }
 
         if ($do_privacy_vendor) {
@@ -214,6 +220,7 @@ sub post_bug_after_creation {
                 blocked      => $bug->bug_id,
             };
             $privacy_vendor_bug = _file_child_bug($bug, $vars, 'privacy-vendor', $bug_data);
+            push(@dep_bug_comment, "Bug " . $privacy_vendor_bug->id . " - " . $privacy_vendor_bug->short_desc);
         }
     };
 
@@ -233,6 +240,11 @@ sub post_bug_after_creation {
         $vars->{message} = 'moz_project_review_creation_failed';
         $vars->{message_error} = $error;
     }
+
+    if (scalar @dep_bug_comment) {
+        $bug->add_comment(join("\n", @dep_bug_comment));
+        $bug->update();
+    }
 }
 
 sub _file_child_bug {