Fixes ReferentialMetadata#validate_periods. Refs #2035

1 files changed, 9 insertions, 5 deletions

diff --git a/app/models/referential_metadata.rb b/app/models/referential_metadata.rb
index 0c792a9b9..e752a6ca0 100644
--- a/app/models/referential_metadata.rb
+++ b/app/models/referential_metadata.rb
@@ -87,18 +87,22 @@ class ReferentialMetadata < ActiveRecord::Base
   validate :validate_periods
 
   def validate_periods
-    Rails.logger.debug "Validate periods"
+    periods_are_valid = true
+
     unless periods.all?(&:valid?)
-      errors.add(:periods, :invalid)
+      periods_are_valid = false
     end
 
     periods.each do |period|
-      Rails.logger.debug "Validate period #{period.inspect} : #{period.intersect?(periods)}"
       if period.intersect?(periods)
-        period.errors.add(:base, :invalid)
-        Rails.logger.debug "period errors #{period.errors}"
+        period.errors.add(:base, I18n.t("referentials.errors.overlapped_period"))
+        periods_are_valid = false
       end
     end
+
+    unless periods_are_valid
+      errors.add(:periods, :invalid)
+    end
   end
 
   def periods_attributes=(attributes = {})