Problem/Motivation

There's functions deprecated for removal in core 10

Proposed resolution

remove the usage, make sure no mentions left

Remaining tasks

review/commit

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

CommentFileSizeAuthor
#2 3261252-2.patch5.46 KBandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost created an issue. See original summary.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Active » Needs review
Related issues: +#3055193: [Symfony 5] The "Symfony\Component\HttpFoundation\File\MimeType\MimeTypeGuesser" class is deprecated since Symfony 4.3, use "Symfony\Component\Mime\MimeTypes" instead.
FileSize
3261252-2.patch5.46 KB

Some parts should be removed in #3115308: Remove _access_rest_csrf route requirement

Also BC shims could use separate issue #3055193: [Symfony 5] The "Symfony\Component\HttpFoundation\File\MimeType\MimeTypeGuesser" class is deprecated since Symfony 4.3, use "Symfony\Component\Mime\MimeTypes" instead.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Needs review » Reviewed & tested by the community

This looks good for just removing deprecations in system.module itself. There are a lot of test deprecations in test modules but they all need to stay because they test the actual deprecation system.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Status: Reviewed & tested by the community » Fixed

Committed 6385a8f and pushed to 10.0.x. Thanks!

  • alexpott committed 6385a8f on 10.0.x 
    Issue #3261252 by andypost: Remove deprecated system.module functions

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.