Merge pull request #287 from symfony-cmf/analysis-6VQE1a

Apply fixes from StyleCI
symfony-cmf · Feb 19, 2024 · bf5d807 · bf5d807
2 parents 921c713 + b50b8d0
commit bf5d807
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 21 deletions.
diff --git a/src/Doctrine/Phpcr/NonTranslatableMetadataListener.php b/src/Doctrine/Phpcr/NonTranslatableMetadataListener.php
@@ -23,9 +23,6 @@
  */
 class NonTranslatableMetadataListener implements EventSubscriber
 {
-    /**
-     * @return array
-     */
     public function getSubscribedEvents(): array
     {
         return [

diff --git a/src/Doctrine/Phpcr/TranslatableMetadataListener.php b/src/Doctrine/Phpcr/TranslatableMetadataListener.php
@@ -28,9 +28,6 @@ public function __construct(
     ) {
     }
 
-    /**
-     * @return array
-     */
     public function getSubscribedEvents(): array
     {
         return [

diff --git a/src/Templating/Helper/Cmf.php b/src/Templating/Helper/Cmf.php
@@ -33,9 +33,6 @@ class Cmf
     private ManagerRegistry $doctrineRegistry;
     private ?string $doctrineManagerName;
 
-    /**
-     * @var DocumentManager
-     */
     protected DocumentManager $dm;
 
     public function __construct(
@@ -117,8 +114,6 @@ public function getPath($document): bool|string
 
     /**
      * Finds a document by path.
-     *
-     * @return object|null
      */
     public function find($path): ?object
     {
@@ -131,8 +126,6 @@ public function find($path): ?object
      * @param string|object $pathOrDocument the identifier of the class (path or document object)
      * @param string        $locale         the language to try to load
      * @param bool          $fallback       set to true if the language fallback mechanism should be used
-     *
-     * @return object|null
      */
     public function findTranslation($pathOrDocument, string $locale, bool $fallback = true): ?object
     {
@@ -188,8 +181,6 @@ private function getDocument($document, ?bool $ignoreRole = false, ?string $clas
      * @param string|bool $offset     string node name to which to skip to or false
      * @param bool|null   $ignoreRole if the role should be ignored or null if publish workflow should be ignored
      * @param string|null $class      class name to filter on
-     *
-     * @return array
      */
     public function findMany(array $paths = [], $limit = false, $offset = false, ?bool $ignoreRole = false, ?string $class = null): array
     {
@@ -295,8 +286,6 @@ public function getChild($parent, string $name): object|bool|null
      *                                  null if publish workflow should be
      *                                  ignored (defaults to false)
      * @param string|null   $class      class name to filter on (optional)
-     *
-     * @return array
      */
     public function getChildren($parent, $limit = false, $offset = false, $filter = null, $ignoreRole = false, $class = null): array
     {

diff --git a/tests/Unit/PublishWorkflow/Voter/PublishTimePeriodVoterTest.php b/tests/Unit/PublishWorkflow/Voter/PublishTimePeriodVoterTest.php
@@ -18,7 +18,6 @@
 use Symfony\Cmf\Bundle\CoreBundle\PublishWorkflow\Voter\PublishTimePeriodVoter;
 use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
 use Symfony\Component\Security\Core\Authorization\Voter\VoterInterface;
-use function is_subclass_of;
 
 class PublishTimePeriodVoterTest extends TestCase
 {

diff --git a/tests/Unit/PublishWorkflow/Voter/PublishableVoterTest.php b/tests/Unit/PublishWorkflow/Voter/PublishableVoterTest.php
@@ -17,10 +17,7 @@
 use Symfony\Cmf\Bundle\CoreBundle\PublishWorkflow\PublishWorkflowChecker;
 use Symfony\Cmf\Bundle\CoreBundle\PublishWorkflow\Voter\PublishableVoter;
 use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
-use Symfony\Component\Security\Core\Authorization\Voter\CacheableVoterInterface;
-use Symfony\Component\Security\Core\Authorization\Voter\Voter;
 use Symfony\Component\Security\Core\Authorization\Voter\VoterInterface;
-use function is_subclass_of;
 
 class PublishableVoterTest extends TestCase
 {