WC_Stripe_Email_Failed_Preorder_Authentication

Complexity

Total Complexity

11

Size/Duplication

Total Lines	94
Duplicated Lines	21.28 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	20	20	2
A	trigger()	0	9	4
A	send_email()	0	10	2
A	get_default_subject()	0	3	1
A	get_default_heading()	0	3	1
A	get_custom_message()	0	3	1

<?php
if ( ! defined( 'ABSPATH' ) ) {
	exit; // Exit if accessed directly.
}

/**
 * Failed Renewal/Pre-Order Authentication Notification
 *
 * @extends WC_Stripe_Email_Failed_Authentication
 */
class WC_Stripe_Email_Failed_Preorder_Authentication extends WC_Stripe_Email_Failed_Authentication {
	/**
	 * Holds the message, which is entered by admins when sending the email.
	 *
	 * @var string
	 */
	protected $custom_message;

	/**
	 * Constructor.
	 *
	 * @param WC_Email[] $email_classes All existing instances of WooCommerce emails.
	 */
	public function __construct( $email_classes = array() ) {

This method seems to be duplicated in your project.

		$this->id             = 'failed_preorder_sca_authentication';
		$this->title          = __( 'Pre-order Payment Action Needed', 'woocommerce-gateway-stripe' );
		$this->description    = __( 'This is an order notification sent to the customer once a pre-order is complete, but additional payment steps are required.', 'woocommerce-gateway-stripe' );
		$this->customer_email = true;

		$this->template_html  = 'emails/failed-preorder-authentication.php';
		$this->template_plain = 'emails/plain/failed-preorder-authentication.php';
		$this->template_base  = plugin_dir_path( WC_STRIPE_MAIN_FILE ) . 'templates/';

		// Use the "authentication required" hook to add the correct, later hook.
		add_action( 'wc_gateway_stripe_process_payment_authentication_required', array( $this, 'trigger' ) );

		if ( isset( $email_classes['WC_Pre_Orders_Email_Pre_Order_Available'] ) ) {
			$this->original_email = $email_classes['WC_Pre_Orders_Email_Pre_Order_Available'];
		}

		// We want all the parent's methods, with none of its properties, so call its parent's constructor, rather than my parent constructor.
		parent::__construct();
	}

	/**
	 * When autnentication is required, this adds another action to `wc_pre_orders_pre_order_completed`
	 * in order to send the authentication required email when the custom pre-orders message is available.
	 *
	 * @param WC_Order $order The order whose payment is failing.
	 */
	public function trigger( $order ) {
		if ( class_exists( 'WC_Pre_Orders_Order' ) && WC_Pre_Orders_Order::order_contains_pre_order( $order->get_id() ) ) {
			if ( isset( $this->original_email ) ) {
				remove_action( 'wc_pre_order_status_completed_notification', array( $this->original_email, 'trigger' ), 10, 2 );
			}

			add_action( 'wc_pre_orders_pre_order_completed', array( $this, 'send_email' ), 10, 2 );
		}
	}

	/**
	 * Triggers the email while also disconnecting the original Pre-Orders email.
	 *
	 * @param WC_Order $order The order that is being paid.
	 * @param string   $message The message, which should be added to the email.
	 */
	public function send_email( $order, $message ) {
		$this->custom_message = $message;

		parent::trigger( $order );

It seems like you call parent on a different method (trigger() instead of send_email()). Are you sure this is correct? If so, you might want to change this to $this->trigger().

		// Restore the action of the original email for other bulk actions.
		if ( isset( $this->original_email ) ) {
			add_action( 'wc_pre_order_status_completed_notification', array( $this->original_email, 'trigger' ), 10, 2 );
		}
	}

	/**
	 * Returns the default subject of the email (modifyable in settings).
	 *
	 * @return string
	 */
	public function get_default_subject() {
		return __( 'Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe' );
	}

	/**
	 * Returns the default heading of the email (modifyable in settings).
	 *
	 * @return string
	 */
	public function get_default_heading() {
		return __( 'Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe' );
	}

	/**
	 * Returns the custom message, entered by the admin.
	 *
	 * @return string
	 */
	public function get_custom_message() {
		return $this->custom_message;
	}
}