Unfollowing now works.

steveklabnik · steveklabnik · commit e7630a1f4eb4 · 2010-08-14T20:45:48.000-04:00
You can unfollow people. Sweet.
diff --git a/controllers/hacker.rb b/controllers/hacker.rb
@@ -29,6 +29,25 @@
 
 end
 
+#this lets you unfollow a Hacker
+get '/hackers/:name/unfollow' do
+	#we have to be logged in to unfollow someone
+	require_login! :return => "/hackers/#{params[:name]}/unfollow"
+
+	#find the hacker with the given name
+	@hacker = Hacker.first(:username => params[:name])
+
+	#follow them!
+	current_user.unfollow! @hacker
+
+	#set a message
+	flash[:notice] = "No longer following #{params[:name]}."
+
+	#redirect back to your page!
+	redirect "/hackers/#{current_user.username}"
+
+end
+
 #this lets us see followers
 get '/hackers/:name/followers' do
 	#find the hacker with the given name
diff --git a/features/hacker.feature b/features/hacker.feature
@@ -54,3 +54,12 @@ Feature: Hacker management
 		When I go to the hacker page for "steve"
 		And I follow "1"
 		Then I should see "fela"
+	Scenario: Unfollowing a Hacker
+		Given I'm logged in as "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 "Unfollow fela"
+		Then I should be on the hacker page for "steve"
+		And I should see "No longer following fela."
+		And I should see "Following: 0"
diff --git a/models/hacker.rb b/models/hacker.rb
@@ -82,6 +82,14 @@ def follow! followee
 		followee.followers << self
 		followee.save
 	end
+	
+	#this function makes the hacker unfollow the followee
+	def unfollow! followee
+		following_ids.delete(followee.id)
+		save
+		followee.followers_ids.delete(id)
+		followee.save
+	end
 
 	private 
 
diff --git a/views/hackers/show.haml b/views/hackers/show.haml
@@ -7,3 +7,4 @@
 - if logged_in? && @hacker.username != current_user.username
 	%a{:href => "/messages/new/to/#{@hacker.username}" }= "Send #{@hacker.username} a message"
 	%a{:href => "/hackers/#{@hacker.username}/follow"} Follow #{@hacker.username}
+	%a{:href => "/hackers/#{@hacker.username}/unfollow"} Unfollow #{@hacker.username}
