Closed (won't fix)
Project:
WSCCI
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
22 May 2012 at 00:14 UTC
Updated:
19 Jun 2012 at 19:45 UTC
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 commentedAlready resolved in core.
Comment #1.0
Crell commentedUpdated issue summary.