WordPress caught an error with one of your plugins, Better Notifications for WP

  • Resolved adamkolenda

    (@adamkolenda)


    3 years, 2 months ago

    Hi,

    It looks as the plugin causes an outage on the WordPress site, After activation the plugin and setting up the initial notification which in deed worked fine I have logged out of the platform and while trying to log back in I got following WordPress window:

    “There has been a critical error on this website. Please check your site admin email inbox for instructions.”

    When checking the mailbox I can read the following:

    Since WordPress 5.2 there is a built-in feature that detects when a plugin or theme causes a fatal error on your site, and notifies you with this automated email.

In this case, WordPress caught an error with one of your plugins, Better Notifications for WP.

When seeking help with this issue, you may be asked for some of the following information:
WordPress version 5.8.1
Current theme: Hestia (version 3.0.19)
Current plugin: Better Notifications for WP (version 1.8.6) PHP version 7.4.6

Error Details
=============
An error of type E_ERROR was caused in line 919 of the file /var/www/html/wp-content/plugins/bnfw/bnfw.php. Error message: Uncaught ArgumentCountError: Too few arguments to function BNFW::user_login(), 1 passed in /var/www/html/wp-includes/class-wp-hook.php on line 303 and exactly 2 expected in /var/www/html/wp-content/plugins/bnfw/bnfw.php:919
Stack trace:
#0 /var/www/html/wp-includes/class-wp-hook.php(303): BNFW->user_login()
#1 /var/www/html/wp-includes/class-wp-hook.php(327): WP_Hook->apply_filters()
#2 /var/www/html/wp-includes/plugin.php(470): WP_Hook->do_action()
#3 /var/www/html/wp-content/plugins/miniorange-radius-client/mo_submit_otp.php(50): do_action()
#4 /var/www/html/wp-includes/class-wp-hook.php(303): Mo_Rad_pass2login->mo_rad_submit()
#5 /var/www/html/wp-includes/class-wp-hook.php(327): WP_Hook->apply_filters()
#6 /var/www/html/wp-includes/plugin.php(470): WP_Hook->do_action()
#7 /var/www/html/wp-settings.php(578): do_action()
#8 /var/www/html/wp-config.php(135): require_once('/var/www/html/w...')
#9 /var/www/html/wp-load.php(50): require_once('/var/www/html/w...')
#10 /var/www/html/wp-login.php(12): require('/var

    I’m only able to access the site again if I remove the plugin completely from the site. However, I think the plugin provides the functionality I need and therefore would appreciate if it can be repaired so that I can use it on my site.

Viewing 5 replies - 1 through 5 (of 5 total)
  • Plugin Author bnfw

    (@voltronik)

    Hi @adamkolenda,
    Thanks for your message.

    I think this issue is actually caused by the miniorange-radius-client plugin which is using an outdated method of registering or logging in users. WordPress has required two parameters for the user_login function for a few years now and whilst BNFW does support this, if another plugin only provides one parameter, it can cause problems, which i your case has been highlighted in the BNFW plugin.

    If you deactivate the miniorange-radius-client plugin and re-test, does this fix the issue?

    Thread Starter adamkolenda

    (@adamkolenda)

    Thanks @voltronik I will investigate this with the Radius developers.

    However why would BNFW plugin be involved in authentication? I though this was only notification plugin?

    Thanks

    Plugin Author bnfw

    (@voltronik)

    Hi @adamkolenda,
    Yes, that’s correct however, the notification is triggered off of a hook within WordPress which is filterable and if the other plugin is overwriting that with their own version then the last thing in this chain, BNFW in this case, will highlight the error. The same thing happens with WooCommerce and BNFW on occasion too for example, as WooCommerce overwrites the default WP user registration with their own version.

    Thread Starter adamkolenda

    (@adamkolenda)

    Thanks,

    That is now fixed with the Radius Vendor and all works fine. Topic can be closed. Many thanks for your help

    Plugin Author bnfw

    (@voltronik)

    Hi @adamkolenda,
    Great! Glad you got it working and that the other plugin author was quick to respond too.

    If you like BNFW and the support, please feel free to leave an honest review.

Viewing 5 replies - 1 through 5 (of 5 total)
  • The topic ‘WordPress caught an error with one of your plugins, Better Notifications for WP’ is closed to new replies.