Handle accidental subject deletion and error handling for #4269

app/controllers/transcribe_controller.rb

@@ -615,8 +615,16 @@ def record_translation_deed
   private
 
   def rollback_article_categories(destroy_ids, unset_ids)
-    Article.where(id: destroy_ids, provenance: nil).destroy_all if destroy_ids
     Article.where(id: unset_ids).update(categories: []) if unset_ids
+
+    if destroy_ids
+      candidate_articles = Article.where(id: destroy_ids, provenance: nil)
+      candidate_articles.each do |candidate|
+        if candidate.pages.count == 1 # don't delete uncategorized subjects that appear elsewhere
+          candidate.destroy!
+        end
+      end
+    end
   end
 
   def page_params

lib/tasks/remediator.rake

@@ -69,11 +69,18 @@ namespace :fromthepage do
       pages = pages.uniq
       # for each page, resave the page so we get updated page source and page article links
       pages.each do |page|
-        print "\nResaving page #{page.title}\n"
+        print "\nResaving page #{page.id}\t#{page.title}\n"
         # print the page_article_link count before and after
         print "Before: #{page.page_article_links.count} links\n"
         page.source_text+=' '
-        page.save!
+        begin
+          page.save!
+          # catch any exceptions and print them
+        rescue => e
+          print "\nERROR SAVING PAGE #{page.id}!!!\n\n"
+          print e.message
+          print "\nERROR SAVING PAGE #{page.id}!!!\n\n"
+        end
         print "After: #{page.page_article_links.count} links\n"
       end
     end