adium 3176:bd12f98a66c3: If no password is set, use NULL, not ""...
commits at adium.im
commits at adium.im
Tue Apr 27 16:09:44 UTC 2010
details: http://hg.adium.im/adium/rev/bd12f98a66c3
revision: 3176:bd12f98a66c3
author: Evan Schoenberg
date: Tue Apr 27 11:09:33 2010 -0500
If no password is set, use NULL, not "", as the password to send libpurple, as code in various places checks for NULL rather than a 0-length string. This, in turn, fixes prompting for the password if no password is entered for a Jabber account which needs one. The form-based request is ugly, but at least now it's functional.
diffs (12 lines):
diff -r d23b7f99e0d0 -r bd12f98a66c3 Plugins/Purple Service/CBPurpleAccount.m
--- a/Plugins/Purple Service/CBPurpleAccount.m Mon Apr 26 20:14:11 2010 -0500
+++ b/Plugins/Purple Service/CBPurpleAccount.m Tue Apr 27 11:09:33 2010 -0500
@@ -1932,7 +1932,7 @@
- (void)continueConnectWithConfiguredProxy
{
//Set password and connect
- purple_account_set_password(account, [password UTF8String]);
+ purple_account_set_password(account, ([password length] ? [password UTF8String] : NULL));
//Set our current status state after filtering its statusMessage as appropriate. This will take us online in the process.
AIStatus *statusState = [self valueForProperty:@"StatusState"];
More information about the commits
mailing list