Follower page exists, plus refactoring.

steveklabnik · steveklabnik · commit 42ef0f96a019 · 2010-08-14T20:00:00.000-04:00
Added a follow! method to hacker, this takes another Hacker and follows
them.
diff --git a/controllers/hacker.rb b/controllers/hacker.rb
@@ -19,8 +19,7 @@
 	@hacker = Hacker.first(:username => params[:name])
 
 	#follow them!
-	Hacker.push_uniq(current_user.id, :following => @hacker.id)
-	Hacker.push_uniq(@hacker.id, :followers => current_user.id)
+	current_user.follow! @hacker
 
 	#set a message
 	flash[:notice] = "Now following #{params[:name]}."
@@ -29,3 +28,12 @@
 	redirect "/hackers/#{current_user.username}"
 
 end
+
+#this lets us see followers
+get '/hackers/:name/followers' do
+	#find the hacker with the given name
+	@hacker = Hacker.first(:username => params[:name])
+
+	#render our page
+	haml :"hackers/followers"
+end
diff --git a/features/hacker.feature b/features/hacker.feature
@@ -40,3 +40,10 @@ Feature: Hacker management
 		And I should see "Following: 1"
 		And I go to the hacker page for "fela"
 		And I should see "Followers: 1"
+	Scenario: Seeing Followers
+		Given there's a hacker with the username "steve"
+		And there's a hacker with the username "fela"
+		And the hacker "steve" is following the hacker "fela"
+		When I go to the hacker page for "fela"
+		And I follow "1"
+		Then I should see "steve"
diff --git a/features/step_definitions/hacker_steps.rb b/features/step_definitions/hacker_steps.rb
@@ -0,0 +1,5 @@
+Given /^the hacker "([^"]*)" is following the hacker "([^"]*)"$/ do |u1, u2|
+	follower = Hacker.first(:username => u1)
+	followee = Hacker.first(:username => u2)
+	follower.follow! followee
+end
diff --git a/features/step_definitions/user_steps.rb b/features/step_definitions/user_steps.rb
diff --git a/models/hacker.rb b/models/hacker.rb
@@ -16,11 +16,13 @@ class Hacker
 	#this is a flag to let us know if this Hacker can administrate the site or not.
 	key :admin, Boolean, :default => false
 
-	#the list of emails this user is following
-	key :following, Array, :typecast => 'ObjectId'
+	#the list of hackers this hacker is following
+	key :following_ids, Array
+	many :following, :in => :following_ids, :class_name => 'Hacker'
 
-	#the list of emails of users that are following this user
-	key :followers, Array, :typecast => 'ObjectId'
+	#the list of hackers that are following this hacker
+	key :followers_ids, Array
+	many :followers, :in => :followers_ids, :class_name => 'Hacker'
 
 	#we don't want to store the password (or the confirmation), so we just make an accessor
 	attr_accessor :password, :password_confirmation
@@ -70,6 +72,14 @@ def gravatar_url
 		"http://www.gravatar.com/avatar/#{MD5::md5(email.downcase)}"
 	end
 
+	#this function makes the hacker follow the followee
+	def follow! followee
+		following << followee
+		save
+		followee.followers << self
+		followee.save
+	end
+
 	private 
 
 	#we're going to use the SHA1 encryption method for now.
diff --git a/views/hackers/followers.haml b/views/hackers/followers.haml
@@ -0,0 +1,3 @@
+%ul
+  - @hacker.followers.each do |follower|
+    %li= follower.username
diff --git a/views/hackers/show.haml b/views/hackers/show.haml
@@ -2,7 +2,7 @@
 %p
 	%img{:src => @hacker.gravatar_url}
 %p Following: #{@hacker.following.count}
-%p Followers: #{@hacker.followers.count}
+%p Followers: <a href="/hackers/#{@hacker.username}/followers">#{@hacker.followers.count}</a>
 
 - if logged_in? && @hacker.username != current_user.username
 	%a{:href => "/messages/new/to/#{@hacker.username}" }= "Send #{@hacker.username} a message"

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+%ul`
	`2`	`+ - @hacker.followers.each do \|follower\|`
	`3`	`+ %li= follower.username`