Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Google Contacts CSV has BOM #1

Open
schorsch opened this issue Nov 17, 2011 · 2 comments
Open

Google Contacts CSV has BOM #1

schorsch opened this issue Nov 17, 2011 · 2 comments
Labels
bug

Comments

@schorsch
Copy link
Member

The ruby csv parser fails when importing google contacts export because it has a UTF16 BOM in it.

We should strip all BOM, and if present autodetect the incoming encoding and convert it to utf-8
@vijendra vijendra mentioned this issue Jan 26, 2012
Closed
vijendra added a commit to vijendra/CSV-Import that referenced this issue Jan 26, 2012
@vijendra
CSV import issue salesking#1
7a14e02
@vijendra
Copy link
Contributor

Can I get a test problematic CSV file?

vijendra added a commit to vijendra/CSV-Import that referenced this issue Jan 26, 2012
@vijendra
CSV import fix. Issue salesking#1
5ed5df1
@vijendra vijendra mentioned this issue Jan 26, 2012
Merged
@schorsch
Copy link
Member Author

i changed the toppic and checked the BOM issue, which is not resolved yet.

Here is an example of how it could be done, but in ruby192 there might be other methods
http://self.d-struct.org/195/howto-remove-byte-order-mark-with-ruby-and-iconv

I will upload a failing csv, too

vijendra added a commit to vijendra/CSV-Import that referenced this issue Jan 28, 2012
@vijendra
BOM handling issue salesking#1
574ca1c
vijendra added a commit to vijendra/CSV-Import that referenced this issue Jan 28, 2012
@vijendra
BOM handling issue salesking#1
66cb370
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@schorsch @vijendra