ansaurus

Question

Howto: "users who have visited your profile..."

Answer 1

+1 A:

Why not roll your own?

class User < ActiveRecord::Base
  has_many :user_visits
  has_many :visitors, :through => :user_visits, :source => :visitor
end

class UserVisit < ActiveRecord::Base
  # user_id :integer
  # visitor_id :integer
  # visited_at :datetime

  belongs_to :user
  belongs_to :visitor, :foreign_key => 'visitor_id', :class_name => 'User'
end


User.first.visitors = [user_1, user2]

You could use STI in you're going to have that many *vists join tables. YMMV as it complicates things such as caching. I'd rather having a little "wetness" on the code side if it makes for a faster experience.

Matt Darby 2009-03-13 16:42:03

Alright. I will skip STI for as it might get to complicated. I was just wondering why the "has_many :visitors, :through => :user_visits" is nessary? Would User.user_visits not return the same result?

atmorell 2009-03-14 01:10:16

User.user_visits will return a UserVisit object, not a User object that actually visited the User's page.You don't *need* it, but it's much cleaner with it; otherwise you'd have to do: @user.user_vists.first.user instead of @user.visitors.first

Matt Darby 2009-03-14 02:42:23

Answer 2

A:

Hmmm this is really giving me some problems:

User.first.visitors
ActiveRecord::HasManyThroughSourceAssociationNotFoundError: 
Could not find the source association(s) :visitor or :visitors in model UserVisit.  
Try 'has_many :visitors, :through => :user_visits, :source => <name>'.  
Is it one of :user?

I tried to add :source => :user to my user model. However then I only get the owner when doing user.visitors.

This is my migration:

class CreateUserVisits < ActiveRecord::Migration
    def self.up
     create_table :user_visits do |t|
      t.integer :user_id
      t.integer :visit_id
     end
    end

    def self.down
    drop_table :user_visits
    end
end

Any thoughts?

atmorell 2009-03-15 18:00:31

If you want information about the visit, then you'll need to work with @user.user_vists. @user.visitors will only return Users that visited the original User.

Matt Darby 2009-03-15 18:53:54

Matt, Active Record gives me the error above without :source => :user added to has_many :visitors,.... When I do user1.visitors each visit record is just the owner and not the visitor itself. user1.visitors => [user1, user1, user1]

atmorell 2009-03-15 20:42:06

Look at my original answer, I updated it to correct an association.

Matt Darby 2009-03-15 21:10:39

Much better :) I was wondering how you would create a new user_visit? I would just do @user.user_visits.build(:visitor_id => current_user.id).save but maybe there is a cleaner way? Should I delete my last answer as all problems has been sorted out in your first answer?

atmorell 2009-03-16 08:16:32

I added a few methods to my user model: http://github.com/mbleigh/acts-as-readable/blob/37cd25589fd85fc643ffacd32b131e52f549bb60/lib/acts_as_readable.rb Now I can do user1.visited_by!(user2) etc

atmorell 2009-03-16 09:54:57

ansaurus

tags:

views:

answers:

Howto: "users who have visited your profile..."

related questions