woothemes / sensei

apigen/hook-docs.php

Doc Comments +3 added lines

@@ -10,6 +10,9 @@
 	private static $found_files            = array();
 	private static $custom_hooks_found     = '';
 
+	/**
+	 * @param string $pattern
+	 */
 	private static function get_files( $pattern, $flags = 0, $path = '' ) {
 
 	    if ( ! $path && ( $dir = dirname( $pattern ) ) != '.' ) {

Spacing +60 added lines, -60 removed lines

@@ -10,32 +10,32 @@ 
 	private static $found_files            = array();
 	private static $custom_hooks_found     = '';
 
-	private static function get_files( $pattern, $flags = 0, $path = '' ) {
+	private static function get_files($pattern, $flags = 0, $path = '') {
 
-	    if ( ! $path && ( $dir = dirname( $pattern ) ) != '.' ) {
+	    if ( ! $path && ($dir = dirname($pattern)) != '.') {
 
 	        if ($dir == '\\' || $dir == '/') { $dir = ''; } // End IF Statement
 
-	        return self::get_files(basename( $pattern ), $flags, $dir . '/' );
+	        return self::get_files(basename($pattern), $flags, $dir.'/');
 
 	    } // End IF Statement
 
-	    $paths = glob( $path . '*', GLOB_ONLYDIR | GLOB_NOSORT );
-	    $files = glob( $path . $pattern, $flags );
+	    $paths = glob($path.'*', GLOB_ONLYDIR | GLOB_NOSORT);
+	    $files = glob($path.$pattern, $flags);
 
-	    if ( is_array( $paths ) ) {
-		    foreach ( $paths as $p ) {
+	    if (is_array($paths)) {
+		    foreach ($paths as $p) {
 			    $found_files = array();
-		   		$retrieved_files = (array) self::get_files( $pattern, $flags, $p . '/' );
-		   		foreach ( $retrieved_files as $file ) {
-			   		if ( ! in_array( $file, self::$found_files ) )
+		   		$retrieved_files = (array) self::get_files($pattern, $flags, $p.'/');
+		   		foreach ($retrieved_files as $file) {
+			   		if ( ! in_array($file, self::$found_files))
 			   			$found_files[] = $file;
 		   		}
 
-		   		self::$found_files = array_merge( self::$found_files, $found_files );
+		   		self::$found_files = array_merge(self::$found_files, $found_files);
 
-		   		if ( is_array( $files ) && is_array( $found_files ) ) {
-		   			$files = array_merge( $files, $found_files );
+		   		if (is_array($files) && is_array($found_files)) {
+		   			$files = array_merge($files, $found_files);
 		   		}
 
 		    } // End FOREACH Loop
@@ -43,29 +43,29 @@ 
 	    return $files;
     }
 
-	private static function get_hook_link( $hook, $details = array() ) {
-		if ( ! empty( $details['class'] ) ) {
-			$link = 'http://docs.woothemes.com/wc-apidocs/source-class-' . $details['class'] . '.html#' . $details['line'];
-		} elseif ( ! empty( $details['function'] ) ) {
-			$link = 'http://docs.woothemes.com/wc-apidocs/source-function-' . $details['function'] . '.html#' . $details['line'];
+	private static function get_hook_link($hook, $details = array()) {
+		if ( ! empty($details['class'])) {
+			$link = 'http://docs.woothemes.com/wc-apidocs/source-class-'.$details['class'].'.html#'.$details['line'];
+		} elseif ( ! empty($details['function'])) {
+			$link = 'http://docs.woothemes.com/wc-apidocs/source-function-'.$details['function'].'.html#'.$details['line'];
 		} else {
-			$link = 'https://github.com/woothemes/woocommerce/search?utf8=%E2%9C%93&q=' . $hook;
+			$link = 'https://github.com/woothemes/woocommerce/search?utf8=%E2%9C%93&q='.$hook;
 		}
 
-		return '<a href="' . $link . '">' . $hook . '</a>';
+		return '<a href="'.$link.'">'.$hook.'</a>';
 	}
 
 	public static function process_hooks() {
 		// If we have one, get the PHP files from it.
-		$template_files 	= self::get_files( '*.php', GLOB_MARK, '../templates/' );
+		$template_files = self::get_files('*.php', GLOB_MARK, '../templates/');
 		$template_files[]	= '../includes/wc-template-functions.php';
 		$template_files[]	= '../includes/wc-template-hooks.php';
 
-		$shortcode_files 	= self::get_files( '*.php', GLOB_MARK, '../includes/shortcodes/' );
-		$widget_files	 	= self::get_files( '*.php', GLOB_MARK, '../includes/widgets/' );
-		$admin_files 		= self::get_files( '*.php', GLOB_MARK, '../includes/admin/' );
-		$class_files 		= self::get_files( '*.php', GLOB_MARK, '../includes/' );
-		$other_files		= array(
+		$shortcode_files 	= self::get_files('*.php', GLOB_MARK, '../includes/shortcodes/');
+		$widget_files = self::get_files('*.php', GLOB_MARK, '../includes/widgets/');
+		$admin_files 		= self::get_files('*.php', GLOB_MARK, '../includes/admin/');
+		$class_files 		= self::get_files('*.php', GLOB_MARK, '../includes/');
+		$other_files = array(
 			'../woocommerce.php'
 		);
 
@@ -86,34 +86,34 @@ 
 		echo '<h1>Action and Filter Hook Reference</h1>';
 		echo '<div class="description"><p>The following is a full list of actions and filters found in WooCommerce core.</p></div>';
 
-		foreach ( self::$files_to_scan as $heading => $files ) {
+		foreach (self::$files_to_scan as $heading => $files) {
 			self::$custom_hooks_found = array();
 
-			foreach ( $files as $f ) {
-				self::$current_file = basename( $f );
-				$tokens             = token_get_all( file_get_contents( $f ) );
+			foreach ($files as $f) {
+				self::$current_file = basename($f);
+				$tokens             = token_get_all(file_get_contents($f));
 				$token_type         = false;
 				$current_class      = '';
 				$current_function   = '';
 
-				if ( in_array( self::$current_file, $scanned ) ) {
+				if (in_array(self::$current_file, $scanned)) {
 					continue;
 				}
 
 				$scanned[] = self::$current_file;
 
-				foreach ( $tokens as $index => $token ) {
-					if ( is_array( $token ) ) {
-						if ( $token[0] == T_CLASS ) {
+				foreach ($tokens as $index => $token) {
+					if (is_array($token)) {
+						if ($token[0] == T_CLASS) {
 							$token_type = 'class';
-						} elseif ( $token[0] == T_FUNCTION ) {
+						} elseif ($token[0] == T_FUNCTION) {
 							$token_type = 'function';
-						} elseif ( $token[1] === 'do_action' ) {
+						} elseif ($token[1] === 'do_action') {
 							$token_type = 'action';
-						} elseif ( $token[1] === 'apply_filters' ) {
+						} elseif ($token[1] === 'apply_filters') {
 							$token_type = 'filter';
-						} elseif ( $token_type && ! empty( trim( $token[1] ) ) ) {
-							switch ( $token_type ) {
+						} elseif ($token_type && ! empty(trim($token[1]))) {
+							switch ($token_type) {
 								case 'class' :
 									$current_class = $token[1];
 								break;
@@ -122,15 +122,15 @@ 
 								break;
 								case 'filter' :
 								case 'action' :
-									$hook = trim( $token[1], "'" );
-									if ( isset( self::$custom_hooks_found[ $hook ] ) ) {
-										self::$custom_hooks_found[ $hook ]['file'][] = self::$current_file;
+									$hook = trim($token[1], "'");
+									if (isset(self::$custom_hooks_found[$hook])) {
+										self::$custom_hooks_found[$hook]['file'][] = self::$current_file;
 									} else {
-    									self::$custom_hooks_found[ $hook ] = array(
+    									self::$custom_hooks_found[$hook] = array(
 											'line'     => $token[2],
 											'class'    => $current_class,
 											'function' => $current_function,
-											'file'     => array( self::$current_file ),
+											'file'     => array(self::$current_file),
 											'type'     => $token_type
 										);
 									}
@@ -142,24 +142,24 @@ 
 				}
 			}
 
-			foreach ( self::$custom_hooks_found as $hook => $details ) {
-				if ( ! strstr( $hook, 'woocommerce' ) && ! strstr( $hook, 'product' ) && ! strstr( $hook, 'wc_' ) ) {
-					unset( self::$custom_hooks_found[ $hook ] );
+			foreach (self::$custom_hooks_found as $hook => $details) {
+				if ( ! strstr($hook, 'woocommerce') && ! strstr($hook, 'product') && ! strstr($hook, 'wc_')) {
+					unset(self::$custom_hooks_found[$hook]);
 				}
 			}
 
-			ksort( self::$custom_hooks_found );
+			ksort(self::$custom_hooks_found);
 
-			if ( ! empty( self::$custom_hooks_found ) ) {
-				echo '<h2>' . $heading . '</h2>';
+			if ( ! empty(self::$custom_hooks_found)) {
+				echo '<h2>'.$heading.'</h2>';
 
 				echo '<table class="summary"><thead><tr><th>Hook</th><th>Type</th><th>File(s)</th></tr></thead><tbody>';
 
-				foreach ( self::$custom_hooks_found as $hook => $details ) {
+				foreach (self::$custom_hooks_found as $hook => $details) {
 					echo '<tr>
-						<td>' . self::get_hook_link( $hook, $details ) . '</td>
-						<td>' . $details['type'] . '</td>
-						<td>' . implode( ', ', array_unique( $details['file'] ) ) . '</td>
+						<td>' . self::get_hook_link($hook, $details).'</td>
+						<td>' . $details['type'].'</td>
+						<td>' . implode(', ', array_unique($details['file'])).'</td>
 					</tr>' . "\n";
 				}
 
@@ -169,13 +169,13 @@ 
 
 		echo '</div><div id="footer">';
 
-		$html   = file_get_contents( '../wc-apidocs/todo.html' );
-		$header = current( explode( '<div id="content">', $html ) );
-		$header = str_replace( '<li class="active">', '<li>', $header );
-		$header = str_replace( '<li class="hooks">', '<li class="active">', $header );
-		$footer = end( explode( '<div id="footer">', $html ) );
+		$html   = file_get_contents('../wc-apidocs/todo.html');
+		$header = current(explode('<div id="content">', $html));
+		$header = str_replace('<li class="active">', '<li>', $header);
+		$header = str_replace('<li class="hooks">', '<li class="active">', $header);
+		$footer = end(explode('<div id="footer">', $html));
 
-		file_put_contents( '../wc-apidocs/hook-docs.html', $header . ob_get_clean() . $footer );
+		file_put_contents('../wc-apidocs/hook-docs.html', $header.ob_get_clean().$footer);
 		echo "Hook docs generated :)\n";
 	}
 }

includes/class-sensei-analysis.php

Doc Comments +8 added lines, -1 removed lines

@@ -21,6 +21,7 @@ 
 	/**
 	 * Constructor
 	 * @since  1.0.0
+	 * @param string $file
 	 * @return  void
 	 */
 	public function __construct ( $file ) {
@@ -215,6 +216,7 @@ 
 	/**
 	 * An individual users' profile view for analysis, showing their Courses
 	 * @since  1.2.0
+	 * @param integer $user_id
 	 * @return void
 	 */
 	public function analysis_user_profile_view( $user_id ) {
@@ -242,6 +244,7 @@ 
 	/**
 	 * An individual Course view for analysis, showing the Courses Lessons
 	 * @since  1.2.0
+	 * @param integer $course_id
 	 * @return void
 	 */
 	public function analysis_course_view( $course_id ) {
@@ -269,6 +272,8 @@ 
 	/**
 	 * An individual Course view for analysis, showing a specific Learners Lessons
 	 * @since  1.2.0
+	 * @param integer $course_id
+	 * @param integer $user_id
 	 * @return void
 	 */
 	public function analysis_user_course_view( $course_id, $user_id ) {
@@ -296,6 +301,7 @@ 
 	/**
 	 * An individual Course view for analysis, showing all the Learners
 	 * @since  1.2.0
+	 * @param integer $course_id
 	 * @return void
 	 */
 	public function analysis_course_users_view( $course_id ) {
@@ -323,6 +329,7 @@ 
 	/**
 	 * An individual Lesson view for analysis, showing all the Learners
 	 * @since  1.2.0
+	 * @param integer $lesson_id
 	 * @return void
 	 */
 	public function analysis_lesson_users_view( $lesson_id ) {
@@ -633,7 +640,7 @@ 
 	/**
 	 * Write array data to CSV
 	 * @since  1.2.0
-	 * @param  array  $report_array data array
+	 * @param  array  $report_data data array
 	 * @return void
 	 */
 	public function report_write_download( $report_data = array() ) {

Spacing +185 added lines, -185 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Analysis Class
@@ -23,23 +23,23 @@ 
 	 * @since  1.0.0
 	 * @return  void
 	 */
-	public function __construct ( $file ) {
+	public function __construct($file) {
 		$this->name = __('Analysis', 'woothemes-sensei');
 		$this->file = $file;
 		$this->page_slug = 'sensei_analysis';
 
 		// Admin functions
-		if ( is_admin() ) {
-			add_action( 'admin_menu', array( $this, 'analysis_admin_menu' ), 10);
-			add_action( 'analysis_wrapper_container', array( $this, 'wrapper_container'  ) );
-			if ( isset( $_GET['page'] ) && ( $_GET['page'] == $this->page_slug ) ) {
-				add_action( 'admin_print_scripts', array( $this, 'enqueue_scripts' ) );
-				add_action( 'admin_print_styles', array( $this, 'enqueue_styles' ) );
+		if (is_admin()) {
+			add_action('admin_menu', array($this, 'analysis_admin_menu'), 10);
+			add_action('analysis_wrapper_container', array($this, 'wrapper_container'));
+			if (isset($_GET['page']) && ($_GET['page'] == $this->page_slug)) {
+				add_action('admin_print_scripts', array($this, 'enqueue_scripts'));
+				add_action('admin_print_styles', array($this, 'enqueue_styles'));
 			}
 
-			add_action( 'admin_init', array( $this, 'report_download_page' ) );
+			add_action('admin_init', array($this, 'report_download_page'));
 
-			add_filter( 'user_search_columns', array( $this, 'user_search_columns_filter' ), 10, 3 );
+			add_filter('user_search_columns', array($this, 'user_search_columns_filter'), 10, 3);
 		} // End If Statement
 	} // End __construct()
 
@@ -53,9 +53,9 @@ 
 	public function analysis_admin_menu() {
 		global $menu, $woocommerce;
 
-		if ( current_user_can( 'manage_sensei_grades' ) ) {
+		if (current_user_can('manage_sensei_grades')) {
 
-            add_submenu_page( 'sensei', __('Analysis', 'woothemes-sensei'),  __('Analysis', 'woothemes-sensei') , 'manage_sensei_grades', 'sensei_analysis', array( $this, 'analysis_page' ) );
+            add_submenu_page('sensei', __('Analysis', 'woothemes-sensei'), __('Analysis', 'woothemes-sensei'), 'manage_sensei_grades', 'sensei_analysis', array($this, 'analysis_page'));
 
 		}
 
@@ -69,7 +69,7 @@ 
 	 * @since 1.0.0
 	 * @return void
 	 */
-	public function enqueue_scripts () {
+	public function enqueue_scripts() {
 		// None for now
 
 	} // End enqueue_scripts()
@@ -82,11 +82,11 @@ 
 	 * @since 1.0.0
 	 * @return void
 	 */
-	public function enqueue_styles () {
+	public function enqueue_styles() {
 
-		wp_enqueue_style( Sensei()->token . '-admin' );
+		wp_enqueue_style(Sensei()->token.'-admin');
 
-		wp_enqueue_style( 'woothemes-sensei-settings-api', Sensei()->plugin_url . 'assets/css/settings.css', '', Sensei()->version );
+		wp_enqueue_style('woothemes-sensei-settings-api', Sensei()->plugin_url.'assets/css/settings.css', '', Sensei()->version);
 
 	} // End enqueue_styles()
 
@@ -98,14 +98,14 @@ 
 	public function load_data_table_files() {
 
 		// Load Analysis Classes
-		$classes_to_load = array(	'list-table',
+		$classes_to_load = array('list-table',
 									'analysis-overview',
 									'analysis-user-profile',
 									'analysis-course',
 									'analysis-lesson'
 									);
-		foreach ( $classes_to_load as $class_file ) {
-            Sensei()->load_class( $class_file );
+		foreach ($classes_to_load as $class_file) {
+            Sensei()->load_class($class_file);
 		} // End For Loop
 	} // End load_data_table_files()
 
@@ -116,13 +116,13 @@ 
 	 * @param  undefined  $optional_data optional constructor arguments
 	 * @return object                 class instance object
 	 */
-	public function load_data_object( $name = '', $data = 0, $optional_data = null ) {
+	public function load_data_object($name = '', $data = 0, $optional_data = null) {
 		// Load Analysis data
-		$object_name = 'WooThemes_Sensei_Analysis_' . $name . '_List_Table';
-		if ( is_null($optional_data) ) {
-			$sensei_analysis_object = new $object_name( $data );
+		$object_name = 'WooThemes_Sensei_Analysis_'.$name.'_List_Table';
+		if (is_null($optional_data)) {
+			$sensei_analysis_object = new $object_name($data);
 		} else {
-			$sensei_analysis_object = new $object_name( $data, $optional_data );
+			$sensei_analysis_object = new $object_name($data, $optional_data);
 		}
 		$sensei_analysis_object->prepare_items();
 		return $sensei_analysis_object;
@@ -139,40 +139,40 @@ 
 		$course_id = 0;
 		$lesson_id = 0;
 		$user_id = 0;
-		if( isset( $_GET['course_id'] ) ) {
-			$course_id = intval( $_GET['course_id'] );
+		if (isset($_GET['course_id'])) {
+			$course_id = intval($_GET['course_id']);
 		}
-		if( isset( $_GET['lesson_id'] ) ) {
-			$lesson_id = intval( $_GET['lesson_id'] );
+		if (isset($_GET['lesson_id'])) {
+			$lesson_id = intval($_GET['lesson_id']);
 		}
-		if( isset( $_GET['user_id'] ) ) {
-			$user_id = intval( $_GET['user_id'] );
+		if (isset($_GET['user_id'])) {
+			$user_id = intval($_GET['user_id']);
 		}
-		$type = isset( $_GET['view'] ) ? esc_html( $_GET['view'] ) : false;
+		$type = isset($_GET['view']) ? esc_html($_GET['view']) : false;
 
-		if ( 0 < $lesson_id ) {
+		if (0 < $lesson_id) {
 			// Viewing a specific Lesson and all its Learners
-			$this->analysis_lesson_users_view( $lesson_id );
+			$this->analysis_lesson_users_view($lesson_id);
 		}
-		elseif ( 0 < $course_id && !$user_id && 'user' == $type ) {
+		elseif (0 < $course_id && ! $user_id && 'user' == $type) {
 			// Viewing a specific Course and all its Learners
-			$this->analysis_course_users_view( $course_id );
+			$this->analysis_course_users_view($course_id);
 		}
-		elseif ( 0 < $course_id && 0 < $user_id ) {
+		elseif (0 < $course_id && 0 < $user_id) {
 			// Viewing a specific Learner on a specific Course, showing their Lessons
-			$this->analysis_user_course_view( $course_id, $user_id );
+			$this->analysis_user_course_view($course_id, $user_id);
 		}
-		elseif( 0 < $course_id ) {
+		elseif (0 < $course_id) {
 			// Viewing a specific Course and all it's Lessons
-			$this->analysis_course_view( $course_id );
+			$this->analysis_course_view($course_id);
 		}
-		elseif ( 0 < $user_id ) {
+		elseif (0 < $user_id) {
 			// Viewing a specific Learner, and their Courses
-			$this->analysis_user_profile_view( $user_id );
+			$this->analysis_user_profile_view($user_id);
 		}
 		else {
 			// Overview of all Learners, all Courses, or all Lessons
-			$this->analysis_default_view( $type );
+			$this->analysis_default_view($type);
 		} // End If Statement
 	} // End analysis_page()
 
@@ -181,35 +181,35 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function analysis_default_view( $type ) {
+	public function analysis_default_view($type) {
 
 		// Load Analysis data
-		$sensei_analysis_overview = $this->load_data_object( 'Overview', $type );
+		$sensei_analysis_overview = $this->load_data_object('Overview', $type);
 		// Wrappers
-		do_action( 'analysis_before_container' );
-		do_action( 'analysis_wrapper_container', 'top' );
+		do_action('analysis_before_container');
+		do_action('analysis_wrapper_container', 'top');
 		$this->analysis_headers();
 		?>
 		<div id="poststuff" class="sensei-analysis-wrap">
 			<div class="sensei-analysis-sidebar">
 				<?php
-				do_action( 'sensei_analysis_before_stats_boxes' );
-				foreach ( $sensei_analysis_overview->stats_boxes() as $key => $value ) {
-					$this->render_stats_box( esc_html( $key ), esc_html( $value ) );
+				do_action('sensei_analysis_before_stats_boxes');
+				foreach ($sensei_analysis_overview->stats_boxes() as $key => $value) {
+					$this->render_stats_box(esc_html($key), esc_html($value));
 				} // End For Loop
-				do_action( 'sensei_analysis_after_stats_boxes' );
+				do_action('sensei_analysis_after_stats_boxes');
 				?>
 			</div>
 			<div class="sensei-analysis-main">
 				<?php $sensei_analysis_overview->display(); ?>
 			</div>
 			<div class="sensei-analysis-extra">
-				<?php do_action( 'sensei_analysis_extra' ); ?>
+				<?php do_action('sensei_analysis_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'analysis_wrapper_container', 'bottom' );
-		do_action( 'analysis_after_container' );
+		do_action('analysis_wrapper_container', 'bottom');
+		do_action('analysis_after_container');
 	} // End analysis_default_view()
 
 	/**
@@ -217,26 +217,26 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function analysis_user_profile_view( $user_id ) {
+	public function analysis_user_profile_view($user_id) {
 
 		// Load Analysis data
-		$sensei_analysis_user_profile = $this->load_data_object( 'User_Profile', $user_id );
+		$sensei_analysis_user_profile = $this->load_data_object('User_Profile', $user_id);
 		// Wrappers
-		do_action( 'analysis_before_container' );
-		do_action( 'analysis_wrapper_container', 'top' );
-		$this->analysis_headers( array( 'nav' => 'user_profile' ) );
+		do_action('analysis_before_container');
+		do_action('analysis_wrapper_container', 'top');
+		$this->analysis_headers(array('nav' => 'user_profile'));
 		?>
 		<div id="poststuff" class="sensei-analysis-wrap user-profile">
 			<div class="sensei-analysis-main">
 				<?php $sensei_analysis_user_profile->display(); ?>
 			</div>
 			<div class="sensei-analysis-extra">
-				<?php do_action( 'sensei_analysis_extra' ); ?>
+				<?php do_action('sensei_analysis_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'analysis_wrapper_container', 'bottom' );
-		do_action( 'analysis_after_container' );
+		do_action('analysis_wrapper_container', 'bottom');
+		do_action('analysis_after_container');
 	} // End analysis_user_profile_view()
 
 	/**
@@ -244,26 +244,26 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function analysis_course_view( $course_id ) {
+	public function analysis_course_view($course_id) {
 
 		// Load Analysis data
-		$sensei_analysis_course = $this->load_data_object( 'Course', $course_id );
+		$sensei_analysis_course = $this->load_data_object('Course', $course_id);
 		// Wrappers
-		do_action( 'analysis_before_container' );
-		do_action( 'analysis_wrapper_container', 'top' );
-		$this->analysis_headers( array( 'nav' => 'course' ) );
+		do_action('analysis_before_container');
+		do_action('analysis_wrapper_container', 'top');
+		$this->analysis_headers(array('nav' => 'course'));
 		?>
 		<div id="poststuff" class="sensei-analysis-wrap course-profile">
 			<div class="sensei-analysis-main">
 				<?php $sensei_analysis_course->display(); ?>
 			</div>
 			<div class="sensei-analysis-extra">
-				<?php do_action( 'sensei_analysis_extra' ); ?>
+				<?php do_action('sensei_analysis_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'analysis_wrapper_container', 'bottom' );
-		do_action( 'analysis_after_container' );
+		do_action('analysis_wrapper_container', 'bottom');
+		do_action('analysis_after_container');
 	} // End analysis_course_view()
 
 	/**
@@ -271,26 +271,26 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function analysis_user_course_view( $course_id, $user_id ) {
+	public function analysis_user_course_view($course_id, $user_id) {
 
 		// Load Analysis data
-		$sensei_analysis_user_course = $this->load_data_object( 'Course', $course_id, $user_id );
+		$sensei_analysis_user_course = $this->load_data_object('Course', $course_id, $user_id);
 		// Wrappers
-		do_action( 'analysis_before_container' );
-		do_action( 'analysis_wrapper_container', 'top' );
-		$this->analysis_headers( array( 'nav' => 'user_course' ) );
+		do_action('analysis_before_container');
+		do_action('analysis_wrapper_container', 'top');
+		$this->analysis_headers(array('nav' => 'user_course'));
 		?>
 		<div id="poststuff" class="sensei-analysis-wrap course-profile">
 			<div class="sensei-analysis-main">
 				<?php $sensei_analysis_user_course->display(); ?>
 			</div>
 			<div class="sensei-analysis-extra">
-				<?php do_action( 'sensei_analysis_extra' ); ?>
+				<?php do_action('sensei_analysis_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'analysis_wrapper_container', 'bottom' );
-		do_action( 'analysis_after_container' );
+		do_action('analysis_wrapper_container', 'bottom');
+		do_action('analysis_after_container');
 	} // End analysis_user_course_view()
 
 	/**
@@ -298,26 +298,26 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function analysis_course_users_view( $course_id ) {
+	public function analysis_course_users_view($course_id) {
 
 		// Load Analysis data
-		$sensei_analysis_course_users = $this->load_data_object( 'Course', $course_id );
+		$sensei_analysis_course_users = $this->load_data_object('Course', $course_id);
 		// Wrappers
-		do_action( 'analysis_before_container' );
-		do_action( 'analysis_wrapper_container', 'top' );
-		$this->analysis_headers( array( 'nav' => 'course_users' ) );
+		do_action('analysis_before_container');
+		do_action('analysis_wrapper_container', 'top');
+		$this->analysis_headers(array('nav' => 'course_users'));
 		?>
 		<div id="poststuff" class="sensei-analysis-wrap course-profile">
 			<div class="sensei-analysis-main">
 				<?php $sensei_analysis_course_users->display(); ?>
 			</div>
 			<div class="sensei-analysis-extra">
-				<?php do_action( 'sensei_analysis_extra' ); ?>
+				<?php do_action('sensei_analysis_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'analysis_wrapper_container', 'bottom' );
-		do_action( 'analysis_after_container' );
+		do_action('analysis_wrapper_container', 'bottom');
+		do_action('analysis_after_container');
 	} // End analysis_course_users_view()
 
 	/**
@@ -325,26 +325,26 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function analysis_lesson_users_view( $lesson_id ) {
+	public function analysis_lesson_users_view($lesson_id) {
 
 		// Load Analysis data
-		$sensei_analysis_lesson_users = $this->load_data_object( 'Lesson', $lesson_id );
+		$sensei_analysis_lesson_users = $this->load_data_object('Lesson', $lesson_id);
 		// Wrappers
-		do_action( 'analysis_before_container' );
-		do_action( 'analysis_wrapper_container', 'top' );
-		$this->analysis_headers( array( 'nav' => 'lesson_users' ) );
+		do_action('analysis_before_container');
+		do_action('analysis_wrapper_container', 'top');
+		$this->analysis_headers(array('nav' => 'lesson_users'));
 		?>
 		<div id="poststuff" class="sensei-analysis-wrap course-profile">
 			<div class="sensei-analysis-main">
 				<?php $sensei_analysis_lesson_users->display(); ?>
 			</div>
 			<div class="sensei-analysis-extra">
-				<?php do_action( 'sensei_analysis_extra' ); ?>
+				<?php do_action('sensei_analysis_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'analysis_wrapper_container', 'bottom' );
-		do_action( 'analysis_after_container' );
+		do_action('analysis_wrapper_container', 'bottom');
+		do_action('analysis_after_container');
 	} // End analysis_lesson_users_view()
 
 	/**
@@ -354,7 +354,7 @@ 
 	 * @param  $data string stats data
 	 * @return void
 	 */
-	public function render_stats_box( $title, $data ) {
+	public function render_stats_box($title, $data) {
 		?><div class="postbox">
 			<h3><span><?php echo $title; ?></span></h3>
 			<div class="inside">
@@ -368,14 +368,14 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function analysis_headers( $args = array( 'nav' => 'default' ) ) {
+	public function analysis_headers($args = array('nav' => 'default')) {
 
-		$function = 'analysis_' . $args['nav'] . '_nav';
+		$function = 'analysis_'.$args['nav'].'_nav';
 		$this->$function();
 		?>
 			<p class="powered-by-woo">
 
-                <?php _e( 'Powered by', 'woothemes-sensei' ); ?>
+                <?php _e('Powered by', 'woothemes-sensei'); ?>
 
                 <a href="http://www.woothemes.com/" title="WooThemes">
 
@@ -386,7 +386,7 @@ 
             </p>
 
 		<?php
-		do_action( 'sensei_analysis_after_headers' );
+		do_action('sensei_analysis_after_headers');
 	} // End analysis_headers()
 
 	/**
@@ -395,10 +395,10 @@ 
 	 * @param $which string
 	 * @return void
 	 */
-	public function wrapper_container( $which ) {
-		if ( 'top' == $which ) {
-			?><div id="woothemes-sensei" class="wrap <?php echo esc_attr( $this->token ); ?>"><?php
-		} elseif ( 'bottom' == $which ) {
+	public function wrapper_container($which) {
+		if ('top' == $which) {
+			?><div id="woothemes-sensei" class="wrap <?php echo esc_attr($this->token); ?>"><?php
+		} elseif ('bottom' == $which) {
 			?></div><!--/#woothemes-sensei--><?php
 		} // End If Statement
 	} // End wrapper_container()
@@ -410,24 +410,24 @@ 
 	 */
 	public function analysis_default_nav() {
 
-		$title = sprintf( '<a href="%s">%s</a>', esc_url( add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ) ), esc_html( $this->name ) );
-		$view = isset($_GET['view']) ? esc_html( $_GET['view'] ) : '';
-		switch ( $view ) { 
+		$title = sprintf('<a href="%s">%s</a>', esc_url(add_query_arg(array('page' => $this->page_slug), admin_url('admin.php'))), esc_html($this->name));
+		$view = isset($_GET['view']) ? esc_html($_GET['view']) : '';
+		switch ($view) { 
 			case 'courses' :
-				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', __( 'Courses', 'woothemes-sensei' ) );
+				$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', __('Courses', 'woothemes-sensei'));
 				break;
 
 			case 'lessons' :
-				$title .= sprintf( '&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;%s</span>', __( 'Lessons', 'woothemes-sensei' ) );
+				$title .= sprintf('&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;%s</span>', __('Lessons', 'woothemes-sensei'));
 				break;
 
 			case 'users' :
 			default :
-				$title .= sprintf( '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;%s</span>', __( 'Learners', 'woothemes-sensei' ) );
+				$title .= sprintf('&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;%s</span>', __('Learners', 'woothemes-sensei'));
 				break;
 		}
 		?>
-			<h2><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h2>
 		<?php
 	} // End analysis_default_nav()
 
@@ -438,17 +438,17 @@ 
 	 */
 	public function analysis_user_profile_nav() {
 
-		$title = sprintf( '<a href="%s">%s</a>', esc_url( add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['user_id'] ) && 0 < intval( $_GET['user_id'] ) ) {
+		$title = sprintf('<a href="%s">%s</a>', esc_url(add_query_arg(array('page' => $this->page_slug), admin_url('admin.php'))), esc_html($this->name));
+		if (isset($_GET['user_id']) && 0 < intval($_GET['user_id'])) {
 
-			$user_id = intval( $_GET['user_id'] );
-			$url = esc_url( add_query_arg( array( 'page' => $this->page_slug, 'user' => $user_id ), admin_url( 'admin.php' ) ) );
-            $user_name = Sensei()->learners->get_learner_full_name( $user_id );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', $url, $user_name );
+			$user_id = intval($_GET['user_id']);
+			$url = esc_url(add_query_arg(array('page' => $this->page_slug, 'user' => $user_id), admin_url('admin.php')));
+            $user_name = Sensei()->learners->get_learner_full_name($user_id);
+			$title .= sprintf('&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', $url, $user_name);
 
 		} // End If Statement
 		?>
-			<h2><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h2>
 		<?php
 	} // End analysis_user_profile_nav()
 
@@ -459,22 +459,22 @@ 
 	 */
 	public function analysis_user_course_nav() {
 
-		$title = sprintf( '<a href="%s">%s</a>', esc_url( add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['user_id'] ) && 0 < intval( $_GET['user_id'] ) ) {
-			$user_id = intval( $_GET['user_id'] );
-			$user_data = get_userdata( $user_id );
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'user_id' => $user_id ), admin_url( 'admin.php' ) );
-            $user_name = Sensei()->learners->get_learner_full_name( $user_id );
-            $title .= sprintf( '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', $url, $user_name );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), $user_data->display_name );
+		$title = sprintf('<a href="%s">%s</a>', esc_url(add_query_arg(array('page' => $this->page_slug), admin_url('admin.php'))), esc_html($this->name));
+		if (isset($_GET['user_id']) && 0 < intval($_GET['user_id'])) {
+			$user_id = intval($_GET['user_id']);
+			$user_data = get_userdata($user_id);
+			$url = add_query_arg(array('page' => $this->page_slug, 'user_id' => $user_id), admin_url('admin.php'));
+            $user_name = Sensei()->learners->get_learner_full_name($user_id);
+            $title .= sprintf('&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', $url, $user_name);
+			$title .= sprintf('&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), $user_data->display_name);
 		} // End If Statement
-		if ( isset( $_GET['course_id'] ) ) { 
-			$course_id = intval( $_GET['course_id'] );
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
+		if (isset($_GET['course_id'])) { 
+			$course_id = intval($_GET['course_id']);
+			$url = add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id), admin_url('admin.php'));
+			$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($course_id));
 		}
 		?>
-			<h2><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h2>
 		<?php
 	} // End analysis_user_course_nav()
 
@@ -485,14 +485,14 @@ 
 	 */
 	public function analysis_course_nav() {
 
-		$title = sprintf( '<a href="%s">%s</a>', add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['course_id'] ) ) { 
-			$course_id = intval( $_GET['course_id'] );
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>',esc_url( $url ), get_the_title( $course_id ) );
+		$title = sprintf('<a href="%s">%s</a>', add_query_arg(array('page' => $this->page_slug), admin_url('admin.php')), esc_html($this->name));
+		if (isset($_GET['course_id'])) { 
+			$course_id = intval($_GET['course_id']);
+			$url = add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id), admin_url('admin.php'));
+			$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($course_id));
 		}
 		?>
-			<h2><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h2>
 		<?php
 	} // End analysis_course_nav()
 
@@ -503,14 +503,14 @@ 
 	 */
 	public function analysis_course_users_nav() {
 
-		$title = sprintf( '<a href="%s">%s</a>', add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['course_id'] ) ) { 
-			$course_id = intval( $_GET['course_id'] );
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
+		$title = sprintf('<a href="%s">%s</a>', add_query_arg(array('page' => $this->page_slug), admin_url('admin.php')), esc_html($this->name));
+		if (isset($_GET['course_id'])) { 
+			$course_id = intval($_GET['course_id']);
+			$url = add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id), admin_url('admin.php'));
+			$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($course_id));
 		}
 		?>
-			<h2><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h2>
 		<?php
 	} // End analysis_course_users_nav()
 
@@ -521,17 +521,17 @@ 
 	 */
 	public function analysis_lesson_users_nav() {
 
-		$title = sprintf( '<a href="%s">%s</a>', add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['lesson_id'] ) ) { 
-			$lesson_id = intval( $_GET['lesson_id'] );
-			$course_id = intval( get_post_meta( $lesson_id, '_lesson_course', true ) );
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $lesson_id ), admin_url( 'admin.php' ) );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $lesson_id ) );
+		$title = sprintf('<a href="%s">%s</a>', add_query_arg(array('page' => $this->page_slug), admin_url('admin.php')), esc_html($this->name));
+		if (isset($_GET['lesson_id'])) { 
+			$lesson_id = intval($_GET['lesson_id']);
+			$course_id = intval(get_post_meta($lesson_id, '_lesson_course', true));
+			$url = add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id), admin_url('admin.php'));
+			$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($course_id));
+			$url = add_query_arg(array('page' => $this->page_slug, 'lesson_id' => $lesson_id), admin_url('admin.php'));
+			$title .= sprintf('&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($lesson_id));
 		}
 		?>
-			<h2><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h2>
 		<?php
 	} // End analysis_lesson_users_nav()
 
@@ -542,59 +542,59 @@ 
 	 */
 	public function report_download_page() {
 		// Check if is a report
-		if ( !empty( $_GET['sensei_report_download'] ) ) {
-			$report = sanitize_text_field( $_GET['sensei_report_download'] );
+		if ( ! empty($_GET['sensei_report_download'])) {
+			$report = sanitize_text_field($_GET['sensei_report_download']);
 
 			// Simple verification to ensure intent, Note that a Nonce is per user, so the URL can't be shared
-			if ( !wp_verify_nonce( $_REQUEST['_sdl_nonce'], 'sensei_csv_download-' . $report ) ) {
-				wp_die( __('Invalid request', 'woothemes-sensei') );
+			if ( ! wp_verify_nonce($_REQUEST['_sdl_nonce'], 'sensei_csv_download-'.$report)) {
+				wp_die(__('Invalid request', 'woothemes-sensei'));
 			}
 
 			// Setup the variables we might need
-			$filename = apply_filters( 'sensei_csv_export_filename', $report );
+			$filename = apply_filters('sensei_csv_export_filename', $report);
 			$course_id = 0;
 			$lesson_id = 0;
 			$user_id = 0;
-			if( isset( $_GET['course_id'] ) ) {
-				$course_id = intval( $_GET['course_id'] );
+			if (isset($_GET['course_id'])) {
+				$course_id = intval($_GET['course_id']);
 			}
-			if( isset( $_GET['lesson_id'] ) ) {
-				$lesson_id = intval( $_GET['lesson_id'] );
+			if (isset($_GET['lesson_id'])) {
+				$lesson_id = intval($_GET['lesson_id']);
 			}
-			if( isset( $_GET['user_id'] ) ) {
-				$user_id = intval( $_GET['user_id'] );
+			if (isset($_GET['user_id'])) {
+				$user_id = intval($_GET['user_id']);
 			}
-			$type = isset( $_GET['view'] ) ? esc_html( $_GET['view'] ) : false;
+			$type = isset($_GET['view']) ? esc_html($_GET['view']) : false;
 
-			if ( 0 < $lesson_id ) {
+			if (0 < $lesson_id) {
 				// Viewing a specific Lesson and all its Learners
-				$sensei_analysis_report_object = $this->load_report_object( 'Lesson', $lesson_id );
+				$sensei_analysis_report_object = $this->load_report_object('Lesson', $lesson_id);
 			}
-			elseif ( 0 < $course_id && 0 < $user_id ) {
+			elseif (0 < $course_id && 0 < $user_id) {
 				// Viewing a specific User on a specific Course
-				$sensei_analysis_report_object = $this->load_report_object( 'Course', $course_id, $user_id );
+				$sensei_analysis_report_object = $this->load_report_object('Course', $course_id, $user_id);
 			}
-			elseif( 0 < $course_id ) {
+			elseif (0 < $course_id) {
 				// Viewing a specific Course and all it's Lessons, or it's Learners
-				$sensei_analysis_report_object = $this->load_report_object( 'Course', $course_id );
+				$sensei_analysis_report_object = $this->load_report_object('Course', $course_id);
 			}
-			elseif ( 0 < $user_id ) {
+			elseif (0 < $user_id) {
 				// Viewing a specific Learner, and their Courses
-				$sensei_analysis_report_object = $this->load_report_object( 'User_Profile', $user_id );
+				$sensei_analysis_report_object = $this->load_report_object('User_Profile', $user_id);
 			}
 			else {
 				// Overview of all Learners, all Courses, or all Lessons
-				$sensei_analysis_report_object = $this->load_report_object( 'Overview', $type );
+				$sensei_analysis_report_object = $this->load_report_object('Overview', $type);
 			} // End If Statement
 
 			// Handle the headers
-			$this->report_set_headers( $filename );
+			$this->report_set_headers($filename);
 
 			// Collate the data, there could be many different reports for a single object
-			$report_data_array = $sensei_analysis_report_object->generate_report( $report );
+			$report_data_array = $sensei_analysis_report_object->generate_report($report);
 
 			// Output the data
-			$this->report_write_download( $report_data_array );
+			$this->report_write_download($report_data_array);
 
 			// Cleanly exit
 			exit;
@@ -607,9 +607,9 @@ 
 	 * @param  string $filename name of report file
 	 * @return void
 	 */
-	public function report_set_headers( $filename = '' ) {
-		header( 'Content-Type: text/csv' );
-		header( 'Content-Disposition: attachment;filename=' . $filename . '.csv');
+	public function report_set_headers($filename = '') {
+		header('Content-Type: text/csv');
+		header('Content-Disposition: attachment;filename='.$filename.'.csv');
 	} // End report_set_headers()
 
 	/**
@@ -620,12 +620,12 @@ 
 	 * @param  undefined  $optional_data optional constructor arguments
 	 * @return object                 class instance object
 	 */
-	public function load_report_object( $name = '', $data = 0, $optional_data = null ) {
-		$object_name = 'WooThemes_Sensei_Analysis_' . $name . '_List_Table';
-		if ( is_null($optional_data) ) {
-			$sensei_analysis_report_object = new $object_name( $data );
+	public function load_report_object($name = '', $data = 0, $optional_data = null) {
+		$object_name = 'WooThemes_Sensei_Analysis_'.$name.'_List_Table';
+		if (is_null($optional_data)) {
+			$sensei_analysis_report_object = new $object_name($data);
 		} else {
-			$sensei_analysis_report_object = new $object_name( $data, $optional_data );
+			$sensei_analysis_report_object = new $object_name($data, $optional_data);
 		}
 		return $sensei_analysis_report_object;
 	} // End load_report_object()
@@ -636,7 +636,7 @@ 
 	 * @param  array  $report_array data array
 	 * @return void
 	 */
-	public function report_write_download( $report_data = array() ) {
+	public function report_write_download($report_data = array()) {
 		$fp = fopen('php://output', 'w');
 		foreach ($report_data as $row) {
 			fputcsv($fp, $row);
@@ -652,9 +652,9 @@ 
 	 * @param  object $user_query_object     WP_User_Query Object
 	 * @return array $search_columns         array of user columns to search
 	 */
-	public function user_search_columns_filter( $search_columns, $search, $user_query_object ) {
+	public function user_search_columns_filter($search_columns, $search, $user_query_object) {
 		// Alter $search_columns to include the fields you want to search on
-		array_push( $search_columns, 'display_name' );
+		array_push($search_columns, 'display_name');
 		return $search_columns;
 	}
 

includes/class-sensei-emails.php

Doc Comments +5 added lines, -1 removed lines

@@ -40,6 +40,7 @@ 
 	 * Constructor for the email class hooks in all emails that can be sent.
 	 *
 	 * @access public
+	 * @param string $file
 	 * @return void
 	 */
 	function __construct( $file ) {
@@ -140,7 +141,7 @@ 
 	 * Wraps a message in the sensei mail template.
 	 *
 	 * @access public
-	 * @param mixed $content
+	 * @param null|string $content
 	 * @return string
 	 */
 	function wrap_message( $content ) {
@@ -198,6 +199,9 @@ 
 		remove_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
 	}
 
+	/**
+	 * @param string $email_template
+	 */
 	function get_content( $email_template ) {
 
 		$message = $this->load_template( $email_template );

Spacing +94 added lines, -94 removed lines

@@ -42,22 +42,22 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function __construct( $file ) {
+	function __construct($file) {
 
 		$this->init();
 
 		// Hooks for sending emails during Sensei events
-		add_action( 'sensei_user_quiz_grade', array( $this, 'learner_graded_quiz' ), 10, 4 );
-		add_action( 'sensei_course_status_updated', array( $this, 'learner_completed_course' ), 10, 4 );
-		add_action( 'sensei_course_status_updated', array( $this, 'teacher_completed_course' ), 10, 4 );
-		add_action( 'sensei_user_course_start', array( $this, 'teacher_started_course' ), 10, 2 );
-        add_action( 'sensei_user_lesson_end', array( $this, 'teacher_completed_lesson' ), 10, 2 );
-		add_action( 'sensei_user_quiz_submitted', array( $this, 'teacher_quiz_submitted' ), 10, 5 );
-		add_action( 'sensei_new_private_message', array( $this, 'teacher_new_message' ), 10, 1 );
-		add_action( 'sensei_private_message_reply', array( $this, 'new_message_reply' ), 10, 2 );
+		add_action('sensei_user_quiz_grade', array($this, 'learner_graded_quiz'), 10, 4);
+		add_action('sensei_course_status_updated', array($this, 'learner_completed_course'), 10, 4);
+		add_action('sensei_course_status_updated', array($this, 'teacher_completed_course'), 10, 4);
+		add_action('sensei_user_course_start', array($this, 'teacher_started_course'), 10, 2);
+        add_action('sensei_user_lesson_end', array($this, 'teacher_completed_lesson'), 10, 2);
+		add_action('sensei_user_quiz_submitted', array($this, 'teacher_quiz_submitted'), 10, 5);
+		add_action('sensei_new_private_message', array($this, 'teacher_new_message'), 10, 1);
+		add_action('sensei_private_message_reply', array($this, 'new_message_reply'), 10, 2);
 
 		// Let 3rd parties unhook the above via this hook
-		do_action( 'sensei_emails', $this );
+		do_action('sensei_emails', $this);
 	}
 
 	/**
@@ -65,15 +65,15 @@ 
 	 */
 	function init() {
 
-		$this->emails['learner-graded-quiz'] = include( 'emails/class-woothemes-sensei-email-learner-graded-quiz.php' );
-		$this->emails['learner-completed-course'] = include( 'emails/class-woothemes-sensei-email-learner-completed-course.php' );
-		$this->emails['teacher-completed-course'] = include( 'emails/class-woothemes-sensei-email-teacher-completed-course.php' );
-        $this->emails['teacher-started-course'] = include( 'emails/class-woothemes-sensei-email-teacher-started-course.php' );
-        $this->emails['teacher-completed-lesson'] = include( 'emails/class-woothemes-sensei-email-teacher-completed-lesson.php' );
-        $this->emails['teacher-quiz-submitted'] = include( 'emails/class-woothemes-sensei-email-teacher-quiz-submitted.php' );
-		$this->emails['teacher-new-message'] = include( 'emails/class-woothemes-sensei-email-teacher-new-message.php' );
-		$this->emails['new-message-reply'] = include( 'emails/class-woothemes-sensei-email-new-message-reply.php' );
-		$this->emails = apply_filters( 'sensei_email_classes', $this->emails );
+		$this->emails['learner-graded-quiz'] = include('emails/class-woothemes-sensei-email-learner-graded-quiz.php');
+		$this->emails['learner-completed-course'] = include('emails/class-woothemes-sensei-email-learner-completed-course.php');
+		$this->emails['teacher-completed-course'] = include('emails/class-woothemes-sensei-email-teacher-completed-course.php');
+        $this->emails['teacher-started-course'] = include('emails/class-woothemes-sensei-email-teacher-started-course.php');
+        $this->emails['teacher-completed-lesson'] = include('emails/class-woothemes-sensei-email-teacher-completed-lesson.php');
+        $this->emails['teacher-quiz-submitted'] = include('emails/class-woothemes-sensei-email-teacher-quiz-submitted.php');
+		$this->emails['teacher-new-message'] = include('emails/class-woothemes-sensei-email-teacher-new-message.php');
+		$this->emails['new-message-reply'] = include('emails/class-woothemes-sensei-email-new-message-reply.php');
+		$this->emails = apply_filters('sensei_email_classes', $this->emails);
 	}
 
 	/**
@@ -95,15 +95,15 @@ 
 	function get_from_name() {
 
 
-		if ( ! $this->_from_name ) {
-			if( isset( Sensei()->settings->settings['email_from_name'] ) && '' != Sensei()->settings->settings['email_from_name'] ) {
+		if ( ! $this->_from_name) {
+			if (isset(Sensei()->settings->settings['email_from_name']) && '' != Sensei()->settings->settings['email_from_name']) {
 				$this->_from_name = Sensei()->settings->settings['email_from_name'];
 			} else {
-				$this->_from_name = get_bloginfo( 'name' );
+				$this->_from_name = get_bloginfo('name');
 			}
 		}
 
-		return wp_specialchars_decode( $this->_from_name );
+		return wp_specialchars_decode($this->_from_name);
 	}
 
 	/**
@@ -115,11 +115,11 @@ 
 	function get_from_address() {
 
 
-		if ( ! $this->_from_address ) {
-			if( isset( Sensei()->settings->settings['email_from_address'] ) && '' != Sensei()->settings->settings['email_from_address'] ) {
+		if ( ! $this->_from_address) {
+			if (isset(Sensei()->settings->settings['email_from_address']) && '' != Sensei()->settings->settings['email_from_address']) {
 				$this->_from_address = Sensei()->settings->settings['email_from_address'];
 			} else {
-				$this->_from_address = get_bloginfo( 'admin_email' );
+				$this->_from_address = get_bloginfo('admin_email');
 			}
 		}
 
@@ -143,13 +143,13 @@ 
 	 * @param mixed $content
 	 * @return string
 	 */
-	function wrap_message( $content ) {
+	function wrap_message($content) {
 
 		$html = '';
 
-		$html .= $this->load_template( 'header' );
-		$html .= wpautop( wptexturize( $content ) );
-		$html .= $this->load_template( 'footer' );
+		$html .= $this->load_template('header');
+		$html .= wpautop(wptexturize($content));
+		$html .= $this->load_template('footer');
 
 		return $html;
 	}
@@ -166,16 +166,16 @@ 
 	 * @param string $content_type (default: "text/html")
 	 * @return void
 	 */
-	function send( $to, $subject, $message, $headers = "Content-Type: text/html\r\n", $attachments = "", $content_type = 'text/html' ) {
+	function send($to, $subject, $message, $headers = "Content-Type: text/html\r\n", $attachments = "", $content_type = 'text/html') {
 		global $email_template;
 
 		// Set content type
 		$this->_content_type = $content_type;
 
 		// Filters for the email
-		add_filter( 'wp_mail_from', array( $this, 'get_from_address' ) );
-		add_filter( 'wp_mail_from_name', array( $this, 'get_from_name' ) );
-		add_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
+		add_filter('wp_mail_from', array($this, 'get_from_address'));
+		add_filter('wp_mail_from_name', array($this, 'get_from_name'));
+		add_filter('wp_mail_content_type', array($this, 'get_content_type'));
 
         // Send
         $send_email = true;
@@ -186,40 +186,40 @@ 
          * @since 1.8.0
          * @param bool $send_email default true
          */
-        if( apply_filters('sensei_send_emails', $send_email,$to, $subject, $message )  ){
+        if (apply_filters('sensei_send_emails', $send_email, $to, $subject, $message)) {
 
-            wp_mail( $to, $subject, $message, $headers, $attachments );
+            wp_mail($to, $subject, $message, $headers, $attachments);
 
         }
 
 		// Unhook filters
-		remove_filter( 'wp_mail_from', array( $this, 'get_from_address' ) );
-		remove_filter( 'wp_mail_from_name', array( $this, 'get_from_name' ) );
-		remove_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
+		remove_filter('wp_mail_from', array($this, 'get_from_address'));
+		remove_filter('wp_mail_from_name', array($this, 'get_from_name'));
+		remove_filter('wp_mail_content_type', array($this, 'get_content_type'));
 	}
 
-	function get_content( $email_template ) {
+	function get_content($email_template) {
 
-		$message = $this->load_template( $email_template );
+		$message = $this->load_template($email_template);
 
-		$html = $this->wrap_message( $message );
+		$html = $this->wrap_message($message);
 
-		return apply_filters( 'sensei_email', $html, $email_template );
+		return apply_filters('sensei_email', $html, $email_template);
 	}
 
-	function load_template( $template = '' ) {
+	function load_template($template = '') {
 		global  $email_template;
 
-		if( ! $template ) return;
+		if ( ! $template) return;
 
-		$email_template = $template . '.php';
-		$template = Sensei_Templates::template_loader( '' );
+		$email_template = $template.'.php';
+		$template = Sensei_Templates::template_loader('');
 
 		ob_start();
 
-		do_action( 'sensei_before_email_template', $email_template );
-		include( $template );
-		do_action( 'sensei_after_email_template', $email_template );
+		do_action('sensei_before_email_template', $email_template);
+		include($template);
+		do_action('sensei_after_email_template', $email_template);
 
 		return ob_get_clean();
 	}
@@ -230,22 +230,22 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function learner_graded_quiz( $user_id, $quiz_id, $grade, $passmark ) {
+	function learner_graded_quiz($user_id, $quiz_id, $grade, $passmark) {
 
 
 		$send = false;
 
-		if( isset( Sensei()->settings->settings['email_learners'] ) ) {
-			if( in_array( 'learner-graded-quiz', (array) Sensei()->settings->settings['email_learners'] ) ) {
+		if (isset(Sensei()->settings->settings['email_learners'])) {
+			if (in_array('learner-graded-quiz', (array) Sensei()->settings->settings['email_learners'])) {
 				$send = true;
 			}
 		} else {
 			$send = true;
 		}
 
-		if( $send ) {
+		if ($send) {
 			$email = $this->emails['learner-graded-quiz'];
-			$email->trigger( $user_id, $quiz_id, $grade, $passmark );
+			$email->trigger($user_id, $quiz_id, $grade, $passmark);
 		}
 	}
 
@@ -255,26 +255,26 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function learner_completed_course( $status = 'in-progress', $user_id = 0, $course_id = 0, $comment_id = 0 ) {
+	function learner_completed_course($status = 'in-progress', $user_id = 0, $course_id = 0, $comment_id = 0) {
 
 
-		if( 'complete' != $status ) {
+		if ('complete' != $status) {
 			return;
 		}
 
 		$send = false;
 
-		if( isset( Sensei()->settings->settings['email_learners'] ) ) {
-			if( in_array( 'learner-completed-course', (array) Sensei()->settings->settings['email_learners'] ) ) {
+		if (isset(Sensei()->settings->settings['email_learners'])) {
+			if (in_array('learner-completed-course', (array) Sensei()->settings->settings['email_learners'])) {
 				$send = true;
 			}
 		} else {
 			$send = true;
 		}
 
-		if( $send ) {
+		if ($send) {
 			$email = $this->emails['learner-completed-course'];
-			$email->trigger( $user_id, $course_id );
+			$email->trigger($user_id, $course_id);
 		}
 	}
 
@@ -284,26 +284,26 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function teacher_completed_course( $status = 'in-progress', $learner_id = 0, $course_id = 0, $comment_id = 0 ) {
+	function teacher_completed_course($status = 'in-progress', $learner_id = 0, $course_id = 0, $comment_id = 0) {
 
 
-		if( 'complete' != $status ) {
+		if ('complete' != $status) {
 			return;
 		}
 
 		$send = false;
 
-		if( isset( Sensei()->settings->settings['email_teachers'] ) ) {
-			if( in_array( 'teacher-completed-course', (array) Sensei()->settings->settings['email_teachers'] ) ) {
+		if (isset(Sensei()->settings->settings['email_teachers'])) {
+			if (in_array('teacher-completed-course', (array) Sensei()->settings->settings['email_teachers'])) {
 				$send = true;
 			}
 		} else {
 			$send = true;
 		}
 
-		if( $send ) {
+		if ($send) {
 			$email = $this->emails['teacher-completed-course'];
-			$email->trigger( $learner_id, $course_id );
+			$email->trigger($learner_id, $course_id);
 		}
 	}
 
@@ -313,22 +313,22 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function teacher_started_course( $learner_id = 0, $course_id = 0 ) {
+	function teacher_started_course($learner_id = 0, $course_id = 0) {
 
 
 		$send = false;
 
-		if( isset( Sensei()->settings->settings['email_teachers'] ) ) {
-			if( in_array( 'teacher-started-course', (array) Sensei()->settings->settings['email_teachers'] ) ) {
+		if (isset(Sensei()->settings->settings['email_teachers'])) {
+			if (in_array('teacher-started-course', (array) Sensei()->settings->settings['email_teachers'])) {
 				$send = true;
 			}
 		} else {
 			$send = true;
 		}
 
-		if( $send ) {
+		if ($send) {
 			$email = $this->emails['teacher-started-course'];
-			$email->trigger( $learner_id, $course_id );
+			$email->trigger($learner_id, $course_id);
 		}
 	}
 
@@ -341,22 +341,22 @@ 
      * @return void
      * @since 1.9.0
      */
-    function teacher_completed_lesson( $learner_id = 0, $lesson_id = 0 ) {
+    function teacher_completed_lesson($learner_id = 0, $lesson_id = 0) {
 
 
         $send = false;
 
-        if( isset( Sensei()->settings->settings[ 'email_teachers' ] ) ) {
-            if( in_array( 'teacher-completed-lesson', (array) Sensei()->settings->settings[ 'email_teachers' ]) ) {
+        if (isset(Sensei()->settings->settings['email_teachers'])) {
+            if (in_array('teacher-completed-lesson', (array) Sensei()->settings->settings['email_teachers'])) {
                 $send = true;
             }
         } else {
             $send = true;
         }
 
-        if( $send ) {
+        if ($send) {
             $email = $this->emails['teacher-completed-lesson'];
-            $email->trigger( $learner_id, $lesson_id );
+            $email->trigger($learner_id, $lesson_id);
         }
     }
 
@@ -366,25 +366,25 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function teacher_quiz_submitted( $learner_id = 0, $quiz_id = 0, $grade = 0, $passmark = 0, $quiz_grade_type = 'manual' ) {
+	function teacher_quiz_submitted($learner_id = 0, $quiz_id = 0, $grade = 0, $passmark = 0, $quiz_grade_type = 'manual') {
 
 
 
 		$send = false;
 
 		// Only trigger if the quiz was marked as manual grading, or auto grading didn't complete
-		if( 'manual' == $quiz_grade_type || is_wp_error( $grade ) ) {
-			if( isset( Sensei()->settings->settings['email_teachers'] ) ) {
-				if( in_array( 'teacher-quiz-submitted', (array) Sensei()->settings->settings['email_teachers'] ) ) {
+		if ('manual' == $quiz_grade_type || is_wp_error($grade)) {
+			if (isset(Sensei()->settings->settings['email_teachers'])) {
+				if (in_array('teacher-quiz-submitted', (array) Sensei()->settings->settings['email_teachers'])) {
 					$send = true;
 				}
 			} else {
 				$send = true;
 			}
 
-			if( $send ) {
+			if ($send) {
 				$email = $this->emails['teacher-quiz-submitted'];
-				$email->trigger( $learner_id, $quiz_id );
+				$email->trigger($learner_id, $quiz_id);
 			}
 
 		}
@@ -396,22 +396,22 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function teacher_new_message( $message_id = 0 ) {
+	function teacher_new_message($message_id = 0) {
 
 
 		$send = false;
 
-		if( isset( Sensei()->settings->settings['email_teachers'] ) ) {
-			if( in_array( 'teacher-new-message', (array) Sensei()->settings->settings['email_teachers'] ) ) {
+		if (isset(Sensei()->settings->settings['email_teachers'])) {
+			if (in_array('teacher-new-message', (array) Sensei()->settings->settings['email_teachers'])) {
 				$send = true;
 			}
 		} else {
 			$send = true;
 		}
 
-		if( $send ) {
+		if ($send) {
 			$email = $this->emails['teacher-new-message'];
-			$email->trigger( $message_id );
+			$email->trigger($message_id);
 		}
 	}
 
@@ -421,22 +421,22 @@ 
 	 * @access public
 	 * @return void
 	 */
-	function new_message_reply( $comment, $message ) {
+	function new_message_reply($comment, $message) {
 
 
 		$send = false;
 
-		if( isset( Sensei()->settings->settings['email_global'] ) ) {
-			if( in_array( 'new-message-reply', (array) Sensei()->settings->settings['email_global'] ) ) {
+		if (isset(Sensei()->settings->settings['email_global'])) {
+			if (in_array('new-message-reply', (array) Sensei()->settings->settings['email_global'])) {
 				$send = true;
 			}
 		} else {
 			$send = true;
 		}
 
-		if( $send ) {
+		if ($send) {
 			$email = $this->emails['new-message-reply'];
-			$email->trigger( $comment, $message );
+			$email->trigger($comment, $message);
 		}
 	}
 
@@ -447,4 +447,4 @@ 
  * for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Emails extends Sensei_Emails{}
+class WooThemes_Sensei_Emails extends Sensei_Emails {}

includes/class-sensei-grading.php

Doc Comments +5 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
 	 * Constructor
 	 * @since  1.3.0
      *
-     * @param $file
+     * @param string $file
 	 */
 	public function __construct ( $file ) {
 		$this->name = __( 'Grading', 'woothemes-sensei' );
@@ -330,6 +330,7 @@ 
 	/**
 	 * Return array of valid statuses for either Course or Lesson
 	 * @since  1.7.0
+	 * @param string $type
 	 * @return array
 	 */
 	public function get_stati( $type ) {
@@ -529,7 +530,7 @@ 
      * the grades as the Grader selected theme and saves the total grade and
      * individual question grades.
      *
-     * @return bool
+     * @return false|null
      */
     public function admin_process_grading_submission() {
 
@@ -906,8 +907,8 @@ 
      * Grading logic specifically for the gap fill questions
      *
      * @since 1.9.0
-     * @param $question_id
-     * @param $user_answer
+     * @param integer $question_id
+     * @param string $user_answer
      *
      * @return bool | int false or the grade given to the user answer
      */

Spacing +243 added lines, -243 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Grading Class
@@ -24,30 +24,30 @@ 
      *
      * @param $file
 	 */
-	public function __construct ( $file ) {
-		$this->name = __( 'Grading', 'woothemes-sensei' );
+	public function __construct($file) {
+		$this->name = __('Grading', 'woothemes-sensei');
 		$this->file = $file;
 		$this->page_slug = 'sensei_grading';
 
 		// Admin functions
-		if ( is_admin() ) {
-			add_action( 'admin_menu', array( $this, 'grading_admin_menu' ), 20);
-			add_action( 'grading_wrapper_container', array( $this, 'wrapper_container'  ) );
-			if ( isset( $_GET['page'] ) && ( $_GET['page'] == $this->page_slug ) ) {
-				add_action( 'admin_print_scripts', array( $this, 'enqueue_scripts' ) );
-				add_action( 'admin_print_styles', array( $this, 'enqueue_styles' ) );
+		if (is_admin()) {
+			add_action('admin_menu', array($this, 'grading_admin_menu'), 20);
+			add_action('grading_wrapper_container', array($this, 'wrapper_container'));
+			if (isset($_GET['page']) && ($_GET['page'] == $this->page_slug)) {
+				add_action('admin_print_scripts', array($this, 'enqueue_scripts'));
+				add_action('admin_print_styles', array($this, 'enqueue_styles'));
 			}
 
-			add_action( 'admin_init', array( $this, 'admin_process_grading_submission' ) );
+			add_action('admin_init', array($this, 'admin_process_grading_submission'));
 
-			add_action( 'admin_notices', array( $this, 'add_grading_notices' ) );
+			add_action('admin_notices', array($this, 'add_grading_notices'));
 //			add_action( 'sensei_grading_notices', array( $this, 'sensei_grading_notices' ) );
 		} // End If Statement
 
 		// Ajax functions
-		if ( is_admin() ) {
-			add_action( 'wp_ajax_get_lessons_dropdown', array( $this, 'get_lessons_dropdown' ) );
-			add_action( 'wp_ajax_get_redirect_url', array( $this, 'get_redirect_url' ) );
+		if (is_admin()) {
+			add_action('wp_ajax_get_lessons_dropdown', array($this, 'get_lessons_dropdown'));
+			add_action('wp_ajax_get_redirect_url', array($this, 'get_redirect_url'));
 		} // End If Statement
 	} // End __construct()
 
@@ -60,8 +60,8 @@ 
 	public function grading_admin_menu() {
 		global $menu;
 
-		if ( current_user_can( 'manage_sensei_grades' ) ) {
-			$grading_page = add_submenu_page('sensei', __('Grading', 'woothemes-sensei'),  __('Grading', 'woothemes-sensei') , 'manage_sensei_grades', $this->page_slug, array( $this, 'grading_page' ) );
+		if (current_user_can('manage_sensei_grades')) {
+			$grading_page = add_submenu_page('sensei', __('Grading', 'woothemes-sensei'), __('Grading', 'woothemes-sensei'), 'manage_sensei_grades', $this->page_slug, array($this, 'grading_page'));
 		}
 
 	} // End grading_admin_menu()
@@ -74,13 +74,13 @@ 
 	 * @since 1.3.0
 	 * @return void
 	 */
-	public function enqueue_scripts () {
+	public function enqueue_scripts() {
 
 
-		$suffix = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ? '' : '.min';
+		$suffix = defined('SCRIPT_DEBUG') && SCRIPT_DEBUG ? '' : '.min';
 
 		// Load Grading JS
-		wp_enqueue_script( 'sensei-grading-general', Sensei()->plugin_url . 'assets/js/grading-general' . $suffix . '.js', array( 'jquery' ), Sensei()->version );
+		wp_enqueue_script('sensei-grading-general', Sensei()->plugin_url.'assets/js/grading-general'.$suffix.'.js', array('jquery'), Sensei()->version);
 
 	} // End enqueue_scripts()
 
@@ -92,11 +92,11 @@ 
 	 * @since 1.0.0
 	 * @return void
 	 */
-	public function enqueue_styles () {
+	public function enqueue_styles() {
 
-		wp_enqueue_style( Sensei()->token . '-admin' );
+		wp_enqueue_style(Sensei()->token.'-admin');
 
-		wp_enqueue_style( 'woothemes-sensei-settings-api', Sensei()->plugin_url . 'assets/css/settings.css', '', Sensei()->version );
+		wp_enqueue_style('woothemes-sensei-settings-api', Sensei()->plugin_url.'assets/css/settings.css', '', Sensei()->version);
 
 	} // End enqueue_styles()
 
@@ -108,12 +108,12 @@ 
 	public function load_data_table_files() {
 
 		// Load Grading Classes
-		$classes_to_load = array(	'list-table',
+		$classes_to_load = array('list-table',
 									'grading-main',
 									'grading-user-quiz'
 									);
-		foreach ( $classes_to_load as $class_file ) {
-			Sensei()->load_class( $class_file );
+		foreach ($classes_to_load as $class_file) {
+			Sensei()->load_class($class_file);
 		} // End For Loop
 	} // End load_data_table_files()
 
@@ -125,16 +125,16 @@ 
 	 * @param  undefined  $optional_data optional constructor arguments
 	 * @return object                 class instance object
 	 */
-	public function load_data_object( $name = '', $data = 0, $optional_data = null ) {
+	public function load_data_object($name = '', $data = 0, $optional_data = null) {
 		// Load Analysis data
-		$object_name = 'WooThemes_Sensei_Grading_' . $name;
-		if ( is_null($optional_data) ) {
-			$sensei_grading_object = new $object_name( $data );
+		$object_name = 'WooThemes_Sensei_Grading_'.$name;
+		if (is_null($optional_data)) {
+			$sensei_grading_object = new $object_name($data);
 		}
 		else {
-			$sensei_grading_object = new $object_name( $data, $optional_data );
+			$sensei_grading_object = new $object_name($data, $optional_data);
 		} // End If Statement
-		if ( 'Main' == $name ) {
+		if ('Main' == $name) {
 			$sensei_grading_object->prepare_items();
 		} // End If Statement
 		return $sensei_grading_object;
@@ -148,7 +148,7 @@ 
 	 */
 	public function grading_page() {
 
-		if ( isset( $_GET['quiz_id'] ) && 0 < intval( $_GET['quiz_id'] ) && isset( $_GET['user'] ) && 0 < intval( $_GET['user'] ) ) {
+		if (isset($_GET['quiz_id']) && 0 < intval($_GET['quiz_id']) && isset($_GET['user']) && 0 < intval($_GET['user'])) {
 			$this->grading_user_quiz_view();
 		}
 		else {
@@ -164,23 +164,23 @@ 
 	public function grading_default_view() {
 
 		// Load Grading data
-		if( !empty( $_GET['course_id'] ) ) {
-			$course_id = intval( $_GET['course_id'] );
+		if ( ! empty($_GET['course_id'])) {
+			$course_id = intval($_GET['course_id']);
 		}
-		if( !empty( $_GET['lesson_id'] ) ) {
-			$lesson_id = intval( $_GET['lesson_id'] );
+		if ( ! empty($_GET['lesson_id'])) {
+			$lesson_id = intval($_GET['lesson_id']);
 		}
-		if( !empty( $_GET['user_id'] ) ) {
-			$user_id = intval( $_GET['user_id'] );
+		if ( ! empty($_GET['user_id'])) {
+			$user_id = intval($_GET['user_id']);
 		}
-		if( !empty( $_GET['view'] ) ) {
-			$view = esc_html( $_GET['view'] );
+		if ( ! empty($_GET['view'])) {
+			$view = esc_html($_GET['view']);
 		}
-		$sensei_grading_overview = $this->load_data_object( 'Main', compact( 'course_id', 'lesson_id', 'user_id', 'view' ) );
+		$sensei_grading_overview = $this->load_data_object('Main', compact('course_id', 'lesson_id', 'user_id', 'view'));
 
 		// Wrappers
-		do_action( 'grading_before_container' );
-		do_action( 'grading_wrapper_container', 'top' );
+		do_action('grading_before_container');
+		do_action('grading_wrapper_container', 'top');
 		$this->grading_headers();
 		?>
 		<div id="poststuff" class="sensei-grading-wrap">
@@ -188,12 +188,12 @@ 
 				<?php $sensei_grading_overview->display(); ?>
 			</div>
 			<div class="sensei-grading-extra">
-				<?php do_action( 'sensei_grading_extra' ); ?>
+				<?php do_action('sensei_grading_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'grading_wrapper_container', 'bottom' );
-		do_action( 'grading_after_container' );
+		do_action('grading_wrapper_container', 'bottom');
+		do_action('grading_after_container');
 	} // End grading_default_view()
 
 	/**
@@ -206,17 +206,17 @@ 
 		// Load Grading data
 		$user_id = 0;
 		$quiz_id = 0;
-		if( isset( $_GET['user'] ) ) {
-			$user_id = intval( $_GET['user'] );
+		if (isset($_GET['user'])) {
+			$user_id = intval($_GET['user']);
 		}
-		if( isset( $_GET['quiz_id'] ) ) {
-			$quiz_id = intval( $_GET['quiz_id'] );
+		if (isset($_GET['quiz_id'])) {
+			$quiz_id = intval($_GET['quiz_id']);
 		}
-		$sensei_grading_user_profile = $this->load_data_object( 'User_Quiz', $user_id, $quiz_id );
+		$sensei_grading_user_profile = $this->load_data_object('User_Quiz', $user_id, $quiz_id);
 		// Wrappers
-		do_action( 'grading_before_container' );
-		do_action( 'grading_wrapper_container', 'top' );
-		$this->grading_headers( array( 'nav' => 'user_quiz' ) );
+		do_action('grading_before_container');
+		do_action('grading_wrapper_container', 'top');
+		$this->grading_headers(array('nav' => 'user_quiz'));
 		?>
 		<div id="poststuff" class="sensei-grading-wrap user-profile">
 			<div class="sensei-grading-main">
@@ -225,8 +225,8 @@ 
 			</div>
 		</div>
 		<?php
-		do_action( 'grading_wrapper_container', 'bottom' );
-		do_action( 'grading_after_container' );
+		do_action('grading_wrapper_container', 'bottom');
+		do_action('grading_after_container');
 	} // End grading_user_quiz_view()
 
 	/**
@@ -234,15 +234,15 @@ 
 	 * @since  1.3.0
 	 * @return void
 	 */
-	public function grading_headers( $args = array( 'nav' => 'default' ) ) {
+	public function grading_headers($args = array('nav' => 'default')) {
 
 
-		$function = 'grading_' . $args['nav'] . '_nav';
+		$function = 'grading_'.$args['nav'].'_nav';
 		$this->$function();
 		?>
-			<p class="powered-by-woo"><?php _e( 'Powered by', 'woothemes-sensei' ); ?><a href="http://www.woothemes.com/" title="WooThemes"><img src="<?php echo Sensei()->plugin_url; ?>assets/images/woothemes.png" alt="WooThemes" /></a></p>
+			<p class="powered-by-woo"><?php _e('Powered by', 'woothemes-sensei'); ?><a href="http://www.woothemes.com/" title="WooThemes"><img src="<?php echo Sensei()->plugin_url; ?>assets/images/woothemes.png" alt="WooThemes" /></a></p>
 		<?php
-		do_action( 'sensei_grading_after_headers' );
+		do_action('sensei_grading_after_headers');
 	} // End grading_headers()
 
 	/**
@@ -251,10 +251,10 @@ 
 	 * @param $which string
 	 * @return void
 	 */
-	public function wrapper_container( $which ) {
-		if ( 'top' == $which ) {
-			?><div id="woothemes-sensei" class="wrap <?php echo esc_attr( $this->token ); ?>"><?php
-		} elseif ( 'bottom' == $which ) {
+	public function wrapper_container($which) {
+		if ('top' == $which) {
+			?><div id="woothemes-sensei" class="wrap <?php echo esc_attr($this->token); ?>"><?php
+		} elseif ('bottom' == $which) {
 			?></div><!--/#woothemes-sensei--><?php
 		} // End If Statement
 	} // End wrapper_container()
@@ -267,29 +267,29 @@ 
 	public function grading_default_nav() {
 		global  $wp_version;
 
-		$title = sprintf( '<a href="%s">%s</a>', esc_url(add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['course_id'] ) ) { 
-			$course_id = intval( $_GET['course_id'] );
-			if ( version_compare($wp_version, '4.1', '>=') ) {
-				$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
-				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
+		$title = sprintf('<a href="%s">%s</a>', esc_url(add_query_arg(array('page' => $this->page_slug), admin_url('admin.php'))), esc_html($this->name));
+		if (isset($_GET['course_id'])) { 
+			$course_id = intval($_GET['course_id']);
+			if (version_compare($wp_version, '4.1', '>=')) {
+				$url = add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id), admin_url('admin.php'));
+				$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($course_id));
 			}
 			else {
-				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', get_the_title( $course_id ) ); 
+				$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', get_the_title($course_id)); 
 			}
 		}
-		if ( isset( $_GET['lesson_id'] ) ) { 
-			$lesson_id = intval( $_GET['lesson_id'] );
-			$title .= '&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;' . get_the_title( intval( $lesson_id ) ) . '</span>'; 
+		if (isset($_GET['lesson_id'])) { 
+			$lesson_id = intval($_GET['lesson_id']);
+			$title .= '&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;'.get_the_title(intval($lesson_id)).'</span>'; 
 		}
-		if ( isset( $_GET['user_id'] ) && 0 < intval( $_GET['user_id'] ) ) {
+		if (isset($_GET['user_id']) && 0 < intval($_GET['user_id'])) {
 
-            $user_name = Sensei()->learners->get_learner_full_name( $_GET['user_id'] );
-			$title .= '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;' . $user_name . '</span>';
+            $user_name = Sensei()->learners->get_learner_full_name($_GET['user_id']);
+			$title .= '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;'.$user_name.'</span>';
 
 		} // End If Statement
 		?>
-			<h2><?php echo apply_filters( 'sensei_grading_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_grading_nav_title', $title); ?></h2>
 		<?php
 	} // End grading_default_nav()
 
@@ -301,29 +301,29 @@ 
 	public function grading_user_quiz_nav() {
 		global  $wp_version;
 
-		$title = sprintf( '<a href="%s">%s</a>', add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['quiz_id'] ) ) { 
-			$quiz_id = intval( $_GET['quiz_id'] );
-			$lesson_id = get_post_meta( $quiz_id, '_quiz_lesson', true );
-			$course_id = get_post_meta( $lesson_id, '_lesson_course', true );
-			if ( version_compare($wp_version, '4.1', '>=') ) {
-				$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
-				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
+		$title = sprintf('<a href="%s">%s</a>', add_query_arg(array('page' => $this->page_slug), admin_url('admin.php')), esc_html($this->name));
+		if (isset($_GET['quiz_id'])) { 
+			$quiz_id = intval($_GET['quiz_id']);
+			$lesson_id = get_post_meta($quiz_id, '_quiz_lesson', true);
+			$course_id = get_post_meta($lesson_id, '_lesson_course', true);
+			if (version_compare($wp_version, '4.1', '>=')) {
+				$url = add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id), admin_url('admin.php'));
+				$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($course_id));
 			}
 			else {
-				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', get_the_title( $course_id ) ); 
+				$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', get_the_title($course_id)); 
 			}
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $lesson_id ), admin_url( 'admin.php' ) );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $lesson_id ) );
+			$url = add_query_arg(array('page' => $this->page_slug, 'lesson_id' => $lesson_id), admin_url('admin.php'));
+			$title .= sprintf('&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($lesson_id));
 		}
-		if ( isset( $_GET['user'] ) && 0 < intval( $_GET['user'] ) ) {
+		if (isset($_GET['user']) && 0 < intval($_GET['user'])) {
 
-            $user_name = Sensei()->learners->get_learner_full_name( $_GET['user'] );
-			$title .= '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;' . $user_name . '</span>';
+            $user_name = Sensei()->learners->get_learner_full_name($_GET['user']);
+			$title .= '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;'.$user_name.'</span>';
 
 		} // End If Statement
 		?>
-			<h2><?php echo apply_filters( 'sensei_grading_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_grading_nav_title', $title); ?></h2>
 		<?php
 	} // End grading_user_quiz_nav()
 
@@ -332,9 +332,9 @@ 
 	 * @since  1.7.0
 	 * @return array
 	 */
-	public function get_stati( $type ) {
+	public function get_stati($type) {
 		$statuses = array();
-		switch( $type ) {
+		switch ($type) {
 			case 'course' :
 				$statuses = array(
 					'in-progress',
@@ -364,7 +364,7 @@ 
 	 * @param  array $args (default: array())
 	 * @return object
 	 */
-	public function count_statuses( $args = array() ) {
+	public function count_statuses($args = array()) {
 		global  $wpdb;
 
         /**
@@ -375,71 +375,71 @@ 
          * @since 1.8.0
          * @param array $args
          */
-        $args = apply_filters( 'sensei_count_statuses_args', $args );
+        $args = apply_filters('sensei_count_statuses_args', $args);
 
-		if ( 'course' == $args['type'] ) {
+		if ('course' == $args['type']) {
 			$type = 'sensei_course_status';
 		}
 		else {
 			$type = 'sensei_lesson_status';
 		}
-		$cache_key = 'sensei-' . $args['type'] . '-statuses';
+		$cache_key = 'sensei-'.$args['type'].'-statuses';
 
 		$query = "SELECT comment_approved, COUNT( * ) AS total FROM {$wpdb->comments} WHERE comment_type = %s ";
 
         // Restrict to specific posts
-		if ( isset( $args['post__in'] ) && !empty( $args['post__in'] ) && is_array( $args['post__in'] ) ) {
-			$query .= ' AND comment_post_ID IN (' . implode( ',', array_map( 'absint', $args['post__in'] ) ) . ')';
+		if (isset($args['post__in']) && ! empty($args['post__in']) && is_array($args['post__in'])) {
+			$query .= ' AND comment_post_ID IN ('.implode(',', array_map('absint', $args['post__in'])).')';
 		}
-		elseif ( !empty( $args['post_id'] ) ) {
-			$query .= $wpdb->prepare( ' AND comment_post_ID = %d', $args['post_id'] );
+		elseif ( ! empty($args['post_id'])) {
+			$query .= $wpdb->prepare(' AND comment_post_ID = %d', $args['post_id']);
 		}
 		// Restrict to specific users
-		if ( isset( $args['user_id'] ) && is_array( $args['user_id'] ) ) {
-			$query .= ' AND user_id IN (' . implode( ',', array_map( 'absint', $args['user_id'] ) ) . ')';
+		if (isset($args['user_id']) && is_array($args['user_id'])) {
+			$query .= ' AND user_id IN ('.implode(',', array_map('absint', $args['user_id'])).')';
 		}
-		elseif ( !empty( $args['user_id'] ) ) {
-			$query .= $wpdb->prepare( ' AND user_id = %d', $args['user_id'] );
+		elseif ( ! empty($args['user_id'])) {
+			$query .= $wpdb->prepare(' AND user_id = %d', $args['user_id']);
 		}
 		$query .= ' GROUP BY comment_approved';
 
-		$counts = wp_cache_get( $cache_key, 'counts' );
-		if ( false === $counts ) {
-			$sql = $wpdb->prepare( $query, $type );
-			$results = (array) $wpdb->get_results( $sql, ARRAY_A );
-			$counts = array_fill_keys( $this->get_stati( $type ), 0 );
+		$counts = wp_cache_get($cache_key, 'counts');
+		if (false === $counts) {
+			$sql = $wpdb->prepare($query, $type);
+			$results = (array) $wpdb->get_results($sql, ARRAY_A);
+			$counts = array_fill_keys($this->get_stati($type), 0);
 
-			foreach ( $results as $row ) {
-				$counts[ $row['comment_approved'] ] = $row['total'];
+			foreach ($results as $row) {
+				$counts[$row['comment_approved']] = $row['total'];
 			}
-			wp_cache_set( $cache_key, $counts, 'counts' );
+			wp_cache_set($cache_key, $counts, 'counts');
 		}
 
-		if( ! isset( $counts['graded'] ) ) {
+		if ( ! isset($counts['graded'])) {
 			$counts['graded'] = 0;
 		}
 
-		if( ! isset( $counts['ungraded'] ) ) {
+		if ( ! isset($counts['ungraded'])) {
 			$counts['ungraded'] = 0;
 		}
 
-		if( ! isset( $counts['passed'] ) ) {
+		if ( ! isset($counts['passed'])) {
 			$counts['passed'] = 0;
 		}
 
-		if( ! isset( $counts['failed'] ) ) {
+		if ( ! isset($counts['failed'])) {
 			$counts['failed'] = 0;
 		}
 
-		if( ! isset( $counts['in-progress'] ) ) {
+		if ( ! isset($counts['in-progress'])) {
 			$counts['in-progress'] = 0;
 		}
 
-		if( ! isset( $counts['complete'] ) ) {
+		if ( ! isset($counts['complete'])) {
 			$counts['complete'] = 0;
 		}
 
-		return apply_filters( 'sensei_count_statuses', $counts, $type );
+		return apply_filters('sensei_count_statuses', $counts, $type);
 	} // End sensei_count_statuses()
 
 	/**
@@ -447,11 +447,11 @@ 
 	 * @since  1.7.0
 	 * @return string
 	 */
-	public function courses_drop_down_html( $selected_course_id = 0 ) {
+	public function courses_drop_down_html($selected_course_id = 0) {
 
 		$html = '';
 
-		$course_args = array(   'post_type'         => 'course',
+		$course_args = array('post_type'         => 'course',
 								'posts_per_page'       => -1,
 								'orderby'           => 'title',
 								'order'             => 'ASC',
@@ -459,12 +459,12 @@ 
 								'suppress_filters'  => 0,
 								'fields'            => 'ids',
 							);
-		$courses = get_posts( apply_filters( 'sensei_grading_filter_courses', $course_args ) );
+		$courses = get_posts(apply_filters('sensei_grading_filter_courses', $course_args));
 
-		$html .= '<option value="">' . __( 'Select a course', 'woothemes-sensei' ) . '</option>';
-		if ( count( $courses ) > 0 ) {
-			foreach ($courses as $course_id){
-				$html .= '<option value="' . esc_attr( absint( $course_id ) ) . '" ' . selected( $course_id, $selected_course_id, false ) . '>' . esc_html( get_the_title( $course_id ) ) . '</option>' . "\n";
+		$html .= '<option value="">'.__('Select a course', 'woothemes-sensei').'</option>';
+		if (count($courses) > 0) {
+			foreach ($courses as $course_id) {
+				$html .= '<option value="'.esc_attr(absint($course_id)).'" '.selected($course_id, $selected_course_id, false).'>'.esc_html(get_the_title($course_id)).'</option>'."\n";
 			} // End For Loop
 		} // End If Statement
 
@@ -485,20 +485,20 @@ 
 		$course_data = array();
 		parse_str($data, $course_data);
 
-		$course_id = intval( $course_data['course_id'] );
+		$course_id = intval($course_data['course_id']);
 
-		$html = $this->lessons_drop_down_html( $course_id );
+		$html = $this->lessons_drop_down_html($course_id);
 
 		echo $html;
 		die(); // WordPress may print out a spurious zero without this can be particularly bad if using JSON
 	}
 
-	public function lessons_drop_down_html( $course_id = 0, $selected_lesson_id = 0 ) {
+	public function lessons_drop_down_html($course_id = 0, $selected_lesson_id = 0) {
 
 		$html = '';
-		if ( 0 < intval( $course_id ) ) {
+		if (0 < intval($course_id)) {
 
-			$lesson_args = array( 'post_type'       => 'lesson',
+			$lesson_args = array('post_type'       => 'lesson',
 								'posts_per_page'       => -1,
 								'orderby'           => 'title',
 								'order'             => 'ASC',
@@ -508,12 +508,12 @@ 
 								'suppress_filters'  => 0,
 								'fields'            => 'ids',
 								);
-			$lessons = get_posts( apply_filters( 'sensei_grading_filter_lessons', $lesson_args ) );
+			$lessons = get_posts(apply_filters('sensei_grading_filter_lessons', $lesson_args));
 
-			$html .= '<option value="">' . __( 'Select a lesson', 'woothemes-sensei' ) . '</option>';
-			if ( count( $lessons ) > 0 ) {
-				foreach ( $lessons as $lesson_id ){
-					$html .= '<option value="' . esc_attr( absint( $lesson_id ) ) . '" ' . selected( $lesson_id, $selected_lesson_id, false ) . '>' . esc_html( get_the_title( $lesson_id ) ) . '</option>' . "\n";
+			$html .= '<option value="">'.__('Select a lesson', 'woothemes-sensei').'</option>';
+			if (count($lessons) > 0) {
+				foreach ($lessons as $lesson_id) {
+					$html .= '<option value="'.esc_attr(absint($lesson_id)).'" '.selected($lesson_id, $selected_lesson_id, false).'>'.esc_html(get_the_title($lesson_id)).'</option>'."\n";
 				} // End For Loop
 			} // End If Statement
 
@@ -534,10 +534,10 @@ 
     public function admin_process_grading_submission() {
 
         // NEEDS REFACTOR/OPTIMISING, such as combining the various meta data stored against the sensei_user_answer entry
-        if( ! isset( $_POST['sensei_manual_grade'] )
-            || ! wp_verify_nonce( $_POST['_wp_sensei_manual_grading_nonce'], 'sensei_manual_grading' )
-            || ! isset( $_GET['quiz_id'] )
-            || $_GET['quiz_id'] != $_POST['sensei_manual_grade'] ) {
+        if ( ! isset($_POST['sensei_manual_grade'])
+            || ! wp_verify_nonce($_POST['_wp_sensei_manual_grading_nonce'], 'sensei_manual_grading')
+            || ! isset($_GET['quiz_id'])
+            || $_GET['quiz_id'] != $_POST['sensei_manual_grade']) {
 
             return false; //exit and do not grade
 
@@ -547,30 +547,30 @@ 
         $user_id = $_GET['user'];
 
 
-        $questions = Sensei_Utils::sensei_get_quiz_questions( $quiz_id );
-        $quiz_lesson_id =  Sensei()->quiz->get_lesson_id( $quiz_id );
+        $questions = Sensei_Utils::sensei_get_quiz_questions($quiz_id);
+        $quiz_lesson_id = Sensei()->quiz->get_lesson_id($quiz_id);
         $quiz_grade = 0;
         $count = 0;
         $quiz_grade_total = $_POST['quiz_grade_total'];
         $all_question_grades = array();
         $all_answers_feedback = array();
 
-        foreach( $questions as $question ) {
+        foreach ($questions as $question) {
 
             ++$count;
             $question_id = $question->ID;
 
-            if( isset( $_POST[ 'question_' . $question_id ] ) ) {
+            if (isset($_POST['question_'.$question_id])) {
 
                 $question_grade = 0;
-                if( $_POST[ 'question_' . $question_id ] == 'right' ) {
+                if ($_POST['question_'.$question_id] == 'right') {
 
-                    $question_grade = $_POST[ 'question_' . $question_id . '_grade' ];
+                    $question_grade = $_POST['question_'.$question_id.'_grade'];
 
                 }
 
                 // add data to the array that will, after the loop, be stored on the lesson status
-                $all_question_grades[ $question_id ] = $question_grade;
+                $all_question_grades[$question_id] = $question_grade;
 
                 // tally up the total quiz grade
                 $quiz_grade += $question_grade;
@@ -579,41 +579,41 @@ 
 
             // Question answer feedback / notes
             $question_feedback = '';
-            if( isset( $_POST[ 'questions_feedback' ][ $question_id ] ) ){
+            if (isset($_POST['questions_feedback'][$question_id])) {
 
-                $question_feedback = wp_unslash( $_POST[ 'questions_feedback' ][ $question_id ] );
+                $question_feedback = wp_unslash($_POST['questions_feedback'][$question_id]);
 
             }
-            $all_answers_feedback[ $question_id ] = $question_feedback;
+            $all_answers_feedback[$question_id] = $question_feedback;
 
         } // end for each $questions
 
         //store all question grades on the lesson status
-        Sensei()->quiz->set_user_grades( $all_question_grades, $quiz_lesson_id , $user_id );
+        Sensei()->quiz->set_user_grades($all_question_grades, $quiz_lesson_id, $user_id);
 
         //store the feedback from grading
-        Sensei()->quiz->save_user_answers_feedback( $all_answers_feedback, $quiz_lesson_id , $user_id );
+        Sensei()->quiz->save_user_answers_feedback($all_answers_feedback, $quiz_lesson_id, $user_id);
 
         // $_POST['all_questions_graded'] is set when all questions have been graded
         // in the class sensei grading user quiz -> display()
-        if( $_POST['all_questions_graded'] == 'yes' ) {
+        if ($_POST['all_questions_graded'] == 'yes') {
 
             // set the users total quiz grade
-			if ( 0 < intval( $quiz_grade_total ) ) {
-            $grade = abs( round( ( doubleval( $quiz_grade ) * 100 ) / ( $quiz_grade_total ), 2 ) );
+			if (0 < intval($quiz_grade_total)) {
+            $grade = abs(round((doubleval($quiz_grade) * 100) / ($quiz_grade_total), 2));
 			}
 			else {
 				$grade = 0;
 			}
-            Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id );
+            Sensei_Utils::sensei_grade_quiz($quiz_id, $grade, $user_id);
 
             // Duplicating what Frontend->sensei_complete_quiz() does
-            $pass_required = get_post_meta( $quiz_id, '_pass_required', true );
-            $quiz_passmark = abs( round( doubleval( get_post_meta( $quiz_id, '_quiz_passmark', true ) ), 2 ) );
+            $pass_required = get_post_meta($quiz_id, '_pass_required', true);
+            $quiz_passmark = abs(round(doubleval(get_post_meta($quiz_id, '_quiz_passmark', true)), 2));
             $lesson_metadata = array();
-            if ( $pass_required ) {
+            if ($pass_required) {
                 // Student has reached the pass mark and lesson is complete
-                if ( $quiz_passmark <= $grade ) {
+                if ($quiz_passmark <= $grade) {
                     $lesson_status = 'passed';
                 }
                 else {
@@ -626,9 +626,9 @@ 
             }
             $lesson_metadata['grade'] = $grade; // Technically already set as part of "WooThemes_Sensei_Utils::sensei_grade_quiz()" above
 
-            Sensei_Utils::update_lesson_status( $user_id, $quiz_lesson_id, $lesson_status, $lesson_metadata );
+            Sensei_Utils::update_lesson_status($user_id, $quiz_lesson_id, $lesson_status, $lesson_metadata);
 
-            if(  in_array( $lesson_status, array( 'passed', 'graded'  ) ) ) {
+            if (in_array($lesson_status, array('passed', 'graded'))) {
 
                 /**
                  * Summary.
@@ -640,27 +640,27 @@ 
                  * @param int  $user_id
                  * @param int $quiz_lesson_id
                  */
-                do_action( 'sensei_user_lesson_end', $user_id, $quiz_lesson_id );
+                do_action('sensei_user_lesson_end', $user_id, $quiz_lesson_id);
 
             } // end if in_array
 
         }// end if $_POST['all_que...
 
-        if( isset( $_POST['sensei_grade_next_learner'] ) && strlen( $_POST['sensei_grade_next_learner'] ) > 0 ) {
+        if (isset($_POST['sensei_grade_next_learner']) && strlen($_POST['sensei_grade_next_learner']) > 0) {
 
-            $load_url = add_query_arg( array( 'message' => 'graded' ) );
+            $load_url = add_query_arg(array('message' => 'graded'));
 
-        } elseif ( isset( $_POST['_wp_http_referer'] ) ) {
+        } elseif (isset($_POST['_wp_http_referer'])) {
 
-            $load_url = add_query_arg( array( 'message' => 'graded' ), $_POST['_wp_http_referer'] );
+            $load_url = add_query_arg(array('message' => 'graded'), $_POST['_wp_http_referer']);
 
         } else {
 
-            $load_url = add_query_arg( array( 'message' => 'graded' ) );
+            $load_url = add_query_arg(array('message' => 'graded'));
 
         }
 
-        wp_safe_redirect( esc_url_raw( $load_url ) );
+        wp_safe_redirect(esc_url_raw($load_url));
         exit;
 
     } // end admin_process_grading_submission
@@ -671,13 +671,13 @@ 
 		$lesson_data = array();
 		parse_str($data, $lesson_data);
 
-		$lesson_id = intval( $lesson_data['lesson_id'] );
-		$course_id = intval( $lesson_data['course_id'] );
-		$grading_view = sanitize_text_field( $lesson_data['view'] );
+		$lesson_id = intval($lesson_data['lesson_id']);
+		$course_id = intval($lesson_data['course_id']);
+		$grading_view = sanitize_text_field($lesson_data['view']);
 
 		$redirect_url = '';
-		if ( 0 < $lesson_id && 0 < $course_id ) {
-			$redirect_url = esc_url_raw( apply_filters( 'sensei_ajax_redirect_url', add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $lesson_id, 'course_id' => $course_id, 'view' => $grading_view ), admin_url( 'admin.php' ) ) ) );
+		if (0 < $lesson_id && 0 < $course_id) {
+			$redirect_url = esc_url_raw(apply_filters('sensei_ajax_redirect_url', add_query_arg(array('page' => $this->page_slug, 'lesson_id' => $lesson_id, 'course_id' => $course_id, 'view' => $grading_view), admin_url('admin.php'))));
 		} // End If Statement
 
 		echo $redirect_url;
@@ -685,11 +685,11 @@ 
 	}
 
 	public function add_grading_notices() {
-		if( isset( $_GET['page'] ) && $this->page_slug == $_GET['page'] && isset( $_GET['message'] ) && $_GET['message'] ) {
-			if( 'graded' == $_GET['message'] ) {
+		if (isset($_GET['page']) && $this->page_slug == $_GET['page'] && isset($_GET['message']) && $_GET['message']) {
+			if ('graded' == $_GET['message']) {
 				$msg = array(
 					'updated',
-					__( 'Quiz Graded Successfully!', 'woothemes-sensei' ),
+					__('Quiz Graded Successfully!', 'woothemes-sensei'),
 				);
 			}
 			?>
@@ -701,9 +701,9 @@ 
 	}
 
 	public function sensei_grading_notices() {
-		if ( isset( $_GET['action'] ) && 'graded' == $_GET['action'] ) {
+		if (isset($_GET['action']) && 'graded' == $_GET['action']) {
 			echo '<div class="grading-notice updated">';
-				echo '<p>' . __( 'Quiz Graded Successfully!', 'woothemes-sensei' ) . '</p>';
+				echo '<p>'.__('Quiz Graded Successfully!', 'woothemes-sensei').'</p>';
 			echo '</div>';
 		} // End If Statement
 	} // End sensei_grading_notices()
@@ -726,16 +726,16 @@ 
      *
      * @return int $quiz_grade total sum of all question grades
      */
-    public static function grade_quiz_auto( $quiz_id = 0, $submitted = array(), $total_questions = 0, $quiz_grade_type = 'auto' ) {
+    public static function grade_quiz_auto($quiz_id = 0, $submitted = array(), $total_questions = 0, $quiz_grade_type = 'auto') {
 
-        if( ! ( intval( $quiz_id ) > 0 )  || ! $submitted
-            || $quiz_grade_type != 'auto' ) {
+        if ( ! (intval($quiz_id) > 0) || ! $submitted
+            || $quiz_grade_type != 'auto') {
             return false; // exit early
         }
 
 
         $user_id = get_current_user_id();
-        $lesson_id =  Sensei()->quiz->get_lesson_id(  $quiz_id ) ;
+        $lesson_id = Sensei()->quiz->get_lesson_id($quiz_id);
         $quiz_autogradable = true;
 
         /**
@@ -749,23 +749,23 @@ 
          *      'gap-fill'.
          * }
          */
-        $autogradable_question_types = apply_filters( 'sensei_autogradable_question_types', array( 'multiple-choice', 'boolean', 'gap-fill' ) );
+        $autogradable_question_types = apply_filters('sensei_autogradable_question_types', array('multiple-choice', 'boolean', 'gap-fill'));
 
         $grade_total = 0;
         $all_question_grades = array();
-        foreach( $submitted as $question_id => $answer ) {
+        foreach ($submitted as $question_id => $answer) {
 
             // check if the question is autogradable, either by type, or because the grade is 0
-            $question_type = Sensei()->question->get_question_type( $question_id );
-			$achievable_grade = Sensei()->question->get_question_grade( $question_id );
+            $question_type = Sensei()->question->get_question_type($question_id);
+			$achievable_grade = Sensei()->question->get_question_grade($question_id);
 			// Question has a zero grade, so skip grading
-			if ( 0 == $achievable_grade ) {
-				$all_question_grades[ $question_id ] = $achievable_grade;
+			if (0 == $achievable_grade) {
+				$all_question_grades[$question_id] = $achievable_grade;
 			}
-            elseif ( in_array( $question_type, $autogradable_question_types ) ) {
+            elseif (in_array($question_type, $autogradable_question_types)) {
                 // Get user question grade
-                $question_grade = Sensei_Utils::sensei_grade_question_auto( $question_id, $question_type, $answer, $user_id );
-                $all_question_grades[ $question_id ] = $question_grade;
+                $question_grade = Sensei_Utils::sensei_grade_question_auto($question_id, $question_type, $answer, $user_id);
+                $all_question_grades[$question_id] = $question_grade;
                 $grade_total += $question_grade;
 
             } else {
@@ -778,27 +778,27 @@ 
         }// end for each question
 
         // Only if the whole quiz was autogradable do we set a grade
-        if ( $quiz_autogradable ) {
+        if ($quiz_autogradable) {
 
-            $quiz_total = Sensei_Utils::sensei_get_quiz_total( $quiz_id );
+            $quiz_total = Sensei_Utils::sensei_get_quiz_total($quiz_id);
 			// Check for zero total from grades
-			if ( 0 < $quiz_total ) {
-            $grade = abs( round( ( doubleval( $grade_total ) * 100 ) / ( $quiz_total ), 2 ) );
+			if (0 < $quiz_total) {
+            $grade = abs(round((doubleval($grade_total) * 100) / ($quiz_total), 2));
 			}
 			else {
 				$grade = 0;
 			}
-            Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id, $quiz_grade_type );
+            Sensei_Utils::sensei_grade_quiz($quiz_id, $grade, $user_id, $quiz_grade_type);
 
         } else {
 
-            $grade = new WP_Error( 'autograde', __( 'This quiz is not able to be automatically graded.', 'woothemes-sensei' ) );
+            $grade = new WP_Error('autograde', __('This quiz is not able to be automatically graded.', 'woothemes-sensei'));
 
         }
 
         // store the auto gradable grades. If the quiz is not auto gradable the grades can be use as the default
         // when doing manual grading.
-        Sensei()->quiz-> set_user_grades( $all_question_grades, $lesson_id, $user_id );
+        Sensei()->quiz-> set_user_grades($all_question_grades, $lesson_id, $user_id);
 
         return $grade;
 
@@ -818,22 +818,22 @@ 
      *
      * @return int $question_grade
      */
-    public static function grade_question_auto( $question_id = 0, $question_type = '', $answer = '', $user_id = 0 ) {
+    public static function grade_question_auto($question_id = 0, $question_type = '', $answer = '', $user_id = 0) {
 
-        if( intval( $user_id ) == 0 ) {
+        if (intval($user_id) == 0) {
 
             $user_id = get_current_user_id();
 
         }
 
-        if( ! ( intval( $question_id ) > 0 ) ) {
+        if ( ! (intval($question_id) > 0)) {
 
             return false;
 
         }
 
 
-        Sensei()->question->get_question_type( $question_id );
+        Sensei()->question->get_question_type($question_id);
 
         /**
          * Applying a grade before the auto grading takes place.
@@ -847,42 +847,42 @@ 
          * @param string $question_type one of the Sensei question type.
          * @param string $answer user supplied question answer
          */
-        $question_grade = apply_filters( 'sensei_pre_grade_question_auto', false, $question_id, $question_type, $answer );
+        $question_grade = apply_filters('sensei_pre_grade_question_auto', false, $question_id, $question_type, $answer);
 
-        if ( false !== $question_grade ) {
+        if (false !== $question_grade) {
 
             return $question_grade;
 
         }
 
         // auto grading core
-        if( in_array( $question_type ,  array( 'multiple-choice'  , 'boolean'  ) )   ){
+        if (in_array($question_type, array('multiple-choice', 'boolean'))) {
 
-            $right_answer = (array) get_post_meta( $question_id, '_question_right_answer', true );
+            $right_answer = (array) get_post_meta($question_id, '_question_right_answer', true);
 
-            if( 0 == get_magic_quotes_gpc() ) {
-                $answer = wp_unslash( $answer );
+            if (0 == get_magic_quotes_gpc()) {
+                $answer = wp_unslash($answer);
             }
             $answer = (array) $answer;
-            if ( is_array( $right_answer ) && count( $right_answer ) == count( $answer ) ) {
+            if (is_array($right_answer) && count($right_answer) == count($answer)) {
                 // Loop through all answers ensure none are 'missing'
                 $all_correct = true;
-                foreach ( $answer as $check_answer ) {
-                    if ( !in_array( $check_answer, $right_answer ) ) {
+                foreach ($answer as $check_answer) {
+                    if ( ! in_array($check_answer, $right_answer)) {
                         $all_correct = false;
                     }
                 }
                 // If all correct then grade
-                if ( $all_correct ) {
-                    $question_grade = Sensei()->question->get_question_grade( $question_id );
+                if ($all_correct) {
+                    $question_grade = Sensei()->question->get_question_grade($question_id);
                 }
             }
 
-        } elseif( 'gap-fill' == $question_type ){
+        } elseif ('gap-fill' == $question_type) {
 
-            $question_grade = self::grade_gap_fill_question( $question_id ,$answer );
+            $question_grade = self::grade_gap_fill_question($question_id, $answer);
 
-        } else{
+        } else {
 
             /**
              * Grading questions that are not auto gradable.
@@ -895,7 +895,7 @@ 
              * @param string $question_type one of the Sensei question type.
              * @param string $answer user supplied question answer
              */
-            $question_grade = ( int ) apply_filters( 'sensei_grade_question_auto', $question_grade, $question_id, $question_type, $answer );
+            $question_grade = (int) apply_filters('sensei_grade_question_auto', $question_grade, $question_id, $question_type, $answer);
 
         } // end if $question_type
 
@@ -911,13 +911,13 @@ 
      *
      * @return bool | int false or the grade given to the user answer
      */
-    public static function grade_gap_fill_question( $question_id, $user_answer ){
+    public static function grade_gap_fill_question($question_id, $user_answer) {
 
-        $right_answer = get_post_meta( $question_id, '_question_right_answer', true );
-        $gapfill_array = explode( '||', $right_answer );
+        $right_answer = get_post_meta($question_id, '_question_right_answer', true);
+        $gapfill_array = explode('||', $right_answer);
 
-        if( 0 == get_magic_quotes_gpc() ) { // deprecated from PHP 5.4 but we still support PHP 5.2
-            $user_answer = wp_unslash( $user_answer );
+        if (0 == get_magic_quotes_gpc()) { // deprecated from PHP 5.4 but we still support PHP 5.2
+            $user_answer = wp_unslash($user_answer);
         }
 
         /**
@@ -930,53 +930,53 @@ 
          *
          * @since 1.9.0
          */
-        $do_case_sensitive_comparison = apply_filters('sensei_gap_fill_case_sensitive_grading', false );
+        $do_case_sensitive_comparison = apply_filters('sensei_gap_fill_case_sensitive_grading', false);
 
-        if( $do_case_sensitive_comparison ){
+        if ($do_case_sensitive_comparison) {
 
             // Case Sensitive Check that the 'gap' is "exactly" equal to the given answer
-            if ( trim(($gapfill_array[1])) == trim( $user_answer ) ) {
+            if (trim(($gapfill_array[1])) == trim($user_answer)) {
 
-                return Sensei()->question->get_question_grade( $question_id );
+                return Sensei()->question->get_question_grade($question_id);
 
-            } else if (@preg_match('/' . $gapfill_array[1] . '/i', null) !== FALSE) {
+            } else if (@preg_match('/'.$gapfill_array[1].'/i', null) !== FALSE) {
 
-                if (preg_match('/' . $gapfill_array[1] . '/i', $user_answer)) {
+                if (preg_match('/'.$gapfill_array[1].'/i', $user_answer)) {
 
                     return Sensei()->question->get_question_grade($question_id);
 
-                }else{
+                } else {
 
                     return false;
 
                 }
 
-            }else{
+            } else {
 
                 return false;
 
             }
 
-        }else{
+        } else {
 
             // Case Sensitive Check that the 'gap' is "exactly" equal to the given answer
-            if ( trim(strtolower($gapfill_array[1])) == trim(strtolower( $user_answer )) ) {
+            if (trim(strtolower($gapfill_array[1])) == trim(strtolower($user_answer))) {
 
-               return Sensei()->question->get_question_grade( $question_id );
+               return Sensei()->question->get_question_grade($question_id);
 
-            } else if (@preg_match('/' . $gapfill_array[1] . '/i', null) !== FALSE) {
+            } else if (@preg_match('/'.$gapfill_array[1].'/i', null) !== FALSE) {
 
-                if (preg_match('/' . $gapfill_array[1] . '/i', $user_answer)) {
+                if (preg_match('/'.$gapfill_array[1].'/i', $user_answer)) {
 
-                    return  Sensei()->question->get_question_grade( $question_id );
+                    return  Sensei()->question->get_question_grade($question_id);
 
-                }else{
+                } else {
 
                     return false;
 
                 }
 
-            }else{
+            } else {
 
                 return false;
 
@@ -993,4 +993,4 @@ 
  * for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Grading extends Sensei_Grading{}
+class WooThemes_Sensei_Grading extends Sensei_Grading {}

includes/class-sensei-learners.php

Doc Comments +1 added lines

@@ -21,6 +21,7 @@
 	/**
 	 * Constructor
 	 * @since  1.6.0
+	 * @param string $file
 	 * @return  void
 	 */
 	public function __construct ( $file ) {

Spacing +145 added lines, -145 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Learners Class
@@ -23,30 +23,30 @@ 
 	 * @since  1.6.0
 	 * @return  void
 	 */
-	public function __construct ( $file ) {
-		$this->name = __( 'Learner Management', 'woothemes-sensei' );;
+	public function __construct($file) {
+		$this->name = __('Learner Management', 'woothemes-sensei'); ;
 		$this->file = $file;
 		$this->page_slug = 'sensei_learners';
 
 		// Admin functions
-		if ( is_admin() ) {
-			add_action( 'admin_menu', array( $this, 'learners_admin_menu' ), 30);
-			add_action( 'learners_wrapper_container', array( $this, 'wrapper_container'  ) );
-			if ( isset( $_GET['page'] ) && ( $_GET['page'] == $this->page_slug ) ) {
-				add_action( 'admin_print_scripts', array( $this, 'enqueue_scripts' ) );
-				add_action( 'admin_print_styles', array( $this, 'enqueue_styles' ) );
+		if (is_admin()) {
+			add_action('admin_menu', array($this, 'learners_admin_menu'), 30);
+			add_action('learners_wrapper_container', array($this, 'wrapper_container'));
+			if (isset($_GET['page']) && ($_GET['page'] == $this->page_slug)) {
+				add_action('admin_print_scripts', array($this, 'enqueue_scripts'));
+				add_action('admin_print_styles', array($this, 'enqueue_styles'));
 			}
 
-			add_action( 'admin_init', array( $this, 'add_new_learners' ) );
+			add_action('admin_init', array($this, 'add_new_learners'));
 
-			add_action( 'admin_notices', array( $this, 'add_learner_notices' ) );
+			add_action('admin_notices', array($this, 'add_learner_notices'));
 		} // End If Statement
 
 		// Ajax functions
-		if ( is_admin() ) {
-			add_action( 'wp_ajax_get_redirect_url_learners', array( $this, 'get_redirect_url' ) );
-			add_action( 'wp_ajax_remove_user_from_post', array( $this, 'remove_user_from_post' ) );
-			add_action( 'wp_ajax_sensei_json_search_users', array( $this, 'json_search_users' ) );
+		if (is_admin()) {
+			add_action('wp_ajax_get_redirect_url_learners', array($this, 'get_redirect_url'));
+			add_action('wp_ajax_remove_user_from_post', array($this, 'remove_user_from_post'));
+			add_action('wp_ajax_sensei_json_search_users', array($this, 'json_search_users'));
 		}
 	} // End __construct()
 
@@ -59,8 +59,8 @@ 
 	public function learners_admin_menu() {
 		global $menu;
 
-		if ( current_user_can( 'manage_sensei_grades' ) ) {
-			$learners_page = add_submenu_page( 'sensei', $this->name, $this->name, 'manage_sensei_grades', $this->page_slug, array( $this, 'learners_page' ) );
+		if (current_user_can('manage_sensei_grades')) {
+			$learners_page = add_submenu_page('sensei', $this->name, $this->name, 'manage_sensei_grades', $this->page_slug, array($this, 'learners_page'));
 		}
 
 	} // End learners_admin_menu()
@@ -73,26 +73,26 @@ 
 	 * @since 1.6.0
 	 * @return void
 	 */
-	public function enqueue_scripts () {
+	public function enqueue_scripts() {
 
-		$suffix = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ? '' : '.min';
+		$suffix = defined('SCRIPT_DEBUG') && SCRIPT_DEBUG ? '' : '.min';
 
 		// Load Learners JS
-		wp_enqueue_script( 'sensei-learners-general',
-            Sensei()->plugin_url . 'assets/js/learners-general' . $suffix . '.js',
-                            array('jquery','select2','sensei-chosen-ajax' ), Sensei()->version, true );
+		wp_enqueue_script('sensei-learners-general',
+            Sensei()->plugin_url.'assets/js/learners-general'.$suffix.'.js',
+                            array('jquery', 'select2', 'sensei-chosen-ajax'), Sensei()->version, true);
 
 		$data = array(
-			'remove_generic_confirm' => __( 'Are you sure you want to remove this user?', 'woothemes-sensei' ),
-			'remove_from_lesson_confirm' => __( 'Are you sure you want to remove the user from this lesson?', 'woothemes-sensei' ),
-			'remove_from_course_confirm' => __( 'Are you sure you want to remove the user from this course?', 'woothemes-sensei' ),
-            'remove_from_purchased_course_confirm' => __( 'Are you sure you want to remove the user from this course? This order associate with this will also be set to canceled.', 'woothemes-sensei' ),
-			'remove_user_from_post_nonce' => wp_create_nonce( 'remove_user_from_post_nonce' ),
-            'search_users_nonce' => wp_create_nonce( 'search-users' ),
-            'selectplaceholder'=> __( 'Select Learner', 'woothemes-sensei' )
+			'remove_generic_confirm' => __('Are you sure you want to remove this user?', 'woothemes-sensei'),
+			'remove_from_lesson_confirm' => __('Are you sure you want to remove the user from this lesson?', 'woothemes-sensei'),
+			'remove_from_course_confirm' => __('Are you sure you want to remove the user from this course?', 'woothemes-sensei'),
+            'remove_from_purchased_course_confirm' => __('Are you sure you want to remove the user from this course? This order associate with this will also be set to canceled.', 'woothemes-sensei'),
+			'remove_user_from_post_nonce' => wp_create_nonce('remove_user_from_post_nonce'),
+            'search_users_nonce' => wp_create_nonce('search-users'),
+            'selectplaceholder'=> __('Select Learner', 'woothemes-sensei')
 		);
 
-		wp_localize_script( 'sensei-learners-general', 'woo_learners_general_data', $data );
+		wp_localize_script('sensei-learners-general', 'woo_learners_general_data', $data);
 
 	} // End enqueue_scripts()
 
@@ -104,9 +104,9 @@ 
 	 * @since 1.6.0
 	 * @return void
 	 */
-	public function enqueue_styles () {
+	public function enqueue_styles() {
 
-		wp_enqueue_style( Sensei()->token . '-admin' );
+		wp_enqueue_style(Sensei()->token.'-admin');
 
 	} // End enqueue_styles()
 
@@ -118,11 +118,11 @@ 
 	public function load_data_table_files() {
 
 		// Load Learners Classes
-		$classes_to_load = array(	'list-table',
+		$classes_to_load = array('list-table',
 									'learners-main',
 									);
-		foreach ( $classes_to_load as $class_file ) {
-			Sensei()->load_class( $class_file );
+		foreach ($classes_to_load as $class_file) {
+			Sensei()->load_class($class_file);
 		} // End For Loop
 	} // End load_data_table_files()
 
@@ -134,15 +134,15 @@ 
 	 * @param  undefined  $optional_data optional constructor arguments
 	 * @return object                 class instance object
 	 */
-	public function load_data_object( $name = '', $data = 0, $optional_data = null ) {
+	public function load_data_object($name = '', $data = 0, $optional_data = null) {
 		// Load Analysis data
-		$object_name = 'WooThemes_Sensei_Learners_' . $name;
-		if ( is_null($optional_data) ) {
-			$sensei_learners_object = new $object_name( $data );
+		$object_name = 'WooThemes_Sensei_Learners_'.$name;
+		if (is_null($optional_data)) {
+			$sensei_learners_object = new $object_name($data);
 		} else {
-			$sensei_learners_object = new $object_name( $data, $optional_data );
+			$sensei_learners_object = new $object_name($data, $optional_data);
 		} // End If Statement
-		if ( 'Main' == $name ) {
+		if ('Main' == $name) {
 			$sensei_learners_object->prepare_items();
 		} // End If Statement
 		return $sensei_learners_object;
@@ -159,16 +159,16 @@ 
 		// Load Learners data
 		$course_id = 0;
 		$lesson_id = 0;
-		if( isset( $_GET['course_id'] ) ) {
-			$course_id = intval( $_GET['course_id'] );
+		if (isset($_GET['course_id'])) {
+			$course_id = intval($_GET['course_id']);
 		}
-		if( isset( $_GET['lesson_id'] ) ) {
-			$lesson_id = intval( $_GET['lesson_id'] );
+		if (isset($_GET['lesson_id'])) {
+			$lesson_id = intval($_GET['lesson_id']);
 		}
-		$sensei_learners_main = $this->load_data_object( 'Main', $course_id, $lesson_id );
+		$sensei_learners_main = $this->load_data_object('Main', $course_id, $lesson_id);
 		// Wrappers
-		do_action( 'learners_before_container' );
-		do_action( 'learners_wrapper_container', 'top' );
+		do_action('learners_before_container');
+		do_action('learners_wrapper_container', 'top');
 		$this->learners_headers();
 		?>
 		<div id="poststuff" class="sensei-learners-wrap">
@@ -176,12 +176,12 @@ 
 				<?php $sensei_learners_main->display(); ?>
 			</div>
 			<div class="sensei-learners-extra">
-				<?php do_action( 'sensei_learners_extra' ); ?>
+				<?php do_action('sensei_learners_extra'); ?>
 			</div>
 		</div>
 		<?php
-		do_action( 'learners_wrapper_container', 'bottom' );
-		do_action( 'learners_after_container' );
+		do_action('learners_wrapper_container', 'bottom');
+		do_action('learners_after_container');
 	} // End learners_default_view()
 
 	/**
@@ -189,15 +189,15 @@ 
 	 * @since  1.6.0
 	 * @return void
 	 */
-	public function learners_headers( $args = array( 'nav' => 'default' ) ) {
+	public function learners_headers($args = array('nav' => 'default')) {
 
 
-		$function = 'learners_' . $args['nav'] . '_nav';
+		$function = 'learners_'.$args['nav'].'_nav';
 		$this->$function();
 		?>
-			<p class="powered-by-woo"><?php _e( 'Powered by', 'woothemes-sensei' ); ?><a href="http://www.woothemes.com/" title="WooThemes"><img src="<?php echo Sensei()->plugin_url; ?>assets/images/woothemes.png" alt="WooThemes" /></a></p>
+			<p class="powered-by-woo"><?php _e('Powered by', 'woothemes-sensei'); ?><a href="http://www.woothemes.com/" title="WooThemes"><img src="<?php echo Sensei()->plugin_url; ?>assets/images/woothemes.png" alt="WooThemes" /></a></p>
 		<?php
-		do_action( 'sensei_learners_after_headers' );
+		do_action('sensei_learners_after_headers');
 	} // End learners_headers()
 
 	/**
@@ -206,10 +206,10 @@ 
 	 * @param $which string
 	 * @return void
 	 */
-	public function wrapper_container( $which ) {
-		if ( 'top' == $which ) {
-			?><div id="woothemes-sensei" class="wrap <?php echo esc_attr( $this->token ); ?>"><?php
-		} elseif ( 'bottom' == $which ) {
+	public function wrapper_container($which) {
+		if ('top' == $which) {
+			?><div id="woothemes-sensei" class="wrap <?php echo esc_attr($this->token); ?>"><?php
+		} elseif ('bottom' == $which) {
 			?></div><!--/#woothemes-sensei--><?php
 		} // End If Statement
 	} // End wrapper_container()
@@ -220,18 +220,18 @@ 
 	 * @return void
 	 */
 	public function learners_default_nav() {
-		$title = sprintf( '<a href="%s">%s</a>', esc_url( add_query_arg( array( 'page' => $this->page_slug ), admin_url( 'admin.php' ) ) ), esc_html( $this->name ) );
-		if ( isset( $_GET['course_id'] ) ) { 
-			$course_id = intval( $_GET['course_id'] );
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id, 'view' => 'learners' ), admin_url( 'admin.php' ) );
-			$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
+		$title = sprintf('<a href="%s">%s</a>', esc_url(add_query_arg(array('page' => $this->page_slug), admin_url('admin.php'))), esc_html($this->name));
+		if (isset($_GET['course_id'])) { 
+			$course_id = intval($_GET['course_id']);
+			$url = add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id, 'view' => 'learners'), admin_url('admin.php'));
+			$title .= sprintf('&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url($url), get_the_title($course_id));
 		}
-		if ( isset( $_GET['lesson_id'] ) ) { 
-			$lesson_id = intval( $_GET['lesson_id'] );
-			$title .= '&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;' . get_the_title( intval( $lesson_id ) ) . '</span>'; 
+		if (isset($_GET['lesson_id'])) { 
+			$lesson_id = intval($_GET['lesson_id']);
+			$title .= '&nbsp;&nbsp;<span class="lesson-title">&gt;&nbsp;&nbsp;'.get_the_title(intval($lesson_id)).'</span>'; 
 		}
 		?>
-			<h2><?php echo apply_filters( 'sensei_learners_nav_title', $title ); ?></h2>
+			<h2><?php echo apply_filters('sensei_learners_nav_title', $title); ?></h2>
 		<?php
 	} // End learners_default_nav()
 
@@ -240,13 +240,13 @@ 
 		// Parse POST data
 		$data = $_POST['data'];
 		$course_data = array();
-		parse_str( $data, $course_data );
+		parse_str($data, $course_data);
 
-		$course_cat = intval( $course_data['course_cat'] );
+		$course_cat = intval($course_data['course_cat']);
 
-		$redirect_url = apply_filters( 'sensei_ajax_redirect_url', add_query_arg( array( 'page' => $this->page_slug, 'course_cat' => $course_cat ), admin_url( 'admin.php' ) ) );
+		$redirect_url = apply_filters('sensei_ajax_redirect_url', add_query_arg(array('page' => $this->page_slug, 'course_cat' => $course_cat), admin_url('admin.php')));
 
-		echo esc_url_raw( $redirect_url );
+		echo esc_url_raw($redirect_url);
 		die();
 	}
 
@@ -257,38 +257,38 @@ 
 
 		// Security check
 		$nonce = '';
-		if ( isset($_POST['remove_user_from_post_nonce']) ) {
-			$nonce = esc_html( $_POST['remove_user_from_post_nonce'] );
+		if (isset($_POST['remove_user_from_post_nonce'])) {
+			$nonce = esc_html($_POST['remove_user_from_post_nonce']);
 		}
-		if ( ! wp_verify_nonce( $nonce, 'remove_user_from_post_nonce' ) ) {
-			die( $return );
+		if ( ! wp_verify_nonce($nonce, 'remove_user_from_post_nonce')) {
+			die($return);
 		}
 
 		// Parse POST data
 		$data = $_POST['data'];
 		$action_data = array();
-		parse_str( $data, $action_data );
+		parse_str($data, $action_data);
 
-		if( $action_data['user_id'] && $action_data['post_id'] && $action_data['post_type'] ) {
+		if ($action_data['user_id'] && $action_data['post_id'] && $action_data['post_type']) {
 
-			$user_id = intval( $action_data['user_id'] );
-			$post_id = intval( $action_data['post_id'] );
-			$post_type = sanitize_text_field( $action_data['post_type'] );
-            $order_id = sanitize_text_field( $action_data['order_id'] );
+			$user_id = intval($action_data['user_id']);
+			$post_id = intval($action_data['post_id']);
+			$post_type = sanitize_text_field($action_data['post_type']);
+            $order_id = sanitize_text_field($action_data['order_id']);
 
-			$user = get_userdata( $user_id );
+			$user = get_userdata($user_id);
 
-			switch( $post_type ) {
+			switch ($post_type) {
 
 				case 'course':
-					$removed = Sensei_Utils::sensei_remove_user_from_course( $post_id, $user_id );
+					$removed = Sensei_Utils::sensei_remove_user_from_course($post_id, $user_id);
 
-                    if( ! empty( $order_id ) && Sensei_WC::is_woocommerce_active()  ){
+                    if ( ! empty($order_id) && Sensei_WC::is_woocommerce_active()) {
 
                         $order = new WC_Order($order_id);
 
-                        if (!empty($order)) {
-                            $order->update_status( 'cancelled' );
+                        if ( ! empty($order)) {
+                            $order->update_status('cancelled');
                         }
 
                     }
@@ -296,126 +296,126 @@ 
 				break;
 
 				case 'lesson':
-					$removed = Sensei_Utils::sensei_remove_user_from_lesson( $post_id, $user_id );
+					$removed = Sensei_Utils::sensei_remove_user_from_lesson($post_id, $user_id);
 				break;
 
 			}
 
-			if( $removed ) {
+			if ($removed) {
 				$return = 'removed';
 			}
 
 		}
 
-		die( $return );
+		die($return);
 	}
 
 	public function json_search_users() {
 
 
-		check_ajax_referer( 'search-users', 'security' );
+		check_ajax_referer('search-users', 'security');
 
-		$term = sanitize_text_field( stripslashes( $_GET['term'] ) );
+		$term = sanitize_text_field(stripslashes($_GET['term']));
 
-		if ( empty( $term ) ) {
+		if (empty($term)) {
 			die();
 		}
 
-		$default = isset( $_GET['default'] ) ? $_GET['default'] : __( 'None', 'woocommerce' );
+		$default = isset($_GET['default']) ? $_GET['default'] : __('None', 'woocommerce');
 
-		$found_users = array( '' => $default );
+		$found_users = array('' => $default);
 
-		$users_query = new WP_User_Query( apply_filters( 'sensei_json_search_users_query', array(
+		$users_query = new WP_User_Query(apply_filters('sensei_json_search_users_query', array(
 			'fields'         => 'all',
 			'orderby'        => 'display_name',
-			'search'         => '*' . $term . '*',
-			'search_columns' => array( 'ID', 'user_login', 'user_email', 'user_nicename','user_firstname','user_lastname' )
-		), $term ) );
+			'search'         => '*'.$term.'*',
+			'search_columns' => array('ID', 'user_login', 'user_email', 'user_nicename', 'user_firstname', 'user_lastname')
+		), $term));
 
 		$users = $users_query->get_results();
 
-		if ( $users ) {
-			foreach ( $users as $user ) {
-                $full_name = Sensei()->learners->get_learner_full_name( $user->ID );
+		if ($users) {
+			foreach ($users as $user) {
+                $full_name = Sensei()->learners->get_learner_full_name($user->ID);
 
-                if( trim($user->display_name ) == trim( $full_name ) ){
+                if (trim($user->display_name) == trim($full_name)) {
 
                     $name = $full_name;
 
-                }else{
+                } else {
 
-                    $name = $full_name . ' ['. $user->display_name .']';
+                    $name = $full_name.' ['.$user->display_name.']';
 
                 }
 
-                $found_users[ $user->ID ] = $name  . ' (#' . $user->ID . ' &ndash; ' . sanitize_email( $user->user_email ) . ')';
+                $found_users[$user->ID] = $name.' (#'.$user->ID.' &ndash; '.sanitize_email($user->user_email).')';
 			}
 		}
 
-		wp_send_json( $found_users );
+		wp_send_json($found_users);
 	}
 
 	public function add_new_learners() {
 
 		$result = false;
 
-		if( ! isset( $_POST['add_learner_submit'] ) ) return $result;
+		if ( ! isset($_POST['add_learner_submit'])) return $result;
 
-		if ( ! isset( $_POST['add_learner_nonce'] ) || ! wp_verify_nonce( $_POST['add_learner_nonce'], 'add_learner_to_sensei' ) ) return $result;
+		if ( ! isset($_POST['add_learner_nonce']) || ! wp_verify_nonce($_POST['add_learner_nonce'], 'add_learner_to_sensei')) return $result;
 
-		if( ( ! isset( $_POST['add_user_id'] ) || '' ==  $_POST['add_user_id'] ) || ! isset( $_POST['add_post_type'] ) || ! isset( $_POST['add_course_id'] ) || ! isset( $_POST['add_lesson_id'] ) ) return $result;
+		if (( ! isset($_POST['add_user_id']) || '' == $_POST['add_user_id']) || ! isset($_POST['add_post_type']) || ! isset($_POST['add_course_id']) || ! isset($_POST['add_lesson_id'])) return $result;
 
 		$post_type = $_POST['add_post_type'];
-		$user_id = absint( $_POST['add_user_id'] );
-		$course_id = absint( $_POST['add_course_id'] );
+		$user_id = absint($_POST['add_user_id']);
+		$course_id = absint($_POST['add_course_id']);
 
-		switch( $post_type ) {
+		switch ($post_type) {
 			case 'course':
 
-				$result = Sensei_Utils::user_start_course( $user_id, $course_id );
+				$result = Sensei_Utils::user_start_course($user_id, $course_id);
 
 				// Complete each lesson if course is set to be completed
-				if( isset( $_POST['add_complete_course'] ) && 'yes' == $_POST['add_complete_course'] ) {
+				if (isset($_POST['add_complete_course']) && 'yes' == $_POST['add_complete_course']) {
 
-					$lesson_ids = Sensei()->course->course_lessons( $course_id, 'any', 'ids' );
+					$lesson_ids = Sensei()->course->course_lessons($course_id, 'any', 'ids');
 
-					foreach( $lesson_ids as $id ) {
-						Sensei_Utils::sensei_start_lesson( $id, $user_id, true );
+					foreach ($lesson_ids as $id) {
+						Sensei_Utils::sensei_start_lesson($id, $user_id, true);
 					}
 
 					// Updates the Course status and it's meta data
-					Sensei_Utils::user_complete_course( $course_id, $user_id );
+					Sensei_Utils::user_complete_course($course_id, $user_id);
 
-					do_action( 'sensei_user_course_end', $user_id, $course_id );
+					do_action('sensei_user_course_end', $user_id, $course_id);
 				}
 
 			break;
 
 			case 'lesson':
-                $lesson_id = absint( $_POST['add_lesson_id'] );
+                $lesson_id = absint($_POST['add_lesson_id']);
 				$complete = false;
-				if( isset( $_POST['add_complete_lesson'] ) && 'yes' == $_POST['add_complete_lesson'] ) {
+				if (isset($_POST['add_complete_lesson']) && 'yes' == $_POST['add_complete_lesson']) {
 					$complete = true;
 				}
 
-				$result = Sensei_Utils::sensei_start_lesson( $lesson_id, $user_id, $complete );
+				$result = Sensei_Utils::sensei_start_lesson($lesson_id, $user_id, $complete);
 
 				// Updates the Course status and it's meta data
-				Sensei_Utils::user_complete_course( $course_id, $user_id );
+				Sensei_Utils::user_complete_course($course_id, $user_id);
 
 			break;
 		}
 
 		// Set redirect URL after adding user to course/lesson
-		$query_args = array( 'page' => $this->page_slug, 'view' => 'learners' );
+		$query_args = array('page' => $this->page_slug, 'view' => 'learners');
 
-		if( $result ) {
+		if ($result) {
 
-			if( $course_id ) {
+			if ($course_id) {
 				$query_args['course_id'] = $course_id;
 			}
 
-			if( $lesson_id ) {
+			if ($lesson_id) {
 				$query_args['lesson_id'] = $lesson_id;
 			}
 
@@ -425,23 +425,23 @@ 
 			$query_args['message'] = 'error';
 		}
 
-		$redirect_url = apply_filters( 'sensei_learners_add_learner_redirect_url', add_query_arg( $query_args, admin_url( 'admin.php' ) ) );
+		$redirect_url = apply_filters('sensei_learners_add_learner_redirect_url', add_query_arg($query_args, admin_url('admin.php')));
 
-		wp_safe_redirect( esc_url_raw( $redirect_url ) );
+		wp_safe_redirect(esc_url_raw($redirect_url));
 		exit;
 	}
 
 	public function add_learner_notices() {
-		if( isset( $_GET['page'] ) && $this->page_slug == $_GET['page'] && isset( $_GET['message'] ) && $_GET['message'] ) {
-			if( 'success' == $_GET['message'] ) {
+		if (isset($_GET['page']) && $this->page_slug == $_GET['page'] && isset($_GET['message']) && $_GET['message']) {
+			if ('success' == $_GET['message']) {
 				$msg = array(
 					'updated',
-					__( 'Learner added successfully!', 'woothemes-sensei' ),
+					__('Learner added successfully!', 'woothemes-sensei'),
 				);
 			} else {
 				$msg = array(
 					'error',
-					__( 'Error adding learner.', 'woothemes-sensei' ),
+					__('Error adding learner.', 'woothemes-sensei'),
 				);
 			}
 			?>
@@ -463,25 +463,25 @@ 
      *
      * @return string $full_name
      */
-    public function get_learner_full_name( $user_id ){
+    public function get_learner_full_name($user_id) {
 
         $full_name = '';
 
-        if( empty( $user_id ) || ! ( 0 < intval( $user_id ) )
-            || !( get_userdata( $user_id ) ) ){
+        if (empty($user_id) || ! (0 < intval($user_id))
+            || ! (get_userdata($user_id))) {
             return false;
         }
 
         // get the user details
-        $user = get_user_by( 'id', $user_id );
+        $user = get_user_by('id', $user_id);
 
-        if( ! empty( $user->first_name  ) && ! empty( $user->last_name  )  ){
+        if ( ! empty($user->first_name) && ! empty($user->last_name)) {
 
-            $full_name = trim( $user->first_name   ) . ' ' . trim( $user->last_name  );
+            $full_name = trim($user->first_name).' '.trim($user->last_name);
 
-        }else{
+        } else {
 
-            $full_name =  $user->display_name;
+            $full_name = $user->display_name;
 
         }
 
@@ -492,7 +492,7 @@ 
          * @param $full_name
          * @param $user_id
          */
-        return apply_filters( 'sensei_learner_full_name' , $full_name , $user_id );
+        return apply_filters('sensei_learner_full_name', $full_name, $user_id);
 
     } // end get_learner_full_name
 
@@ -503,4 +503,4 @@ 
  * for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Learners extends Sensei_Learners{}
+class WooThemes_Sensei_Learners extends Sensei_Learners {}

includes/class-sensei-list-table.php

Doc Comments +1 added lines

@@ -44,6 +44,7 @@
 	/**
 	 * Constructor
 	 * @since  1.2.0
+	 * @param string $token
 	 * @return  void
 	 */
 	public function __construct ( $token ) {

Spacing +38 added lines, -38 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Generic List Table Class
@@ -46,17 +46,17 @@ 
 	 * @since  1.2.0
 	 * @return  void
 	 */
-	public function __construct ( $token ) {
+	public function __construct($token) {
 		// Class Variables
 		$this->token = $token;
 
-		parent::__construct( array(
-								'singular' => 'wp_list_table_' . $this->token, // Singular label
-								'plural'   => 'wp_list_table_' . $this->token . 's', // Plural label
+		parent::__construct(array(
+								'singular' => 'wp_list_table_'.$this->token, // Singular label
+								'plural'   => 'wp_list_table_'.$this->token.'s', // Plural label
 								'ajax'     => false // No Ajax for this table
-		) );
+		));
 		// Actions
-		add_action( 'sensei_before_list_table', array( $this, 'table_search_form' ), 5 );
+		add_action('sensei_before_list_table', array($this, 'table_search_form'), 5);
 	} // End __construct()
 
 	/**
@@ -64,7 +64,7 @@ 
 	 * @param  array $columns Existing columns
 	 * @return array          Modified columns
 	 */
-	public function remove_sortable_columns( $columns ) {
+	public function remove_sortable_columns($columns) {
 		return array();
 	}
 
@@ -73,14 +73,14 @@ 
 	 * @since  1.2.0
 	 * @param string $which, helps you decide if you add the markup after (bottom) or before (top) the list
 	 */
-	public function extra_tablenav( $which ) {
-		if ( $which == "top" ){
+	public function extra_tablenav($which) {
+		if ($which == "top") {
 			//The code that goes before the table is here
-			do_action( 'sensei_before_list_table' );
+			do_action('sensei_before_list_table');
 		} // End If Statement
-		if ( $which == "bottom" ){
+		if ($which == "bottom") {
 			//The code that goes after the table is there
-			do_action( 'sensei_after_list_table' );
+			do_action('sensei_after_list_table');
 		} // End If Statement
 	} // End extra_tablenav()
 
@@ -90,20 +90,20 @@ 
 	 * @return void
 	 */
 	public function table_search_form() {
-		if ( empty( $_REQUEST['s'] ) && !$this->has_items() ) {
+		if (empty($_REQUEST['s']) && ! $this->has_items()) {
 			return;
 		}
 		?><form method="get">
 			<?php
-			if( isset( $_GET ) && count( $_GET ) > 0 ) {
-				foreach( $_GET as $k => $v ) {
-					if( 's' != $k ) {
+			if (isset($_GET) && count($_GET) > 0) {
+				foreach ($_GET as $k => $v) {
+					if ('s' != $k) {
 						?><input type="hidden" name="<?php echo $k; ?>" value="<?php echo $v; ?>" /><?php
 					}
 				}
 			}
 			?>
-			<?php $this->search_box( __( 'Search Users' ,'woothemes-sensei' ), 'search_id'); ?>
+			<?php $this->search_box(__('Search Users', 'woothemes-sensei'), 'search_id'); ?>
 		</form><?php
 	} // End table_search_form()
 
@@ -131,11 +131,11 @@ 
 	 * @return array
 	 */
 	function get_column_info() {
-		if ( isset( $this->_column_headers ) )
+		if (isset($this->_column_headers))
 			return $this->_column_headers;
 
 		$columns = $this->get_columns();
-		$hidden = get_hidden_columns( $this->screen );
+		$hidden = get_hidden_columns($this->screen);
 
 		$sortable_columns = $this->get_sortable_columns();
 		/**
@@ -148,22 +148,22 @@ 
 		 *
 		 * @param array $sortable_columns An array of sortable columns.
 		 */
-		$_sortable = apply_filters( "manage_{$this->screen->id}_sortable_columns", $sortable_columns );
+		$_sortable = apply_filters("manage_{$this->screen->id}_sortable_columns", $sortable_columns);
 
 		$sortable = array();
-		foreach ( $_sortable as $id => $data ) {
-			if ( empty( $data ) )
+		foreach ($_sortable as $id => $data) {
+			if (empty($data))
 				continue;
 
 			$data = (array) $data;
-			if ( !isset( $data[1] ) )
+			if ( ! isset($data[1]))
 				$data[1] = false;
 
 			$sortable[$id] = $data;
 		}
 
         $primary = $this->get_primary_column_name();
-        $this->_column_headers = array( $columns, $hidden, $sortable, $primary );
+        $this->_column_headers = array($columns, $hidden, $sortable, $primary);
 
 		return $this->_column_headers;
 	}
@@ -174,27 +174,27 @@ 
 	 * @since  1.7.0
 	 * @param object $item The current item
 	 */
-	function single_row( $item ) {
+	function single_row($item) {
 		static $row_class = '';
-		$row_class = ( $row_class == '' ? ' class="alternate"' : '' );
+		$row_class = ($row_class == '' ? ' class="alternate"' : '');
 
-		echo '<tr' . $row_class . '>';
+		echo '<tr'.$row_class.'>';
 
-		$column_data = $this->get_row_data( $item );
+		$column_data = $this->get_row_data($item);
 
-		list( $columns, $hidden ) = $this->get_column_info();
+		list($columns, $hidden) = $this->get_column_info();
 
-		foreach ( $columns as $column_name => $column_display_name ) {
+		foreach ($columns as $column_name => $column_display_name) {
 			$class = "class='$column_name column-$column_name'";
 
 			$style = '';
-			if ( in_array( $column_name, $hidden ) )
+			if (in_array($column_name, $hidden))
 				$style = ' style="display:none;"';
 
 			$attributes = "$class$style";
 
 			echo "<td $attributes>";
-			if ( isset($column_data[$column_name]) ) {
+			if (isset($column_data[$column_name])) {
 				echo $column_data[$column_name];
 			}
 			echo "</td>";
@@ -208,8 +208,8 @@ 
 	 * @access public
 	 * @abstract
 	 */
-	protected function get_row_data( $item ) {
-		die( 'either function WooThemes_Sensei_List_Table::get_row_data() must be over-ridden in a sub-class or WooThemes_Sensei_List_Table::single_row() should be.' );
+	protected function get_row_data($item) {
+		die('either function WooThemes_Sensei_List_Table::get_row_data() must be over-ridden in a sub-class or WooThemes_Sensei_List_Table::single_row() should be.');
 	}
 
 	/**
@@ -220,7 +220,7 @@ 
 	 */
 	function no_items() {
 
-		 _e( 'No items found.', 'woothemes-sensei' );
+		 _e('No items found.', 'woothemes-sensei');
 
 	} // End no_items()
 
@@ -238,9 +238,9 @@ 
 	 * @since  1.2.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		// This will be output Above the table headers on the left
-		echo apply_filters( 'sensei_list_bulk_actions', '' );
+		echo apply_filters('sensei_list_bulk_actions', '');
 	} // End bulk_actions()
 
 } // End Class

includes/class-sensei-messages.php

Doc Comments +1 added lines, -1 removed lines

@@ -299,7 +299,7 @@
      * @param  integer $sender_id   ID of sender
      * @param  integer $receiver_id ID of receiver
      * @param  string  $message     Message content
-     * @param  string  $post_id     ID of post related to message
+     * @param  integer  $post_id     ID of post related to message
      * @return mixed                Message ID on success, boolean false on failure
      */
     private function save_new_message_post( $sender_id = 0, $receiver_id = 0, $message = '', $post_id = 0 ) {

Spacing +177 added lines, -177 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Messages Class
@@ -21,61 +21,61 @@ 
 	 * Constructor.
 	 * @since  1.6.0
 	 */
-	public function __construct () {
+	public function __construct() {
 		$this->post_type = 'sensei_message';
-		$this->meta_fields = array( 'sender', 'receiver' );
+		$this->meta_fields = array('sender', 'receiver');
 
 		// Add Messages page to admin menu
-		add_action( 'admin_menu', array( $this, 'add_menu_item' ), 40 );
-		add_action( 'add_meta_boxes', array( $this, 'add_meta_box' ), 10, 2 );
-		add_action( 'admin_menu', array( $this, 'remove_meta_box' ) );
+		add_action('admin_menu', array($this, 'add_menu_item'), 40);
+		add_action('add_meta_boxes', array($this, 'add_meta_box'), 10, 2);
+		add_action('admin_menu', array($this, 'remove_meta_box'));
 
 		// Save new private message
-		add_action( 'init', array( $this, 'save_new_message' ), 1 );
+		add_action('init', array($this, 'save_new_message'), 1);
 
 		// Monitor when new reply is posted
-		add_action( 'comment_post', array( $this, 'message_reply_received' ), 10, 1 );
+		add_action('comment_post', array($this, 'message_reply_received'), 10, 1);
 
         // Block WordPress from sending comment update emails for the messages post type
-        add_filter('comment_notification_recipients', array( $this, 'stop_wp_comment_emails' ),  20, 2  );
+        add_filter('comment_notification_recipients', array($this, 'stop_wp_comment_emails'), 20, 2);
 
         // Block WordPress from sending comment moderator emails on the sensei messages post types
-        add_filter('comment_moderation_recipients', array( $this, 'stop_wp_comment_emails' ),  20, 2  );
+        add_filter('comment_moderation_recipients', array($this, 'stop_wp_comment_emails'), 20, 2);
 
 		// Process saving of message posts
-		add_action( 'save_post', array( $this, 'save_message' ) );
+		add_action('save_post', array($this, 'save_message'));
 
 		// Add message links to courses & lessons
-		add_action( 'sensei_single_course_content_inside_before', array( $this, 'send_message_link' ), 35 );
+		add_action('sensei_single_course_content_inside_before', array($this, 'send_message_link'), 35);
 
         // add message link to lesson
-        add_action( 'sensei_single_lesson_content_inside_before', array( $this, 'send_message_link' ), 30, 2 );
+        add_action('sensei_single_lesson_content_inside_before', array($this, 'send_message_link'), 30, 2);
 
         // add message link to lesson
-        add_action( 'sensei_single_quiz_questions_before', array( $this, 'send_message_link' ), 10, 2 );
+        add_action('sensei_single_quiz_questions_before', array($this, 'send_message_link'), 10, 2);
 
 		// Hide messages and replies from users who do not have access
-        add_action( 'template_redirect', array( $this, 'message_login' ), 10, 1 );
-        add_action( 'pre_get_posts', array( $this, 'message_list' ), 10, 1 );
-        add_filter( 'the_title', array( $this, 'message_title' ), 10, 2 );
-        add_filter( 'the_content', array( $this, 'message_content' ), 10, 1 );
-        add_filter( 'comments_array', array( $this, 'message_replies' ), 100, 1 );
-        add_filter( 'get_comments_number', array( $this, 'message_reply_count' ), 100, 2 );
-        add_filter( 'comments_open', array( $this, 'message_replies_open' ), 100, 2 );
+        add_action('template_redirect', array($this, 'message_login'), 10, 1);
+        add_action('pre_get_posts', array($this, 'message_list'), 10, 1);
+        add_filter('the_title', array($this, 'message_title'), 10, 2);
+        add_filter('the_content', array($this, 'message_content'), 10, 1);
+        add_filter('comments_array', array($this, 'message_replies'), 100, 1);
+        add_filter('get_comments_number', array($this, 'message_reply_count'), 100, 2);
+        add_filter('comments_open', array($this, 'message_replies_open'), 100, 2);
 	} // End __construct()
 
 	public function add_menu_item() {
 
-		if( ! isset( Sensei()->settings->settings['messages_disable'] ) || ! Sensei()->settings->settings['messages_disable'] ) {
-			add_submenu_page( 'sensei', __( 'Messages', 'woothemes-sensei'),  __( 'Messages', 'woothemes-sensei') , 'edit_courses', 'edit.php?post_type=sensei_message' );
+		if ( ! isset(Sensei()->settings->settings['messages_disable']) || ! Sensei()->settings->settings['messages_disable']) {
+			add_submenu_page('sensei', __('Messages', 'woothemes-sensei'), __('Messages', 'woothemes-sensei'), 'edit_courses', 'edit.php?post_type=sensei_message');
 		}
 	}
 
-	public function add_meta_box( $post_type, $post ) {
+	public function add_meta_box($post_type, $post) {
 
-		if( ! $post_type == $this->post_type ) return;
+		if ( ! $post_type == $this->post_type) return;
 
-		add_meta_box( $this->post_type . '-data', __( 'Message Information', 'woothemes-sensei' ), array( $this, 'meta_box_content' ), $this->post_type, 'normal', 'default' );
+		add_meta_box($this->post_type.'-data', __('Message Information', 'woothemes-sensei'), array($this, 'meta_box_content'), $this->post_type, 'normal', 'default');
 
 	}
 
@@ -85,25 +85,25 @@ 
 		$settings = array(
 			array(
 				'id' 			=> 'sender',
-				'label'			=> __( 'Message sent by:', 'woothemes-sensei' ),
-				'description'	=> __( 'The username of the learner who sent this message.', 'woothemes-sensei' ),
+				'label'			=> __('Message sent by:', 'woothemes-sensei'),
+				'description'	=> __('The username of the learner who sent this message.', 'woothemes-sensei'),
 				'type'			=> 'text',
 				'default'		=> '',
-				'placeholder'	=> __( 'Learner username', 'woothemes-sensei' ),
+				'placeholder'	=> __('Learner username', 'woothemes-sensei'),
 			),
 			array(
 				'id' 			=> 'receiver',
-				'label'			=> __( 'Message received by:', 'woothemes-sensei' ),
-				'description'	=> __( 'The username of the teacher who received this message.', 'woothemes-sensei' ),
+				'label'			=> __('Message received by:', 'woothemes-sensei'),
+				'description'	=> __('The username of the teacher who received this message.', 'woothemes-sensei'),
 				'type'			=> 'text',
 				'default'		=> '',
-				'placeholder'	=> __( 'Teacher username', 'woothemes-sensei' ),
+				'placeholder'	=> __('Teacher username', 'woothemes-sensei'),
 			),
 		);
 
-		$message_posttype = get_post_meta( $post->ID, '_posttype', true );
+		$message_posttype = get_post_meta($post->ID, '_posttype', true);
 
-		if( isset( $message_posttype ) && $message_posttype ) {
+		if (isset($message_posttype) && $message_posttype) {
 
 			$args = array(
 				'post_type' => $message_posttype,
@@ -113,58 +113,58 @@ 
 				'post_status' => 'publish',
 			);
 
-			$posts = get_posts( $args );
+			$posts = get_posts($args);
 
-			$post_options[0] = sprintf( __( 'Select %1$s', 'woothemes-sensei' ), $message_posttype );
-			foreach( $posts as $post_item ) {
-				$post_options[ $post_item->ID ] = $post_item->post_title;
+			$post_options[0] = sprintf(__('Select %1$s', 'woothemes-sensei'), $message_posttype);
+			foreach ($posts as $post_item) {
+				$post_options[$post_item->ID] = $post_item->post_title;
 			}
 
 			$settings[] = array(
 				'id' 			=> 'post',
-				'label'			=> sprintf( __( 'Message from %1$s:', 'woothemes-sensei' ), $message_posttype ),
-				'description'	=> sprintf( __( 'The %1$s to which this message relates.', 'woothemes-sensei' ), $message_posttype ),
+				'label'			=> sprintf(__('Message from %1$s:', 'woothemes-sensei'), $message_posttype),
+				'description'	=> sprintf(__('The %1$s to which this message relates.', 'woothemes-sensei'), $message_posttype),
 				'type'			=> 'select',
 				'default'		=> 0,
 				'options'		=> $post_options,
 			);
 		}
 
-		$html = Sensei()->admin->render_settings( $settings, $post->ID, 'message-info' );
+		$html = Sensei()->admin->render_settings($settings, $post->ID, 'message-info');
 
 		echo $html;
 	}
 
-	public function save_message( $post_id = 0 ) {
+	public function save_message($post_id = 0) {
 		global $post;
 
-		if( $this->post_type != get_post_type() ) return;
+		if ($this->post_type != get_post_type()) return;
 
-		if( isset( $_POST['sender'] ) && $_POST['sender'] ) {
-			update_post_meta( $post_id, '_sender', $_POST['sender'] );
+		if (isset($_POST['sender']) && $_POST['sender']) {
+			update_post_meta($post_id, '_sender', $_POST['sender']);
 		}
 
-		if( isset( $_POST['receiver'] ) && $_POST['receiver'] ) {
-			update_post_meta( $post_id, '_receiver', $_POST['receiver'] );
+		if (isset($_POST['receiver']) && $_POST['receiver']) {
+			update_post_meta($post_id, '_receiver', $_POST['receiver']);
 		}
 
-		if( isset( $_POST['post'] ) && $_POST['post'] ) {
-			update_post_meta( $post_id, '_post', $_POST['post'] );
+		if (isset($_POST['post']) && $_POST['post']) {
+			update_post_meta($post_id, '_post', $_POST['post']);
 		}
 
-		remove_action( 'save_post', array( $this, 'save_message' ) );
+		remove_action('save_post', array($this, 'save_message'));
 
-		wp_update_post( array( 'ID' => $post_id, 'comment_status' => 'open' ) );
+		wp_update_post(array('ID' => $post_id, 'comment_status' => 'open'));
 
-		add_action( 'save_post', array( $this, 'save_message' ) );
+		add_action('save_post', array($this, 'save_message'));
 	}
 
-	public function send_message_link( $post_id = 0, $user_id = 0 ) {
+	public function send_message_link($post_id = 0, $user_id = 0) {
 		global  $post;
 
         // only show the link for the allowed post types:
         $allowed_post_types = array('lesson', 'course', 'quiz');
-		if ( ! in_array( get_post_type() , $allowed_post_types ) ) {
+		if ( ! in_array(get_post_type(), $allowed_post_types)) {
 
 			return;
 
@@ -172,28 +172,28 @@ 
 
 		$html = '';
 
-		if( ! isset( Sensei()->settings->settings['messages_disable'] ) || ! Sensei()->settings->settings['messages_disable'] ) {
+		if ( ! isset(Sensei()->settings->settings['messages_disable']) || ! Sensei()->settings->settings['messages_disable']) {
 
-			if( ! is_user_logged_in() ) return;
+			if ( ! is_user_logged_in()) return;
 
-			if( isset( $_GET['contact'] ) ) {
-				$html .= $this->teacher_contact_form( $post );
+			if (isset($_GET['contact'])) {
+				$html .= $this->teacher_contact_form($post);
 			} else {
-				$href = add_query_arg( array( 'contact' => $post->post_type ) );
-
-				if( 'lesson' == $post->post_type ) {
-					$contact_button_text = __( 'Contact Lesson Teacher', 'woothemes-sensei' );
-				} elseif( 'course' == $post->post_type ) {
-					$contact_button_text = __( 'Contact Course Teacher', 'woothemes-sensei' );
-				}else{
-                    $contact_button_text = __( 'Contact Teacher', 'woothemes-sensei' );
+				$href = add_query_arg(array('contact' => $post->post_type));
+
+				if ('lesson' == $post->post_type) {
+					$contact_button_text = __('Contact Lesson Teacher', 'woothemes-sensei');
+				} elseif ('course' == $post->post_type) {
+					$contact_button_text = __('Contact Course Teacher', 'woothemes-sensei');
+				} else {
+                    $contact_button_text = __('Contact Teacher', 'woothemes-sensei');
                 }
 
-				$html .= '<p><a class="button send-message-button" href="' . esc_url($href) . '#private_message">' . $contact_button_text . '</a></p>';
+				$html .= '<p><a class="button send-message-button" href="'.esc_url($href).'#private_message">'.$contact_button_text.'</a></p>';
 			}
 
-			if( isset( $this->message_notice ) && isset( $this->message_notice['type'] ) && isset( $this->message_notice['notice'] ) ) {
-				$html .= '<div class="sensei-message ' . $this->message_notice['type'] . '">' . $this->message_notice['notice'] . '</div>';
+			if (isset($this->message_notice) && isset($this->message_notice['type']) && isset($this->message_notice['notice'])) {
+				$html .= '<div class="sensei-message '.$this->message_notice['type'].'">'.$this->message_notice['notice'].'</div>';
 			}
 
 		}
@@ -201,40 +201,40 @@ 
 		echo $html;
 	}
 
-	public function teacher_contact_form( $post ) {
+	public function teacher_contact_form($post) {
 
-		if( ! is_user_logged_in() ) return;
+		if ( ! is_user_logged_in()) return;
 
 		global $current_user;
 		wp_get_current_user();
 
 		$html = '';
 
-		if( ! isset( $post->ID ) ) return $html;
+		if ( ! isset($post->ID)) return $html;
 
         //confirm private message
         $confirmation = '';
-        if( isset( $_GET[ 'send' ] ) && 'complete' == $_GET[ 'send' ] ) {
+        if (isset($_GET['send']) && 'complete' == $_GET['send']) {
 
             $confirmation_message = __('Your private message has been sent.', 'woothemes-sensei');
-            $confirmation = '<div class="sensei-message tick">' . $confirmation_message . '</div>';
+            $confirmation = '<div class="sensei-message tick">'.$confirmation_message.'</div>';
 
         }
 
-		$html .= '<h3 id="private_message">' . __( 'Send Private Message', 'woothemes-sensei' ) . '</h3>';
+		$html .= '<h3 id="private_message">'.__('Send Private Message', 'woothemes-sensei').'</h3>';
         $html .= '<p>';
-        $html .=  $confirmation;
+        $html .= $confirmation;
         $html .= '</p>';
 		$html .= '<form name="contact-teacher" action="" method="post" class="contact-teacher">';
 			$html .= '<p class="form-row form-row-wide">';
-				$html .= '<textarea name="contact_message" placeholder="' . __( 'Enter your private message.', 'woothemes-sensei' ) . '"></textarea>';
+				$html .= '<textarea name="contact_message" placeholder="'.__('Enter your private message.', 'woothemes-sensei').'"></textarea>';
 			$html .= '</p>';
 			$html .= '<p class="form-row">';
-				$html .= '<input type="hidden" name="post_id" value="' . $post->ID . '" />';
-				$html .= '<input type="hidden" name="sender_id" value="' . $current_user->ID . '" />';
-				$html .= '<input type="hidden" name="receiver_id" value="' . $post->post_author . '" />';
-				$html .= wp_nonce_field( 'message_teacher', 'sensei_message_teacher_nonce', true, false );
-				$html .= '<input type="submit" class="send_message" value="' . __( 'Send Message', 'woothemes-sensei' ) . '" />';
+				$html .= '<input type="hidden" name="post_id" value="'.$post->ID.'" />';
+				$html .= '<input type="hidden" name="sender_id" value="'.$current_user->ID.'" />';
+				$html .= '<input type="hidden" name="receiver_id" value="'.$post->post_author.'" />';
+				$html .= wp_nonce_field('message_teacher', 'sensei_message_teacher_nonce', true, false);
+				$html .= '<input type="submit" class="send_message" value="'.__('Send Message', 'woothemes-sensei').'" />';
 			$html .= '</p>';
 			$html .= '<div class="fix"></div>';
 		$html .= '</form>';
@@ -244,30 +244,30 @@ 
 
 	public function save_new_message() {
 
-		if( ! isset( $_POST['sensei_message_teacher_nonce'] ) ) return;
+		if ( ! isset($_POST['sensei_message_teacher_nonce'])) return;
 
-		if( ! wp_verify_nonce( $_POST['sensei_message_teacher_nonce'], 'message_teacher' ) ) return;
+		if ( ! wp_verify_nonce($_POST['sensei_message_teacher_nonce'], 'message_teacher')) return;
 
-		$message_id = $this->save_new_message_post( $_POST['sender_id'], $_POST['receiver_id'], $_POST['contact_message'], $_POST['post_id'] );
+		$message_id = $this->save_new_message_post($_POST['sender_id'], $_POST['receiver_id'], $_POST['contact_message'], $_POST['post_id']);
 
 	}
 
-	public function message_reply_received( $comment_id = 0 ) {
+	public function message_reply_received($comment_id = 0) {
 
 		// Get comment object
-    	$comment = get_comment( $comment_id );
+    	$comment = get_comment($comment_id);
 
-		if( is_null( $comment ) ) return;
+		if (is_null($comment)) return;
 
 		// Get message post object
-		$message = get_post( $comment->comment_post_ID );
+		$message = get_post($comment->comment_post_ID);
 
-		if( $message->post_type != $this->post_type ) return;
+		if ($message->post_type != $this->post_type) return;
 
 		// Force comment to be approved
-		wp_set_comment_status( $comment_id, 'approve' );
+		wp_set_comment_status($comment_id, 'approve');
 
-		do_action( 'sensei_private_message_reply', $comment, $message );
+		do_action('sensei_private_message_reply', $comment, $message);
 	}
 
     /**
@@ -280,11 +280,11 @@ 
      * @param int $comment_id
      * @return array;
      */
-    public function stop_wp_comment_emails( $emails , $comment_id ){
+    public function stop_wp_comment_emails($emails, $comment_id) {
 
-        $comment = get_comment( $comment_id );
-        if( isset( $comment->comment_post_ID ) &&
-            'sensei_message' == get_post_type( $comment->comment_post_ID )  ){
+        $comment = get_comment($comment_id);
+        if (isset($comment->comment_post_ID) &&
+            'sensei_message' == get_post_type($comment->comment_post_ID)) {
 
             // empty the emails array to ensure no emails are sent for this comment
             $emails = array();
@@ -302,45 +302,45 @@ 
      * @param  string  $post_id     ID of post related to message
      * @return mixed                Message ID on success, boolean false on failure
      */
-    private function save_new_message_post( $sender_id = 0, $receiver_id = 0, $message = '', $post_id = 0 ) {
+    private function save_new_message_post($sender_id = 0, $receiver_id = 0, $message = '', $post_id = 0) {
 
     	$message_id = false;
 
-    	if( $sender_id && $receiver_id && $message && $post_id ) {
+    	if ($sender_id && $receiver_id && $message && $post_id) {
 
-    		$title = wp_trim_words( $message, 8, '...' );
+    		$title = wp_trim_words($message, 8, '...');
 
     		// Set up post data for message
 	    	$message_data = array(
 	            'post_type'      => $this->post_type,
-	            'post_title'     => esc_html( $title ),
-	            'post_content'   => esc_html( $message ),
+	            'post_title'     => esc_html($title),
+	            'post_content'   => esc_html($message),
 	            'post_status'    => 'publish',
 	            'ping_status'    => 'closed',
 	            'comment_status' => 'open',
 	            'post_excerpt'   => '',
-	            'post_author'	 => intval( $sender_id )
+	            'post_author'	 => intval($sender_id)
 	        );
 
 	    	// Insert post
-	        $message_id = wp_insert_post( $message_data );
+	        $message_id = wp_insert_post($message_data);
 
-	        if( ! is_wp_error( $message_id ) ) {
+	        if ( ! is_wp_error($message_id)) {
 
 	        	// Add sender to message meta
-	        	$sender = get_userdata( $sender_id );
-	        	add_post_meta( $message_id, '_sender', $sender->user_login );
+	        	$sender = get_userdata($sender_id);
+	        	add_post_meta($message_id, '_sender', $sender->user_login);
 
 	        	// Add receiver to message meta
-	        	$receiver = get_userdata( $receiver_id );
-		        add_post_meta( $message_id, '_receiver', $receiver->user_login );
+	        	$receiver = get_userdata($receiver_id);
+		        add_post_meta($message_id, '_receiver', $receiver->user_login);
 
 		        // Add lesson/course ID to message meta
-		        $post = get_post( $post_id );
-		        add_post_meta( $message_id, '_posttype', $post->post_type );
-		        add_post_meta( $message_id, '_post', $post->ID );
+		        $post = get_post($post_id);
+		        add_post_meta($message_id, '_posttype', $post->post_type);
+		        add_post_meta($message_id, '_post', $post->ID);
 
-		        do_action( 'sensei_new_private_message', $message_id );
+		        do_action('sensei_new_private_message', $message_id);
 
 		    } else {
 
@@ -358,22 +358,22 @@ 
 	 * @param  integer $user_id    ID of user
 	 * @return boolean             True if user has access to this message
 	 */
-	private function view_message( $message_id, $user_id = 0) {
+	private function view_message($message_id, $user_id = 0) {
 
-		if( ! is_user_logged_in() ) return false;
+		if ( ! is_user_logged_in()) return false;
 
-		if( $user_id == 0 ) {
+		if ($user_id == 0) {
 			global $current_user;
 			wp_get_current_user();
 			$user_login = $current_user->user_login;
 		}
 
 		// Get allowed users
-		$receiver = get_post_meta( $message_id, '_receiver', true );
-		$sender = get_post_meta( $message_id, '_sender', true );
+		$receiver = get_post_meta($message_id, '_receiver', true);
+		$sender = get_post_meta($message_id, '_sender', true);
 
 		// Check if user is allowed to view the message
-		if( in_array( $user_login, array( $receiver, $sender ) ) ) {
+		if (in_array($user_login, array($receiver, $sender))) {
 			return true;
 		}
 
@@ -400,27 +400,27 @@ 
      * @return void
      */
 
-    public function message_login () {
+    public function message_login() {
 
-        if ( is_user_logged_in() ) {
+        if (is_user_logged_in()) {
 
             return;
         }
 
         $settings = Sensei()->settings->get_settings();
-        if( isset( $settings[ 'my_course_page' ] )
-            && 0 < intval( $settings[ 'my_course_page' ] ) ){
+        if (isset($settings['my_course_page'])
+            && 0 < intval($settings['my_course_page'])) {
 
-            $my_courses_page_id = $settings[ 'my_course_page' ];
+            $my_courses_page_id = $settings['my_course_page'];
 
             $my_courses_url = get_permalink($my_courses_page_id);
 
         }
 
-        if ( is_single() && is_singular( $this->post_type )
-            || is_post_type_archive( $this->post_type ) ) {
+        if (is_single() && is_singular($this->post_type)
+            || is_post_type_archive($this->post_type)) {
 
-            if ( isset($my_courses_url) ) {
+            if (isset($my_courses_url)) {
 
                 wp_redirect($my_courses_url, 303);
                 exit;
@@ -439,12 +439,12 @@ 
      * @param  array $query Original query
      * @return void
      */
-	public function message_list( $query ) {
+	public function message_list($query) {
 		global $current_user;
 
-		if( is_admin() ) return;
+		if (is_admin()) return;
 
-		if( is_post_type_archive( $this->post_type ) && $query->is_main_query() ) {
+		if (is_post_type_archive($this->post_type) && $query->is_main_query()) {
 			wp_get_current_user();
 			$username = $current_user->user_login;
 
@@ -462,7 +462,7 @@ 
 				'compare' => '='
 			);
 
-			$query->set( 'meta_query', $meta_query );
+			$query->set('meta_query', $meta_query);
 
 			return;
 		}
@@ -474,11 +474,11 @@ 
 	 * @param  integer $post_id ID of post
 	 * @return string           Modified string if user does not have access to this message
 	 */
-	public function message_title( $title = '', $post_id = null ) {
+	public function message_title($title = '', $post_id = null) {
 
-		if( is_single() && is_singular( $this->post_type ) && in_the_loop() && get_post_type( $post_id ) == $this->post_type ) {
-			if( ! is_user_logged_in() || ! $this->view_message( $post_id ) ) {
-				$title = __( 'You are not allowed to view this message.', 'woothemes-sensei' );
+		if (is_single() && is_singular($this->post_type) && in_the_loop() && get_post_type($post_id) == $this->post_type) {
+			if ( ! is_user_logged_in() || ! $this->view_message($post_id)) {
+				$title = __('You are not allowed to view this message.', 'woothemes-sensei');
 			}
 		}
 
@@ -490,12 +490,12 @@ 
 	 * @param  string $content Original message content
 	 * @return string          Empty string if user does not have access to this message
 	 */
-	public function message_content( $content ) {
+	public function message_content($content) {
 		global $post;
 
-		if( is_single() && is_singular( $this->post_type ) && in_the_loop() ) {
-			if( ! is_user_logged_in() || ! $this->view_message( $post->ID ) ) {
-				$content = __( 'Please log in to view your messages.', 'woothemes-sensei' );
+		if (is_single() && is_singular($this->post_type) && in_the_loop()) {
+			if ( ! is_user_logged_in() || ! $this->view_message($post->ID)) {
+				$content = __('Please log in to view your messages.', 'woothemes-sensei');
 			}
 		}
 
@@ -507,11 +507,11 @@ 
 	 * @param  array $comments Array of replies
 	 * @return array           Empty array if user does not have access to this message
 	 */
-	public function message_replies( $comments ) {
+	public function message_replies($comments) {
 		global $post;
 
-		if( is_single() && is_singular( $this->post_type ) && in_the_loop() ) {
-			if( ! is_user_logged_in() || ! $this->view_message( $post->ID ) ) {
+		if (is_single() && is_singular($this->post_type) && in_the_loop()) {
+			if ( ! is_user_logged_in() || ! $this->view_message($post->ID)) {
 				$comments = array();
 			}
 		}
@@ -525,11 +525,11 @@ 
 	 * @param  integer $post_id ID of post
 	 * @return integer          0 if user does not have access to this message
 	 */
-	public function message_reply_count( $count, $post_id ) {
+	public function message_reply_count($count, $post_id) {
 		global $post;
 
-		if( is_single() && is_singular( $this->post_type ) && in_the_loop() ) {
-			if( ! is_user_logged_in() || ! $this->view_message( $post->ID ) ) {
+		if (is_single() && is_singular($this->post_type) && in_the_loop()) {
+			if ( ! is_user_logged_in() || ! $this->view_message($post->ID)) {
 				$count = 0;
 			}
 		}
@@ -543,11 +543,11 @@ 
 	 * @param  integer $post_id ID of post
 	 * @return boolean          False if user does not have access to this message
 	 */
-	public function message_replies_open( $open, $post_id ) {
+	public function message_replies_open($open, $post_id) {
 		global $post;
 
-		if( is_single() && is_singular( $this->post_type ) && in_the_loop() ) {
-			if( ! is_user_logged_in() || ! $this->view_message( $post->ID ) ) {
+		if (is_single() && is_singular($this->post_type) && in_the_loop()) {
+			if ( ! is_user_logged_in() || ! $this->view_message($post->ID)) {
 				$open = false;
 			}
 		}
@@ -560,17 +560,17 @@ 
      *
      * @since 1.9.0
      */
-    public static function the_message_sent_by_title(){
+    public static function the_message_sent_by_title() {
 
-        $sender_username = get_post_meta( get_the_ID() , '_sender', true );
-        if( $sender_username ) {
+        $sender_username = get_post_meta(get_the_ID(), '_sender', true);
+        if ($sender_username) {
 
-            $sender = get_user_by( 'login', $sender_username ); ?>
+            $sender = get_user_by('login', $sender_username); ?>
 
             <p class="message-meta">
                 <small>
                     <em>
-                        <?php printf( __( 'Sent by %1$s on %2$s.', 'woothemes-sensei' ), $sender->display_name, get_the_date() ); ?>
+                        <?php printf(__('Sent by %1$s on %2$s.', 'woothemes-sensei'), $sender->display_name, get_the_date()); ?>
                     </em>
                 </small>
             </p>
@@ -589,11 +589,11 @@ 
 
         global $post;
 
-        $content_post_id = get_post_meta( $post->ID, '_post', true );
-        if( $content_post_id ) {
-            $title = sprintf( __( 'Re: %1$s', 'woothemes-sensei' ), '<a href="' . get_permalink( $content_post_id ) . '">' . get_the_title( $content_post_id ) . '</a>' );
+        $content_post_id = get_post_meta($post->ID, '_post', true);
+        if ($content_post_id) {
+            $title = sprintf(__('Re: %1$s', 'woothemes-sensei'), '<a href="'.get_permalink($content_post_id).'">'.get_the_title($content_post_id).'</a>');
         } else {
-            $title = get_the_title( $post->ID );
+            $title = get_the_title($post->ID);
         }
 
         ?>
@@ -610,7 +610,7 @@ 
                  * @param string $template
                  * @param string $post_type
                  */
-                echo apply_filters( 'sensei_single_title', $title, $post->post_type );
+                echo apply_filters('sensei_single_title', $title, $post->post_type);
                 ?>
 
             </h1>
@@ -629,18 +629,18 @@ 
      *
      * @return string
      */
-    public static function the_archive_header( ){
+    public static function the_archive_header( ) {
 
         $html = '';
         $html .= '<header class="archive-header"><h1>';
-        $html .= __( 'My Messages', 'woothemes-sensei' );
+        $html .= __('My Messages', 'woothemes-sensei');
         $html .= '</h1></header>';
 
         /**
          * Filter the sensei messages archive title.
          * @since 1.0.0
          */
-        echo apply_filters( 'sensei_message_archive_title', $html );
+        echo apply_filters('sensei_message_archive_title', $html);
 
     } // get_archive_header()
 
@@ -650,23 +650,23 @@ 
      * @since 1.9.0
      * @param $post_id
      */
-    public static function the_message_title( $message_post_id ){
+    public static function the_message_title($message_post_id) {
 
-        $content_post_id = get_post_meta( $message_post_id, '_post', true );
+        $content_post_id = get_post_meta($message_post_id, '_post', true);
 
-        if( $content_post_id ) {
+        if ($content_post_id) {
 
-            $title = sprintf( __( 'Re: %1$s', 'woothemes-sensei' ), get_the_title( $content_post_id ) );
+            $title = sprintf(__('Re: %1$s', 'woothemes-sensei'), get_the_title($content_post_id));
 
         } else {
 
-            $title = get_the_title( $message_post_id );
+            $title = get_the_title($message_post_id);
 
         }
 
         ?>
         <h2>
-            <a href="<?php esc_url( get_permalink( $message_post_id ) );?>">
+            <a href="<?php esc_url(get_permalink($message_post_id)); ?>">
                 <?php echo  $title; ?>
             </a>
 
@@ -680,13 +680,13 @@ 
      *
      * @param $message_post_id
      */
-    public  static function the_message_sender( $message_post_id ){
+    public  static function the_message_sender($message_post_id) {
 
-        $sender_username = get_post_meta( $message_post_id, '_sender', true );
-        $sender = get_user_by( 'login', $sender_username );
+        $sender_username = get_post_meta($message_post_id, '_sender', true);
+        $sender = get_user_by('login', $sender_username);
 
-        if( $sender_username && $sender instanceof WP_User ) {
-            $sender_display_name = sprintf( __( 'Sent by %1$s on %2$s.', 'woothemes-sensei' ), $sender->display_name, get_the_date() );
+        if ($sender_username && $sender instanceof WP_User) {
+            $sender_display_name = sprintf(__('Sent by %1$s on %2$s.', 'woothemes-sensei'), $sender->display_name, get_the_date());
             ?>
             <p class="message-meta">
                 <small>
@@ -704,13 +704,13 @@ 
      *
      * @since 1.9.0
      */
-    public static function the_my_messages_link(){
-        if( ! Sensei()->settings->get('messages_disable')  ) {
+    public static function the_my_messages_link() {
+        if ( ! Sensei()->settings->get('messages_disable')) {
             ?>
             <p class="my-messages-link-container">
-                <a class="my-messages-link" href="<?php echo get_post_type_archive_link( 'sensei_message' ); ?>"
-                   title="<?php _e( 'View & reply to private messages sent to your course & lesson teachers.', 'woothemes-sensei' ); ?>">
-                    <?php _e( 'My Messages', 'woothemes-sensei' ); ?>
+                <a class="my-messages-link" href="<?php echo get_post_type_archive_link('sensei_message'); ?>"
+                   title="<?php _e('View & reply to private messages sent to your course & lesson teachers.', 'woothemes-sensei'); ?>">
+                    <?php _e('My Messages', 'woothemes-sensei'); ?>
                 </a>
             </p>
             <?php
@@ -724,4 +724,4 @@ 
  * for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Messages extends Sensei_Messages{}
+class WooThemes_Sensei_Messages extends Sensei_Messages {}

includes/class-sensei-modules.php

Doc Comments +1 added lines

@@ -1942,6 +1942,7 @@
      * Hooked into 'get_terms'
      *
      * @since 1.8.0
+     * @param string[] $taxonomies
      */
     public function append_teacher_name_to_module( $terms, $taxonomies, $args )
     {

Spacing +259 added lines, -259 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 /**
  * Sensei Modules Class
@@ -21,33 +21,33 @@ 
     private $order_page_slug;
     public $taxonomy;
 
-    public function __construct( $file )
+    public function __construct($file)
     {
         $this->file = $file;
         $this->dir = dirname($this->file);
-        $this->assets_dir = trailingslashit($this->dir) . 'assets';
+        $this->assets_dir = trailingslashit($this->dir).'assets';
         $this->assets_url = esc_url(trailingslashit(plugins_url('/assets/', $this->file)));
         $this->taxonomy = 'module';
         $this->order_page_slug = 'module-order';
 
         // setup taxonomy
-        add_action( 'init', array( $this, 'setup_modules_taxonomy' ), 10 );
+        add_action('init', array($this, 'setup_modules_taxonomy'), 10);
 
         // Manage lesson meta boxes for taxonomy
-        add_action('add_meta_boxes', array($this, 'modules_metaboxes'), 20, 2 );
+        add_action('add_meta_boxes', array($this, 'modules_metaboxes'), 20, 2);
 
         // Save lesson meta box
         add_action('save_post', array($this, 'save_lesson_module'), 10, 1);
 
         //Reset the none modules lessons transient
-        add_action( 'save_post', array( 'Sensei_Core_Modules', 'reset_none_modules_transient' ) );
+        add_action('save_post', array('Sensei_Core_Modules', 'reset_none_modules_transient'));
 
         // Frontend styling
         add_action('wp_enqueue_scripts', array($this, 'enqueue_styles'));
 
         // Admin styling
         add_action('admin_enqueue_scripts', array($this, 'admin_enqueue_styles'));
-        add_action('admin_enqueue_scripts', array($this, 'admin_enqueue_scripts'),  20 , 2 );
+        add_action('admin_enqueue_scripts', array($this, 'admin_enqueue_scripts'), 20, 2);
 
         // Handle module completion record
         add_action('sensei_lesson_status_updated', array($this, 'update_lesson_status_module_progress'), 10, 3);
@@ -55,20 +55,20 @@ 
         add_action('wp', array($this, 'save_module_progress'), 10);
 
         // Handle module ordering
-        add_action('admin_menu', array($this, 'register_modules_admin_menu_items'), 30 );
+        add_action('admin_menu', array($this, 'register_modules_admin_menu_items'), 30);
         add_filter('manage_edit-course_columns', array($this, 'course_columns'), 11, 1);
         add_action('manage_posts_custom_column', array($this, 'course_column_content'), 11, 2);
 
         // Ensure modules always show under courses
-        add_action( 'admin_menu', array( $this, 'remove_lessons_menu_model_taxonomy' ) , 10 );
-        add_action( 'admin_menu', array( $this, 'remove_courses_menu_model_taxonomy' ) , 10 );
-        add_action( 'admin_menu', array( $this, 'redirect_to_lesson_module_taxonomy_to_course' ) , 20 );
+        add_action('admin_menu', array($this, 'remove_lessons_menu_model_taxonomy'), 10);
+        add_action('admin_menu', array($this, 'remove_courses_menu_model_taxonomy'), 10);
+        add_action('admin_menu', array($this, 'redirect_to_lesson_module_taxonomy_to_course'), 20);
 
         // Add course field to taxonomy
-        add_action($this->taxonomy . '_add_form_fields', array($this, 'add_module_fields'), 50, 1);
-        add_action($this->taxonomy . '_edit_form_fields', array($this, 'edit_module_fields'), 1, 1);
-        add_action('edited_' . $this->taxonomy, array($this, 'save_module_course'), 10, 2);
-        add_action('created_' . $this->taxonomy, array($this, 'save_module_course'), 10, 2);
+        add_action($this->taxonomy.'_add_form_fields', array($this, 'add_module_fields'), 50, 1);
+        add_action($this->taxonomy.'_edit_form_fields', array($this, 'edit_module_fields'), 1, 1);
+        add_action('edited_'.$this->taxonomy, array($this, 'save_module_course'), 10, 2);
+        add_action('created_'.$this->taxonomy, array($this, 'save_module_course'), 10, 2);
         add_action('wp_ajax_sensei_json_search_courses', array($this, 'search_courses_json'));
 
         // Manage module taxonomy archive page
@@ -80,10 +80,10 @@ 
         add_filter('body_class', array($this, 'module_archive_body_class'));
 
         // add modules to the single course template
-        add_action( 'sensei_single_course_content_inside_after', array($this, 'load_course_module_content_template') , 8 );
+        add_action('sensei_single_course_content_inside_after', array($this, 'load_course_module_content_template'), 8);
 
         //Single Course modules actions. Add to single-course/course-modules.php
-        add_action('sensei_single_course_modules_before',array( $this,'course_modules_title' ), 20);
+        add_action('sensei_single_course_modules_before', array($this, 'course_modules_title'), 20);
 
         // Set up display on single lesson page
         add_filter('sensei_breadcrumb_output', array($this, 'module_breadcrumb_link'), 10, 2);
@@ -95,22 +95,22 @@ 
         add_filter('sensei_analysis_course_column_data', array($this, 'analysis_course_column_data'), 10, 3);
 
         // Manage module taxonomy columns
-        add_filter('manage_edit-' . $this->taxonomy . '_columns', array($this, 'taxonomy_column_headings'), 1, 1);
-        add_filter('manage_' . $this->taxonomy . '_custom_column', array($this, 'taxonomy_column_content'), 1, 3);
+        add_filter('manage_edit-'.$this->taxonomy.'_columns', array($this, 'taxonomy_column_headings'), 1, 1);
+        add_filter('manage_'.$this->taxonomy.'_custom_column', array($this, 'taxonomy_column_content'), 1, 3);
         add_filter('sensei_module_lesson_list_title', array($this, 'sensei_course_preview_titles'), 10, 2);
 
         //store new modules created on the course edit screen
-        add_action( 'wp_ajax_sensei_add_new_module_term', array( 'Sensei_Core_Modules','add_new_module_term' ) );
+        add_action('wp_ajax_sensei_add_new_module_term', array('Sensei_Core_Modules', 'add_new_module_term'));
 
         // for non admin users, only show taxonomies that belong to them
-        add_filter('get_terms', array( $this, 'filter_module_terms' ), 20, 3 );
-        add_filter('get_object_terms', array( $this, 'filter_course_selected_terms' ), 20, 3 );
+        add_filter('get_terms', array($this, 'filter_module_terms'), 20, 3);
+        add_filter('get_object_terms', array($this, 'filter_course_selected_terms'), 20, 3);
 
         // add the teacher name next to the module term in for admin users
-        add_filter('get_terms', array( $this, 'append_teacher_name_to_module' ), 70, 3 );
+        add_filter('get_terms', array($this, 'append_teacher_name_to_module'), 70, 3);
 
         // remove the default modules  metabox
-        add_action('admin_init',array( 'Sensei_Core_Modules' , 'remove_default_modules_box' ));
+        add_action('admin_init', array('Sensei_Core_Modules', 'remove_default_modules_box'));
 
     } // end constructor
 
@@ -127,9 +127,9 @@ 
      * @return void
      * @deprecated since 1.9.0
      */
-    public function change_module_term_slug( $term_id, $tt_id, $taxonomy ){
+    public function change_module_term_slug($term_id, $tt_id, $taxonomy) {
 
-        _deprecated_function('change_module_term_slug', '1.9.0' );
+        _deprecated_function('change_module_term_slug', '1.9.0');
 
     }// end add_module_term_group
 
@@ -143,20 +143,20 @@ 
      *
      * @return void
      */
-    public function modules_metaboxes( $post_type, $post )
+    public function modules_metaboxes($post_type, $post)
     {
-        if ('lesson' == $post_type ) {
+        if ('lesson' == $post_type) {
 
             // Remove default taxonomy meta box from Lesson edit screen
-            remove_meta_box($this->taxonomy . 'div', 'lesson', 'side');
+            remove_meta_box($this->taxonomy.'div', 'lesson', 'side');
 
             // Add custom meta box to limit module selection to one per lesson
-            add_meta_box($this->taxonomy . '_select', __('Lesson Module', 'woothemes-sensei'), array($this, 'lesson_module_metabox'), 'lesson', 'side', 'default');
+            add_meta_box($this->taxonomy.'_select', __('Lesson Module', 'woothemes-sensei'), array($this, 'lesson_module_metabox'), 'lesson', 'side', 'default');
         }
 
-        if( 'course' == $post_type ){
+        if ('course' == $post_type) {
             // Course modules selection metabox
-            add_meta_box( $this->taxonomy . '_course_mb', __('Course Modules', 'woothemes-sensei'), array( $this, 'course_module_metabox'), 'course', 'side', 'core');
+            add_meta_box($this->taxonomy.'_course_mb', __('Course Modules', 'woothemes-sensei'), array($this, 'course_module_metabox'), 'course', 'side', 'core');
         }
     }
 
@@ -192,21 +192,21 @@ 
             $modules = $this->get_course_modules($lesson_course);
 
             // Build the HTML to output
-            $html .= '<input type="hidden" name="' . esc_attr('woo_lesson_' . $this->taxonomy . '_nonce') . '" id="' . esc_attr('woo_lesson_' . $this->taxonomy . '_nonce') . '" value="' . esc_attr(wp_create_nonce(plugin_basename($this->file))) . '" />';
+            $html .= '<input type="hidden" name="'.esc_attr('woo_lesson_'.$this->taxonomy.'_nonce').'" id="'.esc_attr('woo_lesson_'.$this->taxonomy.'_nonce').'" value="'.esc_attr(wp_create_nonce(plugin_basename($this->file))).'" />';
             if (is_array($modules) && count($modules) > 0) {
-                $html .= '<select id="lesson-module-options" name="lesson_module" class="widefat">' . "\n";
-                $html .= '<option value="">' . __('None', 'woothemes-sensei') . '</option>';
+                $html .= '<select id="lesson-module-options" name="lesson_module" class="widefat">'."\n";
+                $html .= '<option value="">'.__('None', 'woothemes-sensei').'</option>';
                 foreach ($modules as $module) {
-                    $html .= '<option value="' . esc_attr(absint($module->term_id)) . '"' . selected($module->term_id, $lesson_module, false) . '>' . esc_html($module->name) . '</option>' . "\n";
+                    $html .= '<option value="'.esc_attr(absint($module->term_id)).'"'.selected($module->term_id, $lesson_module, false).'>'.esc_html($module->name).'</option>'."\n";
                 }
-                $html .= '</select>' . "\n";
+                $html .= '</select>'."\n";
             } else {
-                $course_url = admin_url('post.php?post=' . urlencode($lesson_course) . '&action=edit');
-                $html .= '<p>' . sprintf(__('No modules are available for this lesson yet. %1$sPlease add some to %3$sthe course%4$s.%2$s', 'woothemes-sensei'), '<em>', '</em>', '<a href="' . esc_url($course_url) . '">', '</a>') . '</p>';
+                $course_url = admin_url('post.php?post='.urlencode($lesson_course).'&action=edit');
+                $html .= '<p>'.sprintf(__('No modules are available for this lesson yet. %1$sPlease add some to %3$sthe course%4$s.%2$s', 'woothemes-sensei'), '<em>', '</em>', '<a href="'.esc_url($course_url).'">', '</a>').'</p>';
             } // End If Statement
 
         } else {
-            $html .= '<p>' . sprintf(__('No modules are available for this lesson yet. %1$sPlease select a course first.%2$s', 'woothemes-sensei'), '<em>', '</em>') . '</p>';
+            $html .= '<p>'.sprintf(__('No modules are available for this lesson yet. %1$sPlease select a course first.%2$s', 'woothemes-sensei'), '<em>', '</em>').'</p>';
         } // End If Statement
 
         // Output the HTML
@@ -225,38 +225,38 @@ 
         global $post;
 
         // Verify post type and nonce
-        if ((get_post_type() != 'lesson') || !isset($_POST['woo_lesson_' . $this->taxonomy . '_nonce'] )
-            ||!wp_verify_nonce($_POST['woo_lesson_' . $this->taxonomy . '_nonce'], plugin_basename($this->file))) {
+        if ((get_post_type() != 'lesson') || ! isset($_POST['woo_lesson_'.$this->taxonomy.'_nonce'])
+            ||! wp_verify_nonce($_POST['woo_lesson_'.$this->taxonomy.'_nonce'], plugin_basename($this->file))) {
             return $post_id;
         }
 
         // Check if user has permissions to edit lessons
         $post_type = get_post_type_object($post->post_type);
-        if (!current_user_can($post_type->cap->edit_post, $post_id)) {
+        if ( ! current_user_can($post_type->cap->edit_post, $post_id)) {
             return $post_id;
         }
 
         // Check if user has permissions to edit this specific post
-        if (!current_user_can('edit_post', $post_id)) {
+        if ( ! current_user_can('edit_post', $post_id)) {
             return $post_id;
         }
 
         // Cast module ID as an integer if selected, otherwise leave as empty string
-        if ( isset( $_POST['lesson_module'] ) ) {
+        if (isset($_POST['lesson_module'])) {
 
-            if( empty ( $_POST['lesson_module'] ) ){
-                wp_delete_object_term_relationships($post_id, $this->taxonomy  );
+            if (empty ($_POST['lesson_module'])) {
+                wp_delete_object_term_relationships($post_id, $this->taxonomy);
                 return true;
             }
 
-            $module_id = intval( $_POST['lesson_module'] );
+            $module_id = intval($_POST['lesson_module']);
 
             // Assign lesson to selected module
             wp_set_object_terms($post_id, $module_id, $this->taxonomy, false);
 
             // Set default order for lesson inside module
-            if (!get_post_meta($post_id, '_order_module_' . $module_id, true)) {
-                update_post_meta($post_id, '_order_module_' . $module_id, 0);
+            if ( ! get_post_meta($post_id, '_order_module_'.$module_id, true)) {
+                update_post_meta($post_id, '_order_module_'.$module_id, 0);
             }
         }
 
@@ -314,7 +314,7 @@ 
         $module_courses = array();
         if (isset($courses) && is_array($courses)) {
             foreach ($courses as $course) {
-                $module_courses[] =   array( 'id' =>$course->ID, 'details'=>$course->post_title );
+                $module_courses[] = array('id' =>$course->ID, 'details'=>$course->post_title);
             }
         }
 
@@ -324,8 +324,8 @@ 
                     for="module_courses"><?php _e('Course(s)', 'woothemes-sensei'); ?></label></th>
             <td>
                 <input type="hidden"
-                       data-defaults="<?php echo esc_attr( json_encode($module_courses)); ?>"
-                       value="<?php echo esc_attr( json_encode($module_courses) ); ?>"
+                       data-defaults="<?php echo esc_attr(json_encode($module_courses)); ?>"
+                       value="<?php echo esc_attr(json_encode($module_courses)); ?>"
                        id="module_courses" name="module_courses"
                        class="ajax_chosen_select_courses"
                        placeholder="<?php esc_attr_e('Search for courses...', 'woothemes-sensei'); ?>"
@@ -370,11 +370,11 @@ 
         }
 
         // Add module to selected courses
-        if ( isset( $_POST['module_courses'] ) && ! empty( $_POST['module_courses'] ) ) {
+        if (isset($_POST['module_courses']) && ! empty($_POST['module_courses'])) {
 
-            $course_ids = explode( ",", $_POST['module_courses'] );
+            $course_ids = explode(",", $_POST['module_courses']);
 
-            foreach ( $course_ids as $course_id ) {
+            foreach ($course_ids as $course_id) {
 
                 wp_set_object_terms($course_id, $module_id, $this->taxonomy, true);
 
@@ -438,11 +438,11 @@ 
      * @since 1.8.0
      * @return void
      */
-    public function single_course_modules(){
+    public function single_course_modules() {
 
-        _deprecated_function('Sensei_Modules->single_course_modules','Sensei 1.9.0', 'Sensei()->modules->load_course_module_content_template');
+        _deprecated_function('Sensei_Modules->single_course_modules', 'Sensei 1.9.0', 'Sensei()->modules->load_course_module_content_template');
         // only show modules on the course that has modules
-        if( is_singular( 'course' ) && has_term( '', 'module' )  )  {
+        if (is_singular('course') && has_term('', 'module')) {
 
             $this->load_course_module_content_template();
 
@@ -459,15 +459,15 @@ 
 
         if (method_exists('WooThemes_Sensei_Utils', 'is_preview_lesson') && WooThemes_Sensei_Utils::is_preview_lesson($lesson_id)) {
             $is_user_taking_course = WooThemes_Sensei_Utils::sensei_check_for_activity(array('post_id' => $course_id, 'user_id' => $current_user->ID, 'type' => 'sensei_course_status'));
-            if (!$is_user_taking_course) {
+            if ( ! $is_user_taking_course) {
                 if (method_exists('WooThemes_Sensei_Frontend', 'sensei_lesson_preview_title_text')) {
                     $title_text = Sensei()->frontend->sensei_lesson_preview_title_text($course_id);
                     // Remove brackets for display here
                     $title_text = str_replace('(', '', $title_text);
                     $title_text = str_replace(')', '', $title_text);
-                    $title_text = '<span class="preview-label">' . $title_text . '</span>';
+                    $title_text = '<span class="preview-label">'.$title_text.'</span>';
                 }
-                $title .= ' ' . $title_text;
+                $title .= ' '.$title_text;
             }
         }
 
@@ -481,8 +481,8 @@ 
         if (is_singular('lesson')) {
             if (has_term('', $this->taxonomy, $post->ID)) {
                 $module = $this->get_lesson_module($post->ID);
-                if( $module ) {
-                    $html .= ' ' . $separator . ' <a href="' . esc_url($module->url) . '" title="' .  __('Back to the module', 'woothemes-sensei') . '">' . $module->name . '</a>';
+                if ($module) {
+                    $html .= ' '.$separator.' <a href="'.esc_url($module->url).'" title="'.__('Back to the module', 'woothemes-sensei').'">'.$module->name.'</a>';
                 }
             }
         }
@@ -490,7 +490,7 @@ 
         if (is_tax($this->taxonomy)) {
             if (isset($_GET['course_id']) && 0 < intval($_GET['course_id'])) {
                 $course_id = intval($_GET['course_id']);
-                $html .= '<a href="' . esc_url(get_permalink($course_id)) . '" title="' .  __('Back to the course', 'woothemes-sensei') . '">' . get_the_title($course_id) . '</a>';
+                $html .= '<a href="'.esc_url(get_permalink($course_id)).'" title="'.__('Back to the course', 'woothemes-sensei').'">'.get_the_title($course_id).'</a>';
             }
         }
         return $html;
@@ -505,18 +505,18 @@ 
      */
     public function module_archive_template($template) {
 
-        if ( ! is_tax($this->taxonomy) ) {
+        if ( ! is_tax($this->taxonomy)) {
             return $template;
         }
 
         $file = 'archive-lesson.php';
-        $find = array( $file, Sensei()->template_url . $file );
+        $find = array($file, Sensei()->template_url.$file);
 
         // locate the template file
         $template = locate_template($find);
-        if (!$template) {
+        if ( ! $template) {
 
-            $template = Sensei()->plugin_path() . 'templates/' . $file;
+            $template = Sensei()->plugin_path().'templates/'.$file;
 
         }
 
@@ -542,7 +542,7 @@ 
             // Set order of lessons
             if (version_compare(Sensei()->version, '1.6.0', '>=')) {
                 $module_id = $query->queried_object_id;
-                $query->set('meta_key', '_order_module_' . $module_id);
+                $query->set('meta_key', '_order_module_'.$module_id);
                 $query->set('orderby', 'meta_value_num date');
             } else {
                 $query->set('orderby', 'menu_order');
@@ -603,10 +603,10 @@ 
                     $status = __('In progress', 'woothemes-sensei');
                     $class = 'in-progress';
                 }
-                echo '<p class="status ' . esc_attr($class) . '">' . $status . '</p>';
+                echo '<p class="status '.esc_attr($class).'">'.$status.'</p>';
             }
 
-            echo '<p class="archive-description module-description">' . apply_filters('sensei_module_archive_description', nl2br($module->description), $module->term_id) . '</p>';
+            echo '<p class="archive-description module-description">'.apply_filters('sensei_module_archive_description', nl2br($module->description), $module->term_id).'</p>';
         }
     }
 
@@ -731,7 +731,7 @@ 
     public function save_user_module_progress($module_id = 0, $course_id = 0, $user_id = 0)
     {
         $module_progress = $this->calculate_user_module_progress($user_id, $module_id, $course_id);
-        update_user_meta(intval($user_id), '_module_progress_' . intval($course_id) . '_' . intval($module_id), intval($module_progress));
+        update_user_meta(intval($user_id), '_module_progress_'.intval($course_id).'_'.intval($module_id), intval($module_progress));
 
         do_action('sensei_module_save_user_progress', $course_id, $module_id, $user_id, $module_progress);
     }
@@ -748,9 +748,9 @@ 
      */
     public function get_user_module_progress($module_id = 0, $course_id = 0, $user_id = 0)
     {
-        $module_progress = get_user_meta(intval($user_id), '_module_progress_' . intval($course_id) . '_' . intval($module_id), true);
+        $module_progress = get_user_meta(intval($user_id), '_module_progress_'.intval($course_id).'_'.intval($module_id), true);
         if ($module_progress) {
-            return (float)$module_progress;
+            return (float) $module_progress;
         }
         return false;
     }
@@ -803,7 +803,7 @@ 
         }
         $module_progress = ($completed_count / $lesson_count) * 100;
 
-        return (float)$module_progress;
+        return (float) $module_progress;
     }
 
     /**
@@ -816,7 +816,7 @@ 
     public function register_modules_admin_menu_items()
     {
         //add the modules link under the Course main menu
-        add_submenu_page('edit.php?post_type=course', __('Modules', 'woothemes-sensei'), __('Modules', 'woothemes-sensei'), 'manage_categories', 'edit-tags.php?taxonomy=module','' );
+        add_submenu_page('edit.php?post_type=course', __('Modules', 'woothemes-sensei'), __('Modules', 'woothemes-sensei'), 'manage_categories', 'edit-tags.php?taxonomy=module', '');
 
         // Regsiter new admin page for module ordering
         $hook = add_submenu_page('edit.php?post_type=course', __('Order Modules', 'woothemes-sensei'), __('Order Modules', 'woothemes-sensei'), 'edit_lessons', $this->order_page_slug, array($this, 'module_order_screen'));
@@ -843,19 +843,19 @@ 
             $ordered = $this->save_course_module_order(esc_attr($_POST['module-order']), esc_attr($_POST['course_id']));
 
             if ($ordered) {
-                $html .= '<div class="updated fade">' . "\n";
-                $html .= '<p>' . __('The module order has been saved for this course.', 'woothemes-sensei') . '</p>' . "\n";
-                $html .= '</div>' . "\n";
+                $html .= '<div class="updated fade">'."\n";
+                $html .= '<p>'.__('The module order has been saved for this course.', 'woothemes-sensei').'</p>'."\n";
+                $html .= '</div>'."\n";
             }
         }
 
         $courses = Sensei()->course->get_all_courses();
 
-        $html .= '<form action="' . admin_url('edit.php') . '" method="get">' . "\n";
-        $html .= '<input type="hidden" name="post_type" value="course" />' . "\n";
-        $html .= '<input type="hidden" name="page" value="' . esc_attr($this->order_page_slug) . '" />' . "\n";
-        $html .= '<select id="module-order-course" name="course_id">' . "\n";
-        $html .= '<option value="">' . __('Select a course', 'woothemes-sensei') . '</option>' . "\n";
+        $html .= '<form action="'.admin_url('edit.php').'" method="get">'."\n";
+        $html .= '<input type="hidden" name="post_type" value="course" />'."\n";
+        $html .= '<input type="hidden" name="page" value="'.esc_attr($this->order_page_slug).'" />'."\n";
+        $html .= '<select id="module-order-course" name="course_id">'."\n";
+        $html .= '<option value="">'.__('Select a course', 'woothemes-sensei').'</option>'."\n";
 
         foreach ($courses as $course) {
             if (has_term('', $this->taxonomy, $course->ID)) {
@@ -863,30 +863,30 @@ 
                 if (isset($_GET['course_id'])) {
                     $course_id = intval($_GET['course_id']);
                 }
-                $html .= '<option value="' . esc_attr(intval($course->ID)) . '" ' . selected($course->ID, $course_id, false) . '>' . get_the_title($course->ID) . '</option>' . "\n";
+                $html .= '<option value="'.esc_attr(intval($course->ID)).'" '.selected($course->ID, $course_id, false).'>'.get_the_title($course->ID).'</option>'."\n";
             }
         }
 
-        $html .= '</select>' . "\n";
-        $html .= '<input type="submit" class="button-primary module-order-select-course-submit" value="' . __('Select', 'woothemes-sensei') . '" />' . "\n";
-        $html .= '</form>' . "\n";
+        $html .= '</select>'."\n";
+        $html .= '<input type="submit" class="button-primary module-order-select-course-submit" value="'.__('Select', 'woothemes-sensei').'" />'."\n";
+        $html .= '</form>'."\n";
 
         if (isset($_GET['course_id'])) {
             $course_id = intval($_GET['course_id']);
             if ($course_id > 0) {
                 $modules = $this->get_course_modules($course_id);
-                $modules = $this->append_teacher_name_to_module( $modules, array( 'module' ), array() );
+                $modules = $this->append_teacher_name_to_module($modules, array('module'), array());
                 if ($modules) {
 
                     $order = $this->get_course_module_order($course_id);
 
-                    $order_string='';
+                    $order_string = '';
                     if ($order) {
                         $order_string = implode(',', $order);
                     }
 
-                    $html .= '<form id="editgrouping" method="post" action="" class="validate">' . "\n";
-                    $html .= '<ul class="sortable-module-list">' . "\n";
+                    $html .= '<form id="editgrouping" method="post" action="" class="validate">'."\n";
+                    $html .= '<ul class="sortable-module-list">'."\n";
                     $count = 0;
                     foreach ($modules as $module) {
                         $count++;
@@ -900,14 +900,14 @@ 
                         if ($count % 2 != 0) {
                             $class .= ' alternate';
                         }
-                        $html .= '<li class="' . esc_attr($class) . '"><span rel="' . esc_attr($module->term_id) . '" style="width: 100%;"> ' . $module->name . '</span></li>' . "\n";
+                        $html .= '<li class="'.esc_attr($class).'"><span rel="'.esc_attr($module->term_id).'" style="width: 100%;"> '.$module->name.'</span></li>'."\n";
                     }
-                    $html .= '</ul>' . "\n";
+                    $html .= '</ul>'."\n";
 
-                    $html .= '<input type="hidden" name="module-order" value="' . $order_string . '" />' . "\n";
-                    $html .= '<input type="hidden" name="course_id" value="' . $course_id . '" />' . "\n";
-                    $html .= '<input type="submit" class="button-primary" value="' . __('Save module order', 'woothemes-sensei') . '" />' . "\n";
-                    $html .= '<a href="' . admin_url('post.php?post=' . $course_id . '&action=edit') . '" class="button-secondary">' . __('Edit course', 'woothemes-sensei') . '</a>' . "\n";
+                    $html .= '<input type="hidden" name="module-order" value="'.$order_string.'" />'."\n";
+                    $html .= '<input type="hidden" name="course_id" value="'.$course_id.'" />'."\n";
+                    $html .= '<input type="submit" class="button-primary" value="'.__('Save module order', 'woothemes-sensei').'" />'."\n";
+                    $html .= '<a href="'.admin_url('post.php?post='.$course_id.'&action=edit').'" class="button-secondary">'.__('Edit course', 'woothemes-sensei').'</a>'."\n";
                 }
             }
         }
@@ -944,7 +944,7 @@ 
     {
         if ($column == 'module_order') {
             if (has_term('', $this->taxonomy, $course_id)) {
-                echo '<a class="button-secondary" href="' . admin_url('edit.php?post_type=course&page=module-order&course_id=' . urlencode(intval($course_id))) . '">' . __('Order modules', 'woothemes-sensei') . '</a>';
+                echo '<a class="button-secondary" href="'.admin_url('edit.php?post_type=course&page=module-order&course_id='.urlencode(intval($course_id))).'">'.__('Order modules', 'woothemes-sensei').'</a>';
             }
         }
     }
@@ -1036,7 +1036,7 @@ 
                 $args['post_type'] = 'lesson';
                 $lessons = get_posts($args);
                 $total_lessons = count($lessons);
-                $column_data = '<a href="' . admin_url('edit.php?module=' . urlencode($module->slug) . '&post_type=lesson') . '">' . intval($total_lessons) . '</a>';
+                $column_data = '<a href="'.admin_url('edit.php?module='.urlencode($module->slug).'&post_type=lesson').'">'.intval($total_lessons).'</a>';
                 break;
         }
 
@@ -1054,7 +1054,7 @@ 
     public function analysis_overview_column_title($columns)
     {
 
-        if ( isset( $_GET['view'] ) && 'lessons' == $_GET['view'] ) {
+        if (isset($_GET['view']) && 'lessons' == $_GET['view']) {
             $new_columns = array();
             if (is_array($columns) && 0 < count($columns)) {
                 foreach ($columns as $column => $title) {
@@ -1082,15 +1082,15 @@ 
      * @param  WP_Post $lesson
      * @return array              Updated column data
      */
-    public function analysis_overview_column_data($columns, $lesson )
+    public function analysis_overview_column_data($columns, $lesson)
     {
 
-        if ( isset( $_GET['view'] ) && 'lessons' == $_GET['view'] ) {
+        if (isset($_GET['view']) && 'lessons' == $_GET['view']) {
             $lesson_module = '';
             $lesson_module_list = wp_get_post_terms($lesson->ID, $this->taxonomy);
             if (is_array($lesson_module_list) && count($lesson_module_list) > 0) {
                 foreach ($lesson_module_list as $single_module) {
-                    $lesson_module = '<a href="' . esc_url(admin_url('edit-tags.php?action=edit&taxonomy=' . urlencode($this->taxonomy) . '&tag_ID=' . urlencode($single_module->term_id))) . '">' . $single_module->name . '</a>';
+                    $lesson_module = '<a href="'.esc_url(admin_url('edit-tags.php?action=edit&taxonomy='.urlencode($this->taxonomy).'&tag_ID='.urlencode($single_module->term_id))).'">'.$single_module->name.'</a>';
                     break;
                 }
             }
@@ -1111,7 +1111,7 @@ 
      */
     public function analysis_course_column_title($columns)
     {
-        if ( isset( $_GET['view'] ) && 'lessons' == $_GET['view'] ) {
+        if (isset($_GET['view']) && 'lessons' == $_GET['view']) {
             $columns['lesson_module'] = __('Module', 'woothemes-sensei');
         }
         return $columns;
@@ -1126,15 +1126,15 @@ 
      * @param  WP_Post $lesson
      * @return array              Updated columns data
      */
-    public function analysis_course_column_data($columns, $lesson )
+    public function analysis_course_column_data($columns, $lesson)
     {
 
-        if ( isset( $_GET['course_id'] ) ) {
+        if (isset($_GET['course_id'])) {
             $lesson_module = '';
             $lesson_module_list = wp_get_post_terms($lesson->ID, $this->taxonomy);
             if (is_array($lesson_module_list) && count($lesson_module_list) > 0) {
                 foreach ($lesson_module_list as $single_module) {
-                    $lesson_module = '<a href="' . esc_url(admin_url('edit-tags.php?action=edit&taxonomy=' . urlencode($this->taxonomy) . '&tag_ID=' . urlencode($single_module->term_id))) . '">' . $single_module->name . '</a>';
+                    $lesson_module = '<a href="'.esc_url(admin_url('edit-tags.php?action=edit&taxonomy='.urlencode($this->taxonomy).'&tag_ID='.urlencode($single_module->term_id))).'">'.$single_module->name.'</a>';
                     break;
                 }
             }
@@ -1161,7 +1161,7 @@ 
     public function get_lesson_module($lesson_id = 0)
     {
         $lesson_id = intval($lesson_id);
-        if ( ! ( intval( $lesson_id > 0) ) ) {
+        if ( ! (intval($lesson_id > 0))) {
             return false;
         }
 
@@ -1169,13 +1169,13 @@ 
         $modules = wp_get_post_terms($lesson_id, $this->taxonomy);
 
         //check if error returned
-        if( empty( $modules ) || isset( $modules['errors']  ) ){
+        if (empty($modules) || isset($modules['errors'])) {
             return false;
         }
 
        // get the last item in the array there should be only be 1 really.
        // this method works for all php versions.
-       foreach( $modules as $module ){
+       foreach ($modules as $module) {
            break;
        }
 
@@ -1188,7 +1188,7 @@ 
         if (isset($course_id) && 0 < $course_id) {
 
             // the course should contain the same module taxonomy term for this to be valid
-            if( ! has_term( $module, $this->taxonomy, $course_id)){
+            if ( ! has_term($module, $this->taxonomy, $course_id)) {
                 return false;
             }
 
@@ -1262,8 +1262,8 @@ 
     {
 
 
-        wp_register_style($this->taxonomy . '-frontend', esc_url($this->assets_url) . 'css/modules-frontend.css', Sensei()->version );
-        wp_enqueue_style($this->taxonomy . '-frontend');
+        wp_register_style($this->taxonomy.'-frontend', esc_url($this->assets_url).'css/modules-frontend.css', Sensei()->version);
+        wp_enqueue_style($this->taxonomy.'-frontend');
     }
 
     /**
@@ -1273,37 +1273,37 @@ 
      *
      * @return void
      */
-    public function admin_enqueue_scripts( $hook ) {
+    public function admin_enqueue_scripts($hook) {
 
         /**
          * Filter the page hooks where modules admin script can be loaded on.
          *
          * @param array $white_listed_pages
          */
-        $script_on_pages_white_list = apply_filters( 'sensei_module_admin_script_page_white_lists', array(
+        $script_on_pages_white_list = apply_filters('sensei_module_admin_script_page_white_lists', array(
             'edit-tags.php',
             'course_page_module-order',
             'post-new.php',
             'post.php'
-        ) );
+        ));
 
-        if ( ! in_array( $hook, $script_on_pages_white_list ) ) {
+        if ( ! in_array($hook, $script_on_pages_white_list)) {
             return;
         }
 
-        $suffix = defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ? '' : '.min';
+        $suffix = defined('SCRIPT_DEBUG') && SCRIPT_DEBUG ? '' : '.min';
 
-        wp_enqueue_script( 'sensei-chosen', Sensei()->plugin_url . 'assets/chosen/chosen.jquery' . $suffix . '.js', array( 'jquery' ), Sensei()->version , true);
-        wp_enqueue_script( 'sensei-chosen-ajax', Sensei()->plugin_url . 'assets/chosen/ajax-chosen.jquery' . $suffix . '.js', array( 'jquery', 'sensei-chosen' ), Sensei()->version , true );
-        wp_enqueue_script( $this->taxonomy . '-admin', esc_url( $this->assets_url ) . 'js/modules-admin' . $suffix . '.js', array( 'jquery', 'sensei-chosen', 'sensei-chosen-ajax', 'jquery-ui-sortable', 'select2' ), Sensei()->version, true );
+        wp_enqueue_script('sensei-chosen', Sensei()->plugin_url.'assets/chosen/chosen.jquery'.$suffix.'.js', array('jquery'), Sensei()->version, true);
+        wp_enqueue_script('sensei-chosen-ajax', Sensei()->plugin_url.'assets/chosen/ajax-chosen.jquery'.$suffix.'.js', array('jquery', 'sensei-chosen'), Sensei()->version, true);
+        wp_enqueue_script($this->taxonomy.'-admin', esc_url($this->assets_url).'js/modules-admin'.$suffix.'.js', array('jquery', 'sensei-chosen', 'sensei-chosen-ajax', 'jquery-ui-sortable', 'select2'), Sensei()->version, true);
 
         // localized module data
         $localize_modulesAdmin = array(
-            'search_courses_nonce' => wp_create_nonce( 'search-courses' ),
-            'selectPlaceholder'    => __( 'Search for courses', 'woothemes-sensei' )
+            'search_courses_nonce' => wp_create_nonce('search-courses'),
+            'selectPlaceholder'    => __('Search for courses', 'woothemes-sensei')
         );
 
-        wp_localize_script( $this->taxonomy . '-admin' ,'modulesAdmin', $localize_modulesAdmin );
+        wp_localize_script($this->taxonomy.'-admin', 'modulesAdmin', $localize_modulesAdmin);
     }
 
     /**
@@ -1316,8 +1316,8 @@ 
     public function admin_enqueue_styles() {
 
 
-        wp_register_style($this->taxonomy . '-sortable', esc_url($this->assets_url) . 'css/modules-admin.css','',Sensei()->version );
-        wp_enqueue_style($this->taxonomy . '-sortable');
+        wp_register_style($this->taxonomy.'-sortable', esc_url($this->assets_url).'css/modules-admin.css', '', Sensei()->version);
+        wp_enqueue_style($this->taxonomy.'-sortable');
 
     }
 
@@ -1331,9 +1331,9 @@ 
      */
     public function course_modules_title( ) {
 
-       if( sensei_module_has_lessons() ){
+       if (sensei_module_has_lessons()) {
 
-            echo '<header><h2>' . __('Modules', 'woothemes-sensei') . '</h2></header>';
+            echo '<header><h2>'.__('Modules', 'woothemes-sensei').'</h2></header>';
 
         }
 
@@ -1346,18 +1346,18 @@ 
      * @since 1.8.0
      * @return void
      */
-    public function load_course_module_content_template(){
+    public function load_course_module_content_template() {
 
         // load backwards compatible template name if it exists in the users theme
-        $located_template= locate_template( Sensei()->template_url . 'single-course/course-modules.php' );
-        if( $located_template ){
+        $located_template = locate_template(Sensei()->template_url.'single-course/course-modules.php');
+        if ($located_template) {
 
-            Sensei_Templates::get_template( 'single-course/course-modules.php' );
+            Sensei_Templates::get_template('single-course/course-modules.php');
             return;
 
         }
 
-        Sensei_Templates::get_template( 'single-course/modules.php' );
+        Sensei_Templates::get_template('single-course/modules.php');
 
     } // end course_module_content
 
@@ -1370,15 +1370,15 @@ 
      * @param $term_id
      * @return array $lessons
      */
-    public function get_lessons( $course_id , $term_id ){
+    public function get_lessons($course_id, $term_id) {
 
-        $lesson_query = $this->get_lessons_query( $course_id, $term_id );
+        $lesson_query = $this->get_lessons_query($course_id, $term_id);
 
-        if( isset( $lesson_query->posts ) ){
+        if (isset($lesson_query->posts)) {
 
             return $lesson_query->posts;
 
-        }else{
+        } else {
 
             return array();
 
@@ -1395,9 +1395,9 @@ 
      * @param $term_id
      * @return WP_Query $lessons_query
      */
-    public function get_lessons_query( $course_id , $term_id ){
+    public function get_lessons_query($course_id, $term_id) {
 
-        if( empty( $term_id ) || empty( $course_id ) ){
+        if (empty($term_id) || empty($course_id)) {
 
             return array();
 
@@ -1418,7 +1418,7 @@ 
                 array(
                     'taxonomy' => 'module',
                     'field' => 'id',
-                    'terms' => intval( $term_id )
+                    'terms' => intval($term_id)
                 )
             ),
             'orderby' => 'menu_order',
@@ -1426,12 +1426,12 @@ 
             'suppress_filters' => 0
         );
 
-        if (version_compare( Sensei()->version, '1.6.0', '>=')) {
-            $args['meta_key'] = '_order_module_' . intval( $term_id );
+        if (version_compare(Sensei()->version, '1.6.0', '>=')) {
+            $args['meta_key'] = '_order_module_'.intval($term_id);
             $args['orderby'] = 'meta_value_num date';
         }
 
-        $lessons_query = new WP_Query( $args );
+        $lessons_query = new WP_Query($args);
 
         return $lessons_query;
 
@@ -1446,37 +1446,37 @@ 
      *
      * @return array $non_module_lessons
      */
-    public function get_none_module_lessons( $course_id ){
+    public function get_none_module_lessons($course_id) {
 
         $non_module_lessons = array();
 
         //exit if there is no course id passed in
-        if( empty( $course_id ) || 'course' != get_post_type( $course_id ) ) {
+        if (empty($course_id) || 'course' != get_post_type($course_id)) {
 
             return $non_module_lessons;
         }
 
         //save some time and check if we already have the saved
-        if( get_transient( 'sensei_'. $course_id .'_none_module_lessons') ){
+        if (get_transient('sensei_'.$course_id.'_none_module_lessons')) {
 
-            return get_transient( 'sensei_'. $course_id .'_none_module_lessons');
+            return get_transient('sensei_'.$course_id.'_none_module_lessons');
 
         }
 
         // create terms array which must be excluded from other arrays
-        $course_modules = $this->get_course_modules( $course_id );
+        $course_modules = $this->get_course_modules($course_id);
 
         //exit if there are no module on this course
-        if( empty( $course_modules ) || ! is_array( $course_modules ) ){
+        if (empty($course_modules) || ! is_array($course_modules)) {
 
-            return  Sensei()->course->course_lessons( $course_id );
+            return  Sensei()->course->course_lessons($course_id);
 
         }
 
         $terms = array();
-        foreach( $course_modules as $module ){
+        foreach ($course_modules as $module) {
 
-            array_push( $terms ,  $module->term_id );
+            array_push($terms, $module->term_id);
 
         }
 
@@ -1487,7 +1487,7 @@ 
             'meta_query' => array(
                 array(
                     'key' => '_lesson_course',
-                    'value' => intval( $course_id ),
+                    'value' => intval($course_id),
                     'compare' => '='
                 )
             ),
@@ -1504,11 +1504,11 @@ 
             'suppress_filters' => 0
         );
 
-        $wp_lessons_query = new WP_Query( $args );
+        $wp_lessons_query = new WP_Query($args);
 
-        if( isset( $wp_lessons_query->posts) && count( $wp_lessons_query->posts ) > 0  ){
+        if (isset($wp_lessons_query->posts) && count($wp_lessons_query->posts) > 0) {
             $non_module_lessons = $wp_lessons_query->get_posts();
-            set_transient( 'sensei_'. $course_id .'_none_module_lessons', $non_module_lessons, 10 * DAY_IN_SECONDS );
+            set_transient('sensei_'.$course_id.'_none_module_lessons', $non_module_lessons, 10 * DAY_IN_SECONDS);
         }
 
         return $non_module_lessons;
@@ -1519,7 +1519,7 @@ 
      *
      * @since 1.8.0
      */
-    public function setup_modules_taxonomy(){
+    public function setup_modules_taxonomy() {
 
         $labels = array(
             'name' => __('Modules', 'woothemes-sensei'),
@@ -1556,11 +1556,11 @@ 
             'show_in_nav_menus' => false,
             'show_in_quick_edit' => false,
             'show_ui' => true,
-            'rewrite' => array('slug' => $modules_rewrite_slug ),
+            'rewrite' => array('slug' => $modules_rewrite_slug),
             'labels' => $labels
         );
 
-        register_taxonomy( 'module' , array('course', 'lesson'), $args);
+        register_taxonomy('module', array('course', 'lesson'), $args);
 
     }// end setup_modules_taxonomy
 
@@ -1572,12 +1572,12 @@ 
      * @since 1.8.0
      * @return void
      */
-    function redirect_to_lesson_module_taxonomy_to_course( ){
+    function redirect_to_lesson_module_taxonomy_to_course( ) {
 
-        global $typenow , $taxnow;
+        global $typenow, $taxnow;
 
-        if( 'lesson'== $typenow && 'module'==$taxnow ){
-            wp_safe_redirect( esc_url_raw( 'edit-tags.php?taxonomy=module&post_type=course'  ) );
+        if ('lesson' == $typenow && 'module' == $taxnow) {
+            wp_safe_redirect(esc_url_raw('edit-tags.php?taxonomy=module&post_type=course'));
         }
 
     }// end redirect to course taxonomy
@@ -1590,18 +1590,18 @@ 
      * @since 1.8.0
      * @return void
      */
-    public function remove_lessons_menu_model_taxonomy(){
+    public function remove_lessons_menu_model_taxonomy() {
         global $submenu;
 
-        if( ! isset( $submenu['edit.php?post_type=lesson'] ) || !is_array( $submenu['edit.php?post_type=lesson'] ) ){
+        if ( ! isset($submenu['edit.php?post_type=lesson']) || ! is_array($submenu['edit.php?post_type=lesson'])) {
             return; // exit
         }
 
         $lesson_main_menu = $submenu['edit.php?post_type=lesson'];
-        foreach( $lesson_main_menu as $index => $sub_item ){
+        foreach ($lesson_main_menu as $index => $sub_item) {
 
-            if( 'edit-tags.php?taxonomy=module&amp;post_type=lesson' == $sub_item[2] ){
-                unset( $submenu['edit.php?post_type=lesson'][ $index ]);
+            if ('edit-tags.php?taxonomy=module&amp;post_type=lesson' == $sub_item[2]) {
+                unset($submenu['edit.php?post_type=lesson'][$index]);
             }
         }
 
@@ -1615,18 +1615,18 @@ 
      * @since 1.8.0
      * @return void
      */
-    public function remove_courses_menu_model_taxonomy(){
+    public function remove_courses_menu_model_taxonomy() {
         global $submenu;
 
-        if( ! isset( $submenu['edit.php?post_type=course'] ) || !is_array( $submenu['edit.php?post_type=course'] ) ){
+        if ( ! isset($submenu['edit.php?post_type=course']) || ! is_array($submenu['edit.php?post_type=course'])) {
             return; // exit
         }
 
         $course_main_menu = $submenu['edit.php?post_type=course'];
-        foreach( $course_main_menu as $index => $sub_item ){
+        foreach ($course_main_menu as $index => $sub_item) {
 
-            if( 'edit-tags.php?taxonomy=module&amp;post_type=course' == $sub_item[2] ){
-                unset( $submenu['edit.php?post_type=course'][ $index ]);
+            if ('edit-tags.php?taxonomy=module&amp;post_type=course' == $sub_item[2]) {
+                unset($submenu['edit.php?post_type=course'][$index]);
             }
         }
 
@@ -1642,12 +1642,12 @@ 
      * @param string $term_name
      * @return array $owners { type WP_User }. Empty array if none if found.
      */
-    public static function get_term_authors( $term_name ){
+    public static function get_term_authors($term_name) {
 
-        $terms = get_terms( array( 'module') , array( 'name__like'=>$term_name, 'hide_empty' => false )  );
+        $terms = get_terms(array('module'), array('name__like'=>$term_name, 'hide_empty' => false));
 
         $owners = array();
-        if( empty( $terms ) ){
+        if (empty($terms)) {
 
             return $owners;
 
@@ -1657,11 +1657,11 @@ 
 
 
         //if there are more handle them appropriately and get the ones we really need that matches the desired name exactly
-        foreach( $terms as $term){
-            if( $term->name == $term_name ){
+        foreach ($terms as $term) {
+            if ($term->name == $term_name) {
 
                 // look for the author in the slug
-                $owners[] = Sensei_Core_Modules::get_term_author( $term->slug  );
+                $owners[] = Sensei_Core_Modules::get_term_author($term->slug);
 
             }// end if term name
 
@@ -1682,30 +1682,30 @@ 
      * @param $slug
      * @return WP_User $author if no author is found or invalid term is passed the admin user will be returned.
      */
-    public static function get_term_author( $slug='' ){
+    public static function get_term_author($slug = '') {
 
-        $term_owner = get_user_by( 'email', get_bloginfo( 'admin_email' ) );
+        $term_owner = get_user_by('email', get_bloginfo('admin_email'));
 
-        if( empty( $slug ) ){
+        if (empty($slug)) {
 
             return $term_owner;
 
         }
 
         // look for the author in the slug
-        $slug_parts = explode( '-', $slug );
+        $slug_parts = explode('-', $slug);
 
-        if( count( $slug_parts ) > 1 ){
+        if (count($slug_parts) > 1) {
 
             // get the user data
             $possible_user_id = $slug_parts[0];
-            $author = get_userdata( $possible_user_id );
+            $author = get_userdata($possible_user_id);
 
             // if the user doesnt exist for the first part of the slug
             // then this slug was also created by admin
-            if( is_a( $author, 'WP_User' ) ){
+            if (is_a($author, 'WP_User')) {
 
-                $term_owner =  $author;
+                $term_owner = $author;
 
             }
         }
@@ -1722,48 +1722,48 @@ 
      *
      * @param WP_Post $post Post object.
      */
-    public function course_module_metabox( $post ) {
+    public function course_module_metabox($post) {
 
         $tax_name = 'module';
-        $taxonomy = get_taxonomy( 'module' );
+        $taxonomy = get_taxonomy('module');
 
         ?>
         <div id="taxonomy-<?php echo $tax_name; ?>" class="categorydiv">
             <ul id="<?php echo $tax_name; ?>-tabs" class="category-tabs">
                 <li class="tabs"><a href="#<?php echo $tax_name; ?>-all"><?php echo $taxonomy->labels->all_items; ?></a></li>
-                <li class="hide-if-no-js"><a href="#<?php echo $tax_name; ?>-pop"><?php _e( 'Most Used' ); ?></a></li>
+                <li class="hide-if-no-js"><a href="#<?php echo $tax_name; ?>-pop"><?php _e('Most Used'); ?></a></li>
             </ul>
 
             <div id="<?php echo $tax_name; ?>-pop" class="tabs-panel" style="display: none;">
                 <ul id="<?php echo $tax_name; ?>checklist-pop" class="categorychecklist form-no-clear" >
-                    <?php $popular_ids = wp_popular_terms_checklist( $tax_name ); ?>
+                    <?php $popular_ids = wp_popular_terms_checklist($tax_name); ?>
                 </ul>
             </div>
 
             <div id="<?php echo $tax_name; ?>-all" class="tabs-panel">
                 <?php
-                $name = ( $tax_name == 'category' ) ? 'post_category' : 'tax_input[' . $tax_name . ']';
+                $name = ($tax_name == 'category') ? 'post_category' : 'tax_input['.$tax_name.']';
                 echo "<input type='hidden' name='{$name}[]' value='0' />"; // Allows for an empty term set to be sent. 0 is an invalid Term ID and will be ignored by empty() checks.
                 ?>
                 <ul id="<?php echo $tax_name; ?>checklist" data-wp-lists="list:<?php echo $tax_name; ?>" class="categorychecklist form-no-clear">
-                    <?php wp_terms_checklist( $post->ID, array( 'taxonomy'=>$tax_name , 'popular_cats' => $popular_ids ) ); ?>
+                    <?php wp_terms_checklist($post->ID, array('taxonomy'=>$tax_name, 'popular_cats' => $popular_ids)); ?>
                 </ul>
             </div>
-            <?php if ( current_user_can( $taxonomy->cap->edit_terms ) ) : ?>
+            <?php if (current_user_can($taxonomy->cap->edit_terms)) : ?>
                 <div id="<?php echo $tax_name; ?>-adder" class="wp-hidden-children">
                     <h4>
                         <a id="sensei-<?php echo $tax_name; ?>-add-toggle" href="#<?php echo $tax_name; ?>-add" class="hide-if-no-js">
                             <?php
                             /* translators: %s: add new taxonomy label */
-                            printf( __( '+ %s' ), $taxonomy->labels->add_new_item );
+                            printf(__('+ %s'), $taxonomy->labels->add_new_item);
                             ?>
                         </a>
                     </h4>
                     <p id="sensei-<?php echo $tax_name; ?>-add" class="category-add wp-hidden-child">
                         <label class="screen-reader-text" for="new<?php echo $tax_name; ?>"><?php echo $taxonomy->labels->add_new_item; ?></label>
-                        <input type="text" name="new<?php echo $tax_name; ?>" id="new<?php echo $tax_name; ?>" class="form-required form-input-tip" value="<?php echo esc_attr( $taxonomy->labels->new_item_name ); ?>" aria-required="true"/>
-                        <a class="button" id="sensei-<?php echo $tax_name; ?>-add-submit" class="button category-add-submit"><?php echo esc_attr( $taxonomy->labels->add_new_item ); ?></a>
-                        <?php wp_nonce_field( '_ajax_nonce-add-' . $tax_name, 'add_module_nonce' ); ?>
+                        <input type="text" name="new<?php echo $tax_name; ?>" id="new<?php echo $tax_name; ?>" class="form-required form-input-tip" value="<?php echo esc_attr($taxonomy->labels->new_item_name); ?>" aria-required="true"/>
+                        <a class="button" id="sensei-<?php echo $tax_name; ?>-add-submit" class="button category-add-submit"><?php echo esc_attr($taxonomy->labels->add_new_item); ?></a>
+                        <?php wp_nonce_field('_ajax_nonce-add-'.$tax_name, 'add_module_nonce'); ?>
                         <span id="<?php echo $tax_name; ?>-ajax-response"></span>
                     </p>
                 </div>
@@ -1783,55 +1783,55 @@ 
     public static function add_new_module_term( ) {
 
 
-        if( ! isset( $_POST[ 'security' ] ) || ! wp_verify_nonce( $_POST[ 'security' ], '_ajax_nonce-add-module'  ) ){
-            wp_send_json_error( array('error'=> 'wrong security nonce') );
+        if ( ! isset($_POST['security']) || ! wp_verify_nonce($_POST['security'], '_ajax_nonce-add-module')) {
+            wp_send_json_error(array('error'=> 'wrong security nonce'));
         }
 
         // get the term an create the new term storing infomration
-        $term_name = sanitize_text_field( $_POST['newTerm'] );
+        $term_name = sanitize_text_field($_POST['newTerm']);
 
-        if( current_user_can('manage_options' ) ) {
+        if (current_user_can('manage_options')) {
 
-            $term_slug = str_ireplace(' ', '-', trim( $term_name ) );
+            $term_slug = str_ireplace(' ', '-', trim($term_name));
 
         } else {
 
-            $term_slug =  get_current_user_id() . '-' . str_ireplace(' ', '-', trim( $term_name ) );
+            $term_slug = get_current_user_id().'-'.str_ireplace(' ', '-', trim($term_name));
 
         }
 
-        $course_id = sanitize_text_field( $_POST['course_id'] );
+        $course_id = sanitize_text_field($_POST['course_id']);
 
         // save the term
-        $slug = wp_insert_term( $term_name,'module', array('slug'=> $term_slug)  );
+        $slug = wp_insert_term($term_name, 'module', array('slug'=> $term_slug));
 
         // send error for all errors except term exits
-        if( is_wp_error( $slug ) ){
+        if (is_wp_error($slug)) {
 
             // prepare for possible term name and id to be passed down if term exists
             $term_data = array();
 
             // if term exists also send back the term name and id
-            if( isset( $slug->errors['term_exists'] ) ){
+            if (isset($slug->errors['term_exists'])) {
 
-                $term = get_term_by( 'slug', $term_slug, 'module');
+                $term = get_term_by('slug', $term_slug, 'module');
                 $term_data['name'] = $term_name;
                 $term_data['id'] = $term->term_id;
 
                 // set the object terms
-                wp_set_object_terms( $course_id, $term->term_id, 'module', true );
+                wp_set_object_terms($course_id, $term->term_id, 'module', true);
             }
 
-            wp_send_json_error(array( 'errors'=>$slug->errors , 'term'=> $term_data ) );
+            wp_send_json_error(array('errors'=>$slug->errors, 'term'=> $term_data));
 
         }
 
         //make sure the new term is checked for this course
 
-        wp_set_object_terms( $course_id, $slug['term_id'], 'module', true );
+        wp_set_object_terms($course_id, $slug['term_id'], 'module', true);
 
         // Handle request then generate response using WP_Ajax_Response
-        wp_send_json_success( array( 'termId' => $slug['term_id'], 'termName' => $term_name ) );
+        wp_send_json_success(array('termId' => $slug['term_id'], 'termName' => $term_name));
 
     }
 
@@ -1843,26 +1843,26 @@ 
      *
      * @since 1.8.0
      */
-    public function filter_module_terms( $terms, $taxonomies, $args ){
+    public function filter_module_terms($terms, $taxonomies, $args) {
 
         //dont limit for admins and other taxonomies. This should also only apply to admin
-        if( current_user_can( 'manage_options' ) || !in_array( 'module', $taxonomies ) || ! is_admin()  ){
+        if (current_user_can('manage_options') || ! in_array('module', $taxonomies) || ! is_admin()) {
             return $terms;
         }
 
         // avoid infinite call loop
-        remove_filter('get_terms', array( $this, 'filter_module_terms' ), 20, 3 );
+        remove_filter('get_terms', array($this, 'filter_module_terms'), 20, 3);
 
         // in certain cases the array is passed in as reference to the parent term_id => parent_id
-        if( isset( $args['fields'] ) && 'id=>parent' == $args['fields'] ){
+        if (isset($args['fields']) && 'id=>parent' == $args['fields']) {
             // change only scrub the terms ids form the array keys
-            $terms = array_keys( $terms );
+            $terms = array_keys($terms);
         }
 
-        $teachers_terms =  $this->filter_terms_by_owner( $terms, get_current_user_id() );
+        $teachers_terms = $this->filter_terms_by_owner($terms, get_current_user_id());
 
         // add filter again as removed above
-        add_filter('get_terms', array( $this, 'filter_module_terms' ), 20, 3 );
+        add_filter('get_terms', array($this, 'filter_module_terms'), 20, 3);
 
         return $teachers_terms;
     }// end filter_module_terms
@@ -1875,25 +1875,25 @@ 
      *
      * @since 1.8.0
      */
-    public function filter_course_selected_terms( $terms, $course_ids_array, $taxonomies ){
+    public function filter_course_selected_terms($terms, $course_ids_array, $taxonomies) {
 
         //dont limit for admins and other taxonomies. This should also only apply to admin
-        if( current_user_can( 'manage_options' ) || ! is_admin() || empty( $terms )
+        if (current_user_can('manage_options') || ! is_admin() || empty($terms)
             // only apply this to module only taxonomy queries so 1 taxonomy only:
-            ||  count( $taxonomies ) > 1 || !in_array( 'module', $taxonomies )  ){
+            ||  count($taxonomies) > 1 || ! in_array('module', $taxonomies)) {
             return $terms;
         }
 
-        $term_objects = $this->filter_terms_by_owner( $terms, get_current_user_id() );
+        $term_objects = $this->filter_terms_by_owner($terms, get_current_user_id());
 
         // if term objects were passed in send back objects
         // if term id were passed in send that back
-        if( is_object( $terms[0] ) ){
+        if (is_object($terms[0])) {
             return $term_objects;
         }
 
         $terms = array();
-        foreach( $term_objects as $term_object ){
+        foreach ($term_objects as $term_object) {
             $terms[] = $term_object->term_id;
         }
 
@@ -1911,20 +1911,20 @@ 
      * @param $user_id
      * @return array
      */
-    public function filter_terms_by_owner( $terms, $user_id ){
+    public function filter_terms_by_owner($terms, $user_id) {
 
         $users_terms = array();
 
-        foreach( $terms as $index => $term ){
+        foreach ($terms as $index => $term) {
 
-            if( is_numeric( $term ) ){
+            if (is_numeric($term)) {
                 // the term id was given, get the term object
-                $term = get_term( $term, 'module' );
+                $term = get_term($term, 'module');
             }
 
-            $author = Sensei_Core_Modules::get_term_author( $term->slug );
+            $author = Sensei_Core_Modules::get_term_author($term->slug);
 
-            if ( $user_id == $author->ID ) {
+            if ($user_id == $author->ID) {
                 // add the term to the teachers terms
                 $users_terms[] = $term;
             }
@@ -1943,34 +1943,34 @@ 
      *
      * @since 1.8.0
      */
-    public function append_teacher_name_to_module( $terms, $taxonomies, $args )
+    public function append_teacher_name_to_module($terms, $taxonomies, $args)
     {
 
         // only for admin users ont he module taxonomy
-        if ( empty( $terms ) || !current_user_can('manage_options') || !in_array('module', $taxonomies) || !is_admin()) {
+        if (empty($terms) || ! current_user_can('manage_options') || ! in_array('module', $taxonomies) || ! is_admin()) {
             return $terms;
         }
 
         // in certain cases the array is passed in as reference to the parent term_id => parent_id
         // simply return this as wp doesn't need an array of stdObject Term
-        if (isset( $args['fields'] ) && 'id=>parent' == $args['fields']) {
+        if (isset($args['fields']) && 'id=>parent' == $args['fields']) {
 
             return $terms;
 
         }
 
         // loop through and update all terms adding the author name
-        foreach( $terms as $index => $term ){
+        foreach ($terms as $index => $term) {
 
-            if( is_numeric( $term ) ){
+            if (is_numeric($term)) {
                 // the term id was given, get the term object
-                $term = get_term( $term, 'module' );
+                $term = get_term($term, 'module');
             }
 
-            $author = Sensei_Core_Modules::get_term_author( $term->slug );
+            $author = Sensei_Core_Modules::get_term_author($term->slug);
 
-            if( ! user_can( $author, 'manage_options' ) ) {
-                $term->name = $term->name . ' (' . $author->display_name . ') ';
+            if ( ! user_can($author, 'manage_options')) {
+                $term->name = $term->name.' ('.$author->display_name.') ';
             }
 
             // add the term to the teachers terms
@@ -1999,23 +1999,23 @@ 
      * @sine 1.9.0
      * @param $post_id
      */
-    public static function reset_none_modules_transient ( $post_id ){
+    public static function reset_none_modules_transient($post_id) {
 
         // this should only apply to course and lesson post types
-        if( in_array( get_post_type( $post_id ), array( 'course', 'lesson' ) ) ){
+        if (in_array(get_post_type($post_id), array('course', 'lesson'))) {
 
             $course_id = '';
 
-            if( 'lesson' == get_post_type( $post_id ) ){
+            if ('lesson' == get_post_type($post_id)) {
 
-                $course_id = Sensei()->lesson->get_course_id( $post_id );
+                $course_id = Sensei()->lesson->get_course_id($post_id);
 
             }
 
 
-            if( !empty( $course_id ) ){
+            if ( ! empty($course_id)) {
 
-                delete_transient( 'sensei_'. $course_id .'_none_module_lessons' );
+                delete_transient('sensei_'.$course_id.'_none_module_lessons');
 
             }
 
@@ -2030,9 +2030,9 @@ 
      * @deprecated since 1.9.0
      *
      */
-    public static function deprecate_sensei_single_course_modules_content(){
+    public static function deprecate_sensei_single_course_modules_content() {
 
-        sensei_do_deprecated_action( 'sensei_single_course_modules_content','1.9.0','sensei_single_course_modules_before or sensei_single_course_modules_after' );
+        sensei_do_deprecated_action('sensei_single_course_modules_content', '1.9.0', 'sensei_single_course_modules_before or sensei_single_course_modules_after');
 
     }
 
@@ -2043,12 +2043,12 @@ 
      *
      * @since 1.9.0
      */
-    public static function setup_single_course_module_loop(){
+    public static function setup_single_course_module_loop() {
 
         global $sensei_modules_loop, $post;
         $course_id = $post->ID;
 
-        $modules = Sensei()->modules->get_course_modules( $course_id );
+        $modules = Sensei()->modules->get_course_modules($course_id);
 
         //initial setup
         $sensei_modules_loop['total'] = 0;
@@ -2056,22 +2056,22 @@ 
         $sensei_modules_loop['current'] = -1;
 
         // exit if this course doesn't have modules
-        if( !$modules || empty( $modules )  ){
+        if ( ! $modules || empty($modules)) {
             return;
         }
 
 
         $lessons_in_all_modules = array();
-        foreach( $modules as $term ){
+        foreach ($modules as $term) {
 
-            $lessons_in_this_module = Sensei()->modules->get_lessons( $course_id , $term->term_id);
-            $lessons_in_all_modules = array_merge(  $lessons_in_all_modules, $lessons_in_this_module  );
+            $lessons_in_this_module = Sensei()->modules->get_lessons($course_id, $term->term_id);
+            $lessons_in_all_modules = array_merge($lessons_in_all_modules, $lessons_in_this_module);
 
         }
 
 
         //setup all of the modules loop variables
-        $sensei_modules_loop['total'] = count( $modules );
+        $sensei_modules_loop['total'] = count($modules);
         $sensei_modules_loop['modules'] = $modules;
         $sensei_modules_loop['current'] = -1;
         $sensei_modules_loop['course_id'] = $course_id;
@@ -2084,7 +2084,7 @@ 
      * @since 1.9.0
      *
      */
-    public static function teardown_single_course_module_loop(){
+    public static function teardown_single_course_module_loop() {
 
         global $sensei_modules_loop, $wp_query, $post;
 

includes/class-sensei-notices.php

Doc Comments -8 removed lines

@@ -33,10 +33,6 @@ 
 	* 
 	*
 	* 
-	* @param string $message 
-	* @param string $type defaults to alert options( alert, tick , download , note   )
-	*
-	* @return void
 	*/
 
 	public function add_notice( $content ,  $type = 'alert'   ){
@@ -47,10 +43,6 @@ 
 	/**
 	*  Output all notices added 
 	* 
-	* @param string $message 
-	* @param string $type
-	*
-	* @return void
 	*/
 
 	public function print_notices(){

Spacing +11 added lines, -11 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Notices Class
@@ -13,7 +13,7 @@ 
  * @since 1.6.3
  */
 
-class Sensei_Notices{
+class Sensei_Notices {
 
 	/**
 	*  @var $notices
@@ -23,7 +23,7 @@ 
 	/**
 	*  constructor 
  	*/
-	public function __construct(){
+	public function __construct() {
 		//initialize the notices variable
 		$this->notices = array();
 	}
@@ -39,9 +39,9 @@ 
 	* @return void
 	*/
 
-	public function add_notice( $content ,  $type = 'alert'   ){
+	public function add_notice($content, $type = 'alert') {
 		// append the new notice
-		$this->notices[] = array('content' => $content , 'type'=> $type );
+		$this->notices[] = array('content' => $content, 'type'=> $type);
 	} // end add_notice()
 
 	/**
@@ -53,12 +53,12 @@ 
 	* @return void
 	*/
 
-	public function print_notices(){
-		if(  count( $this->notices ) > 0  ){
+	public function print_notices() {
+		if (count($this->notices) > 0) {
 			foreach ($this->notices  as  $notice) {
 
-				$classes = 'sensei-message '. $notice['type'];
-				$html = '<div class="'. $classes . '">'. $notice['content'] . '</div>';
+				$classes = 'sensei-message '.$notice['type'];
+				$html = '<div class="'.$classes.'">'.$notice['content'].'</div>';
 
 				echo $html; 
 			}
@@ -73,7 +73,7 @@ 
 	* @return void
 	*/
 
-	public function clear_notices(){
+	public function clear_notices() {
 		// assign an empty array to clear all existing notices
 		$this->notices = array();
 	} // end clear_notices()
@@ -85,4 +85,4 @@ 
  * for backward compatibility
  * @since 1.9.0
  */
-class Woothemes_Sensei_Notices extends Sensei_Notices{}
+class Woothemes_Sensei_Notices extends Sensei_Notices {}