minor refactor for readability (#1330)

app/models/form/subsection.rb

@@ -35,6 +35,10 @@ class Form::Subsection
     :in_progress
   end
 
+  def complete?(log)
+    status(log) == :completed
+  end
+
   def is_incomplete?(log)
     %i[not_started in_progress].include?(status(log))
   end
@@ -58,4 +62,8 @@ class Form::Subsection
       end
     end
   end
+
+  def not_displayed_in_tasklist?(log)
+    !displayed_in_tasklist?(log)
+  end
 end

app/models/log.rb

@@ -130,14 +130,12 @@ private
   end
 
   def all_fields_completed?
-    subsection_statuses = form.subsections.map { |subsection| subsection.status(self) if subsection.displayed_in_tasklist?(self) }.uniq.compact
+    form.subsections.all? { |subsection| subsection.complete?(self) || subsection.not_displayed_in_tasklist?(self) }
-    subsection_statuses == [:completed]
   end
 
   def all_fields_nil?
     not_started_statuses = %i[not_started cannot_start_yet]
-    subsection_statuses = form.subsections.map { |subsection| subsection.status(self) }.uniq
+    form.subsections.all? { |subsection| not_started_statuses.include? subsection.status(self) }
-    subsection_statuses.all? { |status| not_started_statuses.include?(status) }
   end
 
   def reset_invalidated_dependent_fields!