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

Merge pull request #675 from bukalapak/master

Browse Source 
porting #668 to master branch
This commit is contained in:
Ryan Bates 2013-05-03 16:15:31 -07:00
commit 60cf6a67ef
2 changed files with 11 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

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

10
spec/cancan/controller_resource_spec.rb
View File

@ -67,6 +67,16 @@ describe CanCan::ControllerResource do
@controller.instance_variable_get(:@project).should == project
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")
resource = CanCan::ControllerResource.new(@controller, :authorize => true)
resource.send(:resource_class).should == Admin::Dashboard
end
it "should build a new resource with hash if params[:id] is not specified" do
@params.merge!(:action => "create", :project => {:name => "foobar"})
resource = CanCan::ControllerResource.new(@controller)