[horde] LDAP Prefs + Horde 3.0 + IMP 4.0 Prob

[Edwin Culp]

I installed the 2nd version of the patch because on my test machine.
I am using Spanish and I am not able save my kronolith and other
prefs.  I had hoped that the problem was that the Spanish default
was somehow breaking the saving of the prefs.  Unfortunately, it
wasn't:-(  The patch works fine and does seem to be a good idea.
Is anyone else seeing the problem with the multivalued attribute and
getting them all written?  Has anyone found a solution/work arround?

Thanks,

ed


Quoting KaalH! <kaalh at smol.org>:

| yeah, I've got a solution ! :)
| 
| The problem is "hordePrefs:: aW1wOnF1b3RlX3ByZWZpeDo+IA=="
| 
| This appends because you are not using english as default language.
| eg: french translation default identity name is "identité par défaut"
| 
| the non-us caracters are the bug cause.
| 
| my proposition/solution is to base64_encode() the entries instead of 
| utf8_encode() them.
| the bad news is that preferences in the ldap tree are no more human
| readable.
| 
| the patch : 
| 
| Index: ldap.php
| ===================================================================
| RCS file: /repository/horde/lib/Prefs/ldap.php,v
| retrieving revision 1.36
| diff -u -r1.36 ldap.php
| --- ldap.php    20 May 2002 11:22:01 -0000      1.36
| +++ ldap.php    22 May 2002 13:19:38 -0000
| @@ -351,7 +351,7 @@
|               * server.
|               */
|              foreach ($result[0][$field] as $prefstr) {
| -                $prefstr = utf8_decode($prefstr);
| +                $prefstr = base64_decode($prefstr);
|  
|                  /* If the string doesn't contain a colon delimeter, skip it.
| 
| */
|                  if (substr_count($prefstr, ':') == 0) {
| @@ -436,7 +436,7 @@
|          foreach($prefs as $pref) {
|              $entry = sprintf("%s:%s:%s", $this->getScope($pref), $pref,
|                               Prefs::getValue($pref));
| -            $new_values[$this->params['field']][] = utf8_encode($entry);
| +            $new_values[$this->params['field']][] = base64_encode($entry);
|          }
| 
| 
| ##################################################################
| 
| another way is to base64_encode only the pref value : 
| horde:identities:[base64'd value]
| then the scope & pref name are still human readable in the ldap tree.
| 
| the patch :
| 
| Index: ldap.php
| ===================================================================
| RCS file: /repository/horde/lib/Prefs/ldap.php,v
| retrieving revision 1.36
| diff -u -r1.36 ldap.php
| --- ldap.php    20 May 2002 11:22:01 -0000      1.36
| +++ ldap.php    22 May 2002 13:30:10 -0000
| @@ -361,6 +361,9 @@
|                  /* Split the string into its scope:name:value components.
| */
|                  list($scope, $pref, $val) = split(':', $prefstr, 3);
|  
| +               /* Decode the pref value */
| +               $pref = base64_decode($pref);
| +
|                  /* Only retrieve this preference if it's in our scope. */
|                  if ((strcmp($scope, $this->scope) == 0) ||
|                      (strcmp($scope, 'horde') == 0)) {
| @@ -435,7 +438,7 @@
|          $new_values = array();
|          foreach($prefs as $pref) {
|              $entry = sprintf("%s:%s:%s", $this->getScope($pref), $pref,
| -                             Prefs::getValue($pref));
| +                             base64_encode(Prefs::getValue($pref)));
|              $new_values[$this->params['field']][] = utf8_encode($entry);
|          }
| 
| 
| 
| 
| -- 
| KaalH!



-------------------------------------------------------------
 http://insourcery.com - Mergence of Business and Technology  
          a "Griffin Plaza Partners, LLC" Company


-------------------------------------------------------------
 http://insourcery.com - Mergence of Business and Technology  
          a "Griffin Plaza Partners, LLC" Company