- Issue created by @jacobupal
- last update
8 months ago 54 pass - Status changed to Needs review
8 months ago 6:49am 4 January 2024 - last update
8 months ago 53 pass, 1 fail 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.
Never mind, with that test failed, no wonder the error went away, it seems to have just deleted the query.
- last update
8 months ago 53 pass, 1 fail jacobupal → changed the visibility of the branch 3412226-warning-array-to to hidden.
jacobupal → changed the visibility of the branch 3412226-warning-array-to to active.
- last update
8 months ago 53 pass, 1 fail - last update
8 months ago 54 pass - last update
8 months ago 54 pass 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
7 months ago 54 pass - Status changed to Fixed
7 months 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.