Changeset 8167 for trunk/bp-notifications/bp-notifications-classes.php

Timestamp:

03/26/2014 10:55:04 PM (11 years ago)

Author:

boonebgorges

Message:

Allow all notifications to be fetched by passing 'both' to is_new param of BP_Notifications_Notification::get()

Fixes #5315

Props rodrigorznd

File:

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

trunk/bp-notifications/bp-notifications-classes.php

@@ -320 +320 @@
 
         // is_new
-        if ( ! empty( $args['is_new'] ) ) {
-            $where_conditions['is_new'] = "is_new = 1";
-        } elseif ( isset( $args['is_new'] ) && ( 0 === $args['is_new'] || false === $args['is_new'] ) ) {
-            $where_conditions['is_new'] = "is_new = 0";
+        if ( 'both' !== $args['is_new'] ) {
+            if ( ! empty( $args['is_new'] ) ) {
+                $where_conditions['is_new'] = "is_new = 1";
+            } elseif ( isset( $args['is_new'] ) && ( 0 === $args['is_new'] || false === $args['is_new'] ) ) {
+                $where_conditions['is_new'] = "is_new = 0";
+            }
         }
 
@@ -538 +540 @@
      *     @type string|array $component_action Name of the action to
      *           filter by. Can be an array of actions.
-     *     @type bool $is_new Whether to limit the query to is_new (unread)
-     *           notifications. Default: true.
+     *     @type bool $is_new Whether to limit to new notifications. True
+     *           returns only new notifications, false returns only non-new
+     *           notifications. 'both' returns all. Default: true.
      *     @type string $search_terms Term to match against component_name
      *           or component_action fields.