Clean up constructor methods and improve image size retrieval error handling

Author: murdercode

src/LaravelShortcodePlus.php

@@ -22,7 +22,9 @@
 
 final class LaravelShortcodePlus
 {
-    public function __construct(protected string $content = '') {}
+    public function __construct(protected string $content = '')
+    {
+    }
 
     public static function source(string $source): LaravelShortcodePlus
     {

src/LaravelShortcodePlusServiceProvider.php

@@ -43,7 +43,9 @@ public function configurePackage(Package $package): void
             ->hasMigration('create_laravel-shortcode-plus_table');
     }
 
-    public function packageRegistered() {}
+    public function packageRegistered()
+    {
+    }
 
     public function packageBooted(): void
     {

src/Shortcodes/PhotoShortcode.php

@@ -95,7 +95,6 @@ public function register($shortcode): string
      */
     public static function getImageHeight(string $path, int $width = 0): float|int
     {
-
         $localPath = storage_path('app/public/'.$path);
 
         // Check if file exists
@@ -106,6 +105,12 @@ public static function getImageHeight(string $path, int $width = 0): float|int
         // Get image sizes
         $sizes = [];
         $imageSizes = @getimagesize($localPath);
+
+        // Check if getimagesize was successful
+        if ($imageSizes === false) {
+            return 0;
+        }
+
         $sizes['width'] = $imageSizes[0];
         $sizes['height'] = $imageSizes[1];