Nathan

Hi Martin,
We can’t take the risk installing this on a production site again, so we found another solution. Sorry!

Nathan

Thanks for such a quick fix!

Thanks @kingyes!

Hi Rick,
Sorry, but the issue is with your code. I thought I explained well exactly what was not working. MigrationsRunner::MIGRATION_OPTION is not defined anywhere.

If anyone else needs to fix this, we just commented out all the code in updateMigrationsSetting() so we could make this thing create the missing table.

We have the same issue. It looks like your Manual Migration runner is still trying to use the legacy way of tracking completed migrations
update_option( MigrationsRunner::MIGRATION_OPTION, $completedMigrations );

However, MigrationsRunner::MIGRATION_OPTION is not defined.

No, sorry. This just works for the shop page. The approach is basically to disable Connect on the pages you want to create a template for. Any more customization would require changes to the code.

If you want to use Elementor for everything, you really should just deactivate Connect. It replaces woo templates with its own that are not compatibile with Elementor. The code above only deactivates Connect on the shop page.

Actually I did, I wrote some code to disable connect on the shop page when Elementor is turned on.

function sprout_fix_gc() {
	if ( is_plugin_active( 'genesis-connect-woocommerce/genesis-connect-woocommerce.php' ) && is_plugin_active( 'elementor/elementor.php' ) && is_shop() ) {
		add_filter( 'template_include', array( &$woocommerce, 'template_loader' ) );
		remove_filter( 'template_include', 'gencwooc_template_loader', 20 );
	}
}
add_action( 'wp', 'sprout_fix_gc' );

I hope that helps you too!

• This reply was modified 4 years, 2 months ago by Nathan.

Just following up to see if you will be able to apply this fix or something similar so it doesn’t break again on the next update. Thanks!

Okay, once I realized that was the problem, I just moved the gif echo to the end:

<?php

/* Connection: close and Content-Length headers are sent, so that browsers disconnect
 * early on to minimize connection time */

//The following outputs a 1x1 pixel gif. It is not really need for the functionality but nice to have.
ignore_user_abort(true);
// ob_end_clean();
$gif = base64_decode('R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7');
$length = strlen( $gif );

header('Content-type: image/gif');
header('Connection: close');
header("Content-Length: $length");
//End of gif output.

require_once ("../../../wp-load.php");
require_once ("config.php");
require_once ("source/Tracking/RequestTracker.php");

if (isset($_GET[WPAM_PluginConfig::$RefKey])
        && get_option(WPAM_PluginConfig::$AffEnableImpressions)) {
    try {
        $requestTracker = new WPAM_Tracking_RequestTracker();
        $request_data = array_map('strip_tags', $_GET);
        $requestTracker->handleImpression($request_data);
    } catch (Exception $e) {
        wp_die("WPAM FAILED: " . $e->getMessage());
    }
}

echo $gif;

exit;

Ah ha! I found it. The issue was different. I looked at the log file and realized it wasn’t the header! It was the gif output.

[28-Aug-2020 16:43:16 UTC] PHP Warning: Cannot modify header information - headers already sent by (output started at /home/USER/public_html/wp-content/plugins/affiliates-manager/imp.php:15) in /home/USER/public_html/wp-content/plugins/affiliates-manager/classes/ClickTracking.php on line 40

line 40 is trying to set a cookie in the header!

if(!empty($aff_id)){
    $cookie_life_time = wpam_get_cookie_life_time();
    setcookie('wpam_id', $aff_id, $cookie_life_time, "/", COOKIE_DOMAIN);
}

• This reply was modified 4 years, 3 months ago by Nathan.

Sadly, it’s still there ??

header('Content-type: image/gif');
header('Connection: close');

// Output a 1x1 pixel transparent gif
ob_start();
echo base64_decode('R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7');
$length = ob_get_length();

header("Content-Length: $length");

It says the error is on line 21, so the ob_end_flush(). It looks like data has already been sent out by time you are adding headers on line 10 and 11. Even it that is not true, line 18 is sending headers after an echo, which would also throw an error.

We deactivated that feature, but are still getting the error. I’ll send my contact info to you. Thanks!

Anyone able to help explain why this is breaking elementor? Thanks!