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

[Edwin Culp]

Kaalh,

Thanks.  I patched my test installation and did some quick testing and 
it seems to fix my prefs problem. 

I would sure like to see it committed including the base64_[en|de]code()

Thanks again,

ed



Quoting Kaalh! <kaalh at smol.org>:

| well ... got some new stuff ;)
| 
| I've modified the storage format, now it's using one multi-valued attribute 
| per application ( hordePrefs, impPrefs, turbaPrefs ... )
| $conf['prefs']['params']['field'] is no longer used
| 
| this patch assume that all shared prefs are stored in hordePrefs attribute, 
| hope this is ok.
| 
| the patch (contain the 2nd version of my previous patch modified) : 
| http://kaalh.smol.org/horde/ldap.php.diff
| 
| the schema diff :
| http://kaalh.smol.org/horde/horde.schema.diff
| 
| now you deal with all applications preferences.
| 
| -- 
| Kaalh!
| 
| 
| Surlignage Edwin Culp <eculp at MexComUSA.Net>:
| 
| > 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
| > 
| > -- 
| > Horde mailing list
| > Frequently Asked Questions: http://horde.org/faq/
| > To unsubscribe, mail: horde-unsubscribe at lists.horde.org
| > 
| 
| -- 
| Horde mailing list
| Frequently Asked Questions: http://horde.org/faq/
| To unsubscribe, mail: horde-unsubscribe at lists.horde.org
| 


-----------------------------------------------------------------------
       The illiterate of the 21st century will not be
             those who cannot read and write,
       but those who cannot learn, unlearn and relearn.
                                                --Alvin Toffler

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