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.4 deprecations #86

Closed
2 tasks done
lekoala opened this issue Nov 21, 2024 · 1 comment
Closed
2 tasks done

php 8.4 deprecations #86

lekoala opened this issue Nov 21, 2024 · 1 comment

Comments

@lekoala
Copy link

lekoala commented Nov 21, 2024

Module version(s) affected

2

Description

Running with php 8.4 gives deprecation errors that are easily fixable

Deprecation Notice: SilverStripe\VendorPlugin\Methods\CopyMethod::__construct(): Implicitly marking parameter $filesystem as nullable is deprecated, the explicit nullable type must be used instead
Deprecation Notice: SilverStripe\VendorPlugin\Methods\SymlinkMethod::__construct(): Implicitly marking parameter $filesystem as nullable is deprecated, the explicit nullable type must be used instead

How to reproduce

install php 8.4
run composer
see the deprecation error

Possible Solution

this is going to probably be the case on many modules so maybe a coordinated action using rector automation would be the best path

Additional Context

No response

Validations

  • Check that there isn't already an issue that reports the same bug
  • Double check that your reproduction steps work in a fresh installation of silverstripe/installer (with any code examples you've provided)
@GuySartorelli
Copy link
Member

Closing as a duplicate of silverstripe/.github#331
We haven't added PHP 8.4 support yet - and when we do it will be for CMS 6 only as per the major release policy.

@GuySartorelli GuySartorelli closed this as not planned Won't fix, can't repro, duplicate, stale Nov 25, 2024
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