Merge pull request #15859 from Godmartinz/remove_name_null_warning

snipe · web-flow · commit 3a683d84a727 · 2024-11-21T14:38:03.000Z
Adds a ternary for blank emails for mail notifications
diff --git a/app/Listeners/CheckoutableListener.php b/app/Listeners/CheckoutableListener.php
@@ -307,7 +307,7 @@ private function getNotifiables($event){
             return $event->checkedOutTo->manager?->email ?? '';
         }
         else{
-            return $event->checkedOutTo->email;
+            return $event->checkedOutTo?->email ?? '';
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -307,7 +307,7 @@ private function getNotifiables($event){`
`307`	`307`	`return $event->checkedOutTo->manager?->email ?? '';`
`308`	`308`	`}`
`309`	`309`	`else{`
`310`		`- return $event->checkedOutTo->email;`
	`310`	`+ return $event->checkedOutTo?->email ?? '';`
`311`	`311`	`}`
`312`	`312`	`}`
`313`	`313`