As we discovered recently UTF-8 files containing the BOM do not have the character stripped by PHP when read from the filesystem or included in code. This affects certain import files used by Feeds, in particular the column names of csv files can become mangled.

Patch forthcoming which removes the BOM character as part of the Fetcher step.

CommentFileSizeAuthor
#2 953538.2.patch3.76 KByhahn
#1 953538.1.patch3.6 KByhahn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yhahn’s picture

yhahn CreditAttribution: yhahn commented
FileSize
953538.1.patch3.6 KB
yhahn’s picture

yhahn CreditAttribution: yhahn commented
FileSize
953538.2.patch3.76 KB

V2 of patch, keeps changes contained to Fetcher classes by using sanitizeRaw() and sanitizeFile() methods.

alex_b’s picture

alex_b CreditAttribution: alex_b commented
Status: Needs review » Reviewed & tested by the community

Rock!

yhahn’s picture

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

http://drupal.org/cvs?commit=441930

Status: Fixed » Closed (fixed)

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