Re: [commits] Horde branch master updated. 275b9c579ef363227ec8d024975f02c70d33c8a5

Michael J Rubinsky <[email protected]>
Newsgroups gmane.comp.horde.devel
Message-ID <20140425211811.Horde.IHGK5bOeHHBbs5xEPjO-fQ1@h4.theupstairsroom.com>
Quoting Michael M Slusarz <[email protected]>:

> Quoting Michael J Rubinsky <[email protected]>:
>
>> Quoting Michael M Slusarz <[email protected]>:
>>
>>> Quoting "Michael J. Rubinsky" <[email protected]>:
>>>
>>>> The branch "master" has been updated.
>>>> The following is a summary of the commits.
>>>>
>>>> from: 25906b6e43200719eabf42dd606524b1c7bc7393
>>>>
>>>> 275b9c5 This makes much more sense as a default.
>>>>
>>>> Summary:  
>>>> http://github.com/horde/horde/compare/25906b6e43200719eabf42dd606524b1c7bc7393...275b9c579ef363227ec8d024975f02c70d33c8a5
>>>>
>>>> -----------------------------------------------------------------------
>>>>
>>>> commit 275b9c579ef363227ec8d024975f02c70d33c8a5
>>>> Author: Michael J Rubinsky <[email protected]>
>>>> Date:   Fri Apr 25 15:53:53 2014 -0400
>>>>
>>>>  This makes much more sense as a default.
>>>>
>>>> horde/config/prefs.php |    2 +-
>>>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>>>
>>>> http://github.com/horde/horde/commit/275b9c579ef363227ec8d024975f02c70d33c8a5
>>>
>>> That's not going to work.  Chicken/egg problem - you can't grab  
>>> the value of a preference from within the file used to initialize  
>>> the preference value.
>>>
>>> To set a default prefs value dependent on another prefs value you  
>>> have to use something like the prefs_init hook.
>>
>> I half thought that might be the case, but when I tested it, it  
>> successfully set the default. Perhaps it was just the specifics of  
>> how I tested?
>
> It could depend on the order the prefs are stored in prefs.php.  But  
> can't guarantee that order.  (Or else the default identity happens  
> to be 0, and getValue() is returning an empty-ish value).
>
> michael
>

In my case, the default is actually 2, so it's most likely the order.  
Either way, you're right of course. I'll hook it.

-- 
mike
The Horde Project
http://www.horde.org
https://www.facebook.com/hordeproject
https://www.twitter.com/hordeproject

-- 
dev mailing list
Frequently Asked Questions: http://wiki.horde.org/FAQ
To unsubscribe, mail: [email protected]
smime.p7s (application/pkcs7-signature, 5.7 KB) - not displayed
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.