New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New extension hook entry_auto_read #5505
Conversation
For extensions to be notified of articles being automatically marked as read for various reasons
lib/Minz/ExtensionManager.php
Outdated
'entry_auto_read' => array( // function(FreshRSS_Entry $entry, string $why) -> none | ||
'list' => array(), | ||
'signature' => 'PassArguments', | ||
), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use [] for arrays ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another time, not to mess the rest of the PR here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Partial revert of FreshRSS#5505 We should not apply the auto mark as read rule based on existing titles for updated articles, because the match would most of the time be on that same article's title.
$entry->applyFilterActions($titlesAsRead); | ||
if ($readWhenSameTitleInFeed > 0) { | ||
$titlesAsRead[$entry->title()] = true; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was wrong. Reverted in #5561
Partial revert of #5505 We should not apply the auto mark as read rule based on existing titles for updated articles, because the match would most of the time be on that same article's title.
For extensions to be notified of articles being automatically marked as read for various reasons