Skip to content
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

複数配送時のインデックスの参照先に誤りがあったため修正 #2187

Merged
merged 2 commits into from
Mar 9, 2017

Conversation

k-yamamura
Copy link
Contributor

概要(Overview・Refs Issue)

#2053
この対応で複数配送時にお届け先削除が可能になっているが、
お届け先の追加、削除、追加を繰り返して商品の追加を行うとエラーが発生したため参照先を修正。

Copy link
Contributor

@ryo-endo ryo-endo left a comment

Choose a reason for hiding this comment

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

修正内容問題ないと思います。

いままではloopのindexを利用して関連付けを行っていたため、お届け先を削除した再にindexの不整合が起こっていました。

今回の変更で、FormTypeが持つパラメータを利用するようになったので、不整合が起こらなくなったことを確認しました。

@ryo-endo
Copy link
Contributor

ryo-endo commented Mar 9, 2017

こちら、3.0.14の開発期間中に発生した不具合です。
受注の編集でエラーが発生するのは避けたいため、3.0.14に修正を含めます。

@ryo-endo ryo-endo merged commit fca130c into EC-CUBE:master Mar 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants