Check for null on webhook url

snipe · snipe · commit 7a312d075c08 · 2025-03-05T11:59:35.000Z
Signed-off-by: snipe &lt;snipe@snipe.net&gt;
diff --git a/app/Models/Setting.php b/app/Models/Setting.php
@@ -247,7 +247,7 @@ public static function fileSizeConvert($bytes): string
      *
      * @return string
      */
-    public function routeNotificationForSlack(): string
+    public function routeNotificationForSlack(): ?string
     {
         // At this point the endpoint is the same for everything.
         //  In the future this may want to be adapted for individual notifications.
@@ -259,7 +259,7 @@ public function routeNotificationForSlack(): string
      *
      * @return string
      */
-    public function routeNotificationForMail(): string
+    public function routeNotificationForMail(): ?string
     {
         // At this point the endpoint is the same for everything.
         //  In the future this may want to be adapted for individual notifications.

Original file line number	Diff line number	Diff line change
`@@ -247,7 +247,7 @@ public static function fileSizeConvert($bytes): string`
`247`	`247`	`*`
`248`	`248`	`* @return string`
`249`	`249`	`*/`
`250`		`- public function routeNotificationForSlack(): string`
	`250`	`+ public function routeNotificationForSlack(): ?string`
`251`	`251`	`{`
`252`	`252`	`// At this point the endpoint is the same for everything.`
`253`	`253`	`// In the future this may want to be adapted for individual notifications.`
`@@ -259,7 +259,7 @@ public function routeNotificationForSlack(): string`
`259`	`259`	`*`
`260`	`260`	`* @return string`
`261`	`261`	`*/`
`262`		`- public function routeNotificationForMail(): string`
	`262`	`+ public function routeNotificationForMail(): ?string`
`263`	`263`	`{`
`264`	`264`	`// At this point the endpoint is the same for everything.`
`265`	`265`	`// In the future this may want to be adapted for individual notifications.`