Change the way we detect documents that were revoked but published an…

…yway to decouple the dependency on the previous issue (ensures edge cases in special filings or holidays aren’t an issue).

app/presenters/problematic_document_presenter.rb

@@ -29,15 +29,6 @@ def documents_scheduled_but_unpublished
   end
 
   def revoked_and_published_documents
-    previous_publication_date = issue.previous.publication_date
-
-    revoked_pi_numbers =
-      PublicInspectionIssue.
-      find_by_publication_date(previous_publication_date).
-      public_inspection_documents.
-      revoked.
-      map(&:document_number)
-
     published_doc_numbers =
       Entry.all(
         :conditions => {
@@ -46,9 +37,12 @@ def revoked_and_published_documents
       ).
       map(&:document_number)
 
-    revoked_pi_numbers.
-      select{|doc_number| published_doc_numbers.include? doc_number}.
-      map{|doc_number| PublicInspectionDocument.find_by_document_number(doc_number) }
+    PublicInspectionDocument.all(
+      :conditions => {
+        :document_number => published_doc_numbers,
+        :publication_date => nil
+      }
+    )
   end
 
   def documents_published_without_public_inspection