diff --git a/lib/cancan/controller_resource.rb b/lib/cancan/controller_resource.rb index 6c41efd..06a8d15 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.camelize.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)