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.
Remove all deprecated code in core file system e.g. under core/lib/Drupal/Core/File
and related tests.
Comment | File | Size | Author |
---|---|---|---|
#9 | 3093564-9.drupal.Remove-all-deprecated-code-in-core-File-system.patch | 7.12 KB | longwave |
#6 | After apply patch.png | 146.12 KB | Vinodhini.E |
#4 | deprecated_filesystem-3093564-4.patch | 19.13 KB | Dinesh18 |
#2 | deprecated_filesystem-3093564-2.patch | 3.91 KB | Dinesh18 |
Comments
Comment #2
Dinesh18 CreditAttribution: Dinesh18 as a volunteer commentedHere is the patch which removes the deprecated code. Kindly review
Comment #4
Dinesh18 CreditAttribution: Dinesh18 as a volunteer commentedHere is the updated patch. It should pass.
Comment #5
Dinesh18 CreditAttribution: Dinesh18 as a volunteer commentedComment #6
Vinodhini.E CreditAttribution: Vinodhini.E at UniMity Solutions Pvt Limited commented@Dinesh18, I have applied a patch and tested using drupal check module. I am getting two errors, please refer below image.
Comment #7
Berdirthis is actually more about file.inc removals, which is #3062757: Remove deprecated legacy include files from Drupal 9 , so we likely need to wait on that.
Comment #8
BerdirThat issue landed, so we can reroll this.
Comment #9
longwaveRerolled.
Comment #10
BerdirPretty straight-forward now, can't see anything else in the File component.
#6: This is not the same as deprecation *usage* removal patches. This removes the actual deprecated code and drupal-check is not really relevant for this. To test patches like this for Drupal 9, you need to instead search for @deprecated and @trigger_error. The things you reported are related to new symfony deprecations and not something to be fixed here.
Comment #12
catchFixed on commit:
Committed 8d6d914 and pushed to 9.0.x. Thanks!