The function drupal_add_http_header have two params, but the module send only one.

Example now:
drupal_add_http_header('Content-Type: text/csv; utf-8');

Example correct implementation:
drupal_add_http_header('Content-Type', 'text/csv; utf-8');

CommentFileSizeAuthor
#1 bad_implementation_of_drupal_add_http_header-1863948-0.patch759 bytesMaxMendez
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MaxMendez’s picture

MaxMendez CreditAttribution: MaxMendez commented
Status: Active » Reviewed & tested by the community
FileSize
bad_implementation_of_drupal_add_http_header-1863948-0.patch759 bytes

Patch here, thanks for this module

anarcat’s picture

anarcat CreditAttribution: anarcat commented
Status: Reviewed & tested by the community » Fixed

Please to not mark patches RTBC yourself, usually you mark needs review and someone else tests it. :) But since the patch is quite trivial and probably works after review, I just committed it.

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.