Fix regression from r23575 -- wl_notificationtimestamp didn't get updated anymore...
authorBrion Vibber <brion@users.mediawiki.org>
Wed, 11 Jul 2007 19:21:34 +0000 (19:21 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Wed, 11 Jul 2007 19:21:34 +0000 (19:21 +0000)
Doing a WHERE on x=NULL doesn't work; need to do 'x IS NULL' here.
Perhaps we should make the DB functions smarter to handle that case to do what we'd expect it to do (as we do turning arrays into 'IN (...)') but this'll fix it for now.

includes/UserMailer.php

index 3529c15..eac8727 100644 (file)
@@ -376,7 +376,7 @@ class EmailNotification {
                                ), array( /* WHERE */
                                        'wl_title' => $title->getDBkey(),
                                        'wl_namespace' => $title->getNamespace(),
-                                       'wl_notificationtimestamp' => NULL
+                                       'wl_notificationtimestamp IS NULL'
                                ), 'UserMailer::NotifyOnChange'
                        );
                        # FIXME what do we do on failure ?