Removed useless error handling

This commit is contained in:
Michael 2020-05-12 09:05:37 +00:00
parent 1d9394d409
commit cd82c527f5

View file

@ -58,14 +58,12 @@ class Delivery
if ($cmd == self::MAIL) { if ($cmd == self::MAIL) {
$target_item = DBA::selectFirst('mail', [], ['id' => $target_id]); $target_item = DBA::selectFirst('mail', [], ['id' => $target_id]);
if (!DBA::isResult($target_item)) { if (!DBA::isResult($target_item)) {
self::setFailedQueue($cmd, $target_item);
return; return;
} }
$uid = $target_item['uid']; $uid = $target_item['uid'];
} elseif ($cmd == self::SUGGESTION) { } elseif ($cmd == self::SUGGESTION) {
$target_item = DBA::selectFirst('fsuggest', [], ['id' => $target_id]); $target_item = DBA::selectFirst('fsuggest', [], ['id' => $target_id]);
if (!DBA::isResult($target_item)) { if (!DBA::isResult($target_item)) {
self::setFailedQueue($cmd, $target_item);
return; return;
} }
$uid = $target_item['uid']; $uid = $target_item['uid'];
@ -75,7 +73,6 @@ class Delivery
} else { } else {
$item = Model\Item::selectFirst(['parent'], ['id' => $target_id]); $item = Model\Item::selectFirst(['parent'], ['id' => $target_id]);
if (!DBA::isResult($item) || empty($item['parent'])) { if (!DBA::isResult($item) || empty($item['parent'])) {
self::setFailedQueue($cmd, $target_item);
return; return;
} }
$parent_id = intval($item['parent']); $parent_id = intval($item['parent']);
@ -97,7 +94,6 @@ class Delivery
if (empty($target_item)) { if (empty($target_item)) {
Logger::log('Item ' . $target_id . "wasn't found. Quitting here."); Logger::log('Item ' . $target_id . "wasn't found. Quitting here.");
self::setFailedQueue($cmd, $target_item);
return; return;
} }