Notify about a new mail only if it's UNSEEN (#1388965)

pull/62/head
Aleksander Machniak 11 years ago
parent b6fa7d54b8
commit 6e8f2a7448

@ -35,6 +35,9 @@ class newmail_notifier extends rcube_plugin
private $rc;
private $notified;
private $opt = array();
private $exceptions = array();
/**
* Plugin initialization
@ -49,13 +52,34 @@ class newmail_notifier extends rcube_plugin
$this->add_hook('preferences_save', array($this, 'prefs_save'));
}
else { // if ($this->rc->task == 'mail') {
$this->add_hook('new_messages', array($this, 'notify'));
// add script when not in ajax and not in frame
if ($this->rc->output->type == 'html' && empty($_REQUEST['_framed'])) {
$this->add_texts('localization/');
$this->rc->output->add_label('newmail_notifier.title', 'newmail_notifier.body');
$this->include_script('newmail_notifier.js');
}
if ($this->rc->action == 'refresh') {
// Load configuration
$this->load_config();
$this->opt['basic'] = $this->rc->config->get('newmail_notifier_basic');
$this->opt['sound'] = $this->rc->config->get('newmail_notifier_sound');
$this->opt['desktop'] = $this->rc->config->get('newmail_notifier_desktop');
if (!empty($this->opt)) {
// Get folders to skip checking for
$exceptions = array('drafts_mbox', 'sent_mbox', 'trash_mbox');
foreach ($exceptions as $folder) {
$folder = $this->rc->config->get($folder);
if (strlen($folder) && $folder != 'INBOX') {
$this->exceptions[] = $folder;
}
}
$this->add_hook('new_messages', array($this, 'notify'));
}
}
}
}
@ -132,43 +156,30 @@ class newmail_notifier extends rcube_plugin
*/
function notify($args)
{
// Already notified or non-automatic check
if ($this->notified || !empty($_GET['_refresh'])) {
// Already notified or unexpected input
if ($this->notified || empty($args['diff']['new'])) {
return $args;
}
// Get folders to skip checking for
if (empty($this->exceptions)) {
$this->delimiter = $this->rc->storage->get_hierarchy_delimiter();
$exceptions = array('drafts_mbox', 'sent_mbox', 'trash_mbox');
foreach ($exceptions as $folder) {
$folder = $this->rc->config->get($folder);
if (strlen($folder) && $folder != 'INBOX') {
$this->exceptions[] = $folder;
}
}
}
$mbox = $args['mailbox'];
$mbox = $args['mailbox'];
$storage = $this->rc->get_storage();
$delimiter = $storage->get_hierarchy_delimiter();
// Skip exception (sent/drafts) folders (and their subfolders)
foreach ($this->exceptions as $folder) {
if (strpos($mbox.$this->delimiter, $folder.$this->delimiter) === 0) {
if (strpos($mbox.$delimiter, $folder.$delimiter) === 0) {
return $args;
}
}
$this->notified = true;
// Load configuration
$this->load_config();
// Check if any of new messages is UNSEEN
$deleted = $this->rc->config->get('skip_deleted') ? 'UNDELETED ' : '';
$search = $deleted . 'UNSEEN UID ' . $args['diff']['new'];
$unseen = $storage->search_once($mbox, $search);
$basic = $this->rc->config->get('newmail_notifier_basic');
$sound = $this->rc->config->get('newmail_notifier_sound');
$desktop = $this->rc->config->get('newmail_notifier_desktop');
if ($unseen->count()) {
$this->notified = true;
if ($basic || $sound || $desktop) {
$this->rc->output->command('plugin.newmail_notifier',
array('basic' => $basic, 'sound' => $sound, 'desktop' => $desktop));
}

@ -19,10 +19,10 @@
<email>alec@alec.pl</email>
<active>yes</active>
</lead>
<date>2012-02-07</date>
<date>2013-03-16</date>
<version>
<release>0.4</release>
<api>0.3</api>
<release>0.5</release>
<api>0.5</api>
</version>
<stability>
<release>stable</release>

@ -1096,16 +1096,17 @@ class rcube_imap extends rcube_storage
/**
* Returns current status of folder
* Returns current status of a folder (compared to the last time use)
*
* We compare the maximum UID to determine the number of
* new messages because the RECENT flag is not reliable.
*
* @param string $folder Folder name
* @param array $diff Difference data
*
* @return int Folder status
* @return int Folder status
*/
public function folder_status($folder = null)
public function folder_status($folder = null, &$diff = array())
{
if (!strlen($folder)) {
$folder = $this->folder;
@ -1126,6 +1127,9 @@ class rcube_imap extends rcube_storage
// got new messages
if ($new['maxuid'] > $old['maxuid']) {
$result += 1;
// get new message UIDs range, that can be used for example
// to get the data of these messages
$diff['new'] = ($old['maxuid'] + 1 < $new['maxuid'] ? ($old['maxuid']+1).':' : '') . $new['maxuid'];
}
// some messages has been deleted
if ($new['cnt'] < $old['cnt']) {

@ -807,13 +807,14 @@ abstract class rcube_storage
/**
* Returns current status of a folder
* Returns current status of a folder (compared to the last time use)
*
* @param string $folder Folder name
* @param array $diff Difference data
*
* @return int Folder status
*/
abstract function folder_status($folder = null);
abstract function folder_status($folder = null, &$diff = array());
/**

@ -52,12 +52,12 @@ foreach ($a_mailboxes as $mbox_name) {
}
// Get mailbox status
$status = $RCMAIL->storage->folder_status($mbox_name);
$status = $RCMAIL->storage->folder_status($mbox_name, $diff);
if ($status & 1) {
// trigger plugin hook
$RCMAIL->plugins->exec_hook('new_messages',
array('mailbox' => $mbox_name, 'is_current' => $is_current));
array('mailbox' => $mbox_name, 'is_current' => $is_current, 'diff' => $diff));
}
rcmail_send_unread_count($mbox_name, true, null,

Loading…
Cancel
Save