[dev] Re: [cvs] commit: framework/Horde/Horde Registry.php

[Michael M Slusarz]

Quoting Chuck Hagenbuch <chuck at horde.org>:

> Quoting Michael M Slusarz <slusarz at curecanti.org>:
>
>>   Log:
>>   very, very bad - this completely nuked an application's 
>> preferences if called after the application had been initialized.
>
> Perhaps something like this:
>
>          if ($app == $this->_currentApp) {
>              return false;
>          }
>
> ... should be added to the loadPrefs() definition? (might want it to return
> true, but still).

It seems to work okay right now, so I'll hold off on adding this check 
for now.

> Also, why is the loadprefs *ever* needed in getImageDir()?

Don't know.  I assume Marko put it in there for a reason - he is probably the
better person to ask.  i didn't want to remove it for fear that it 
would fix my
problem but break something else.

michael

_______________________________________
Michael Slusarz [slusarz at curecanti.org]