adding :singular option to support has_one associations in load/authorize resource - closes #93

This commit is contained in:
Ryan Bates
2010-08-06 11:03:10 -07:00
parent 6998e8bdd1
commit 84f4c904b7
3 changed files with 41 additions and 4 deletions

View File

@@ -67,6 +67,9 @@ module CanCan
# [:+through+]
# Load this resource through another one. This should match the name of the parent instance variable.
#
# [:+singular+]
# Pass +true+ if this is a singular resource through a +has_one+ association.
#
# [:+parent+]
# True or false depending on if the resource is considered a parent resource. This defaults to +true+ if a resource
# name is given which does not match the controller.

View File

@@ -41,12 +41,21 @@ module CanCan
def load_resource_instance
if !parent? && new_actions.include?(@params[:action].to_sym)
@params[name] ? resource_base.new(@params[name]) : resource_base.new
elsif id_param
resource_base.find(id_param)
build_resource
elsif id_param || @options[:singular]
find_resource
end
end
def build_resource
method_name = @options[:singular] ? "build_#{name}" : "new"
resource_base.send(*[method_name, @params[name]].compact)
end
def find_resource
@options[:singular] ? resource_base.send(name) : resource_base.find(id_param)
end
def authorization_action
parent? ? :read : @params[:action].to_sym
end
@@ -77,8 +86,13 @@ module CanCan
# The object that methods (such as "find", "new" or "build") are called on.
# If the :through option is passed it will go through an association on that instance.
# If the :singular option is passed it won't use the association because it needs to be handled later.
def resource_base
through_resource ? through_resource.send(name.to_s.pluralize) : resource_class
if through_resource
@options[:singular] ? through_resource : through_resource.send(name.to_s.pluralize)
else
resource_class
end
end
# The object to load this resource through.