shuhaib864 / xmpphp

Automatically exported from code.google.com/p/xmpphp
0 stars 0 forks source link

Rename Log Class - Name is to common and leads to problems #35

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Install PEAR
2. Add PEAR to the default include path like its by most hosters

What is the expected output? What do you see instead?
Expected would be the inclusion of the XMPPHP Log Class, instead the PEAR
Log.php will be included. This leads to various PHP errors like missing
constants or methods.

What version of the product are you using? On what operating system?
Tested with the latest SVN release of XMPPHP on openSUSE 11.0 with PHP
5.2.5 running on Apache 2.2.6.

Please provide any additional information below.
To fix this issue i would suggest to either prefix the Log class name with
the XMPPHP_ namespace or prefix the require paths with the XMPPHP path.
(require_once 'Log.php'; to require_once 'XMPPHP/Log.php'; or require_once
'XMLObj.php'; to require_once 'XMPPHP/XMLObj.php';)

The best would be to apply both fixes and also apply the namespacing to all
other classes to provide further missinclusion and to be more standard
compliant ;)

Original issue reported on code.google.com by ota.ma...@gmail.com on 3 Sep 2008 at 12:41

GoogleCodeExporter commented 8 years ago
Log class name is XMPPHP_Log

Original comment by nathanfr...@gmail.com on 26 Nov 2008 at 10:04