Show hosts for scheduled jobs

Searching inside scheduled jobs doesn't really work, but at least that's
consistent with the old implementation.

app/controllers/api/v2/job_invocations_controller.rb

@@ -270,6 +270,11 @@ def parent_scope
       def set_hosts_and_template_invocations
         @pattern_template_invocations = @job_invocation.pattern_template_invocations.includes(:input_values)
         @hosts = @job_invocation.targeting.hosts.authorized(:view_hosts, Host)
+
+        unless params[:search].nil?
+          @hosts = @hosts.joins(:template_invocations)
+                         .where(:template_invocations => { :job_invocation_id => @job_invocation.id})
+        end
         @template_invocations = @job_invocation.template_invocations
                                                .where(host: @hosts)
                                                .includes(:input_values)

webpack/JobInvocationDetail/JobInvocationHostTable.js

@@ -50,16 +50,16 @@ const JobInvocationHostTable = ({
     filter = selectedFilter,
     search = urlSearchQuery
   ) => {
-    const baseFilter = `job_invocation.id = ${id}`;
     const dropdownFilterClause =
       filter && filter !== 'all_statuses'
-        ? `and job_invocation.result = ${filter}`
-        : '';
-    const searchQueryClause = search ? `and (${search})` : '';
-    return `${baseFilter} ${dropdownFilterClause} ${searchQueryClause}`;
+        ? `job_invocation.result = ${filter}`
+        : null;
+    const parts = [dropdownFilterClause, search];
+    return parts.filter((x) => x).map((fragment) => `(${fragment}})`).join(' AND ');
   };
 
-  const defaultParams = { search: constructFilter() };
+  const filter = constructFilter();
+  const defaultParams = filter != '' ? { search: filter } : {};
   if (urlPage) defaultParams.page = Number(urlPage);
   if (urlPerPage) defaultParams.per_page = Number(urlPerPage);
   const [expandedHost, setExpandedHost] = useState([]);