Re: [phpBT-dev] [PATCH] minor change to installer

[Ulf E. <ulf...@fa...>]

* Ulf Erikson [2005-02-08 21:45]:
> I had some troubles on a server where the installer always reported 
> success on "test database connection". It turned out 'db_error' was 
> defined 'DB_Error'. Is this a known issue/bug for one or another version 
> of MySQL and/or php? Anyway, this patch use DB::isError for error 
> checking and patches isError to handle both cases..

This is due to php5. It seems that 'get_class()' no longer guarantees 
lowercase answers. Since PEAR itself fails to detect errors you get 
rather unpredictable results.. Newer versions of PEAR should fix this. 
Will you please upgrade before the next release candidate?

When trying to use the latest version of PEAR/DB I found that there is 
no PEAR_PATH used. Adding a PEAR_PATH for each include/require (such 
that they look like the included PEAR distro) it all works nicely again.


Next issue will be MySQL 4.1. You need to use mysqli.php instead of 
mysql.php from that version. Do you plan to support it in the release? 
That would require some testing..

-- 
Ulf