Comment 3 for bug 544316

Revision history for this message
Malcolm Fitzgerald (malcolm-notyourhomework) wrote : Re: [Bug 544316] [NEW] no need to define ICONURL into cgiStyleDocsSample and wbError

On 23/03/2010, at 10:32 AM, Xavier Brochard wrote:

> Le Lundi 22 Mars 2010 23:43:04, vous avez écrit :
>> On 23/03/2010, at 4:02 AM, Xavier Brochard wrote:
>>> Public bug reported:
>>>
>>> in cgiStyleDocsSample no need to hard coding ICONURL, it works perfectly
>>> without this line: $ICONURL = "/icons/default"; # why is it hard encoded
>>> ?
>>>
>>> in wbError ICONURL is further defined before call to printFoot()
>>>
>>> $ICONURL = $ICONURL || '/icons/default';
>>
>> This code only ever worked on rent-a-database.com. It's probably a
>> quick-fix that was never cleaned up.
>
> I'm not sure for wbError
> because a few lines upon there is a commented call to mainDefaults() to fix
> ICONURL. Can ICONURL be undefined sometimes ?

I remember a time when I saw an error message for ICONURL on my installations. It has been such a long time since I've seen it. mainDefaults() handles all that now.

Malcolm