Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
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.