Filter on `wp_unique_post_slug` issue

[ahegyes]

Hello!

Version 12.0.0 introduced a new filter function called bp_core_set_unique_directory_page_slug attached to the filter wp_unique_post_slug.

My understanding is that the function wants to avoid any duplicate post_name entries across the post types buddypress and page. However, it seems to fail in doing that and causes issues in certain cases.

For example, it tries to accomplish its goal by first querying a list of posts of type buddypress and page (​https://github.com/buddypress/buddypress/blob/b5f1c43598581173edb3cbd2686ae075f912ae87/src/bp-core/bp-core-functions.php#L1002-L1008). However, since it doesn't use the limit or numberposts argument, only the top 5 posts will be returned instead of all of them.

Moreover, it doesn't handle hiearchies properly. Normally, pages are allowed to have the same post_name as long as they have different parents. However, this function doesn't take that into account.

In my scenario, I tried adding a page named Program as child of page B. However, this function's get_posts included another page named Program which is a child of page A. WordPress allows this, but this function changed the slug to program-2 for the child of B because the child of A happened to be amongst those 5 queried posts.

[imath]

Thanks for your report. I'll look at it asap.

This PR also improves how Slug check is performed only requesting for pages / BuddyPress directories potentially having the same post_name and not being a child of another item.

Trac ticket: ​https://buddypress.trac.wordpress.org/ticket/9086

[imath]

Hi @ahegyes

I believe the above PR is fixing the issue, can you confirm?
Thanks in advance.

[emaralive]

@imath, tested under the following conditions:

Without PR #256:

1. Created page A, slug = a
2. Created page Program as child of page A, slug = program
3. Created page B, slug = b
4. Created page Program as child of page B, slug = program-2

With PR #256:

1. Created page A, slug = a
2. Created page Program as child of page A, slug = program
3. Created page B, slug = b
4. Created page Program as child of page B, slug = program

Conclusion:
PR #256 provides a remedy for the reported issue.

[imath]

Hi @emaralive

Thanks a billion for your tests and confirmation ❤️. I'll commit the PR shortly and we'll respect our plan about packaging a 12.4.0 release tomorrow 🤝

[imath]

In 13777:

Improve the page / buddypress post types slugs unicity checks

To avoid slug conflicts between the page & buddypress post types as both URI Schemas put their slugs right after the site domain, 12.0.0 introduced a filter to extend the WordPress built-in check performed by wp_unique_post_slug().

This filter needed to be improve to :

• make sure to allow WordPress sub-pages to have the same slugs than the BP Directory ones as in this case the URI Schemas puts the parent page slug before the sub-page one.
• only request the page or buddypress post types having the same post_name than the page or buddypress post type being inserted.

Props ahegyes, emaralive

See #9086 (trunk)
Closes ​https://github.com/buddypress/buddypress/pull/256

[imath]

In 13778:

Improve the page / buddypress post types slugs unicity checks

To avoid slug conflicts between the page & buddypress post types as both URI Schemas put their slugs right after the site domain, 12.0.0 introduced a filter to extend the WordPress built-in check performed by wp_unique_post_slug().

This filter needed to be improve to :

• make sure to allow WordPress sub-pages to have the same slugs than the BP Directory ones as in this case the URI Schemas puts the parent page slug before the sub-page one.
• only request the page or buddypress post types having the same post_name than the page or buddypress post type being inserted.

Props ahegyes, emaralive

Fixes #9086 (branch 12.0)