mmucklo /
DtcQueueBundle
| Conditions | 1 |
| Paths | 1 |
| Total Lines | 15 |
| Code Lines | 10 |
| Lines | 0 |
| Ratio | 0 % |
| Tests | 10 |
| CRAP Score | 1 |
| Changes | 0 | ||
| 1 | <?php |
||
| 20 | 1 | protected function getOldLiveRuns() |
|
| 21 | { |
||
| 22 | /** @var EntityManager $objectManager */ |
||
| 23 | 1 | $objectManager = $this->getObjectManager(); |
|
| 24 | /** @var QueryBuilder $queryBuilder */ |
||
| 25 | 1 | $queryBuilder = $objectManager->createQueryBuilder(); |
|
| 26 | 1 | $queryBuilder->select(['r']) |
|
| 27 | 1 | ->from($this->getRunClass(), 'r'); |
|
| 28 | 1 | $time = time() - 86400; |
|
| 29 | 1 | $date = new \DateTime("@$time"); |
|
| 30 | 1 | $queryBuilder->where('r.lastHeartbeatAt < :date'); |
|
| 31 | 1 | $queryBuilder->setParameter(':date', $date); |
|
| 32 | |||
| 33 | 1 | return $queryBuilder->getQuery()->getResult(); |
|
| 34 | } |
||
| 35 | } |
||
| 36 |
This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass.
Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type.