diff --git a/app/controllers/case_logs_controller.rb b/app/controllers/case_logs_controller.rb index ca9804056..34e257fb3 100644 --- a/app/controllers/case_logs_controller.rb +++ b/app/controllers/case_logs_controller.rb @@ -7,7 +7,7 @@ class CaseLogsController < ApplicationController before_action :find_resource, except: %i[create index edit] def index - set_session_filters if params[:status].present? + set_session_filters @pagy, @case_logs = pagy(filtered_case_logs) @@ -121,14 +121,21 @@ private end def filtered_case_logs - user_case_logs = current_user.case_logs - status_filter = JSON.parse(session[:case_logs_filters])["status"] if session[:case_logs_filters].present? - return user_case_logs unless status_filter + query = CaseLog.for_organisation(current_user.organisation) + if session[:case_logs_filters].present? + filters = JSON.parse(session[:case_logs_filters]) + filters.each do |category, values| + next if values.reject(&:empty?).blank? - user_case_logs.filter_by_status(status_filter) + query = query.public_send("filter_by_#{category}", values, current_user) + end + end + query.all.includes(:owning_organisation, :managing_organisation) end def set_session_filters - session[:case_logs_filters] = { status: params[:status] }.to_json + new_filters = session[:case_logs_filters].present? ? JSON.parse(session[:case_logs_filters]) : {} + %i[status years user].each { |filter| new_filters[filter] = params[filter] if params[filter].present? } + session[:case_logs_filters] = new_filters.to_json end end diff --git a/app/helpers/filters_helper.rb b/app/helpers/filters_helper.rb index aaee74fb6..f448bbeb5 100644 --- a/app/helpers/filters_helper.rb +++ b/app/helpers/filters_helper.rb @@ -1,9 +1,11 @@ module FiltersHelper - def filter_selected?(filter) - return true unless session[:case_logs_filters] + def filter_selected?(filter, value) + return false unless session[:case_logs_filters] selected_filters = JSON.parse(session[:case_logs_filters]) - selected_filters["status"].present? && selected_filters["status"].include?(filter.to_s) + return false if selected_filters[filter].blank? + + selected_filters[filter].include?(value.to_s) end def status_filters diff --git a/app/models/case_log.rb b/app/models/case_log.rb index 2425e347f..3dee49aa9 100644 --- a/app/models/case_log.rb +++ b/app/models/case_log.rb @@ -34,7 +34,20 @@ class CaseLog < ApplicationRecord belongs_to :managing_organisation, class_name: "Organisation" scope :for_organisation, ->(org) { where(owning_organisation: org).or(where(managing_organisation: org)) } - scope :filter_by_status, ->(status) { where status: status } + scope :filter_by_status, ->(status, _user = nil) { where status: status } + scope :filter_by_years, lambda { |years, _user = nil| + first_year = years.shift + query = filter_by_year(first_year) + years.each { |year| query = query.or(filter_by_year(year)) } + query.all + } + scope :filter_by_year, ->(year) { where(startdate: Time.utc(year.to_i, 4, 1)...Time.utc(year.to_i + 1, 4, 1)) } + + scope :filter_by_user, lambda { |selected_user, user| + if !selected_user.include?("all") && user.present? + where(id: PaperTrail::Version.where(item_type: "CaseLog", event: "create", whodunnit: user.to_global_id.uri.to_s).map(&:item_id)) + end + } AUTOGENERATED_FIELDS = %w[id status created_at updated_at discarded_at].freeze OPTIONAL_FIELDS = %w[postcode_known la_known first_time_property_let_as_social_housing tenant_code propcode].freeze @@ -409,7 +422,6 @@ private end self.nocharge = household_charge&.zero? ? 1 : 0 self.underoccupation_benefitcap = 3 if renewal == 1 && collection_start_year == 2021 - self.ethnic = ethnic || ethnic_group self.housingneeds = get_housingneeds if is_renewal? self.underoccupation_benefitcap = 2 if collection_start_year == 2021 diff --git a/app/views/case_logs/_log_filters.erb b/app/views/case_logs/_log_filters.erb index 4e5593a62..888028700 100644 --- a/app/views/case_logs/_log_filters.erb +++ b/app/views/case_logs/_log_filters.erb @@ -2,9 +2,9 @@

Filters

-