Give_New_Donor_Register_Email

Complexity

Total Complexity

9

Size/Duplication

Total Lines	173
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	init()	0	33	1
A	get_default_email_message()	0	16	1
A	setup_email_notification()	0	6	1
B	email_preview_header()	0	75	6

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 25 and the first side effect is on line 14.

/**
 * New Donor Register Email
 *
 * @package     Give
 * @subpackage  Classes/Emails
 * @copyright   Copyright (c) 2016, WordImpress
 * @license     https://opensource.org/licenses/gpl-license GNU Public License
 * @since       2.0
 */

// Exit if access directly.
if ( ! defined( 'ABSPATH' ) ) {
	exit;
}

if ( ! class_exists( 'Give_New_Donor_Register_Email' ) ) :

	/**
	 * Give_New_Donor_Register_Email
	 *
	 * @abstract
	 * @since       2.0
	 */
	class Give_New_Donor_Register_Email extends Give_Email_Notification {

		/**
		 * Create a class instance.
		 *
		 * @access  public
		 * @since   2.0
		 */
		public function init() {
			$this->load( array(
				'id'                    => 'new-donor-register',
				'label'                 => __( 'New Donor Register', 'give' ),
				'description'           => __( 'New Donor Register Notification will be sent to recipient(s) when new donor registered.', 'give' ),
				'has_recipient_field'   => true,
				'notification_status'   => 'enabled',
				'has_preview_header'    => true,
				'email_tag_context'     => array( 'donor', 'general' ),
				'form_metabox_setting'  => false,
				'default_email_subject' => sprintf(
					/* translators: %s: site name */
					esc_attr__( '[%s] New User Registration', 'give' ),
					get_bloginfo( 'name' )
				),
				'default_email_massage' => $this->get_default_email_message(),
			) );

			// Setup action hook.
			add_action(
				"give_{$this->config['id']}_email_notification",
				array( $this, 'setup_email_notification' ),
				10,
				2
			);

			add_filter(
				'give_email_preview_header',
				array( $this, 'email_preview_header' ),
				10,
				2
			);
		}

		/**
		 * Get default email message.
		 *
		 * @since  2.0
		 * @access public
		 *
		 * @return string
		 */
		function get_default_email_message() {

It is generally recommended to explicitly declare the visibility for methods.

			$message = esc_attr__( 'New user registration on your site {sitename}:', 'give' ) . "\r\n\r\n";
			$message .= esc_attr__( 'Username: {username}', 'give' ) . "\r\n\r\n";
			$message .= esc_attr__( 'E-mail: {user_email}', 'give' ) . "\r\n";

			/**
			 * Filter the default email message
			 *
			 * @since 2.0
			 */
			return apply_filters(
				"give_{$this->config['id']}_get_default_email_message",
				$message,
				$this
			);
		}


		/**
		 * Send new donor register notifications.
		 *
		 * @since  2.0
		 * @access public
		 *
		 * @param int   $user_id   User ID.
		 * @param array $user_data User Information.
		 *
		 * @return string

Should the return type not be string|null?

		 */
		public function setup_email_notification( $user_id, $user_data ) {
			$this->recipient_email = $user_data['user_email'];
			$this->send_email_notification( array(
				'user_id' => $user_id,
			) );
		}


		/**
		 * email preview header.
		 *
		 * @since  2.0
		 * @access public
		 *
		 * @param string                        $email_preview_header
		 * @param Give_New_Donor_Register_Email $email
		 * @return string

Should the return type not be string|null?

		 */
		public function email_preview_header( $email_preview_header, $email ) {
			// Bailout.
			if ( $this->config['id'] !== $email->config['id'] ) {
				return $email_preview_header;
			}

			// Payment receipt switcher
			$user_id = give_check_variable( give_clean( $_GET ), 'isset', 0, 'user_id' );

			// Get payments.
			$donors  = new Give_API();
			$donors  = give_check_variable( $donors->get_customers(), 'empty', array(), 'donors' );

The method get_customers() does not seem to exist on object<Give_API>.

			$options = array();

			// Default option.
			$options[0] = esc_html__( 'No donor(s) found.', 'give' );

			// Provide nice human readable options.
			if ( $donors ) {
				$options[0] = esc_html__( '- Select a donor -', 'give' );
				foreach ( $donors as $donor ) {
					// Exclude customers for which wp user not exist.
					if ( ! $donor['info']['user_id'] ) {
						continue;
					}
					$options[ $donor['info']['user_id'] ] = esc_html( '#' . $donor['info']['customer_id'] . ' - ' . $donor['info']['email'] );
				}
			}

			// Start constructing HTML output.
			$email_preview_header = '<div style="margin:0;padding:10px 0;width:100%;background-color:#FFF;border-bottom:1px solid #eee; text-align:center;">';

			// Inline JS function for switching donations.
			$request_url = $_SERVER['REQUEST_URI'];

			// Remove payment id query param if set from request url.
			if ( $user_id ) {
				$request_url_data = wp_parse_url( $_SERVER['REQUEST_URI'] );
				$query            = $request_url_data['query'];
				$query            = str_replace( "&user_id={$user_id}", '', $query );

				$request_url = home_url( '/?' . str_replace( '', '', $query ) );
			}

			$email_preview_header .= '<script>
				 function change_preview(){
				  var transactions = document.getElementById("give_preview_email_user_id");
			        var selected_trans = transactions.options[transactions.selectedIndex];
				        if (selected_trans){
				            var url_string = "' . $request_url . '&user_id=" + selected_trans.value;
				                window.location = url_string;
				        }
				    }
			    </script>';

			$email_preview_header .= '<label for="give_preview_email_user_id" style="font-size:12px;color:#333;margin:0 4px 0 0;">' . esc_html__( 'Preview email with a donor:', 'give' ) . '</label>';

			// The select field with 100 latest transactions
			$email_preview_header .= Give()->html->select( array(
				'name'             => 'preview_email_user_id',
				'selected'         => $user_id,
				'id'               => 'give_preview_email_user_id',
				'class'            => 'give-preview-email-donor-id',
				'options'          => $options,
				'chosen'           => false,
				'select_atts'      => 'onchange="change_preview()"',
				'show_option_all'  => false,
				'show_option_none' => false,
			) );

			// Closing tag
			$email_preview_header .= '</div>';

			echo $email_preview_header;
		}
	}

endif; // End class_exists check

return Give_New_Donor_Register_Email::get_instance();