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 not supported #18

Open
itajackass opened this issue Jan 10, 2023 · 2 comments
Open

PHP 8 not supported #18

itajackass opened this issue Jan 10, 2023 · 2 comments

Comments

@itajackass
Copy link

itajackass commented Jan 10, 2023

Hi, if I try to launch:

composer require lyquidity/xbrl:dev-master lyquidity/xpath2:dev-master lyquidity/utilities:dev-master lyquidity/xml:dev-master --prefer-dist

i get error:

 Package lyquidity/xpath2 at version dev-master has requirements incompatible with your PHP version, PHP extensions
 and Composer version:
   - lyquidity/xpath2 dev-master requires php ^7.0 which does not match your installed version 8.0.13.

Any chance to get all compatible with php 8?

@bseddon
Copy link
Owner

bseddon commented Jan 10, 2023

The XPath2 package is used to support XBRL formulas. Formulas have not been tested with PHP 8.x. My guess is that if the constraint is relaxed it will be used with 8.1 then logs will be filled with deprecation messages. I'll look at checking with 8.x but its not something I'll be able to do today.

@bseddon
Copy link
Owner

bseddon commented Jan 11, 2023

Last night the automated test suite ran using PHP 8.0.13. Because all the tests completed successfully, the composer.json has been updated to allow PHP versions >=7.0

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

2 participants