レビューの「作成日時」に編集日時が反映される

[imamura623]

■現象

レビューを編集すると、レビューの「作成日時」がレビューの編集日時に更新されてしまう。

■原因

レビューの編集時にレビューの作成日時を更新していた。

■修正内容

2.10/2.12

更新日時はDBに記録しないようにして、作成日時が更新されないようにした。

2.13

更新日時には新たにDBに作成したu_datetimeカラムに保存させることで、作成日時を保存するu_datetimeが編集時に更新されないようにした。

■関連情報

元のDescription

​http://sns.openpne.jp/?m=pc&a=page_fh_diary&target_c_diary_id=17988 より転記

マイレビューを編集したら、
「レビューの作成日時」
が現在のものに変わりました。

こういう仕様が正しいんですかね？
作成日時と編集日時は別にしておかないと、なんかおかしい気がするんですけど…。

[nakasone]

以下のような確認を行ったところ再現できました。

■確認ブラウザ

• !FireFox2

■確認バージョン

• 2.10.x
• 2.12.x
• trunk

■手順

1. マイレビュー→レビューの編集
2. レビューの作成日時が編集した日時に更新されている

[asma]

このチケットの対応として、c_reviewテーブルにr_datetimeしか無いため 1)コードだけを変更して、更新日時はDBに記録されない方法 2)DBにu_datetimeカラムを追加して、作成日時、更新日時の両方がDBに記録される方法

のどちらで対応するべきでしょうか？ もしくは、他に対応方法があるのでしょうか？ 海老原昂輔 18:22 安定版は 1), 開発版は 2) という形で対応するのはどうでしょう。

まあテンプレート側を「更新日時」にするという手もあると思いますけどね。 あずま(masayuki-as) 18:27 なるほど、でも、更新日時のみを表示しているところあります？ 18:28 なければ、

安定版は 1), 開発版は 2) という形で対応

でいこうと思います。

[asma]

• r7838 2.12
• r7839 2.10

対応。更新の際、更新日時でr_datetimeは更新されない

[asma]

• r7847,r7846 trunk

DBに更新日時を記録するように対応。表示は文言通り、作成日時に。

[kiwa]

descriptionを wiki:pne-guidelines のフォーマットにあわせてください

[kiwa]

descriptionを編集しました。

[kiwa]

r7992にて、2.13用のupdate sqlのファイル名が「～for2.13.1～」になっていたのを「～for2.13.2～」に修正しました。

[ebihara]

• update用SQLのファイル名にスペルミスがあります。 culumn ではなく、 column ではないでしょうか
• このupdate用SQLだと、追加した u_datetime カラムの値がデフォルト値の "0000-00-00 00:00:00" となってしまいます。 r_datetime と等価にするべきではないでしょうか
• PostgreSQL版update用SQLが以下の点で適切でないように思えます。
  • datetime 型ではなく timestamp 型ではないでしょうか
  • デフォルト値は "0000-01-01 00:00:00" ではないでしょうか

[ebihara]

以下のリビジョンで修正しました。ご確認ください。

• SQLのファイル名を修正： r8013
• PostgreSQL版update用SQLの構文が間違っていたのを修正： r8014
• updateSQL実行後に r_datetime と u_datetime の値が等しくなるようにした： r8015

[ogawa]

r8026 で2.12.xのインデントとスペースの修正をしました。

[ogawa]

2.13系のコードについて、以下の点が問題ないか確認してください。

• レビュー作成時に u_datetime が更新されていない
• h_home のマイフレンド最新レビューなどの表示系では u_datetime ではなく、r_datetime を使ったままになっている

[ebihara]

r8034 でレビュー作成時に u_datetime の更新をおこなっていなかったのを修正しました。

h_home のマイフレンド最新レビューなどの表示系では u_datetime ではなく、r_datetime を使ったままになっている

については、どの場面においても r_datetime を使用するのが適当であると判断し修正しません。

[kiwa]

確認しました。問題ありません。update sqlのチェックは2.13.２の動作テスト期間に行います。

#2947 2.13.2で追加されたupdate sqlの確認を行う