Page MenuHomePhabricator

wl_notificationtimestamp NULL check but wl_notificationtimestamp is '0'
Closed, ResolvedPublic

Description

Author: suse-tux

Description:
In the function clearAllNotifications in includes/User.php
'wl_notificationtimestamp' => 0 must be changed to
'wl_notificationtimestamp' => NULL
because otherwise the mysql-check in the notifyOnPageChange() function in
includes/UserMailer.php does not work (checks for NULL and not for '0').
I think it's a bug so i attached a patch for it.


Version: 1.8.x
Severity: normal

Details

Reference
bz6023

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:17 PM
bzimport set Reference to bz6023.
bzimport added a subscriber: Unknown Object (MLST).

suse-tux wrote:

patch which fixes the "problem"

Attached:

  • Bug 7442 has been marked as a duplicate of this bug. ***

jimmy.collins wrote:

Fixed in r16696.