Refactor associations

[excid3]

I noticed this could be refactored into a standard Rails association with a custom query:
https://github.com/mikeh-dev/KartCalendar/blob/72baf6065bf0fd4d02216f7a2bf721510db23fcf/app/models/user.rb#L25

Before:

  def followed_championships
    Championship.joins(:follows).where(follows: { user_id: id })
  end

After

has_many :followed_championships, ->(user) { joins(:follows).where(follows: {user_id: user.id}) }

[mikeh-dev]

After a little stumbling with your suggestion, I went with this, which still seemed to get the job done a little more succinctly than I was before.

Probably due to a fault on my behalf, I was running into an error with your suggestion as it was looking for a user_id in events.

Anyway, this works for me and I can now just user current_user.followed_events in the view. I guess your approach would have enabled me to not even use current_user as it would have already been assigned in the association but I couldn't work it out.

  has_many :followed_events, through: :follows, source: :followable, source_type: 'Event'

I think I can also now remove my event_users table and just use the follows table.

Thanks again Chris!