Kentaro Ohkouchi

[4.2] monolog.yml に stream を追加

3

## 概要(Overview・Refs Issue) 本番環境の出力先は stream が推奨されている https://symfony.com/doc/5.4/logging.html#where-logs-are-stored ## 方針(Policy) monolog.yml に stream を追加 ## 実装に関する補足(Appendix) `APP_ENV=dev` の場合は [symfony/cli](https://symfony.com/doc/current/setup/symfony_server.html) で実行すると、 SQLの実行内容も確認できる ## テスト（Test) symfony/cli にてログ出力を確認 ## 相談（Discussion） ## マイナーバージョン互換性保持のための制限事項チェックリスト - [...

## 概要(Overview) - 店舗所在地、お客様情報共に、郵便番号は出力されない - 2系の納品書には出力されていた ## 期待する内容(Expect) or 要望 (Requirement) 店舗所在地、お客様情報に郵便番号が出力される ## 再現手順(Procedure) 納品書出力する ### 環境 (environment) + EC-CUBE: 4.0.5 + PHP: 7.4.5 ## 関連情報 (Ref) - #4652

外税表示

5

現在は総額表示のみの対応だが、外税表示にも切り替えられるようにしたい

## 概要(Overview) 値引の明細(OrderItem)を追加した際、購入確認画面等で `-￥100` という表示になってしまう。 ## 期待する内容(Expect) or 要望 (Requirement) `￥-100` としたい。 こんな感じで修正できるが、ロケールごとにフォーマットが異なるので、いまいちな感じ ```diff --- a/src/Eccube/Twig/Extension/EccubeExtension.php +++ b/src/Eccube/Twig/Extension/EccubeExtension.php @@ -190,6 +190,10 @@ class EccubeExtension extends \Twig_Extension $currency = $this->app['config']['currency']; $formatter...

インデックスを効果的に利用できるように管理画面のマルチ検索窓と詳細検索の改善

1

現在のマルチ検索窓では、部分一致のため、インデックスが利用できない。 特に、商品マスタ検索画面では、詳細検索項目のフィールドが少ないため、更に不便。 以下のように、用途に応じて検索パターンを限定することにより、インデックスを付与した際に効果的に利用できると思われる - マルチ検索窓 - 部分一致 - 詳細検索項目 - 完全一致 or 前方一致

各種イベント、例外発生時にログを出力する

例えば、仮会員登録メールなど、 https:// を強制したい場合に意図しない結果となってしまう。 不具合ではないが、何らかの注意喚起をしておきたい。 (ロードバランサーを使用した場合など、 https:// の判定が難しくなるケースもある) 関連 #703

ゴミ箱機能

2

ユニーク制約 + 論理削除が組み合わさると、一旦論理削除したデータが再登録できない問題が発生する。 論理削除したデータは、ゴミ箱に表示するようにし、復活できるようにすれば、上記の問題も解決するし、利便性も向上するのではないだろうか。

dtb_product_class.product_code のインデックス

4

`dtb_product_class.product_code` は、キーになりうる情報なので、インデックスを付与するべき。 また、 できれば NOT NULL, ユニーク制約を付与したい。

2系では、複合キーになっていたテーブルが、3系ではサロゲートキーになったが、ユニークキーが存在しないテーブルがあり、不整合が発生しやすい状況となっている。 del_flg の存在するテーブルは難しいが、 del_flg の無いテーブルはユニークキーの制約を作成するべき。 以下のテーブルが該当する - dtb_delivery_fee - dtb_delivery_time