chewventure

That’s awesome, thanks for the quick response!

Will have a go on my site and see how things go (y)

Oh apologies, I didn’t see that in the description!

Yes, definitely could add some jquery as a workaround but would also be nice if it could be an option built into the plugin too ??

I agree with @francesca111 , this should have been caught in a basic integration smoke test.

How has a fix not been put out for this yet? It’s not acceptable seeing as people are paying for this plugin.

Sorry, that was a typo when stripping out personal data from my function.

Hmm, I don’t know why it would work on 2 sites, but not on my other one. Really a mystery – in the end I added the jQuery manually and it works. So strange.

Probably never get to the bottom of it!

@andrewza did you have any luck with the investigation with this?

Ok, so I wrote my own very clunky cron job to clear the Membership level if the user has no active Subscriptions using:

include_once("wp-config.php");
include_once("wp-includes/wp-db.php");

global $wpdb;

$sql = "UPDATE wp_pmpro_memberships_users
SET wp_pmpro_memberships_users.status = 'admin_cancelled'
WHERE wp_pmpro_memberships_users.user_id IN 
    (
    /*Select all users that have an active Membership but no active Subscription*/
    SELECT pmpro.user_id FROM (SELECT * FROM wp_pmpro_memberships_users) as pmpro
    JOIN wp_pmpro_membership_levels as pmprol ON pmprol.id=pmpro.membership_id
    JOIN wp_users ON pmpro.user_id = wp_users.ID
    WHERE pmpro.status = 'active'
    AND pmpro.user_id NOT IN 
        (
        /*Select all users that have an active Subscription*/
        SELECT meta.meta_value 
        FROM wp_posts as p
        JOIN wp_postmeta as meta
        ON meta.post_id = p.id
        JOIN wp_users as u
        ON meta.meta_value = u.id
        WHERE post_type LIKE 'shop_subscription'
        AND post_status LIKE 'wc-active'
        AND meta.meta_key = '_customer_user'
        )
    )";
	
$results = $wpdb->get_results($sql);

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

Ok, I found a solution from https://stackoverflow.com/questions/36079651/silence-declaration-should-be-compatible-warnings-in-php-7

In my case I had to add arguments to the WC_Product_Variable_Subscription::get_available_variations() function

Ok, I found a solution from https://stackoverflow.com/questions/36079651/silence-declaration-should-be-compatible-warnings-in-php-7

In my case I had to add arguments to the WC_Product_Variable_Subscription::get_available_variations() function

Hmm, that’s odd. Thanks for looking into it.

I wonder why downgrading the php fixes it in our situation…

Will try to find a way to get the WooCommerce login details and follow up that way.

@eastofwest – I’m working for a client that does not have access to the log-in details for WooCommerce and is not on good relationship with the original developer, thus this is our only support channel.

Please don’t close this as we need to understand why we have to use an older, insecure version of php to have WooCommerce Core and WooCommerce Subscription play nicely together.

Ok, so I’ve rolled back my PHP version to 7.1.33 and the warning has disappeared.

I’m guessing this is an integration issue between the plugins that WooCommerce needs to fix?

Ok, so I’ve rolled back my PHP version to 7.1.33 and the warning has disappeared.

I’m guessing this is an integration issue between the plugins that WooCommerce needs to fix?

Thanks for the response @andrewza

Thank you @rynald0s

Hi @andrewza – did you have any success with the testing?

Cheers