Support » Plugin: bbPress - Moderation Tools » Error when approving topic or reply

  • Resolved Antipole

    (@antipole)


    When I approve a reply from the front-end button, I get two PHP warnings logged:

    Warning: Missing argument 4 for gdbbx_attachments_front::save_reply() in /home/antipole/public_html/ovni-owners-test-site/wp-content/plugins/gd-bbpress-toolbox/modules/attachments/front.php on line 175
    Warning: Missing argument 5 for gdbbx_attachments_front::save_reply() in /home/antipole/public_html/ovni-owners-test-site/wp-content/plugins/gd-bbpress-toolbox/modules/attachments/front.php on line 175

    I have tested this with the minimum of plug-ins active, namely: bbPress; gd bbpress toolbox; bbPress Moderation Tools; Error Log Monitor; Log Emails,; Stop emails & User Switching.

    I have been in contact with the author of gd-bbpress-toolbox, who advises:

    This is a problem caused by the ‘bbPress Moderation Tools’ plugin that executes bbPress actions ‘bbp_new_reply’ and ‘bbp_new_topic’ with wrong signature (some arguments are missing). My plugin depends on that fifth argument. If any plugin should run the actions bbPress runs in its core, they need to match the arguments, and this plugin is not doing that, so I can’t help you with it until the author of the plugin fixes it. You can try contacting plugin author to fix the plugin,

    Can you look at this, please.

Viewing 6 replies - 1 through 6 (of 6 total)
  • Plugin Author Digital Arm

    (@digital-arm)

    Apologies for the delay looking into this for you. If you would like to update to the latest version (1.2.2) the plugin has been updated to include all required arguments for the bbp_new_reply and bbp_new_topic actions.

    This should resolve the issue but please let us know if you experience any problems.

    Thread Starter Antipole

    (@antipole)

    Thanks for looking into this. I have updated and tested, but am getting the following logged. (Have replaced some details of my file structure by {file} and database name with {my database name} for security reasons.) Please advise. Thanks.

    Warning: Use of undefined constant time – assumed ‘time’ (this will throw an Error in a future version of PHP) in {file}/wp-content/plugins/moderation-tools-for-bbpress/incs/moderation.php on line 245

    WordPress database error Table ‘{my database name}.n’ doesn’t exist for query SELECT * FROM n WHERE user_id IN (2) AND item_id IN (9363) AND secondary_item_id IN (164) AND component_name IN (‘forums’) AND component_action IN (‘bbp_new_reply’) AND is_new = 1 made by require(‘wp-blog-header.php’), require_once(‘wp-load.php’), require_once(‘wp-config.php’), require_once(‘wp-settings.php’), do_action(‘init’), WP_Hook->do_action, WP_Hook->apply_filters, bbPressModToolsPlugin_Moderation->handle_actions, bbPressModToolsPlugin_Moderation->handle_moderation_action, do_action(‘bbp_new_reply’), WP_Hook->do_action, WP_Hook->apply_filters, bbp_buddypress_add_notification, bp_notifications_add_notification, BP_Notifications_Notification::get

    WordPress database error Incorrect table name ” for query SHOW FULL COLUMNS FROM made by require(‘wp-blog-header.php’), require_once(‘wp-load.php’), require_once(‘wp-config.php’), require_once(‘wp-settings.php’), do_action(‘init’), WP_Hook->do_action, WP_Hook->apply_filters, bbPressModToolsPlugin_Moderation->handle_actions, bbPressModToolsPlugin_Moderation->handle_moderation_action, do_action(‘bbp_new_reply’), WP_Hook->do_action, WP_Hook->apply_filters, bbp_buddypress_add_notification, bp_notifications_add_notification, BP_Notifications_Notification->save, BP_Notifications_Notification::_insert

    Warning: Cannot modify header information – headers already sent by (output started at {file}/wp-content/plugins/moderation-tools-for-bbpress/incs/moderation.php:245) in {file}/wp-includes/pluggable.php on line 1223

    Plugin Author Digital Arm

    (@digital-arm)

    That seems to be a slightly different issue which I’ve just fixed and released a new hotfix for (1.2.3).

    I’m not sure if that will fix your database errors but please update and try again.

    Thread Starter Antipole

    (@antipole)

    Thanks for the prompt response. v1.2.3 has eliminated the first and last of the above errors, but I am still seeing the two database errors.

    On my test site now running v1.2.3, I have taken a fresh copy of the database from my live site. I have deactivated all plugins that have anything to do with bbPress (apart from bbPress itself and BuddyPress) and its forums etc. and I am still seeing the database errors.

    As the errors mention this plugin, please can you take another look? Thanks.

    Plugin Author Digital Arm

    (@digital-arm)

    Would you be able to create an Admin user on your test site so I can take a quick look? If so please email the details to support@digitalarm.co.uk.

    Thread Starter Antipole

    (@antipole)

    Thank you very much for investigating and promptly fixing this in v1.2.4. Excellent support!

    You mentioned in your email that the topic author will receive a notification even if the reply is held for moderation. With my plugins active (which were not during your testing on my site), I do not see this issue because I have taken over the issuing of notifications and do it from my own plugin. This works whether a topic/reply is published from the front or backend. For moderated topics/replies, this only happens when the topic is finally published, whether from the front or backend.

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘Error when approving topic or reply’ is closed to new replies.