Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PHP 8.2 comatibility #314

Closed
TiiFuchs opened this issue Mar 17, 2023 · 5 comments
Closed

PHP 8.2 comatibility #314

TiiFuchs opened this issue Mar 17, 2023 · 5 comments

Comments

@TiiFuchs
Copy link

I get this error message using PHP 8.2 when just entering takeout:

Deprecated: Return type of HumbugBox3130\KevinGH\RequirementChecker\RequirementCollection::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar://[...]/.composer/vendor/tightenco/takeout/builds/takeout/.box/src/RequirementCollection.php on line 12

I guess that's a PHP compatibility problem.

I can't find any information on the README which PHP versions are officially supported.

@vblinden
Copy link

You should update to the latest version.

composer global require tightenco/takeout:2.4.0

I had the same problem, but had an older 1.* version installed.

@scheMeZa
Copy link

@vblinden is correct.
I can confirm as I am using takeout with PHP 8.2

@tonysm
Copy link
Contributor

tonysm commented Jun 16, 2023

Hey @TiiFuchs

I'm using Takeout (v2.5.0 at the time of writing) with PHP 8.2.6. What version of Takeout are you using? I think the suggestion from @vblinden should fix it.

I'm gonna close the issue, but pls feel free to reopen it and leave a comment if that doesn't work for you.

@tonysm tonysm closed this as completed Jun 16, 2023
@cheesegrits
Copy link

@tonysm maybe update the installation instructions? I just had the same issue, followed the instructions, the composer install command given installs 1.x. Maybe give a more specific example for folk with PHP 8.2?

@tonysm
Copy link
Contributor

tonysm commented Nov 4, 2023

@cheesegrits we're working on something that should fix issues like this one. See #326 if you're curious.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants