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] Handle float values in options facet parser #3722

Merged
merged 2 commits into from
Oct 13, 2023

Conversation

saschanowak
Copy link
Contributor

@saschanowak saschanowak commented Jul 11, 2023

What this pr does

Cast solr value to string to handle float values correctly.
When the result values are 0.01, 0.02, 0.03 the last count wins with the array key 0
see https://3v4l.org/2LOHk#v8.1.20

Version

Tested on dev-master with TYPO3 12.4.3

Maintainers notes:

Port:

  • release-11.0.x/ELTS
  • release-11.2.x/ELTS
  • release-11.5.x

@dkd-kaehm dkd-kaehm added the BACKPORTABLE The changes SHOULD be backported label Jul 20, 2023
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.

The changes are fine,
but we want to add Unit test case for that case.

Most probably Tests/Unit/Domain/Search/ResultSet/ResultSetReconstitutionProcessorTest.php is the right place for test.

dkd-kaehm pushed a commit to dkd-kaehm/ext-solr that referenced this pull request Oct 12, 2023
dkd-kaehm pushed a commit to dkd-kaehm/ext-solr that referenced this pull request Oct 12, 2023
@dkd-kaehm dkd-kaehm merged commit b9299f5 into TYPO3-Solr:main Oct 13, 2023
dkd-kaehm added a commit that referenced this pull request Oct 13, 2023
@dkd-kaehm dkd-kaehm mentioned this pull request Oct 13, 2023
1 task
dkd-kaehm pushed a commit to dkd-kaehm/ext-solr that referenced this pull request Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BACKPORTABLE The changes SHOULD be backported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants