Problem/Motivation

Follow-up from #3432882: Remove deprecated code in File module we missed removing the following deprecated methods from \Drupal\file\Upload\InputStreamUploadedFile

  • isValid()
  • getErrorMessage()
  • getError()

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3446967

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

kim.pepper created an issue. See original summary.

kim.pepper opened merge request !8057

kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
commented
Status: Active » Needs review
alexpott’s picture

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

looks great - given these are bad method calls throwing implementations it's not hard to remove them.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Issue tags: +Major version only

  • longwave committed cde2856c on 11.0.x 
    Issue #3446967 by kim.pepper, alexpott: [11.x] Remove deprecated methods...

  • longwave committed c5ea6561 on 11.x 
    Issue #3446967 by kim.pepper, alexpott: [11.x] Remove deprecated methods...
longwave’s picture

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

Committed and pushed c5ea6561fc to 11.x and cde2856cef to 11.0.x. Thanks!

Status: Fixed » Closed (fixed)

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

quietone’s picture

quietone commented
Version: 11.x-dev » 11.0.x-dev

Update version to the branch this applies to.