これ`Plan`の`pdf_name`って固定なんですか? Controllerじゃなくてmodelに置くべき処理な気がします。
``` ruby
class Plan
  def pdf_name
    "#{id}.pdf"
  end
end
```