Improve bbPress compatibility with Jetpack's 'Widget Visibility'

[sgr33n]

Hi,

I'm trying to use the JetPack Widget Visibility option with bbPress, but it's impossibile to identify the first page in order to show some bbPress widget only on it. Even if would be possible to set it as a static page (the one used by bbPress with default guid "forums"), it doesn't recognize it until i deactivate bbPress. For all the other page Jetpack make it possibile to recognize bbPress forums, topics and replies by their different post types, so the only problematic page is the first one. I think this could be a bug because the page with guid "forums" should pass through JetPack Widget Visibility, but when bbPress is active it doesn't recognize it anymore.

Thanks.

[netweb]

Confirmed, using bbPress with Jetpack's Widget Visibility you are not able to assign widgets to the bbPress root /forums, using a page titled forums or a custom page eg. forumpage using Jetpack's visibilty option post type = forum or page = forums/forumpage etc.

http://example.com/forums <- Standard bbPress - Only Widget Visibility option is if post type = forum does not display any widgets.

http://example.com/forums <- Using a WP page titled forums and bbPress root slug forums using Widget Visibility post type = forum or page = forums does not display any widgets.

http://example.com/forumpage<- Using a WP page titled forumpage and bbPress root slug forumpage using Widget Visibility post type = forum or page = forumpage does not display any widgets.

Single Forums, Topics and Replies appear to work fine, just the root forums page is the issue.

[johnjamesjacoby]

Any luck with topics views? I suspect any theme comparability page probably has problems as we'll.

[netweb]

Replying to johnjamesjacoby:

Any luck with topics views? I suspect any theme comparability page probably has problems as we'll.

Single Forums, Topics and Replies appear to work fine, just the root forums page is the issue.

[jeherve]

Replying to netweb:

http://example.com/forums <- Standard bbPress - Only Widget Visibility option is if post type = forum does not display any widgets.

http://example.com/forums <- Using a WP page titled forums and bbPress root slug forums using Widget Visibility post type = forum or page = forums does not display any widgets.

http://example.com/forumpage<- Using a WP page titled forumpage and bbPress root slug forumpage using Widget Visibility post type = forum or page = forumpage does not display any widgets.

I'm not really familiar with bbPress, so you'll excuse me if my question seems stupid.

How would you check for each one of these situations using standard ​conditional tags, or custom bbPress conditional tags? We might have to customize Jetpack to add bbPress compatibility.

[netweb]

Replying to jeherve:

How would you check for each one of these situations using standard ​conditional tags, or custom bbPress conditional tags? We might have to customize Jetpack to add bbPress compatibility.

is_bbpress() ​/trunk/includes/common/template-tags.php?rev=4919#L1012

[sgr33n]

That also happen with ​http://ww2.bbpress.org/forums/view/no-replies/ and ​http://ww2.bbpress.org/forums/view/popular/

[jeherve]

Thanks! I created a ticket in the plugins trac, and we'll see about adding extra checks when bbPress is active:

• ​http://plugins.trac.wordpress.org/ticket/2050

[johnjamesjacoby]

Closing this here, so we can focus on getting Jetpack to support this with bbPress's conditional tags.

[SGr33n]

For the records, I discovered that the forum root is sensitive to visibility "page" is "archive page". Nothing about the two page views links.

[SGr33n]

This would solve temporarily the issue.

Replace \wp-content\plugins\jetpack\modules\widget-visibility\widget-conditions.php with the attached one.

Anyway if you used the page is_archive page, as told before, this will affect also the forum root.

[SGr33n]

Replace this to add page is bbpress condition

[SGr33n]

Started a new ticket.

[netweb]

@SGr33n Let's see what the Jetpack team come up with:
​https://github.com/Automattic/jetpack/issues/315 and ​https://github.com/Automattic/jetpack/pull/167 and
​https://github.com/Automattic/jetpack/pull/434

[SGr33n]

I see, but... still waiting :P