1
period3
case wrong in 2.0.6
  • 2004/5/13 16:29

  • period3

  • Just popping in

  • Posts: 2

  • Since: 2004/5/13


Hi,

I'm reading through the XOOPS 'class/database/databasefactory.php' file, and I notice that
it is instantiating a new class instance as follows:

$class = 'Xoops'.ucfirst(XOOPS_DB_TYPE).'DatabaseProxy';

Given that XOOPS_DB_TYPE is entirely lowercase (eg.mysql), then this would translate to:
$class = 'XoopsMysqlDatabaseProxy';

However, this class does not exist. The only class I have
is in mysqldatabase.php, and is called XoopsMySQLDatabaseProxy. The case is wrong.

Still, XOOPS doesn't seem to have any problems, so I guess it works. It appears PHP isn't case sensitive with class names. Still, I think it would be a good idea to fix this so future releases of PHP don't break Xoops.

Login

Who's Online

350 user(s) are online (287 user(s) are browsing Support Forums)


Members: 0


Guests: 350


more...

Donat-O-Meter

Stats
Goal: $100.00
Due Date: Nov 30
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $100.00
Make donations with PayPal!

Latest GitHub Commits