[buddypress-trac] [BuddyPress] #3265: bp-default - move sidebar to footer

[buddypress-trac at lists.automattic.com]

#3265: bp-default - move sidebar to footer
--------------------------+-----------------------------
  Reporter:  svenl77      |      Owner:
      Type:  enhancement  |     Status:  new
  Priority:  normal       |  Milestone:  Awaiting Review
 Component:  Theme        |    Version:  1.2
Resolution:               |   Keywords:  bp-default
--------------------------+-----------------------------

Comment (by svenl77):

 By reading the code of the locate_template() function , I can see it comes
 with 3 parameters.

 locate_template($template_names, $load = false, $require_once = true )

 the third parameter “$require_once” is ignored in buddypress and default
 is true

 locate_template( array( ‘sidebar.php’ ), true )

 that means the conflict could just happen by using something like this

 locate_template( array( ‘sidebar.php’ ), true, false ); where
 $require_once is false.

 and only if we add $require_once as false in the footer too.

 I did some testing, and even if a plugin has  locate_template( array(
 ‘sidebar.php’ ), true, false ).

 There will not be another sidebar rendered if we use locate_template(
 array( ‘sidebar.php’ ), true, true) in the footer.

-- 
Ticket URL: <https://trac.buddypress.org/ticket/3265#comment:5>
BuddyPress <http://buddypress.org/>
BuddyPress