diff --git a/app/models/dossier.rb b/app/models/dossier.rb index 6a84c8bf984..5118da8ee04 100644 --- a/app/models/dossier.rb +++ b/app/models/dossier.rb @@ -656,12 +656,12 @@ def expiration_notification_date def close_to_expiration? return false if en_instruction? - expiration_notification_date < Time.zone.now && expiration_notification_date > Expired::REMAINING_WEEKS_BEFORE_EXPIRATION.weeks.ago + Expired::REMAINING_WEEKS_BEFORE_EXPIRATION.weeks.ago < expiration_notification_date end def has_expired? return false if en_instruction? - expiration_notification_date < Time.zone.now && expiration_notification_date < Expired::REMAINING_WEEKS_BEFORE_EXPIRATION.weeks.ago + expiration_notification_date < Expired::REMAINING_WEEKS_BEFORE_EXPIRATION.weeks.ago end def after_notification_expiration_date @@ -841,27 +841,27 @@ def build_attestation end end - def author_is_user(author) + def is_user?(author) author.is_a?(User) end - def author_is_administration(author) + def is_administration?(author) author.is_a?(Instructeur) || author.is_a?(Administrateur) || author.is_a?(SuperAdmin) end - def author_is_automatic(author) + def is_automatic?(author) author == :automatic end def hide_and_keep_track!(author, reason) transaction do - if author_is_administration(author) && can_be_deleted_by_administration?(reason) + if is_administration?(author) && can_be_deleted_by_administration?(reason) update(hidden_by_administration_at: Time.zone.now, hidden_by_reason: reason) log_dossier_operation(author, :supprimer, self) - elsif author_is_user(author) && can_be_deleted_by_user? + elsif is_user?(author) && can_be_deleted_by_user? update(hidden_by_user_at: Time.zone.now, dossier_transfer_id: nil, hidden_by_reason: reason) log_dossier_operation(author, :supprimer, self) - elsif author_is_automatic(author) && can_be_deleted_by_automatic?(reason) + elsif is_automatic?(author) && can_be_deleted_by_automatic?(reason) update(hidden_by_expired_at: Time.zone.now, hidden_by_reason: reason) log_automatic_dossier_operation(:supprimer, self) else @@ -879,9 +879,9 @@ def hide_and_keep_track!(author, reason) def restore(author) transaction do - if author_is_administration(author) + if is_administration?(author) update(hidden_by_administration_at: nil) - elsif author_is_user(author) + elsif is_user?(author) update(hidden_by_user_at: nil) end diff --git a/config/locales/views/dossier_mailer/notify_near_deletion_to_user/fr.yml b/config/locales/views/dossier_mailer/notify_near_deletion_to_user/fr.yml index 84479d47ac3..f6a52e6f5e0 100644 --- a/config/locales/views/dossier_mailer/notify_near_deletion_to_user/fr.yml +++ b/config/locales/views/dossier_mailer/notify_near_deletion_to_user/fr.yml @@ -20,5 +20,5 @@ fr: one: Vous pouvez télécharger votre dossier au format PDF depuis l’onglet « Expirant » sur la page other: "Vous pouvez télécharger vos dossiers au format PDF depuis l’onglet « Expirant » sur la page " footer_en_construction: - one: Depuis la page de votre dossier vous avez la possibilité de :
- prolonger la durée de conservation pour un délai de 6 mois
- contacter l'administration qui gère votre dossier via la messagerie - other: Depuis la page de vos dossiers vous avez la possibilité de :
- prolonger la durée de conservation pour un délai de 6 mois
- contacter l'administration qui gère votre dossier via la messagerie + one: Depuis la page de votre dossier vous avez la possibilité de :
- prolonger la durée de conservation
- contacter l'administration qui gère votre dossier via la messagerie + other: Depuis la page de vos dossiers vous avez la possibilité de :
- prolonger la durée de conservation
- contacter l'administration qui gère votre dossier via la messagerie diff --git a/spec/mailers/dossier_mailer_spec.rb b/spec/mailers/dossier_mailer_spec.rb index 1cc81b05e93..cad8960f613 100644 --- a/spec/mailers/dossier_mailer_spec.rb +++ b/spec/mailers/dossier_mailer_spec.rb @@ -211,7 +211,7 @@ def notify_deletion_to_administration(hidden_dossier, to_email) expect(subject.body).to include("N° #{dossier.id} ") expect(subject.body).to include(dossier.procedure.libelle) expect(subject.body).to include("Votre compte reste activé") - expect(subject.body).to include("Depuis la page de votre dossier vous avez la possibilité de :
- prolonger la durée de conservation pour un délai de 6 mois") + expect(subject.body).to include("Depuis la page de votre dossier vous avez la possibilité de :
- prolonger la durée de conservation") end end