Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.7.6

Bug #285 Log::factory() needs to return a reference
Submitted: 2003-11-23 21:10 UTC
From: jon Assigned: cain
Status: Closed Package: HTML_Template_Xipe
PHP Version: 4CVS-2003-11-23 (stable) OS: FreeBSD
Roadmaps: (Not assigned)    
Subscription  


 [2003-11-23 21:10 UTC] jon
Description: ------------ The Log::factory() call returns a reference to the new Log object. This patch corrects the Log::factory() call in Main.php: Index: Main.php =================================================================== RCS file: /repository/pear/HTML_Template_Xipe/Xipe/Main.php,v retrieving revision 1.17 diff -u -r1.17 Main.php --- Main.php 2 Jul 2003 09:13:23 -0000 1.17 +++ Main.php 23 Nov 2003 21:10:54 -0000 @@ -749,7 +749,7 @@ if ($this->logObject==null || !is_object($this->logObject) || !$this->_logFileName) { $this->_logFileName = $this->_compiledFilePrefix.$this->getOption('logFileExtension'); - $this->logObject = Log::factory('file',$this->_logFileName); + $this->logObject = &Log::factory('file',$this->_logFileName); $this->_log('---------------------'); //FIXXME write the options in the log-file but nicer!!! than here $this->_log('options: '.serialize($this->options));

Comments

 [2003-12-01 21:50 UTC] cain at php dot net
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.
 [2009-05-24 04:03 UTC] jon (Jon Parise)
The following patch has been added/updated: Patch Name: 1.php Revision: 1243119791 URL: http://pear.php.net/bugs/patch-display.php?bug=285&patch=1.php&revision=1243119791&display=1