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

[KaalH!]

I've just updated the ldap.php patch. 

Changes : 
* no more utf8_(en|de)code , base64 is enough.
* bugfix : apps with no prefs stored were prevent to retrieve shared prefs.

http://kaalh.smol.org/horde/ldap.php.diff

-- 
KaalH!


Surlignage 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