From 062c5f5895d917c08dabb59729b77f9a593bb97a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Melaine=20G=C3=A9rard?= <pro@melaine-gerard.fr>
Date: Wed, 22 Jan 2025 20:17:44 +0100
Subject: [PATCH] :sparkles: Lint fix

---
 phpstan.dist.neon                            |  3 ++
 src/Controller/FilesController.php           |  6 +--
 src/Entity/ParentDirectory.php               |  2 +
 src/Entity/User.php                          |  3 --
 src/Repository/ParentDirectoryRepository.php | 48 ++++++++++----------
 5 files changed, 33 insertions(+), 29 deletions(-)

diff --git a/phpstan.dist.neon b/phpstan.dist.neon
index 23ff84e..2b5f313 100644
--- a/phpstan.dist.neon
+++ b/phpstan.dist.neon
@@ -11,6 +11,9 @@ parameters:
         containerXmlPath: var/cache/dev/App_KernelDevDebugContainer.xml
     strictRules:
         noVariableVariables: false
+    ignoreErrors:
+        -
+            identifier: property.unusedType
 
 
     reportUnmatchedIgnoredErrors: false
diff --git a/src/Controller/FilesController.php b/src/Controller/FilesController.php
index d1ec375..2a0a083 100755
--- a/src/Controller/FilesController.php
+++ b/src/Controller/FilesController.php
@@ -68,7 +68,7 @@ class FilesController extends AbstractController
             $filename = basename((string) $file['path']);
             if (!str_starts_with($filename, '.')) {
                 // On vérifie si l'utilisateur a le droit d'accéder au fichier (vérifier que owner_role du parentDirectory correspondant est bien le folderRole de l'utilisateur)
-                $pathFile = explode('/', $file['path']);
+                $pathFile = explode('/', (string) $file['path']);
                 if ('' !== $path) {
                     $parentDir = $this->parentDirectoryRepository->findOneBy(['name' => $pathFile[0]]);
 
@@ -313,8 +313,8 @@ class FilesController extends AbstractController
 
             $name = $data['name'];
 
-            if (explode('/', $name) > 1) {
-                $name = explode('/', $name)[0];
+            if (count(explode('/', (string) $name)) > 1) {
+                $name = explode('/', (string) $name)[0];
             }
 
             if ($defaultAdapter->directoryExists($basePath . '/' . $name)) {
diff --git a/src/Entity/ParentDirectory.php b/src/Entity/ParentDirectory.php
index 174b7ec..379bce2 100644
--- a/src/Entity/ParentDirectory.php
+++ b/src/Entity/ParentDirectory.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace App\Entity;
 
 use App\Enum\RoleEnum;
diff --git a/src/Entity/User.php b/src/Entity/User.php
index 154a06c..087726f 100755
--- a/src/Entity/User.php
+++ b/src/Entity/User.php
@@ -41,9 +41,6 @@ class User implements UserInterface, PasswordAuthenticatedUserInterface
     #[ORM\Column(enumType: RoleEnum::class)]
     private ?RoleEnum $folder_role = null;
 
-    /**
-     * @return int|null
-     */
     public function getId(): ?int
     {
         return $this->id;
diff --git a/src/Repository/ParentDirectoryRepository.php b/src/Repository/ParentDirectoryRepository.php
index 89d56ae..0628959 100644
--- a/src/Repository/ParentDirectoryRepository.php
+++ b/src/Repository/ParentDirectoryRepository.php
@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace App\Repository;
 
 use App\Entity\ParentDirectory;
@@ -16,28 +18,28 @@ class ParentDirectoryRepository extends ServiceEntityRepository
         parent::__construct($registry, ParentDirectory::class);
     }
 
-//    /**
-//     * @return ParentDirectory[] Returns an array of ParentDirectory objects
-//     */
-//    public function findByExampleField($value): array
-//    {
-//        return $this->createQueryBuilder('p')
-//            ->andWhere('p.exampleField = :val')
-//            ->setParameter('val', $value)
-//            ->orderBy('p.id', 'ASC')
-//            ->setMaxResults(10)
-//            ->getQuery()
-//            ->getResult()
-//        ;
-//    }
+    //    /**
+    //     * @return ParentDirectory[] Returns an array of ParentDirectory objects
+    //     */
+    //    public function findByExampleField($value): array
+    //    {
+    //        return $this->createQueryBuilder('p')
+    //            ->andWhere('p.exampleField = :val')
+    //            ->setParameter('val', $value)
+    //            ->orderBy('p.id', 'ASC')
+    //            ->setMaxResults(10)
+    //            ->getQuery()
+    //            ->getResult()
+    //        ;
+    //    }
 
-//    public function findOneBySomeField($value): ?ParentDirectory
-//    {
-//        return $this->createQueryBuilder('p')
-//            ->andWhere('p.exampleField = :val')
-//            ->setParameter('val', $value)
-//            ->getQuery()
-//            ->getOneOrNullResult()
-//        ;
-//    }
+    //    public function findOneBySomeField($value): ?ParentDirectory
+    //    {
+    //        return $this->createQueryBuilder('p')
+    //            ->andWhere('p.exampleField = :val')
+    //            ->setParameter('val', $value)
+    //            ->getQuery()
+    //            ->getOneOrNullResult()
+    //        ;
+    //    }
 }