質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.33%

  • Ruby on Rails

    5105questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

rails5:scaffoldでupdateした時のエラー

解決済

回答 1

投稿 2017/02/15 11:22 ・編集 2017/02/15 11:55

  • 評価
  • クリップ 0
  • VIEW 246

0xfee1900d

score 1

前提・実現したいこと

現在 Ruby on RailsでWebアプリケーションを作成しております.
scaffoldを使ってMicropost的なものを作っているのですが,標準の
/micropost/:id
から
/micropost/:token
とIDではなく,ランダムに生成した文字列( token )でアクセスするようにしております.

これで詳細表示と新規作成はできるのですが,編集した時に以下のようなエラーがでており解決できない状態です.

発生している問題・エラーメッセージ

編集画面のフォームからpostするとtokenが入る場所にidが入ってしまいます.その結果以下のようなエラーが出ます.

NoMethodError in MicropostController#update
undefined method `update' for nil:NilClass

pryで見たときはこのようになっております.
編集画面へアクセス

Started GET "/micropost/ql05NTxxxxx/edit" for 10.0.2.2 at 2017-02-15 02:36:34 +0000
Processing by MicropostController#edit as HTML
  Parameters: {"access_token"=>"ql05NTxxxxx"}
  Micropost Load (2.0ms)  SELECT  "micropost".* FROM "micropost" WHERE "micropost"."token" = ? LIMIT ?  [["token", "ql05NTxxxxx"], ["LIMIT", 1]]
  Rendering micropost/edit.html.erb within layouts/application

submitした時

Started PATCH "/micropost/2" for 10.0.2.2 at 2017-02-15 02:36:43 +0000
Processing by MicropostController#update as HTML
  Parameters: {"utf8"=>"✓", "authenticity_token"=>"wb8ni196rHHcoJdMYxDSg1DA==", "micropost"=>{"title"=>"テスト", "content"=>"この投稿はテストです\r\n\r\n追加しました", "status"=>"published"}, "commit"=>"登録", "token"=>"2"

補足情報(言語/FW/ツール等のバージョンなど)

Ruby 2.3.3
Ruby on Rails 5.0.0.1

Controllerにて以下のように設定しております

  private
    # Use callbacks to share common setup or constraints between actions.
    def set_micropost
      @micropost = Micropost.find_by( token: params[ :token ] )
    end

    # Never trust parameters from the scary internet, only allow the white list through.
    def micropost_params
      params.require(:micropost).permit(:title, :content, :status)
    end

new.html.erb, edit.html.erbは以下の通りになっております

<section>
    <h2>フォーム</h2>
    <%= render 'form', micropost: @micropost %>
</section>

_form.html.erb

<%= form_for(micropost) do |f| %>
  <% if micropost.errors.any? %>
    <div id="error_explanation">
      <h2><%= pluralize(micropost.errors.count, "error") %> prohibited this micropost from being saved:</h2>

      <ul>
      <% micropost.errors.full_messages.each do |message| %>
        <li><%= message %></li>
      <% end %>
      </ul>
    </div>
  <% end %>

  <dl>
      <dt><span class="default-pink">必須</span><strong>タイトル</strong></dt>
      <dd>
          <%= f.text_field :title %>
      </dd>
      <dt><span class="default-pink">必須</span><strong>本文</strong></dt>
      <dd>
          <%= f.text_area :content %>
      </dd>
      <dt><span class="default-pink">必須</span><strong>公開状態</strong></dt>
      <dd class="select">
          <%= f.select :status, [ ["公開", "published"], ['非公開', "unpublish"], ["下書き", "draft"] ], :selected => "published" %>
      </dd>
  </dl>
  <%= f.submit "登録", class: "default-pink" %>
<% end %>

この問題を解決したく思います.
皆様のお知恵をお借りできればと思います.よろしくお願い致します.

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+2

解決法

def update
の一行目に以下を追加
@micropost=Micropost.find_by(token: params[:token])
これで動きます。

def set_micropost
@micropost=Micropost.find(params[:id])
end
というのがあれば
def set_micropost
@micropost=Micropost.find_by(token: params[:token])
end

にしてupdate内の
さっき追加した行を消すとよりベターです。

また
config/routes.rbを修正してください。
resouces :micropostsをやめ

post  'micropost/(:token')' => 'micropost#create'
patch  'micropost/:token' => 'micropost#update'

と追記してください。
これでも無理だったら、残念ですが_form.html.erbを
new と editに結合してそれぞれ
form_forを

form_for(@micropost,url: {action: :create,token: @micropost.token})
form_for(@micropost,url: {action: :update,token: @micropost.token})

としてください
と追記してください。
これでも無理だったら、残念ですが_form.html.erbを
new と edit,
_form.html.erbの form_forを

render partial: 'form',locals: {micropost: @micropost,url: {action: :create,token: @micropost.token}}

render partial: 'form',locals: (micropost:@micropost,url: {action: :update,token: @micropost.token})

form_for(@micropost,url: url)

それぞれとしてください

投稿 2017/02/15 11:38

編集 2017/02/15 12:33

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/02/15 11:42

    ご回答ありがとうございます.
    質問に書き忘れておりましたが,すでにご回答頂いたようにset_micropostで設定しております.

    pryで値を見ましたがやはりtokenの部分にidが入ってしまいます…

    キャンセル

  • 2017/02/15 11:50

    そうすると問題は_form.html.erbですね

    キャンセル

  • 2017/02/15 11:59

    何度もご回答いただきありがとうございます.
    _form.html.erbは質問に追記したとおりで,生成されたもののhtmlタグを変更したのみです.
    どこを修正すればいいかさっぱり見当がつきません.お手数おかけしますが教えていただけるとありがたいです

    キャンセル

  • 2017/02/15 12:34 編集

    config/routes.rb
    を修正してください。
    resouces :micropostsをやめ
    post 'micropost/(:token')' => 'micropost#create'
    patch 'micropost/:token' => 'micropost#update'
    と追記してください。

    キャンセル

  • 2017/02/15 12:31

    ありがとうございます!
    routes.rbを変更してもうまく動作しませんでしたが,ご回答に追記していただいた通りフォームをそれぞれで分ける方法でうまくいきました.

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

ただいまの回答率

91.33%

関連した質問

同じタグがついた質問を見る

  • Ruby on Rails

    5105questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。