Skip to:
Content

bbPress.org

Opened 8 years ago

Closed 8 years ago

#2361 closed defect (fixed)

Breadcrumbs 'include_current' argument doesn't work

Reported by: mrcl Owned by: johnjamesjacoby
Milestone: 2.4 Priority: normal
Severity: minor Version: 2.3.2
Component: General Keywords: needs-patch
Cc:

Description

On bbp_get_breadcrumb(), the include_current argument isn't working due to a improperly called variable, ignoring argument value set by the user and passing the default value instead.

On http://bbpress.trac.wordpress.org/browser/tags/2.3.2/includes/common/template-tags.php#L2280, where is $r['pre_current_text'], should be $r['current_text'].

I'm sorry, I'm new here. I tried to learn how to submit a patch, but I got confused.

Change History (2)

#1 @johnjamesjacoby
8 years ago

  • Milestone changed from Awaiting Review to 2.4

#2 @johnjamesjacoby
8 years ago

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

In 5005:

Use the correct array index string in bbp_get_breadcrumb() ( 'current_text' VS 'pre_current_text' ). Props mrcl. Fixes #2361.

Note: See TracTickets for help on using tickets.