Fix validation of array data

[paulbalandan]

Description
Fixes #4510

This changes the behavior of _array_search_dot. Currently, the function returns early when it matched a non-null element in its value traversal, leaving the remaining elements in the loop as untouched. This PR accumulates the finds in an array, filters out the nulls. If resulting array contains only one element, then we'll return that element instead of an array in order to preserve BC. Otherwise, an array containing matches will be returned.

@luispastendev, can you check this? This passes locally on your use case .

Checklist:

• Securely signed commits
• Component(s) with PHPdocs
• Unit testing, with >80% coverage
• User guide updated
• Conforms to style guide

[luispastendev]

Thank you very much Paul! I tested the changes and now it works fine, only one detail @WinterSilence detect another use case when an empty array is sent, it evaluates it as true, what do you think about it?

#4510 (comment)

[paulbalandan]

Thank you very much Paul! I tested the changes and now it works fine, only one detail @WinterSilence detect another use case when an empty array is sent, it evaluates it as true, what do you think about it?

#4510 (comment)

My latest push contains some fix for empty arrays. Can you check also?

[luispastendev]

Perfect I was solved! thank you very much for your time paul!

[MGatner]

See my one comment, everything else looks great.