Skip to content

Fix html:cancel regression from commit ""STR-286 and STR-1116: Action…#35

Open
nrmnrm wants to merge 1 commit intoweblegacy:mainfrom
nrmnrm:main
Open

Fix html:cancel regression from commit ""STR-286 and STR-1116: Action…#35
nrmnrm wants to merge 1 commit intoweblegacy:mainfrom
nrmnrm:main

Conversation

@nrmnrm
Copy link

@nrmnrm nrmnrm commented Jun 17, 2024

…Form populate and reset strategy""

@nrmnrm nrmnrm changed the title Fix html_cancel regression from commit ""STR-286 and STR-1116: Action… Fix html:cancel regression from commit ""STR-286 and STR-1116: Action… Jun 18, 2024
@ste-gr ste-gr self-assigned this Jun 27, 2024
@ste-gr
Copy link
Member

ste-gr commented Jun 28, 2024

Hi @nrmnrm,

sorry for the delay (vacation).
Thank you for your work! I think the change is from apache/struts1@e2602ab / STR-1674 - Cancellation does not work as intended without form.

As I saw in the Issue, the goal was that no population would be performed on a cancellation. So I created a new PR #37.

Please take a look at it.

Greetings
Stefan

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.

2 participants