Bug 1237185 - hide 'cab review' custom field behind a "click through"…

… to direct people to servicenow
mozilla-bteam · Jan 11, 2016 · 9e478dd · 9e478dd
1 parent 9bc226a
commit 9e478dd
Show file tree

Hide file tree

Showing 5 changed files with 86 additions and 6 deletions.
diff --git a/extensions/BMO/template/en/default/hook/bug/edit-custom_field.html.tmpl b/extensions/BMO/template/en/default/hook/bug/edit-custom_field.html.tmpl
@@ -60,3 +60,45 @@
     RETURN;
   END;
 %]
+
+[%
+  IF field.name == 'cf_cab_review' AND bug.cf_cab_review == '---';
+    field.hidden = 1;
+    value = '---';
+%]
+  [%# lifted from bug/field.html.tmpl %]
+  <tr>
+    [% PROCESS "bug/field-label.html.tmpl" hidden=0 %]
+    <td class="field_value" id="field_container_[% field.name FILTER html %]" colspan="2">
+      <span id="cab-review-gate">
+        <a href="https://mozilla.service-now.com/change_request.do?sysparm_stack=change_request_list.do&amp;sys_id=-1&amp;sysparm_query=active=true" target="_blank">ServiceNow Change Request</a>
+        (<a href="#" id="cab-review-gate-close">use flag</a>)
+      </span>
+      <span id="cab-review-edit" style="display:none">
+        <input type="hidden" id="[% field.name FILTER html %]_dirty">
+        <select id="[% field.name FILTER html %]" name="[% field.name FILTER html %]">
+          [%
+            FOREACH legal_value = field.legal_values;
+              NEXT IF NOT legal_value.is_active AND NOT value.contains(legal_value.name).size;
+              NEXT UNLESS bug.check_can_change_field(field.name, '---', legal_value.name)
+                OR value.contains(legal_value.name).size;
+              %]
+              <option value="[% legal_value.name FILTER html %]"
+                      id="v[% legal_value.id FILTER html %]_[% field.name FILTER html %]"
+                      [% IF value.contains(legal_value.name).size %]
+                        selected
+                      [% ELSIF bug AND !legal_value.is_visible_on_bug(bug) %]
+                        class="bz_hidden_option" disabled
+                      [% END %]
+              >[% display_value(field.name, legal_value.name) FILTER html %]</option>
+          [%
+            END;
+          %]
+        </select>
+      </span>
+    </td>
+  </tr>
+[%
+    RETURN;
+  END;
+%]
diff --git a/extensions/BMO/web/js/edit_bug.js b/extensions/BMO/web/js/edit_bug.js
@@ -45,3 +45,12 @@ YAHOO.util.Event.onDOMReady(function() {
       new_comment.style.width = comment_width;
   }
 });
+
+$(function() {
+  $('#cab-review-gate-close')
+    .click(function(event) {
+      event.preventDefault();
+      $('#cab-review-gate').hide();
+      $('#cab-review-edit').show();
+    });
+})(jQuery);
diff --git a/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl b/extensions/BugModal/template/en/default/bug_modal/edit.html.tmpl
@@ -772,11 +772,32 @@
 
     [% UNLESS cf_hidden_in_product('cf_cab_review', bug.product, bug.component, bug) %]
       [% rendered_custom_fields.push('cf_cab_review') %]
-      [% INCLUDE bug_modal/field.html.tmpl
-          field = bug_fields.cf_cab_review
-          field_type = bug_fields.cf_cab_review.type
-          hide_on_view = bug.cf_cab_review == "---"
-      %]
+      [% IF bug.cf_cab_review == "---" %]
+        [% WRAPPER bug_modal/field.html.tmpl
+            field = bug_fields.cf_cab_review
+            field_type = bug_fields.cf_cab_review.type
+            container = 1
+            hide_on_view = 1
+        %]
+          <span id="cab-review-gate">
+            <a href="https://mozilla.service-now.com/change_request.do?sysparm_stack=change_request_list.do&amp;sys_id=-1&amp;sysparm_query=active=true" target="_blank">ServiceNow Change Request</a>
+            <button id="cab-review-gate-close" type="button" class="minor">Use Flag</button>
+          </span>
+          <span id="cab-review-edit" style="display:none">
+            [% INCLUDE bug_modal/field.html.tmpl
+                field = bug_fields.cf_cab_review
+                field_type = bug_fields.cf_cab_review.type
+                inline = 1
+                no_indent = 1
+            %]
+          </span>
+        [% END %]
+      [% ELSE %]
+        [% INCLUDE bug_modal/field.html.tmpl
+            field = bug_fields.cf_cab_review
+            field_type = bug_fields.cf_cab_review.type
+        %]
+      [% END %]
     [% END %]
 
   [% END %]

diff --git a/extensions/BugModal/web/bug_modal.js b/extensions/BugModal/web/bug_modal.js
@@ -898,6 +898,14 @@ $(function() {
             }
         });
 
+    // cab review 'gate'
+    $('#cab-review-gate-close')
+        .click(function(event) {
+            event.preventDefault();
+            $('#cab-review-gate').hide();
+            $('#cab-review-edit').show();
+        });
+
     // custom textarea fields
     $('.edit-textarea-btn')
         .click(function(event) {

diff --git a/scripts/migrate-cab-review.pl b/scripts/migrate-cab-review.pl
@@ -11,7 +11,7 @@
 use warnings;
 
 use FindBin qw($RealBin);
-use lib "$RealBin/../../..";
+use lib "$RealBin/..";
 
 use Bugzilla;
 use Bugzilla::Bug;