woothemes / sensei

includes/emails/class-woothemes-sensei-email-teacher-new-message.php

<?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_Teacher_New_Message' ) ) :

/**
 * Teacher New Message
 *
 * An email sent to the teacher when one of their students sends them a private message.
 *
 * @package Users
 * @author Automattic
 *
 * @since		1.6.0
 */
class WooThemes_Sensei_Email_Teacher_New_Message {

	var $template;

The visibility should be declared for property $template.

	var $subject;

The visibility should be declared for property $subject.

	var $heading;

The visibility should be declared for property $heading.

	var $recipient;

The visibility should be declared for property $recipient.

	var $learner;

The visibility should be declared for property $learner.

	var $teacher;

The visibility should be declared for property $teacher.

	var $message;

The visibility should be declared for property $message.

	/**
	 * Constructor
	 */
	function __construct() {

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

		$this->template = 'teacher-new-message';
		$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] You have received a new private message', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
		$this->heading = apply_filters( 'sensei_email_heading', __( 'Your student has sent you a private message', 'woothemes-sensei' ), $this->template );
	}

	/**
	 * trigger function.
	 *
	 * @access public
     * @param integer $message_id
	 * @return void
	 */
	function trigger( $message_id = 0 ) {

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

		global  $sensei_email_data;

		$this->message = get_post( $message_id );

		$learner_username = get_post_meta( $message_id, '_sender', true );
		$this->learner = get_user_by( 'login', $learner_username );

		$teacher_username = get_post_meta( $message_id, '_receiver', true );
		$this->teacher = get_user_by( 'login', $teacher_username );

		$content_type = get_post_meta( $message_id, '_posttype', true );

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

		$content_id = get_post_meta( $message_id, '_post', true );
		$content_title = get_the_title( $content_id );

        // 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,
			'teacher_id'		=> $this->teacher->ID,
			'learner_id'		=> $this->learner->ID,
			'learner_name'		=> $this->learner->display_name,
			'message_id'		=> $message_id,
			'message'			=> $this->message->post_content,
			'content_title'		=> $content_title,
			'content_type'		=> $content_type,
		), $this->template );

		// Set recipient (teacher)
		$this->recipient = stripslashes( $this->teacher->user_email );

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

		wp_safe_redirect( esc_url_raw( add_query_arg( array( 'send' => 'complete' ) ) ) );
		exit;
	}
}

endif;

return new WooThemes_Sensei_Email_Teacher_New_Message();