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

Don't call deprecated code from non-deprecated code #10503

Closed
2 tasks
maxime-rainville opened this issue Sep 11, 2022 · 2 comments
Closed
2 tasks

Don't call deprecated code from non-deprecated code #10503

maxime-rainville opened this issue Sep 11, 2022 · 2 comments

Comments

@maxime-rainville
Copy link
Contributor

maxime-rainville commented Sep 11, 2022

Some of our deprecated methods and classes might still be called from code we do not intent to remove in CMS5. To facilitate upgrades, we should aim to not use deprecated code within CMS4.

Acceptance criteria

  • Update code so that deprecation methods and classes are no longer being used where possible
  • Scope is core modules

Recommended approach

  • Using the 4 branch and using CI, turn on deprecation warnings for silverstripe/installer and silverstripe/recipe-kitchen-sink and see what deprecation warnings bubble up
  • Ensure that we have a reliable way to surface deprecation warnings from up-stream dependencies such as symfony and guzzle e.g. investigate using this https://symfony.com/doc/current/components/phpunit_bridge.html#usage

Notes

  • In some instances there will be no immediate migration path for code in 4 because the new API only exists in 5
@maxime-rainville
Copy link
Contributor Author

Let's get #10500 done first before we scope this one.

@emteknetnz
Copy link
Member

Duplicate of #10542

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

No branches or pull requests

3 participants