Only send notification if the item is assigned to a person

Signed-off-by: snipe <snipe@snipe.net>
This commit is contained in:
snipe 2024-02-22 12:03:07 +00:00
parent 54552fc95c
commit 3351998efd

View file

@ -1156,7 +1156,10 @@ class ReportsController extends Controller
$logItem = $logItem_res[0]; $logItem = $logItem_res[0];
} }
if (!$assetItem->assignedTo->locale){ // Only send notification if assigned
if ($assetItem->assignedTo) {
if (!$assetItem->assignedTo->locale) {
Notification::locale(Setting::getSettings()->locale)->send( Notification::locale(Setting::getSettings()->locale)->send(
$assetItem->assignedTo, $assetItem->assignedTo,
new CheckoutAssetNotification($assetItem, $assetItem->assignedTo, $logItem->user, $acceptance, $logItem->note) new CheckoutAssetNotification($assetItem, $assetItem->assignedTo, $logItem->user, $acceptance, $logItem->note)
@ -1167,6 +1170,7 @@ class ReportsController extends Controller
new CheckoutAssetNotification($assetItem, $assetItem->assignedTo, $logItem->user, $acceptance, $logItem->note) new CheckoutAssetNotification($assetItem, $assetItem->assignedTo, $logItem->user, $acceptance, $logItem->note)
); );
} }
}
return redirect()->route('reports/unaccepted_assets')->with('success', trans('admin/reports/general.reminder_sent')); return redirect()->route('reports/unaccepted_assets')->with('success', trans('admin/reports/general.reminder_sent'));
} }