From 51c97e7e192e82cc3b563a15bcaac286b4f8e422 Mon Sep 17 00:00:00 2001 From: Arun Agrawal Date: Wed, 3 Jul 2013 20:48:49 +0200 Subject: removed deprecated increment_open_transactions --- activerecord/CHANGELOG.md | 4 ++++ .../lib/active_record/connection_adapters/abstract_adapter.rb | 4 ---- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/activerecord/CHANGELOG.md b/activerecord/CHANGELOG.md index 9cd011aef5..878807151b 100644 --- a/activerecord/CHANGELOG.md +++ b/activerecord/CHANGELOG.md @@ -1,3 +1,7 @@ +* Remove deprecated `increment_open_transactions`. + + *Arun Agrawal* + * Remove deprecated `PostgreSQLAdapter#outside_transaction?` method. You can use `#transaction_open?` instead. diff --git a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb index e232cad982..5ef70487dd 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb @@ -387,10 +387,6 @@ module ActiveRecord @transaction.number end - def increment_open_transactions - ActiveSupport::Deprecation.warn "#increment_open_transactions is deprecated and has no effect" - end - def decrement_open_transactions ActiveSupport::Deprecation.warn "#decrement_open_transactions is deprecated and has no effect" end -- cgit v1.2.3 From b23501112e40f8e8fff3676147c8a4aa8d8a4aa3 Mon Sep 17 00:00:00 2001 From: Arun Agrawal Date: Wed, 3 Jul 2013 21:40:20 +0200 Subject: Remove deprecated `decrement_open_transactions`. --- activerecord/CHANGELOG.md | 4 ++++ .../lib/active_record/connection_adapters/abstract_adapter.rb | 4 ---- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/activerecord/CHANGELOG.md b/activerecord/CHANGELOG.md index 878807151b..e13aa74de8 100644 --- a/activerecord/CHANGELOG.md +++ b/activerecord/CHANGELOG.md @@ -1,3 +1,7 @@ +* Remove deprecated `decrement_open_transactions`. + + *Arun Agrawal* + * Remove deprecated `increment_open_transactions`. *Arun Agrawal* diff --git a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb index 5ef70487dd..5887e6e0cd 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb @@ -387,10 +387,6 @@ module ActiveRecord @transaction.number end - def decrement_open_transactions - ActiveSupport::Deprecation.warn "#decrement_open_transactions is deprecated and has no effect" - end - def transaction_joinable=(joinable) message = "#transaction_joinable= is deprecated. Please pass the :joinable option to #begin_transaction instead." ActiveSupport::Deprecation.warn message -- cgit v1.2.3 From 9105c59af05d3219e2811cab0642badbb4af5398 Mon Sep 17 00:00:00 2001 From: Arun Agrawal Date: Wed, 3 Jul 2013 20:52:56 +0200 Subject: Remove deprecated `transaction_joinable=` in favor of `begin_transaction` with `:joinable` option. --- activerecord/CHANGELOG.md | 5 +++++ .../lib/active_record/connection_adapters/abstract_adapter.rb | 6 ------ activerecord/test/cases/transactions_test.rb | 9 --------- 3 files changed, 5 insertions(+), 15 deletions(-) diff --git a/activerecord/CHANGELOG.md b/activerecord/CHANGELOG.md index e13aa74de8..d64d23613f 100644 --- a/activerecord/CHANGELOG.md +++ b/activerecord/CHANGELOG.md @@ -1,3 +1,8 @@ +* Remove deprecated `transaction_joinable=` in favor of `begin_transaction` + with `:joinable` option. + + *Arun Agrawal* + * Remove deprecated `decrement_open_transactions`. *Arun Agrawal* diff --git a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb index 5887e6e0cd..cfff7202a3 100644 --- a/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb +++ b/activerecord/lib/active_record/connection_adapters/abstract_adapter.rb @@ -387,12 +387,6 @@ module ActiveRecord @transaction.number end - def transaction_joinable=(joinable) - message = "#transaction_joinable= is deprecated. Please pass the :joinable option to #begin_transaction instead." - ActiveSupport::Deprecation.warn message - @transaction.joinable = joinable - end - def create_savepoint end diff --git a/activerecord/test/cases/transactions_test.rb b/activerecord/test/cases/transactions_test.rb index f4814f1c42..6ba7464203 100644 --- a/activerecord/test/cases/transactions_test.rb +++ b/activerecord/test/cases/transactions_test.rb @@ -593,14 +593,5 @@ if current_adapter?(:PostgreSQLAdapter) assert_equal original_salary, Developer.find(1).salary end - - test "#transaction_joinable= is deprecated" do - Developer.transaction do - conn = Developer.connection - assert conn.current_transaction.joinable? - assert_deprecated { conn.transaction_joinable = false } - assert !conn.current_transaction.joinable? - end - end end end -- cgit v1.2.3 From 50cbc03d18c5984347965a94027879623fc44cce Mon Sep 17 00:00:00 2001 From: Arun Agrawal Date: Wed, 3 Jul 2013 21:48:52 +0200 Subject: Remove deprecated `scope` use without passing a callable object. Removed tests from deprecated code. --- activerecord/CHANGELOG.md | 4 ++++ activerecord/lib/active_record/scoping/named.rb | 13 ------------- activerecord/test/cases/scoping/named_scoping_test.rb | 10 ---------- 3 files changed, 4 insertions(+), 23 deletions(-) diff --git a/activerecord/CHANGELOG.md b/activerecord/CHANGELOG.md index d64d23613f..5a8876c282 100644 --- a/activerecord/CHANGELOG.md +++ b/activerecord/CHANGELOG.md @@ -1,3 +1,7 @@ +* Remove deprecated `scope` use without passing a callable object. + + *Arun Agrawal* + * Remove deprecated `transaction_joinable=` in favor of `begin_transaction` with `:joinable` option. diff --git a/activerecord/lib/active_record/scoping/named.rb b/activerecord/lib/active_record/scoping/named.rb index e7d5e6ce84..7c51aa6979 100644 --- a/activerecord/lib/active_record/scoping/named.rb +++ b/activerecord/lib/active_record/scoping/named.rb @@ -141,19 +141,6 @@ module ActiveRecord def scope(name, body, &block) extension = Module.new(&block) if block - # Check body.is_a?(Relation) to prevent the relation actually being - # loaded by respond_to? - if body.is_a?(Relation) || !body.respond_to?(:call) - ActiveSupport::Deprecation.warn( - "Using #scope without passing a callable object is deprecated. For " \ - "example `scope :red, where(color: 'red')` should be changed to " \ - "`scope :red, -> { where(color: 'red') }`. There are numerous gotchas " \ - "in the former usage and it makes the implementation more complicated " \ - "and buggy. (If you prefer, you can just define a class method named " \ - "`self.red`.)" - ) - end - singleton_class.send(:define_method, name) do |*args| if body.respond_to?(:call) scope = all.scoping { body.call(*args) } diff --git a/activerecord/test/cases/scoping/named_scoping_test.rb b/activerecord/test/cases/scoping/named_scoping_test.rb index abd0b8621a..72c9787b84 100644 --- a/activerecord/test/cases/scoping/named_scoping_test.rb +++ b/activerecord/test/cases/scoping/named_scoping_test.rb @@ -435,16 +435,6 @@ class NamedScopingTest < ActiveRecord::TestCase end end - def test_eager_scopes_are_deprecated - klass = Class.new(ActiveRecord::Base) - klass.table_name = 'posts' - - assert_deprecated do - klass.scope :welcome_2, klass.where(:id => posts(:welcome).id) - end - assert_equal [posts(:welcome).title], klass.welcome_2.map(&:title) - end - def test_eager_default_scope_relations_are_remove klass = Class.new(ActiveRecord::Base) klass.table_name = 'posts' -- cgit v1.2.3