restful_principles.rb

# You know REST... yeah... This is not RESTful

class UsersController < ApplicationController

  def subscribe_mailing_list
    current_user.subscribe(params[:mailing_list])
    redirect_to users_path, :notice => "You've been subscribed"
  end
  
  def unsubscribe_mailing_list
    current_user.unsubscribe(params[:mailing_list])
    redirect_to users_path, :notice => "You have been unsubscribed"
  end

end

# Would be better off doing:


class SubscriptionsController < ApplicationController

  def create
    current_user.subscribe(params[:id])
    redirect_to users_path, :notice => "You've been subscribed"
  end
  
  def destroy
    current_user.unsubscribe(params[:id])
    redirect_to users_path, :notice => "You have been unsubscribed"
  end

end

# It's just better form.