declarative_authorization 0.5.4 → 0.5.5

data/CHANGELOG

@@ -1,3 +1,7 @@
+** RELEASE 0.5.5 (Jan 10, 2012)
+
+* Update of handling of association proxies for Rails 3.2
+
 ** RELEASE 0.5.4 (Nov 30, 2011)
 
 * Cumulative loading of authorization rules [Damian Curso/sb]

data/lib/declarative_authorization/authorization.rb

@@ -60,6 +60,14 @@ module Authorization
   def self.default_role= (role)
     @@default_role = role.to_sym
   end
+
+  def self.is_a_association_proxy? (object)
+    if Rails.version < "3.2"
+      object.respond_to?(:proxy_reflection)
+    else
+      object.respond_to?(:proxy_association)
+    end
+  end
   
   # Authorization::Engine implements the reference monitor.  It may be used
   # for querying the permission and retrieving obligations under which
@@ -155,7 +163,7 @@ module Authorization
       #
       # Example: permit!( :edit, :object => user.posts )
       #
-      if options[:object].respond_to?( :proxy_reflection ) && options[:object].respond_to?( :new )
+      if Authorization.is_a_association_proxy?(options[:object]) && options[:object].respond_to?(:new)
         options[:object] = options[:object].new
       end
       

data/lib/declarative_authorization/in_controller.rb

@@ -170,7 +170,7 @@ module Authorization
       context = object = nil
       if object_or_sym.nil?
         context = self.class.decl_auth_context
-      elsif !object_or_sym.respond_to?(:proxy_reflection) and object_or_sym.is_a?(Symbol)
+      elsif !Authorization.is_a_association_proxy?(object_or_sym) and object_or_sym.is_a?(Symbol)
         context = object_or_sym
       else
         object = object_or_sym

data/lib/declarative_authorization/obligation_scope.rb

@@ -140,9 +140,13 @@ module Authorization
     # Returns the model associated with the given path.
     def model_for (path)
       reflection = reflection_for(path)
-      
-      if reflection.respond_to?(:proxy_reflection)
-        reflection.proxy_reflection.klass
+
+      if Authorization.is_a_association_proxy?(reflection)
+        if Rails.version < "3.2"
+          reflection.proxy_reflection.klass
+        else
+          reflection.proxy_association.reflection.klass
+        end
       elsif reflection.respond_to?(:klass)
         reflection.klass
       else
@@ -167,7 +171,7 @@ module Authorization
 
       reflection = path.empty? ? top_level_model : begin
         parent = reflection_for( path[0..-2] )
-        if !parent.respond_to?(:proxy_reflection) and parent.respond_to?(:klass)
+        if !Authorization.is_a_association_proxy?(parent) and parent.respond_to?(:klass)
           parent.klass.reflect_on_association( path.last )
         else
           parent.reflect_on_association( path.last )
@@ -182,7 +186,7 @@ module Authorization
 
       # Claim alias for join table
       # TODO change how this is checked
-      if !reflection.respond_to?(:proxy_reflection) and !reflection.respond_to?(:proxy_scope) and reflection.is_a?(ActiveRecord::Reflection::ThroughReflection)
+      if !Authorization.is_a_association_proxy?(reflection) and !reflection.respond_to?(:proxy_scope) and reflection.is_a?(ActiveRecord::Reflection::ThroughReflection)
         join_table_path = path[0..-2] + [reflection.options[:through]]
         reflection_for(join_table_path, true)
       end

data/test/model_test.rb

@@ -88,7 +88,11 @@ class TestModelSecurityModel < ActiveRecord::Base
   using_access_control
 end
 class TestModelSecurityModelWithFind < ActiveRecord::Base
-  set_table_name "test_model_security_models"
+  if Rails.version < "3.2"
+    set_table_name "test_model_security_models"
+  else
+    self.table_name = "test_model_security_models"
+  end
   has_many :test_attrs
   belongs_to :test_attr
   using_access_control :include_read => true, 
@@ -209,6 +213,9 @@ class NamedScopeModelTest < Test::Unit::TestCase
       end
     }
     Authorization::Engine.instance(reader)
+    TestModel.delete_all
+    TestAttrThrough.delete_all
+    TestAttr.delete_all
 
     allowed_model = TestModel.create!
     allowed_model.test_attrs.create!(:attr => 1).test_attr_throughs.create!
@@ -383,6 +390,7 @@ class NamedScopeModelTest < Test::Unit::TestCase
       end
     }
     Authorization::Engine.instance(reader)
+    TestModel.delete_all
 
     test_model_1 = TestModel.create!
     TestModel.create!
@@ -911,6 +919,9 @@ class NamedScopeModelTest < Test::Unit::TestCase
         end
       }
       Authorization::Engine.instance(reader)
+      TestModel.delete_all
+      TestAttrThrough.delete_all
+      TestAttr.delete_all
 
       test_attr_through_1 = TestAttrThrough.create!
       test_item = NWayJoinItem.create!
@@ -1061,6 +1072,8 @@ class NamedScopeModelTest < Test::Unit::TestCase
       end
     }
     Authorization::Engine.instance(reader)
+    TestModel.delete_all
+    TestAttr.delete_all
 
     test_model_1 = TestModel.create!
     test_model_2 = TestModel.create!
@@ -1803,6 +1816,9 @@ class ModelTest < Test::Unit::TestCase
       end
     }
     Authorization::Engine.instance(reader)
+    TestModel.delete_all
+    TestAttr.delete_all
+    TestAttrThrough.delete_all
 
     test_model_1 = TestModel.create! :content => 'test_1'
     test_model_2 = TestModel.create! :content => 'test_2'

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 5
-  - 4
-  version: 0.5.4
+  - 5
+  version: 0.5.5
 platform: ruby
 authors: 
 - Steffen Bartsch
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-11-30 00:00:00 +01:00
+date: 2012-01-10 00:00:00 +01:00
 default_executable: 
 dependencies: []