fix(dossier): should refresh dossier footer when removing options from multiselect
This commit is contained in:
parent
dd85a495e0
commit
3a54e44b57
8 changed files with 28 additions and 13 deletions
|
@ -5,8 +5,9 @@ class Champs::OptionsController < ApplicationController
|
|||
|
||||
def remove
|
||||
champ = policy_scope(Champ).includes(:champs).find(params[:champ_id])
|
||||
champ.remove_option([params[:option]].compact)
|
||||
champ.remove_option([params[:option]].compact, true)
|
||||
champs = champ.private? ? champ.dossier.champs_private_all : champ.dossier.champs_public_all
|
||||
@dossier = champ.private? ? nil : champ.dossier
|
||||
@to_show, @to_hide, @to_update = champs_to_turbo_update({ params[:champ_id] => true }, champs)
|
||||
end
|
||||
end
|
||||
|
|
|
@ -62,8 +62,12 @@ class Champs::DropDownListChamp < Champ
|
|||
options.include?(value)
|
||||
end
|
||||
|
||||
def remove_option(options)
|
||||
update_column(:value, nil)
|
||||
def remove_option(options, touch = false)
|
||||
if touch
|
||||
update(value: nil)
|
||||
else
|
||||
update_column(:value, nil)
|
||||
end
|
||||
end
|
||||
|
||||
private
|
||||
|
|
|
@ -70,8 +70,12 @@ class Champs::LinkedDropDownListChamp < Champ
|
|||
options.include?(primary_value) || options.include?(secondary_value)
|
||||
end
|
||||
|
||||
def remove_option(options)
|
||||
update_column(:value, nil)
|
||||
def remove_option(options, touch = false)
|
||||
if touch
|
||||
update(value: nil)
|
||||
else
|
||||
update_column(:value, nil)
|
||||
end
|
||||
end
|
||||
|
||||
private
|
||||
|
|
|
@ -51,8 +51,13 @@ class Champs::MultipleDropDownListChamp < Champ
|
|||
(selected_options - options).size != selected_options.size
|
||||
end
|
||||
|
||||
def remove_option(options)
|
||||
update_column(:value, (selected_options - options).to_json)
|
||||
def remove_option(options, touch = false)
|
||||
value = (selected_options - options).to_json
|
||||
if touch
|
||||
update(value:)
|
||||
else
|
||||
update_columns(value:)
|
||||
end
|
||||
end
|
||||
|
||||
def focusable_input_id
|
||||
|
|
|
@ -1 +1 @@
|
|||
= render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update }
|
||||
= render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update, dossier: @dossier }
|
||||
|
|
|
@ -1 +1 @@
|
|||
= render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update }
|
||||
= render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update, dossier: nil }
|
||||
|
|
|
@ -13,3 +13,7 @@
|
|||
|
||||
= turbo_stream.remove_all(".editable-champ .spinner-removable")
|
||||
= turbo_stream.hide_all(".editable-champ .spinner")
|
||||
|
||||
- if dossier.present?
|
||||
= turbo_stream.replace_all '.dossier-edit-sticky-footer' do
|
||||
= render Dossiers::EditFooterComponent.new(dossier:, annotation: false)
|
||||
|
|
|
@ -1,4 +1 @@
|
|||
= render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update, dossier: @dossier, annotation: false }
|
||||
|
||||
= turbo_stream.replace_all '.dossier-edit-sticky-footer' do
|
||||
= render Dossiers::EditFooterComponent.new(dossier: @dossier, annotation: false)
|
||||
= render partial: 'shared/dossiers/update_champs', locals: { to_show: @to_show, to_hide: @to_hide, to_update: @to_update, dossier: @dossier }
|
||||
|
|
Loading…
Reference in a new issue