search

Mangopay / mangopay2-php-sdk

PHP SDK for MANGOPAY
https://packagist.org/packages/mangopay/php-sdk-v2
MIT License
123 stars 134 forks source link

PHP 8.2 compatibility #588

Open Nek- opened 1 year ago

Nek- commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 11 months ago

yes, that's a really disturbing issue!

florentdestremau commented 9 months ago

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

franck-grenier commented 9 months ago

+1

florentdestremau commented 9 months ago

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

https://github.com/Mangopay/mangopay2-php-sdk/pull/601

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

williamdes commented 6 months ago

See https://github.com/Mangopay/mangopay2-php-sdk/issues/626

mralaminahamed commented 6 months ago

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

fredericdelordm commented 6 months ago

Hello everyone, looking into it