Disable attr_accessible for rails >= 4.0.

Author: aepstein

.gitignore

@@ -3,3 +3,4 @@ garlic
 nbproject
 rdoc
 gemfiles/*.lock
+log/*

test/model_test.rb

@@ -26,7 +26,9 @@ class TestModel < ActiveRecord::Base
     :class_name => "TestAttrThrough", :source => :test_attr_throughs,
     :conditions => "test_attrs.attr = 1"
 
-  attr_accessible :content, :test_attr_through_id, :country_id
+  if Rails.version < "4"
+    attr_accessible :content, :test_attr_through_id, :country_id
+  end
 
   # TODO currently not working in Rails 3
   if Rails.version < "3"
@@ -75,9 +77,11 @@ class TestAttr < ActiveRecord::Base
   has_many :test_attr_throughs
   has_many :test_model_security_model_with_finds
   attr_reader :role_symbols
-  attr_accessible :test_model, :test_another_model, :attr, :branch, :company, :test_attr,
-	  :test_a_third_model, :n_way_join_item, :n_way_join_item_id, :test_attr_through_id, 
-	  :test_model_id, :test_another_model_id
+  if Rails.version < "4"
+    attr_accessible :test_model, :test_another_model, :attr, :branch, :company, :test_attr,
+	    :test_a_third_model, :n_way_join_item, :n_way_join_item_id, :test_attr_through_id, 
+	    :test_model_id, :test_another_model_id
+	end
   def initialize (*args)
     @role_symbols = []
     super(*args)
@@ -91,7 +95,9 @@ class TestAttrThrough < ActiveRecord::Base
 class TestModelSecurityModel < ActiveRecord::Base
   has_many :test_attrs
   using_access_control
-  attr_accessible :attr, :attr_2, :test_attrs
+  if Rails.version < "4"
+    attr_accessible :attr, :attr_2, :test_attrs
+  end
 end
 class TestModelSecurityModelWithFind < ActiveRecord::Base
   if Rails.version < "3.2"
@@ -103,19 +109,25 @@ class TestModelSecurityModelWithFind < ActiveRecord::Base
   belongs_to :test_attr
   using_access_control :include_read => true, 
     :context => :test_model_security_models
-  attr_accessible :test_attr, :attr
+  if Rails.version < "4"
+    attr_accessible :test_attr, :attr
+  end
 end
 
 class Branch < ActiveRecord::Base
   has_many :test_attrs
   belongs_to :company
-  attr_accessible :name, :company
+  if Rails.version < "4"
+    attr_accessible :name, :company
+  end
 end
 class Company < ActiveRecord::Base
   has_many :test_attrs
   has_many :branches
   belongs_to :country
-  attr_accessible :name, :country, :country_id
+  if Rails.version < "4"
+    attr_accessible :name, :country, :country_id
+  end
 end
 class SmallCompany < Company
   def self.decl_auth_context
@@ -125,7 +137,9 @@ def self.decl_auth_context
 class Country < ActiveRecord::Base
   has_many :test_models
   has_many :companies
-  attr_accessible :name
+  if Rails.version < "4"
+    attr_accessible :name
+  end
 end
 
 class NamedScopeModelTest < Test::Unit::TestCase
@@ -781,7 +795,13 @@ def test_with_contains
     user = MockUser.new(:test_role,
                         :id => test_model_1.test_attrs.first.id)
     assert_equal 1, TestModel.with_permissions_to(:read, :user => user).length
-    assert_equal 1, TestModel.with_permissions_to(:read, :user => user).find(:all, :conditions => {:id => test_model_1.id}).length
+    if Rails.version < "4"
+      assert_equal 1, TestModel.with_permissions_to(:read, :user => user).
+        find(:all, :conditions => {:id => test_model_1.id}).length
+    else
+      assert_equal 1, TestModel.with_permissions_to(:read, :user => user).
+        where( :id => test_model_1.id ).length
+    end
 
     TestModel.delete_all
     TestAttr.delete_all

test/test_helper.rb

@@ -114,6 +114,7 @@ def warn?; end
 else
   class TestApp
     class Application < ::Rails::Application
+      config.secret_key_base = "testingpurposesonly"
       config.active_support.deprecation = :stderr
       database_path = File.expand_path('../database.yml', __FILE__)
       if Rails.version.start_with? '3.0.'
@@ -128,9 +129,11 @@ class ApplicationController < ActionController::Base
   end
   #Rails::Application.routes.draw do
   Rails.application.routes.draw do
-    match '/name/spaced_things(/:action)' => 'name/spaced_things'
-    match '/deep/name_spaced/things(/:action)' => 'deep/name_spaced/things'
-    match '/:controller(/:action(/:id))'
+    match '/name/spaced_things(/:action)' => 'name/spaced_things',
+      :via => [ :get, :post ]
+    match '/deep/name_spaced/things(/:action)' => 'deep/name_spaced/things',
+      :via => [ :get, :post ]
+    match '/:controller(/:action(/:id))', :via => [ :get, :post ]
   end
 end
 
@@ -141,8 +144,12 @@ class ApplicationController < ActionController::Base
 
 class Test::Unit::TestCase
   include Authorization::TestHelper
+end
 
-  def request! (user, action, reader, params = {})
+class ActionController::TestCase
+  include Authorization::TestHelper
+
+  def request!(user, action, reader, params = {})
     action = action.to_sym if action.is_a?(String)
     @controller.current_user = user
     @controller.authorization_engine = Authorization::Engine.new(reader)