Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 12 years ago

#4242 closed defect (bug) (fixed)

Child Page - Page Titles

Reported by: mwbarker's profile mwbarker Owned by:
Milestone: 1.7 Priority: normal
Severity: normal Version: 1.5.5
Component: Core Keywords: has-patch needs-testing
Cc: jkudish

Description

If you assign BuddyPress pages as child pages, the page titles are not displayed correctly in the browser.

i.e. for the main Groups Directory page.

If it is a main (parent) page, title displays
User Groups Directory | Site Title

If it is a child page, title displays
Directory| Site TItle

You should be assign BuddyPress pages as child pages without breaking the page title generated in the <title> tags.

Attachments (2)

4242-1.patch (639 bytes) - added by DJPaul 13 years ago.
4242-2.diff (693 bytes) - added by jkudish 12 years ago.
refresh patch, checking both slug and root_slug

Download all attachments as: .zip

Change History (14)

#1 @DJPaul
13 years ago

  • Milestone changed from Awaiting Review to 1.6

Moving to 1.6 for investigation.

@DJPaul
13 years ago

#2 @DJPaul
13 years ago

  • Keywords has-patch dev-feedback added

The issue seems to be in the bp_get_name_from_root_slug() function. I've attached a patch that seems to work, but would appreciate someone to test this on lots of different combinations of page/component mappings, or feedback from another dev, before commit.

#3 @boonebgorges
13 years ago

I'll have to do some more testing, but I'm guessing it'll be necessary to check *both* the root_slug and slug properties. So a decision will have to be made about order of precedence.

#4 @DJPaul
12 years ago

  • Keywords needs-testing added; has-patch dev-feedback removed

#5 @DJPaul
12 years ago

  • Keywords needs-patch added; needs-testing removed
  • Milestone changed from 1.6 to Future Release

#6 @DJPaul
12 years ago

Moved to a future release as Boone and I don't think we should make this change at this stage of 1.6. As we are already in beta and have no unit test coverage for this, we could break things if we missed a non-obvious case by doing this in a hurry.

#7 @DJPaul
12 years ago

  • Milestone changed from Future Release to 1.7

#8 @k12onos
12 years ago

The patch does not work anymore on 1.6.1. I would really appreciate it if you could update the patch to work on 1.6.1 since 1.7 release will take some time.

@jkudish
12 years ago

refresh patch, checking both slug and root_slug

#9 @jkudish
12 years ago

  • Cc jkudish added
  • Keywords has-patch added; needs-patch removed

#10 @boonebgorges
12 years ago

  • Keywords needs-testing added

Ugh, this is the kind of thing that's scary to fix without unit tests. At the very least, it'd be nice if someone had the time to do some manual tests with different kinds of nested setups, to make sure that potential slug/root_slug conflicts are detected correctly.

#11 @jkudish
12 years ago

I agree that it needs more testing. FWIW, however, I did test at least a few basic scenarios before submitting the patch :)

#12 @johnjamesjacoby
12 years ago

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

(In [6616]) Fix bug causing page name/title to be incorrect when directory page has parntes. Props jkudish. Fixes #4242.

Note: See TracTickets for help on using tickets.