| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.1.2 |
| Bug #284 | Log::factory() needs to return a reference | ||
|---|---|---|---|
| Submitted: | 2003-11-23 21:06 UTC | ||
| From: | jon | Assigned: | quipo |
| Status: | Closed | Package: | DB_QueryTool |
| PHP Version: | 4CVS-2003-11-23 (stable) | OS: | FreeBSD |
| Roadmaps: | (Not assigned) | ||
[2003-11-23 21:06 UTC] jon
Description:
------------
The Log::factory() call returns a reference to the new Log object. This patch corrects the Log::factory() call in Query.php:
Index: Query.php
===================================================================
RCS file: /repository/pear/DB_QueryTool/QueryTool/Query.php,v
retrieving revision 1.33
diff -u -r1.33 Query.php
--- Query.php 23 Sep 2003 11:28:13 -0000 1.33
+++ Query.php 23 Nov 2003 21:06:28 -0000
@@ -1792,7 +1792,7 @@
return;
}
if (!$this->_logObject) {
- $this->_logObject = Log::factory('file',$this->options['logfile']);
+ $this->_logObject = &Log::factory('file',$this->options['logfile']);
}
if ($text==='start query' || $text==='end query') {
Comments
[2004-03-25 14:27 UTC] quipo
This bug has been fixed in CVS.
In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.
In case this was a pear.php.net website problem, the change will show
up on the website in short time.
Thank you for the report, and for helping us make PEAR better.