void return

Author: nicklog

phpcs.xml

@@ -13,6 +13,5 @@
 
     <rule ref="./vendor/doctrine/coding-standard/lib/Doctrine/ruleset.xml">
         <exclude name="SlevomatCodingStandard.Classes.SuperfluousInterfaceNaming.SuperfluousSuffix"/>
-        <exclude name="SlevomatCodingStandard.Commenting.UselessInheritDocComment.UselessInheritDocComment"/>
     </rule>
 </ruleset>

src/Command/ClearSessionCommand.php

@@ -24,9 +24,6 @@ public function __construct(EntityManagerInterface $entityManager)
         parent::__construct();
     }
 
-    /**
-     * @inheritDoc
-     */
     protected function configure() : void
     {
         $this->setName('shapecode:doctrine-session:clear');

src/Entity/SessionInterface.php

@@ -6,9 +6,6 @@
 
 use DateTime;
 
-/**
- * Interface SessionInterface
- */
 interface SessionInterface
 {
     public function getSessionId() : string;

src/Repository/SessionRepository.php

@@ -20,29 +20,23 @@ public function findOneBySessionId(string $sessionId) : ?SessionInterface
         return $session;
     }
 
-    /**
-     * @return mixed
-     */
-    public function purge()
+    public function purge() : void
     {
         $qb = $this->createQueryBuilder('r');
         $qb->delete();
         $qb->where($qb->expr()->lt('r.endOfLife', ':endOfLife'));
         $qb->setParameter('endOfLife', new DateTime(), Types::DATETIME_MUTABLE);
 
-        return $qb->getQuery()->execute();
+        $qb->getQuery()->execute();
     }
 
-    /**
-     * @return mixed
-     */
-    public function destroy(string $sessionId)
+    public function destroy(string $sessionId) : void
     {
         $qb = $this->createQueryBuilder('r');
         $qb->delete();
         $qb->where($qb->expr()->eq('r.sessionId', ':session_id'));
         $qb->setParameter('session_id', $sessionId, Types::STRING);
 
-        return $qb->getQuery()->execute();
+        $qb->getQuery()->execute();
     }
 }

src/Repository/SessionRepositoryInterface.php

@@ -11,13 +11,7 @@ interface SessionRepositoryInterface extends ObjectRepository
 {
     public function findOneBySessionId(string $sessionId) : ?SessionInterface;
 
-    /**
-     * @return mixed
-     */
-    public function purge();
+    public function purge() : void;
 
-    /**
-     * @return mixed
-     */
-    public function destroy(string $sessionId);
+    public function destroy(string $sessionId) : void;
 }

src/Session/Handler/DoctrineHandler.php

@@ -23,9 +23,6 @@ public function __construct(EntityManagerInterface $entityManager)
         $this->entityManager = $entityManager;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function close() : bool
     {
         return true;