Skip to:
Content

Opened 7 months ago

Last modified 6 months ago

#7552 new enhancement

Introduce 'visibility' parameter for registering BP nav items

Reported by: r-a-y Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Navigation Keywords: needs-patch
Cc:

Description

Split from #7349, the following patch introduces a new 'visibility' parameter in bp_core_new_nav_item().

If set to false, this allows for a nav item to be hidden, but the screen function to still be registered.

More details can be found from this comment and on:
https://buddypress.trac.wordpress.org/ticket/7349#comment:13

Attachments (1)

7552.01.patch (2.1 KB) - added by r-a-y 7 months ago.

Download all attachments as: .zip

Change History (4)

@r-a-y
7 months ago

#1 @DJPaul
6 months ago

  • Component changed from Core to Navigation
  • Milestone changed from Under Consideration to 3.0

I like the principle, let's take a look.

#2 @DJPaul
6 months ago

  • Keywords needs-patch added; has-patch dev-feedback removed
  • Milestone changed from 3.0 to Future Release

I don't believe we can remove the show_for_displayed_user parameter.

#3 @r-a-y
6 months ago

Have you tested the patch? It works.

Do the following:

  • Make sure you are logged out.
  • Navigate to any user's Messages page (example.com/members/user/messages/)
  • With the patch, you get redirected to login. Without the patch, you get 404'd.
Last edited 6 months ago by r-a-y (previous) (diff)
Note: See TracTickets for help on using tickets.