Merge pull request #10434 from annando/issue-10433

Issue 10433: Only notify when there had been changes
This commit is contained in:
Hypolite Petovan 2021-06-26 09:21:35 -04:00 committed by GitHub
commit 3fe6789c9c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 2 deletions

View File

@ -159,6 +159,10 @@ class Item
$fields['vid'] = Verb::getID($fields['verb']);
}
if (!empty($fields['edited'])) {
$previous = Post::selectFirst(['edited'], $condition);
}
$rows = Post::update($fields, $condition);
if (is_bool($rows)) {
return $rows;
@ -203,8 +207,8 @@ class Item
}
// We only need to notfiy others when it is an original entry from us.
// Only call the notifier when the item has some content relevant change.
if ($item['origin'] && in_array('edited', array_keys($fields))) {
// Only call the notifier when the item had been edited and records had been changed.
if ($item['origin'] && !empty($fields['edited']) && ($previous['edited'] != $fields['edited'])) {
$notify_items[] = $item['id'];
}
}