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

同一フォーム利用時にhtml属性のidが被る時があるため、被るフォーム項目はid属性を出力しないように対応 #3824

Merged
merged 8 commits into from
Sep 14, 2018

Conversation

k-yamamura
Copy link
Contributor

概要(Overview・Refs Issue)

ブロック管理で同一フォームを利用した時にフォーム項目のid属性が被っていたため明示的に出力できないように対応

{% form_theme form 'Form/form_div_layout.twig' %}

を宣言後、

{{ form_widget(form.name, {'id': null} ) }}

{'id': null}を設定するとhtmlへ展開された時に、

<input type="text" name="name">

とid属性が出力されないようになる。

@ryo-endo ryo-endo added this to the 4.0 milestone Sep 11, 2018
@okazy okazy self-assigned this Sep 12, 2018
@chihiro-adachi
Copy link
Contributor

@okazy
form_div_layout.twigで実装するのは影響範囲が広すぎるので、form _selfで定義できるか確認してもらえませんか。
https://symfony.com/doc/current/form/form_customization.html

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.

現状の実装で動作問題ありませんでした。
form _self で同一twigファイル内に定義してみましたが、そちらの方法ではうまく動作しませんでした。

widget_attributesはバージョンアップでもこれ以上変更される可能性は低いと思いますので、こちらのプルリクの内容で取り込ませていただきます。

@okazy okazy merged commit 65284bf into EC-CUBE:4.0 Sep 14, 2018
@ryo-endo ryo-endo added affected:plugin プラグインに影響する可能性のある実装変更 affected:template フロントテンプレートの変更 and removed Status: ready-for-merge affected:plugin プラグインに影響する可能性のある実装変更 labels Sep 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:template フロントテンプレートの変更
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants