Merge pull request #6465 from tchak/dossier-rebase

Rebase dossiers en brouillon

app/jobs/dossier_rebase_job.rb

@@ -0,0 +1,8 @@
+class DossierRebaseJob < ApplicationJob
+  # If by the time the job runs the Dossier has been deleted, ignore the rebase
+  discard_on ActiveRecord::RecordNotFound
+
+  def perform(dossier)
+    dossier.rebase!
+  end
+end

app/models/champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/address_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/annuaire_education_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/carte_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/checkbox_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/civilite_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/commune_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/date_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/datetime_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/decimal_number_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/departement_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/dossier_link_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/drop_down_list_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/email_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/engagement_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/explication_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/header_section_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/iban_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/integer_number_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/linked_drop_down_list_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/multiple_drop_down_list_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/number_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/pays_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/phone_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/piece_justificative_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/region_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/repetition_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/siret_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/text_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/textarea_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/titre_identite_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/champs/yes_no_champ.rb

@@ -6,6 +6,7 @@
 #  data                           :jsonb
 #  fetch_external_data_exceptions :string           is an Array
 #  private                        :boolean          default(FALSE), not null
+#  rebased_at                     :datetime
 #  row                            :integer
 #  type                           :string
 #  value                          :string

app/models/concerns/dossier_rebase_concern.rb

@@ -0,0 +1,121 @@
+module DossierRebaseConcern
+  extend ActiveSupport::Concern
+
+  def rebase!
+    if brouillon? && revision != procedure.published_revision
+      transaction do
+        rebase
+      end
+    end
+  end
+
+  private
+
+  def rebase
+    attachments_to_purge = []
+    geo_areas_to_delete = []
+    changes_by_type_de_champ = revision.compare(procedure.published_revision).group_by { |change| change[:stable_id] }
+
+    changes_by_type_de_champ.each do |stable_id, changes|
+      type_de_champ = find_type_de_champ_by_stable_id(stable_id)
+      published_type_de_champ = find_type_de_champ_by_stable_id(stable_id, published: true)
+
+      changes.each do |change|
+        case change[:op]
+        when :add
+          add_new_champs_for_revision(published_type_de_champ)
+        when :remove
+          delete_champs_for_revision(type_de_champ)
+        end
+      end
+    end
+
+    flattened_all_champs.each do |champ|
+      changes_by_stable_id = (changes_by_type_de_champ[champ.stable_id] || [])
+        .filter { |change| change[:op] == :update }
+
+      update_champ_for_revision(champ) do |update|
+        changes_by_stable_id.each do |change|
+          case change[:attribute]
+          when :type_champ
+            update[:type] = "Champs::#{change[:to].classify}Champ"
+            update[:value] = nil
+            update[:external_id] = nil
+            update[:data] = nil
+            geo_areas_to_delete += champ.geo_areas
+            if champ.piece_justificative_file.attached?
+              attachments_to_purge << champ.piece_justificative_file
+            end
+          when :drop_down_options
+            update[:value] = nil
+          when :carte_layers
+            geo_areas_to_delete += champ.geo_areas
+          end
+          update[:rebased_at] = Time.zone.now
+        end
+      end
+    end
+
+    self.update_column(:revision_id, procedure.published_revision_id)
+    attachments_to_purge.each(&:purge_later)
+    geo_areas_to_delete.each(&:destroy)
+  end
+
+  def add_new_champs_for_revision(published_type_de_champ)
+    if published_type_de_champ.parent
+      find_champs_by_stable_id(published_type_de_champ.parent.stable_id).each do |champ_repetition|
+        champ_repetition.rows.size.times do |row|
+          champ = published_type_de_champ.champ.build(row: row)
+          champ_repetition.champs << champ
+        end
+      end
+    else
+      champ = published_type_de_champ.build_champ
+      self.champs << champ
+    end
+  end
+
+  def update_champ_for_revision(champ)
+    published_type_de_champ = find_type_de_champ_by_stable_id(champ.stable_id, published: true)
+    return if !published_type_de_champ
+
+    update = {}
+
+    yield update
+
+    if champ.type_de_champ != published_type_de_champ
+      update[:type_de_champ_id] = published_type_de_champ.id
+    end
+
+    if update.present?
+      champ.update_columns(update)
+    end
+  end
+
+  def delete_champs_for_revision(published_type_de_champ)
+    Champ.where(id: find_champs_by_stable_id(published_type_de_champ.stable_id).map(&:id))
+      .destroy_all
+  end
+
+  def flattened_all_types_de_champ(published: false)
+    revision = published ? procedure.published_revision : self.revision
+    types_de_champ = revision.types_de_champ + revision.types_de_champ_private
+    (types_de_champ + types_de_champ.filter(&:repetition?).flat_map(&:types_de_champ))
+      .index_by(&:stable_id)
+  end
+
+  def flattened_all_champs
+    all_champs = (champs + champs_private)
+    all_champs + all_champs.filter(&:repetition?).flat_map(&:champs)
+  end
+
+  def find_type_de_champ_by_stable_id(stable_id, published: false)
+    flattened_all_types_de_champ(published: published)[stable_id]
+  end
+
+  def find_champs_by_stable_id(stable_id)
+    flattened_all_champs.filter do |champ|
+      champ.stable_id == stable_id
+    end
+  end
+end

app/models/dossier.rb

@@ -36,6 +36,7 @@
 class Dossier < ApplicationRecord
   self.ignored_columns = [:en_construction_conservation_extension]
   include DossierFilteringConcern
+  include DossierRebaseConcern
 
   include Discard::Model
   self.discard_column = :hidden_at

app/models/procedure.rb

@@ -714,6 +714,9 @@ class Procedure < ApplicationRecord
   def publish_revision!
     update!(draft_revision: create_new_revision, published_revision: draft_revision)
     published_revision.touch(:published_at)
+    dossiers.state_brouillon.find_each do |dossier|
+      DossierRebaseJob.perform_later(dossier)
+    end
   end
 
   def cnaf_enabled?

app/models/procedure_revision.rb

@@ -134,11 +134,11 @@ class ProcedureRevision < ApplicationRecord
       to_sids = to_h.keys
 
       removed = (from_sids - to_sids).map do |sid|
-        { op: :remove, label: from_h[sid].libelle, private: from_h[sid].private?, position: from_sids.index(sid) }
+        { op: :remove, label: from_h[sid].libelle, private: from_h[sid].private?, position: from_sids.index(sid), stable_id: sid }
       end
 
       added = (to_sids - from_sids).map do |sid|
-        { op: :add, label: to_h[sid].libelle, private: to_h[sid].private?, position: to_sids.index(sid) }
+        { op: :add, label: to_h[sid].libelle, private: to_h[sid].private?, position: to_sids.index(sid), stable_id: sid }
       end
 
       kept = from_sids.intersection(to_sids)
@@ -147,7 +147,7 @@ class ProcedureRevision < ApplicationRecord
         .map { |sid| [sid, from_sids.index(sid), to_sids.index(sid)] }
         .filter { |_, from_index, to_index| from_index != to_index }
         .map do |sid, from_index, to_index|
-        { op: :move, label: from_h[sid].libelle, private: from_h[sid].private?, from: from_index, to: to_index, position: to_index }
+        { op: :move, label: from_h[sid].libelle, private: from_h[sid].private?, from: from_index, to: to_index, position: to_index, stable_id: sid }
       end
 
       changed = kept
@@ -172,7 +172,8 @@ class ProcedureRevision < ApplicationRecord
         label: from_type_de_champ.libelle,
         private: from_type_de_champ.private?,
         from: from_type_de_champ.type_champ,
-        to: to_type_de_champ.type_champ
+        to: to_type_de_champ.type_champ,
+        stable_id: from_type_de_champ.stable_id
       }
     end
     if from_type_de_champ.libelle != to_type_de_champ.libelle
@@ -182,7 +183,8 @@ class ProcedureRevision < ApplicationRecord
         label: from_type_de_champ.libelle,
         private: from_type_de_champ.private?,
         from: from_type_de_champ.libelle,
-        to: to_type_de_champ.libelle
+        to: to_type_de_champ.libelle,
+        stable_id: from_type_de_champ.stable_id
       }
     end
     if from_type_de_champ.description != to_type_de_champ.description
@@ -192,7 +194,8 @@ class ProcedureRevision < ApplicationRecord
         label: from_type_de_champ.libelle,
         private: from_type_de_champ.private?,
         from: from_type_de_champ.description,
-        to: to_type_de_champ.description
+        to: to_type_de_champ.description,
+        stable_id: from_type_de_champ.stable_id
       }
     end
     if from_type_de_champ.mandatory? != to_type_de_champ.mandatory?
@@ -202,7 +205,8 @@ class ProcedureRevision < ApplicationRecord
         label: from_type_de_champ.libelle,
         private: from_type_de_champ.private?,
         from: from_type_de_champ.mandatory?,
-        to: to_type_de_champ.mandatory?
+        to: to_type_de_champ.mandatory?,
+        stable_id: from_type_de_champ.stable_id
       }
     end
     if to_type_de_champ.drop_down_list?
@@ -213,7 +217,8 @@ class ProcedureRevision < ApplicationRecord
           label: from_type_de_champ.libelle,
           private: from_type_de_champ.private?,
           from: from_type_de_champ.drop_down_list_options,
-          to: to_type_de_champ.drop_down_list_options
+          to: to_type_de_champ.drop_down_list_options,
+          stable_id: from_type_de_champ.stable_id
         }
       end
     elsif to_type_de_champ.carte?
@@ -224,7 +229,8 @@ class ProcedureRevision < ApplicationRecord
           label: from_type_de_champ.libelle,
           private: from_type_de_champ.private?,
           from: from_type_de_champ.carte_optional_layers,
-          to: to_type_de_champ.carte_optional_layers
+          to: to_type_de_champ.carte_optional_layers,
+          stable_id: from_type_de_champ.stable_id
         }
       end
     elsif to_type_de_champ.piece_justificative?
@@ -235,7 +241,8 @@ class ProcedureRevision < ApplicationRecord
           label: from_type_de_champ.libelle,
           private: from_type_de_champ.private?,
           from: from_type_de_champ.piece_justificative_template_filename,
-          to: to_type_de_champ.piece_justificative_template_filename
+          to: to_type_de_champ.piece_justificative_template_filename,
+          stable_id: from_type_de_champ.stable_id
         }
       end
     elsif to_type_de_champ.repetition?

app/views/shared/dossiers/editable_champs/_champ_label_content.html.haml

@@ -5,3 +5,7 @@
 - if champ.updated_at.present? && seen_at.present?
   %span.updated-at{ class: highlight_if_unseen_class(seen_at, champ.updated_at) }
     = "modifié le #{try_format_datetime(champ.updated_at)}"
+
+- if champ.rebased_at.present? && champ.rebased_at > (seen_at || champ.updated_at) && current_user.owns_or_invite?(champ.dossier)
+  %span.updated-at.highlighted
+    Le type de ce champ où sa description a été modifiée par l'administration. Vérifier son contenu.

db/migrate/20211013131241_add_rebased_at_to_champs.rb

@@ -0,0 +1,5 @@
+class AddRebasedAtToChamps < ActiveRecord::Migration[6.1]
+  def change
+    add_column :champs, :rebased_at, :datetime
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2021_10_20_104237) do
+ActiveRecord::Schema.define(version: 2021_10_20_114237) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -190,6 +190,7 @@ ActiveRecord::Schema.define(version: 2021_10_20_104237) do
     t.string "external_id"
     t.string "fetch_external_data_exceptions", array: true
     t.jsonb "value_json"
+    t.datetime "rebased_at"
     t.index ["dossier_id"], name: "index_champs_on_dossier_id"
     t.index ["etablissement_id"], name: "index_champs_on_etablissement_id"
     t.index ["parent_id"], name: "index_champs_on_parent_id"

spec/models/dossier_spec.rb

@@ -1459,4 +1459,71 @@ describe Dossier do
 
     it { expect(dossier.spreadsheet_columns(types_de_champ: [])).to include(["État du dossier", "Brouillon"]) }
   end
+
+  describe "#rebase" do
+    let(:procedure) { create(:procedure, :with_type_de_champ_mandatory, :with_yes_no, :with_repetition, :with_datetime) }
+    let(:dossier) { create(:dossier, procedure: procedure) }
+
+    let(:yes_no_type_de_champ) { procedure.types_de_champ.find { |tdc| tdc.type_champ == TypeDeChamp.type_champs.fetch(:yes_no) } }
+
+    let(:text_type_de_champ) { procedure.types_de_champ.find(&:mandatory?) }
+    let(:text_champ) { dossier.champs.find(&:mandatory?) }
+    let(:rebased_text_champ) { dossier.champs.find { |c| c.type_champ == TypeDeChamp.type_champs.fetch(:text) } }
+
+    let(:datetime_type_de_champ) { procedure.types_de_champ.find { |tdc| tdc.type_champ == TypeDeChamp.type_champs.fetch(:datetime) } }
+    let(:datetime_champ) { dossier.champs.find { |c| c.type_champ == TypeDeChamp.type_champs.fetch(:datetime) } }
+    let(:rebased_datetime_champ) { dossier.champs.find { |c| c.type_champ == TypeDeChamp.type_champs.fetch(:date) } }
+
+    let(:repetition_type_de_champ) { procedure.types_de_champ.find { |tdc| tdc.type_champ == TypeDeChamp.type_champs.fetch(:repetition) } }
+    let(:repetition_text_type_de_champ) { repetition_type_de_champ.types_de_champ.find { |tdc| tdc.type_champ == TypeDeChamp.type_champs.fetch(:text) } }
+    let(:repetition_champ) { dossier.champs.find { |c| c.type_champ == TypeDeChamp.type_champs.fetch(:repetition) } }
+    let(:rebased_repetition_champ) { dossier.champs.find { |c| c.type_champ == TypeDeChamp.type_champs.fetch(:repetition) } }
+
+    before do
+      procedure.publish!
+      procedure.draft_revision.add_type_de_champ({
+        type_champ: TypeDeChamp.type_champs.fetch(:text),
+        libelle: "Un champ text"
+      })
+      procedure.draft_revision.find_or_clone_type_de_champ(text_type_de_champ).update(mandatory: false, libelle: "nouveau libelle")
+      procedure.draft_revision.find_or_clone_type_de_champ(datetime_type_de_champ).update(type_champ: TypeDeChamp.type_champs.fetch(:date))
+      procedure.draft_revision.find_or_clone_type_de_champ(repetition_text_type_de_champ).update(libelle: "nouveau libelle dans une repetition")
+      procedure.draft_revision.add_type_de_champ({
+        type_champ: TypeDeChamp.type_champs.fetch(:checkbox),
+        libelle: "oui ou non",
+        parent_id: repetition_type_de_champ.stable_id
+      })
+      procedure.draft_revision.remove_type_de_champ(yes_no_type_de_champ.stable_id)
+
+      datetime_champ.update(value: Date.today.to_s)
+      text_champ.update(value: 'bonjour')
+    end
+
+    it "updates the brouillon champs with the latest revision changes" do
+      revision_id = dossier.revision_id
+      libelle = text_type_de_champ.libelle
+
+      expect(dossier.revision).to eq(procedure.published_revision)
+      expect(dossier.champs.size).to eq(4)
+      expect(repetition_champ.rows.size).to eq(1)
+      expect(repetition_champ.rows[0].size).to eq(1)
+
+      procedure.publish_revision!
+      perform_enqueued_jobs
+      procedure.reload
+      dossier.reload
+
+      expect(procedure.revisions.size).to eq(3)
+      expect(dossier.revision).to eq(procedure.published_revision)
+      expect(dossier.champs.size).to eq(4)
+      expect(rebased_text_champ.value).to eq(text_champ.value)
+      expect(rebased_text_champ.type_de_champ_id).not_to eq(text_champ.type_de_champ_id)
+      expect(rebased_datetime_champ.type_champ).to eq(TypeDeChamp.type_champs.fetch(:date))
+      expect(rebased_datetime_champ.value).to be_nil
+      expect(rebased_repetition_champ.rows.size).to eq(1)
+      expect(rebased_repetition_champ.rows[0].size).to eq(2)
+      expect(rebased_text_champ.rebased_at).not_to be_nil
+      expect(rebased_datetime_champ.rebased_at).not_to be_nil
+    end
+  end
 end

spec/models/procedure_revision_spec.rb

@@ -174,7 +174,8 @@ describe ProcedureRevision do
           {
             op: :add,
             label: "Un champ text",
-            private: false
+            private: false,
+            stable_id: new_type_de_champ.stable_id
           }
         ])
 
@@ -186,12 +187,14 @@ describe ProcedureRevision do
             label: type_de_champ_first.libelle,
             private: false,
             from: type_de_champ_first.libelle,
-            to: "modifier le libelle"
+            to: "modifier le libelle",
+            stable_id: type_de_champ_first.stable_id
           },
           {
             op: :add,
             label: "Un champ text",
-            private: false
+            private: false,
+            stable_id: new_type_de_champ.stable_id
           }
         ])
         expect(new_revision.types_de_champ.first.revision).to eq(new_revision)
@@ -204,19 +207,22 @@ describe ProcedureRevision do
             label: type_de_champ_first.libelle,
             private: false,
             from: type_de_champ_first.libelle,
-            to: "modifier le libelle"
+            to: "modifier le libelle",
+            stable_id: type_de_champ_first.stable_id
           },
           {
             op: :add,
             label: "Un champ text",
-            private: false
+            private: false,
+            stable_id: new_type_de_champ.stable_id
           },
           {
             op: :move,
             label: type_de_champ_second.libelle,
             private: false,
             from: 1,
-            to: 2
+            to: 2,
+            stable_id: type_de_champ_second.stable_id
           }
         ])
         expect(new_revision.types_de_champ.last.revision).to eq(revision)
@@ -226,12 +232,14 @@ describe ProcedureRevision do
           {
             op: :remove,
             label: type_de_champ_first.libelle,
-            private: false
+            private: false,
+            stable_id: type_de_champ_first.stable_id
           },
           {
             op: :add,
             label: "Un champ text",
-            private: false
+            private: false,
+            stable_id: new_type_de_champ.stable_id
           }
         ])
 
@@ -241,12 +249,14 @@ describe ProcedureRevision do
           {
             op: :remove,
             label: type_de_champ_first.libelle,
-            private: false
+            private: false,
+            stable_id: type_de_champ_first.stable_id
           },
           {
             op: :add,
             label: "Un champ text",
-            private: false
+            private: false,
+            stable_id: new_type_de_champ.stable_id
           },
           {
             op: :update,
@@ -254,7 +264,8 @@ describe ProcedureRevision do
             label: type_de_champ_second.libelle,
             private: false,
             from: type_de_champ_second.description,
-            to: "une description"
+            to: "une description",
+            stable_id: type_de_champ_second.stable_id
           },
           {
             op: :update,
@@ -262,7 +273,8 @@ describe ProcedureRevision do
             label: type_de_champ_second.libelle,
             private: false,
             from: false,
-            to: true
+            to: true,
+            stable_id: type_de_champ_second.stable_id
           }
         ])
 
@@ -272,12 +284,14 @@ describe ProcedureRevision do
           {
             op: :remove,
             label: type_de_champ_first.libelle,
-            private: false
+            private: false,
+            stable_id: type_de_champ_first.stable_id
           },
           {
             op: :add,
             label: "Un champ text",
-            private: false
+            private: false,
+            stable_id: new_type_de_champ.stable_id
           },
           {
             op: :update,
@@ -285,7 +299,8 @@ describe ProcedureRevision do
             label: type_de_champ_second.libelle,
             private: false,
             from: type_de_champ_second.description,
-            to: "une description"
+            to: "une description",
+            stable_id: type_de_champ_second.stable_id
           },
           {
             op: :update,
@@ -293,7 +308,8 @@ describe ProcedureRevision do
             label: type_de_champ_second.libelle,
             private: false,
             from: false,
-            to: true
+            to: true,
+            stable_id: type_de_champ_second.stable_id
           },
           {
             op: :update,
@@ -301,7 +317,8 @@ describe ProcedureRevision do
             label: "sub type de champ",
             private: false,
             from: "text",
-            to: "drop_down_list"
+            to: "drop_down_list",
+            stable_id: new_revision.types_de_champ.last.types_de_champ.first.stable_id
           },
           {
             op: :update,
@@ -309,7 +326,8 @@ describe ProcedureRevision do
             label: "sub type de champ",
             private: false,
             from: [],
-            to: ["one", "two"]
+            to: ["one", "two"],
+            stable_id: new_revision.types_de_champ.last.types_de_champ.first.stable_id
           }
         ])
 
@@ -319,12 +337,14 @@ describe ProcedureRevision do
           {
             op: :remove,
             label: type_de_champ_first.libelle,
-            private: false
+            private: false,
+            stable_id: type_de_champ_first.stable_id
           },
           {
             op: :add,
             label: "Un champ text",
-            private: false
+            private: false,
+            stable_id: new_type_de_champ.stable_id
           },
           {
             op: :update,
@@ -332,7 +352,8 @@ describe ProcedureRevision do
             label: type_de_champ_second.libelle,
             private: false,
             from: type_de_champ_second.description,
-            to: "une description"
+            to: "une description",
+            stable_id: type_de_champ_second.stable_id
           },
           {
             op: :update,
@@ -340,7 +361,8 @@ describe ProcedureRevision do
             label: type_de_champ_second.libelle,
             private: false,
             from: false,
-            to: true
+            to: true,
+            stable_id: type_de_champ_second.stable_id
           },
           {
             op: :update,
@@ -348,7 +370,8 @@ describe ProcedureRevision do
             label: "sub type de champ",
             private: false,
             from: "text",
-            to: "carte"
+            to: "carte",
+            stable_id: new_revision.types_de_champ.last.types_de_champ.first.stable_id
           },
           {
             op: :update,
@@ -356,7 +379,8 @@ describe ProcedureRevision do
             label: "sub type de champ",
             private: false,
             from: [],
-            to: [:cadastres, :znieff]
+            to: [:cadastres, :znieff],
+            stable_id: new_revision.types_de_champ.last.types_de_champ.first.stable_id
           }
         ])
       end