search

tulsawebdevs / django-multi-gtfs

Django app to import and export General Transit Feed Specification (GTFS)
http://tulsawebdevs.org/
Apache License 2.0
50 stars 32 forks source link

Strip " (issue #64) #65

Closed jclgoodwin closed 6 years ago

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-0.4%) to 99.591% when pulling adad52fa32dd160681f22e6eca64a41a6c845258 on jclgoodwin:strip-quotes into 434fd5b0fe1d2b0c571e37ac700a1c0e3b3332d9 on tulsawebdevs:master.

jwhitlock commented 6 years ago

Thanks for the PR!

There's some py2 / py3 differences in unicode string processing that would require some small changes. There's many examples in the code.

However, I'm closing in favor of PR #68, which handles this and similar issues at the CSV reader instead of in the value processor. I copied your test case to show that this also handles issue #64.