How to prevent Tightenco collect()-helper from overwriting Illuminate collect-helper?

[IYSaschi]

Dear community & LdapRecord-team

I got problems with tightenco collect()-helper overwriting Illuminate collect()-helper

Loadorder autoload_files.php:

'265b4faa2b3a9766332744949e83bf97' => $vendorDir . '**/laravel/framework/src/Illuminate/Collections/helpers.php',**
    'c7a3c339e7e14b60e06a2d7fcce9476b' => $vendorDir . '/laravel/framework/src/Illuminate/Events/functions.php',
    'f0906e6318348a765ffb6eb24e0d0938' => $vendorDir . '/laravel/framework/src/Illuminate/Foundation/helpers.php',
    '58571171fd5812e6e447dce228f52f4d' => $vendorDir . '**/laravel/framework/src/Illuminate/Support/helpers.php**',
    '6124b4c8570aa390c21fafd04a26c69f' => $vendorDir . '/myclabs/deep-copy/src/DeepCopy/deep_copy.php',
    'fe62ba7e10580d903cc46d808b5961a4' => $vendorDir . **'/tightenco/collect/src/Collect/Support/helpers.php**',
    'caf31cc6ec7cf2241cb6f12c226c3846' => $vendorDir . '**/tightenco/collect/src/Collect/Support/alias.php**', 

My Loadorder seems right, Illuminate is loaded first. So tightenco should recocnize that a helper with name "collect()" is already present, or em I wrong?

Because phpstan throws the following erro:

should return Illuminate\Support\Collection but returns
Tightenco\Collect\Support\Collection.

Tightenco\Collect always overwrites Illuminate and I don't know how to fix it. I also dont think that this package is needed at all because its: (statement from packagist) "Deprecated: With the separation of Illuminate's Collections package, Collect is no longer necessary 🎉 . We will maintain it for the 8.0 and 9.0 releases of Laravel for the convenience of package maintainers, and will stop mirroring new major versions after 9.0, but keep the existing package here for continued support." https://packagist.org/packages/tightenco/collect

This is also pointed out on tightencos github-page: https://github.com/tighten/collect

There are also some other threats about this problem e.g.
tighten/collect#92
laravel/framework#23420
https://github.com/composer/composer/pull/1051/files#diff-33037407f7f293f37df2258567b56598R238

Till now I got no solution for this problem, maybe you can help!
Thank you!

Greetings IYSaschi

[stevebauman]

Hi @IYSaschi,

LdapRecord v3 is coming out which will remove usage of tightenco/collect and replace it with the core Laravel illuminate/collections package.

However, I'm surprised this is happening, since tightenco/collect checks if Laravel collections exist:

https://github.com/tighten/collect/blob/d4772fe52bbc823d1d2e3d289d3f1ca8c0f3323b/src/Collect/Support/helpers.php#L8-L9

If you're running the latest Laravel v10 and this issue is occurring, then I may have to release LdapRecord v3 sooner than anticipated.

[IYSaschi]

Thanks for your fast reply,

currently I' am useing laravel v9 I will try to upgrade to version 10.
I will get back to you and share the information I have gathered.

[Owlwasrowk]

I have the same problem and my laravel version is already v10...

[IYSaschi]

I'am back,

sadly it was not possible for me to upgrade my project to laravel 10. To many dependencies won't work.

[IYSaschi]

Hey @stevebauman ,

I got a solution for the problem, If somebody has the same problem here is the solution, which works for me with laravel 9.52.0:
Simple add the following part to your composer.json ->

  "replace": {
        "tightenco/collect" : "9.52.0"
    },

[stevebauman]

That's great to hear you were able to find a resolution. Thanks for following up @IYSaschi! 🙏

[IYSaschi]

before I forget my solution is not working, and its not possible to use your framework since tightenco/collection gets removed

[IYSaschi]

sorry :)