Custom component directory title

[imath]

Since r8080, a new parameter 'directory_title' has been introduced to allow a component to set the directory page title.

It's nice but i think the change applied to bp_modify_page_title() introduced a little regression considering 1.9.x.

Before the page title of the custom component was set using bp_get_name_from_root_slug(), now the function bp_get_directory_title() first checks for component's 'directory_title' argument to see if it's set, which is always the case. Problem is that it's set with an empty value. So custom component that used to have a page title set thanks to bp_get_name_from_root_slug() will no more have a page title.

The patch attached simply changes isset in favor of ! empty which is fixing the issue.

[boonebgorges]

imath - This looks good at a glance, but can you please give a specific example of a plugin or customization that illustrates the issue? I want to be able to see it for myself.

[imath]

Replying to boonebgorges:

I want to be able to see it for myself.

I think it's all plugins that have a directory page actually, i've seen it while working on BuddyPress Skeleton Component (branch 1.7) and just saw it on BuddyDrive as i need to quicky update it as of changes in plupload in WP 3.9.

The page title is not showing the component's name <title> | Site name </title>

[boonebgorges]

Thanks, imath.

[boonebgorges]

In 8276:

Use empty() rather than isset() when checking for fallback directory_title component properties

isset() always returns true, even for empty values.

Fixes #5544

Props imath