Changeset 7533

Timestamp:

11/08/2013 05:45:34 AM (11 years ago)

Author:

johnjamesjacoby

Message:

Make sure notifications always have date. See #5148.

File:

• trunk/bp-members/bp-members-notifications.php (modified) (2 diffs)

trunk/bp-members/bp-members-notifications.php

@@ -24 +24 @@
  * @param string $secondary_item_id
  * @param string $date_notified
- * @param string $is_new
- * @return boolean True on success, false on fail
- */
-function bp_core_add_notification( $item_id, $user_id, $component_name, $component_action, $secondary_item_id = 0, $date_notified = false, $is_new = true ) {
-
-    // Bail if notifications is not active
-    if ( ! bp_is_active( 'notifications' ) ) {
-        return false;
-    }
-
-    $args = array(
+ * @param int $is_new
+ * @return boolean True on success, false on fail
+ */
+function bp_core_add_notification( $item_id, $user_id, $component_name, $component_action, $secondary_item_id = 0, $date_notified = false, $is_new = 1 ) {
+
+    // Bail if notifications is not active
+    if ( ! bp_is_active( 'notifications' ) ) {
+        return false;
+    }
+
+    // Notifications must always have a time
+    if ( false === $date_notified ) {
+        $date_notified = bp_core_current_time();
+    }
+
+    // Add the notification
+    return bp_notifications_add_notification( array(
         'item_id'           => $item_id,
         'user_id'           => $user_id,
@@ -42 +48 @@
         'date_notified'     => $date_notified,
         'is_new'            => $is_new
-    );
-
-    return bp_notifications_add_notification( $args );
+    ) );
 }