Browse Source

Simplifications of SecurityAdvisoryRepo

Jordi Boggiano 5 years ago
parent
commit
8f548b333f
1 changed files with 5 additions and 17 deletions
  1. 5 17
      src/Packagist/WebBundle/Entity/SecurityAdvisoryRepository.php

+ 5 - 17
src/Packagist/WebBundle/Entity/SecurityAdvisoryRepository.php

@@ -20,16 +20,8 @@ class SecurityAdvisoryRepository extends ServiceEntityRepository
             WHERE s.packageName = :name
             ORDER BY s.id DESC';
 
-        $stmt = $this->getEntityManager()->getConnection()
-            ->executeQuery(
-                $sql,
-                ['name' => $name],
-                []
-            );
-        $result = $stmt->fetchAll();
-        $stmt->closeCursor();
-
-        return $result;
+        return $this->getEntityManager()->getConnection()
+            ->fetchAll($sql, ['name' => $name]);
     }
 
     public function searchSecurityAdvisories(array $packageNames, int $updatedSince): array
@@ -40,18 +32,14 @@ class SecurityAdvisoryRepository extends ServiceEntityRepository
             (count($packageNames) > 0 ? ' AND s.packageName IN (:packageNames)' : '')
             .' ORDER BY s.id DESC';
 
-        $stmt = $this->getEntityManager()->getConnection()
-            ->executeQuery(
+        return $this->getEntityManager()->getConnection()
+            ->fetchAll(
                 $sql,
                 [
                     'packageNames' => $packageNames,
-                    'updatedSince' => (new \DateTime('@' . $updatedSince))->format('Y-m-d H:i:s'),
+                    'updatedSince' => date('Y-m-d H:i:s', $updatedSince),
                 ],
                 ['packageNames' => Connection::PARAM_STR_ARRAY]
             );
-        $result = $stmt->fetchAll();
-        $stmt->closeCursor();
-
-        return $result;
     }
 }