Sí, sí, perdona. Tienes toda la razón :)

On Sun, Mar 24, 2013 at 1:46 AM, Leandro Crisol <notifications@github.com>wrote:

> In app/models/user.rb:
>
> >      @cache_skills ||= Set.new(self.users_skills.find(:all).collect {|s| s.role})
> >      @cache_skills.include?(skill)
> >    end
> >
> > +  def has_skill?(skill)
> > +    if self.users_skills.find(:first, :conditions => ['role = ?', skill]) != nil
>
> En todo caso, ¿no sería así? Solo añado una negación al principio de línea:
>
> !self.users_skills.find(:first, :conditions => ['role = ?', skill]).nil?
>
> Si no recuerdo mal, x.nil? se evalúa a true si x es nil, pero mi condición
> original es true cuando el objeto *no es nil*.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/gamersmafia/gamersmafia/pull/505/files#r3503258>
> .
>



-- 
Juan Alonso