WooThemes_Sensei_Email_Teacher_Completed_Course

Complexity

Total Complexity

3

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	trigger()	0	33	2

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

/**
 * Teacher Completed Course
 *
 * An email sent to the teacher when one of their students completes a course.
 *
 * @package Users
 * @author Automattic
 *
 * @since		1.6.0
 */
class WooThemes_Sensei_Email_Teacher_Completed_Course {

	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.

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

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

		$this->template = 'teacher-completed-course';
		$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] Your student has completed a course', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
		$this->heading = apply_filters( 'sensei_email_heading', __( 'Your student has completed a course', 'woothemes-sensei' ), $this->template );
	}

	/**
	 * trigger function.
	 *
     * @param int $learner_id
     * @param int $course_id
     *
	 * @return void
	 */
	function trigger( $learner_id = 0, $course_id = 0 ) {

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

		global  $sensei_email_data;

		// Get learner user object
		$this->learner = new WP_User( $learner_id );

		// Get teacher ID and user object
		$teacher_id = get_post_field( 'post_author', $course_id, 'raw' );
		$this->teacher = new WP_User( $teacher_id );

		// Get passed status
		$passed = __( 'passed', 'woothemes-sensei' );
		if( ! Sensei_Utils::sensei_user_passed_course( $course_id, $learner_id ) ) {
			$passed = __( 'failed', 'woothemes-sensei' );
		}

		// Construct data array
		$sensei_email_data = apply_filters( 'sensei_email_data', array(
			'template'			=> $this->template,
			'heading'			=> $this->heading,
			'teacher_id'		=> $teacher_id,
			'learner_id'		=> $learner_id,
			'learner_name'		=> $this->learner->display_name,
			'course_id'			=> $course_id,
			'passed'			=> $passed,
		), $this->template );

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

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

endif;

return new WooThemes_Sensei_Email_Teacher_Completed_Course();