[cvs] commit: horde/docs CHANGES horde/config prefs.php.dist framework/Horde/Horde Registry.php framework/Prefs Identity.php Prefs.php framework/Prefs/Prefs imsp.php ldap.php session.php sql.php

Chuck Hagenbuch chuck at horde.org
Wed Nov 22 20:47:04 PST 2006


chuck       2006-11-22 20:47:04 PST

  Modified files:
    docs                 CHANGES 
    config               prefs.php.dist 
    Horde/Horde          Registry.php 
    Prefs                Identity.php Prefs.php 
    Prefs/Prefs          imsp.php ldap.php session.php sql.php 
  Log:
  [cjh] Rewrite the Prefs code to keep the same Prefs object through multiple
  scope changes, instead of re-loading the Horde prefs, or prefs for
  a scope that had already been loaded. Fixes bugs when making changes
  to preferences and then switching scopes again (Bug #2838).
  
  This is a big change to the prefs system, and while I've tested all of
  it as best as I can, and the SQL driver I'm pretty confident in, this
  needs testing before we even think about merging it. It would all be
  BC, though.
  
  Please let me know (or update the bug) if this works or doesn't work
  for you, especially people using lots of hooks and other more edge
  cases, and also let me know if you think it should be merged or not.
  
  Revision  Changes    Path
  1.289     +8 -7      framework/Horde/Horde/Registry.php
  1.5       +9 -18     framework/Prefs/Identity.php
  1.181     +172 -154  framework/Prefs/Prefs.php
  1.14      +70 -92    framework/Prefs/Prefs/imsp.php
  1.111     +70 -128   framework/Prefs/Prefs/ldap.php
  1.41      +12 -37    framework/Prefs/Prefs/session.php
  1.122     +92 -147   framework/Prefs/Prefs/sql.php
  1.103     +13 -5     horde/config/prefs.php.dist
  1.893     +4 -0      horde/docs/CHANGES

  Chora Links:
  http://cvs.horde.org/diff.php/framework/Horde/Horde/Registry.php?r1=1.288&r2=1.289&ty=u
  http://cvs.horde.org/diff.php/framework/Prefs/Identity.php?r1=1.4&r2=1.5&ty=u
  http://cvs.horde.org/diff.php/framework/Prefs/Prefs.php?r1=1.180&r2=1.181&ty=u
  http://cvs.horde.org/diff.php/framework/Prefs/Prefs/imsp.php?r1=1.13&r2=1.14&ty=u
  http://cvs.horde.org/diff.php/framework/Prefs/Prefs/ldap.php?r1=1.110&r2=1.111&ty=u
  http://cvs.horde.org/diff.php/framework/Prefs/Prefs/session.php?r1=1.40&r2=1.41&ty=u
  http://cvs.horde.org/diff.php/framework/Prefs/Prefs/sql.php?r1=1.121&r2=1.122&ty=u
  http://cvs.horde.org/diff.php/horde/config/prefs.php.dist?r1=1.102&r2=1.103&ty=u
  http://cvs.horde.org/diff.php/horde/docs/CHANGES?r1=1.892&r2=1.893&ty=u


More information about the cvs mailing list