move custom Exceptions in exception folder close #436

vincent-peugnet · Oct 13, 2024 · 0c8bbb7 · 0c8bbb7
1 parent a171bec
commit 0c8bbb7
Show file tree

Hide file tree

Showing 21 changed files with 56 additions and 26 deletions.
diff --git a/app/class/Config.php b/app/class/Config.php
@@ -2,6 +2,8 @@
 
 namespace Wcms;
 
+use Wcms\Exception\Filesystemexception;
+
 abstract class Config
 {
     protected static $pagetable = 'mystore';

diff --git a/app/class/Controlleradmin.php b/app/class/Controlleradmin.php
@@ -3,6 +3,7 @@
 namespace Wcms;
 
 use RuntimeException;
+use Wcms\Exception\Filesystemexception;
 
 class Controlleradmin extends Controller
 {

diff --git a/app/class/Controllerapimedia.php b/app/class/Controllerapimedia.php
@@ -5,6 +5,7 @@
 use AltoRouter;
 use Error;
 use RuntimeException;
+use Wcms\Exception\Filesystemexception;
 
 class Controllerapimedia extends Controllerapi
 {

diff --git a/app/class/Controllermedia.php b/app/class/Controllermedia.php
@@ -4,6 +4,7 @@
 
 use AltoRouter;
 use RuntimeException;
+use Wcms\Exception\Filesystemexception\Folderexception;
 use Wcms\Exception\Forbiddenexception;
 
 class Controllermedia extends Controller

diff --git a/app/class/Controllerpage.php b/app/class/Controllerpage.php
@@ -4,6 +4,7 @@
 
 use DateTimeImmutable;
 use RuntimeException;
+use Wcms\Exception\Filesystemexception;
 
 class Controllerpage extends Controller
 {

diff --git a/app/class/Filesystemexception.php → app/class/Exception/Filesystemexception.php b/app/class/Filesystemexception.php → app/class/Exception/Filesystemexception.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace Wcms;
+namespace Wcms\Exception;
 
 use RuntimeException;
 

diff --git a/app/class/Chmodexception.php → ...on/Filesystemexception/Chmodexception.php b/app/class/Chmodexception.php → ...on/Filesystemexception/Chmodexception.php
@@ -1,6 +1,8 @@
 <?php
 
-namespace Wcms;
+namespace Wcms\Exception\Filesystemexception;
+
+use Wcms\Exception\Filesystemexception;
 
 class Chmodexception extends Filesystemexception
 {

diff --git a/app/class/Exception/Filesystemexception/Fileexception.php b/app/class/Exception/Filesystemexception/Fileexception.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Wcms\Exception\Filesystemexception;
+
+use Wcms\Exception\Filesystemexception;
+
+class Fileexception extends Filesystemexception
+{
+}
diff --git a/app/class/Exception/Filesystemexception/Folderexception.php b/app/class/Exception/Filesystemexception/Folderexception.php
@@ -0,0 +1,9 @@
+<?php
+
+namespace Wcms\Exception\Filesystemexception;
+
+use Wcms\Exception\Filesystemexception;
+
+class Folderexception extends Filesystemexception
+{
+}
diff --git a/app/class/Notfoundexception.php → ...Filesystemexception/Notfoundexception.php b/app/class/Notfoundexception.php → ...Filesystemexception/Notfoundexception.php
@@ -1,8 +1,9 @@
 <?php
 
-namespace Wcms;
+namespace Wcms\Exception\Filesystemexception;
 
 use Throwable;
+use Wcms\Exception\Filesystemexception;
 
 /**
  * File not found

diff --git a/app/class/Unlinkexception.php → ...n/Filesystemexception/Unlinkexception.php b/app/class/Unlinkexception.php → ...n/Filesystemexception/Unlinkexception.php
@@ -1,8 +1,9 @@
 <?php
 
-namespace Wcms;
+namespace Wcms\Exception\Filesystemexception;
 
 use Throwable;
+use Wcms\Exception\Filesystemexception;
 
 /**
  * Error while trying to unlink file on the filesystem

diff --git a/app/class/Fileexception.php b/app/class/Fileexception.php
diff --git a/app/class/Folderexception.php b/app/class/Folderexception.php
diff --git a/app/class/Fs.php b/app/class/Fs.php
@@ -4,7 +4,12 @@
 
 use DomainException;
 use LogicException;
-use RuntimeException;
+use Wcms\Exception\Filesystemexception;
+use Wcms\Exception\Filesystemexception\Chmodexception;
+use Wcms\Exception\Filesystemexception\Fileexception;
+use Wcms\Exception\Filesystemexception\Folderexception;
+use Wcms\Exception\Filesystemexception\Notfoundexception;
+use Wcms\Exception\Filesystemexception\Unlinkexception;
 
 /**
  * File system related tools. No Wcms specific param should be set here.

diff --git a/app/class/Media.php b/app/class/Media.php
@@ -5,6 +5,7 @@
 use DateTime;
 use DomainException;
 use RuntimeException;
+use Wcms\Exception\Filesystemexception\Fileexception;
 
 class Media extends Item
 {

diff --git a/app/class/Modelmedia.php b/app/class/Modelmedia.php
@@ -8,7 +8,9 @@
 use RuntimeException;
 use Imagick;
 use ImagickException;
-use RangeException;
+use Wcms\Exception\Filesystemexception;
+use Wcms\Exception\Filesystemexception\Fileexception;
+use Wcms\Exception\Filesystemexception\Folderexception;
 use Wcms\Exception\Forbiddenexception;
 
 class Modelmedia extends Model

diff --git a/app/class/Modelpage.php b/app/class/Modelpage.php
@@ -10,6 +10,8 @@
 use InvalidArgumentException;
 use RangeException;
 use RuntimeException;
+use Wcms\Exception\Filesystemexception;
+use Wcms\Exception\Filesystemexception\Notfoundexception;
 
 class Modelpage extends Modeldb
 {

diff --git a/app/class/Servicefont.php b/app/class/Servicefont.php
@@ -2,7 +2,8 @@
 
 namespace Wcms;
 
-use RuntimeException ;
+use RuntimeException;
+use Wcms\Exception\Filesystemexception;
 
 class Servicefont
 {

diff --git a/app/class/Servicerss.php b/app/class/Servicerss.php
@@ -12,7 +12,10 @@
 use LogicException;
 use RuntimeException;
 use Wcms\Exception\Database\Notfoundexception;
-use Wcms\Notfoundexception as WcmsNotfoundexception;
+use Wcms\Exception\Filesystemexception;
+use Wcms\Exception\Filesystemexception\Folderexception;
+use Wcms\Exception\Filesystemexception\Notfoundexception as FilesystemexceptionNotfoundexception;
+use Wcms\Exception\Filesystemexception\Unlinkexception;
 
 class Servicerss
 {
@@ -233,7 +236,7 @@ public static function removeatom(string $id): void
     {
         try {
             Fs::deletefile(self::atomfile($id));
-        } catch (WcmsNotfoundexception $e) {
+        } catch (FilesystemexceptionNotfoundexception $e) {
             // do nothing, this means file is already deleted
         } catch (Unlinkexception $e) {
             throw new RuntimeException("RSS atom file deletion error", 0, $e);

diff --git a/app/class/Servicetags.php b/app/class/Servicetags.php
@@ -2,7 +2,9 @@
 
 namespace Wcms;
 
-use Exception;
+use Wcms\Exception\Filesystemexception;
+use Wcms\Exception\Filesystemexception\Fileexception;
+use Wcms\Exception\Filesystemexception\Notfoundexception;
 
 class Servicetags
 {
@@ -16,7 +18,7 @@ public function taglist(): array
         try {
             $tagfile = Fs::readfile(Model::TAGS_FILE);
         } catch (Notfoundexception $e) {
-            // This mean the tag fille does not exist
+            // This mean the tag file does not exist
             return [];
         }
         $tagdata = json_decode($tagfile, true);

diff --git a/app/fn/fn.php b/app/fn/fn.php
@@ -1,6 +1,6 @@
 <?php
 
-use Wcms\Folderexception;
+use Wcms\Exception\Filesystemexception\Folderexception;
 
 function readablesize($bytes, $base = 2 ** 10)
 {