clean up job listing, bug fixes

app/controllers/delayed_jobs_controller.rb

@@ -1,5 +1,5 @@
 class DelayedJobsController < ApplicationController
   def index
-    @delayed_jobs = Delayed::Job.order("created_at desc").paginate(params[:page])
+    @delayed_jobs = Delayed::Job.where("handler not like ? and handler not like ?", "%method_name: :update_related%", "%method_name: :process!%").order("created_at desc").paginate(params[:page])
   end
 end

app/models/post.rb

@@ -532,14 +532,14 @@ class Post < ActiveRecord::Base
     def add_pool!(pool)
       return if belongs_to_pool?(pool)
       self.pool_string = "#{pool_string} pool:#{pool.id}".strip
-      update_column(:pool_string, pool_string)
+      update_column(:pool_string, pool_string) unless new_record?
       pool.add!(self)
     end
     
     def remove_pool!(pool)
       return unless belongs_to_pool?(pool)
       self.pool_string = pool_string.gsub(/(?:\A| )pool:#{pool.id}(?:\Z| )/, " ").strip
-      update_column(:pool_string, pool_string)
+      update_column(:pool_string, pool_string) unless new_record?
       pool.remove!(self)
     end
   end

app/presenters/upload_presenter.rb

@@ -6,7 +6,7 @@ class UploadPresenter < Presenter
   def status(template)
     case @upload.status
     when /duplicate: (\d+)/
-      template.link_to(@upload.status, template.__send__(:post_path, $1))
+      template.link_to(@upload.status.gsub(/error: RuntimeError - /, ""), template.__send__(:post_path, $1))
       
     else
       @upload.status

app/views/delayed_jobs/index.html.erb

@@ -3,6 +3,7 @@
 <table class="striped" width="100%">
   <thead>
     <tr>
+      <th>ID</th>
       <% if CurrentUser.is_admin? %>
         <th>Handler</th>
       <% end %>
@@ -16,8 +17,9 @@
   <tbody>
     <% @delayed_jobs.each do |job| %>
       <tr>
+        <td><%= job.id %></td>
         <% if CurrentUser.is_admin? %>
-          <td><%= job.handler %></td>
+          <td><%= job.handler[:method_name] %></td>
         <% end %>
         <td><%= job.attempts %></td>
         <td><%= job.priority %></td>