[データベース]詳細閲覧時にデータの更新日が更新される不具合を修正しました

[gakigaki]

概要

データベースの詳細画面を閲覧した際に、意図せず最終更新日が現在の日時で上書きされてしまう不具合を修正しました。

変更の目的

「更新日型」として設定された項目が、データの詳細を閲覧するだけで毎回更新されてしまう問題を解決するため。本来、この項目はデータが実際に編集された際にのみ更新されるべきです。

変更内容

証跡用の更新日(updated_at)と、画面表示用の最終更新日を分離することで対応しました。

• databases_inputsテーブルに、最終更新日を管理するためのlast_col_updated_atカラムを追加しました。
• 項目の更新時に、Observerを利用してlast_col_updated_atを更新するようにしました。
• 画面やCSVに出力する更新日型のデータを、従来のupdated_atから新設したlast_col_updated_atに変更しました。

テスト

以下の手順で、修正が正しく反映されていることを画面上で確認しました。

1. 「更新日型」の項目を持つデータベースを用意し、データを登録します。
2. 登録したデータの詳細画面を閲覧します。
3. 一覧画面に戻り、最終更新日が閲覧した日時に変わっていないことを確認します。
4. データを編集して保存し、一覧画面で最終更新日が正しく更新されていることを確認します。
5. CSVを出力し、「更新日型」の項目が画面と同様の値になっていることを確認します。

特記事項

• NC2およびNC3からの移行プログラムは、今回の修正範囲外です。

レビュー完了希望日

関連Pull requests/Issues

参考

DB変更の有無

有り

チェックリスト

• プルリクエストにわかりやすいタイトルとラベルを付けました。https://github.com/opensource-workshop/connect-cms/wiki/Pull-requests-Rule

[masaton0216]

@gakigaki
対応お疲れ様です。

1件コメントというか検討したい内容です。
明確に答えを持ち合わせてないのですが、一時保存の場合も更新されてしまうのはどうすべきかな？と。
更新日は、閲覧者に対する「登録内容の更新お知らせ」的な意味合いがあるかと思うので、そう考えると一時保存時はまだ閲覧者に対して変更内容を見せていない段階の為、更新日も更新しない、が正しいような気もしますが、どうでしょ？

[gakigaki]

@masaton0216
レビューありがとうございます。

一時保存については当改修内容で問題ないという認識です。
データベースプラグインにおいて、一時保存状態のデータは一般ユーザーから見えなくなります。
よって、一般ユーザーは一時保存状態の更新日を確認できないためです。

[masaton0216]

確認ありがとうございます。マージします。