-
Notifications
You must be signed in to change notification settings - Fork 64
symfony4.4へのマイグレーションについて記載 #87
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
_pages/quickstart/update_40_41.md
Outdated
|
|
||
| ```diff | ||
| - $pageRepository = $container->get(PageRepository::class); | ||
| + $entityManager = $container->get('doctrine'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$container->get('doctrine') で返ってくるのは ManagerRegistry ですね
| + $entityManager = $container->get('doctrine'); | |
| + $entityManager = $container->get('doctrine')->getManager(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@nanasess
ありがとうございます。修正しました。
|
Customer と Member で Serializable の実装が必須になったのも追記した方がよい��も。 |
|
あと、最近のリリースで直ってるかもしれませんが、一部のメッセージが翻訳されない問題がありました。 |
たぶんこの修正かなと思うのですが、Rangeバリデータでmin/max両方設定されていると、出力されるエラーメッセージが変わるようです。
こちら了解です、確認して追記します。 |
|
2点修正します。 テストコードバリデーションメッセージ一部のバリデーションメッセージが変更になっています。 その他の仕様変更非会員購入時のお客様情報保持形式の変更非会員購入時にはお客様情報をsessionに保存していますが、その保持形式が Customer エンティティから Form データの配列へ変更になりました。 - $NonMember = $this->session->get('eccube.front.shopping.nonmember')
+ $NonMember = $this->orderHelper->getNonMember('eccube.front.shopping.nonmember') |
- 非会員購入時のお客様情報取得方法の変更について記載
|
こちら追記しました。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
内容確認しました。
No description provided.