Revert "Merge pull request #7907 from tchak/feat-cond-repetitions"

This reverts commit 1900f8ff4c, reversing
changes made to febffc2d64.

# Conflicts:
#	app/models/dossier.rb
This commit is contained in:
Paul Chavard 2022-12-14 11:25:45 +01:00
parent b740c75f09
commit 0d10d36cb0
11 changed files with 144 additions and 229 deletions

View file

@ -1,8 +1,7 @@
class TypesDeChampEditor::BlockComponent < ApplicationComponent class TypesDeChampEditor::BlockComponent < ApplicationComponent
def initialize(block:, coordinates:, upper_coordinates: []) def initialize(block:, coordinates:)
@block = block @block = block
@coordinates = coordinates @coordinates = coordinates
@upper_coordinates = upper_coordinates
end end
private private

View file

@ -1,3 +1,3 @@
%ul.types-de-champ-block{ id: block_id, data: sortable_options } %ul.types-de-champ-block{ id: block_id, data: sortable_options }
- @coordinates.each.with_index do |coordinate, i| - @coordinates.each.with_index do |coordinate, i|
= render TypesDeChampEditor::ChampComponent.new(coordinate: coordinate, upper_coordinates: @upper_coordinates + @coordinates.take(i)) = render TypesDeChampEditor::ChampComponent.new(coordinate: coordinate, upper_coordinates: @coordinates.take(i))

View file

@ -118,6 +118,6 @@ class TypesDeChampEditor::ChampComponent < ApplicationComponent
end end
def conditional_enabled? def conditional_enabled?
!type_de_champ.private? !type_de_champ.private? && !coordinate.child?
end end
end end

View file

@ -87,7 +87,7 @@
- if type_de_champ.block? - if type_de_champ.block?
.flex.justify-start.section.ml-1 .flex.justify-start.section.ml-1
.editor-block.flex-grow.cell .editor-block.flex-grow.cell
= render TypesDeChampEditor::BlockComponent.new(block: coordinate, coordinates: coordinate.revision_types_de_champ, upper_coordinates: @upper_coordinates) = render TypesDeChampEditor::BlockComponent.new(block: coordinate, coordinates: coordinate.revision_types_de_champ)
= render TypesDeChampEditor::AddChampButtonComponent.new(revision: coordinate.revision, parent: coordinate, is_annotation: coordinate.private?) = render TypesDeChampEditor::AddChampButtonComponent.new(revision: coordinate.revision, parent: coordinate, is_annotation: coordinate.private?)
- if conditional_enabled? - if conditional_enabled?

View file

@ -55,13 +55,7 @@ module Administrateurs
def retrieve_coordinate_and_uppers def retrieve_coordinate_and_uppers
@tdc = draft_revision.find_and_ensure_exclusive_use(params[:stable_id]) @tdc = draft_revision.find_and_ensure_exclusive_use(params[:stable_id])
@coordinate = draft_revision.coordinate_for(@tdc) @coordinate = draft_revision.coordinate_for(@tdc)
@upper_tdcs = @coordinate.upper_siblings.map(&:type_de_champ)
upper_coordinates = @coordinate.upper_siblings
if @coordinate.child?
upper_coordinates += @coordinate.parent.upper_siblings
end
@upper_tdcs = upper_coordinates.map(&:type_de_champ)
end end
def draft_revision def draft_revision

View file

@ -200,7 +200,7 @@ module Users
respond_to do |format| respond_to do |format|
format.html { render :brouillon } format.html { render :brouillon }
format.turbo_stream do format.turbo_stream do
@to_shows, @to_hides = @dossier.champs_public_all @to_shows, @to_hides = @dossier.champs_public
.filter(&:conditional?) .filter(&:conditional?)
.partition(&:visible?) .partition(&:visible?)
.map { |champs| champs_to_one_selector(champs) } .map { |champs| champs_to_one_selector(champs) }
@ -221,7 +221,7 @@ module Users
respond_to do |format| respond_to do |format|
format.html { render :modifier } format.html { render :modifier }
format.turbo_stream do format.turbo_stream do
@to_shows, @to_hides = @dossier.champs_public_all @to_shows, @to_hides = @dossier.champs_public
.filter(&:conditional?) .filter(&:conditional?)
.partition(&:visible?) .partition(&:visible?)
.map { |champs| champs_to_one_selector(champs) } .map { |champs| champs_to_one_selector(champs) }

View file

@ -239,7 +239,7 @@ class Champ < ApplicationRecord
private private
def champs_for_condition def champs_for_condition
dossier.champs.filter { _1.row.nil? || _1.row == row } private? ? dossier.champs_private : dossier.champs_public
end end
def html_id def html_id

View file

@ -79,10 +79,8 @@ class Dossier < ApplicationRecord
has_one_attached :justificatif_motivation has_one_attached :justificatif_motivation
has_many :champs
has_many :champs_public, -> { root.public_ordered }, class_name: 'Champ', inverse_of: false, dependent: :destroy has_many :champs_public, -> { root.public_ordered }, class_name: 'Champ', inverse_of: false, dependent: :destroy
has_many :champs_private, -> { root.private_ordered }, class_name: 'Champ', inverse_of: false, dependent: :destroy has_many :champs_private, -> { root.private_ordered }, class_name: 'Champ', inverse_of: false, dependent: :destroy
has_many :champs_public_all, -> { public_only }, class_name: 'Champ', inverse_of: false
has_many :prefilled_champs_public, -> { root.public_only.prefilled }, class_name: 'Champ', inverse_of: false, dependent: :destroy has_many :prefilled_champs_public, -> { root.public_only.prefilled }, class_name: 'Champ', inverse_of: false, dependent: :destroy
has_many :commentaires, inverse_of: :dossier, dependent: :destroy has_many :commentaires, inverse_of: :dossier, dependent: :destroy
has_many :invites, dependent: :destroy has_many :invites, dependent: :destroy

View file

@ -78,7 +78,6 @@ class DossierPreloader
def load_dossier(dossier, champs, children_by_parent = {}) def load_dossier(dossier, champs, children_by_parent = {})
champs_public, champs_private = champs.partition(&:public?) champs_public, champs_private = champs.partition(&:public?)
dossier.association(:champs).target = []
load_champs(dossier, :champs_public, champs_public, dossier, children_by_parent) load_champs(dossier, :champs_public, champs_public, dossier, children_by_parent)
load_champs(dossier, :champs_private, champs_private, dossier, children_by_parent) load_champs(dossier, :champs_private, champs_private, dossier, children_by_parent)
@ -98,7 +97,6 @@ class DossierPreloader
champ.association(:parent).target = parent champ.association(:parent).target = parent
end end
end end
dossier.association(:champs).target += champs
parent.association(name).target = champs.sort_by do |champ| parent.association(name).target = champs.sort_by do |champ|
[champ.row, positions[dossier.revision_id][champ.type_de_champ_id]] [champ.row, positions[dossier.revision_id][champ.type_de_champ_id]]

View file

@ -1,10 +1,10 @@
describe Administrateurs::ConditionsController, type: :controller do describe Administrateurs::ConditionsController, type: :controller do
include Logic include Logic
before { sign_in(procedure.administrateurs.first.user) }
context 'without bloc repetition' do
let(:procedure) { create(:procedure, types_de_champ_public: [{ type: :integer_number }] * 3) } let(:procedure) { create(:procedure, types_de_champ_public: [{ type: :integer_number }] * 3) }
let(:first_coordinate) { procedure.draft_revision.revision_types_de_champ.first }
let(:second_coordinate) { procedure.draft_revision.revision_types_de_champ.first }
let(:third_coordinate) { procedure.draft_revision.revision_types_de_champ.first }
let(:first_tdc) { procedure.draft_revision.types_de_champ.first } let(:first_tdc) { procedure.draft_revision.types_de_champ.first }
let(:second_tdc) { procedure.draft_revision.types_de_champ.second } let(:second_tdc) { procedure.draft_revision.types_de_champ.second }
let(:third_tdc) { procedure.draft_revision.types_de_champ.third } let(:third_tdc) { procedure.draft_revision.types_de_champ.third }
@ -116,44 +116,3 @@ describe Administrateurs::ConditionsController, type: :controller do
end end
end end
end end
context 'with a repetiton bloc' do
let(:procedure) do
create(:procedure, types_de_champ_public: [
{ type: :integer_number, libelle: 'top_1' },
{
type: :repetition,
libelle: 'repetition',
children: [
{ type: :integer_number, libelle: 'child_1' },
{ type: :integer_number, libelle: 'child_2' }
]
}
])
end
let(:tdcs) { procedure.draft_revision.types_de_champ }
let(:top) { tdcs.find_by(libelle: 'top_1') }
let(:repetition) { tdcs.find_by(libelle: 'repetition') }
let(:child_1) { tdcs.find_by(libelle: 'child_1') }
let(:child_2) { tdcs.find_by(libelle: 'child_2') }
let(:default_params) do
{
procedure_id: procedure.id,
stable_id: child_2.stable_id
}
end
describe '#add_row' do
before do
post :add_row, params: default_params, format: :turbo_stream
end
it do
expect(child_2.reload.condition).to eq(empty_operator(empty, empty))
expect(assigns(:coordinate)).to eq(procedure.draft_revision.coordinate_for(child_2))
expect(assigns(:upper_tdcs)).to eq([child_1, top])
end
end
end
end

View file

@ -289,87 +289,52 @@ describe 'The user' do
include Logic include Logic
context 'with a repetition' do context 'with a repetition' do
let(:stable_id) { 999 }
let(:condition) { greater_than_eq(champ_value(stable_id), constant(18)) }
let(:procedure) do let(:procedure) do
create(:procedure, :published, :for_individual, procedure = create(:procedure, :published, :for_individual,
types_de_champ_public: [ types_de_champ_public: [
{ type: :integer_number, libelle: 'age', stable_id: }, { type: :integer_number, libelle: 'age' },
{ {
type: :repetition, libelle: 'repetition', condition:, children: [ type: :repetition, libelle: 'repetition', children: [
{ type: :text, libelle: 'nom', mandatory: true } { type: :text, libelle: 'nom', mandatory: true }
] ]
} }
]) ])
age = procedure.published_revision.types_de_champ.where(libelle: 'age').first
repetition = procedure.published_revision.types_de_champ.repetition.first
repetition.update(condition: greater_than_eq(champ_value(age.stable_id), constant(18)))
procedure
end end
scenario 'submit a dossier with an hidden mandatory champ within a repetition', js: true do scenario 'submit a dossier with an hidden mandatory champ within a repetition', js: true do
log_in(user, procedure) log_in(user, procedure)
fill_individual fill_individual
fill_in('age', with: 10) fill_in('age', with: 10)
click_on 'Déposer le dossier' click_on 'Déposer le dossier'
expect(page).to have_current_path(merci_dossier_path(user_dossier)) expect(page).to have_current_path(merci_dossier_path(user_dossier))
end end
end end
context 'with a condition inside repetition' do
let(:a_stable_id) { 999 }
let(:b_stable_id) { 9999 }
let(:a_condition) { ds_eq(champ_value(a_stable_id), constant(true)) }
let(:b_condition) { ds_eq(champ_value(b_stable_id), constant(true)) }
let(:condition) { ds_or([a_condition, b_condition]) }
let(:procedure) do
create(:procedure, :published, :for_individual,
types_de_champ_public: [
{ type: :checkbox, libelle: 'champ_a', stable_id: a_stable_id },
{
type: :repetition, libelle: 'repetition', children: [
{ type: :checkbox, libelle: 'champ_b', stable_id: b_stable_id },
{ type: :text, libelle: 'champ_c', condition: }
]
}
])
end
scenario 'fill a dossier', js: true do
log_in(user, procedure)
fill_individual
expect(page).to have_no_css('label', text: 'champ_c', visible: true)
check('champ_a')
wait_for_autosave
expect(page).to have_css('label', text: 'champ_c', visible: true)
uncheck('champ_a')
wait_for_autosave
expect(page).to have_no_css('label', text: 'champ_c', visible: true)
check('champ_b')
wait_for_autosave
expect(page).to have_css('label', text: 'champ_c', visible: true)
end
end
context 'with a required conditionnal champ' do context 'with a required conditionnal champ' do
let(:stable_id) { 999 }
let(:condition) { greater_than_eq(champ_value(stable_id), constant(18)) }
let(:procedure) do let(:procedure) do
create(:procedure, :published, :for_individual, procedure = create(:procedure, :published, :for_individual,
types_de_champ_public: [ types_de_champ_public: [
{ type: :integer_number, libelle: 'age', stable_id: }, { type: :integer_number, libelle: 'age' },
{ type: :text, libelle: 'nom', mandatory: true, condition: } { type: :text, libelle: 'nom', mandatory: true }
]) ])
age, nom = procedure.draft_revision.types_de_champ.all
nom.update(condition: greater_than_eq(champ_value(age.stable_id), constant(18)))
procedure
end end
scenario 'submit a dossier with an hidden mandatory champ ', js: true do scenario 'submit a dossier with an hidden mandatory champ ', js: true do
log_in(user, procedure) log_in(user, procedure)
fill_individual fill_individual
click_on 'Déposer le dossier' click_on 'Déposer le dossier'
expect(page).to have_current_path(merci_dossier_path(user_dossier)) expect(page).to have_current_path(merci_dossier_path(user_dossier))
end end
@ -388,21 +353,23 @@ describe 'The user' do
end end
context 'with a visibilite in cascade' do context 'with a visibilite in cascade' do
let(:age_stable_id) { 999 }
let(:permis_stable_id) { 9999 }
let(:tonnage_stable_id) { 99999 }
let(:permis_condition) { greater_than_eq(champ_value(age_stable_id), constant(18)) }
let(:tonnage_condition) { ds_eq(champ_value(permis_stable_id), constant(true)) }
let(:parking_condition) { less_than_eq(champ_value(tonnage_stable_id), constant(20)) }
let(:procedure) do let(:procedure) do
create(:procedure, :published, :for_individual, procedure = create(:procedure, :for_individual).tap do |p|
types_de_champ_public: [ p.draft_revision.add_type_de_champ(type_champ: :integer_number, libelle: 'age')
{ type: :integer_number, libelle: 'age', stable_id: age_stable_id }, p.draft_revision.add_type_de_champ(type_champ: :yes_no, libelle: 'permis de conduire')
{ type: :yes_no, libelle: 'permis de conduire', stable_id: permis_stable_id, condition: permis_condition }, p.draft_revision.add_type_de_champ(type_champ: :integer_number, libelle: 'tonnage')
{ type: :integer_number, libelle: 'tonnage', stable_id: tonnage_stable_id, condition: tonnage_condition }, p.draft_revision.add_type_de_champ(type_champ: :text, libelle: 'parking')
{ type: :text, libelle: 'parking', condition: parking_condition } end
])
age, permis, tonnage, parking = procedure.draft_revision.types_de_champ.all
permis.update(condition: greater_than_eq(champ_value(age.stable_id), constant(18)))
tonnage.update(condition: ds_eq(champ_value(permis.stable_id), constant(true)))
parking.update(condition: less_than_eq(champ_value(tonnage.stable_id), constant(20)))
procedure.publish!
procedure
end end
scenario 'fill a dossier', js: true do scenario 'fill a dossier', js: true do