Skip to content

Commit

Permalink
Merge pull request #43 from alpianon/patch-1
Browse files Browse the repository at this point in the history 
fix mailbox filter
  • Loading branch information
@stremlau
stremlau committed Jan 23, 2020
2 parents d50a1bc + 8207eda commit c87b0be
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion html5_notifier.php
View file
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ function show_notification($args)
}
$subject = ((!empty($mbox)) ? rcube_charset::convert($mbox, 'UTF7-IMAP') . ': ' : '') . $msg->get('subject');

if(strtolower($_SESSION['username']) == strtolower($RCMAIL->user->data['username']) && !in_array($mbox, $excluded_directories))
if(strtolower($_SESSION['username']) == strtolower($RCMAIL->user->data['username']) && !in_array($args['mailbox'], $excluded_directories))
{
$RCMAIL->output->command("plugin.showNotification", array(
'duration' => $RCMAIL->config->get('html5_notifier_duration'),
Expand Down

1 comment on commit c87b0be

@Takika
Copy link
Contributor

@Takika Takika commented on c87b0be Jan 24, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's not a fix. You set $mbox in the previous switch() statement and after it you use the previously changed $mbox variable as the in_array parameter.
And pls. fix the indentation too.

Please sign in to comment.