[5.1] Add support for nested indexed arrays to array_pluck and data_get

[JosephSilber]

This has been requested a lot, and for a long time. It's immensely useful in countless situations.

To take just one example:

$tags = User::with('tags.posts.comments')->find(1)->tags;

$comments = $tags->pluck('posts.*.comments')->collapse()->unique();

This would have previously required way more code and/or an additional query¹.

You can go as far as you need, with as many stars as you need:

$colors = $users->pluck('orders.*.items.*.product.colors')->collapse()->unique();

There are too many use cases to list here. This really puts data_get and array_pluck on steroids!

---

¹ That excellent article is what actually reminded me about this. See the comments there.

[taylorotwell]

Is this a breaking change if someone actually uses * as an array key and then tries to access that value?

[JosephSilber]

It is. It's not a very common scenario, but it's not BC in that instance.

[taylorotwell]

Going to hold off on this for now.

[tobz-nz]

+1 on this. Would be super useful.
Also, if you're putting *'s in you're array keys, you're probably doing something wrong.

[Riesjart2]

+1

[Gummibeer]

👍

[vlakoff]

Looks like we actually don't support more than 2 wildcards, because of the return:

if ($segment === '*') {
    ...
    return in_array('*', $key) ? Arr::collapse($result) : $result;
}

[vlakoff]

Mmm, but Arr::pluck calls data_get so it should be recursive.