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 compatibility #588

Open
Nek- opened this issue Mar 6, 2023 · 9 comments · May be fixed by #640
Open

PHP 8.2 compatibility #588

Nek- opened this issue Mar 6, 2023 · 9 comments · May be fixed by #640

Comments

@Nek-
Copy link

Nek- commented Mar 6, 2023

It seems that this library is mostly compatible with PHP 8.2 but triggers a lot of deprecation warnings: PHP Deprecated: Creation of dynamic property. They should be fixed to avoid too many logs using PHP 8.2.

@bfolliot
Copy link

bfolliot commented Oct 3, 2023

Hi, when do you plan to move forward on this subject? As with @Nek- , the numerous warnings from your library are polluting our logs.

@bfolliot
Copy link

bfolliot commented Oct 3, 2023

The one that comes up most often for me:

Deprecated: Creation of dynamic property MangoPay\Libraries\HttpCurl::$logger is deprecated in vendor/mangopay/php-sdk-v2/MangoPay/Libraries/HttpBase.php on line 23

@hrsa
Copy link

hrsa commented Dec 11, 2023

yes, that's a really disturbing issue!

@florentdestremau
Copy link

little UP here as this seems to have been left out.

@franck-grenier
Copy link

+1

@florentdestremau
Copy link

florentdestremau commented Jan 25, 2024

seem like there is already a PR done for this, it's sad maintainers don't take notice.

#601

@iulian03 you seem to be the latest contributor, would you have some time to check on this ?

@williamdes
Copy link
Contributor

See #626

@mralaminahamed
Copy link

Thanks, @williamdes, I'm actively addressing #626, focusing on resolving all PHP-related issues.

@mralaminahamed mralaminahamed linked a pull request Apr 30, 2024 that will close this issue
5 tasks
@fredericdelordm
Copy link
Contributor

Hello everyone, looking into it

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

Successfully merging a pull request may close this issue.

8 participants