[kronolith] reminder.php and @localhost email problem

[W. Tasin]

Hi all,

I tried to trace back the problem with reminder.php not getting the
default identity of a user.

The problem - in my case using preferences from sql backend - is the
character conversion call in Prefs_sql::_convertFromDriver($value)
respectively the last parameter inside String::convertCharset(...).

I'm using the german language in my horde/imp/kronolith installation, so
"Standardidentität" (i. e. "default identity") is used in
"horde_prefs"-database.

The conversion from the serialized string version
"....{s:2:"id";s:18:"Standardidentität";s:8:"fullname";s:8:"W.
Tasin";..." to the database entry works fine, but backwards from
database entry to serialized string doesn't work.

The resulting serialized string obtained from the sql database is:
"....{s:2:"id";s:18:"Standardidentität";s:8:"fullname";s:8:"W.
Tasin";..." and so the unserialize-call won't work anymore, because the
string now has the length 17.

My patch correct this behaviour, but I don't know if this should be done
in String::convertCharset(); I leave this decision to one of the horde
developer team...

HTH
Ciao

Walter



-------------- next part --------------
--- framework/Prefs/Prefs/sql.php	Fri Mar 12 17:10:27 2004
+++ framework/Prefs/Prefs/sql.php	Tue Mar 30 09:38:41 2004
@@ -362,7 +362,7 @@
      */
     function _convertFromDriver($value) 
     {
-        return String::convertCharset($value, $this->_params['charset']);
+        return String::convertCharset($value, $this->_params['charset'], NLS::getCharset());
     }
 
     /**