管理画面トップ 売上状況が9時間ずれる問題修正

[dk-umebius]

概要(Overview・Refs Issue)

#4426 管理画面トップ 売上状況が9時間ずれる問題修正

コミュニティより
https://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=23411&forum=11&post_id=96550#forumpost96550

方針(Policy)

SUBSTRING関数を使わない

テスト（Test)

MySQLとPostgreSQLで動作確認

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[ryo-endo]

@okazy 開発コミュニティにもあるように、意思決定につかう数字の集計不具合なので、bug:lowではないかなー。(基準があれなので判断難しいけどMiddleにしました。)

@dk-umebius ありがとうございます。変更行数があるので、確認に少々お時間かかりそうです。

[okazy]

修正コード確認して処理の前後で処理の意図が変わらないことを確認しました。
また、テストコードを追加しました。

[okazy]

動作確認して問題ありませんでした。
SQLを整理したため、twigへ渡しているパラメータ salesThisMonth, salesToday, salesYesterday 内の order_day が渡らなくなっていますが、twig内で利用しておらず、またカスタマイズもされない部分かと思いますので取り込んで問題ないかと思います。

[okazy]

ありがとうございます！取り込みました。