Problem/Motivation
@deprecated things are deprecated, including _drupal_add_http_header() and other header-related functions from bootstrap.inc.
HTTP headers are now handed by the Symfony Request/Response objects. Let's finish the job of encapsulating responses by removing the global header-related functions.
The affected functions include:
drupal_set_preferred_header_name()drupal_add_http_header()- Depends on the status ofdrupal_process_attached()#2382545: Cleanup drupal_process_attached and _drupal_add_* documentationdrupal_get_http_header()drupal_page_header()drupal_send_headers()
Note that FinishResponseSubscriber calls drupal_get_http_header(), in order to retain backward compatibility, which will be obsoleted by this patch.
Proposed resolution
Remove these functions from Drupal core.
Remaining tasks
User interface changes
API changes
Beta phase evaluation
| Issue category | Task because this issue does not fix anything that's broken, other than deprecation. |
|---|---|
| Issue priority | Normal because it's not a pressing issue. |
| Prioritized changes | Prioritized because it removes @deprecated code. |
Comments
Comment #1
mile23Comment #2
mile23Immediately blocked by this: #2467759: Refactor drupal_process_attached() so it doesn't depend on drupal_add_http_header()
Comment #3
wim leers#2467759: Refactor drupal_process_attached() so it doesn't depend on drupal_add_http_header() already removed
_drupal_add_http_header()and_drupal_get_http_header()..AFAICT this is all done? :)
Comment #4
mile23Indeedy. Looks like all the mentioned functions have disappeared.