TypeError: implode(): Argument #1 ($array) must be of type array, string given in implode() (line 587 of modules/contrib/search_api_swiftype/src/Plugin/search_api/backend/SwiftypeBackend.php

Hi,
I have resolved the "TypeError: implode(): Argument #1 ($array)" issue and created a patch.
I have also attached the patch file .
kindly review it,
Thank!.

@sarwan_verma,

Could you please ensure that you only pick up issues that are not already assigned to someone else? If an issue is already assigned, it's important to take note and avoid working on it. Overlooking this wastes time for both of us, as well as for others who may be working on the same issue. Remember, contribution is not just about earning credit; it's about respecting the efforts and time of everyone involved. Let's be mindful, considerate, and make the best use of our time.

There is no reason to be annoyed. You both tried to solve the problem.
I created a new branch based on your changes.

Thanks for the quick fix, this helped a lot, because now we know, that a cURL timeout is the underlying problem in our case.

But with the current solution from the MR from #6, the $response variable is just completely empty, which results in a log message like this:

Failed creating/updating documents: "cURL error 28: Operation timed out after 30001 milliseconds with 0 bytes received (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for https://api.swiftype.com/api/v1/engines/SWIFTYPE-ENGINE-ID/document_types/entity-node/documents/bulk_create_or_update_verbose". Response:

I guess the error handling should be able to deal with all of those cases:

• No response at all (e.g. when a cURL error like ours occurred)
• Array response (already covered)
• String response (already covered)

Additionally, it would also be nice, if the external IDs of the items to index may be included in the log message - because those are mostly very important to track done any underlying issues.

ushma → changed the visibility of the branch 10.x to hidden.

ushma → changed the visibility of the branch 10.x to active.

@hctom Added logic to handle empty response case. Also attached a patch for the same. Please review.