fix a bug in Mojolicious::Plugin::Minion::Admin where filtering jobs …

…by task or queue would not reset the offset
mojolicious · Dec 8, 2017 · 4d9bc93 · 4d9bc93
1 parent ee1ecf3
commit 4d9bc93
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/Changes b/Changes
@@ -6,6 +6,8 @@
     Minion::Backend::Pg.
   - Updated Font Awesome to version 5.
   - Improved Minion::Job to clear signal handlers in job processes.
+  - Fixed a bug in Mojolicious::Plugin::Minion::Admin where filtering jobs by
+    task or queue would not reset the offset.
 
 8.03  2017-11-21
   - Removed -f option from worker command.

diff --git a/lib/Mojolicious/Plugin/Minion/resources/templates/minion/jobs.html.ep b/lib/Mojolicious/Plugin/Minion/resources/templates/minion/jobs.html.ep
@@ -112,6 +112,7 @@
         % my $i = 0;
         % for my $job (@$jobs) {
           % $i++;
+          % my $base = url_with->query(offset => 0);
           <tbody>
             <tr>
               <td>
@@ -123,12 +124,12 @@
                 </a>
               </td>
               <td>
-                <a href="<%= url_with->query([task => $job->{task}]) %>">
+                <a href="<%= $base->clone->query([task => $job->{task}]) %>">
                   <%= $job->{task} %>
                 </a>
               </td>
               <td>
-                <a href="<%= url_with->query([queue => $job->{queue}]) %>">
+                <a href="<%= $base->clone->query([queue => $job->{queue}]) %>">
                   <%= $job->{queue} %>
                 </a>
               </td>