Fixed form deletion when records are or are not dependent
This commit is contained in:
parent
cf759a4cc7
commit
0c7d31bac0
|
@ -15,7 +15,7 @@ class FormsController < ApplicationController
|
||||||
def create
|
def create
|
||||||
@form = Form.new(form_params)
|
@form = Form.new(form_params)
|
||||||
if @form.save
|
if @form.save
|
||||||
redirect_to @form, notice: 'Form was successfully created.'
|
redirect_to forms_path, notice: 'Form was successfully created.'
|
||||||
else
|
else
|
||||||
render :new
|
render :new
|
||||||
end
|
end
|
||||||
|
@ -33,10 +33,17 @@ class FormsController < ApplicationController
|
||||||
end
|
end
|
||||||
|
|
||||||
def destroy
|
def destroy
|
||||||
|
@form = Form.find(params[:id])
|
||||||
|
# Manually delete or handle associated records here
|
||||||
|
@form.onboarding_items.destroy_all
|
||||||
|
@form.forms_roles.destroy_all
|
||||||
@form.destroy
|
@form.destroy
|
||||||
redirect_to forms_url, notice: 'Form was successfully destroyed.'
|
redirect_to forms_url, notice: 'Form was successfully destroyed.'
|
||||||
|
rescue ActiveRecord::InvalidForeignKey
|
||||||
|
redirect_to forms_url, alert: 'Form could not be deleted. There are dependent records still associated.'
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
||||||
private
|
private
|
||||||
def set_form
|
def set_form
|
||||||
@form = Form.find(params[:id])
|
@form = Form.find(params[:id])
|
||||||
|
|
|
@ -73,6 +73,12 @@ class WorkersController < ApplicationController
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def onboarding
|
||||||
|
@worker = worker.find(params[:id])
|
||||||
|
@onboarding_items = @worker.onboarding_items.includes(:form)
|
||||||
|
end
|
||||||
|
|
||||||
|
|
||||||
private
|
private
|
||||||
def worker_params
|
def worker_params
|
||||||
params.require(:worker).permit(
|
params.require(:worker).permit(
|
||||||
|
|
|
@ -6,7 +6,6 @@ class Form < ApplicationRecord
|
||||||
accepts_nested_attributes_for :forms_roles, allow_destroy: true
|
accepts_nested_attributes_for :forms_roles, allow_destroy: true
|
||||||
|
|
||||||
validates :name, presence: true
|
validates :name, presence: true
|
||||||
validates :role, inclusion: { in: %w[Participant Worker Vendor Employer] }
|
|
||||||
# Include more validations as necessary
|
# Include more validations as necessary
|
||||||
|
|
||||||
after_save :update_onboarding_items
|
after_save :update_onboarding_items
|
||||||
|
|
|
@ -1,19 +1,30 @@
|
||||||
<h1><%= @form.name %></h1>
|
<div class="container mt-5">
|
||||||
|
<div class="row justify-content-center">
|
||||||
<p>
|
<div class="col-md-8">
|
||||||
|
<div class="card">
|
||||||
|
<div class="card-header">
|
||||||
|
<h1 class="card-title"><%= @form.name %></h1>
|
||||||
|
</div>
|
||||||
|
<div class="card-body">
|
||||||
|
<p class="card-text">
|
||||||
<strong>Required/Optional:</strong>
|
<strong>Required/Optional:</strong>
|
||||||
<%= @form.required ? 'Required' : 'Optional' %>
|
<%= @form.required ? 'Required' : 'Optional' %>
|
||||||
</p>
|
</p>
|
||||||
|
<p class="card-text">
|
||||||
<p>
|
|
||||||
<strong>Program:</strong>
|
<strong>Program:</strong>
|
||||||
<%= @form.program %>
|
<%= @form.program %>
|
||||||
</p>
|
</p>
|
||||||
|
<p class="card-text">
|
||||||
<p>
|
|
||||||
<strong>Who/Role:</strong>
|
<strong>Who/Role:</strong>
|
||||||
<%= @form.role %>
|
<%# Assuming you have adjusted @form.role to handle multiple roles %>
|
||||||
|
<%= @form.roles.map(&:name).join(', ') %>
|
||||||
</p>
|
</p>
|
||||||
|
</div>
|
||||||
<%= link_to 'Edit', edit_form_path(@form), class: 'btn btn-info' %> |
|
<div class="card-footer text-muted d-flex justify-content-between">
|
||||||
|
<%= link_to 'Edit', edit_form_path(@form), class: 'btn btn-info' %>
|
||||||
<%= link_to 'Back to Forms', forms_path, class: 'btn btn-secondary' %>
|
<%= link_to 'Back to Forms', forms_path, class: 'btn btn-secondary' %>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
|
Loading…
Reference in New Issue