Duplicates filter in EagerLoadIterator removes valid paths

[tantchen]

At this point i have ["station","aircraft","aircraftConfig","person","owner"] in $values because aircraft starts with aircraftConfig the reject callback removes aircraft.

The expected output should be ["aircraft","aircraftConfig","owner","person","station"]
but it is ["aircraftConfig","owner","person","station"] with the result that every aircraft will be fetched with a singe select.

eloquent/src/QueryBuilder/EagerLoading/EagerLoadIterator.php

Line 76 in 98bbec0

fn($path) => $values->contains(fn($check) => $path !== $check && Str::startsWith($check, $path))

[tantchen]

May be a possible fix ...

   /**
     * Get the paths as a collection.
     *
     * Before returning the paths, we filter out any duplicates. For example, if the iterator
     * yields `user` and `user.country`, we only want `user.country` to be in the collection.
     *
     * @return Collection
     */
    public function collect(): Collection
    {
        $values = collect($this);

        return $values->unique()->reject(
            fn ($path) => $values->contains(fn ($check) => $path !== $check && Str::startsWith($check, $path.'.'))
        )->sort()->values();
    }

[lindyhopchris]

Thanks for reporting, this definitely is a bug. I've added it to the to-do list to address.