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

Authorizing fields to have child nodes, and fix their parsing in the … #1

Merged
merged 1 commit into from
Mar 23, 2016

Conversation

romainruaud
Copy link
Collaborator

…converter to proper snake case.

@afoucret afoucret merged commit 912e9cf into Smile-SA:master Mar 23, 2016
@Fdec Fdec mentioned this pull request Jul 1, 2016
@romainruaud romainruaud deleted the fix_indices-converter branch October 27, 2016 13:26
botisSmile pushed a commit to botisSmile/elasticsuite that referenced this pull request Sep 8, 2021
peterjaap added a commit to peterjaap/elasticsuite that referenced this pull request Apr 15, 2022
When no category URL suffix is defined, the get function returns NULL causing this error: Deprecated Functionality: str_replace(): Passing null to parameter Smile-SA#1 ($search) of type array|string is deprecated
vsushkov added a commit to vsushkov/elasticsuite that referenced this pull request Apr 21, 2022
Sometimes one of the `$optionTextValues` could be `null`. And in PHP 8 `Passing null to parameter Smile-SA#1 ($string) of type string is deprecated`. So I suggest to explicitly cast all values to string.
vsushkov added a commit to vsushkov/elasticsuite that referenced this pull request Apr 21, 2022
Sometimes one of the `$optionTextValues` could be `null`. And in PHP 8 `Passing null to parameter Smile-SA#1 ($string) of type string is deprecated`. So I suggest to explicitly cast all values to string.
ecrocombe pushed a commit to ecrocombe/elasticsuite that referenced this pull request Sep 12, 2023
rbayet pushed a commit to ecrocombe/elasticsuite that referenced this pull request Sep 12, 2023
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