When a content type has subscriptions turned off, the check in notifications_content_type_enabled() incorrectly returns true. This is caused by the fact that the in_array call isn't using strict checking (despite the fact that in the function's comments it says "PHP Note: We need to use strict checking for in_array(), http://es.php.net/manual/en/function.in-array.php#91911")

Pretty simple patch but it fixes another important part of notifications.

Comments

webflo’s picture

Assigned: Unassigned » webflo
Status: Active » Needs review
webflo’s picture

Status: Needs review » Fixed

Commit f30f473 on 7.x-1.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.