[Propel1] Fixed data collector
This commit is contained in:
parent
e712ab5de5
commit
eb759c59a8
@ -56,8 +56,6 @@ class PropelDataCollector extends DataCollector
|
||||
'queries' => $this->buildQueries(),
|
||||
'querycount' => $this->countQueries(),
|
||||
);
|
||||
|
||||
$this->data['queriesduration'] = $this->countQueriesDuration();
|
||||
}
|
||||
|
||||
/**
|
||||
@ -90,9 +88,19 @@ class PropelDataCollector extends DataCollector
|
||||
return $this->data['querycount'];
|
||||
}
|
||||
|
||||
public function getQueriesDuration()
|
||||
/**
|
||||
* Returns the total time of queries.
|
||||
*
|
||||
* @return float The total time of queries
|
||||
*/
|
||||
public function getTime()
|
||||
{
|
||||
return $this->data['queriesduration'];
|
||||
$time = 0;
|
||||
foreach ($this->data['queries'] as $query) {
|
||||
$time += (float) $query['time'];
|
||||
}
|
||||
|
||||
return $time;
|
||||
}
|
||||
|
||||
/**
|
||||
@ -132,21 +140,4 @@ class PropelDataCollector extends DataCollector
|
||||
{
|
||||
return count($this->logger->getQueries());
|
||||
}
|
||||
|
||||
/**
|
||||
* Count the queries duration.
|
||||
*
|
||||
* @return int The number of queries.
|
||||
*/
|
||||
private function countQueriesDuration()
|
||||
{
|
||||
$duration = 0;
|
||||
foreach ($this->data['queries'] as $queries) {
|
||||
foreach ($queries as $query) {
|
||||
$duration += $query['time'];
|
||||
}
|
||||
}
|
||||
|
||||
return $duration;
|
||||
}
|
||||
}
|
||||
|
@ -69,6 +69,7 @@ class PropelDataCollectorTest extends Propel1TestCase
|
||||
)
|
||||
), $c->getQueries());
|
||||
$this->assertEquals(2, $c->getQueryCount());
|
||||
$this->assertEquals(0.012, $c->getTime());
|
||||
}
|
||||
|
||||
private function createCollector($queries)
|
||||
|
Reference in New Issue
Block a user