diff --git a/app/controllers/folders_controller.rb b/app/controllers/folders_controller.rb
index 209fb60e..77d44ee5 100644
--- a/app/controllers/folders_controller.rb
+++ b/app/controllers/folders_controller.rb
@@ -17,7 +17,8 @@ class FoldersController < ApplicationController
 
   def destroy
     # Relocate all documents in this folder to the root "folder"
-    Document.where(folder_id: @folder.id).update_all(folder_id: nil)
+    # TODO - I think we can handle this at the model association level with dependent: nullify, but I've never used it
+    Document.with_deleted.where(folder_id: @folder.id).update_all(folder_id: nil)
 
     # Relocate all child folders in this folder to the root "folder"
     Folder.where(parent_folder_id: @folder.id).update_all(parent_folder_id: nil)
diff --git a/app/controllers/page_collection_submissions_controller.rb b/app/controllers/page_collection_submissions_controller.rb
index 99a17aed..7dab6782 100644
--- a/app/controllers/page_collection_submissions_controller.rb
+++ b/app/controllers/page_collection_submissions_controller.rb
@@ -2,7 +2,7 @@ class PageCollectionSubmissionsController < ApplicationController
   before_action :set_page_collection, only: [:index]
   before_action :set_page_collection_submission, only: [:show, :edit, :update, :destroy, :approve, :pass]
 
-  before_action :require_collection_ownership, only: [:index, :edit, :update, :destroy, :pass, :approve]
+  before_action :require_collection_ownership, only: [:index]
 
   # GET /page_collection_submissions
   def index
diff --git a/app/controllers/share_comments_controller.rb b/app/controllers/share_comments_controller.rb
index 980d7248..9e5cd209 100644
--- a/app/controllers/share_comments_controller.rb
+++ b/app/controllers/share_comments_controller.rb
@@ -1,4 +1,6 @@
 class ShareCommentsController < ApplicationController
+  before_action :authenticate_user!
+
   before_action :set_share_comment, only: [:update, :destroy]
 
   # POST /share_comments
diff --git a/app/models/folder.rb b/app/models/folder.rb
index 406adbb0..de82df8b 100644
--- a/app/models/folder.rb
+++ b/app/models/folder.rb
@@ -1,4 +1,6 @@
 class Folder < ApplicationRecord
+  has_many :documents
+
   belongs_to :parent_folder, optional: true, class_name: Folder.name, foreign_key: :parent_folder_id
   belongs_to :user