14873: Removes legacy setting as all controllers were migrated.

When needing to stop the callback chain, instead of returning false
now we need to call: throw(:abort)
The removed setting allowed the old behavior, and it isn't neede anymore.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <ldipentima@veritasgenetics.com>

diff --git a/services/api/config/initializers/new_framework_defaults.rb b/services/api/config/initializers/new_framework_defaults.rb
index b8dca33a373171cceda0b9d9f3f2d5f6f1dc68e9..2e2f0b1810df54f5a5a4b15e6a07c5be98e78f67 100644 (file)
--- a/services/api/config/initializers/new_framework_defaults.rb
+++ b/services/api/config/initializers/new_framework_defaults.rb
@@ -24,6 +24,3 @@ ActiveSupport.to_time_preserves_timezone = false
 
 # Require `belongs_to` associations by default. Previous versions had false.
 Rails.application.config.active_record.belongs_to_required_by_default = false
-
-# Do not halt callback chains when a callback returns false. Previous versions had true.
-ActiveSupport.halt_callback_chains_on_return_false = true