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

Deprecation Warnings with v2 and php 8.0 #637

Open
tolry opened this issue Sep 29, 2021 · 1 comment
Open

Deprecation Warnings with v2 and php 8.0 #637

tolry opened this issue Sep 29, 2021 · 1 comment
Labels

Comments

@tolry
Copy link

tolry commented Sep 29, 2021

Describe the bug

php 8.0 deprecated libxml_disable_entity_loader since it's no longer needed with the updated libxml version used

To Reproduce

Run version 2.x with deprecation activated (e.g. error_reporting(-1);)

Expected behavior

Getting no deprecation warnings.

Your Environment

  • Which version of this library are you using? 2.12.22
  • Which version of php are you using? 8.0.11

I have a fix ready here and could submit this, if wanted.

@tolry tolry added the bug? label Sep 29, 2021
@pabloolvcastro
Copy link

pabloolvcastro commented Feb 28, 2023

I was wondering when this would get fixed. We are currently facing some problems with classes that implement basic PHP interfaces, and the return type has changed in PHP 8. e.g. Recurly_Pager (pager.php) and the interface Iterator. We are running the library version 2.12.37.

pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue Mar 2, 2023
The changes promoted by this commint mantain compatibility with previous versions as well. A mix of valid return types with the annotation ReturnTypeWillChange was used to guarantee compatibility e.g mixed new type

Ref
recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue Mar 2, 2023
The changes promoted by this commit maintain compatibility with previous versions as well. A mix of valid return types with the annotation ReturnTypeWillChange was used to guarantee compatibility, e.g. mixed new type.

Ref
recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue Mar 2, 2023
This is an optional commit. It provides tools to help with compatibility checks for PHP 8+.

Ref recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue Jul 4, 2023
The changes promoted by this commit maintain compatibility with previous versions as well. A mix of valid return types with the annotation ReturnTypeWillChange was used to guarantee compatibility, e.g. mixed new type.

Ref
recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue Jul 4, 2023
This is an optional commit. It provides tools to help with compatibility checks for PHP 8+.

Ref recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue Nov 1, 2023
The changes promoted by this commit maintain compatibility with previous versions as well. A mix of valid return types with the annotation ReturnTypeWillChange was used to guarantee compatibility, e.g. mixed new type.

Ref
recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue Nov 1, 2023
This is an optional commit. It provides tools to help with compatibility checks for PHP 8+.

Ref recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue May 7, 2024
The changes promoted by this commit maintain compatibility with previous versions as well. A mix of valid return types with the annotation ReturnTypeWillChange was used to guarantee compatibility, e.g. mixed new type.

Ref
recurly#637
pabloolvcastro pushed a commit to pabloolvcastro/recurly-client-php that referenced this issue May 7, 2024
This is an optional commit. It provides tools to help with compatibility checks for PHP 8+.

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

No branches or pull requests

2 participants