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.
This branch changes file_download() from calling the drupal_access_denied()/drupal_not_found() functions to throwing the Symfony exceptions directly. However, it does not change the corresponding PHPDoc to match, and there's follow-up issues to sweep all of core to do this (#1587850: Replace drupal_not_found() with throw NotFoundHttpException, #1591604: Replace drupal_access_denied() with throw AccessDeniedHttpException), so I suggest removing this outlier from the initial kernel patch. Here's the revert patch to do so.
Comment | File | Size | Author |
---|---|---|---|
kernel-file_download-cleanup.patch | 1.06 KB | effulgentsia |
Comments
Comment #1
Crell CreditAttribution: Crell commentedAlready resolved in core.
Comment #1.0
Crell CreditAttribution: Crell commentedUpdated issue summary.