Problem/Motivation

Drupal\file\Plugin\rest\resource\FileUploadResource::__construct() contains a BC layer for one of its arguments.

Steps to reproduce

Proposed resolution

Remove the fallback.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3295626-2.patch2.14 KBlongwave

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Needs review
StatusFileSize
new 3295626-2.patch2.14 KB
catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks good.

  • decfbd1 committed on 10.0.x 
    Issue #3295626 by longwave, catch: Remove BC layer in Drupal\file\Plugin...
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed
Issue tags: +Deprecation Removal

Agreed, looks good, committed.

catch’s picture

catch
he/him
Primary language English
commented
Status: Fixed » Reviewed & tested by the community

@Gábor Hojtsy this looks like the 10.1.x commit is missing.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Version: 10.0.x-dev » 10.1.x-dev
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed ac0b8ca and pushed to 10.1.x. Thanks!

  • alexpott committed ac0b8ca on 10.1.x 
    Issue #3295626 by longwave, catch: Remove BC layer in Drupal\file\Plugin...

Status: Fixed » Closed (fixed)

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