Skip to content

Resolved #4829 where searching tables in SQL manager could generate false error message#4831

Merged
TomJaeger merged 1 commit into7.devfrom
bug/7.x/sql-manager-false-error
Jun 16, 2025
Merged

Resolved #4829 where searching tables in SQL manager could generate false error message#4831
TomJaeger merged 1 commit into7.devfrom
bug/7.x/sql-manager-false-error

Conversation

@intoeetive
Copy link
Contributor

Resolved #4829 where searching tables in SQL manager could generate false error message

@intoeetive intoeetive added this to the 7.x milestone May 11, 2025
@intoeetive intoeetive added the Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on. label May 11, 2025
@robinsowell
Copy link
Contributor

I'd test this one, but can't replicate!

@intoeetive
Copy link
Contributor Author

@robinsowell the page does not error out, it still works. It's just if you enter anything in the search box and hit enter, it will present you with the red rectangle with message (and not with this fix)

@intoeetive intoeetive modified the milestones: 7.x, 7.5.12 May 14, 2025
@TomJaeger TomJaeger modified the milestones: 7.5.12, 7.5.13 May 14, 2025
@TomJaeger TomJaeger requested a review from robinsowell June 4, 2025 20:18
@robinsowell
Copy link
Contributor

Oh man- so it doesn't happen in Firefox. I could replicate in chrome. And this does fix it.

FF doesn't send bulk_action_submit, chrome does- same exact behavior.
Screenshot 2025-06-13 at 1 44 31 PM

In any case- I could replicate and this did indeed fix it. Yay!

@TomJaeger TomJaeger merged commit 532ad56 into 7.dev Jun 16, 2025
22 checks passed
@TomJaeger TomJaeger deleted the bug/7.x/sql-manager-false-error branch June 16, 2025 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug: Accepted Bug has been confirmed, is reproducible, and ready to work on.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Searching table names on the SQL Manager page gives an error

3 participants