Critical error while save settings on PHP 8.4.22

  • MooMee

    (@moomee8668)


    2 days ago

    Hello,
    I try to change scanning schedule from “hourly” to “daily” and got message:

    Warning: Undefined array key “scan-hour-am” in /{hosting-path}/wp-content/plugins/website-file-changes-monitor/classes/admin/class-admin-manager.php on line 510

    Fatal error: Uncaught TypeError: strtoupper(): Argument #1 ($string) must be of type string, null given in /{hosting-path}/wp-content/plugins/website-file-changes-monitor/classes/admin/class-admin-manager.php:510 Stack trace: #0 /{hosting-path}/wp-content/plugins/website-file-changes-monitor/classes/admin/class-admin-manager.php(510): strtoupper() #1 /{hosting-path}/wp-includes/class-wp-hook.php(341): MFM\Admin\Admin_Manager::save_options() #2 /{hosting-path}/wp-includes/class-wp-hook.php(365): WP_Hook->apply_filters() #3 /{hosting-path}/wp-includes/plugin.php(522): WP_Hook->do_action() #4 /{hosting-path}/wp-admin/admin.php(242): do_action() #5 {main} thrown in /{hosting-path}/wp-content/plugins/website-file-changes-monitor/classes/admin/class-admin-manager.php on line 510

    And got the same message when try to change other settings (i.e. “from” email)

    WordPress 7.0
    PHP version8.4.22 (Supports 64bit values)

Viewing 2 replies - 1 through 2 (of 2 total)
  • Thread Starter MooMee

    (@moomee8668)

    1 day, 20 hours ago

    // phpcs:ignore Generic.Strings.UnnecessaryStringConcat.Found

    does not work, because it is about concatenation, but error caused by strtoupper() function.

    I don’t know why $posted[‘scan-hour-am’] become NULL (in settings form there are no field am/pm), but to prevent this error can add in wp-content/plugins/website-file-changes-monitor/classes/admin/class-admin-manager.php before line 510:

    if ( !isset( $posted['scan-hour-am'] ) ) {
    $posted['scan-hour-am'] = get_site_option( MFM_PREFIX . 'scan-hour-am', Settings_Helper::get_settings_default_value( 'scan-hour-am' ) );
    };

    Plugin Support Lucian Padureanu

    (@lucianwpwhite)

    1 day, 19 hours ago

    Hello there @moomee8668 !

    I am really sorry to hear about this unexpected issue!

    Would it be possible to contact us directly at support@melapress.com so we can exchange a few more details in a more secure channel about the issue? I am confident we will sort this one out so looking forward to your update!

Viewing 2 replies - 1 through 2 (of 2 total)

You must be logged in to reply to this topic.