Merge pull request #10292 from annando/fix-mail

Fix inserting mails
This commit is contained in:
Hypolite Petovan 2021-05-23 07:39:00 -04:00 committed by GitHub
commit 4d214bd9cb
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -36,15 +36,14 @@ use Friendica\Worker\Delivery;
class Mail class Mail
{ {
/** /**
* Insert received private message * Insert private message
* *
* @param array $msg * @param array $msg
* @param bool $notifiction
* @return int|boolean Message ID or false on error * @return int|boolean Message ID or false on error
*/ */
public static function insert($msg) public static function insert($msg, $notifiction = true)
{ {
$user = User::getById($msg['uid']);
if (!isset($msg['reply'])) { if (!isset($msg['reply'])) {
$msg['reply'] = DBA::exists('mail', ['parent-uri' => $msg['parent-uri']]); $msg['reply'] = DBA::exists('mail', ['parent-uri' => $msg['parent-uri']]);
} }
@ -63,6 +62,10 @@ class Mail
$msg['created'] = (!empty($msg['created']) ? DateTimeFormat::utc($msg['created']) : DateTimeFormat::utcNow()); $msg['created'] = (!empty($msg['created']) ? DateTimeFormat::utc($msg['created']) : DateTimeFormat::utcNow());
$msg['author-id'] = Contact::getIdForURL($msg['from-url'], 0, false);
$msg['uri-id'] = ItemURI::insert(['uri' => $msg['uri'], 'guid' => $msg['guid']]);
$msg['parent-uri-id'] = ItemURI::getIdByURI($msg['parent-uri']);
DBA::lock('mail'); DBA::lock('mail');
if (DBA::exists('mail', ['uri' => $msg['uri'], 'uid' => $msg['uid']])) { if (DBA::exists('mail', ['uri' => $msg['uri'], 'uid' => $msg['uid']])) {
@ -71,12 +74,8 @@ class Mail
return false; return false;
} }
$msg['author-id'] = Contact::getIdForURL($msg['from-url'], 0, false);
$msg['uri-id'] = ItemURI::insert(['uri' => $msg['uri'], 'guid' => $msg['guid']]);
$msg['parent-uri-id'] = ItemURI::getIdByURI($msg['parent-uri']);
if ($msg['reply']) { if ($msg['reply']) {
$reply = DBA::selectFirst('mail', ['uri', 'uri-id'], ['parent-uri' => $mail['parent-uri'], 'reply' => false]); $reply = DBA::selectFirst('mail', ['uri', 'uri-id'], ['parent-uri' => $msg['parent-uri'], 'reply' => false]);
$msg['thr-parent'] = $reply['uri']; $msg['thr-parent'] = $reply['uri'];
$msg['thr-parent-id'] = $reply['uri-id']; $msg['thr-parent-id'] = $reply['uri-id'];
@ -95,6 +94,8 @@ class Mail
DBA::update('conv', ['updated' => DateTimeFormat::utcNow()], ['id' => $msg['convid']]); DBA::update('conv', ['updated' => DateTimeFormat::utcNow()], ['id' => $msg['convid']]);
} }
if ($notifiction) {
$user = User::getById($msg['uid']);
// send notifications. // send notifications.
$notif_params = [ $notif_params = [
'type' => Notification\Type::MAIL, 'type' => Notification\Type::MAIL,
@ -108,6 +109,7 @@ class Mail
notification($notif_params); notification($notif_params);
Logger::info('Mail is processed, notification was sent.', ['id' => $msg['id'], 'uri' => $msg['uri']]); Logger::info('Mail is processed, notification was sent.', ['id' => $msg['id'], 'uri' => $msg['uri']]);
}
return $msg['id']; return $msg['id'];
} }
@ -195,9 +197,7 @@ class Mail
$replyto = $convuri; $replyto = $convuri;
} }
$post_id = null; $post_id = self::insert(
$success = DBA::insert(
'mail',
[ [
'uid' => local_user(), 'uid' => local_user(),
'guid' => $guid, 'guid' => $guid,
@ -214,13 +214,9 @@ class Mail
'uri' => $uri, 'uri' => $uri,
'parent-uri' => $replyto, 'parent-uri' => $replyto,
'created' => DateTimeFormat::utcNow() 'created' => DateTimeFormat::utcNow()
] ], false
); );
if ($success) {
$post_id = DBA::lastInsertId();
}
/** /**
* *
* When a photo was uploaded into the message using the (profile wall) ajax * When a photo was uploaded into the message using the (profile wall) ajax
@ -301,8 +297,7 @@ class Mail
return -4; return -4;
} }
DBA::insert( self::insert(
'mail',
[ [
'uid' => $recipient['uid'], 'uid' => $recipient['uid'],
'guid' => $guid, 'guid' => $guid,
@ -320,7 +315,7 @@ class Mail
'parent-uri' => $me['url'], 'parent-uri' => $me['url'],
'created' => DateTimeFormat::utcNow(), 'created' => DateTimeFormat::utcNow(),
'unknown' => 1 'unknown' => 1
] ], false
); );
return 0; return 0;