Skip to content

I fixed issue #94 per the comment -- added force_encoding statement#127

Open
aglover wants to merge 1 commit intoflexera-public:masterfrom
aglover:master
Open

I fixed issue #94 per the comment -- added force_encoding statement#127
aglover wants to merge 1 commit intoflexera-public:masterfrom
aglover:master

Conversation

@aglover
Copy link
Copy Markdown

@aglover aglover commented Apr 12, 2012

I had this issue in production and this fixed it for us.

#94

Change is simple -- xml_text.force_encoding 'utf-8'

@konstantin-dzreev
Copy link
Copy Markdown
Contributor

This does not seem to work with ruby 1.8.7

@aglover
Copy link
Copy Markdown
Author

aglover commented Apr 26, 2012

Yep, it isn't an issue on 1.8.7! :) I did not test pre-1.9.2. Sorry about that. Any suggestion other than checking RUBY_VERSION before the call?

@JoshReedSchramm
Copy link
Copy Markdown

Is anyone still looking at this pull request? If not I can pick it up and see if we can fix it in 1.8.7

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants