Skip to content

GEM: ファイルアップロード画面 vs ウィジェットで、検索Formデータが混線する#2063

Merged
xtakahashi-hiroyuki merged 6 commits into
masterfrom
feature/1976-gem-file-upload-view-fix
May 22, 2026
Merged

GEM: ファイルアップロード画面 vs ウィジェットで、検索Formデータが混線する#2063
xtakahashi-hiroyuki merged 6 commits into
masterfrom
feature/1976-gem-file-upload-view-fix

Conversation

@xtakahashi-hiroyuki

@xtakahashi-hiroyuki xtakahashi-hiroyuki commented May 19, 2026

Copy link
Copy Markdown
Contributor

対応内容

closes #1976

バグ原因

ファイルアップロード画面のSearchFormViewデータが、Widget画面処理で別データで上書きされていた

修正内容

上書きしないように(キーを変更。重複しないように)

動作確認・スクリーンショット(任意)

メイン修正

2026-05-20_20h55_22

デグレチェック

Widget以外の画面にデグレがないことを確認した

  • Top画面 > EntityListParts

レビュー観点・補足情報(任意)

横展開チェック

iplass-gem/src/main/java/org/iplass/mtp/impl/view/top/parts
の中で、今回の問題(以下)と同じ実装がないか、横展開チェックした

  • キー衝突
  • セットしたのにクリアしてない

→ 問題なし

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

ファイルアップロード画面とTOP画面ウィジェット(EntityListParts)の描画が同一リクエスト内で走る際に、SearchFormViewrequest に格納する属性名が衝突して上書きされる問題(#1976)を回避するため、EntityListParts 側の属性キーを変更して衝突を防ぐPRです。

Changes:

  • EntityListParts の JSP(parts / widget)で参照する SearchFormView の request 属性名を searchFormViewlistPartsSearchFormView に変更
  • MetaEntityListParts が設定する属性名を同様に変更し、clearAttribute() で後片付けを追加

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
iplass-gem/src/main/resources/META-INF/resources/jsp/gem/generic/search/listWidget.jsp EntityListParts(Widget) 側で参照する SearchFormView の request 属性キーを変更
iplass-gem/src/main/resources/META-INF/resources/jsp/gem/generic/search/list.jsp EntityListParts(Parts) 側で参照する SearchFormView の request 属性キーを変更
iplass-gem/src/main/java/org/iplass/mtp/impl/view/top/parts/MetaEntityListParts.java EntityListParts が request に積む属性キー変更+属性クリア処理を追加

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@xtakahashi-hiroyuki xtakahashi-hiroyuki force-pushed the feature/1976-gem-file-upload-view-fix branch from 7522f68 to e642a1c Compare May 21, 2026 02:38

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

@yuichi-github yuichi-github requested a review from h-terazaki May 22, 2026 04:02
@xtakahashi-hiroyuki xtakahashi-hiroyuki merged commit de27023 into master May 22, 2026
5 checks passed
@xtakahashi-hiroyuki xtakahashi-hiroyuki deleted the feature/1976-gem-file-upload-view-fix branch May 22, 2026 07:30
xtakahashi-hiroyuki added a commit that referenced this pull request May 25, 2026
* searchFormViewをrequestに設定するキーを変更(混線防止)

* clearAttributeが漏れていたので追加

* レビュー対応:partsCntはclearしない

* コメント微修正

* レビュー対応:キー名を改善

* レビュー対応:キー名改善

---------

Co-authored-by: Takahashi.Hiroyuki <xtakahashi.hiroyuki4@contract.dentsusoken.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GEM: ファイルアップロード画面 vs ウィジェットで、検索Formデータが混線する

4 participants