Change membership level access does not send an email about pmpro-multiple-memberships-per-user

[michaelbeil]

A member reported this in ticket 359121 (moderators only).

When using the Multiple Memberships Per User Plugin, and changing a user’s membership level or adding a membership from User Admin panel, notification emails are not being sent to the user. Test emails however from the Member email settings however, do seem to be working.

[ipokkel]

Moderators only: https://www.paidmembershipspro.com/forums/topic/emails-dont-send-when-admin-cancels-membership/

The sendAdminChangeAdminEmail and sendAdminChangeEmail parts of the profile.php are commented out here.

pmpro-multiple-memberships-per-user/includes/profile.php

Lines 360 to 386 in dbba958

	// $subscription_id = -1;
	// foreach($old_levels as $checklevel) {
	// if($checklevel->id == $leveltodel) {
	// $subscription_id = $checklevel->subscription_id;
	// break;
	// }
	// }
	// $wpdb->query("UPDATE $wpdb->pmpro_memberships_users SET status ='admin_cancelled', enddate ='".current_time('mysql')."' WHERE id=$subscription_id");
	// if(is_array($_REQUEST['cancel_subscription']) && array_key_exists($arraykey, $_REQUEST['cancel_subscription']) && !empty($_REQUEST['cancel_subscription'][$arraykey])) {
	// $other_order_ids = $wpdb->get_col("SELECT id FROM $wpdb->pmpro_membership_orders WHERE user_id = '" . $user_id . "' AND status = 'success' AND membership_id = $leveltodel ORDER BY id DESC");
	//
	// foreach($other_order_ids as $order_id)
	// {
	// $c_order = new MemberOrder($order_id);
	// $c_order->cancel();
	// }
	// }
	// //email to admin
	// $pmproemail = new PMProEmail();
	// $pmproemail->sendAdminChangeAdminEmail(get_userdata($user_id));
	//
	// //send email
	// if(is_array($_REQUEST['send_admin_change_email']) && array_key_exists($arraykey, $_REQUEST['send_admin_change_email']) && !empty($_REQUEST['send_admin_change_email'][$arraykey])) {
	// //email to member
	// $pmproemail = new PMProEmail();
	// $pmproemail->sendAdminChangeEmail(get_userdata($user_id));
	// }

[andrewlimaza]

I think this gets disabled as the function needs to support MMPU, which it currently doesn't do.