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.

CommentFileSizeAuthor
kernel-file_download-cleanup.patch1.06 KBeffulgentsia
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Crell’s picture

Status: Needs review » Closed (won't fix)

Already resolved in core.

Crell’s picture

Issue summary: View changes

Updated issue summary.