no need to define ICONURL into cgiStyleDocsSample and wbError

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.

Malcolm

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 ?

Xavier
<email address hidden> - 09 54 06 16 26

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