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.

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

Fix build #430

Merged
merged 8 commits into from
Dec 10, 2013
Merged

Fix build #430

merged 8 commits into from
Dec 10, 2013

Conversation

bf4
Copy link
Collaborator

@bf4 bf4 commented Dec 9, 2013

No description provided.

@ghost ghost assigned bf4 Dec 9, 2013
@bf4
Copy link
Collaborator Author

bf4 commented Dec 10, 2013

SQLITE3 tests failing on Travis, unable to duplicate locally or fix on Travis...

#<Taggablodel id: 2, name: "Taggable 1", type: nil>
#<ActiveRecord::StatementInvalid: SQLite3::SQLException: unrecognized token: "'o": SELECT "tags".* FROM "tags"  WHERE (lower(name) = 'o' OR lower(name) = 'two')>

if defined?(Encoding)
string.to_s.force_encoding('BINARY')
else
string.to_s.mb_chars
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SQLITE3 queries require 8 bit ascii, but mb_chars seems to occasionally render 'one' as some invalid utf-8 'o', which crashes sqlite3. This fixes that.

bf4 added a commit that referenced this pull request Dec 10, 2013
@bf4 bf4 merged commit 0e6742c into mbleigh:master Dec 10, 2013
@bf4 bf4 deleted the fix_build branch December 10, 2013 18:20
tekniklr pushed a commit to tekniklr/acts-as-taggable-on that referenced this pull request Mar 19, 2021
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.

2 participants