Discourse "Publish to Discourse" Breaking with Beaver Builder Plugin

support wordpress
1

Hello!

Discourse is breaking when I click the “Publish to Discourse” button in a post.

The issue goes away when I disable the Beaver Builder plugin, however, after contacting the Beaver Builder team, this was their response:

The issue is with the discourse plugin, they are using a fillter wrong, one of our team reported it to them last year and even took the time to give them the fix as you can see here get_excerpt hook expects 2 arguments in WP 4.5 and above · Issue #432 · discourse/wp-discourse · GitHub there isnt anything we can do from our end im afraid.

Can you please address this issue? Here is the error from debug.log:

[20-Jan-2023 02:38:01 UTC] PHP Fatal error: Uncaught ArgumentCountError: Too few arguments to function FLBuilderCompatibility::fix_rest_excerpt_filter(), 1 passed in .../wp-includes/class-wp-hook.php on line 308 and exactly 2 expected in .../wp-content/plugins/bb-plugin/classes/class-fl-builder-compatibility.php:1180
Stack trace:
#0 .../wp-includes/class-wp-hook.php(308): FLBuilderCompatibility::fix_rest_excerpt_filter()
#1 .../wp-includes/plugin.php(205): WP_Hook->apply_filters()
#2 .../wp-content/plugins/wp-discourse/lib/discourse-publish.php(238): apply_filters()
#3 .../wp-content/plugins/wp-discourse/lib/discourse-publish.php(177): WPDiscourse\DiscoursePublish\DiscoursePublish->sync_to_discourse_work()
#4 .../wp-content/plugins/wp-discourse/lib/discourse-publish.php(126): WPDiscourse\DiscoursePublish\DiscoursePublish->sync_to_discourse()
#5 .../wp-content/plugins/wp-discourse/admin/discourse-sidebar/discourse-sidebar.php(516): WPDiscourse\DiscoursePublish\DiscoursePublish->publish_post_after_save()
#6 .../wp-includes/rest-api/class-wp-rest-server.php(1171): WPDiscourse\Admin\DiscourseSidebar->publish_topic()
#7 .../wp-includes/rest-api/class-wp-rest-server.php(1018): WP_REST_Server->respond_to_request()
#8 .../wp-includes/rest-api/class-wp-rest-server.php(442): WP_REST_Server->dispatch()
#9 .../wp-includes/rest-api.php(410): WP_REST_Server->serve_request()
#10 .../wp-includes/class-wp-hook.php(308): rest_api_loaded()
#11 .../wp-includes/class-wp-hook.php(332): WP_Hook->apply_filters()
#12 .../wp-includes/plugin.php(565): WP_Hook->do_action()
#13 .../wp-includes/class-wp.php(399): do_action_ref_array()
#14 .../wp-includes/class-wp.php(780): WP->parse_request()
#15 .../wp-includes/functions.php(1332): WP->main()
#16 .../wp-blog-header.php(16): wp()
#17 .../index.php(17): require('/home/...')
#18 {main}
thrown in .../wp-content/plugins/bb-plugin/classes/class-fl-builder-compatibility.php on line 1180

Thank you!

Jake

1 Like
2

Just letting you know that I did apply Beaver Builder’s recommended code fix from the pull request, and it fixed the problem. Can you all please apply this fix to the plugin?

1 Like
3

I can confirm so much that applying that modification didn’t break anything right away on OceanWP & classic editor.

4

@TEKFused Thanks, we will take a look at the issue and PR.

1 Like
5

Thank you very much!

6

Looks like they disabled issues for that plugin, for people finding this and need the patch the pull request is still there and still open Update for compatibility with get_the_excerpt filter by codente · Pull Request #433 · discourse/wp-discourse · GitHub

7

The policy towards issues and PRs for this plugin has recently been tweaked. This particular PR will be merged soon and be included in the next release. PRs to this plugin are definitely appreciated. As noted in the readme, all issues should be reported in wordpress.

1 Like