Skip to content

Fix(dashboard): reject invalid reports_id when adding a widget#359

Merged
stonebuzz merged 1 commit into
mainfrom
support_44193
May 21, 2026
Merged

Fix(dashboard): reject invalid reports_id when adding a widget#359
stonebuzz merged 1 commit into
mainfrom
support_44193

Conversation

@Rom1-B
Copy link
Copy Markdown
Contributor

@Rom1-B Rom1-B commented May 21, 2026

Checklist before requesting a review

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !44193

Submitting the "add report" form without selecting a report posted -1 (the empty dropdown sentinel value) into the reports_id column, which is defined UNSIGNED NOT NULL and rejects negative values. The form now validates the posted value before inserting. The migration purges any existing invalid rows and enforces the unsigned constraint on older installs.

Screenshots (if appropriate):

@Rom1-B Rom1-B requested a review from stonebuzz May 21, 2026 12:26
@stonebuzz stonebuzz merged commit 040e96a into main May 21, 2026
3 checks passed
@stonebuzz stonebuzz deleted the support_44193 branch May 21, 2026 14:50
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