WordPress.org

Ready to get started?Download WordPress

Forums

WordPress File Monitor Plus
[resolved] Small bug in Other Cron code (3 posts)

  1. ionainteractive
    Member
    Posted 2 years ago #

    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/

  2. Scott Cariss
    Member
    Plugin Author

    Posted 2 years ago #

    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

  3. ionainteractive
    Member
    Posted 2 years ago #

    No worries! Glad to help resolve! :-)

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic