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

受注編集画面のtwigのdivのid重複 #1724

Closed
izayoi256 opened this issue Sep 3, 2016 · 7 comments
Closed

受注編集画面のtwigのdivのid重複 #1724

izayoi256 opened this issue Sep 3, 2016 · 7 comments
Assignees
Milestone

Comments

@izayoi256
Copy link
Contributor

配送商品の要素に対して配送先のインデックスを利用しているためidが重複する場合がある。

https://github.com/EC-CUBE/ec-cube/blob/3.0.10/src/Eccube/Resource/template/admin/Order/edit.twig#L548 (その他、配送商品の要素でidにshippingIndexを指定している箇所全て)

@ryo-endo ryo-endo added the bug label Sep 7, 2016
@ryo-endo ryo-endo added this to the 3.0.x milestone Sep 7, 2016
@ryo-endo ryo-endo changed the title 受注編集画面のid重複 受注編集画面のtwigのdivのid重複 Oct 3, 2016
@Yangsin Yangsin added bug:Middle and removed bug labels Oct 13, 2016
@Yangsin
Copy link

Yangsin commented Jan 6, 2017

@izayoi256
#1725 #1758 の修正でこちらも修正されている気がするのですがいかがでしょうか。

@izayoi256
Copy link
Contributor Author

@Yangsin
直っていません。複数配送先機能を有効にすると、同配送先内の配送商品のidが重複します。
screenshot from 2017-01-10 13-19-54

@Yangsin Yangsin modified the milestones: 3.0.14, 3.0.x Jan 10, 2017
@Yangsin
Copy link

Yangsin commented Jan 10, 2017

そういうことですね。。。確認ありがとうございます。

@t-nagahashi
Copy link
Contributor

本件、#2053 で同時に対応されているようです。

@t-nagahashi
Copy link
Contributor

動作確認はOKです。

■修正前(デモサイト 3.0.13)
「shipment_item__id--0」が複数ある状態。
default

■修正後(#2053 適用後)
「shipment_item__id--0--0」
「shipment_item__id--0--1」
と、インクリメントされた状態となっています。

2053

@t-nagahashi
Copy link
Contributor

#2053 のソースチェックもOK。

@ryo-endo
Copy link
Contributor

ryo-endo commented Feb 8, 2017

私のほうでも重複消えていること確認しました。
OKそうですのでCLOSEします。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants