Skip to:

Opened 11 years ago

Closed 11 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


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 11 years ago.
4242-2.diff (693 bytes) - added by jkudish 11 years ago.
refresh patch, checking both slug and root_slug

Download all attachments as: .zip

Change History (14)

#1 @DJPaul
11 years ago

  • Milestone changed from Awaiting Review to 1.6

Moving to 1.6 for investigation.

11 years ago

#2 @DJPaul
11 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
11 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
11 years ago

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

#5 @DJPaul
11 years ago

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

#6 @DJPaul
11 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
11 years ago

  • Milestone changed from Future Release to 1.7

#8 @k12onos
11 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.

11 years ago

refresh patch, checking both slug and root_slug

#9 @jkudish
11 years ago

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

#10 @boonebgorges
11 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
11 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
11 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.