bbPress titles ‘untitled’
-
Today I noticed that with The SEO Framework (TSF) on bbPress profile titles are not shown correctly.
If I disable TSF the titles are empty, but that’s fixed with this code./* ---------------------------------------------------------------- // Fix broken BBPress titles ---------------------------------------------------------------- */ function ajdg_bbp_title($parts) { if (empty($parts['title']) && is_bbpress()) { $parts['title'] = trim(bbp_title('', '')); } return $parts; } add_filter('document_title_parts', 'ajdg_bbp_title');Yes, bbPress is not free of fault as well in this. But the proposed fix/workaround doesn’t work as long as The SEO Framework is enabled.
I found some similar issues with other plugins and TSF, and all were dismissed as the other plugin having issues. But I lean to an issue in TSF as this is yet another plugin that has this (or a very similar) issue in combination with TSF.
Regardless of where it breaks. I need/want this to work and the only logical solution I can find is that TSF either ignores bbPress profile page titles, or stops overwriting them. Since Topic and Reply titles work fine with TSF, well… TSF is missing something I think.
For example what can change is that TSF checks if the title is empty. If not, use the original string (instead of setting it to ‘untitled’) if TSF doesn’t know what to do with it. Simple enough.
Please advise!
The page I need help with: [log in to see the link]
-
The topics and replies work fine because there’s a WordPress Post Object
post_titleproperty present. But, bbPress’s template would like to showTopic: %s–which currently isn’t the case when TSF is active.——
Thanks for chiming in JJJ (@johnjamesjacoby)! Here are my thoughts:
Titles
What I would like to see is a simple function that processes the title for the current request. Something like:bbp_get_title().In essence,
bbp_get_title()should do everythingbbp_title()does, without the processing/appending of the separator and blog name. Here’s the code I’m envisioning for that function.Therein, filter
bbp_raw_titleshould be replaced, because we don’t work with providing title parts anymore (thanks to the infamous wp_title filter…). I’d like to suggest a new filter that can adjust the output, identical to the function name:bbp_get_title.In extent, you could create
bbp_filter_document_title_parts(), which filtersdocument_title_parts, providing that title. E.g.:add_filter( 'document_title_parts', 'bbp_filter_document_title_parts' ); function bbp_filter_document_title_parts( $title ) { if ( is_bbpress() ) { // You may want to use strlen instead of ?: $title['title'] = bbp_get_title() ?: $title['title']; } return $title; }Keep in mind that you still should support
wp_title; so, removing/changingbbp_title()as it currently functions wouldn’t be wise.Canonical URLs
We also have an issue with the canonical URL. WordPress has, since v4.6, a function calledwp_get_canonical_url(). In that function, there’s a canonical URL you can provide. It’d be great if you could supply the correct canonical URL to that filter 🙂Other SEO functionality
Now, in WordPress, there’s no default way to provide Open Graph, a meta description, or other metadata besides the title tag and canonical URL. So, I’m not sure if you can make it more compatible with SEO plugins–other than the aforementioned augmentation of the canonical URL filter and providing us an integral API function for the title.Query compatibility
As for not detecting a user-query via the WordPress API… well, that requires some arcane rewrite magic, which I don’t believe you should engage with further. I think it’s best to keep that as-is and let other developers use custom functionality, likebp_is_user(), when we desire to expand support.I hope these insights are helpful 🙂 Cheers!
- The topic ‘bbPress titles ‘untitled’’ is closed to new replies.
