WooThemes_Sensei_Email_Learner_Graded_Quiz

Complexity

Total Complexity

4

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	trigger()	0	43	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_Learner_Graded_Quiz' ) ) :

/**
 * Learner Graded Quiz
 *
 * An email sent to the learner when their quiz has been graded (auto or manual).
 *
 * @package Users
 * @author Automattic
 *
 * @since		1.6.0
 */
class WooThemes_Sensei_Email_Learner_Graded_Quiz {

	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 $user;

The visibility should be declared for property $user.

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

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

		$this->template = 'learner-graded-quiz';
		$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] Your quiz has been graded', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
		$this->heading = apply_filters( 'sensei_email_heading', __( 'Your quiz has been graded', 'woothemes-sensei' ), $this->template );
	}

	/**
	 * trigger function.
	 *
     * @param int $user_id
     * @param int $quiz_id
     * @param int $grade
     * @param int $passmark
     *
	 * @return void
	 */
	function trigger ( $user_id = 0, $quiz_id = 0, $grade = 0, $passmark = 0 ) {

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

		global  $sensei_email_data;

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

		// Get passed flag
		$passed = __( 'failed', 'woothemes-sensei' );
		if( $grade >= $passmark ) {
			$passed = __( 'passed', 'woothemes-sensei' );
		}

		// Get grade tye (auto/manual)
		$grade_type = get_post_meta( $quiz_id, '_quiz_grade_type', true );

		if( 'auto' == $grade_type ) {
			$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] You have completed a quiz', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
			$this->heading = apply_filters( 'sensei_email_heading', __( 'You have completed a quiz', 'woothemes-sensei' ), $this->template );
		}

		$lesson_id = get_post_meta( $quiz_id, '_quiz_lesson', true );

		// Construct data array
		$sensei_email_data = apply_filters( 'sensei_email_data', array(
			'template'			=> $this->template,
			'heading'			=> $this->heading,
			'user_id'			=> $user_id,
			'user_name'         => stripslashes( $this->user->display_name ),
			'lesson_id'			=> $lesson_id,
			'quiz_id'			=> $quiz_id,
			'grade'				=> $grade,
			'passmark'			=> $passmark,
			'passed'			=> $passed,
			'grade_type'		=> $grade_type,
		), $this->template );

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

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

endif;

return new WooThemes_Sensei_Email_Learner_Graded_Quiz();