Skip to:
Content

BuddyPress.org

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#4771 closed defect (bug) (duplicate)

Show the Toolbar for logged out users | Toolbar not showing no matter what

Reported by: magichew's profile magichew Owned by:
Milestone: 1.7 Priority: normal
Severity: normal Version: 1.7
Component: Core Keywords: has-patch
Cc:

Description

No matter whether checked or not checked the Toolbar doesn't show to logged out users.

Attachments (1)

4771.01.patch (1021 bytes) - added by r-a-y 12 years ago.

Download all attachments as: .zip

Change History (10)

#1 @r-a-y
12 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 1.7

Confirmed.

According to the WP codex article, show_admin_bar() should be run on plugin loaded.

I've attached a patch that moves the BP admin bar loading from 'bp_init' to 'bp_loaded' and that fixes things.

@r-a-y
12 years ago

#2 @magichew
12 years ago

Patch applied and working.

#3 @DJPaul
12 years ago

  • Keywords commit added

#4 @r-a-y
12 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [6724]) Fix 'Show the Toolbar for logged out users' setting.

bp_core_load_admin_bar() uses the show_admin_bar() function; this
function toggles the WP toolbar visibiility.

WP recommends to use show_admin_bar() on 'plugins_loaded':
https://codex.wordpress.org/Function_Reference/show_admin_bar#Notes

Therefore, this commit changes when bp_core_load_admin_bar() runs
from 'bp_init' to 'bp_loaded'.

Fixes #4771.

#5 @magichew
12 years ago

I understand that you guys are perhaps still figuring this one out but I just wanted to let you all know that this hasn't been patched in the 1.7 beta 1. I've had to manually added the patch again after updating.

#7 @magichew
12 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Hello. Sorry to keep going on about this but this issue still persists in the release version. I had to manually add the patch to make the bar appear.

#8 @r-a-y
12 years ago

  • Keywords commit removed
  • Resolution set to duplicate
  • Status changed from reopened to closed

Closing in favor of #4803 since that ticket is more current and has an updated patch.

magichew: Try the patch in #4803.

#9 @magichew
12 years ago

Okay. Will do. Thanks r-a-y

Note: See TracTickets for help on using tickets.