Browse Source

Fix duplicate quoted message issue

Reverse logic in URL resolving system Embed first then picture
pull/978/head
Timothée Jaussoin 5 years ago
parent
commit
9b2ebe9b3d
  1. 3
      app/Message.php
  2. 20
      app/widgets/ChatActions/ChatActions.php

3
app/Message.php

@ -242,7 +242,8 @@ class Message extends Model
->orderBy('published', 'asc')
->first();
if ($parent && $parent->mid != $this->mid) {
if ($parent && $parent->mid != $this->mid
&& $parent->replaceid != $this->replaceid) {
$this->parentmid = $parent->mid;
}
}

20
app/widgets/ChatActions/ChatActions.php

@ -85,17 +85,19 @@ class ChatActions extends \Movim\Widget\Base
->first();
if ($message && $message->resolved == false) {
$picture = resolvePictureFileFromUrl(trim($message->body));
if ($picture != false) {
$message->file = (array)$picture;
try {
Url::resolve(trim($message->body));
$message->urlid = Url::$id;
$this->rpc('Chat.refreshMessage', $message->mid);
} else {
try {
Url::resolve(trim($message->body));
$message->urlid = Url::$id;
} catch (\Exception $e) {}
if (!$message->urlid) {
$picture = resolvePictureFileFromUrl(trim($message->body));
if ($picture != false) {
$message->file = (array)$picture;
$this->rpc('Chat.refreshMessage', $message->mid);
} catch (\Exception $e) {}
}
}
$message->resolved = true;

Loading…
Cancel
Save