Skip to content

Commit 5981d7a

Browse files
authored
Merge pull request #3294 from derekashauer/test-email-variables-3166
Test email variables 3166
2 parents 30ef202 + bb02ef0 commit 5981d7a

8 files changed

+25
-7
lines changed

classes/class.pmproemail.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -502,7 +502,7 @@ function sendCheckoutAdminEmail($user = NULL, $order = NULL)
502502
case 'checkout_free_admin':
503503
$email = new PMPro_Email_Template_Checkout_Free_Admin( $user, $order );
504504
break;
505-
case 'checkout_paid':
505+
case 'checkout_paid_admin':
506506
$email = new PMPro_Email_Template_Checkout_Paid_Admin( $user, $order );
507507
break;
508508
}

classes/email-templates/class-pmpro-email-template-checkout-check-admin.php

+3
Original file line numberDiff line numberDiff line change
@@ -138,6 +138,9 @@ public function get_email_template_variables() {
138138
$order = $this->order;
139139
$user = $this->user;
140140
$membership_level = pmpro_getSpecificMembershipLevelForUser( $user->ID, $order->membership_id );
141+
if ( empty( $membership_level ) ) {
142+
$membership_level = pmpro_getLevel( $order->membership_id );
143+
}
141144

142145
$confirmation_in_email = get_pmpro_membership_level_meta( $membership_level->id, 'confirmation_in_email', true );
143146
if ( ! empty( $confirmation_in_email ) ) {

classes/email-templates/class-pmpro-email-template-checkout-check.php

+4
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,10 @@ public function get_email_template_variables() {
173173
$order = $this->order;
174174
$user = $this->user;
175175
$membership_level = pmpro_getSpecificMembershipLevelForUser( $user->ID, $order->membership_id );
176+
if ( empty( $membership_level ) ) {
177+
$membership_level = pmpro_getLevel( $order->membership_id );
178+
}
179+
176180
$discount_code = "";
177181
if( $order->getDiscountCode() ) {
178182
$discount_code = "<p>" . __("Discount Code", 'paid-memberships-pro' ) . ": " . $order->discount_code->code . "</p>\n";

classes/email-templates/class-pmpro-email-template-checkout-free-admin.php

+3
Original file line numberDiff line numberDiff line change
@@ -156,6 +156,9 @@ public function get_email_template_variables() {
156156
$order = $this->order;
157157
$user = $this->user;
158158
$membership_level = pmpro_getSpecificMembershipLevelForUser( $user->ID, $order->membership_id );
159+
if ( empty( $membership_level ) ) {
160+
$membership_level = pmpro_getLevel( $order->membership_id );
161+
}
159162

160163
$confirmation_message = '';
161164
$confirmation_in_email = get_pmpro_membership_level_meta( $membership_level->id, 'confirmation_in_email', true );

classes/email-templates/class-pmpro-email-template-checkout-free.php

+3
Original file line numberDiff line numberDiff line change
@@ -153,6 +153,9 @@ public function get_email_template_variables() {
153153
$order = $this->order;
154154
$user = $this->user;
155155
$membership_level = pmpro_getSpecificMembershipLevelForUser( $user->ID, $order->membership_id );
156+
if ( empty( $membership_level ) ) {
157+
$membership_level = pmpro_getLevel( $order->membership_id );
158+
}
156159

157160
$confirmation_message = '';
158161
$confirmation_in_email = get_pmpro_membership_level_meta( $membership_level->id, 'confirmation_in_email', true );

classes/email-templates/class-pmpro-email-template-checkout-paid-admin.php

+3
Original file line numberDiff line numberDiff line change
@@ -144,6 +144,9 @@ public function get_email_template_variables() {
144144
$order = $this->order;
145145
$user = $this->user;
146146
$membership_level = pmpro_getSpecificMembershipLevelForUser( $user->ID, $order->membership_id );
147+
if ( empty( $membership_level ) ) {
148+
$membership_level = pmpro_getLevel( $order->membership_id );
149+
}
147150

148151
$confirmation_in_email = get_pmpro_membership_level_meta( $membership_level->id, 'confirmation_in_email', true );
149152
if ( ! empty( $confirmation_in_email ) ) {

classes/email-templates/class-pmpro-email-template-checkout-paid.php

+3
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,9 @@ public function get_email_template_variables() {
149149
$order = $this->order;
150150
$user = $this->user;
151151
$membership_level = pmpro_getSpecificMembershipLevelForUser( $user->ID, $order->membership_id );
152+
if ( empty( $membership_level ) ) {
153+
$membership_level = pmpro_getLevel( $order->membership_id );
154+
}
152155

153156
$confirmation_message = '';
154157
$confirmation_in_email = get_pmpro_membership_level_meta( $membership_level->id, 'confirmation_in_email', true );

includes/email.php

+5-6
Original file line numberDiff line numberDiff line change
@@ -296,14 +296,13 @@ function pmpro_email_templates_send_test() {
296296

297297
$test_user = $current_user;
298298

299-
300-
//test subscription object
301-
$test_subscription = new PMPro_Subscription( array( 'user_id' => $test_user->ID, 'membership_level_id' => $test_user->membership_level->id, 'next_payment_date' => date( 'Y-m-d', strtotime( '+1 month' ) ) ) );
302-
303299
// Grab the first membership level defined as a "test level" to use
304300
$all_levels = pmpro_getAllLevels( true);
305301
$test_user->membership_level = array_pop( $all_levels );
306302

303+
//test subscription object
304+
$test_subscription = new PMPro_Subscription( array( 'user_id' => $test_user->ID, 'membership_level_id' => $test_user->membership_level->id, 'next_payment_date' => date( 'Y-m-d', strtotime( '+1 month' ) ) ) );
305+
307306
//add notice to email body
308307
add_filter('pmpro_email_body', 'pmpro_email_templates_test_body', 10, 2);
309308

@@ -314,11 +313,11 @@ function pmpro_email_templates_send_test() {
314313
switch($test_email->template) {
315314
case 'cancel':
316315
$send_email = 'sendCancelEmail';
317-
$params = array($test_user);
316+
$params = array($test_user, $test_user->membership_level->id);
318317
break;
319318
case 'cancel_admin':
320319
$send_email = 'sendCancelAdminEmail';
321-
$params = array($current_user, $current_user->membership_level->id);
320+
$params = array($test_user, $test_user->membership_level->id);
322321
break;
323322
case 'cancel_on_next_payment_date':
324323
case 'cancel_on_next_payment_date_admin':

0 commit comments

Comments
 (0)