Add Placement Types management functionality

abachman · abachman · commit 50fb1b5fb4ba · 2025-09-12T14:26:44.000-04:00
- Introduced PlacementTypesController with actions for new, create, edit, and update. - Added routes for placement types. - Implemented views for managing placement types. - Created policies to manage access to placement types. - Added tests for placement types functionality in both request and policy specs. fixes #5778
diff --git a/app/controllers/casa_org_controller.rb b/app/controllers/casa_org_controller.rb
@@ -8,6 +8,7 @@ class CasaOrgController < ApplicationController
   before_action :set_sent_emails, only: %i[edit update]
   before_action :set_contact_topics, only: %i[edit update]
   before_action :set_custom_org_links, only: %i[edit update]
+  before_action :set_placement_types, only: %i[edit update]
   before_action :require_organization!
   after_action :verify_authorized
   before_action :set_active_storage_url_options, only: %i[edit update]
@@ -95,6 +96,10 @@ def set_custom_org_links
     @custom_org_links = @casa_org.custom_org_links
   end
 
+  def set_placement_types
+    @placement_types = @casa_org.placement_types
+  end
+
   def set_active_storage_url_options
     ActiveStorage::Current.url_options = {host: request.base_url}
   end
diff --git a/app/controllers/placement_types_controller.rb b/app/controllers/placement_types_controller.rb
@@ -0,0 +1,47 @@
+class PlacementTypesController < ApplicationController
+  before_action :set_placement_type, only: %i[edit update]
+  after_action :verify_authorized
+  after_action :verify_policy_scoped
+
+  def new
+    @placement_type = policy_scope(PlacementType).new(casa_org: current_organization)
+    authorize @placement_type
+  end
+
+  def create
+    @placement_type = policy_scope(PlacementType).new(placement_type_params)
+    authorize @placement_type
+
+    if @placement_type.save
+      redirect_to edit_casa_org_path(current_organization), notice: "Placement Type was successfully created."
+    else
+      render :new, status: :unprocessable_entity
+    end
+  end
+
+  def edit
+    authorize @placement_type
+  end
+
+  def update
+    authorize @placement_type
+
+    if @placement_type.update(placement_type_params)
+      redirect_to edit_casa_org_path(current_organization), notice: "Placement Type was successfully updated."
+    else
+      render :edit, status: :unprocessable_entity
+    end
+  end
+
+  private
+
+  def set_placement_type
+    @placement_type = policy_scope(PlacementType).find(params[:id])
+  rescue ActiveRecord::RecordNotFound
+    redirect_to edit_casa_org_path
+  end
+
+  def placement_type_params
+    params.require(:placement_type).permit(:name).merge(casa_org: current_organization)
+  end
+end
diff --git a/app/policies/placement_type_policy.rb b/app/policies/placement_type_policy.rb
@@ -0,0 +1,22 @@
+class PlacementTypePolicy < ApplicationPolicy
+  class Scope < ApplicationPolicy::Scope
+    def resolve
+      case user
+      when CasaAdmin
+        scope.where(casa_org: @user.casa_org)
+      when Volunteer, Supervisor
+        scope.none
+      else
+        scope.none
+      end
+    end
+  end
+
+  def edit?
+    is_admin_same_org?
+  end
+
+  alias_method :new?, :edit?
+  alias_method :create?, :edit?
+  alias_method :update?, :edit?
+end
diff --git a/app/views/casa_org/_placement_types.html.erb b/app/views/casa_org/_placement_types.html.erb
@@ -0,0 +1,49 @@
+<div class="row">
+  <div class="col-lg-12">
+    <div class="card-style mb-30">
+      <div class="row align-items-center">
+        <div class="col-md-6">
+          <h3>Placement Types</h3>
+        </div>
+        <div class="col-md-6">
+          <div class="breadcrumb-wrapper">
+            <span class="ml-5">
+              <%= link_to new_placement_type_path, class: "btn-sm main-btn primary-btn btn-hover" do %>
+                <i class="lni lni-plus mr-10"></i>
+                New Placement Type
+              <% end %>
+            </span>
+          </div>
+        </div>
+      </div>
+      <div class="table-wrapper table-responsive">
+        <table class="table striped-table" id="contact-types">
+          <thead>
+            <tr>
+              <th>Name</th>
+              <th>Actions</th>
+            </tr>
+          </thead>
+          <tbody>
+            <% placement_types.each do |placement_type| %>
+              <tr id="placement_type-<%= placement_type.id %>">
+                <td scope="row" class="min-width">
+                  <%= placement_type.name %>
+                </td>
+                <td>
+                  <%= link_to edit_placement_type_path(placement_type) do %>
+                    <div class="action">
+                      <button class="text-danger">
+                        <i class="lni lni-pencil-alt"></i> Edit
+                      </button>
+                    </div>
+                  <% end %>
+                </td>
+              </tr>
+            <% end %>
+          </tbody>
+        </table>
+      </div>
+    </div>
+  </div>
+</div>
diff --git a/app/views/casa_org/edit.html.erb b/app/views/casa_org/edit.html.erb
@@ -166,3 +166,18 @@
 <div class="tables-wrapper">
   <%= render "contact_topics" %>
 </div>
+
+<div class="title-wrapper pt-30">
+  <div class="row align-items-center">
+    <div class="col-md-6">
+      <div class="title mb-30">
+        <h2 id="placement-types">
+          Manage Placement Types
+        </h2>
+      </div>
+    </div>
+  </div>
+</div>
+<div class="tables-wrapper">
+  <%= render "placement_types", placement_types: @placement_types %>
+</div>
diff --git a/app/views/layouts/_sidebar.html.erb b/app/views/layouts/_sidebar.html.erb
@@ -57,7 +57,8 @@
               { title: "Contact Types", icon: "user", path: edit_casa_org_path(current_organization, anchor: 'contact-types'), render_check: policy(:application).modify_organization? },
               { title: "Court Details", icon: "library", path: edit_casa_org_path(current_organization, anchor: 'court-details'), render_check: policy(:application).modify_organization? },
               { title: "Learning Hours", icon: "timer", path: edit_casa_org_path(current_organization, anchor: 'learning-hours'), render_check: policy(:application).modify_organization? },
-              { title: "Case Contact Topics", icon: "comments", path: edit_casa_org_path(current_organization, anchor: 'case-contact-topics'), render_check: policy(:application).modify_organization? }
+              { title: "Case Contact Topics", icon: "comments", path: edit_casa_org_path(current_organization, anchor: 'case-contact-topics'), render_check: policy(:application).modify_organization? },
+              { title: "Placement Types", icon: "comments", path: edit_casa_org_path(current_organization, anchor: 'placement-types'), render_check: policy(:application).modify_organization? }
             ]) %>
           <% end %>
           <% if current_organization.custom_org_links.any?(&:active) %>
diff --git a/app/views/placement_types/_fields.html.erb b/app/views/placement_types/_fields.html.erb
@@ -0,0 +1,18 @@
+<div class="input-style-1">
+  <%= form.label :placement_started_at, "Placement Started At" %>
+  <%= form.date_field :placement_started_at,
+                      value: placement.placement_started_at&.to_date || Time.zone.now,
+                      class: "form-control" %>
+</div>
+<div class="select-style-1">
+  <%= form.label :placement_type_id, "Placement Type" %>
+  <div class="select-position">
+    <%= form.collection_select(
+          :placement_type_id,
+          PlacementType.for_organization(current_organization).order_alphabetically,
+          :id, :name,
+          {include_hidden: false, include_blank: "-Select Placement Type-"},
+          {class: "form-control"}
+        ) %>
+  </div>
+</div>
diff --git a/app/views/placement_types/_form.html.erb b/app/views/placement_types/_form.html.erb
@@ -0,0 +1,31 @@
+<div class="title-wrapper pt-30">
+  <div class="row align-items-center">
+    <div class="col-md-6">
+      <div class="title mb-30">
+        <h1>
+          <%= title %>
+        </h1>
+      </div>
+    </div>
+  </div>
+</div><!-- ==== end title ==== -->
+
+<!-- ========== card start ========== -->
+<div class="card-style mb-30">
+  <%= form_with(model: placement_type, local: true) do |form| %>
+    <div class="alert-box danger-alert">
+      <%= render "/shared/error_messages", resource: placement_type %>
+    </div>
+    <div class="input-style-1">
+      <%= form.label :name, "Name" %>
+      <%= form.text_field :name, class: "form-control", required: true %>
+    </div>
+
+    <div class="actions mb-10">
+      <%= button_tag(type: "submit", class: "btn-sm main-btn primary-btn btn-hover") do %>
+        <i class="lni lni-checkmark-circle mr-5"></i> Submit
+      <% end %>
+    </div>
+  <% end %>
+</div>
+<!-- card end -->
diff --git a/app/views/placement_types/edit.html.erb b/app/views/placement_types/edit.html.erb
@@ -0,0 +1 @@
+<%= render partial: "form", locals: {title: "Edit Placement Type", placement_type: @placement_type} %>
diff --git a/app/views/placement_types/new.html.erb b/app/views/placement_types/new.html.erb
@@ -0,0 +1 @@
+<%= render partial: "form", locals: {title: "New Placement Type", placement_type: @placement_type} %>
diff --git a/config/routes.rb b/config/routes.rb
@@ -142,6 +142,7 @@
   resources :learning_hours_reports, only: %i[index]
   resources :learning_hour_types, only: %i[new create edit update]
   resources :learning_hour_topics, only: %i[new create edit update]
+  resources :placement_types, only: %i[new create edit update]
 
   resources :contact_topics, except: %i[index show delete] do
     delete "soft_delete", on: :member
diff --git a/spec/policies/placement_type_policy_spec.rb b/spec/policies/placement_type_policy_spec.rb
@@ -0,0 +1,40 @@
+require "rails_helper"
+
+RSpec.describe PlacementTypePolicy, type: :policy do
+  let(:casa_org) { create :casa_org }
+  let(:volunteer) { create :volunteer, casa_org: }
+  let(:supervisor) { create :supervisor, casa_org: }
+  let(:casa_admin) { create :casa_admin, casa_org: }
+  let(:all_casa_admin) { create :all_casa_admin }
+
+  let(:placement_type) { create :placement_type, casa_org: }
+
+  subject { described_class }
+
+  permissions :edit?, :new?, :update?, :create? do
+    it "does not permit a nil user" do
+      expect(described_class).not_to permit(nil, placement_type)
+    end
+
+    it "does not permit a volunteer" do
+      expect(described_class).not_to permit(volunteer, placement_type)
+    end
+
+    it "does not permit a supervisor" do
+      expect(described_class).not_to permit(supervisor, placement_type)
+    end
+
+    it "permits a casa admin" do
+      expect(described_class).to permit(casa_admin, placement_type)
+    end
+
+    it "does not permit a casa admin for a different casa org" do
+      other_org_casa_admin = create :casa_admin, casa_org: create(:casa_org)
+      expect(described_class).not_to permit(other_org_casa_admin, placement_type)
+    end
+
+    it "does not permit an all casa admin" do
+      expect(described_class).not_to permit(all_casa_admin, placement_type)
+    end
+  end
+end
diff --git a/spec/requests/placement_types_spec.rb b/spec/requests/placement_types_spec.rb
@@ -0,0 +1,44 @@
+require "rails_helper"
+
+RSpec.describe "PlacementTypes", type: :request do
+  let(:organization) { create(:casa_org) }
+  let(:admin) { create(:casa_admin, casa_org: organization) }
+  let(:placement_type) { create(:placement_type, casa_org: organization) }
+
+  describe "as an admin" do
+    before do
+      sign_in admin
+    end
+
+    describe "GET /edit" do
+      it "returns http success" do
+        get edit_placement_type_path(placement_type)
+        expect(response).to have_http_status(:success)
+      end
+    end
+
+    describe "GET /new" do
+      it "returns http success" do
+        get "/placement_types/new"
+        expect(response).to have_http_status(:success)
+      end
+    end
+
+    describe "POST /create" do
+      it "returns http success" do
+        expect {
+          post "/placement_types", params: {placement_type: {name: "New Placement Type"}}
+        }.to change(PlacementType, :count).by(1)
+        expect(response).to redirect_to(edit_casa_org_path(organization))
+      end
+    end
+
+    describe "PATCH /update" do
+      it "returns http success" do
+        patch placement_type_path(placement_type), params: {placement_type: {name: "Updated Placement Type"}}
+        expect(response).to redirect_to(edit_casa_org_path(organization))
+        expect(placement_type.reload.name).to eq("Updated Placement Type")
+      end
+    end
+  end
+end
diff --git a/spec/views/placement_types/edit.html.erb_spec.rb b/spec/views/placement_types/edit.html.erb_spec.rb
@@ -0,0 +1,22 @@
+require "rails_helper"
+
+RSpec.describe "placement_types/edit.html.erb", type: :view do
+  let(:organization) { build_stubbed :casa_org }
+  let(:admin) { build_stubbed :casa_admin, casa_org: organization }
+  let(:placement_type) { build_stubbed :placement_type, casa_org: organization }
+
+  before do
+    enable_pundit(view, admin)
+    allow(view).to receive(:current_user).and_return(admin)
+    allow(view).to receive(:current_organization).and_return(organization)
+    sign_in admin
+  end
+
+  it "allows editing the placement type" do
+    assign :placement_type, placement_type
+    render template: "placement_types/edit"
+    expect(rendered).to have_text("Edit Placement Type")
+    expect(rendered).to have_field("placement_type[name]", with: placement_type.name)
+    expect(rendered).to have_button("Submit")
+  end
+end
diff --git a/spec/views/placement_types/new.html.erb_spec.rb b/spec/views/placement_types/new.html.erb_spec.rb
@@ -0,0 +1,22 @@
+require "rails_helper"
+
+RSpec.describe "placement_types/new.html.erb", type: :view do
+  let(:organization) { build_stubbed :casa_org }
+  let(:admin) { build_stubbed :casa_admin, casa_org: organization }
+  let(:placement_type) { organization.placement_types.new }
+
+  before do
+    enable_pundit(view, admin)
+    allow(view).to receive(:current_user).and_return(admin)
+    allow(view).to receive(:current_organization).and_return(organization)
+    sign_in admin
+  end
+
+  it "allows creating a placement type" do
+    assign :placement_type, placement_type
+    render template: "placement_types/new"
+    expect(rendered).to have_text("New Placement Type")
+    expect(rendered).to have_field("placement_type[name]")
+    expect(rendered).to have_button("Submit")
+  end
+end

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+<%= render partial: "form", locals: {title: "Edit Placement Type", placement_type: @placement_type} %>`