Merge pull request #133 from chrootchad/bug-132

stevebauman · web-flow · commit 36d6a6345d32 · 2025-11-10T08:37:40.000-05:00
Fix: issue 132 - pathinfo(): Passing null to parameter #1 () of type …
diff --git a/src/Attachment.php b/src/Attachment.php
@@ -81,6 +81,10 @@ public function save(string $path): false|int
      */
     public function extension(): ?string
     {
+        if (! $this->filename) {
+            return null;
+        }
+
         if ($ext = pathinfo($this->filename, PATHINFO_EXTENSION)) {
             return $ext;
         }

Original file line number	Diff line number	Diff line change
`@@ -81,6 +81,10 @@ public function save(string $path): false\|int`
`81`	`81`	`*/`
`82`	`82`	`public function extension(): ?string`
`83`	`83`	`{`
	`84`	`+ if (! $this->filename) {`
	`85`	`+ return null;`
	`86`	`+ }`
	`87`	`+`
`84`	`88`	`if ($ext = pathinfo($this->filename, PATHINFO_EXTENSION)) {`
`85`	`89`	`return $ext;`
`86`	`90`	`}`