[prev in list] [next in list] [prev in thread] [next in thread] 

List:       horde-dev
Subject:    Re: [dev] PHP Notice : Undefined index: gnupg
From:       Jan Schneider <jan () horde ! org>
Date:       2005-09-28 7:34:01
Message-ID: 20050928093401.1nmafjqyxcc8g4g4 () neo ! wg ! de
[Download RAW message or body]

Zitat von Michael M Slusarz <slusarz@bigworm.curecanti.org>:

> Quoting Matt Selsky <selsky@columbia.edu>:
>
>> I can't reproduce this with HEAD.
>>
>> On Tue, Sep 27, 2005 at 03:56:44PM -0400, Serge Grondin wrote:
>>> Hi,
>>>
>>> This patch remove this Notice in
>>> /horde/imp/templates/compose/encrypt_list.inc on line 6.
>>>
>>> The $conf['utils']['gnupg'] is not set in the conf.php file if there is
>>> nothing entered for the file path for the 'gpg' binary.
>>>
>>> Thanks for the excellent work.
>
> I can.  This looks to be more of an issue with the Horde configuration
> creation than with this particular config parameter.  Is there any
> reason we are not creating a "$conf['foo'] = '';" entry in the
> generated conf.php file if an item is blank?  Or is the preferred
> practice to always do an empty() check on those config parameters that
> are not required?

The latter. There was a reason that we don't set empty optional items 
that I can't remember right now.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/


-- 
Horde developers mailing list - Join the hunt: http://horde.org/bounties/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@lists.horde.org
[prev in list] [next in list] [prev in thread] [next in thread]