Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUGFIX] Add empty array defaults in SearchFormViewHelper #4042

Conversation

hnadler
Copy link
Contributor

@hnadler hnadler commented May 16, 2024

What this pr does

Adds empty array defaults in SearchFormViewHelper::getExistingSearchParameters to prevent TYPO3 TypeError.

How to test

Setup ext:solr with "keepExistingParametersForNewSearches = 1"

Fixes: #4040

@davidjakob
Copy link

Thank you for creating this PR!

@dkd-kaehm dkd-kaehm force-pushed the main branch 3 times, most recently from af047de to 8fc413f Compare May 16, 2024 15:31
@dkd-kaehm dkd-kaehm changed the base branch from main to release-12.0.x May 16, 2024 16:20
@dkd-kaehm dkd-kaehm force-pushed the bugfix/4040-searchformviewhelper_breaks_on_empty_queryparams_and_body_in_request branch from b55d50d to 26ca9f2 Compare May 16, 2024 16:24
Adds empty array defaults in SearchFormViewHelper::getExistingSearchParameters to prevent TYPO3 TypeError.

Fixes: TYPO3-Solr#4040
@dkd-kaehm dkd-kaehm force-pushed the bugfix/4040-searchformviewhelper_breaks_on_empty_queryparams_and_body_in_request branch from 26ca9f2 to 438e0eb Compare May 16, 2024 16:25
Copy link
Collaborator

@dkd-kaehm dkd-kaehm left a comment

Choose a reason for hiding this comment

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

💯

@dkd-kaehm dkd-kaehm merged commit 8b6e898 into TYPO3-Solr:release-12.0.x May 16, 2024
7 checks passed
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.

[BUG] SearchFormViewHelper breaks on empty QueryParams and Body in Request.
3 participants