friendship = Friendship.new
friendship.befriender = current_user
friendship.befriendee = @new_friend
- if current_user.is_friends_with?(@new_friend)
- flash[:warning] = t "friendships.make_friend.already_a_friend", :name => @new_friend.display_name
+ if current_user.friends_with?(@new_friend)
+ flash[:warning] = t ".already_a_friend", :name => @new_friend.display_name
+ elsif current_user.friendships.where("created_at >= ?", Time.now.utc - 1.hour).count >= current_user.max_friends_per_hour
+ flash.now[:error] = t ".limit_exceeded"
elsif friendship.save
- flash[:notice] = t "friendships.make_friend.success", :name => @new_friend.display_name
- Notifier.friendship_notification(friendship).deliver_later
+ flash[:notice] = t ".success", :name => @new_friend.display_name
+ UserMailer.friendship_notification(friendship).deliver_later
else
- friendship.add_error(t("friendships.make_friend.failed", :name => @new_friend.display_name))
+ friendship.add_error(t(".failed", :name => @new_friend.display_name))
end
- if params[:referer]
- redirect_to params[:referer]
- else
- redirect_to user_path
- end
+ referer = safe_referer(params[:referer]) if params[:referer]
+
+ redirect_to referer || user_path
end
else
render_unknown_user params[:display_name]
if @friend
if request.post?
- if current_user.is_friends_with?(@friend)
+ if current_user.friends_with?(@friend)
Friendship.where(:befriender => current_user, :befriendee => @friend).delete_all
- flash[:notice] = t "friendships.remove_friend.success", :name => @friend.display_name
+ flash[:notice] = t ".success", :name => @friend.display_name
else
- flash[:error] = t "friendships.remove_friend.not_a_friend", :name => @friend.display_name
+ flash[:error] = t ".not_a_friend", :name => @friend.display_name
end
- if params[:referer]
- redirect_to params[:referer]
- else
- redirect_to user_path
- end
+ referer = safe_referer(params[:referer]) if params[:referer]
+
+ redirect_to referer || user_path
end
else
render_unknown_user params[:display_name]