Don't send notifications to disabled accounts

[tcitworld]

Steps to reproduce

1. Produce an activity, for instance an announcement, that triggers emails to all users

Expected behaviour

All enabled users should receive the email, not the disabled ones.

Actual behaviour

All users receive the email, even the disabled ones.

Server configuration

Operating system: Debian

Web server: Nginx

Database: MySQL

PHP version: PHP 5.6

Nextcloud version: (see Nextcloud admin page) 12.0.0

List of activated apps:

Enabled:
  - activity: 2.5.2
  - admin_audit: 1.2.0
  - admin_notifications: 1.0.0
  - announcementcenter: 3.1.0
  - bookmarks: 0.10.1
  - bruteforcesettings: 1.0.2
  - calendar: 1.5.3
  - circles: 0.11.0
  - comments: 1.2.0
  - contacts: 1.5.3
  - dav: 1.3.0
  - federatedfilesharing: 1.2.0
  - federation: 1.2.0
  - files: 1.7.2
  - files_pdfviewer: 1.1.1
  - files_sharing: 1.4.0
  - files_texteditor: 2.4.1
  - files_trashbin: 1.2.0
  - files_versions: 1.5.0
  - files_videoplayer: 1.1.0
  - firstrunwizard: 2.1
  - gallery: 17.0.0
  - impersonate: 1.0.1
  - logreader: 2.0.0
  - lookup_server_connector: 1.0.0
  - nextcloud_announcements: 1.1
  - notes: 2.3.1
  - notifications: 2.0.0
  - oauth2: 1.0.5
  - password_policy: 1.2.2
  - provisioning_api: 1.2.0
  - quota_warning: 1.0.1
  - serverinfo: 1.2.0
  - sharebymail: 1.2.0
  - socialsharing_diaspora: 1.0.1
  - socialsharing_email: 1.0.1
  - socialsharing_facebook: 1.0.1
  - socialsharing_googleplus: 1.0.1
  - socialsharing_twitter: 1.0.1
  - survey_client: 1.0.0
  - systemtags: 1.2.0
  - tasks: 0.9.5
  - theming: 1.3.0
  - twofactor_backupcodes: 1.1.1
  - twofactor_totp: 1.3.0
  - twofactor_u2f: 1.3.2
  - updatenotification: 1.2.0
  - workflowengine: 1.2.0
Disabled:
  - encryption
  - files_external
  - jsloader
  - news
  - user_external
  - user_ldap

Nextcloud configuration:

    "system": {
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "default_language": "fr",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trusted_domains": [
            "mydomain.tld",
            "www.mydomain.tld"
        ],
        "datadirectory": "\/var\/www\/mydata",
        "overwrite.cli.url": "myserver",
        "dbtype": "mysql",
        "version": "12.0.0.29",
        "dbname": "database name",
        "dbhost": "remote mysql server",
        "dbtableprefix": "oc_",
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "logtimezone": "UTC",
        "installed": true,
        "instanceid": "ocbkwoy9jvzq",
        "appstore.experimental.enabled": true,
        "memcache.local": "\\OC\\Memcache\\Redis",
        "redis": {
            "host": "localhost",
            "port": 6379,
            "timeout": 0
        },
        "mail_smtpmode": "smtp",
        "mail_from_address": "noreply-mydomain",
        "mail_domain": "mydomain.tld",
        "mail_smtphost": "localhost",
        "mail_smtpport": "25",
        "theme": "mycustomtheme",
        "maintenance": false,
        "loglevel": 2,
        "trashbin_retention_obligation": "auto",
        "dbdriveroptions": {
            "1012": "\/etc\/\/mysql\/ssl\/ca-cert.pem",
            "1011": "\/etc\/\/mysql\/ssl\/remotemysql-cert.pem",
            "1010": "\/etc\/mysql\/ssl\/remotemysql-key.pem"
        },
        "mail_smtpauthtype": "PLAIN"
    }
}

Logs

Nextcloud log (data/nextcloud.log)

Nothing relevant.

Browser log

Nothing relevant.