aboutsummaryrefslogtreecommitdiffstats
path: root/Frameworks
diff options
context:
space:
mode:
authorMax <post@wickenrode.com>2016-02-06 00:00:18 +0100
committerMax <post@wickenrode.com>2016-02-06 00:00:18 +0100
commit6457da276ba4a46dcf7e52c0b25edb42d751a13e (patch)
tree37fa21f877a75ca7721a654b7035f9f707fe8e6d /Frameworks
parent21062b7e1c72429d7823901280848a6cd14dc3a6 (diff)
downloadsequelpro-6457da276ba4a46dcf7e52c0b25edb42d751a13e.tar.gz
sequelpro-6457da276ba4a46dcf7e52c0b25edb42d751a13e.tar.bz2
sequelpro-6457da276ba4a46dcf7e52c0b25edb42d751a13e.zip
Add another fallback for charset detection (which might improve compatibility with Sphinx)
Diffstat (limited to 'Frameworks')
-rw-r--r--Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m19
1 files changed, 18 insertions, 1 deletions
diff --git a/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m b/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m
index b09a27ee..e102c24d 100644
--- a/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m
+++ b/Frameworks/SPMySQLFramework/Source/SPMySQLConnection.m
@@ -979,11 +979,28 @@ static uint64_t _elapsedMicroSecondsSinceAbsoluteTime(uint64_t comparisonTime)
// connection, it may be overridden by init_connect commands or connection state changes.
// Default to latin1 for older server versions.
NSString *retrievedEncoding = @"latin1";
+ // character_set_results is the charset the strings received from the server will be in
if ([variables objectForKey:@"character_set_results"]) {
retrievedEncoding = [variables objectForKey:@"character_set_results"];
- } else if ([variables objectForKey:@"character_set"]) {
+ }
+ // not used in 4.1+ (?)
+ else if ([variables objectForKey:@"character_set"]) {
retrievedEncoding = [variables objectForKey:@"character_set"];
}
+ // character_set_client is the charset the server expects strings transmitted by us to be in
+ else if ([variables objectForKey:@"character_set_client"]) {
+ retrievedEncoding = [variables objectForKey:@"character_set_client"]; // fallback for sphinxql
+ }
+ // character_set_connection is used internally by the server for comparisons.
+ // String literals (without a cast) will always be converted from character_set_client to character_set_connection first.
+ // As an example:
+ // * Use a client with "SET NAMES utf8"
+ // * Do a "set @@session.character_set_connection = 'latin1';"
+ // * Finally try a "SELECT '犬';" (also try "select _utf8'犬';" for completeness)
+ // * The result will just show a "?"
+ // So even though we told the server that the client uses utf8 and the results
+ // should be encoded in utf8, too, the character got lost.
+ // This happened because the server did a roundtrip of utf8 -> latin1 -> utf8.
// Update instance variables
if (encoding) [encoding release];