PHP 8.1 displays warnings for php-code-search v 1.10.5 about php-code-search HOT 8 CLOSED

Thank you for the bug report, I will look into this.

from php-code-search.

Great! Thanks again for the reports and prompt/helpful responses 👍

from php-code-search.

@Hypnopompia I've published v1.10.6, which should fix the deprecation warnings you're seeings. Are you able to confirm that the issue is resolved?

from php-code-search.

@patinthehat I can confirm that v.1.10.6 removes the warnings. Thanks!

However, when using the spatie/x-ray package in my laravel project, I am seeing this error:

Call to undefined method Permafrost\PhpCodeSearch\Support\Collections\Collection::merge()

It appears the collect() helper in https://github.com/permafrost-dev/php-code-search/blob/main/src/Support/helpers.php#L95-L100 is taking priority over the laravel collect() helper and returning a different Collection class, so when laravel tries to run the merge() method, it fails.

To resolve this, I just removed the x-ray package from my laravel project and installed the x-ray package globally using composer. Now it runs fine with no errors or warnings. I think x-ray was meant to be included in any project that also includes the ray package, so I'm not sure if there's a better way to handle the conflicting helper functions, but I'll leave that for you to decide. I have a workaround that is working for me.

Thanks so much!

from php-code-search.

@Hypnopompia I also wrote the spatie\x-ray package - I'll look into this issue now and patch it ASAP. Thanks for the updated report!

A few follow up questions:

• Are you using spatie\laravel-ray?
• Can you provide more information from the error you're getting so I can locate the issue?

from php-code-search.

Yes, my composer.json includes:
"spatie/laravel-ray": "^1.28"

The error is thrown when running the ./artisan command:

./artisan

In app.php line 237:
Call to undefined method Permafrost\PhpCodeSearch\Support\Collections\Collection::merge()

in my config/app.php file, line 237 is

'aliases' => Facade::defaultAliases()->merge([

In Facade::defaultAliases(), it looks like this:

Hopefully that helps. Let me know if there is any other information I can provide.

from php-code-search.

@Hypnopompia I've published v1.11.0, could you see if this resolves the issue you reported when x-ray is installed locally (project-level) instead of globally?

from php-code-search.

@patinthehat yep, I can confirm that the error I was seeing is no longer there. Looks good! Thanks so much.

from php-code-search.