• markohoza

    (@markohoza)


    Hi,

    I noticed a bug in the Flow-Flow plugin logging logic.

    In libs/flowflow/core/src/LABase.php, inside refreshCache(), debug messages are logged when WP_DEBUG is disabled:

    if (!defined('WP_DEBUG') || !WP_DEBUG) {
        $this->log('Flow-Flow Debug: refreshCache called ...');
    }

    This condition appears to be inverted. With define('WP_DEBUG', false);, the plugin still writes repeated “Flow-Flow Debug” messages to the error log during cron cache refreshes.

    Expected behavior: debug logs should only be written when WP_DEBUG is enabled.

    Suggested fix:

    if (defined('WP_DEBUG') && WP_DEBUG) {
        $this->log('Flow-Flow Debug: refreshCache called ...');
    }

    The same inverted condition appears multiple times in refreshCache(), so all related debug log checks should be updated.

    Thanks.

You must be logged in to reply to this topic.