Support to "unset" even if one radio is checked

Previously we used jquery-uncheckable-radio to uncheck radios, which has
been broken for a long while. By adding an empty-value radio option along
with Incident's text input, checking it not only indicates that user's
manual input will be used, but also automatically unchecks previously
checked radio input.

Author: sunnavy

html/RTIR/Elements/SelectIncident

@@ -61,6 +61,12 @@
   <input type="hidden" name="<% $Name %>" value="<% $potential[0] %>" />
 % }
 
+% if ( $dformat =~ /RadioButton/ ) {
+  <div class="col-auto mt-1">
+    <input type="radio" name="Incident" value="" class="form-check-input mt-2" />
+  </div>
+% }
+
   <div class="col-auto">
     <input class="form-control" type="text" name="<% $Name %>" value="" data-autocomplete="Tickets" data-autocomplete-limit="Lifecycle = 'incidents'" />
   </div>
@@ -69,14 +75,6 @@
     <input type="submit" name="<% "More$Name" %>" value="<% $config->{'Multiple'}? loc('Add') : loc('Other') %>" class="btn btn-primary" />
   </div>
 % }
-
-% if ( !$config->{'Required'} && $dformat =~ /RadioButton/ ) {
-<script type="text/javascript">
-//<![CDATA[
-jQuery(function(){ jQuery('input[type="radio"][name="<% $Name %>"]').uncheckable() });
-//]]>
-</script>
-% }
 </div>
 <%ARGS>
 $Type

lib/RT/IR.pm

@@ -142,7 +142,6 @@ for my $result_page ( 'Link/FromIncident/', 'Link/ToIncident/', 'Merge/', 'Incid
 use Parse::BooleanLogic;
 my $ticket_sql_parser = Parse::BooleanLogic->new;
 
-RT->AddJavaScript('jquery.uncheckable-radio-0.1.js');
 RT->AddStyleSheets( 'rtir-styles.css' );
 
 # Add the RTIR search result page to the whitelist to allow