- Issue created by @jacobupal
- ๐ฎ๐ณIndia chetan 11
chetan 11 โ made their first commit to this issueโs fork.
- last update
over 1 year ago 54 pass - Status changed to Needs review
over 1 year ago 6:49am 4 January 2024 - last update
over 1 year ago 53 pass, 1 fail - ๐ฌ๐งUnited Kingdom jacobupal Leeds
I'm afraid your addition incurred the same warning, however I've made a different change to the MR that seems to work:
This version checks whether each item in 'keys' is a string, and converts it if not, just before the 'implode' step recombines the whole thing.
- ๐ฌ๐งUnited Kingdom jacobupal Leeds
Never mind, with that test failed, no wonder the error went away, it seems to have just deleted the query.
- last update
over 1 year ago 53 pass, 1 fail - ๐ฌ๐งUnited Kingdom jacobupal Leeds
jacobupal โ changed the visibility of the branch 3412226-warning-array-to to hidden.
- ๐ฌ๐งUnited Kingdom jacobupal Leeds
jacobupal โ changed the visibility of the branch 3412226-warning-array-to to active.
- last update
over 1 year ago 53 pass, 1 fail - last update
over 1 year ago 54 pass - last update
over 1 year ago 54 pass - ๐ฌ๐งUnited Kingdom jacobupal Leeds
OK, I got it passing... the only problem now is that the logged query will not include the hyphen so even though the test string
foo bar baz
worksfoo -bar
would probably fail the test as the '-' becomes the boolean operator 'NOT' and this version strips boolean operators so the result would befoo bar
. So somehow among all the array-to-string stuff the '-' needs to be brought back.This seems to be as far as I can take it for now anyhoo.
Probably the event test should include a hyphen, and maybe even quote marks, for completeness, something like
foo -bar "fizz buzz" -"far boo"
- First commit to issue fork.
- last update
over 1 year ago 54 pass - Status changed to Fixed
over 1 year ago 4:41pm 6 February 2024 - ๐บ๐ธUnited States japerry KVUO
Incremental progress is still progress, so gonna just commit this. If users need more functionality they should open a new issue and reference this one.
-
japerry โ
committed 3811eead on 2.0.x authored by
chetan 11 โ
Issue #3412226 by jacobupal: "Warning: Array to string" when Search API...
-
japerry โ
committed 3811eead on 2.0.x authored by
chetan 11 โ
Automatically closed - issue fixed for 2 weeks with no activity.