Bug 1453272 - Don't set specifically requestable flags, including needinfo, when...

diff --git a/extensions/BugModal/web/bug_modal.js b/extensions/BugModal/web/bug_modal.js
index 598c8c252f2d0089296f4b4b46909a2be5fa4e22..ed3b1afe24845909aaa5d07e91a10e9e8581b869 100644 (file)
--- a/extensions/BugModal/web/bug_modal.js
+++ b/extensions/BugModal/web/bug_modal.js
@@ -810,10 +810,11 @@ $(function() {
             var id = target.prop('id').replace(/^flag(_type)?-(\d+)/, "#requestee$1-$2");
             if (target.val() == '?') {
                 $(id + '-container').show();
-                $(id).focus().select();
+                $(id).focus().select().prop('required', true);
             }
             else {
                 $(id + '-container').hide();
+                $(id).prop('required', false);
             }
         });
 

diff --git a/js/flag.js b/js/flag.js
index 64b4ad8a29d5b4b89e7d3c6f0ba10b83cc2e038a..6a712da7a6044d7f1702ee86f45736caacf53968 100644 (file)
--- a/js/flag.js
+++ b/js/flag.js
@@ -38,9 +38,12 @@ function toggleRequesteeField(flagField, no_focus)
   // of the flag field.
   if (flagField.value == "?") {
       YAHOO.util.Dom.removeClass(requesteeField.parentNode, 'bz_default_hidden');
+      requesteeField.required = true;
       if (!no_focus) requesteeField.focus();
-  } else
+  } else {
       YAHOO.util.Dom.addClass(requesteeField.parentNode, 'bz_default_hidden');
+      requesteeField.required = false;
+  }
 }
 
 // Hides requestee fields when the window is loaded since they shouldn't

diff --git a/qa/t/test_flags.t b/qa/t/test_flags.t
index 34b47e24da438c0a94cad90d1228e828a474e3ea..904584eed7d88bae30c9a34543426f2b19349627 100644 (file)
--- a/qa/t/test_flags.t
+++ b/qa/t/test_flags.t
@@ -270,7 +270,9 @@ ok(
 # the MTA.
 
 $sel->select_ok("flag_type-$flagtype1_id", "label=?");
+$sel->type_ok("requestee_type-$flagtype1_id", $config->{admin_user_login});
 $sel->select_ok("flag_type-$flagtype2_id", "label=?");
+$sel->type_ok("requestee_type-$flagtype2_id", $config->{admin_user_login});
 $sel->select_ok("flag_type-$flagtype3_id", "label=?");
 $sel->type_ok("comment", "Setting all 3 flags to ?");
 $sel->click_ok("commit");
@@ -356,10 +358,12 @@ ok(
   "Inactive SeleniumAttachmentFlag3Test flag type not displayed"
 );
 
-# Let's generate some "flagmail", first with no requestee.
+# Let's generate some "flagmail" first.
 
 $sel->select_ok("flag_type-$aflagtype1_id", "label=?");
+$sel->type_ok("requestee_type-$aflagtype1_id", $config->{admin_user_login});
 $sel->select_ok("flag_type-$aflagtype2_id", "label=?");
+$sel->type_ok("requestee_type-$aflagtype2_id", $config->{admin_user_login});
 $sel->type_ok("comment", "patch for testing purposes only");
 $sel->click_ok("create");
 $sel->wait_for_page_to_load_ok(WAIT_TIME);

diff --git a/qa/t/test_flags2.t b/qa/t/test_flags2.t
index bb23ac00a59f2485a8ee637b92aca9c19711e918..fb62a382ddd31b9ead72405f3602b6f5754a853c 100644 (file)
--- a/qa/t/test_flags2.t
+++ b/qa/t/test_flags2.t
@@ -229,6 +229,7 @@ $sel->select_ok("component", "label=c1");
 $sel->is_editable_ok("flag_type-$flagtype1_id",
   "The selenium bug flag type is not selectable");
 $sel->select_ok("flag_type-$flagtype1_id", "label=?");
+$sel->type_ok("requestee_type-$flagtype1_id", $config->{admin_user_login});
 $sel->type_ok("short_desc", "Create a new selenium flag for c2");
 $sel->type_ok("comment",    ".");
 $sel->click_ok("commit");