\Drupal\file\Upload\FileUploadHandler::handleFileUpload() should alway check that the uploaded file is valid

We will need to update / maybe delete and create a new CR - https://www.drupal.org/node/3375456 →

From @kim.pepper on gitlab:

I don't think we should revert the deprecations on UploadedFileInterface. ::isValid(), ::getError() and ::getErrorMessage() don't really apply to input stream uploaded files.

I don't agree I think we should keep them. Whatever we wrap input stream uploaded files can return TRUE from isValid().

The alternative is to make an addition to UploadedFileInterface and make that responsible for the validation. I think that this might be a good way to go as it put the responsibility in the correct place.

kim.pepper → made their first commit to this issue’s fork.

I think the approach for validation here is a better one that the current version.

RTBC for me.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Rebased and resolved the conflicts. Please review.

I redid the merge and got the same diff as @sakthi_dev so nice resolving.

Committed to 11.x and backported to 10.3.x

larowlan → closed merge request !7289

I've updated the CR to reflect the changes - see https://www.drupal.org/node/3375456 →

I created a new issue to follow this same pattern for moveUploadedFile() 📌 Provide a method for FileUploads to be moved Active

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