From 81b2c31a192082fd09787f8cf08ba5ef2236abfb Mon Sep 17 00:00:00 2001
From: Byron Jones <glob@mozilla.com>
Date: Thu, 22 Jan 2015 13:00:21 +0800
Subject: Bug 1119988: form.dev-engagement-event: disabled accounts causes
 invalid/incomplete bugs to be created

---
 extensions/BMO/Extension.pm | 11 +++++++++++
 1 file changed, 11 insertions(+)

(limited to 'extensions/BMO/Extension.pm')

diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm
index 604de68cb..458be737f 100644
--- a/extensions/BMO/Extension.pm
+++ b/extensions/BMO/Extension.pm
@@ -1390,6 +1390,7 @@ sub _post_dev_engagement {
     Bugzilla->error_mode(ERROR_MODE_DIE);
 
     my $discussion_bug;
+    my @warnings;
     eval {
         # Add attachment containing tab delimited field values for
         # spreadsheet import.
@@ -1485,6 +1486,12 @@ EOF
         foreach my $type (@{ $discussion_bug->flag_types }) {
             next if $type->name ne 'needinfo';
             foreach my $requestee (DEV_ENGAGE_DISCUSS_NEEDINFO()) {
+                # needinfo'ing a disable account throws an error - warn instead
+                my $requestee_object = Bugzilla::User->new({ name => $requestee, cache => 1 });
+                if (!$requestee_object || !$requestee_object->is_enabled) {
+                    push @warnings, "Failed to needinfo $requestee on dev-engagement bug (does not exist or disabled)";
+                    next;
+                }
                 my $needinfo_flag = {
                     type_id   => $type->id,
                     status    => '?',
@@ -1508,6 +1515,10 @@ EOF
     Bugzilla->set_user($old_user);
     Bugzilla->error_mode($error_mode_cache);
 
+    foreach my $warning (@warnings) {
+        warn $warning . "\n";
+    }
+
     # No matter what happened, ensure the parent bug gets marked as updated
     # There's no need to send mail for parent bug
     $parent_bug->update($parent_bug->creation_ts);
-- 
cgit v1.2.3-24-g4f1b