Support » Plugin: WordPress File Monitor Plus » [Plugin: WordPress File Monitor Plus] Small bug in Other Cron code

  • Resolved ionainteractive

    (@ionainteractive)


    There is a small bug in line #137 of classes/wpfmp.class.php (function external_cron_run_check). When checking the $_GET parameters to decide whether or not to process an external cron request, you currently have “isset($_GET[‘sc_wpfmp_key’])” as a disqualifying expression, but it’s negation should be the disqualifier, because that parameter MUST be presented (and match the back-end security_key option value) in order to process. So that expression simply needs an exclamation point before it, to be:

    if( ! isset( $_GET[‘sc_wpfmp_scan’] ) || ! isset( $_GET[‘sc_wpfmp_key’] ) || 1 != $_GET[‘sc_wpfmp_scan’] || $options[‘security_key’] != $_GET[‘sc_wpfmp_key’] || “other” != $options[‘cron_method’] )
    return;

    http://wordpress.org/extend/plugins/wordpress-file-monitor-plus/

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Scott Cariss

    (@l3rady)

    Thankyou very much for reporting this. I made this mistake when refactoring the code *doh*. I have pushed an update to the repository 2.0.1 hopefully this fixes it.

    I hope this bug wasn’t to hard to track down and I’m sorry if it caused you any inconvenience.

    Kind Regards

    Scott

    No worries! Glad to help resolve! 🙂

Viewing 2 replies - 1 through 2 (of 2 total)
  • The topic ‘[Plugin: WordPress File Monitor Plus] Small bug in Other Cron code’ is closed to new replies.