Fix a bug: edge case of birthday in the future in seeds.rb, we should…

… use Date.current instead of DateTime.current since birthday is a date (see #344)
sleede · May 4, 2022 · 5e5822c · 5e5822c
1 parent 5cffdff
commit 5e5822c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ## next deploy
 
 - when generating an avoir, the option "by_wallet" is not present anymore if wallet module is off
+- Fix a bug: edge case of birthday in the future in seeds.rb, we should use Date.current instead of DateTime.current since birthday is a date (see https://github.com/sleede/fab-manager/issues/344)
 
 ## v5.3.13 2022 May 02
 

diff --git a/app/models/statistic_profile.rb b/app/models/statistic_profile.rb
@@ -46,6 +46,6 @@ def age
   private
 
   def check_birthday_in_past
-    errors.add(:birthday, I18n.t('statistic_profile.birthday_in_past')) if birthday.present? && birthday > DateTime.current
+    errors.add(:birthday, I18n.t('statistic_profile.birthday_in_past')) if birthday.present? && birthday > Date.current
   end
 end
diff --git a/db/seeds.rb b/db/seeds.rb
@@ -89,7 +89,7 @@
   admin = User.new(username: 'admin', email: ENV['ADMIN_EMAIL'], password: ENV['ADMIN_PASSWORD'],
                    password_confirmation: Rails.application.secrets.admin_password, group_id: Group.find_by(slug: 'admins').id,
                    profile_attributes: { first_name: 'admin', last_name: 'admin', phone: '0123456789' },
-                   statistic_profile_attributes: { gender: true, birthday: DateTime.current })
+                   statistic_profile_attributes: { gender: true, birthday: Date.current })
   admin.add_role 'admin'
   admin.save!
 end