Changeset 13102 for trunk/src/bp-messages/bp-messages-template.php

Timestamp:

09/09/2021 04:10:21 AM (4 years ago)

Author:

espellcaste

Message:

Adding support to the offset pagination to Messages and Recipients in a thread.

Previously, the BP_Messages_Thread query class would grab ALL messages created for a thread and ALL its recipients. This commit introduces support to the offset pagination
allowing one to paginate both messages and recipients using parameters with a specific limit and offset.

Props imath
Fixes #8508

File:

• trunk/src/bp-messages/bp-messages-template.php (modified) (2 diffs)

trunk/src/bp-messages/bp-messages-template.php

@@ -1626 +1626 @@
  * @param array|string $args {
  *     Array of arguments. All are optional.
- *     @type int    $thread_id         ID of the thread whose messages you are displaying.
- *                                     Default: if viewing a thread, the thread ID will be parsed from
- *                                     the URL (bp_action_variable( 0 )).
- *     @type string $order             'ASC' or 'DESC'. Default: 'ASC'.
- *     @type bool   $update_meta_cache Whether to pre-fetch metadata for
- *                                     queried message items. Default: true.
+ *     @type int      $thread_id         ID of the thread whose messages you are displaying.
+ *                                       Default: if viewing a thread, the thread ID will be parsed from
+ *                                       the URL (bp_action_variable( 0 )).
+ *     @type string   $order             'ASC' or 'DESC'. Default: 'ASC'.
+ *     @type bool     $update_meta_cache Whether to pre-fetch metadata for
+ *                                       queried message items. Default: true.
+ *     @type int|null $page              Page of messages being requested. Default to null, meaning all.
+ *     @type int|null $per_page          Messages to return per page. Default to null, meaning all.
  * }
+ *
  * @return bool True if there are messages to display, otherwise false.
  */
@@ -1638 +1641 @@
     global $thread_template;
 
-    $r = bp_parse_args( $args, array(
-        'thread_id'         => false,
-        'order'             => 'ASC',
-        'update_meta_cache' => true,
-    ), 'thread_has_messages' );
+    $r = bp_parse_args(
+        $args,
+        array(
+            'thread_id'         => false,
+            'order'             => 'ASC',
+            'update_meta_cache' => true,
+            'page'              => null,
+            'per_page'          => null,
+        ),
+        'thread_has_messages'
+    );
 
     if ( empty( $r['thread_id'] ) && bp_is_messages_component() && bp_is_current_action( 'view' ) ) {