Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
#1399846: Make unused file 'cleanup' configurable removed the use of Drupal\file\FileStorage::retrieveTemporaryFiles().
Proposed resolution
So we can remove the method from the class and interface. Also we need to remove Drupal\file\Tests\RetrieveTemporaryFilesTest.
Remaining tasks
| Task | Novice task? | Contributor instructions | Complete? | |
|---|---|---|---|---|
| Create a patch | Y | Instructions | ||
| Review patch to ensure that it fixes the issue, stays within scope, is properly documented, and follows coding standards | Y | Instructions | Y |
User interface changes
No
API changes
Removal of Drupal\file\FileStorage::retrieveTemporaryFiles()
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | file-remove-temp-2278025-3-D8.patch | 3.96 KB | duellj |
Comments
Comment #1
alexpottComment #2
alexpottComment #3
duellj CreditAttribution: duellj commentedAttached patch removes retrieveTemporaryFiles() from FileStorage class and interface and removes corresponding test class.
Comment #4
biggm CreditAttribution: biggm commentedReviewing now.
Comment #5
biggm CreditAttribution: biggm commentedComment #6
biggm CreditAttribution: biggm commentedComment #7
alexpottCommitted cafa95e and pushed to 8.x. Thanks!