Skip to content

fix(settings): save the anti spam addresses currently in the form - #14004

Open
GVodyanov wants to merge 1 commit into
mainfrom
fix/antispam-save-current-values
Open

GVodyanov wants to merge 1 commit into
mainfrom
fix/antispam-save-current-values

Conversation

@GVodyanov

Copy link
Copy Markdown
Contributor

In Admin → Groupware → Anti Spam, saving after a reset sent the values loaded at page load instead of what was in the form. Reset also didn't clear the component state.

Save now sends the current form values, and a successful reset clears them.

How to test

  1. Enter spam/ham addresses A, save, then reset
  2. Enter addresses B, save, reload
  3. The form shows B (before: A)

Found during the QA round in #13797. AI-assisted (Claude Code); reviewed and tested by me.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

Save submitted the module-level initial state instead of the component's
data. After a Reset the form switched to a new object, so saving new
addresses stored the old ones again while the fields showed the new
values. Reset now also clears the fields only when the deletion worked.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Grigory Vodyanov <scratchx@gmx.com>
@GVodyanov GVodyanov self-assigned this Nov 1, 2026
@nextcloud-command nextcloud-command added the AI assisted This PR contains AI-assisted commits label Nov 1, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review AI assisted This PR contains AI-assisted commits

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants