[buddypress-trac] [BuddyPress Trac] #8280: BP Nouveau throwing notices on including members-loop.php

[buddypress-trac]

#8280: BP Nouveau throwing notices on including members-loop.php
--------------------------+-----------------------------
 Reporter:  sbrajesh      |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Templates     |    Version:  5.0.0
 Severity:  normal        |   Keywords:  needs-patch
--------------------------+-----------------------------
 In BP Nouveau template pack, various items loop uses
 'bp_nouveau_pagination()' to show the pagination.

 The problem with this function is, it checks for specific components and
 screens. If the fixed list of component is not current component, it
 throws notice as shown in attached file.

 This practically makes it impossible to use the loops outside the
 specified context of the function.

 The proposed solution is to set the default values for $pag_count,
 $pag_links, $page_arg and if possible allow filtering $pagination_type.


 Also, to answer why we need this loop is, we include it to take the
 advantage by plugins.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8280>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac