Remove the limit of the explode from Utility::splitCombinedId

This is definitely something to fix within the Search API Grouping module, the method is correct as-is.

I believe an error has been made in the issue description as this has nothing to do with the contrib module "Search API Grouping".

Will check this with @cgoffin

In the mean time I rebased the patch as an MR for now so the project that is using this can continue using it, for now 😉

Had a chat with Christophe. Like I suspected this has nothing to do with the contrib module. But the contrib module mentioned here had a similar issue and implementation. Turns out if you extend the item ID and enrich it with extra data, e.g. the date of the item. That information is lost because of the limit on the splitCombinedId function.

Ok, reading the tests failure it makes perfect sense. Closing again as won't fix and will look into our custom code to fix this in a different way :)

BramDriesen → closed merge request !121