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

Bug #18045 Profiler::enterSection is broken
Submitted: 2010-11-12 05:44 UTC
From: jszczypk Assigned: doconnor
Status: Assigned Package: Benchmark (version 1.2.9)
PHP Version: 5.3.3 OS: OS X
Roadmaps: (Not assigned)    
Subscription  


 [2010-11-12 05:44 UTC] jszczypk (Janusz Szczypka)
Description: ------------ I am getting a lot of "Illegal offset type" for lines 423 and 426 of Profiler.php Looks like somebody forgot to put ["name"] after $item. Expected result: ---------------- Profiler.php lines 420-430 if (isset($this->_callers[$name][$item["name"]])) { $this->_callers[$name][$item["name"]]++; } else { $this->_callers[$name][$item["name"]] = 1; } if (isset($this->_calls[$item["name"]][$name])) { $this->_calls[$item["name"]][$name]++; } else { $this->_calls[$item["name"]][$name] = 1; }

Comments

 [2011-12-10 09:57 UTC] doconnor (Daniel O'Connor)
-Status: Open +Status: Closed -Assigned To: +Assigned To: doconnor
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.
 [2012-01-12 19:20 UTC] jszczypk (Janusz Szczypka)
-Status: Closed +Status: Open -Package Version: 1.2.8 +Package Version: 1.2.9
Line 426 still is not fixed in 1.2.9: Now it is: if (isset($this->_calls[$item][$name])) { It should be: if (isset($this->_calls[$item["name"]][$name])) {