@@ -62,9 +62,11 @@ module Authorization def roles_for( authorizable_obj ) if authorizable_obj.is_a? Class - self.roles.select { |role| role.authorizable_type == authorizable_obj.to_s } + self.roles.find(:all, :conditions => { :authorizable_type => authorizable_obj.to_s}) elsif authorizable_obj - self.roles.select { |role| role.authorizable_type == authorizable_obj.class.base_class.to_s && role.authorizable.id == authorizable_obj.id } + self.roles.find(:all, :conditions => { + :authorizable_type => authorizable_obj.class.base_class.to_s, + :authorizable_id => authorizable_obj.id }) else self.roles.select { |role| role.authorizable.nil? } end lib/publishare/object_roles_table.rb 139b6d5794736daab6a7f6483def2e358856c3de Tim Wong timlikesdogs@gmail.com http://github.com/DocSavage/rails-authorization-plugin/commit/09ce50e2f35e7fd74f5298ca9f99aca7ab770acc 09ce50e2f35e7fd74f5298ca9f99aca7ab770acc 2009-05-11T15:26:12-07:00 2009-04-06T16:22:17-07:00 Don't use Array#select, so that has_no_roles and has_no_roles_for will work. This also prevents the active record objects from being instantiated prematurely. Signed-off-by: Glenn Rempe <glenn@rempe.us> 5bc5aa812b213f93619955b6fa1b95e72104ce67 Glenn Rempe glenn@rempe.us