WooThemes_Sensei_Email_New_Message_Reply

Complexity

Total Complexity

4

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	trigger()	0	49	3

<?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 17 and the first side effect is on line 3.

if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly

if ( ! class_exists( 'WooThemes_Sensei_Email_New_Message_Reply' ) ) :

/**
 * Teacher New Message
 *
 * An email sent to the a user when they receive a reply to the private message.
 *
 * @package Users
 * @author Automattic
 *
 * @since		1.6.0
 */
class WooThemes_Sensei_Email_New_Message_Reply {

    /**
     * @var string
     */
    var $template;

The visibility should be declared for property $template.

    /**
     * @var string
     */
	var $subject;

The visibility should be declared for property $subject.

    /**
     * @var string
     */
    var $heading;

The visibility should be declared for property $heading.

    /**
     * @var string
     */
	var $recipient;

The visibility should be declared for property $recipient.

	var $original_sender;

The visibility should be declared for property $original_sender.

	var $original_receiver;

The visibility should be declared for property $original_receiver.

	var $commenter;

The visibility should be declared for property $commenter.

    /**
     * @var WP_Post
     */
	var $message;

The visibility should be declared for property $message.

    /**
     * @var WP_Comment
     */
	var $comment;

The visibility should be declared for property $comment.

	/**
	 * Constructor
	 *
	 * @access public
	 */
	function __construct() {

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

		$this->template = 'new-message-reply';
		$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] You have a new message', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
		$this->heading = apply_filters( 'sensei_email_heading', __( 'You have received a reply to your private message', 'woothemes-sensei' ), $this->template );
	}

	/**
	 * trigger function.
	 *
     * @param WP_Comment $comment
     * @param string $message
     *
	 * @return void
	 */
	function trigger ( $comment, $message ) {

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

		global  $sensei_email_data;

		$this->comment = $comment;
		$this->message = $message;

It seems like $message of type string is incompatible with the declared type object<WP_Post> of property $message.

		$this->commenter = get_userdata( $comment->user_id );

		$original_sender = get_post_meta( $this->message->ID, '_sender', true );
		$this->original_sender = get_user_by( 'login', $original_sender );

		$original_receiver = get_post_meta( $this->message->ID, '_receiver', true );
		$this->original_receiver = get_user_by( 'login', $original_receiver );

		$content_type = get_post_meta( $this->message->ID, '_posttype', true );

$content_type is not used, you could remove the assignment.

		$content_id = get_post_meta( $this->message->ID, '_post', true );
		$content_title = get_the_title( $content_id );

		$comment_link = get_comment_link( $comment );

        // setup the post type parameter
        $content_type = get_post_type( $content_id );
        if( !$content_type ){
            $content_type ='';
        }

        // Construct data array
        $sensei_email_data = apply_filters( 'sensei_email_data', array(
            'template'			=> $this->template,
            $content_type.'_id' => $content_id,
			'heading'			=> $this->heading,
			'commenter_name'	=> $this->commenter->display_name,
			'message'			=> $this->comment->comment_content,
			'comment_link'		=> $comment_link,
			'content_title'		=> $content_title,
			'content_type'		=> $content_type,
		), $this->template );

		// Set recipient
		if( $this->commenter->user_login == $original_sender ) {
			$this->recipient = stripslashes( $this->original_receiver->user_email );
		} else {
			$this->recipient = stripslashes( $this->original_sender->user_email );
		}

		// Send mail
		Sensei()->emails->send( $this->recipient, $this->subject, Sensei()->emails->get_content( $this->template ) );
	}
}

endif;

return new WooThemes_Sensei_Email_New_Message_Reply();