Fix issue #64 w/ test #99

merged 1 commit into from Jul 28, 2012

2 participants


Fixes issue #64. Uses the same fix as in pull request #68 but adds a test to verify that the change works and that nothing else is broken

@ciaranj ciaranj merged commit d8d1e0b into ciaranj:master Jul 28, 2012

Thank you, although please note I applied some changes to your pull request (your test still passes though.) I don't think you need to do the encoding twice that you're doing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment