Fabien Salathe comments

Results 109 comments of


                                            Fabien Salathe

trafficstars

[Bug]: The PropertyInfo component was not able to guess the type of Company::$edAt. You may need to add a `@var` annotation or use `@OA\Property(type="")` to make its type explicit

For us this break with Symfony 7.1, it's a blocker to upgrade.

fixes deprecations for php 8.4

Looks good, thank you!

Add PHP 8.4 support for 1.7.x by dropping support for PHP < 7.1

You can use dev-master to have the latest version compatible with PHP8.4 (not the best practice but there is no other tags)

Remove deprecation notice on nullable properties in method signatures

There is a PR there: https://github.com/checkout/checkout-sdk-php/pull/285 but it does more than just fixing the issues. IMO it would be easier to release a new minor (with PHP 7.1) with only...

Remove deprecation notice on nullable properties in method signatures

https://packagist.org/packages/checkout/checkout-sdk-php/php-stats#3 FYI almost nobody is using PHP

Remove deprecation notice on nullable properties in method signatures

Okay thank you, do you know when it is plan? It's currently blocking us from upgrading

Remove deprecation notice on nullable properties in method signatures

If anyone else has the same issue, here is the fork working with PHP 8.4: https://packagist.org/packages/bafs/checkout-sdk-php

Replace removed DebugStack with LoggingMiddleware

@mpdude could this be merged?

addText() function and text formatting (new line "\n")

I used `"\n"` and then the regex `$text = preg_replace('~\R~u', '', $text);`.