Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Issue #50 Typo #51

Merged
merged 1 commit into from

3 participants

@skord

This should fix issue #50 where there's a typo with 'ignorables/ignoreables'.

@travisbot

This pull request passes (merged 8ab714a into 7e390e3).

@davidcelis davidcelis merged commit 4386ac5 into from
@davidcelis
Owner

Thanks, dude. Version bump incoming.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 28, 2012
  1. @skord

    typo breaks ignorables

    skord authored
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/recommendable/acts_as_recommended_to.rb
View
4 lib/recommendable/acts_as_recommended_to.rb
@@ -374,9 +374,9 @@ def ignored
# @return [ActiveRecord::Relation] an ActiveRecord::Relation of records that self has ignored
def ignored_for klass
ids = if klass.sti?
- ignores.joins(manual_join(klass, 'ignore')).map(&:ignoreable_id)
+ ignores.joins(manual_join(klass, 'ignore')).map(&:ignorable_id)
else
- ignores.where(:ignoreable_type => klass.to_s).map(&:ignoreable_id)
+ ignores.where(:ignorable_type => klass.to_s).map(&:ignorable_id)
end
klass.where('ID IN (?)', ids)
Something went wrong with that request. Please try again.