Changeset 11828 for trunk/src/bp-groups/bp-groups-template.php

Timestamp:

01/27/2018 01:53:26 PM (7 years ago)

Author:

hnla

Message:

Pass additional $args to 'bp_get_new_group_invite_friend_list()`

This function renders markup for the looped item defaulting to 'li' but leaves it to the template to provide the parent element.

Commit provides two new args 'before' & 'after' to allow the function to return the complete element markup if desired, simplifies templates having to render markup and checks for empty function return.

Props DJPaul.

Fixes #7656

File:

• trunk/src/bp-groups/bp-groups-template.php (modified) (3 diffs)

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

@@ -5019 +5019 @@
             'user_id'   => bp_loggedin_user_id(),
             'group_id'  => false,
-            'separator' => 'li'
+            'before'    => '',
+            'separator' => 'li',
+            'after'     => '',
         ), 'group_invite_friend_list' );
 
@@ -5033 +5035 @@
         $items = array();
 
+        // Build list markup parent elements.
+        $before = '';
+        if ( ! empty( $r['before'] ) ) {
+            $before = $r['before'];
+        }
+
+        $after = '';
+        if ( ! empty( $r['after'] ) ) {
+            $after = $r['after'];
+        }
+
         // Get user's friends who are not in this group already.
         $friends = friends_get_friends_invite_list( $r['user_id'], $r['group_id'] );
@@ -5059 +5072 @@
 
         if ( ! empty( $invitable_friends ) && is_array( $invitable_friends ) ) {
-            $retval = implode( "\n", $invitable_friends );
+            $retval = $before . implode( "\n", $invitable_friends ) . $after;
         } else {
             $retval = false;