From 6c497b8dd91e1d1c9e7014f65df6f366dc4c6595 Mon Sep 17 00:00:00 2001 From: Steven Anderson Date: Wed, 20 Jul 2011 09:31:53 +0100 Subject: [PATCH] Added support for engines and namespaced models. --- lib/cancan/controller_resource.rb | 8 +++++++- spec/cancan/controller_resource_spec.rb | 12 ++++++++++++ 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/lib/cancan/controller_resource.rb b/lib/cancan/controller_resource.rb index 6c41efd..8d4e367 100644 --- a/lib/cancan/controller_resource.rb +++ b/lib/cancan/controller_resource.rb @@ -134,7 +134,7 @@ module CanCan def resource_class case @options[:class] when false then name.to_sym - when nil then name.to_s.camelize.constantize + when nil then namespaced_name.to_s.camelize.constantize when String then @options[:class].constantize else @options[:class] end @@ -203,6 +203,12 @@ module CanCan @name || name_from_controller end + def namespaced_name + @params[:controller].sub("Controller", "").singularize.constantize + rescue NameError + name + end + def name_from_controller @params[:controller].sub("Controller", "").underscore.split('/').last.singularize end diff --git a/spec/cancan/controller_resource_spec.rb b/spec/cancan/controller_resource_spec.rb index 7033c04..0b052b4 100644 --- a/spec/cancan/controller_resource_spec.rb +++ b/spec/cancan/controller_resource_spec.rb @@ -35,6 +35,18 @@ describe CanCan::ControllerResource do @controller.instance_variable_get(:@project).should == project end + it "should attempt to load a resource with the same namespace as the controller when using :: for namespace" do + module MyEngine + class Project < ::Project; end + end + + project = MyEngine::Project.create! + @params.merge!(:controller => "MyEngine::ProjectsController", :action => "show", :id => project.id) + resource = CanCan::ControllerResource.new(@controller) + resource.load_resource + @controller.instance_variable_get(:@project).should == project + end + it "should properly load resource for namespaced controller when using '::' for namespace" do project = Project.create! @params.merge!(:controller => "Admin::ProjectsController", :action => "show", :id => project.id)