Skip to content

Commit 896e31e

Browse files
committed
Cosmetic: Move implicitly-set belongs_to_required config since Rails 5.0 to config/application.rb
1 parent a9ce12c commit 896e31e

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

config/application.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,11 @@ class Application < Rails::Application
3333
# Fixture paths
3434
config.fixture_paths = [Rails.root.join('spec/fixtures')]
3535

36+
# TODO: The following config should be switched to its default value `true`, but not urgent.
37+
# Rails 5.0: Require `belongs_to` associations by default. Previous versions had false.
38+
# https://railsguides.jp/configuring.html#config-active-record-belongs-to-required-by-default
39+
config.active_record.belongs_to_required_by_default = false
40+
3641
# Rails 8.0: `to_time` will always preserve the full timezone in Rails 8.1.
3742
config.active_support.to_time_preserves_timezone = :zone
3843
end

config/initializers/new_framework_defaults.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,4 @@
1717
#ActiveSupport.to_time_preserves_timezone = true
1818

1919
# Require `belongs_to` associations by default. Previous versions had false.
20-
Rails.application.config.active_record.belongs_to_required_by_default = false
20+
#Rails.application.config.active_record.belongs_to_required_by_default = false

0 commit comments

Comments
 (0)