Fixed the issue with rails integration for fixnums #3

merged 1 commit into from Mar 29, 2012


None yet
2 participants

ratnikov commented Mar 29, 2012

Seems like the escaper takes only strings, so when feeding it a fixnum it would blow up.

Fixed by calling to_s on code when feeding into the escaper.

Reproduced the issue in a special spec that checks rails integration

This comment has been minimized.

Show comment Hide comment

igrigorik Mar 29, 2012


Ah, nice catch.


igrigorik commented Mar 29, 2012

Ah, nice catch.

igrigorik added a commit that referenced this pull request Mar 29, 2012

Merge pull request #3 from ratnikov/master
Fixed the issue with rails integration for fixnums

@igrigorik igrigorik merged commit 78771d8 into igrigorik:master Mar 29, 2012

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