🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@chihiro-adachi
Copy link
Contributor

No description provided.

@chihiro-adachi chihiro-adachi changed the title [WIP ]symfony4.4へのマイグレーションについて記載 [WIP] symfony4.4へのマイグレーションについて記載 May 12, 2020

```diff
- $pageRepository = $container->get(PageRepository::class);
+ $entityManager = $container->get('doctrine');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$container->get('doctrine') で返ってくるのは ManagerRegistry ですね

Suggested change
+ $entityManager = $container->get('doctrine');
+ $entityManager = $container->get('doctrine')->getManager();

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nanasess
ありがとうございます。修正しました。

@nanasess
Copy link
Contributor

Customer と Member で Serializable の実装が必須になったのも追記した方がよい��も。
非会員購入では Customer オブジェクトを serialize していたので、プラグインにも影響ありそうです
https://github.com/EC-CUBE/ec-cube/pull/4409/files#diff-5516645542fe3aea96d9f10bfd6135d5
https://github.com/EC-CUBE/ec-cube/pull/4409/files#diff-aaae9bf97a22fe00ccf913f66876aa5a

@nanasess
Copy link
Contributor

あと、最近のリリースで直ってるかもしれませんが、一部のメッセージが翻訳されない問題がありました。

EC-CUBE/ec-cube@642a17b

@chihiro-adachi
Copy link
Contributor Author

@nanasess

あと、最近のリリースで直ってるかもしれませんが、一部のメッセージが翻訳されない問題がありました。

たぶんこの修正かなと思うのですが、Rangeバリデータでmin/max両方設定されていると、出力されるエラーメッセージが変わるようです。
symfony/symfony#32435

Customer と Member で Serializable の実装が必須になったのも追記した方がよいかも。

こちら了解です、確認して追記します。

@chihiro-adachi chihiro-adachi changed the title [WIP] symfony4.4へのマイグレーションについて記載 symfony4.4へのマイグレーションについて記載 Jun 30, 2020
@okazy okazy added the installation インストールや環境にまつわるもの label Jul 3, 2020
@okazy okazy added this to the 4.x milestone Jul 3, 2020
@okazy
Copy link
Contributor

okazy commented Nov 19, 2020

2点修正します。

テストコード

バリデーションメッセージ

一部のバリデーションメッセージが変更になっています。
バリデーションメッセージを検証している自動テストは修正が必要な可能性があります。
プロダクトコードの修正は必要ありません。

その他の仕様変更

非会員購入時のお客様情報保持形式の変更

非会員購入時にはお客様情報をsessionに保存していますが、その保持形式が Customer エンティティから Form データの配列へ変更になりました。
非会員購入時のお客様情報を変更されるようなカスタマイズをされている場合に影響があります。

- $NonMember = $this->session->get('eccube.front.shopping.nonmember')
+ $NonMember = $this->orderHelper->getNonMember('eccube.front.shopping.nonmember')

Customer の Serializable 実装に伴う本体の修正

- 非会員購入時のお客様情報取得方法の変更について記載
@okazy
Copy link
Contributor

okazy commented Nov 19, 2020

こちら追記しました。
#87 (comment)

Copy link
Contributor

@okazy okazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

内容確認しました。

@okazy okazy merged commit c156586 into EC-CUBE:master Nov 19, 2020
@okazy okazy modified the milestones: 4.x, 4.1 Dec 9, 2020
@okazy okazy added the improvement 改善 label Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement 改善 installation インストールや環境にまつわるもの

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants