Ruby On Rails 論理的削除 deleted_flg

Question

###前提・実現したいこと
こんにちは。
Ruby On Rails で物理的削除から論理的削除に変えたいと思っています。

色々なサイトを見たりしながらやっているんですがなかなかうまくいかず...
初心者ですが何卒よろしくお願い申し上げます。

Comments

[IPU]

失敗している箇所のエラーログを貼ってください。多分、もっと手前の部分で失敗しているはずです。

[fkryu]

エラーは出ていない状態でいます（ ; ; ）showページで投稿の削除ボタンを押したらエラー等は出ず、削除もされないといった状態になっています。どこか間違っているのでしょうか。。

Answer 1

modelに delete_flag == false のrecordのみを取得する scope を追加し
controller（#index, #showなど）で利用するのはいかがでしょう?

例）
model

class Hoge < ActiveRecord::Base
  scope :availables, -> { Hoge.where(deleted_flg: false)}
end

controller

class HogeController
  def index
    @hoges = Hoge.availables
    respond_to do |format|
      ..............
    end
  end

  def show
    @hoge = Hoge.availables.find(params[:id])
    respond_to do |format|
      ..............
    end
  end
end

Comments

[fkryu]

ご返事ありがとうございます。

showページで削除をして見たら

ActiveRecord::StatementInvalid in BuildingsController#show

Mysql2::Error: Unknown column 'buildings.delete_flg' in 'where clause': SELECT `buildings`.* FROM `books` WHERE `buildings`.`delete_flg` = 0 AND `buildings`.`id` = 8 LIMIT 1
と出てしまいました。。

[satoshih]

buildings テーブルに delete_flg というカラムが無いというエラーだと思いますが、
migrationは正しく行えてますか？

[fkryu]

はい。
現段階delete_flgのカラムが増え、editで削除したらdelete_flgが1になりindexでは無事表示できているのですが、showアクションで表示ができずにいます。。

[fkryu]

Couldn't find Building without an ID
と出てしまいます。
buildingのidではなくuser_idをfindできていないからだと思うのですが...

[satoshih]

Couldn't find Building without an ID
は find(id) のidに integer以外が渡された時に起きると思うのですが、findに何を渡してますか？

[fkryu]

@buildings = Building.availables.find(params[:user_id])
と記述しています。。

[satoshih]

params[:user_id]には何が入ってますか？
また、Buildingのidで検索しないのであれば、
Building.availables.find_by(user_id: params[:user_id])
のようにしてみてください。

[fkryu]

user_idカラムの数字(8)が入っています。
上記のコードを記述すると情報は表示されるのですが、deleted_flgが1のレコードも表示されてしまいます。。

[satoshih]

deleted_flg は booleanじゃないんですね。。。。
回答に記載したソースはbooleanのつもりで書いてますので、
scopeで定義している where(条件) の条件で正しい値が取れるかどうか、
rails consoleなどで確認してみてください。

[fkryu]

class AddDeleteFlgToBuildings < ActiveRecord::Migration
def change
add_column :buildings, :delete_flg, :boolean, null: false, default: false
end
end

でmigrateしているのでbooleanになっていると思っていたのですが。。

[satoshih]

すいません。MySQLの仕様のようですね。boolean == tinyint(1) のようです。
deleted_flg: false
rails consoleで実行して正しい結果は返ってきませんか？

もし返ってこないようでしたら、 where(deleted_flg: false) をwhere(deleted_flg: 0) にするなど
試してみてください

[fkryu]

すみません。MySQLです。
irb(main):001:0> deleted_flg: false
SyntaxError: (irb):1: syntax error, unexpected ':', expecting end-of-input
deleted_flg: false
......................................
となります。

[satoshih]

rails console で実行する場合は
Building.where(deleted_flg: false)
で試してみてください

[fkryu]

ありがとうございます！
Building.where(deleted_flg: false)の時もBuilding.where(deleted_flg: 0)の時も、削除していないレコード(deleted_flgが0)が表示されています。

[satoshih]

すいません。回答のソースがtypoしてました
scope :availables, -> { Hoge.where(delete_flg: false)}
ではなく
scope :availables, -> { Hoge.where(deleted_flg: false)}
でした

（回答は修正済みです）

[fkryu]

わざわざありがとうございます。
はい。そちらで記述しています！
consoleは問題ないのでしょうか？？

[satoshih]

consoleの結果がdeleted_flgがfalseのものだけが出力されているのでしたら、条件は問題無いです
あとはモデルのscopeが
scope :availables, -> { Building.where(deleted_flg: false) }
になっていれば動くはずです。試してみてください
（最初に書いたscopeは間違っていましたので修正してます）

[fkryu]

ありがとうございます。
エラーはなく表示されるのですが、deleted_flgが0のレコードも1のレコードも全て表示されてしまいます。。
0のレコードだけを表示させるようにはできないでしょうか。。

[satoshih]

状況を整理したいのですが、
rails consoleで Building.where(deleted_flg: false) とした場合の戻り値は期待する値が返ってきていますか？(y/n)
rails console で返ってきた値とcontrollerで検索した値が一致しないですか？(y/n)
BuildingsControllerでBuildingのdeleted_flgがfalseかつ、 paramsのuser_idが設定されているのレコードを検索したい?(y/n)

[fkryu]

はい。
戻り値は理想通りに返ってきています。Y
consoleではdeleted_flgが0のレコードだけなのですが、localhost:3000ではdeleted_flgが1のレコードも表示されてしまいます。Y

はい。deleted_flgが0のレコードでidが設定されているレコードを検索できればとおもいます。Y

[satoshih]

controller経由で取得した際に debug logにSQLコマンドが出るかと思うのですが、
そのSQLコマンドを貼っていただくことは可能ですか？

[fkryu]

すみません。初心者であまりよくわからないのですがどう操作すれば良いのですか？

[fkryu]

controllerで使用するavailablesはメソッドの前につけるイメージでindexアクションはできるようになったのですが、showアクションは最初は
def show
end
だけで何も記述していなかったんです。

[satoshih]

Controllerに before_filter などで 指定されているメソッドがあると思います。
そのメソッドで Buildingを検索するようになっていませんか？

[fkryu]

おそらくshowアクションには何も書けてないと思います。。
初めの質問の最後に画像を添付しましたので確認していただけるとありがたく思います。。

[satoshih]

viewで使用している 変数が @building になっていませんか？
controllerで @buildings （末尾にsがついてる）に値を入れているので、見れていないかもしれないです

[fkryu]

なっています！！
buildingsにすればいいのですか？？

[satoshih]

controllerでセットするモデルは１つだと思うので、
controller側を @building に変更する方がいいかと思います。

[fkryu]

def show
@building = Building.availables.find_by(user_id: params[:user_id])
end
に変更して、エラーは出ないのですが、やはりdeleted_flgが1のレコードも表示されてしまいます。

[satoshih]

rails consoleで
Building.availables.find_by(user_id: params[:user_id])
を実行して、期待する値が取得できているか確認してみていただいてもいいですか？
（params[:user_id] はUserのidに置き換えてください）

[fkryu]

Building.availables.find_by(user_id: params[8])で実行しましたがエラーになってしまいます。。。

[satoshih]

Userのidが 8 なら
Building.availables.find_by(user_id: 8)
で実行してみてください

[fkryu]

何から何まですみません。。
deleted_flg: falseになっているレコードだけが表示されています。

[satoshih]

了解です。
ではcontrollerでセットした値がどこかで上書きされてると思います。
一旦 viewとcontroller の @building を @buildings に変更すると動くかどうか確認していただけますか？

[fkryu]

@building を @buildings に変更して表示されましたが、内容は変わらずです。。

[fkryu]

変更した内容を画像で添付いたしました。

[satoshih]

viewの

current_user.buildings.each
を
current_user.buildings.availables.each
もしくは
@buildings.each
に変更してみてください

[fkryu]

できました！！
かなりいろいろなことがわかり勉強になりました！！
本当にありがとうございます！

Answer 2

gem 'kakurenbo' を使うのではダメでしょうか？
カラムにdeleted_at:datetimeを追加するだけで必要な処理は全てやってくれます。
参考

Answer 3

論理削除なので実際には削除されていないが削除されたように見せかけたいということですよね？
論理削除されたデータはdeleted_flgがtrueになっているので、
deleted_flgがtrueのレコードは画面に表示させないようViewの修正が必要なのではないでしょうか。

Comments

[fkryu]

はい。そうです。
Viewの修正も必要なのですね。
あまりイメージができずにいます。。

[fkryu]

データベースの値を確認したら削除ボタンを押したidの
レコードのeleted_flgが1になってました。

[hn1]

あ、satoshihさんの回答にあるように取得部分で論理削除データを弾いておけばViewの修正は不要でした。
そっちの方がスマートだと思います。

[fkryu]

ありがとうございます。頑張って見ます。。。