Move logic to the view

ELIXIR-Belgium · Jul 22, 2024 · 6b038ed · 6b038ed
1 parent c3c225a
commit 6b038ed
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/app/controllers/investigations_controller.rb b/app/controllers/investigations_controller.rb
@@ -72,11 +72,6 @@ def ro_for_download
     headers["Content-Length"]=ro_file.size.to_s
   end
 
-  def new
-    @investigation = Investigation.new
-    @investigation.is_isa_json_compliant = Seek::Config.isa_json_compliance_enabled ? true : false
-    respond_to(&:html)
-  end
   def create
     @investigation = Investigation.new(investigation_params)
     update_sharing_policies @investigation

diff --git a/app/views/investigations/_form.html.erb b/app/views/investigations/_form.html.erb
@@ -1,4 +1,5 @@
 <% object ||= nil %>
+<% isa_json_compliance_default = @investigation.new_record? ? Seek::Config.isa_json_compliance_enabled : @investigation.is_isa_json_compliant %>
 
 <%= f.error_messages -%>
 
@@ -26,7 +27,7 @@
       <% if @investigation.studies.any? %>
         <%= f.check_box :is_isa_json_compliant, disabled: "disabled"%>
       <% else %>
-        <%= f.check_box :is_isa_json_compliant%>
+        <%= f.check_box :is_isa_json_compliant, checked: isa_json_compliance_default %>
       <% end %>
       <%= "Make #{t('investigation')} compliant to ISA-JSON schemas?" %>
     </label>