rails4 排他処理について

Question

フラグがたって編集不可能になったアイテムに、ユーザーがURL直打ちでアクセスしたときに
リダイレクトさせたいです。通常のリダイレクト処理はわかるのですが、
フラグがたった場合にリダイレクトさせる処理はどう書くのか教えてください

現在はフラグが立っていないアイテムのみ@itemに入れています。

def update
    @item = current_user.items.find_by(id: params[:id],flag: false)
    respond_to do |format|
      if @item.update(item_params)
        format.html { redirect_to @item, notice: 'アイテムを編集しました。' }
        format.json { render :show, status: :ok, location: @item }
      else
        format.html { render :edit }
        format.json { render json: @item.errors, status: :unprocessable_entity }
      end
    end
  end

Answer 1

def update
  @item = current_user.items.find_by(id: params[:id])
  if @item.flag
    redirect_to items_path
    return
  end
  # 残りの更新処理
end

でしょうか。

Comments

[smith]

ありがとうございます。
下記のようにしたら、うまくいきましたが
通常はupdateに書くべきなのでしょうか？

def edit
@item = current_user.items.find_by(id: params[:id])
if @item.trade
redirect_to items_path
end
end

[smith]

先ほどのコードちょっと修正します。
if @item.flag正しくはこっちでした。

[hello-world]

そうですね。URLを直接とのことなので、editに書くのがふさわしいと思います。

[smith]

ありがとう御座います。