sales logs are now working

app/controllers/form_controller.rb

@@ -131,20 +131,24 @@ private
     end
   end
 
-
   def find_resource
-    @log = if new_log_request? || new_log_request_referrer?
+    @log = if params.key?("sales_log")
+             if new_log_request? || new_log_request_referrer?
+               SalesLog.new(owning_organisation: current_user.support? ? nil : current_user.organisation)
+             else
+               current_user.sales_logs.find_by(id: params[:id])
+             end
+           elsif new_log_request? || new_log_request_referrer?
              LettingsLog.new(owning_organisation: current_user.support? ? nil : current_user.organisation)
            else
-             params.key?("sales_log") ? current_user.sales_logs.find_by(id: params[:id]) : current_user.lettings_logs.find_by(id: params[:id])
+             current_user.lettings_logs.find_by(id: params[:id])
            end
   end
 
   def find_resource_by_named_id
     @log = if params[:sales_log_id].present?
              current_user.sales_logs.find_by(id: params[:sales_log_id])
-           else
-             if new_log_request?
+           elsif new_log_request?
              if request.path.include?("sales-logs")
                SalesLog.new
              else
@@ -154,7 +158,6 @@ private
              current_user.lettings_logs.find_by(id: params[:lettings_log_id])
            end
   end
-  end
 
   def new_log_request?
     request.path.split("/").include?("new")

app/helpers/tasklist_helper.rb

@@ -20,7 +20,7 @@ module TasklistHelper
     if log.id
       send(path, log)
     else
-      "/#{log.lettings? ? "lettings" : "sales"}-logs/new/#{next_question_page(subsection, log, current_user)}"
+      "/#{log.lettings? ? 'lettings' : 'sales'}-logs/new/#{next_question_page(subsection, log, current_user)}"
     end
   end