will/cancan
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix namespace split, so we can use / for namespace

Browse Source
This commit is contained in:
Nugroho Herucahyono 2012-06-29 18:27:11 +07:00
parent 6886aecb9a
commit 6c1828acb6
2 changed files with 12 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/cancan/controller_resource.rb
View File

@ -237,7 +237,7 @@ module CanCan
end end
def namespace def namespace
@params[:controller].split("::")[0..-2] @params[:controller].split(/::|\//)[0..-2]
end end
def namespaced_name def namespaced_name

11
spec/cancan/controller_resource_spec.rb
View File

@ -62,6 +62,17 @@ describe CanCan::ControllerResource do
@controller.instance_variable_get(:@project).should == project @controller.instance_variable_get(:@project).should == project
end end
it "has the specified nested resource_class when using / for namespace" do
module Admin
class Dashboard; end
end
@ability.can(:index, "admin/dashboard")
@params.merge!(:controller => "admin/dashboard", :action => "index")
@controller.authorize!(:index, "admin/dashboard")
resource = CanCan::ControllerResource.new(@controller, :authorize => true)
resource.send(:resource_class).should == Admin::Dashboard
end
it "builds a new resource with hash if params[:id] is not specified and authorize on each attribute" do it "builds a new resource with hash if params[:id] is not specified and authorize on each attribute" do
@params.merge!(:action => "create", :project => {:name => "foobar"}) @params.merge!(:action => "create", :project => {:name => "foobar"})
CanCan::ControllerResource.new(@controller, :load => true).process CanCan::ControllerResource.new(@controller, :load => true).process