woothemes / sensei

includes/admin/class-sensei-learner-management.php

Spacing +137 added lines, -137 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
@@ -22,30 +22,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()
 
@@ -58,8 +58,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()
@@ -72,25 +72,25 @@ 
 	 * @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','sensei-core-select2','sensei-chosen-ajax' ), Sensei()->version, true );
+		wp_enqueue_script('sensei-learners-general',
+            Sensei()->plugin_url.'assets/js/learners-general'.$suffix.'.js',
+                            array('jquery', 'sensei-core-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_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_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()
 
@@ -102,9 +102,9 @@ 
 	 * @since 1.6.0
 	 * @return void
 	 */
-	public function enqueue_styles () {
+	public function enqueue_styles() {
 
-		wp_enqueue_style( 'woothemes-sensei-admin' );
+		wp_enqueue_style('woothemes-sensei-admin');
 
 	} // End enqueue_styles()
 
@@ -116,11 +116,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()
@@ -133,15 +133,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;
@@ -158,16 +158,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">
@@ -175,12 +175,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,14 +189,14 @@ 
      * @param array $args
 	 * @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 ) {
+	public function wrapper_container($which) {
+		if ('top' == $which) {
 			?><div id="woothemes-sensei" class="wrap woothemes-sensei"><?php
-		} elseif ( 'bottom' == $which ) {
+		} 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,22 +240,22 @@ 
 		// 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();
 	}
 
 	public function remove_user_from_post() {
 
         // Parse POST data
-        $data = sanitize_text_field( $_POST['data'] );
+        $data = sanitize_text_field($_POST['data']);
         $action_data = array();
-        parse_str( $data, $action_data );
+        parse_str($data, $action_data);
 
 		// Security checks
         // ensure the current user may remove users from post
@@ -263,54 +263,54 @@ 
 
         // check the nonce, valid post
 		$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']);
 		}
-        $post =  get_post( intval( $action_data[ 'post_id' ] ) );
+        $post = get_post(intval($action_data['post_id']));
 
         // validate the user
         $may_remove_user = false;
-        if( current_user_can('manage_sensei')
-            ||  $post->post_author == get_current_user_id() ){
+        if (current_user_can('manage_sensei')
+            ||  $post->post_author == get_current_user_id()) {
 
             $may_remove_user = true;
 
         }
 
-        if( ! wp_verify_nonce( $nonce, 'remove_user_from_post_nonce' )
-            || ! is_a( $post ,'WP_Post' )
-            || ! $may_remove_user ){
+        if ( ! wp_verify_nonce($nonce, 'remove_user_from_post_nonce')
+            || ! is_a($post, 'WP_Post')
+            || ! $may_remove_user) {
 
             die('');
 
         }
 
-		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'] );
+			$user_id = intval($action_data['user_id']);
+			$post_id = intval($action_data['post_id']);
+			$post_type = sanitize_text_field($action_data['post_type']);
 
-			$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);
 
 				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 ) {
-				die( 'removed' );
+			if ($removed) {
+				die('removed');
 			}
 
 		}
@@ -321,109 +321,109 @@ 
 	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_Learner::get_full_name( $user->ID );
+		if ($users) {
+			foreach ($users as $user) {
+                $full_name = Sensei_Learner::get_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;
 			}
 
@@ -433,23 +433,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'),
 				);
 			}
 			?>
@@ -472,9 +472,9 @@ 
      *
      * @return string $full_name
      */
-    public function get_learner_full_name( $user_id ){
+    public function get_learner_full_name($user_id) {
 
-        return Sensei_Learner::get_full_name( $user_id );
+        return Sensei_Learner::get_full_name($user_id);
 
     } // end get_learner_full_name
 
@@ -485,4 +485,4 @@ 
  * @ignore only for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Learners extends Sensei_Learner_Management{}
+class WooThemes_Sensei_Learners extends Sensei_Learner_Management {}

includes/class-sensei-question.php

Spacing +287 added lines, -287 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Question Class
@@ -19,35 +19,35 @@ 
 	 * Constructor.
 	 * @since  1.0.0
 	 */
-	public function __construct () {
+	public function __construct() {
         $this->token = 'question';
 		$this->question_types = $this->question_types();
-		$this->meta_fields = array( 'question_right_answer', 'question_wrong_answers' );
-		if ( is_admin() ) {
+		$this->meta_fields = array('question_right_answer', 'question_wrong_answers');
+		if (is_admin()) {
 			// Custom Write Panel Columns
-			add_filter( 'manage_edit-question_columns', array( $this, 'add_column_headings' ), 10, 1 );
-			add_action( 'manage_posts_custom_column', array( $this, 'add_column_data' ), 10, 2 );
-			add_action( 'add_meta_boxes', array( $this, 'question_edit_panel_metabox' ), 10, 2 );
+			add_filter('manage_edit-question_columns', array($this, 'add_column_headings'), 10, 1);
+			add_action('manage_posts_custom_column', array($this, 'add_column_data'), 10, 2);
+			add_action('add_meta_boxes', array($this, 'question_edit_panel_metabox'), 10, 2);
 
 			// Quesitno list table filters
-			add_action( 'restrict_manage_posts', array( $this, 'filter_options' ) );
-			add_filter( 'request', array( $this, 'filter_actions' ) );
+			add_action('restrict_manage_posts', array($this, 'filter_options'));
+			add_filter('request', array($this, 'filter_actions'));
 
-			add_action( 'save_post', array( $this, 'save_question' ), 10, 1 );
+			add_action('save_post', array($this, 'save_question'), 10, 1);
 		} // End If Statement
 	} // End __construct()
 
 	public function question_types() {
 		$types = array(
-			'multiple-choice' => __( 'Multiple Choice', 'woothemes-sensei' ),
-			'boolean' => __( 'True/False', 'woothemes-sensei' ),
-			'gap-fill' => __( 'Gap Fill', 'woothemes-sensei' ),
-			'single-line' => __( 'Single Line', 'woothemes-sensei' ),
-			'multi-line' => __( 'Multi Line', 'woothemes-sensei' ),
-			'file-upload' => __( 'File Upload', 'woothemes-sensei' ),
+			'multiple-choice' => __('Multiple Choice', 'woothemes-sensei'),
+			'boolean' => __('True/False', 'woothemes-sensei'),
+			'gap-fill' => __('Gap Fill', 'woothemes-sensei'),
+			'single-line' => __('Single Line', 'woothemes-sensei'),
+			'multi-line' => __('Multi Line', 'woothemes-sensei'),
+			'file-upload' => __('File Upload', 'woothemes-sensei'),
 		);
 
-		return apply_filters( 'sensei_question_types', $types );
+		return apply_filters('sensei_question_types', $types);
 	}
 
 	/**
@@ -57,12 +57,12 @@ 
 	 * @param  array $defaults
 	 * @return array $new_columns
 	 */
-	public function add_column_headings ( $defaults ) {
+	public function add_column_headings($defaults) {
 		$new_columns['cb'] = '<input type="checkbox" />';
-		$new_columns['title'] = _x( 'Question', 'column name', 'woothemes-sensei' );
-		$new_columns['question-type'] = _x( 'Type', 'column name', 'woothemes-sensei' );
-		$new_columns['question-category'] = _x( 'Categories', 'column name', 'woothemes-sensei' );
-		if ( isset( $defaults['date'] ) ) {
+		$new_columns['title'] = _x('Question', 'column name', 'woothemes-sensei');
+		$new_columns['question-type'] = _x('Type', 'column name', 'woothemes-sensei');
+		$new_columns['question-category'] = _x('Categories', 'column name', 'woothemes-sensei');
+		if (isset($defaults['date'])) {
 			$new_columns['date'] = $defaults['date'];
 		}
 
@@ -77,27 +77,27 @@ 
 	 * @param  int $id
 	 * @return void
 	 */
-	public function add_column_data ( $column_name, $id ) {
+	public function add_column_data($column_name, $id) {
 		global $wpdb, $post;
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'id':
 				echo $id;
 			break;
 
 			case 'question-type':
-				$question_type = strip_tags( get_the_term_list( $id, 'question-type', '', ', ', '' ) );
+				$question_type = strip_tags(get_the_term_list($id, 'question-type', '', ', ', ''));
 				$output = '&mdash;';
-				if( isset( $this->question_types[ $question_type ] ) ) {
-					$output = $this->question_types[ $question_type ];
+				if (isset($this->question_types[$question_type])) {
+					$output = $this->question_types[$question_type];
 				}
 				echo $output;
 			break;
 
 			case 'question-category':
-				$output = strip_tags( get_the_term_list( $id, 'question-category', '', ', ', '' ) );
-				if( ! $output ) {
+				$output = strip_tags(get_the_term_list($id, 'question-category', '', ', ', ''));
+				if ( ! $output) {
 					$output = '&mdash;';
 				}
 				echo $output;
@@ -110,49 +110,49 @@ 
 
 	} // End add_column_data()
 
-	public function question_edit_panel_metabox( $post_type, $post ) {
-		if( in_array( $post_type, array( 'question', 'multiple_question' ) ) ) {
+	public function question_edit_panel_metabox($post_type, $post) {
+		if (in_array($post_type, array('question', 'multiple_question'))) {
 
-			$metabox_title = __( 'Question', 'woothemes-sensei' );
+			$metabox_title = __('Question', 'woothemes-sensei');
 
-			if( isset( $post->ID ) ) {
+			if (isset($post->ID)) {
 
-                $question_type = Sensei()->question->get_question_type( $post->ID );
+                $question_type = Sensei()->question->get_question_type($post->ID);
 
-				if( $question_type ) {
-					$type = $this->question_types[ $question_type ];
-					if( $type ) {
+				if ($question_type) {
+					$type = $this->question_types[$question_type];
+					if ($type) {
 						$metabox_title = $type;
 					}
 				}
 			}
-			add_meta_box( 'question-edit-panel', $metabox_title, array( $this, 'question_edit_panel' ), 'question', 'normal', 'high' );
-			add_meta_box( 'question-lessons-panel', __( 'Quizzes', 'woothemes-sensei' ), array( $this, 'question_lessons_panel' ), 'question', 'side', 'default' );
-			add_meta_box( 'multiple-question-lessons-panel', __( 'Quizzes', 'woothemes-sensei' ), array( $this, 'question_lessons_panel' ), 'multiple_question', 'side', 'default' );
+			add_meta_box('question-edit-panel', $metabox_title, array($this, 'question_edit_panel'), 'question', 'normal', 'high');
+			add_meta_box('question-lessons-panel', __('Quizzes', 'woothemes-sensei'), array($this, 'question_lessons_panel'), 'question', 'side', 'default');
+			add_meta_box('multiple-question-lessons-panel', __('Quizzes', 'woothemes-sensei'), array($this, 'question_lessons_panel'), 'multiple_question', 'side', 'default');
 		}
 	}
 
 	public function question_edit_panel() {
 		global  $post, $pagenow;
 
-		add_action( 'admin_enqueue_scripts', array( Sensei()->lesson, 'enqueue_scripts' ) );
-		add_action( 'admin_enqueue_scripts', array( Sensei()->lesson, 'enqueue_styles' ) );
+		add_action('admin_enqueue_scripts', array(Sensei()->lesson, 'enqueue_scripts'));
+		add_action('admin_enqueue_scripts', array(Sensei()->lesson, 'enqueue_styles'));
 
 		$html = '<div id="lesson-quiz" class="single-question"><div id="add-question-main">';
 
-		if( 'post-new.php' == $pagenow ) {
+		if ('post-new.php' == $pagenow) {
 
 			$html .= '<div id="add-question-actions">';
-				$html .= Sensei()->lesson->quiz_panel_add( 'question' );
+				$html .= Sensei()->lesson->quiz_panel_add('question');
 			$html .= '</div>';
 
 		} else {
 			$question_id = $post->ID;
 
-			$question_type =  Sensei()->question->get_question_type( $post->ID );
+			$question_type = Sensei()->question->get_question_type($post->ID);
 
 			$html .= '<div id="add-question-metadata"><table class="widefat">';
-				$html .= Sensei()->lesson->quiz_panel_question( $question_type, 0, $question_id, 'question' );
+				$html .= Sensei()->lesson->quiz_panel_question($question_type, 0, $question_id, 'question');
 			$html .= '</table></div>';
 		}
 
@@ -164,20 +164,20 @@ 
 	public function question_lessons_panel() {
 		global $post;
 
-		$no_lessons = sprintf( __( '%1$sThis question does not appear in any quizzes yet.%2$s', 'woothemes-sensei' ), '<em>', '</em>' );
+		$no_lessons = sprintf(__('%1$sThis question does not appear in any quizzes yet.%2$s', 'woothemes-sensei'), '<em>', '</em>');
 
-		if( ! isset( $post->ID ) ) {
+		if ( ! isset($post->ID)) {
 			echo $no_lessons;
 			return;
 		}
 
 		// This retrieves those quizzes the question is directly connected to.
-		$quizzes = get_post_meta( $post->ID, '_quiz_id', false );
+		$quizzes = get_post_meta($post->ID, '_quiz_id', false);
 
 		// Collate all 'multiple_question' quizzes the question is part of.
-		$categories_of_question = wp_get_post_terms( $post->ID, 'question-category', array( 'fields' => 'ids' ) );
-		if ( ! empty( $categories_of_question ) ) {
-			foreach ( $categories_of_question as $term_id ) {
+		$categories_of_question = wp_get_post_terms($post->ID, 'question-category', array('fields' => 'ids'));
+		if ( ! empty($categories_of_question)) {
+			foreach ($categories_of_question as $term_id) {
 				$qargs = array(
 					'fields'           => 'ids',
 					'post_type'        => 'multiple_question',
@@ -191,41 +191,41 @@ 
 					'post_status'      => 'any',
 					'suppress_filters' => 0,
 				);
-				$cat_question_ids = get_posts( $qargs );
-				foreach( $cat_question_ids as $cat_question_id ) {
-					$cat_quizzes = get_post_meta( $cat_question_id, '_quiz_id', false );
-					$quizzes = array_merge( $quizzes, $cat_quizzes );
+				$cat_question_ids = get_posts($qargs);
+				foreach ($cat_question_ids as $cat_question_id) {
+					$cat_quizzes = get_post_meta($cat_question_id, '_quiz_id', false);
+					$quizzes = array_merge($quizzes, $cat_quizzes);
 				}
 			}
-			$quizzes = array_unique( array_filter( $quizzes ) );
+			$quizzes = array_unique(array_filter($quizzes));
 		}
 
-		if( 0 == count( $quizzes ) ) {
+		if (0 == count($quizzes)) {
 			echo $no_lessons;
 			return;
 		}
 
 		$lessons = false;
 
-		foreach( $quizzes as $quiz ) {
+		foreach ($quizzes as $quiz) {
 
-			$lesson_id = get_post_meta( $quiz, '_quiz_lesson', true );
+			$lesson_id = get_post_meta($quiz, '_quiz_lesson', true);
 
-			if( ! $lesson_id ) continue;
+			if ( ! $lesson_id) continue;
 
-			$lessons[ $lesson_id ]['title'] = get_the_title( $lesson_id );
-			$lessons[ $lesson_id ]['link'] = admin_url( 'post.php?post=' . $lesson_id . '&action=edit' );
+			$lessons[$lesson_id]['title'] = get_the_title($lesson_id);
+			$lessons[$lesson_id]['link'] = admin_url('post.php?post='.$lesson_id.'&action=edit');
 		}
 
-		if( ! $lessons ) {
+		if ( ! $lessons) {
 			echo $no_lessons;
 			return;
 		}
 
 		$html = '<ul>';
 
-		foreach( $lessons as $id => $lesson ) {
-			$html .= '<li><a href="' . esc_url( $lesson['link'] ) . '">' . esc_html( $lesson['title'] ) . '</a></li>';
+		foreach ($lessons as $id => $lesson) {
+			$html .= '<li><a href="'.esc_url($lesson['link']).'">'.esc_html($lesson['title']).'</a></li>';
 		}
 
 		$html .= '</ul>';
@@ -234,30 +234,30 @@ 
 
 	}
 
-	public function save_question( $post_id = 0 ) {
+	public function save_question($post_id = 0) {
 
-		if( ! isset( $_POST['post_type']
-            ) || 'question' != $_POST['post_type'] ) {
+		if ( ! isset($_POST['post_type']
+            ) || 'question' != $_POST['post_type']) {
             return;
         }
 
 
 
         //setup the data for saving
-		$data = $_POST ;
+		$data = $_POST;
         $data['quiz_id'] = 0;
 		$data['question_id'] = $post_id;
 
-		if ( ! wp_is_post_revision( $post_id ) ){
+		if ( ! wp_is_post_revision($post_id)) {
 
 			// Unhook function to prevent infinite loops
-			remove_action( 'save_post', array( $this, 'save_question' ) );
+			remove_action('save_post', array($this, 'save_question'));
 
 			// Update question data
-			$question_id = Sensei()->lesson->lesson_save_question( $data, 'question' );
+			$question_id = Sensei()->lesson->lesson_save_question($data, 'question');
 
 			// Re-hook same function
-			add_action( 'save_post', array( $this, 'save_question' ) );
+			add_action('save_post', array($this, 'save_question'));
 		}
 
 		return;
@@ -270,15 +270,15 @@ 
 	public function filter_options() {
 		global $typenow;
 
-		if( is_admin() && 'question' == $typenow ) {
+		if (is_admin() && 'question' == $typenow) {
 
 			$output = '';
 
 			// Question type
-			$selected = isset( $_GET['question_type'] ) ? $_GET['question_type'] : '';
-			$type_options = '<option value="">' . __( 'All types', 'woothemes-sensei' ) . '</option>';
-			foreach( $this->question_types as $label => $type ) {
-				$type_options .= '<option value="' . esc_attr( $label ) . '" ' . selected( $selected, $label, false ) . '>' . esc_html( $type ) . '</option>';
+			$selected = isset($_GET['question_type']) ? $_GET['question_type'] : '';
+			$type_options = '<option value="">'.__('All types', 'woothemes-sensei').'</option>';
+			foreach ($this->question_types as $label => $type) {
+				$type_options .= '<option value="'.esc_attr($label).'" '.selected($selected, $label, false).'>'.esc_html($type).'</option>';
 			}
 
 			$output .= '<select name="question_type" id="dropdown_question_type">';
@@ -286,12 +286,12 @@ 
 			$output .= '</select>';
 
 			// Question category
-			$cats = get_terms( 'question-category', array( 'hide_empty' => false ) );
-			if ( ! empty( $cats ) && ! is_wp_error( $cats ) ) {
-				$selected = isset( $_GET['question_cat'] ) ? $_GET['question_cat'] : '';
-				$cat_options = '<option value="">' . __( 'All categories', 'woothemes-sensei' ) . '</option>';
-				foreach( $cats as $cat ) {
-					$cat_options .= '<option value="' . esc_attr( $cat->slug ) . '" ' . selected( $selected, $cat->slug, false ) . '>' . esc_html( $cat->name ) . '</option>';
+			$cats = get_terms('question-category', array('hide_empty' => false));
+			if ( ! empty($cats) && ! is_wp_error($cats)) {
+				$selected = isset($_GET['question_cat']) ? $_GET['question_cat'] : '';
+				$cat_options = '<option value="">'.__('All categories', 'woothemes-sensei').'</option>';
+				foreach ($cats as $cat) {
+					$cat_options .= '<option value="'.esc_attr($cat->slug).'" '.selected($selected, $cat->slug, false).'>'.esc_html($cat->name).'</option>';
 				}
 
 				$output .= '<select name="question_cat" id="dropdown_question_cat">';
@@ -308,14 +308,14 @@ 
 	 * @param  array $request Current request
 	 * @return array          Modified request
 	 */
-	public function filter_actions( $request ) {
+	public function filter_actions($request) {
 		global $typenow;
 
-		if( is_admin() && 'question' == $typenow ) {
+		if (is_admin() && 'question' == $typenow) {
 
 			// Question type
-			$question_type = isset( $_GET['question_type'] ) ? $_GET['question_type'] : '';
-			if( $question_type ) {
+			$question_type = isset($_GET['question_type']) ? $_GET['question_type'] : '';
+			if ($question_type) {
 				$type_query = array(
 					'taxonomy' => 'question-type',
 					'terms' => $question_type,
@@ -325,8 +325,8 @@ 
 			}
 
 			// Question category
-			$question_cat = isset( $_GET['question_cat'] ) ? $_GET['question_cat'] : '';
-			if( $question_cat ) {
+			$question_cat = isset($_GET['question_cat']) ? $_GET['question_cat'] : '';
+			if ($question_cat) {
 				$cat_query = array(
 					'taxonomy' => 'question-category',
 					'terms' => $question_cat,
@@ -351,16 +351,16 @@ 
      *
      * @return string $question_type | bool
      */
-    public function get_question_type( $question_id ){
+    public function get_question_type($question_id) {
 
-        if( empty( $question_id ) || ! intval( $question_id ) > 0
-            || 'question' != get_post_type( $question_id )   ){
+        if (empty($question_id) || ! intval($question_id) > 0
+            || 'question' != get_post_type($question_id)) {
             return false;
         }
 
         $question_type = 'multiple-choice';
-        $question_types = wp_get_post_terms( $question_id, 'question-type' );
-        foreach( $question_types as $type ) {
+        $question_types = wp_get_post_terms($question_id, 'question-type');
+        foreach ($question_types as $type) {
             $question_type = $type->slug;
         }
 
@@ -377,21 +377,21 @@ 
 	 *
 	 * @return int $question_grade | bool
 	 */
-	public function get_question_grade( $question_id ) {
+	public function get_question_grade($question_id) {
 
-		if ( empty( $question_id ) || ! intval( $question_id ) > 0
-			|| 'question' != get_post_type( $question_id ) ) {
+		if (empty($question_id) || ! intval($question_id) > 0
+			|| 'question' != get_post_type($question_id)) {
 			return false;
 		}
 
-		$question_grade_raw = get_post_meta( $question_id, '_question_grade', true );
+		$question_grade_raw = get_post_meta($question_id, '_question_grade', true);
 		// If not set then default to 1...
-		if ( false === $question_grade_raw || $question_grade_raw == '' ) {
+		if (false === $question_grade_raw || $question_grade_raw == '') {
 			$question_grade = 1;
 		}
 		// ...but allow a grade of 0 for non-marked questions
 		else {
-			$question_grade = intval( $question_grade_raw );
+			$question_grade = intval($question_grade_raw);
 		}
 		return $question_grade;
 
@@ -404,9 +404,9 @@ 
      * @since 1.9.0
      * @param $question_type
      */
-    public static function load_question_template( $question_type ){
+    public static function load_question_template($question_type) {
 
-        Sensei_Templates::get_template  ( 'single-quiz/question_type-' . $question_type . '.php' );
+        Sensei_Templates::get_template('single-quiz/question_type-'.$question_type.'.php');
     }
 
     /**
@@ -417,9 +417,9 @@ 
      * @since 1.9.0
      * @param $question_id
      */
-    public static function the_question_title( $question_id ){
+    public static function the_question_title($question_id) {
 
-        echo self::get_the_question_title( $question_id );
+        echo self::get_the_question_title($question_id);
 
     }// end the_question_title
 
@@ -431,7 +431,7 @@ 
      * @param $question_id
      * @return string
      */
-    public static function get_the_question_title( $question_id ){
+    public static function get_the_question_title($question_id) {
 
         /**
          * Filter the sensei question title
@@ -439,17 +439,17 @@ 
          * @since 1.3.0
          * @param $question_title
          */
-        $title = apply_filters( 'sensei_question_title', get_the_title( $question_id ) );
+        $title = apply_filters('sensei_question_title', get_the_title($question_id));
 
         /**
          * hook document in class-woothemes-sensei-message.php the_title()
          */
-        $title = apply_filters( 'sensei_single_title', $title, 'question');
+        $title = apply_filters('sensei_single_title', $title, 'question');
 
         $title_html  = '<span class="question question-title">';
         $title_html .= $title;
         $title_html .= '<span class="grade"><?php sensi_the_question_grade()?></span>';
-        $title_html .='</span>';
+        $title_html .= '</span>';
 
         return $title_html;
     }
@@ -460,14 +460,14 @@ 
      * @param $question_id
      * @return string
      */
-    public static function get_the_question_description( $question_id ){
+    public static function get_the_question_description($question_id) {
 
-        $question = get_post( $question_id );
+        $question = get_post($question_id);
 
         /**
          * Already documented within WordPress Core
          */
-        return apply_filters( 'the_content', $question->post_content );
+        return apply_filters('the_content', $question->post_content);
 
     }
 
@@ -477,9 +477,9 @@ 
      * @since 1.9.0
      * @param $question_id
      */
-    public static function the_question_description( $question_id  ){
+    public static function the_question_description($question_id) {
 
-        echo self::get_the_question_description( $question_id );
+        echo self::get_the_question_description($question_id);
 
     }
 
@@ -490,38 +490,38 @@ 
      * @param $question_id
      * @return string
      */
-    public static function get_the_question_media( $question_id ){
+    public static function get_the_question_media($question_id) {
 
-        $question_media = get_post_meta( $question_id, '_question_media', true );
+        $question_media = get_post_meta($question_id, '_question_media', true);
         $question_media_link = '';
-        if( 0 < intval( $question_media ) ) {
-            $mimetype = get_post_mime_type( $question_media );
-            if( $mimetype ) {
-                $mimetype_array = explode( '/', $mimetype);
-                if( isset( $mimetype_array[0] ) && $mimetype_array[0] ) {
+        if (0 < intval($question_media)) {
+            $mimetype = get_post_mime_type($question_media);
+            if ($mimetype) {
+                $mimetype_array = explode('/', $mimetype);
+                if (isset($mimetype_array[0]) && $mimetype_array[0]) {
                     $question_media_type = $mimetype_array[0];
-                    $question_media_url = wp_get_attachment_url( $question_media );
-                    $attachment = get_post( $question_media );
+                    $question_media_url = wp_get_attachment_url($question_media);
+                    $attachment = get_post($question_media);
                     $question_media_title = $attachment->post_title;
                     $question_media_description = $attachment->post_content;
-                    switch( $question_media_type ) {
+                    switch ($question_media_type) {
                         case 'image':
-                            $image_size = apply_filters( 'sensei_question_image_size', 'medium', $question_id );
-                            $attachment_src = wp_get_attachment_image_src( $question_media, $image_size );
-                            $question_media_link = '<a class="' . esc_attr( $question_media_type ) . '" title="' . esc_attr( $question_media_title ) . '" href="' . esc_url( $question_media_url ) . '" target="_blank"><img src="' . $attachment_src[0] . '" width="' . $attachment_src[1] . '" height="' . $attachment_src[2] . '" /></a>';
+                            $image_size = apply_filters('sensei_question_image_size', 'medium', $question_id);
+                            $attachment_src = wp_get_attachment_image_src($question_media, $image_size);
+                            $question_media_link = '<a class="'.esc_attr($question_media_type).'" title="'.esc_attr($question_media_title).'" href="'.esc_url($question_media_url).'" target="_blank"><img src="'.$attachment_src[0].'" width="'.$attachment_src[1].'" height="'.$attachment_src[2].'" /></a>';
                             break;
 
                         case 'audio':
-                            $question_media_link = wp_audio_shortcode( array( 'src' => $question_media_url ) );
+                            $question_media_link = wp_audio_shortcode(array('src' => $question_media_url));
                             break;
 
                         case 'video':
-                            $question_media_link = wp_video_shortcode( array( 'src' => $question_media_url ) );
+                            $question_media_link = wp_video_shortcode(array('src' => $question_media_url));
                             break;
 
                         default:
-                            $question_media_filename = basename( $question_media_url );
-                            $question_media_link = '<a class="' . esc_attr( $question_media_type ) . '" title="' . esc_attr( $question_media_title ) . '" href="' . esc_url( $question_media_url ) . '" target="_blank">' . $question_media_filename . '</a>';
+                            $question_media_filename = basename($question_media_url);
+                            $question_media_link = '<a class="'.esc_attr($question_media_type).'" title="'.esc_attr($question_media_title).'" href="'.esc_url($question_media_url).'" target="_blank">'.$question_media_filename.'</a>';
                             break;
                     }
                 }
@@ -529,21 +529,21 @@ 
         }
 
         $output = '';
-        if( $question_media_link ) {
+        if ($question_media_link) {
 
                 $output .= '<div class="question_media_display">';
-                $output .=      $question_media_link;
+                $output .= $question_media_link;
                 $output .= '<dl>';
 
-                if( $question_media_title ) {
+                if ($question_media_title) {
 
-                   $output .= '<dt>'. $question_media_title. '</dt>';
+                   $output .= '<dt>'.$question_media_title.'</dt>';
 
                  }
 
-                if( $question_media_description ) {
+                if ($question_media_description) {
 
-                    $output .= '<dd>' . $question_media_description . '</dd>';
+                    $output .= '<dd>'.$question_media_description.'</dd>';
 
                 }
 
@@ -564,9 +564,9 @@ 
      * @since 1.9.0
      * @param string $question_id
      */
-    public static function the_question_media( $question_id ){
+    public static function the_question_media($question_id) {
 
-        echo self::get_the_question_media( $question_id );
+        echo self::get_the_question_media($question_id);
 
     }
 
@@ -577,11 +577,11 @@ 
      *
      * @param $question_id
      */
-    public static function the_question_hidden_fields( $question_id ){
+    public static function the_question_hidden_fields($question_id) {
         ?>
 
-            <input type="hidden" name="question_id_<?php $question_id;?>" value="<?php $question_id;?>" />
-            <input type="hidden" name="questions_asked[]" value="<?php esc_attr_e( $question_id ); ?>" />
+            <input type="hidden" name="question_id_<?php $question_id; ?>" value="<?php $question_id; ?>" />
+            <input type="hidden" name="questions_asked[]" value="<?php esc_attr_e($question_id); ?>" />
 
         <?php
     }
@@ -592,27 +592,27 @@ 
      * @since 1.9.0
      * @param $question_id
      */
-    public static function answer_feedback_notes( $question_id ){
+    public static function answer_feedback_notes($question_id) {
 
         //IDS
         $quiz_id = get_the_ID();
-        $lesson_id = Sensei()->quiz->get_lesson_id( $quiz_id );
+        $lesson_id = Sensei()->quiz->get_lesson_id($quiz_id);
 
         // Data to check before showing feedback
-        $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, get_current_user_id() );
-        $user_quiz_grade = Sensei_Quiz::get_user_quiz_grade( $lesson_id, get_current_user_id() );
-        $not_empty_user_quiz_grade = !empty( $user_quiz_grade );
-        $reset_quiz_allowed = Sensei_Quiz::is_reset_allowed( $lesson_id );
-        $lesson_completed = Sensei_Utils::user_completed_lesson( $lesson_id );
-        $quiz_grade_type = get_post_meta( $quiz_id , '_quiz_grade_type', true );
+        $user_lesson_status = Sensei_Utils::user_lesson_status($lesson_id, get_current_user_id());
+        $user_quiz_grade = Sensei_Quiz::get_user_quiz_grade($lesson_id, get_current_user_id());
+        $not_empty_user_quiz_grade = ! empty($user_quiz_grade);
+        $reset_quiz_allowed = Sensei_Quiz::is_reset_allowed($lesson_id);
+        $lesson_completed = Sensei_Utils::user_completed_lesson($lesson_id);
+        $quiz_grade_type = get_post_meta($quiz_id, '_quiz_grade_type', true);
 
-        if( ( $lesson_completed  && $not_empty_user_quiz_grade  )
-            ||  ( $lesson_completed && ! $reset_quiz_allowed && 'auto' == $quiz_grade_type )
-            || ( 'auto' == $quiz_grade_type && ! $reset_quiz_allowed && $not_empty_user_quiz_grade ) ) {
+        if (($lesson_completed && $not_empty_user_quiz_grade)
+            ||  ($lesson_completed && ! $reset_quiz_allowed && 'auto' == $quiz_grade_type)
+            || ('auto' == $quiz_grade_type && ! $reset_quiz_allowed && $not_empty_user_quiz_grade)) {
 
-            $answer_notes = Sensei()->quiz->get_user_question_feedback( $lesson_id, $question_id, get_current_user_id() );
+            $answer_notes = Sensei()->quiz->get_user_question_feedback($lesson_id, $question_id, get_current_user_id());
 
-            if( $answer_notes ) { ?>
+            if ($answer_notes) { ?>
 
                 <div class="sensei-message info info-special answer-feedback">
 
@@ -626,7 +626,7 @@ 
                          * @param string $question_id
                          * @param string $lesson_id
                          */
-                        echo apply_filters( 'sensei_question_answer_notes', $answer_notes, $question_id, $lesson_id );
+                        echo apply_filters('sensei_question_answer_notes', $answer_notes, $question_id, $lesson_id);
 
                     ?>
 
@@ -645,72 +645,72 @@ 
      * @since 1.9.0
      * @param string $question_id
      */
-    public static function the_answer_result_indication( $question_id ){
+    public static function the_answer_result_indication($question_id) {
 
-        global $post,  $current_user, $sensei_question_loop;
+        global $post, $current_user, $sensei_question_loop;
 
         // Post Data
         $quiz_id = $sensei_question_loop['quiz_id'];
-        $lesson_id = Sensei()->quiz->get_lesson_id( $quiz_id );
+        $lesson_id = Sensei()->quiz->get_lesson_id($quiz_id);
         $question_item = $sensei_question_loop['current_question'];
 
         // Setup variable needed to determine if the message should show and what it should show
-        $user_quiz_grade = Sensei_Quiz::get_user_quiz_grade( $lesson_id, get_current_user_id() );
-        $lesson_complete = Sensei_Utils::user_completed_lesson( $lesson_id, get_current_user_id() );
-        $reset_quiz_allowed = Sensei_Quiz::is_reset_allowed( $lesson_id );
-        $quiz_grade_type = get_post_meta( $quiz_id, '_quiz_grade_type', true );
+        $user_quiz_grade = Sensei_Quiz::get_user_quiz_grade($lesson_id, get_current_user_id());
+        $lesson_complete = Sensei_Utils::user_completed_lesson($lesson_id, get_current_user_id());
+        $reset_quiz_allowed = Sensei_Quiz::is_reset_allowed($lesson_id);
+        $quiz_grade_type = get_post_meta($quiz_id, '_quiz_grade_type', true);
 
         // retrieve the question total grade
-        $question_grade = Sensei()->question->get_question_grade( $question_id );
+        $question_grade = Sensei()->question->get_question_grade($question_id);
 
         // retrieve grade the user achieved
-        $user_question_grade = Sensei()->quiz->get_user_question_grade( $lesson_id, $question_id, get_current_user_id() );
+        $user_question_grade = Sensei()->quiz->get_user_question_grade($lesson_id, $question_id, get_current_user_id());
 
         // Question ID
         $question_id = $question_item->ID;
 
         // conditions to check
-        $completed_with_valid_grade = $lesson_complete && $user_quiz_grade != '' ;
-        $completed_with_valid_grade_and_reset_not_allowed = $lesson_complete &&  $user_quiz_grade != '' && ! $reset_quiz_allowed ;
-        $grade_type_auto_a_valid_grade_and_reset_not_allowed =  'auto' == $quiz_grade_type && ! $reset_quiz_allowed && $user_quiz_grade != '' ;
+        $completed_with_valid_grade = $lesson_complete && $user_quiz_grade != '';
+        $completed_with_valid_grade_and_reset_not_allowed = $lesson_complete && $user_quiz_grade != '' && ! $reset_quiz_allowed;
+        $grade_type_auto_a_valid_grade_and_reset_not_allowed = 'auto' == $quiz_grade_type && ! $reset_quiz_allowed && $user_quiz_grade != '';
 
-        if (  $completed_with_valid_grade
+        if ($completed_with_valid_grade
             || $completed_with_valid_grade_and_reset_not_allowed
-            || $grade_type_auto_a_valid_grade_and_reset_not_allowed  ) {
+            || $grade_type_auto_a_valid_grade_and_reset_not_allowed) {
 
             $user_correct = false;
-            $answer_message = __( 'Incorrect', 'woothemes-sensei' );
+            $answer_message = __('Incorrect', 'woothemes-sensei');
             $answer_message_class = 'user_wrong';
             // For zero grade mark as 'correct' but add no classes
-            if ( 0 == $question_grade ) {
+            if (0 == $question_grade) {
 
                 $user_correct = true;
                 $answer_message = '';
                 $answer_message_class = '';
 
-            } else if( $user_question_grade > 0 ) {
+            } else if ($user_question_grade > 0) {
 
                 $user_correct = true;
-                $answer_message = sprintf( __( 'Grade: %d', 'woothemes-sensei' ), $user_question_grade );
+                $answer_message = sprintf(__('Grade: %d', 'woothemes-sensei'), $user_question_grade);
                 $answer_message_class = 'user_right';
 
             }
 
             // attach the correct answer if the question is auto gradable and user got it wrong
-            if( !$reset_quiz_allowed && !$user_correct ){
+            if ( ! $reset_quiz_allowed && ! $user_correct) {
 
-                $answer_message .=  ' - '. __('Right Answer:','woothemes-sensei') . ' ' . self::get_correct_answer( $question_item->ID );
+                $answer_message .= ' - '.__('Right Answer:', 'woothemes-sensei').' '.self::get_correct_answer($question_item->ID);
 
             }
 
             // answer feedback
-            $answer_notes = Sensei()->quiz->get_user_question_feedback( $lesson_id, $question_id, $current_user->ID );
-            if( $answer_notes ) {
+            $answer_notes = Sensei()->quiz->get_user_question_feedback($lesson_id, $question_id, $current_user->ID);
+            if ($answer_notes) {
                 $answer_message_class .= ' has_notes';
             }
             ?>
 
-            <div class="answer_message <?php esc_attr_e( $answer_message_class ); ?>">
+            <div class="answer_message <?php esc_attr_e($answer_message_class); ?>">
 
                 <span><?php echo $answer_message; ?></span>
 
@@ -731,39 +731,39 @@ 
      * @param $quiz_id
      * @return array $question_data
      */
-    public static function get_template_data( $question_id, $quiz_id ){
+    public static function get_template_data($question_id, $quiz_id) {
 
-        $lesson_id = Sensei()->quiz->get_lesson_id( $quiz_id  );
+        $lesson_id = Sensei()->quiz->get_lesson_id($quiz_id);
 
-        $reset_allowed = get_post_meta( $quiz_id, '_enable_quiz_reset', true );
+        $reset_allowed = get_post_meta($quiz_id, '_enable_quiz_reset', true);
         //backwards compatibility
-        if( 'on' == $reset_allowed ) {
+        if ('on' == $reset_allowed) {
             $reset_allowed = 1;
         }
 
         // Check again that the lesson is complete
-        $user_lesson_end = Sensei_Utils::user_completed_lesson( Sensei()->quiz->get_lesson_id( $quiz_id), get_current_user_id() );
+        $user_lesson_end = Sensei_Utils::user_completed_lesson(Sensei()->quiz->get_lesson_id($quiz_id), get_current_user_id());
         $user_lesson_complete = false;
-        if ( $user_lesson_end ) {
+        if ($user_lesson_end) {
             $user_lesson_complete = true;
         }
 
         //setup the question data
-        $data[ 'ID' ]                     = $question_id;
-        $data[ 'title' ]                  = get_the_title( $question_id );
-        $data[ 'content' ]                = get_post( $question_id )->post_content;
-        $data[ 'quiz_id' ]                = $quiz_id;
-        $data[ 'lesson_id' ]              = Sensei()->quiz->get_lesson_id( $quiz_id );
-        $data[ 'type' ]                   = Sensei()->question->get_question_type( $question_id );
-        $data[ 'question_grade' ]         = Sensei()->question->get_question_grade(  $question_id  );
-        $data[ 'user_question_grade' ]    = Sensei()->quiz->get_user_question_grade( $lesson_id,  $question_id , get_current_user_id());
-        $data[ 'question_right_answer' ]  = get_post_meta( $question_id , '_question_right_answer', true );
-        $data[ 'question_wrong_answers' ] = get_post_meta( $question_id , '_question_wrong_answers', true );
-        $data[ 'user_answer_entry' ]      = Sensei()->quiz->get_user_question_answer( $lesson_id,  $question_id , get_current_user_id() );
-        $data[ 'lesson_completed' ]       = Sensei_Utils::user_completed_course( $lesson_id, get_current_user_id( ) );
-        $data[ 'quiz_grade_type' ]        = get_post_meta( $quiz_id , '_quiz_grade_type', true );
-        $data[ 'reset_quiz_allowed' ]     = $reset_allowed;
-        $data[ 'lesson_complete' ]        = $user_lesson_complete;
+        $data['ID']                     = $question_id;
+        $data['title']                  = get_the_title($question_id);
+        $data['content']                = get_post($question_id)->post_content;
+        $data['quiz_id']                = $quiz_id;
+        $data['lesson_id']              = Sensei()->quiz->get_lesson_id($quiz_id);
+        $data['type']                   = Sensei()->question->get_question_type($question_id);
+        $data['question_grade']         = Sensei()->question->get_question_grade($question_id);
+        $data['user_question_grade']    = Sensei()->quiz->get_user_question_grade($lesson_id, $question_id, get_current_user_id());
+        $data['question_right_answer']  = get_post_meta($question_id, '_question_right_answer', true);
+        $data['question_wrong_answers'] = get_post_meta($question_id, '_question_wrong_answers', true);
+        $data['user_answer_entry']      = Sensei()->quiz->get_user_question_answer($lesson_id, $question_id, get_current_user_id());
+        $data['lesson_completed']       = Sensei_Utils::user_completed_course($lesson_id, get_current_user_id( ));
+        $data['quiz_grade_type']        = get_post_meta($quiz_id, '_quiz_grade_type', true);
+        $data['reset_quiz_allowed']     = $reset_allowed;
+        $data['lesson_complete']        = $user_lesson_complete;
 
         /**
          * Filter the question template data. This filter fires  in
@@ -777,7 +777,7 @@ 
          * @param string $question_id
          * @param string $quiz_id
          */
-        return apply_filters( 'sensei_get_question_template_data', $data, $question_id, $quiz_id );
+        return apply_filters('sensei_get_question_template_data', $data, $question_id, $quiz_id);
 
     }
 
@@ -793,28 +793,28 @@ 
      *
      * @return array()
      */
-    public static function file_upload_load_question_data ( $question_data, $question_id, $quiz_id ){
+    public static function file_upload_load_question_data($question_data, $question_id, $quiz_id) {
 
 
-        if( 'file-upload' == Sensei()->question->get_question_type( $question_id ) ) {
+        if ('file-upload' == Sensei()->question->get_question_type($question_id)) {
 
             // Get uploaded file
-            $attachment_id = $question_data[ 'user_answer_entry' ];
+            $attachment_id = $question_data['user_answer_entry'];
             $answer_media_url = $answer_media_filename = '';
 
 
             $question_helptext = '';
-            if( isset( $question_data['question_wrong_answers'][0] ) ) {
+            if (isset($question_data['question_wrong_answers'][0])) {
 
-                $question_helptext =  $question_data['question_wrong_answers'][0];
+                $question_helptext = $question_data['question_wrong_answers'][0];
 
             }
 
 
-            if( 0 < intval( $attachment_id ) ) {
+            if (0 < intval($attachment_id)) {
 
-                $answer_media_url = wp_get_attachment_url( $attachment_id );
-                $answer_media_filename = basename( $answer_media_url );
+                $answer_media_url = wp_get_attachment_url($attachment_id);
+                $answer_media_filename = basename($answer_media_url);
 
             }
 
@@ -822,11 +822,11 @@ 
             // Get max upload file size, formatted for display
             // Code copied from wp-admin/includes/media.php:1515
             $upload_size_unit = $max_upload_size = wp_max_upload_size();
-            $sizes = array( 'KB', 'MB', 'GB' );
-            for ( $u = -1; $upload_size_unit > 1024 && $u < count( $sizes ) - 1; $u++ ) {
+            $sizes = array('KB', 'MB', 'GB');
+            for ($u = -1; $upload_size_unit > 1024 && $u < count($sizes) - 1; $u++) {
                 $upload_size_unit /= 1024;
             }
-            if ( $u < 0 ) {
+            if ($u < 0) {
 
                 $upload_size_unit = 0;
                 $u = 0;
@@ -836,14 +836,14 @@ 
                 $upload_size_unit = (int) $upload_size_unit;
 
             }
-            $max_upload_size = sprintf( __( 'Maximum upload file size: %d%s' ), esc_html( $upload_size_unit ), esc_html( $sizes[ $u ] ) );
+            $max_upload_size = sprintf(__('Maximum upload file size: %d%s'), esc_html($upload_size_unit), esc_html($sizes[$u]));
 
             // Assemble all the data needed by the file upload template
-            $question_data[ 'answer_media_url' ]      = $answer_media_url;
-            $question_data[ 'answer_media_filename' ] = $answer_media_filename;
-            $question_data[ 'max_upload_size' ]       = $max_upload_size;
+            $question_data['answer_media_url']      = $answer_media_url;
+            $question_data['answer_media_filename'] = $answer_media_filename;
+            $question_data['max_upload_size']       = $max_upload_size;
 
-            $question_data[ 'question_helptext' ]     = $question_helptext;
+            $question_data['question_helptext']     = $question_helptext;
 
         }// end if is file upload type
 
@@ -863,27 +863,27 @@ 
      *
      * @return array()
      */
-    public static function multiple_choice_load_question_data( $question_data, $question_id, $quiz_id ){
+    public static function multiple_choice_load_question_data($question_data, $question_id, $quiz_id) {
 
-        if( 'multiple-choice' == Sensei()->question->get_question_type( $question_id ) ) {
+        if ('multiple-choice' == Sensei()->question->get_question_type($question_id)) {
 
 
             $answer_type = 'radio';
-            if ( is_array( $question_data[ 'question_right_answer' ] ) && ( 1 < count( $question_data[ 'question_right_answer' ] ) ) ) {
+            if (is_array($question_data['question_right_answer']) && (1 < count($question_data['question_right_answer']))) {
 
                 $answer_type = 'checkbox';
 
             }
 
             // Merge right and wrong answers
-            if ( is_array( $question_data[ 'question_right_answer' ] ) ) {
+            if (is_array($question_data['question_right_answer'])) {
 
-                $merged_options = array_merge( $question_data[ 'question_wrong_answers' ], $question_data[ 'question_right_answer' ] );
+                $merged_options = array_merge($question_data['question_wrong_answers'], $question_data['question_right_answer']);
 
-            }  else {
+            } else {
 
-                array_push( $question_data[ 'question_wrong_answers' ], $question_data[ 'question_right_answer' ] );
-                $merged_options = $question_data[ 'question_wrong_answers' ];
+                array_push($question_data['question_wrong_answers'], $question_data['question_right_answer']);
+                $merged_options = $question_data['question_wrong_answers'];
 
             }
 
@@ -891,24 +891,24 @@ 
             $question_answers_options = array();
             $count = 0;
 
-            foreach( $merged_options as $answer ) {
+            foreach ($merged_options as $answer) {
 
                 $count++;
                 $question_option = array();
 
-                if( ( $question_data[ 'lesson_completed' ] && $question_data[ 'user_quiz_grade' ] != '' )
-                    || ( $question_data[ 'lesson_completed' ] && ! $question_data[ 'reset_quiz_allowed' ] && $question_data[ 'user_quiz_grade' ] != '' )
-                    || ( 'auto' == $question_data[ 'quiz_grade_type' ] && ! $question_data[ 'reset_quiz_allowed' ]  && ! empty( $question_data[ 'user_quiz_grade' ] ) ) ) {
+                if (($question_data['lesson_completed'] && $question_data['user_quiz_grade'] != '')
+                    || ($question_data['lesson_completed'] && ! $question_data['reset_quiz_allowed'] && $question_data['user_quiz_grade'] != '')
+                    || ('auto' == $question_data['quiz_grade_type'] && ! $question_data['reset_quiz_allowed'] && ! empty($question_data['user_quiz_grade']))) {
 
                     $user_correct = false;
 
 
                     // For zero grade mark as 'correct' but add no classes
-                    if ( 0 == $question_data[ 'question_grade' ] ) {
+                    if (0 == $question_data['question_grade']) {
 
                         $user_correct = true;
 
-                    }  else if( $question_data[ 'user_question_grade' ] > 0 ) {
+                    } else if ($question_data['user_question_grade'] > 0) {
 
                         $user_correct = true;
 
@@ -918,20 +918,20 @@ 
 
                 // setup the option specific classes
                 $answer_class = '';
-                if( isset( $user_correct ) && 0 < $question_data[ 'question_grade' ] ) {
-                    if ( is_array( $question_data['question_right_answer'] ) && in_array($answer, $question_data['question_right_answer']) ) {
+                if (isset($user_correct) && 0 < $question_data['question_grade']) {
+                    if (is_array($question_data['question_right_answer']) && in_array($answer, $question_data['question_right_answer'])) {
 
                         $answer_class .= ' right_answer';
 
-                    }  elseif( !is_array($question_data['question_right_answer']) && $question_data['question_right_answer'] == $answer ) {
+                    }  elseif ( ! is_array($question_data['question_right_answer']) && $question_data['question_right_answer'] == $answer) {
 
                         $answer_class .= ' right_answer';
 
-                    } elseif( ( is_array( $question_data['user_answer_entry']  ) && in_array($answer, $question_data['user_answer_entry'] ) )
-                        ||  ( !  $question_data['user_answer_entry'] &&  $question_data['user_answer_entry'] == $answer ) ) {
+                    } elseif ((is_array($question_data['user_answer_entry']) && in_array($answer, $question_data['user_answer_entry']))
+                        ||  ( ! $question_data['user_answer_entry'] && $question_data['user_answer_entry'] == $answer)) {
 
                         $answer_class = 'user_wrong';
-                        if( $user_correct ) {
+                        if ($user_correct) {
 
                             $answer_class = 'user_right';
 
@@ -943,76 +943,76 @@ 
 
                 // determine if the current option must be checked
                 $checked = '';
-                if ( isset( $question_data['user_answer_entry'] ) && 0 < count( $question_data['user_answer_entry'] ) ) {
-                    if ( is_array( $question_data['user_answer_entry'] ) && in_array( $answer, $question_data['user_answer_entry'] ) ) {
+                if (isset($question_data['user_answer_entry']) && 0 < count($question_data['user_answer_entry'])) {
+                    if (is_array($question_data['user_answer_entry']) && in_array($answer, $question_data['user_answer_entry'])) {
 
                         $checked = 'checked="checked"';
 
-                    } elseif ( !is_array( $question_data['user_answer_entry'] ) ) {
+                    } elseif ( ! is_array($question_data['user_answer_entry'])) {
 
-                        $checked = checked( $answer, $question_data['user_answer_entry'] , false );
+                        $checked = checked($answer, $question_data['user_answer_entry'], false);
 
                     }
 
                 } // End If Statement
 
                 //Load the answer option data
-                $question_option[ 'ID' ]          = Sensei()->lesson->get_answer_id( $answer );
-                $question_option[ 'answer' ]      = $answer;
-                $question_option[ 'option_class'] = $answer_class;
-                $question_option[ 'checked']      = $checked;
-                $question_option[ 'count' ]       = $count;
-                $question_option[ 'type' ] = $answer_type;
+                $question_option['ID']          = Sensei()->lesson->get_answer_id($answer);
+                $question_option['answer']      = $answer;
+                $question_option['option_class'] = $answer_class;
+                $question_option['checked']      = $checked;
+                $question_option['count']       = $count;
+                $question_option['type'] = $answer_type;
 
                 // add the speci  fic option to the list of options for this question
-                $question_answers_options[$question_option[ 'ID' ]] = $question_option;
+                $question_answers_options[$question_option['ID']] = $question_option;
 
             } // end for each option
 
 
             // Shuffle the array depending on the settings
             $answer_options_sorted = array();
-            $random_order = get_post_meta( $question_data['ID'], '_random_order', true );
-            if(  $random_order && $random_order == 'yes' ) {
+            $random_order = get_post_meta($question_data['ID'], '_random_order', true);
+            if ($random_order && $random_order == 'yes') {
 
                 $answer_options_sorted = $question_answers_options;
-                shuffle( $answer_options_sorted );
+                shuffle($answer_options_sorted);
 
             } else {
 
                 $answer_order = array();
-                $answer_order_string = get_post_meta( $question_data['ID'], '_answer_order', true );
-                if( $answer_order_string ) {
+                $answer_order_string = get_post_meta($question_data['ID'], '_answer_order', true);
+                if ($answer_order_string) {
 
-                    $answer_order = array_filter( explode( ',', $answer_order_string ) );
-                    if( count( $answer_order ) > 0 ) {
+                    $answer_order = array_filter(explode(',', $answer_order_string));
+                    if (count($answer_order) > 0) {
 
-                        foreach( $answer_order as $answer_id ) {
+                        foreach ($answer_order as $answer_id) {
 
-                            if( isset( $question_answers_options[ $answer_id ] ) ) {
+                            if (isset($question_answers_options[$answer_id])) {
 
-                                $answer_options_sorted[ $answer_id ] = $question_answers_options[ $answer_id ];
-                                unset( $question_answers_options[ $answer_id ] );
+                                $answer_options_sorted[$answer_id] = $question_answers_options[$answer_id];
+                                unset($question_answers_options[$answer_id]);
 
                             }
 
                         }
 
-                        if( count( $question_answers_options ) > 0 ) {
-                            foreach( $question_answers_options as $id => $answer ) {
+                        if (count($question_answers_options) > 0) {
+                            foreach ($question_answers_options as $id => $answer) {
 
-                                $answer_options_sorted[ $id ] = $answer;
+                                $answer_options_sorted[$id] = $answer;
 
                             }
                         }
 
-                    }else{
+                    } else {
 
                         $answer_options_sorted = $question_answers_options;
 
                     }
 
-                }else{
+                } else {
 
                     $answer_options_sorted = $question_answers_options;
 
@@ -1022,7 +1022,7 @@ 
 
 
             // assemble and setup the data for the templates data array
-            $question_data[ 'answer_options' ]    =  $answer_options_sorted;
+            $question_data['answer_options'] = $answer_options_sorted;
 
         }
 
@@ -1042,14 +1042,14 @@ 
      *
      * @return array()
      */
-    public static function gap_fill_load_question_data( $question_data, $question_id, $quiz_id ){
+    public static function gap_fill_load_question_data($question_data, $question_id, $quiz_id) {
 
-        if( 'gap-fill' == Sensei()->question->get_question_type( $question_id ) ) {
+        if ('gap-fill' == Sensei()->question->get_question_type($question_id)) {
 
-            $gapfill_array = explode( '||', $question_data[ 'question_right_answer' ] );
-            $question_data[ 'gapfill_pre' ]  = isset( $gapfill_array[0] ) ? $gapfill_array[0] : '';
-            $question_data[ 'gapfill_gap' ]  = isset( $gapfill_array[1] ) ? $gapfill_array[1] : '';
-            $question_data[ 'gapfill_post' ] = isset( $gapfill_array[2] ) ? $gapfill_array[2] : '';
+            $gapfill_array = explode('||', $question_data['question_right_answer']);
+            $question_data['gapfill_pre']  = isset($gapfill_array[0]) ? $gapfill_array[0] : '';
+            $question_data['gapfill_gap']  = isset($gapfill_array[1]) ? $gapfill_array[1] : '';
+            $question_data['gapfill_post'] = isset($gapfill_array[2]) ? $gapfill_array[2] : '';
 
         }
 
@@ -1064,32 +1064,32 @@ 
      * @param $question_id
      * @return string $correct_answer or empty
      */
-    public static function get_correct_answer( $question_id ){
+    public static function get_correct_answer($question_id) {
 
-        $right_answer = get_post_meta( $question_id, '_question_right_answer', true );
-        $type = Sensei()->question->get_question_type( $question_id );
-        $type_name = __( 'Multiple Choice', 'woothemes-sensei' );
+        $right_answer = get_post_meta($question_id, '_question_right_answer', true);
+        $type = Sensei()->question->get_question_type($question_id);
+        $type_name = __('Multiple Choice', 'woothemes-sensei');
         $grade_type = 'manual-grade';
 
-        if ('boolean'== $type ) {
+        if ('boolean' == $type) {
 
             $right_answer = ucfirst($right_answer);
 
-        }elseif( 'multiple-choice' == $type ) {
+        }elseif ('multiple-choice' == $type) {
 
             $right_answer = (array) $right_answer;
-            $right_answer = implode( ', ', $right_answer );
+            $right_answer = implode(', ', $right_answer);
 
-        }elseif( 'gap-fill' == $type ) {
+        }elseif ('gap-fill' == $type) {
 
-            $right_answer_array = explode( '||', $right_answer );
-            if ( isset( $right_answer_array[0] ) ) { $gapfill_pre = $right_answer_array[0]; } else { $gapfill_pre = ''; }
-            if ( isset( $right_answer_array[1] ) ) { $gapfill_gap = $right_answer_array[1]; } else { $gapfill_gap = ''; }
-            if ( isset( $right_answer_array[2] ) ) { $gapfill_post = $right_answer_array[2]; } else { $gapfill_post = ''; }
+            $right_answer_array = explode('||', $right_answer);
+            if (isset($right_answer_array[0])) { $gapfill_pre = $right_answer_array[0]; } else { $gapfill_pre = ''; }
+            if (isset($right_answer_array[1])) { $gapfill_gap = $right_answer_array[1]; } else { $gapfill_gap = ''; }
+            if (isset($right_answer_array[2])) { $gapfill_post = $right_answer_array[2]; } else { $gapfill_post = ''; }
 
-            $right_answer = $gapfill_pre . ' <span class="highlight">' . $gapfill_gap . '</span> ' . $gapfill_post;
+            $right_answer = $gapfill_pre.' <span class="highlight">'.$gapfill_gap.'</span> '.$gapfill_post;
 
-        }else{
+        } else {
 
             // for non auto gradable question types no answer should be returned.
             $right_answer = '';
@@ -1107,4 +1107,4 @@ 
  * @ignore only for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Question extends Sensei_Question{}
+class WooThemes_Sensei_Question extends Sensei_Question {}

includes/class-sensei-analysis.php

Spacing +182 added lines, -182 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * All functionality pertaining to the Admin Analysis in Sensei.
@@ -19,24 +19,24 @@ 
 	 * @since  1.0.0
 	 * @param string $file
 	 */
-	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 ) ) {
+		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_styles', array( $this, 'enqueue_styles' ) );
+				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()
 
@@ -50,9 +50,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'));
 
 		}
 
@@ -66,11 +66,11 @@ 
 	 * @since 1.0.0
 	 * @return void
 	 */
-	public function enqueue_styles () {
+	public function enqueue_styles() {
 
-		wp_enqueue_style( 'woothemes-sensei-admin' );
+		wp_enqueue_style('woothemes-sensei-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()
 
@@ -82,14 +82,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()
 
@@ -100,13 +100,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 = 'Sensei_Analysis_' . $name . '_List_Table';
-		if ( is_null($optional_data) ) {
-			$sensei_analysis_object = new $object_name( $data );
+		$object_name = '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;
@@ -123,40 +123,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()
 
@@ -165,35 +165,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()
 
 	/**
@@ -202,26 +202,26 @@ 
 	 * @param integer $user_id
 	 * @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()
 
 	/**
@@ -230,26 +230,26 @@ 
 	 * @param integer $course_id
 	 * @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()
 
 	/**
@@ -259,26 +259,26 @@ 
 	 * @param integer $user_id
 	 * @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()
 
 	/**
@@ -287,26 +287,26 @@ 
 	 * @param integer $course_id
 	 * @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()
 
 	/**
@@ -315,26 +315,26 @@ 
 	 * @param integer $lesson_id
 	 * @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()
 
 	/**
@@ -344,7 +344,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">
 			<h2><span><?php echo $title; ?></span></h2>
 			<div class="inside">
@@ -358,14 +358,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">
 
@@ -376,7 +376,7 @@ 
             </p>
 
 		<?php
-		do_action( 'sensei_analysis_after_headers' );
+		do_action('sensei_analysis_after_headers');
 	} // End analysis_headers()
 
 	/**
@@ -385,10 +385,10 @@ 
 	 * @param $which string
 	 * @return void
 	 */
-	public function wrapper_container( $which ) {
-		if ( 'top' == $which ) {
+	public function wrapper_container($which) {
+		if ('top' == $which) {
 			?><div id="woothemes-sensei" class="wrap woothemes-sensei"><?php
-		} elseif ( 'bottom' == $which ) {
+		} elseif ('bottom' == $which) {
 			?></div><!--/#woothemes-sensei--><?php
 		} // End If Statement
 	} // End wrapper_container()
@@ -400,24 +400,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;
 		}
 		?>
-			<h1><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h1>
+			<h1><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h1>
 		<?php
 	} // End analysis_default_nav()
 
@@ -428,17 +428,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_Learner::get_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_Learner::get_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
 		?>
-			<h1><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h1>
+			<h1><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h1>
 		<?php
 	} // End analysis_user_profile_nav()
 
@@ -449,22 +449,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_Learner::get_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_Learner::get_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));
 		}
 		?>
-			<h1><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h1>
+			<h1><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h1>
 		<?php
 	} // End analysis_user_course_nav()
 
@@ -475,14 +475,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));
 		}
 		?>
-			<h1><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h1>
+			<h1><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h1>
 		<?php
 	} // End analysis_course_nav()
 
@@ -493,14 +493,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));
 		}
 		?>
-			<h1><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h1>
+			<h1><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h1>
 		<?php
 	} // End analysis_course_users_nav()
 
@@ -511,17 +511,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));
 		}
 		?>
-			<h1><?php echo apply_filters( 'sensei_analysis_nav_title', $title ); ?></h1>
+			<h1><?php echo apply_filters('sensei_analysis_nav_title', $title); ?></h1>
 		<?php
 	} // End analysis_lesson_users_nav()
 
@@ -532,59 +532,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;
@@ -597,9 +597,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()
 
 	/**
@@ -610,12 +610,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()
@@ -626,7 +626,7 @@ 
 	 * @param  array  $report_data 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);
@@ -642,9 +642,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-list-table.php

Spacing +39 added lines, -39 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Generic Data Table parent Class in Sensei.
@@ -56,18 +56,18 @@ 
 	 * @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()
 
@@ -76,7 +76,7 @@ 
 	 * @param  array $columns Existing columns
 	 * @return array          Modified columns
 	 */
-	public function remove_sortable_columns( $columns ) {
+	public function remove_sortable_columns($columns) {
 		return array();
 	}
 
@@ -85,14 +85,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()
 
@@ -102,22 +102,22 @@ 
 	 * @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 esc_attr( $k ); ?>" value="<?php echo esc_attr( $v ); ?>" />
+                        <input type="hidden" name="<?php echo esc_attr($k); ?>" value="<?php echo esc_attr($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()
 
@@ -145,11 +145,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);
 
 		/**
 		 * Filter the list table sortable columns for a specific screen.
@@ -161,22 +161,22 @@ 
 		 *
 		 * @param array $sortable_columns An array of sortable columns.
 		 */
-		$_sortable = apply_filters( "manage_{$this->screen->id}_sortable_columns", $this->get_sortable_columns() );
+		$_sortable = apply_filters("manage_{$this->screen->id}_sortable_columns", $this->get_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;
 	}
@@ -187,27 +187,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>";
@@ -221,8 +221,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.');
 	}
 
 	/**
@@ -233,7 +233,7 @@ 
 	 */
 	function no_items() {
 
-		 _e( 'No items found.', 'woothemes-sensei' );
+		 _e('No items found.', 'woothemes-sensei');
 
 	} // End no_items()
 
@@ -251,9 +251,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-updates.php

Spacing +436 added lines, -436 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Updates Class
@@ -31,7 +31,7 @@ 
 
 		// Setup object data
 		$this->parent = $parent;
-		$this->updates_run = get_option( 'woothemes-sensei-upgrades', array() );
+		$this->updates_run = get_option('woothemes-sensei-upgrades', array());
 
         // The list of upgrades to run
         $this->updates = array('1.1.0' => array('auto' => array('assign_role_caps' => array('title' => __('Assign role capabilities', 'woothemes-sensei'), 'desc' => __('Assigns Sensei capabilites to the relevant user roles.', 'woothemes-sensei'), 'product' => 'Sensei')),
@@ -75,8 +75,8 @@ 
             ),
         );
 
-		$this->updates = apply_filters( 'sensei_upgrade_functions', $this->updates, $this->updates );
-		$this->version = get_option( 'woothemes-sensei-version' );
+		$this->updates = apply_filters('sensei_upgrade_functions', $this->updates, $this->updates);
+		$this->version = get_option('woothemes-sensei-version');
 
         // Manual Update Screen
         add_action('admin_menu', array($this, 'add_update_admin_screen'), 50);
@@ -120,10 +120,10 @@ 
         <h2><?php _e('Sensei Updates', 'woothemes-sensei'); ?></h2>
 
         <?php
-        $function_name= '';
-        if ( isset($_GET['action']) && $_GET['action'] == 'update'
+        $function_name = '';
+        if (isset($_GET['action']) && $_GET['action'] == 'update'
             && isset($_GET['n']) && intval($_GET['n']) >= 0
-            && ( (isset($_POST['checked'][0]) && '' != $_POST['checked'][0]) || (isset($_GET['functions']) && '' != $_GET['functions']))) {
+            && ((isset($_POST['checked'][0]) && '' != $_POST['checked'][0]) || (isset($_GET['functions']) && '' != $_GET['functions']))) {
 
             // Setup the data variables
             $n = intval($_GET['n']);
@@ -135,12 +135,12 @@ 
 
                 foreach ($_POST['checked'] as $key => $function_name) {
 
-                    if( ! isset(  $_POST[ $function_name.'_nonce_field' ] ) 
-                        || ! wp_verify_nonce( $_POST[ $function_name.'_nonce_field' ] , 'run_'.$function_name ) ){
+                    if ( ! isset($_POST[$function_name.'_nonce_field']) 
+                        || ! wp_verify_nonce($_POST[$function_name.'_nonce_field'], 'run_'.$function_name)) {
 
                         wp_die(
-                            '<h1>' . __( 'Cheatin&#8217; uh?' ) . '</h1>' .
-                            '<p>' . __( 'The nonce supplied in order to run this update function is invalid','woothemes-sensei') . '</p>',
+                            '<h1>'.__('Cheatin&#8217; uh?').'</h1>'.
+                            '<p>'.__('The nonce supplied in order to run this update function is invalid', 'woothemes-sensei').'</p>',
                             403
                         );
 
@@ -157,7 +157,7 @@ 
 
                     } else {
 
-                        _doing_it_wrong( esc_html( $function_name) , 'Is not a valid Sensei updater function', 'Sensei 1.9.0');
+                        _doing_it_wrong(esc_html($function_name), 'Is not a valid Sensei updater function', 'Sensei 1.9.0');
                         return;
 
                     }// End If Statement
@@ -166,7 +166,7 @@ 
                 if ('' == $functions_list) {
                     $functions_list .= $function_name;
                 } else {
-                    $functions_list .= '+' . $function_name;
+                    $functions_list .= '+'.$function_name;
                 } // End If Statement
 
                 // Mark update has having been run
@@ -184,12 +184,12 @@ 
 
             foreach ($functions_array as $key => $function_name) {
 
-                if( ! isset( $_GET[ $function_name.'_nonce' ] )
-                    || ! wp_verify_nonce( $_GET[ $function_name.'_nonce' ] , 'run_'.$function_name ) ){
+                if ( ! isset($_GET[$function_name.'_nonce'])
+                    || ! wp_verify_nonce($_GET[$function_name.'_nonce'], 'run_'.$function_name)) {
 
                     wp_die(
-                        '<h1>' . __( 'Cheatin&#8217; uh?' ) . '</h1>' .
-                        '<p>' . __( 'The nonce supplied in order to run this update function is invalid','woothemes-sensei') . '</p>',
+                        '<h1>'.__('Cheatin&#8217; uh?').'</h1>'.
+                        '<p>'.__('The nonce supplied in order to run this update function is invalid', 'woothemes-sensei').'</p>',
                         403
                     );
 
@@ -206,7 +206,7 @@ 
 
                 } else {
 
-                    _doing_it_wrong( esc_html( $function_name) , 'Is not a valid Sensei updater function', 'Sensei 1.9.0');
+                    _doing_it_wrong(esc_html($function_name), 'Is not a valid Sensei updater function', 'Sensei 1.9.0');
                     return;
 
                 } // End If Statement
@@ -215,7 +215,7 @@ 
                 if ('' == $functions_list) {
                     $functions_list .= $function_name;
                 } else {
-                    $functions_list .= '+' . $function_name;
+                    $functions_list .= '+'.$function_name;
                 } // End If Statement
 
                 $this->set_update_run($function_name);
@@ -224,27 +224,27 @@ 
 
         } // End If Statement
 
-        if (!$done_processing) { ?>
+        if ( ! $done_processing) { ?>
 
             <h3><?php _e('Processing Updates...', 'woothemes-sensei'); ?></h3>
 
             <p>
 
-                <?php _e( "If your browser doesn't start loading the next page automatically, click this button:", 'woothemes-sensei' ); ?>
+                <?php _e("If your browser doesn't start loading the next page automatically, click this button:", 'woothemes-sensei'); ?>
 
                 <?php
-                $next_action_url = add_query_arg( array(
+                $next_action_url = add_query_arg(array(
                     'page' => 'sensei_updates',
                     'action' => 'update',
                     'n' => $n + 50,
-                    'functions' => array( $functions_list ),
-                    $function_name.'_nonce' => wp_create_nonce( 'run_'. $function_name ),
-                ), admin_url( 'admin.php' ) );
+                    'functions' => array($functions_list),
+                    $function_name.'_nonce' => wp_create_nonce('run_'.$function_name),
+                ), admin_url('admin.php'));
                 ?>
 
-                <a class="button"  href="<?php echo esc_url( $next_action_url ); ?>">
+                <a class="button"  href="<?php echo esc_url($next_action_url); ?>">
 
-                    <?php _e( 'Next', 'woothemes-sensei' ); ?>
+                    <?php _e('Next', 'woothemes-sensei'); ?>
 
                 </a>
 
@@ -252,7 +252,7 @@ 
             <script type='text/javascript'>
                 <!--
                 function js_sensei_nextpage() {
-                    location.href = "<?php echo esc_url_raw(  $next_action_url );?>";
+                    location.href = "<?php echo esc_url_raw($next_action_url); ?>";
                 }
                 setTimeout( "js_sensei_nextpage()", 250 );
                 //-->
@@ -331,17 +331,17 @@ 
                                     <td><p><?php echo $type_label; ?></p></td>
                                     <td>
                                         <p>
-                                            <input onclick="javascript:return confirm('<?php echo addslashes( sprintf( __( 'Are you sure you want to run the \'%s\' update?', 'woothemes-sensei' ), $data['title'] ) ); ?>');"
+                                            <input onclick="javascript:return confirm('<?php echo addslashes(sprintf(__('Are you sure you want to run the \'%s\' update?', 'woothemes-sensei'), $data['title'])); ?>');"
                                                    id="update-sensei"
-                                                   class="button<?php if( ! $update_run ) { echo ' button-primary'; } ?>"
+                                                   class="button<?php if ( ! $update_run) { echo ' button-primary'; } ?>"
                                                    type="submit"
-                                                   value="<?php if( $update_run ) { _e( 'Re-run Update', 'woothemes-sensei' ); } else { _e( 'Run Update', 'woothemes-sensei' ); } ?>"
+                                                   value="<?php if ($update_run) { _e('Re-run Update', 'woothemes-sensei'); } else { _e('Run Update', 'woothemes-sensei'); } ?>"
                                                    name="update">
 
                                             <?php
                                             $nonce_action = 'run_'.$update;
                                             $nonce_field_name = $update.'_nonce_field';
-                                            wp_nonce_field( $nonce_action, $nonce_field_name, false, true );
+                                            wp_nonce_field($nonce_action, $nonce_field_name, false, true);
                                             ?>
                                         </p>
                                     </td>
@@ -375,7 +375,7 @@ 
      *
      * @param string $function_name
      */
-    public function function_in_whitelist( $function_name ){
+    public function function_in_whitelist($function_name) {
 
         $function_whitelist = array(
             'status_changes_convert_questions',
@@ -386,7 +386,7 @@ 
 
         );
 
-        return in_array($function_name, $function_whitelist );
+        return in_array($function_name, $function_whitelist);
 
     }// end function_in_whitelist
 
@@ -397,8 +397,8 @@ 
 	 * @param  string $b Second key
 	 * @return integer
 	 */
-	private function sort_updates( $a, $b ) {
-		return strcmp( $a, $b );
+	private function sort_updates($a, $b) {
+		return strcmp($a, $b);
 	}
 
 	/**
@@ -409,39 +409,39 @@ 
 	 * @access public
 	 * @return boolean
 	 */
-	public function update ( $type = 'auto' ) {
+	public function update($type = 'auto') {
 
 		// Only allow admins to run update functions
-		if( ! current_user_can( 'manage_options' ) ) {
+		if ( ! current_user_can('manage_options')) {
             return false;
         }
 
         $this->force_updates();
 
         // Run through all functions
-        foreach ( $this->updates as $version => $value ) {
-            foreach ( $this->updates[$version] as $upgrade_type => $function_to_run ) {
-                if ( $upgrade_type == $type ) {
+        foreach ($this->updates as $version => $value) {
+            foreach ($this->updates[$version] as $upgrade_type => $function_to_run) {
+                if ($upgrade_type == $type) {
                     $updated = false;
                     // Run the update function
-                    foreach ( $function_to_run as $function_name => $update_data ) {
-                        if ( isset( $function_name ) && '' != $function_name ) {
-                            if ( ! in_array( $function_name, $this->updates_run ) ) {
+                    foreach ($function_to_run as $function_name => $update_data) {
+                        if (isset($function_name) && '' != $function_name) {
+                            if ( ! in_array($function_name, $this->updates_run)) {
                                 $updated = false;
-                                if ( method_exists( $this, $function_name ) ) {
+                                if (method_exists($this, $function_name)) {
 
-            $this->updates_run = array_unique( $this->updates_run ); // we only need one reference per update
-			update_option( Sensei()->token . '-upgrades', $this->updates_run );
+            $this->updates_run = array_unique($this->updates_run); // we only need one reference per update
+			update_option(Sensei()->token.'-upgrades', $this->updates_run);
 			return true;
 
-                                } elseif( $this->function_in_whitelist( $function_name ) ) {
+                                } elseif ($this->function_in_whitelist($function_name)) {
 
-                                    $updated = call_user_func( $function_name );
+                                    $updated = call_user_func($function_name);
 
                                 }  // End If Statement
 
-                                if ( $updated ) {
-                                    array_push( $this->updates_run, $function_name );
+                                if ($updated) {
+                                    array_push($this->updates_run, $function_name);
                                 } // End If Statement
                             }
                         } // End If Statement
@@ -450,8 +450,8 @@ 
             } // End For Loop
         } // End For Loop
 
-        $this->updates_run = array_unique( $this->updates_run ); // we only need one reference per update
-        update_option( $this->token . '-upgrades', $this->updates_run );
+        $this->updates_run = array_unique($this->updates_run); // we only need one reference per update
+        update_option($this->token.'-upgrades', $this->updates_run);
 
         return true;
 
@@ -459,7 +459,7 @@ 
 
 	private function force_updates() {
 
-		if( ! isset( $_GET['page'] ) || 'sensei_updates' != $_GET['page'] ) {
+		if ( ! isset($_GET['page']) || 'sensei_updates' != $_GET['page']) {
 
 			// $skip_forced_updates = false;
 			// if( ! get_option( 'woothemes-sensei-force-updates', false ) ) {
@@ -468,8 +468,8 @@ 
 
 			// Force critical updates if only if lessons already exist
 			$skip_forced_updates = false;
-			$lesson_posts = wp_count_posts( 'lesson' );
-			if( ! isset( $lesson_posts->publish ) || ! $lesson_posts->publish ) {
+			$lesson_posts = wp_count_posts('lesson');
+			if ( ! isset($lesson_posts->publish) || ! $lesson_posts->publish) {
 				$skip_forced_updates = true;
 			}
 
@@ -477,69 +477,69 @@ 
 
 			$updates_to_run = array();
 
-			foreach ( $this->updates as $version => $value ) {
-				foreach ( $this->updates[$version] as $upgrade_type => $function_to_run ) {
-					if ( $upgrade_type == 'forced' ) {
-						foreach ( $function_to_run as $function_name => $update_data ) {
+			foreach ($this->updates as $version => $value) {
+				foreach ($this->updates[$version] as $upgrade_type => $function_to_run) {
+					if ($upgrade_type == 'forced') {
+						foreach ($function_to_run as $function_name => $update_data) {
 
-							if( $skip_forced_updates ) {
-								$this->set_update_run( $function_name );
+							if ($skip_forced_updates) {
+								$this->set_update_run($function_name);
 								continue;
 							}
 
-							$update_run = $this->has_update_run( $function_name );
+							$update_run = $this->has_update_run($function_name);
 
-							if( ! $update_run ) {
+							if ( ! $update_run) {
 								$use_the_force = true;
-								$updates_to_run[ $function_name ] = $update_data;
+								$updates_to_run[$function_name] = $update_data;
 							}
 						}
 					}
 				}
 			}
 
-			if( $skip_forced_updates ) {
+			if ($skip_forced_updates) {
 				return;
 			}
 
-			if( $use_the_force && 0 < count( $updates_to_run ) ) {
+			if ($use_the_force && 0 < count($updates_to_run)) {
 
-				$update_title = __( 'Important Sensei updates required', 'woothemes-sensei' );
+				$update_title = __('Important Sensei updates required', 'woothemes-sensei');
 
-				$update_message = '<h1>' . __( 'Important Sensei upgrades required!', 'woothemes-sensei' ) . '</h1>' . "\n";
+				$update_message = '<h1>'.__('Important Sensei upgrades required!', 'woothemes-sensei').'</h1>'."\n";
 
 				// $update_message .= '<h4>' . sprintf( __( 'These updates are only required if you are updating from a previous version of Sensei. If you are installing Sensei for the first time, %1$syou can dismiss this page by clicking here%2$s.', 'woothemes-sensei' ), '<a href="' . add_query_arg( array( 'sensei_skip_forced_updates' => 'true' ) ) . '">', '</a>' ) . '</h4>' ."\n";
 
-				$update_message .= '<p>' . __( 'The latest version of Sensei requires some important database upgrades. In order to run these upgrades you will need to follow the step by step guide below. Your site will not function correctly unless you run these critical updates.', 'woothemes-sensei' ) . '</p>' . "\n";
+				$update_message .= '<p>'.__('The latest version of Sensei requires some important database upgrades. In order to run these upgrades you will need to follow the step by step guide below. Your site will not function correctly unless you run these critical updates.', 'woothemes-sensei').'</p>'."\n";
 
-				$update_message .= '<p><b>' . __( 'To run the upgrades click on each of the links below in the order that they appear.', 'woothemes-sensei' ) . '</b></p>' . "\n";
+				$update_message .= '<p><b>'.__('To run the upgrades click on each of the links below in the order that they appear.', 'woothemes-sensei').'</b></p>'."\n";
 
-				$update_message .= '<p>' . __( 'Clicking each link will open up a new window/tab - do not close that window/tab until you see the message \'Update completed successfully\'. Once you see that message you can close the window/tab and start the next upgrade by clicking on the next link in the list.', 'woothemes-sensei' ) . '</p>' . "\n";
+				$update_message .= '<p>'.__('Clicking each link will open up a new window/tab - do not close that window/tab until you see the message \'Update completed successfully\'. Once you see that message you can close the window/tab and start the next upgrade by clicking on the next link in the list.', 'woothemes-sensei').'</p>'."\n";
 
-				$update_message .= '<p><b>' . __( 'Once all the upgrades have been completed you will be able to use your WordPress site again.', 'woothemes-sensei' ) . '</b></p>' . "\n";
+				$update_message .= '<p><b>'.__('Once all the upgrades have been completed you will be able to use your WordPress site again.', 'woothemes-sensei').'</b></p>'."\n";
 
-				$update_message .= '<ol>' . "\n";
+				$update_message .= '<ol>'."\n";
 
-					foreach( $updates_to_run as $function => $data ) {
+					foreach ($updates_to_run as $function => $data) {
 
-						if( ! isset( $data['title'] ) ) {
+						if ( ! isset($data['title'])) {
 							break;
 						}
 
-						$update_message .= '<li style="margin:5px 0;"><a href="' . admin_url( 'admin.php?page=sensei_updates&action=update&n=0&functions[]=' . $function ) . '" target="_blank">' . $data['title'] . '</a></li>';
+						$update_message .= '<li style="margin:5px 0;"><a href="'.admin_url('admin.php?page=sensei_updates&action=update&n=0&functions[]='.$function).'" target="_blank">'.$data['title'].'</a></li>';
 					}
 
-				$update_message .= '</ol>' . "\n";
+				$update_message .= '</ol>'."\n";
 
-				switch( $version ) {
+				switch ($version) {
 
 					case '1.7.0':
-						$update_message .= '<p><em>' . sprintf( __( 'Want to know what these upgrades are all about? %1$sFind out more here%2$s.', 'woothemes-sensei' ), '<a href="http://develop.woothemes.com/sensei/2014/12/03/important-information-about-sensei-1-7" target="_blank">', '</a>' ) . '</em></p>' . "\n";
+						$update_message .= '<p><em>'.sprintf(__('Want to know what these upgrades are all about? %1$sFind out more here%2$s.', 'woothemes-sensei'), '<a href="http://develop.woothemes.com/sensei/2014/12/03/important-information-about-sensei-1-7" target="_blank">', '</a>').'</em></p>'."\n";
 					break;
 
 				}
 
-				wp_die( $update_message, $update_title );
+				wp_die($update_message, $update_title);
 			}
 		}
 	}
@@ -551,8 +551,8 @@ 
 	 * @since  1.4.0
 	 * @return boolean
 	 */
-	private function has_update_run( $update ) {
-		if ( in_array( $update, $this->updates_run ) ) {
+	private function has_update_run($update) {
+		if (in_array($update, $this->updates_run)) {
 			return true;
 		}
 		return false;
@@ -564,10 +564,10 @@ 
 	 * @param string $update Update to process
 	 * @since  1.4.0
 	 */
-	private function set_update_run( $update ) {
-		array_push( $this->updates_run, $update );
-        $this->updates_run = array_unique( $this->updates_run ); // we only need one reference per update
-		update_option( Sensei()->token . '-upgrades', $this->updates_run );
+	private function set_update_run($update) {
+		array_push($this->updates_run, $update);
+        $this->updates_run = array_unique($this->updates_run); // we only need one reference per update
+		update_option(Sensei()->token.'-upgrades', $this->updates_run);
 	}
 
 	/**
@@ -578,15 +578,15 @@ 
 	 * @return void
 	 */
 	public function assign_role_caps() {
-		foreach ( $this->parent->post_types->role_caps as $role_cap_set  ) {
-			foreach ( $role_cap_set as $role_key => $capabilities_array ) {
+		foreach ($this->parent->post_types->role_caps as $role_cap_set) {
+			foreach ($role_cap_set as $role_key => $capabilities_array) {
 				/* Get the role. */
-				$role = get_role( $role_key );
-				foreach ( $capabilities_array as $cap_name  ) {
+				$role = get_role($role_key);
+				foreach ($capabilities_array as $cap_name) {
 					/* If the role exists, add required capabilities for the plugin. */
-					if ( !empty( $role ) ) {
-						if ( !$role->has_cap( $cap_name ) ) {
-							$role->add_cap( $cap_name );
+					if ( ! empty($role)) {
+						if ( ! $role->has_cap($cap_name)) {
+							$role->add_cap($cap_name);
 						} // End If Statement
 					} // End If Statement
 				} // End For Loop
@@ -602,16 +602,16 @@ 
 	 * @return void
 	 */
 	public function set_default_quiz_grade_type() {
-		$args = array(	'post_type' 		=> 'quiz',
+		$args = array('post_type' 		=> 'quiz',
 						'posts_per_page' 		=> -1,
 						'post_status'		=> 'publish',
 						'suppress_filters' 	=> 0
 						);
-		$quizzes = get_posts( $args );
+		$quizzes = get_posts($args);
 
-		foreach( $quizzes as $quiz ) {
-			update_post_meta( $quiz->ID, '_quiz_grade_type', 'auto' );
-			update_post_meta( $quiz->ID, '_quiz_grade_type_disabled', '' );
+		foreach ($quizzes as $quiz) {
+			update_post_meta($quiz->ID, '_quiz_grade_type', 'auto');
+			update_post_meta($quiz->ID, '_quiz_grade_type_disabled', '');
 		}
 	} // End set_default_quiz_grade_type
 
@@ -623,23 +623,23 @@ 
 	 * @return void
 	 */
 	public function set_default_question_type() {
-		$args = array(	'post_type' 		=> 'question',
+		$args = array('post_type' 		=> 'question',
 						'posts_per_page' 		=> -1,
 						'post_status'		=> 'publish',
 						'suppress_filters' 	=> 0
 						);
-		$questions = get_posts( $args );
+		$questions = get_posts($args);
 
 		$already_run = true;
-		foreach( $questions as $question ) {
-			if( $already_run ) {
-				$terms = wp_get_post_terms( $question->ID, 'question-type' );
-				if( is_array( $terms ) && count( $terms ) > 0 ) {
+		foreach ($questions as $question) {
+			if ($already_run) {
+				$terms = wp_get_post_terms($question->ID, 'question-type');
+				if (is_array($terms) && count($terms) > 0) {
 					break;
 				}
 			}
 			$already_run = false;
-			wp_set_post_terms( $question->ID, array( 'multiple-choice' ), 'question-type' );
+			wp_set_post_terms($question->ID, array('multiple-choice'), 'question-type');
 		}
 
 	} // End set_default_question_type
@@ -651,90 +651,90 @@ 
 	 * @access public
 	 * @return boolean
 	 */
-	public function update_question_answer_data( $n = 50, $offset = 0 ) {
+	public function update_question_answer_data($n = 50, $offset = 0) {
 
 		// Get Total Number of Updates to run
-		$quiz_count_object = wp_count_posts( 'quiz' );
+		$quiz_count_object = wp_count_posts('quiz');
 		$quiz_count_published = $quiz_count_object->publish;
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		} // End If Statement
-		$total_pages = intval( $quiz_count_published / $n );
+		$total_pages = intval($quiz_count_published / $n);
 
 
-		$args = array(	'post_type' 		=> 'quiz',
+		$args = array('post_type' 		=> 'quiz',
 						'posts_per_page' 		=> $n,
 						'offset'			=> $offset,
 						'post_status'		=> 'publish',
 						'suppress_filters' 	=> 0
 						);
-		$quizzes = get_posts( $args );
+		$quizzes = get_posts($args);
 
 		$old_answers = array();
 		$right_answers = array();
 		$old_user_answers = array();
 
-		if( is_array( $quizzes ) ) {
-			foreach( $quizzes as $quiz ) {
+		if (is_array($quizzes)) {
+			foreach ($quizzes as $quiz) {
 				$quiz_id = $quiz->ID;
 
 				// Get current user answers
-				$comments = Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $quiz_id, 'type' => 'sensei_quiz_answers' ), true  );
+				$comments = Sensei_Utils::sensei_check_for_activity(array('post_id' => $quiz_id, 'type' => 'sensei_quiz_answers'), true);
 				// Need to always return an array, even with only 1 item
-				if ( !is_array($comments) ) {
-					$comments = array( $comments );
+				if ( ! is_array($comments)) {
+					$comments = array($comments);
 				}
-				foreach ( $comments as $comment ) {
+				foreach ($comments as $comment) {
 					$user_id = $comment->user_id;
-					$content = maybe_unserialize( base64_decode( $comment->comment_content ) );
-					$old_user_answers[ $quiz_id ][ $user_id ] = $content;
+					$content = maybe_unserialize(base64_decode($comment->comment_content));
+					$old_user_answers[$quiz_id][$user_id] = $content;
 				}
 
 				// Get correct answers
-				$questions = Sensei_Utils::sensei_get_quiz_questions( $quiz_id );
-				if( is_array( $questions ) ) {
-					foreach( $questions as $question ) {
-						$right_answer = get_post_meta( $question->ID, '_question_right_answer', true );
-						$right_answers[ $quiz_id ][ $question->ID ] = $right_answer;
+				$questions = Sensei_Utils::sensei_get_quiz_questions($quiz_id);
+				if (is_array($questions)) {
+					foreach ($questions as $question) {
+						$right_answer = get_post_meta($question->ID, '_question_right_answer', true);
+						$right_answers[$quiz_id][$question->ID] = $right_answer;
 					}
 				}
 			}
 		}
 
-		if( is_array( $right_answers ) ) {
-			foreach( $right_answers as $quiz_id => $question ) {
+		if (is_array($right_answers)) {
+			foreach ($right_answers as $quiz_id => $question) {
 				$count = 0;
-				if( is_array( $question ) ) {
-					foreach( $question as $question_id => $answer ) {
+				if (is_array($question)) {
+					foreach ($question as $question_id => $answer) {
 						++$count;
-						if( isset( $old_user_answers[ $quiz_id ] ) ) {
-							$answers_linkup[ $quiz_id ][ $count ] = $question_id;
+						if (isset($old_user_answers[$quiz_id])) {
+							$answers_linkup[$quiz_id][$count] = $question_id;
 						}
 					}
 				}
 			}
 		}
 
-		if( is_array( $old_user_answers ) ) {
-			foreach( $old_user_answers as $quiz_id => $user_answers ) {
-				foreach( $user_answers as $user_id => $answers ) {
-					foreach( $answers as $answer_id => $user_answer ) {
-						$question_id = $answers_linkup[ $quiz_id ][ $answer_id ];
-						$new_user_answers[ $question_id ] = $user_answer;
-						Sensei_Utils::sensei_grade_question_auto( $question_id, '', $user_answer, $user_id );
+		if (is_array($old_user_answers)) {
+			foreach ($old_user_answers as $quiz_id => $user_answers) {
+				foreach ($user_answers as $user_id => $answers) {
+					foreach ($answers as $answer_id => $user_answer) {
+						$question_id = $answers_linkup[$quiz_id][$answer_id];
+						$new_user_answers[$question_id] = $user_answer;
+						Sensei_Utils::sensei_grade_question_auto($question_id, '', $user_answer, $user_id);
 					}
-					$lesson_id = get_post_meta( $quiz_id, '_quiz_lesson', true );
-					Sensei_Utils::sensei_start_lesson( $lesson_id, $user_id );
-					Sensei_Utils::sensei_save_quiz_answers( $new_user_answers, $user_id );
+					$lesson_id = get_post_meta($quiz_id, '_quiz_lesson', true);
+					Sensei_Utils::sensei_start_lesson($lesson_id, $user_id);
+					Sensei_Utils::sensei_save_quiz_answers($new_user_answers, $user_id);
 				}
 			}
 		}
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
@@ -749,15 +749,15 @@ 
 	 * @return boolean
 	 */
 	public function update_question_grade_points() {
-		$args = array(	'post_type' 		=> 'question',
+		$args = array('post_type' 		=> 'question',
 						'posts_per_page' 		=> -1,
 						'post_status'		=> 'publish',
 						'suppress_filters' 	=> 0
 						);
-		$questions = get_posts( $args );
+		$questions = get_posts($args);
 
-		foreach( $questions as $question ) {
-			update_post_meta( $question->ID, '_question_grade', '1' );
+		foreach ($questions as $question) {
+			update_post_meta($question->ID, '_question_grade', '1');
 		}
 		return true;
 	} // End update_question_grade_points
@@ -769,7 +769,7 @@ 
 	 * @return boolean
 	 */
 	public function convert_essay_paste_questions() {
-		$args = array(	'post_type' 		=> 'question',
+		$args = array('post_type' 		=> 'question',
 						'posts_per_page' 		=> -1,
 						'post_status'		=> 'publish',
 						'tax_query'			=> array(
@@ -781,14 +781,14 @@ 
 						),
 						'suppress_filters' 	=> 0
 						);
-		$questions = get_posts( $args );
+		$questions = get_posts($args);
 
-		foreach( $questions as $question ) {
-			wp_set_object_terms( $question->ID, 'multi-line', 'question-type', false );
+		foreach ($questions as $question) {
+			wp_set_object_terms($question->ID, 'multi-line', 'question-type', false);
 
-			$quiz_id = get_post_meta( $question->ID, '_quiz_id', true );
-			if( 0 < intval( $quiz_id ) ) {
-				add_post_meta( $question->ID, '_quiz_question_order' . $quiz_id, $quiz_id . '0000', true );
+			$quiz_id = get_post_meta($question->ID, '_quiz_id', true);
+			if (0 < intval($quiz_id)) {
+				add_post_meta($question->ID, '_quiz_question_order'.$quiz_id, $quiz_id.'0000', true);
 			}
 		}
 		return true;
@@ -800,33 +800,33 @@ 
 	 * @since  1.5.0
 	 * @return boolean
 	 */
-	public function set_random_question_order( $n = 50, $offset = 0 ) {
+	public function set_random_question_order($n = 50, $offset = 0) {
 
 		// Get Total Number of Updates to run
-		$quiz_count_object = wp_count_posts( 'quiz' );
+		$quiz_count_object = wp_count_posts('quiz');
 		$quiz_count_published = $quiz_count_object->publish;
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		} // End If Statement
-		$total_pages = intval( $quiz_count_published / $n );
+		$total_pages = intval($quiz_count_published / $n);
 
-		$args = array(	'post_type' 		=> 'quiz',
+		$args = array('post_type' 		=> 'quiz',
 						'post_status'		=> 'any',
 						'posts_per_page' 		=> $n,
 						'offset'			=> $offset,
 						'suppress_filters' 	=> 0
 						);
-		$quizzes = get_posts( $args );
+		$quizzes = get_posts($args);
 
-		foreach( $quizzes as $quiz ) {
-			update_post_meta( $quiz->ID, '_random_question_order', 'yes' );
+		foreach ($quizzes as $quiz) {
+			update_post_meta($quiz->ID, '_random_question_order', 'yes');
 		}
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
@@ -840,37 +840,37 @@ 
 	 * @since  1.5.0
 	 * @return boolean
 	 */
-	public function set_default_show_question_count( $n = 50, $offset = 0 ) {
+	public function set_default_show_question_count($n = 50, $offset = 0) {
 
-		$args = array(	'post_type' 		=> 'quiz',
+		$args = array('post_type' 		=> 'quiz',
 						'post_status'		=> 'any',
 						'posts_per_page' 		=> $n,
 						'offset'			=> $offset,
 						'meta_key'			=> '_show_questions',
 						'suppress_filters' 	=> 0
 						);
-		$quizzes = get_posts( $args );
+		$quizzes = get_posts($args);
 
-		$total_quizzes = count( $quizzes );
+		$total_quizzes = count($quizzes);
 
-		if( 0 == intval( $total_quizzes ) ) {
+		if (0 == intval($total_quizzes)) {
 			return true;
 		}
 
-		foreach( $quizzes as $quiz ) {
-			delete_post_meta( $quiz->ID, '_show_questions' );
+		foreach ($quizzes as $quiz) {
+			delete_post_meta($quiz->ID, '_show_questions');
 		}
 
-		$total_pages = intval( $total_quizzes / $n );
+		$total_pages = intval($total_quizzes / $n);
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		} // End If Statement
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
@@ -878,14 +878,14 @@ 
 
 	}
 
-	public function remove_deleted_user_activity( $n = 50, $offset = 0 ) {
+	public function remove_deleted_user_activity($n = 50, $offset = 0) {
 
-		$all_activity = get_comments( array( 'status' => 'approve' ) );
+		$all_activity = get_comments(array('status' => 'approve'));
 		$activity_count = array();
-		foreach( $all_activity as $activity ) {
-			if( '' == $activity->comment_type ) continue;
-			if( strpos( 'sensei_', $activity->comment_type ) != 0 ) continue;
-			if( 0 == $activity->user_id ) continue;
+		foreach ($all_activity as $activity) {
+			if ('' == $activity->comment_type) continue;
+			if (strpos('sensei_', $activity->comment_type) != 0) continue;
+			if (0 == $activity->user_id) continue;
 			$activity_count[] = $activity->comment_ID;
 		}
 
@@ -895,33 +895,33 @@ 
 			'status' => 'approve'
 		);
 
-		$activities = get_comments( $args );
+		$activities = get_comments($args);
 
-		foreach( $activities as $activity ) {
-			if( '' == $activity->comment_type ) continue;
-			if( strpos( 'sensei_', $activity->comment_type ) != 0 ) continue;
-			if( 0 == $activity->user_id ) continue;
+		foreach ($activities as $activity) {
+			if ('' == $activity->comment_type) continue;
+			if (strpos('sensei_', $activity->comment_type) != 0) continue;
+			if (0 == $activity->user_id) continue;
 
-			$user_exists = get_userdata( $activity->user_id );
+			$user_exists = get_userdata($activity->user_id);
 
-			if( ! $user_exists ) {
-				wp_delete_comment( intval( $activity->comment_ID ), true );
+			if ( ! $user_exists) {
+				wp_delete_comment(intval($activity->comment_ID), true);
 				wp_cache_flush();
 			}
 		}
 
-		$total_activities = count( $activity_count );
+		$total_activities = count($activity_count);
 
-		$total_pages = intval( $total_activities / $n );
+		$total_pages = intval($total_activities / $n);
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		} // End If Statement
 
-		if ( $current_page >= $total_pages ) {
+		if ($current_page >= $total_pages) {
 			return true;
 		} else {
 			return false;
@@ -930,16 +930,16 @@ 
 	}
 
 	public function add_teacher_role() {
-		add_role( 'teacher', __( 'Teacher', 'woothemes-sensei' ), array( 'read' => true, 'manage_sensei_grades' => true ) );
+		add_role('teacher', __('Teacher', 'woothemes-sensei'), array('read' => true, 'manage_sensei_grades' => true));
 		return true;
 	}
 
 	public function add_sensei_caps() {
-		$role = get_role( 'administrator' );
+		$role = get_role('administrator');
 
-		if( ! is_null( $role ) ) {
-			$role->add_cap( 'manage_sensei' );
-			$role->add_cap( 'manage_sensei_grades' );
+		if ( ! is_null($role)) {
+			$role->add_cap('manage_sensei');
+			$role->add_cap('manage_sensei_grades');
 		}
 
 		return true;
@@ -953,16 +953,16 @@ 
 			'suppress_filters' 	=> 0
 		);
 
-		$questions = get_posts( $args );
+		$questions = get_posts($args);
 
-		foreach( $questions as $question ) {
+		foreach ($questions as $question) {
 
-			if( ! isset( $question->ID ) ) continue;
+			if ( ! isset($question->ID)) continue;
 
-			$quiz_id = get_post_meta( $question->ID, '_quiz_id', true );
+			$quiz_id = get_post_meta($question->ID, '_quiz_id', true);
 
-			$question_order = get_post_meta( $question->ID, '_quiz_question_order', true );
-			update_post_meta( $question->ID, '_quiz_question_order' . $quiz_id, $question_order );
+			$question_order = get_post_meta($question->ID, '_quiz_question_order', true);
+			update_post_meta($question->ID, '_quiz_question_order'.$quiz_id, $question_order);
 
 		}
 
@@ -971,15 +971,15 @@ 
 
 	public function update_quiz_settings() {
 
-		$settings = get_option( 'woothemes-sensei-settings', array() );
+		$settings = get_option('woothemes-sensei-settings', array());
 
 		$lesson_completion = false;
-		if( isset( $settings['lesson_completion'] ) ) {
+		if (isset($settings['lesson_completion'])) {
 			$lesson_completion = $settings['lesson_completion'];
 		}
 
 		$reset_quiz_allowed = false;
-		if( isset( $settings['quiz_reset_allowed'] ) ) {
+		if (isset($settings['quiz_reset_allowed'])) {
 			$reset_quiz_allowed = $settings['quiz_reset_allowed'];
 		}
 
@@ -990,20 +990,20 @@ 
 			'suppress_filters' 	=> 0
 		);
 
-		$quizzes = get_posts( $args );
+		$quizzes = get_posts($args);
 
-		foreach( $quizzes as $quiz ) {
+		foreach ($quizzes as $quiz) {
 
-			if( ! isset( $quiz->ID ) ) continue;
+			if ( ! isset($quiz->ID)) continue;
 
-			if( isset( $lesson_completion ) && 'passed' == $lesson_completion ) {
-				update_post_meta( $quiz->ID, '_pass_required', 'on' );
+			if (isset($lesson_completion) && 'passed' == $lesson_completion) {
+				update_post_meta($quiz->ID, '_pass_required', 'on');
 			} else {
-				update_post_meta( $quiz->ID, '_quiz_passmark', 0 );
+				update_post_meta($quiz->ID, '_quiz_passmark', 0);
 			}
 
-			if( isset( $reset_quiz_allowed ) && $reset_quiz_allowed ) {
-				update_post_meta( $quiz->ID, '_enable_quiz_reset', 'on' );
+			if (isset($reset_quiz_allowed) && $reset_quiz_allowed) {
+				update_post_meta($quiz->ID, '_enable_quiz_reset', 'on');
 			}
 		}
 
@@ -1018,22 +1018,22 @@ 
 			'suppress_filters' 	=> 0
 		);
 
-		$lessons = get_posts( $args );
+		$lessons = get_posts($args);
 
-		foreach( $lessons as $lesson ) {
+		foreach ($lessons as $lesson) {
 
-			if( ! isset( $lesson->ID ) ) continue;
+			if ( ! isset($lesson->ID)) continue;
 
-			$course_id = get_post_meta( $lesson->ID, '_lesson_course', true);
+			$course_id = get_post_meta($lesson->ID, '_lesson_course', true);
 
-			if( $course_id ) {
-				update_post_meta( $lesson->ID, '_order_' . $course_id, 0 );
+			if ($course_id) {
+				update_post_meta($lesson->ID, '_order_'.$course_id, 0);
 			}
 
-            $module = Sensei()->modules->get_lesson_module( $lesson->ID );
+            $module = Sensei()->modules->get_lesson_module($lesson->ID);
 
-            if( $module ) {
-                update_post_meta( $lesson->ID, '_order_module_' . $module->term_id, 0 );
+            if ($module) {
+                update_post_meta($lesson->ID, '_order_module_'.$module->term_id, 0);
             }
 
 		}
@@ -1042,10 +1042,10 @@ 
 	}
 
 	public function add_editor_caps() {
-		$role = get_role( 'editor' );
+		$role = get_role('editor');
 
-		if( ! is_null( $role ) ) {
-			$role->add_cap( 'manage_sensei_grades' );
+		if ( ! is_null($role)) {
+			$role->add_cap('manage_sensei_grades');
 		}
 
 		return true;
@@ -1064,26 +1064,26 @@ 
 					WHERE m.meta_key = '_question_right_answer' AND m.meta_value LIKE '%|%' AND m.meta_value NOT LIKE '%||%'
 						AND m.post_id = tr.object_id AND tr.term_taxonomy_id = tt.term_taxonomy_id AND tt.term_id = t.term_id
 						AND tt.taxonomy = 'question-type' AND t.slug = 'gap-fill'";
-		$wpdb->query( $sql );
+		$wpdb->query($sql);
 
 		return true;
 	}
 
-	public function update_quiz_lesson_relationship( $n = 50, $offset = 0 ) {
-		$count_object = wp_count_posts( 'quiz' );
+	public function update_quiz_lesson_relationship($n = 50, $offset = 0) {
+		$count_object = wp_count_posts('quiz');
 
 		$count_published = 0;
-		foreach ( $count_object AS $status => $count ) {
+		foreach ($count_object AS $status => $count) {
 			$count_published += $count;
 		}
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		}
-		$total_pages = ceil( $count_published / $n );
+		$total_pages = ceil($count_published / $n);
 
 		$args = array(
 			'post_type' => 'quiz',
@@ -1092,52 +1092,52 @@ 
 			'post_status' => 'any'
 		);
 
-		$quizzes = get_posts( $args );
+		$quizzes = get_posts($args);
 
-		foreach( $quizzes as $quiz ) {
+		foreach ($quizzes as $quiz) {
 
-			if( ! isset( $quiz->ID ) || 0 != $quiz->post_parent ) continue;
+			if ( ! isset($quiz->ID) || 0 != $quiz->post_parent) continue;
 
-			$lesson_id = get_post_meta( $quiz->ID, '_quiz_lesson', true );
+			$lesson_id = get_post_meta($quiz->ID, '_quiz_lesson', true);
 
-			if( empty( $lesson_id ) ) continue;
+			if (empty($lesson_id)) continue;
 
 			$data = array(
 				'ID' => $quiz->ID,
 				'post_parent' => $lesson_id,
 			);
-			wp_update_post( $data );
+			wp_update_post($data);
 
-			update_post_meta( $lesson_id, '_lesson_quiz', $quiz->ID );
+			update_post_meta($lesson_id, '_lesson_quiz', $quiz->ID);
 		}
 
-		if ( $current_page == $total_pages || 0 == $total_pages ) {
+		if ($current_page == $total_pages || 0 == $total_pages) {
 			return true;
 		} else {
 			return false;
 		}
 	}
 
-	function status_changes_fix_lessons( $n = 50, $offset = 0 ) {
+	function status_changes_fix_lessons($n = 50, $offset = 0) {
 		global $wpdb;
 
-		$count_object = wp_count_posts( 'lesson' );
+		$count_object = wp_count_posts('lesson');
 		$count_published = 0;
-		foreach ( $count_object AS $status => $count ) {
+		foreach ($count_object AS $status => $count) {
 			$count_published += $count;
 		}
 
-		if ( 0 == $count_published ) {
+		if (0 == $count_published) {
 			return true;
 		}
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		}
-		$total_pages = ceil( $count_published / $n );
+		$total_pages = ceil($count_published / $n);
 
 		// Get all Lessons with (and without) Quizzes...
 		$args = array(
@@ -1147,26 +1147,26 @@ 
 			'offset' => $offset,
 			'fields' => 'ids'
 		);
-		$lesson_ids = get_posts( $args );
+		$lesson_ids = get_posts($args);
 
 		// ...get all Quiz IDs for the above Lessons
-		$id_list = join( ',', $lesson_ids );
-		$meta_list = $wpdb->get_results( "SELECT post_id, meta_value FROM $wpdb->postmeta WHERE meta_key = '_quiz_lesson' AND meta_value IN ($id_list)", ARRAY_A );
+		$id_list = join(',', $lesson_ids);
+		$meta_list = $wpdb->get_results("SELECT post_id, meta_value FROM $wpdb->postmeta WHERE meta_key = '_quiz_lesson' AND meta_value IN ($id_list)", ARRAY_A);
 		$lesson_quiz_ids = array();
-		if ( !empty($meta_list) ) {
-			foreach ( $meta_list as $metarow ) {
+		if ( ! empty($meta_list)) {
+			foreach ($meta_list as $metarow) {
 				$lesson_id = $metarow['meta_value'];
 				$quiz_id = $metarow['post_id'];
-				$lesson_quiz_ids[ $lesson_id ] = $quiz_id;
+				$lesson_quiz_ids[$lesson_id] = $quiz_id;
 			}
 		}
 
 		// ...check all Quiz IDs for questions
-		$id_list = join( ',', array_values($lesson_quiz_ids) );
-		$meta_list = $wpdb->get_results( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key = '_quiz_id' AND meta_value IN ($id_list)", ARRAY_A );
+		$id_list = join(',', array_values($lesson_quiz_ids));
+		$meta_list = $wpdb->get_results("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key = '_quiz_id' AND meta_value IN ($id_list)", ARRAY_A);
 		$lesson_quiz_ids_with_questions = array();
-		if ( !empty($meta_list) ) {
-			foreach ( $meta_list as $metarow ) {
+		if ( ! empty($meta_list)) {
+			foreach ($meta_list as $metarow) {
 				$quiz_id = $metarow['meta_value'];
 				$lesson_quiz_ids_with_questions[] = $quiz_id;
 			}
@@ -1174,49 +1174,49 @@ 
 
 		// For each quiz check there are questions, if not remove the corresponding meta keys from Quizzes and Lessons
 		// if there are questions on the quiz add the corresponding meta keys to Quizzes and Lessons
-		$d_count = $a_count =0;
-		foreach ( $lesson_quiz_ids AS $lesson_id => $quiz_id ) {
-			if ( !in_array( $quiz_id, $lesson_quiz_ids_with_questions ) ) {
+		$d_count = $a_count = 0;
+		foreach ($lesson_quiz_ids AS $lesson_id => $quiz_id) {
+			if ( ! in_array($quiz_id, $lesson_quiz_ids_with_questions)) {
 
 				// Quiz has no questions, drop the corresponding data
-				delete_post_meta( $quiz_id, '_pass_required' );
-				delete_post_meta( $quiz_id, '_quiz_passmark' );
-				delete_post_meta( $lesson_id, '_quiz_has_questions' );
+				delete_post_meta($quiz_id, '_pass_required');
+				delete_post_meta($quiz_id, '_quiz_passmark');
+				delete_post_meta($lesson_id, '_quiz_has_questions');
 				$d_count++;
 			}
-			else if ( in_array( $quiz_id, $lesson_quiz_ids_with_questions ) ) {
+			else if (in_array($quiz_id, $lesson_quiz_ids_with_questions)) {
 
 				// Quiz has no questions, drop the corresponding data
-				update_post_meta( $lesson_id, '_quiz_has_questions', true );
+				update_post_meta($lesson_id, '_quiz_has_questions', true);
 				$a_count++;
 			}
 		}
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
 		}
 	}
 
-	function status_changes_convert_lessons( $n = 50, $offset = 0 ) {
+	function status_changes_convert_lessons($n = 50, $offset = 0) {
 		global $wpdb;
 
-		wp_defer_comment_counting( true );
+		wp_defer_comment_counting(true);
 
-		$user_count_result = $wpdb->get_var( "SELECT COUNT(*) FROM $wpdb->users " );
+		$user_count_result = $wpdb->get_var("SELECT COUNT(*) FROM $wpdb->users ");
 
-		if ( 0 == $user_count_result ) {
+		if (0 == $user_count_result) {
 			return true;
 		}
 
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		}
 
-		$total_pages = ceil( $user_count_result / $n );
+		$total_pages = ceil($user_count_result / $n);
 
 		// Get all Lessons with Quizzes...
 		$args = array(
@@ -1231,34 +1231,34 @@ 
 			),
 			'fields' => 'ids'
 		);
-		$lesson_ids_with_quizzes = get_posts( $args );
+		$lesson_ids_with_quizzes = get_posts($args);
 		// ...get all Quiz IDs for the above Lessons
-		$id_list = join( ',', $lesson_ids_with_quizzes );
-		$meta_list = $wpdb->get_results( "SELECT post_id, meta_value FROM $wpdb->postmeta WHERE meta_key = '_quiz_lesson' AND meta_value IN ($id_list)", ARRAY_A );
+		$id_list = join(',', $lesson_ids_with_quizzes);
+		$meta_list = $wpdb->get_results("SELECT post_id, meta_value FROM $wpdb->postmeta WHERE meta_key = '_quiz_lesson' AND meta_value IN ($id_list)", ARRAY_A);
 		$lesson_quiz_ids = array();
-		if ( !empty($meta_list) ) {
-			foreach ( $meta_list as $metarow ) {
+		if ( ! empty($meta_list)) {
+			foreach ($meta_list as $metarow) {
 				$lesson_id = $metarow['meta_value'];
 				$quiz_id = $metarow['post_id'];
-				$lesson_quiz_ids[ $lesson_id ] = $quiz_id;
+				$lesson_quiz_ids[$lesson_id] = $quiz_id;
 			}
 		}
 
 		// ...get all Pass Required & Passmarks for the above Lesson/Quizzes
-		$id_list = join( ',', array_values($lesson_quiz_ids) );
-		$meta_list = $wpdb->get_results( "SELECT post_id, meta_key, meta_value FROM $wpdb->postmeta WHERE ( meta_key = '_pass_required' OR meta_key = '_quiz_passmark' ) AND post_id IN ($id_list)", ARRAY_A );
+		$id_list = join(',', array_values($lesson_quiz_ids));
+		$meta_list = $wpdb->get_results("SELECT post_id, meta_key, meta_value FROM $wpdb->postmeta WHERE ( meta_key = '_pass_required' OR meta_key = '_quiz_passmark' ) AND post_id IN ($id_list)", ARRAY_A);
 		$quizzes_pass_required = $quizzes_passmarks = array();
-		if ( !empty($meta_list) ) {
-			foreach ( $meta_list as $metarow ) {
-				if ( !empty($metarow['meta_value']) ) {
+		if ( ! empty($meta_list)) {
+			foreach ($meta_list as $metarow) {
+				if ( ! empty($metarow['meta_value'])) {
 					$quiz_id = $metarow['post_id'];
 					$key = $metarow['meta_key'];
 					$value = $metarow['meta_value'];
-					if ( '_pass_required' == $key ) {
-						$quizzes_pass_required[ $quiz_id ] = $value;
+					if ('_pass_required' == $key) {
+						$quizzes_pass_required[$quiz_id] = $value;
 					}
-					if ( '_quiz_passmark' == $key ) {
-						$quizzes_passmarks[ $quiz_id ] = $value;
+					if ('_quiz_passmark' == $key) {
+						$quizzes_passmarks[$quiz_id] = $value;
 					}
 				}
 			}
@@ -1272,41 +1272,41 @@ 
 		$check_existing_sql = "SELECT comment_ID FROM $wpdb->comments WHERE comment_post_ID = %d AND user_id = %d AND comment_type = 'sensei_lesson_status' ";
 
 		// $per_page users at a time, could be batch run via an admin ajax command, 1 user at a time?
-		$user_ids = $wpdb->get_col( $wpdb->prepare($users_sql, $n, $offset) );
+		$user_ids = $wpdb->get_col($wpdb->prepare($users_sql, $n, $offset));
 
-		foreach ( $user_ids AS $user_id ) {
+		foreach ($user_ids AS $user_id) {
 
 			$lesson_ends = $lesson_grades = $lesson_answers = array();
 
 			// Pre-process the lesson ends
-			$_lesson_ends = $wpdb->get_results( $wpdb->prepare($end_sql, $user_id), ARRAY_A );
-			foreach ( $_lesson_ends as $lesson_end ) {
+			$_lesson_ends = $wpdb->get_results($wpdb->prepare($end_sql, $user_id), ARRAY_A);
+			foreach ($_lesson_ends as $lesson_end) {
 				// This will overwrite existing entries with the newer ones
-				$lesson_ends[ $lesson_end['comment_post_ID'] ] = $lesson_end['comment_date'];
+				$lesson_ends[$lesson_end['comment_post_ID']] = $lesson_end['comment_date'];
 			}
-			unset( $_lesson_ends );
+			unset($_lesson_ends);
 
 			// Pre-process the lesson grades
-			$_lesson_grades = $wpdb->get_results( $wpdb->prepare($grade_sql, $user_id), ARRAY_A );
-			foreach ( $_lesson_grades as $lesson_grade ) {
+			$_lesson_grades = $wpdb->get_results($wpdb->prepare($grade_sql, $user_id), ARRAY_A);
+			foreach ($_lesson_grades as $lesson_grade) {
 				// This will overwrite existing entries with the newer ones (assuming the grade is higher)
-				if ( empty($lesson_grades[ $lesson_grade['comment_post_ID'] ]) || $lesson_grades[ $lesson_grade['comment_post_ID'] ] < $lesson_grade['comment_content'] ) {
-					$lesson_grades[ $lesson_grade['comment_post_ID'] ] = $lesson_grade['comment_content'];
+				if (empty($lesson_grades[$lesson_grade['comment_post_ID']]) || $lesson_grades[$lesson_grade['comment_post_ID']] < $lesson_grade['comment_content']) {
+					$lesson_grades[$lesson_grade['comment_post_ID']] = $lesson_grade['comment_content'];
 				}
 			}
-			unset( $_lesson_grades );
+			unset($_lesson_grades);
 
 			// Pre-process the lesson answers
-			$_lesson_answers = $wpdb->get_results( $wpdb->prepare($answers_sql, $user_id), ARRAY_A );
-			foreach ( $_lesson_answers as $lesson_answer ) {
+			$_lesson_answers = $wpdb->get_results($wpdb->prepare($answers_sql, $user_id), ARRAY_A);
+			foreach ($_lesson_answers as $lesson_answer) {
 				// This will overwrite existing entries with the newer ones
-				$lesson_answers[ $lesson_answer['comment_post_ID'] ] = $lesson_answer['comment_content'];
+				$lesson_answers[$lesson_answer['comment_post_ID']] = $lesson_answer['comment_content'];
 			}
-			unset( $_lesson_answers );
+			unset($_lesson_answers);
 
 			// Grab all the lesson starts for the user
-			$lesson_starts = $wpdb->get_results( $wpdb->prepare($start_sql, $user_id), ARRAY_A );
-			foreach ( $lesson_starts as $lesson_log ) {
+			$lesson_starts = $wpdb->get_results($wpdb->prepare($start_sql, $user_id), ARRAY_A);
+			foreach ($lesson_starts as $lesson_log) {
 
 				$lesson_id = $lesson_log['comment_post_ID'];
 
@@ -1319,23 +1319,23 @@ 
 					'start' => $status_date,
 				);
 				// Check if there is a lesson end
-				if ( !empty($lesson_ends[$lesson_id]) ) {
+				if ( ! empty($lesson_ends[$lesson_id])) {
 					$status_date = $lesson_ends[$lesson_id];
 					// Check lesson has quiz
-					if ( !empty( $lesson_quiz_ids[$lesson_id] ) ) {
+					if ( ! empty($lesson_quiz_ids[$lesson_id])) {
 						// Check for the quiz answers
-						if ( !empty($lesson_answers[$quiz_id]) ) {
+						if ( ! empty($lesson_answers[$quiz_id])) {
 							$meta_data['questions_asked'] = $lesson_answers[$quiz_id];
 						}
 						// Check if there is a quiz grade
 						$quiz_id = $lesson_quiz_ids[$lesson_id];
-						if ( !empty($lesson_grades[$quiz_id]) ) {
+						if ( ! empty($lesson_grades[$quiz_id])) {
 							$meta_data['grade'] = $quiz_grade = $lesson_grades[$quiz_id];
 							// Check if the user has to get the passmark and has or not
-							if ( !empty( $quizzes_pass_required[$quiz_id] ) && $quizzes_passmarks[$quiz_id] <= $quiz_grade ) {
+							if ( ! empty($quizzes_pass_required[$quiz_id]) && $quizzes_passmarks[$quiz_id] <= $quiz_grade) {
 								$status = 'passed';
 							}
-							elseif ( !empty( $quizzes_pass_required[$quiz_id] ) && $quizzes_passmarks[$quiz_id] > $quiz_grade ) {
+							elseif ( ! empty($quizzes_pass_required[$quiz_id]) && $quizzes_passmarks[$quiz_id] > $quiz_grade) {
 								$status = 'failed';
 							}
 							else {
@@ -1363,26 +1363,26 @@ 
 						'comment_author' => '',
 					);
 				// Check it doesn't already exist
-				$sql = $wpdb->prepare( $check_existing_sql, $lesson_id, $user_id );
-				$comment_ID = $wpdb->get_var( $sql );
-				if ( !$comment_ID ) {
+				$sql = $wpdb->prepare($check_existing_sql, $lesson_id, $user_id);
+				$comment_ID = $wpdb->get_var($sql);
+				if ( ! $comment_ID) {
 					// Bypassing WP wp_insert_comment( $data ), so no actions/filters are run
 					$wpdb->insert($wpdb->comments, $data);
 					$comment_ID = (int) $wpdb->insert_id;
 
-					if ( $comment_ID && !empty($meta_data) ) {
-						foreach ( $meta_data as $key => $value ) {
+					if ($comment_ID && ! empty($meta_data)) {
+						foreach ($meta_data as $key => $value) {
 							// Bypassing WP add_comment_meta(() so no actions/filters are run
-							if ( $wpdb->get_var( $wpdb->prepare(
+							if ($wpdb->get_var($wpdb->prepare(
 									"SELECT COUNT(*) FROM $wpdb->commentmeta WHERE comment_id = %d AND meta_key = %s ",
-									$comment_ID, $key ) ) ) {
+									$comment_ID, $key ))) {
 									continue; // Found the meta data already
 							}
-							$result = $wpdb->insert( $wpdb->commentmeta, array(
+							$result = $wpdb->insert($wpdb->commentmeta, array(
 								'comment_id' => $comment_ID,
 								'meta_key' => $key,
 								'meta_value' => $value
-							) );
+							));
 						}
 					}
 				}
@@ -1390,40 +1390,40 @@ 
 		}
 		$wpdb->flush();
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
 		}
 	}
 
-	function status_changes_convert_courses( $n = 50, $offset = 0 ) {
+	function status_changes_convert_courses($n = 50, $offset = 0) {
 		global $wpdb;
 
-		wp_defer_comment_counting( true );
+		wp_defer_comment_counting(true);
 
-		$user_count_result = $wpdb->get_var( "SELECT COUNT(*) FROM $wpdb->users " );
+		$user_count_result = $wpdb->get_var("SELECT COUNT(*) FROM $wpdb->users ");
 
-		if ( 0 == $user_count_result ) {
+		if (0 == $user_count_result) {
 			return true;
 		}
 
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		}
 
-		$total_pages = ceil( $user_count_result / $n );
+		$total_pages = ceil($user_count_result / $n);
 
 		// Get all Lesson => Course relationships
-		$meta_list = $wpdb->get_results( "SELECT $wpdb->postmeta.post_id, $wpdb->postmeta.meta_value FROM $wpdb->postmeta INNER JOIN $wpdb->posts ON ($wpdb->posts.ID = $wpdb->postmeta.post_id) WHERE $wpdb->posts.post_type = 'lesson' AND $wpdb->postmeta.meta_key = '_lesson_course'", ARRAY_A );
+		$meta_list = $wpdb->get_results("SELECT $wpdb->postmeta.post_id, $wpdb->postmeta.meta_value FROM $wpdb->postmeta INNER JOIN $wpdb->posts ON ($wpdb->posts.ID = $wpdb->postmeta.post_id) WHERE $wpdb->posts.post_type = 'lesson' AND $wpdb->postmeta.meta_key = '_lesson_course'", ARRAY_A);
 		$course_lesson_ids = array();
-		if ( !empty($meta_list) ) {
-			foreach ( $meta_list as $metarow ) {
+		if ( ! empty($meta_list)) {
+			foreach ($meta_list as $metarow) {
 				$lesson_id = $metarow['post_id'];
 				$course_id = $metarow['meta_value'];
-				$course_lesson_ids[ $course_id ][] = $lesson_id;
+				$course_lesson_ids[$course_id][] = $lesson_id;
 			}
 		}
 
@@ -1433,13 +1433,13 @@ 
 		$check_existing_sql = "SELECT comment_ID FROM $wpdb->comments WHERE comment_post_ID = %d AND user_id = %d AND comment_type = 'sensei_course_status' ";
 
 		// $per_page users at a time, could be batch run via an admin ajax command, 1 user at a time?
-		$user_ids = $wpdb->get_col( $wpdb->prepare($users_sql, $n, $offset) );
+		$user_ids = $wpdb->get_col($wpdb->prepare($users_sql, $n, $offset));
 
-		foreach ( $user_ids AS $user_id ) {
+		foreach ($user_ids AS $user_id) {
 
 			// Grab all the course starts for the user
-			$course_starts = $wpdb->get_results( $wpdb->prepare($start_sql, $user_id), ARRAY_A );
-			foreach ( $course_starts as $course_log ) {
+			$course_starts = $wpdb->get_results($wpdb->prepare($start_sql, $user_id), ARRAY_A);
+			foreach ($course_starts as $course_log) {
 
 				$course_id = $course_log['comment_post_ID'];
 
@@ -1454,23 +1454,23 @@ 
 					'percent' => 0,
 				);
 				// Check if the course has lessons
-				if ( !empty( $course_lesson_ids[$course_id] ) ) {
+				if ( ! empty($course_lesson_ids[$course_id])) {
 
 					$lessons_completed = 0;
-					$total_lessons = count( $course_lesson_ids[ $course_id ] );
+					$total_lessons = count($course_lesson_ids[$course_id]);
 
 					// Don't use prepare as we need to provide the id join
-					$sql = sprintf($lessons_sql, $user_id, join(', ', $course_lesson_ids[ $course_id ]) );
+					$sql = sprintf($lessons_sql, $user_id, join(', ', $course_lesson_ids[$course_id]));
 					// Get all lesson statuses for this Courses' lessons
-					$lesson_statuses = $wpdb->get_results( $sql, ARRAY_A );
+					$lesson_statuses = $wpdb->get_results($sql, ARRAY_A);
 					// Not enough lesson statuses, thus cannot be complete
-					if ( $total_lessons > count($lesson_statuses) ) {
+					if ($total_lessons > count($lesson_statuses)) {
 						$status = 'in-progress';
 					}
 					// Count each lesson to work out the overall percentage
-					foreach ( $lesson_statuses as $lesson_status ) {
+					foreach ($lesson_statuses as $lesson_status) {
 						$status_date = $lesson_status['comment_date'];
-						switch ( $lesson_status['status'] ) {
+						switch ($lesson_status['status']) {
 							case 'complete': // Lesson has no quiz/questions
 							case 'graded': // Lesson has quiz, but it's not important what the grade was
 							case 'passed':
@@ -1485,7 +1485,7 @@ 
 						}
 					}
 					$meta_data['complete'] = $lessons_completed;
-					$meta_data['percent'] = abs( round( ( doubleval( $lessons_completed ) * 100 ) / ( $total_lessons ), 0 ) );
+					$meta_data['percent'] = abs(round((doubleval($lessons_completed) * 100) / ($total_lessons), 0));
 				}
 				else {
 					// Course has no lessons, therefore cannot be 'complete'
@@ -1502,26 +1502,26 @@ 
 						'comment_author' => '',
 					);
 				// Check it doesn't already exist
-				$sql = $wpdb->prepare( $check_existing_sql, $course_id, $user_id );
-				$comment_ID = $wpdb->get_var( $sql );
-				if ( !$comment_ID ) {
+				$sql = $wpdb->prepare($check_existing_sql, $course_id, $user_id);
+				$comment_ID = $wpdb->get_var($sql);
+				if ( ! $comment_ID) {
 					// Bypassing WP wp_insert_comment( $data ), so no actions/filters are run
 					$wpdb->insert($wpdb->comments, $data);
 					$comment_ID = (int) $wpdb->insert_id;
 
-					if ( $comment_ID && !empty($meta_data) ) {
-						foreach ( $meta_data as $key => $value ) {
+					if ($comment_ID && ! empty($meta_data)) {
+						foreach ($meta_data as $key => $value) {
 							// Bypassing WP wp_insert_comment( $data ), so no actions/filters are run
-							if ( $wpdb->get_var( $wpdb->prepare(
+							if ($wpdb->get_var($wpdb->prepare(
 									"SELECT COUNT(*) FROM $wpdb->commentmeta WHERE comment_id = %d AND meta_key = %s ",
-									$comment_ID, $key ) ) ) {
+									$comment_ID, $key ))) {
 									continue; // Found the meta data already
 							}
-							$result = $wpdb->insert( $wpdb->commentmeta, array(
+							$result = $wpdb->insert($wpdb->commentmeta, array(
 								'comment_id' => $comment_ID,
 								'meta_key' => $key,
 								'meta_value' => $value
-							) );
+							));
 						}
 					}
 				}
@@ -1529,7 +1529,7 @@ 
 		}
 		$wpdb->flush();
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
@@ -1545,70 +1545,70 @@ 
 	 * @param type $offset
 	 * @return boolean
 	 */
-	function status_changes_repair_course_statuses( $n = 50, $offset = 0 ) {
+	function status_changes_repair_course_statuses($n = 50, $offset = 0) {
 		global $wpdb;
 
-		$count_object = wp_count_posts( 'lesson' );
+		$count_object = wp_count_posts('lesson');
 		$count_published = $count_object->publish;
 
-		if ( 0 == $count_published ) {
+		if (0 == $count_published) {
 			return true;
 		}
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		}
-		$total_pages = ceil( $count_published / $n );
+		$total_pages = ceil($count_published / $n);
 
 		$course_lesson_ids = $lesson_user_statuses = array();
 
 		// Get all Lesson => Course relationships
-		$meta_list = $wpdb->get_results( "SELECT $wpdb->postmeta.post_id, $wpdb->postmeta.meta_value FROM $wpdb->postmeta INNER JOIN $wpdb->posts ON ($wpdb->posts.ID = $wpdb->postmeta.post_id) WHERE $wpdb->posts.post_type = 'lesson' AND $wpdb->postmeta.meta_key = '_lesson_course' LIMIT $n OFFSET $offset ", ARRAY_A );
-		if ( !empty($meta_list) ) {
-			foreach ( $meta_list as $metarow ) {
+		$meta_list = $wpdb->get_results("SELECT $wpdb->postmeta.post_id, $wpdb->postmeta.meta_value FROM $wpdb->postmeta INNER JOIN $wpdb->posts ON ($wpdb->posts.ID = $wpdb->postmeta.post_id) WHERE $wpdb->posts.post_type = 'lesson' AND $wpdb->postmeta.meta_key = '_lesson_course' LIMIT $n OFFSET $offset ", ARRAY_A);
+		if ( ! empty($meta_list)) {
+			foreach ($meta_list as $metarow) {
 				$lesson_id = $metarow['post_id'];
 				$course_id = $metarow['meta_value'];
-				$course_lesson_ids[ $course_id ][] = $lesson_id;
+				$course_lesson_ids[$course_id][] = $lesson_id;
 			}
 		}
 
 		// Get all Lesson => Course relationships
-		$status_list = $wpdb->get_results( "SELECT user_id, comment_post_ID, comment_approved FROM $wpdb->comments WHERE comment_type = 'sensei_lesson_status' GROUP BY user_id, comment_post_ID ", ARRAY_A );
-		if ( !empty($status_list) ) {
-			foreach ( $status_list as $status ) {
-				$lesson_user_statuses[ $status['comment_post_ID'] ][ $status['user_id'] ] = $status['comment_approved'];
+		$status_list = $wpdb->get_results("SELECT user_id, comment_post_ID, comment_approved FROM $wpdb->comments WHERE comment_type = 'sensei_lesson_status' GROUP BY user_id, comment_post_ID ", ARRAY_A);
+		if ( ! empty($status_list)) {
+			foreach ($status_list as $status) {
+				$lesson_user_statuses[$status['comment_post_ID']][$status['user_id']] = $status['comment_approved'];
 			}
 		}
 
-		$course_completion = Sensei()->settings->settings[ 'course_completion' ];
+		$course_completion = Sensei()->settings->settings['course_completion'];
 
 		$per_page = 40;
 		$comment_id_offset = $count = 0;
 
 		$course_sql = "SELECT * FROM $wpdb->comments WHERE comment_type = 'sensei_course_status' AND comment_ID > %d LIMIT $per_page";
 		// $per_page users at a time
-		while ( $course_statuses = $wpdb->get_results( $wpdb->prepare($course_sql, $comment_id_offset) ) ) {
+		while ($course_statuses = $wpdb->get_results($wpdb->prepare($course_sql, $comment_id_offset))) {
 
-			foreach ( $course_statuses AS $course_status ) {
+			foreach ($course_statuses AS $course_status) {
 				$user_id = $course_status->user_id;
 				$course_id = $course_status->comment_post_ID;
-				$total_lessons = count( $course_lesson_ids[ $course_id ] );
-				if ( $total_lessons <= 0 ) {
+				$total_lessons = count($course_lesson_ids[$course_id]);
+				if ($total_lessons <= 0) {
 					$total_lessons = 1; // Fix division of zero error, some courses have no lessons
 				}
 				$lessons_completed = 0;
 				$status = 'in-progress';
 
 				// Some Courses have no lessons... (can they ever be complete?)
-				if ( !empty($course_lesson_ids[ $course_id ]) ) {
-					foreach( $course_lesson_ids[ $course_id ] AS $lesson_id ) {
-						$lesson_status = $lesson_user_statuses[ $lesson_id ][ $user_id ];
+				if ( ! empty($course_lesson_ids[$course_id])) {
+					foreach ($course_lesson_ids[$course_id] AS $lesson_id) {
+						$lesson_status = $lesson_user_statuses[$lesson_id][$user_id];
 						// If lessons are complete without needing quizzes to be passed
-						if ( 'passed' != $course_completion ) {
-							switch ( $lesson_status ) {
+						if ('passed' != $course_completion) {
+							switch ($lesson_status) {
 								// A user cannot 'complete' a course if a lesson...
 								case 'in-progress': // ...is still in progress
 								case 'ungraded': // ...hasn't yet been graded
@@ -1620,7 +1620,7 @@ 
 							}
 						}
 						else {
-							switch ( $lesson_status ) {
+							switch ($lesson_status) {
 								case 'complete': // Lesson has no quiz/questions
 								case 'graded': // Lesson has quiz, but it's not important what the grade was
 								case 'passed': // Lesson has quiz and the user passed
@@ -1635,79 +1635,79 @@ 
 						}
 					} // Each lesson
 				} // Check for lessons
-				if ( $lessons_completed == $total_lessons ) {
+				if ($lessons_completed == $total_lessons) {
 					$status = 'complete';
 				}
 				// update the overall percentage of the course lessons complete (or graded) compared to 'in-progress' regardless of the above
 				$metadata = array(
 					'complete' => $lessons_completed,
-					'percent' => abs( round( ( doubleval( $lessons_completed ) * 100 ) / ( $total_lessons ), 0 ) ),
+					'percent' => abs(round((doubleval($lessons_completed) * 100) / ($total_lessons), 0)),
 				);
-				Sensei_Utils::update_course_status( $user_id, $course_id, $status, $metadata );
+				Sensei_Utils::update_course_status($user_id, $course_id, $status, $metadata);
 				$count++;
 
 			} // per course status
 			$comment_id_offset = $course_status->comment_ID;
 		} // all course statuses
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
 		}
 	}
 
-	function status_changes_convert_questions( $n = 50, $offset = 0 ) {
+	function status_changes_convert_questions($n = 50, $offset = 0) {
 		global $wpdb;
 
-		wp_defer_comment_counting( true );
+		wp_defer_comment_counting(true);
 
-		$user_count_result = $wpdb->get_var( "SELECT COUNT(*) FROM $wpdb->users " );
+		$user_count_result = $wpdb->get_var("SELECT COUNT(*) FROM $wpdb->users ");
 
-		if ( 0 == $user_count_result ) {
+		if (0 == $user_count_result) {
 			return true;
 		}
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		}
 
-		$total_pages = ceil( $user_count_result / $n );
+		$total_pages = ceil($user_count_result / $n);
 
 		$users_sql = "SELECT ID FROM $wpdb->users ORDER BY ID ASC LIMIT %d OFFSET %d";
 		$answers_sql = "SELECT * FROM $wpdb->comments WHERE comment_type = 'sensei_user_answer' AND user_id = %d GROUP BY comment_post_ID ";
 		$grades_sql = "SELECT comment_post_ID, comment_content FROM $wpdb->comments WHERE comment_type = 'sensei_user_grade' AND user_id = %d GROUP BY comment_post_ID ";
 		$notes_sql = "SELECT comment_post_ID, comment_content FROM $wpdb->comments WHERE comment_type = 'sensei_answer_notes' AND user_id = %d GROUP BY comment_post_ID ";
 
-		$user_ids = $wpdb->get_col( $wpdb->prepare($users_sql, $n, $offset) );
+		$user_ids = $wpdb->get_col($wpdb->prepare($users_sql, $n, $offset));
 
-		foreach ( $user_ids AS $user_id ) {
+		foreach ($user_ids AS $user_id) {
 
 			$answer_grades = $answer_notes = array();
 
 			// Pre-process the answer grades
-			$_answer_grades = $wpdb->get_results( $wpdb->prepare($grades_sql, $user_id), ARRAY_A );
-			foreach ( $_answer_grades as $answer_grade ) {
+			$_answer_grades = $wpdb->get_results($wpdb->prepare($grades_sql, $user_id), ARRAY_A);
+			foreach ($_answer_grades as $answer_grade) {
 				// This will overwrite existing entries with the newer ones
-				$answer_grades[ $answer_grade['comment_post_ID'] ] = $answer_grade['comment_content'];
+				$answer_grades[$answer_grade['comment_post_ID']] = $answer_grade['comment_content'];
 			}
-			unset( $_answer_grades );
+			unset($_answer_grades);
 
 			// Pre-process the answer notes
-			$_answer_notes = $wpdb->get_results( $wpdb->prepare($notes_sql, $user_id), ARRAY_A );
-			foreach ( $_answer_notes as $answer_note ) {
+			$_answer_notes = $wpdb->get_results($wpdb->prepare($notes_sql, $user_id), ARRAY_A);
+			foreach ($_answer_notes as $answer_note) {
 				// This will overwrite existing entries with the newer ones
-				$answer_notes[ $answer_note['comment_post_ID'] ] = $answer_note['comment_content'];
+				$answer_notes[$answer_note['comment_post_ID']] = $answer_note['comment_content'];
 			}
-			unset( $_answer_notes );
+			unset($_answer_notes);
 
 			// Grab all the questions for the user
 			$sql = $wpdb->prepare($answers_sql, $user_id);
-			$answers = $wpdb->get_results( $sql, ARRAY_A );
-			foreach ( $answers as $answer ) {
+			$answers = $wpdb->get_results($sql, ARRAY_A);
+			foreach ($answers as $answer) {
 
 				// Excape data
 				$answer = wp_slash($answer);
@@ -1717,12 +1717,12 @@ 
 				$meta_data = array();
 
 				// Check if the question has been graded, add as meta
-				if ( !empty($answer_grades[ $answer['comment_post_ID'] ]) ) {
-					$meta_data['user_grade'] = $answer_grades[ $answer['comment_post_ID'] ];
+				if ( ! empty($answer_grades[$answer['comment_post_ID']])) {
+					$meta_data['user_grade'] = $answer_grades[$answer['comment_post_ID']];
 				}
 				// Check if there is an answer note, add as meta
-				if ( !empty($answer_notes[ $answer['comment_post_ID'] ]) ) {
-					$meta_data['answer_note'] = $answer_notes[ $answer['comment_post_ID'] ];
+				if ( ! empty($answer_notes[$answer['comment_post_ID']])) {
+					$meta_data['answer_note'] = $answer_notes[$answer['comment_post_ID']];
 				}
 
 				// Wipe the unnessary data from the main comment
@@ -1735,21 +1735,21 @@ 
 					);
 				$data = array_merge($answer, $data);
 
-				$rval = $wpdb->update( $wpdb->comments, $data, compact( 'comment_ID' ) );
-				if ( $rval ) {
-					if ( !empty($meta_data) ) {
-						foreach ( $meta_data as $key => $value ) {
+				$rval = $wpdb->update($wpdb->comments, $data, compact('comment_ID'));
+				if ($rval) {
+					if ( ! empty($meta_data)) {
+						foreach ($meta_data as $key => $value) {
 							// Bypassing WP wp_insert_comment( $data ), so no actions/filters are run
-							if ( $wpdb->get_var( $wpdb->prepare(
+							if ($wpdb->get_var($wpdb->prepare(
 									"SELECT COUNT(*) FROM $wpdb->commentmeta WHERE comment_id = %d AND meta_key = %s ",
-									$comment_ID, $key ) ) ) {
+									$comment_ID, $key ))) {
 									continue; // Found the meta data already
 							}
-							$result = $wpdb->insert( $wpdb->commentmeta, array(
+							$result = $wpdb->insert($wpdb->commentmeta, array(
 								'comment_id' => $comment_ID,
 								'meta_key' => $key,
 								'meta_value' => $value
-							) );
+							));
 						}
 					}
 				}
@@ -1757,7 +1757,7 @@ 
 		}
 		$wpdb->flush();
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
@@ -1774,10 +1774,10 @@ 
 		global $wpdb;
 
 		// Update 'sensei_user_answer' entries to use comment_approved = 'log' so they don't appear in counts
-		$wpdb->query( "UPDATE $wpdb->comments SET comment_approved = 'log' WHERE comment_type = 'sensei_user_answer' " );
+		$wpdb->query("UPDATE $wpdb->comments SET comment_approved = 'log' WHERE comment_type = 'sensei_user_answer' ");
 
 		// Mark all old Sensei comment types with comment_approved = 'legacy' so they no longer appear in counts, but can be restored if required
-		$wpdb->query( "UPDATE $wpdb->comments SET comment_approved = 'legacy' WHERE comment_type IN ('sensei_course_start', 'sensei_course_end', 'sensei_lesson_start', 'sensei_lesson_end', 'sensei_quiz_asked', 'sensei_user_grade', 'sensei_answer_notes', 'sensei_quiz_grade') " );
+		$wpdb->query("UPDATE $wpdb->comments SET comment_approved = 'legacy' WHERE comment_type IN ('sensei_course_start', 'sensei_course_end', 'sensei_lesson_start', 'sensei_lesson_end', 'sensei_quiz_asked', 'sensei_user_grade', 'sensei_answer_notes', 'sensei_quiz_grade') ");
 
 		return true;
 	}
@@ -1790,50 +1790,50 @@ 
 	 * @param type $offset
 	 * @return boolean
 	 */
-	public function update_comment_course_lesson_comment_counts( $n = 50, $offset = 0 ) {
+	public function update_comment_course_lesson_comment_counts($n = 50, $offset = 0) {
 		global $wpdb;
 
-		$item_count_result = $wpdb->get_var( "SELECT COUNT(*) FROM $wpdb->posts WHERE post_type IN ('course', 'lesson') " );
+		$item_count_result = $wpdb->get_var("SELECT COUNT(*) FROM $wpdb->posts WHERE post_type IN ('course', 'lesson') ");
 
-		if ( 0 == $item_count_result ) {
+		if (0 == $item_count_result) {
 			return true;
 		}
 
 		// Calculate if this is the last page
-		if ( 0 == $offset ) {
+		if (0 == $offset) {
 			$current_page = 1;
 		} else {
-			$current_page = intval( $offset / $n );
+			$current_page = intval($offset / $n);
 		}
 
-		$total_pages = ceil( $item_count_result / $n );
+		$total_pages = ceil($item_count_result / $n);
 
 		// Recalculate all counts
-		$items = $wpdb->get_results( $wpdb->prepare( "SELECT ID FROM $wpdb->posts WHERE post_type IN ('course', 'lesson') LIMIT %d OFFSET %d", $n, $offset ) );
-		foreach ( (array) $items as $post ) {
+		$items = $wpdb->get_results($wpdb->prepare("SELECT ID FROM $wpdb->posts WHERE post_type IN ('course', 'lesson') LIMIT %d OFFSET %d", $n, $offset));
+		foreach ((array) $items as $post) {
 			// Code copied from wp_update_comment_count_now()
-			$new = (int) $wpdb->get_var( $wpdb->prepare("SELECT COUNT(*) FROM $wpdb->comments WHERE comment_post_ID = %d AND comment_approved = '1'", $post->ID) );
-			$wpdb->update( $wpdb->posts, array('comment_count' => $new), array('ID' => $post->ID) );
+			$new = (int) $wpdb->get_var($wpdb->prepare("SELECT COUNT(*) FROM $wpdb->comments WHERE comment_post_ID = %d AND comment_approved = '1'", $post->ID));
+			$wpdb->update($wpdb->posts, array('comment_count' => $new), array('ID' => $post->ID));
 
-			clean_post_cache( $post->ID );
+			clean_post_cache($post->ID);
 		}
 
-		if ( $current_page == $total_pages ) {
+		if ($current_page == $total_pages) {
 			return true;
 		} else {
 			return false;
 		}
 	}
 
-	public function remove_legacy_comments () {
+	public function remove_legacy_comments() {
 		global $wpdb;
 
-		$result = $wpdb->delete( $wpdb->comments, array( 'comment_approved' => 'legacy' ) );
+		$result = $wpdb->delete($wpdb->comments, array('comment_approved' => 'legacy'));
 
 		return true;
 	}
 
-	public function index_comment_status_field () {
+	public function index_comment_status_field() {
 		global $wpdb;
 
 		$wpdb->query("ALTER TABLE `$wpdb->comments` ADD INDEX `comment_type` ( `comment_type` )");
@@ -1852,7 +1852,7 @@ 
      * @since 1.8.0
      * @return bool;
      */
-    public  function enhance_teacher_role ( ) {
+    public  function enhance_teacher_role( ) {
 
         require_once('class-sensei-teacher.php');
         $teacher = new Sensei_Teacher();

templates/single-course/lessons.php

Spacing +7 added lines, -7 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 /**
  * The Template for displaying all single course meta information.
  *
@@ -22,17 +22,17 @@ 
          * @hooked WooThemes_Sensei_Course::load_single_course_lessons_query
          * @since 1.9.0
          */
-        do_action( 'sensei_single_course_lessons_before' );
+        do_action('sensei_single_course_lessons_before');
 
     ?>
 
     <?php
 
     //lessons loaded into loop in the sensei_single_course_lessons_before hook
-    if( have_posts() ):
+    if (have_posts()):
 
         // start course lessons loop
-        while ( have_posts() ): the_post();  ?>
+        while (have_posts()): the_post(); ?>
 
             <article <?php post_class(); ?> >
 
@@ -50,7 +50,7 @@ 
                      * @hooked WooThemes_Sensei_Lesson::the_lesson_thumbnail - 8
                      *
                      */
-                    do_action( 'sensei_single_course_inside_before_lesson', get_the_ID() );
+                    do_action('sensei_single_course_inside_before_lesson', get_the_ID());
 
                 ?>
 
@@ -75,7 +75,7 @@ 
                      *
                      * @since 1.9.0
                      */
-                    do_action( 'sensei_single_course_inside_after_lesson', get_the_ID() );
+                    do_action('sensei_single_course_inside_after_lesson', get_the_ID());
 
                 ?>
 
@@ -94,7 +94,7 @@ 
          *
          * @since 1.9.0
          */
-        do_action( 'sensei_single_course_lessons_after' );
+        do_action('sensei_single_course_lessons_after');
 
     ?>
 

templates/course-results.php

Spacing +11 added lines, -11 removed lines

@@ -11,7 +11,7 @@ 
  */
 ?>
 
-<?php  get_sensei_header();  ?>
+<?php  get_sensei_header(); ?>
 
 <?php
 /**
@@ -21,15 +21,15 @@ 
  *
  * @hooked Sensei_Course_Results::deprecate_sensei_course_results_content_hook() - 20
  */
-do_action( 'sensei_course_results_content_before' );
+do_action('sensei_course_results_content_before');
 ?>
 
 <?php
 global $course;
-$course = get_page_by_path( $wp_query->query_vars['course_results'], OBJECT, 'course' );
+$course = get_page_by_path($wp_query->query_vars['course_results'], OBJECT, 'course');
 ?>
 
-<article <?php post_class( array( 'course', 'post','course-results' ) ); ?> >
+<article <?php post_class(array('course', 'post', 'course-results')); ?> >
 
     <section class="entry fix">
 
@@ -42,7 +42,7 @@ 
          * @param integer $course_id
          *
          */
-        do_action( 'sensei_course_results_content_inside_before', $course->ID );
+        do_action('sensei_course_results_content_inside_before', $course->ID);
         ?>
 
         <header>
@@ -53,7 +53,7 @@ 
 
         </header>
 
-        <?php if ( is_user_logged_in() ):?>
+        <?php if (is_user_logged_in()):?>
 
             <?php
             /**
@@ -65,14 +65,14 @@ 
              *
              * @hooked Sensei_Course_Results::course_info() - 20
              */
-            do_action( 'sensei_course_results_content_inside_before_lessons', $course->ID );
+            do_action('sensei_course_results_content_inside_before_lessons', $course->ID);
             ?>
 
 
             <section class="course-results-lessons">
                 <?php
-                $started_course = Sensei_Utils::user_started_course( $course->ID, get_current_user_id() );
-                if( $started_course ) {
+                $started_course = Sensei_Utils::user_started_course($course->ID, get_current_user_id());
+                if ($started_course) {
 
                     sensei_the_course_results_lessons();
 
@@ -92,7 +92,7 @@ 
          *
          * @hooked Sensei()->course_results->course_info - 20
          */
-        do_action( 'sensei_course_results_content_inside_after', $course->ID );
+        do_action('sensei_course_results_content_inside_after', $course->ID);
         ?>
 
     </section>
@@ -106,7 +106,7 @@ 
  * @since 1.9.0
  *
  */
-do_action( 'sensei_course_results_content_after' );
+do_action('sensei_course_results_content_after');
 ?>
 
 

includes/shortcodes/class-sensei-shortcode-user-courses.php

Spacing +61 added lines, -61 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 /**
  * This class is loaded int WP by the shortcode loader class.
  *
@@ -61,28 +61,28 @@ 
      * @param string $content
      * @param string $shortcode the shortcode that was called for this instance
      */
-    public function __construct( $attributes, $content, $shortcode ){
+    public function __construct($attributes, $content, $shortcode) {
 
-        if(!  is_user_logged_in() ) {
+        if ( ! is_user_logged_in()) {
             // show the login form
             Sensei_Templates::get_template('user/login-form.php');
             return;
         }
 
         // set up all argument need for constructing the course query
-        $this->number = isset( $attributes['number'] ) ? $attributes['number'] : '10';
-        $this->orderby = isset( $attributes['orderby'] ) ? $attributes['orderby'] : 'title';
-        $this->status = isset( $attributes['status'] ) ? $attributes['status'] : 'all';
+        $this->number = isset($attributes['number']) ? $attributes['number'] : '10';
+        $this->orderby = isset($attributes['orderby']) ? $attributes['orderby'] : 'title';
+        $this->status = isset($attributes['status']) ? $attributes['status'] : 'all';
 
         // set the default for menu_order to be ASC
-        if( 'menu_order' == $this->orderby && !isset( $attributes['order']  ) ){
+        if ('menu_order' == $this->orderby && ! isset($attributes['order'])) {
 
-            $this->order =  'ASC';
+            $this->order = 'ASC';
 
-        }else{
+        } else {
 
             // for everything else use the value passed or the default DESC
-            $this->order = isset( $attributes['order']  ) ? $attributes['order'] : 'ASC';
+            $this->order = isset($attributes['order']) ? $attributes['order'] : 'ASC';
 
         }
 
@@ -96,20 +96,20 @@ 
      *
      * @since 1.9.0
      */
-    protected function setup_course_query(){
+    protected function setup_course_query() {
 
-        $status_query = array( 'user_id' => get_current_user_id(), 'type' => 'sensei_course_status' );
-        $user_courses_logs = Sensei_Utils::sensei_check_for_activity( $status_query , true );
-        if ( !is_array($user_courses_logs) ) {
+        $status_query = array('user_id' => get_current_user_id(), 'type' => 'sensei_course_status');
+        $user_courses_logs = Sensei_Utils::sensei_check_for_activity($status_query, true);
+        if ( ! is_array($user_courses_logs)) {
 
-            $user_courses_logs = array( $user_courses_logs );
+            $user_courses_logs = array($user_courses_logs);
 
         }
 
         $completed_ids = $active_ids = array();
-        foreach( $user_courses_logs as $course_status ) {
+        foreach ($user_courses_logs as $course_status) {
 
-            if ( Sensei_Utils::user_completed_course( $course_status, get_current_user_id() ) ) {
+            if (Sensei_Utils::user_completed_course($course_status, get_current_user_id())) {
 
                 $completed_ids[] = $course_status->comment_post_ID;
 
@@ -120,34 +120,34 @@ 
             }
         }
 
-        if( 'completed' == $this->status ){
+        if ('completed' == $this->status) {
 
-            $included_courses =  $completed_ids;
+            $included_courses = $completed_ids;
 
 
-        }elseif( 'active'==$this->status ){
+        }elseif ('active' == $this->status) {
 
-            $included_courses =  $active_ids;
+            $included_courses = $active_ids;
 
-        }else{ // all courses
+        } else { // all courses
 
-            if( empty( $completed_ids ) ){
+            if (empty($completed_ids)) {
 
-                add_action( 'sensei_loop_course_inside_before', array( $this, 'completed_no_course_message_output' ) );
+                add_action('sensei_loop_course_inside_before', array($this, 'completed_no_course_message_output'));
             }
 
-            if( empty( $active_ids ) ){
+            if (empty($active_ids)) {
 
-                add_action( 'sensei_loop_course_inside_before', array( $this, 'active_no_course_message_output' ) );
+                add_action('sensei_loop_course_inside_before', array($this, 'active_no_course_message_output'));
 
             }
 
-            if( empty( $completed_ids ) &&  empty( $active_ids )  ){
+            if (empty($completed_ids) && empty($active_ids)) {
 
                 $included_courses = array('-1000'); // don't show any courses
 
-            }else{
-                $included_courses = Sensei_Utils::array_zip_merge( (array)$active_ids, (array)$completed_ids );
+            } else {
+                $included_courses = Sensei_Utils::array_zip_merge((array) $active_ids, (array) $completed_ids);
             }
 
 
@@ -158,7 +158,7 @@ 
         // if the shortcode is not active or in active and the active and completed
         // tabs show up.
         $number_of_posts = $this->number;
-        if( 'active' != $this->status && 'complete' != $this->status  ){
+        if ('active' != $this->status && 'complete' != $this->status) {
             $number_of_posts = 1000;
         }
 
@@ -169,12 +169,12 @@ 
             'post_status'      => 'publish',
             'orderby'          => $this->orderby,
             'order'            => $this->order,
-            'paged' => empty( $query_var_paged )? 1 : $query_var_paged,
+            'paged' => empty($query_var_paged) ? 1 : $query_var_paged,
             'posts_per_page'   => $number_of_posts,
             'post__in'         => $included_courses,
         );
 
-        $this->query = new WP_Query( $query_args );
+        $this->query = new WP_Query($query_args);
 
     }// end setup _course_query
 
@@ -184,12 +184,12 @@ 
      *
      * @since 1.9.0
      */
-    public function completed_no_course_message_output(){
+    public function completed_no_course_message_output() {
         ?>
         <li class="user-completed">
             <div class="sensei-message info">
 
-                <?php _e( 'You have not completed any courses yet.', 'woothemes-sensei' ); ?>
+                <?php _e('You have not completed any courses yet.', 'woothemes-sensei'); ?>
 
             </div>
         </li>
@@ -202,17 +202,17 @@ 
      *
      * @since 1.9.0
      */
-    public function active_no_course_message_output(){
+    public function active_no_course_message_output() {
         ?>
 
         <li class="user-active">
             <div class="sensei-message info">
 
-                <?php _e( 'You have no active courses.', 'woothemes-sensei' ); ?>
+                <?php _e('You have no active courses.', 'woothemes-sensei'); ?>
 
-                <a href="<?php esc_attr_e( Sensei_Course::get_courses_page_url() ); ?>">
+                <a href="<?php esc_attr_e(Sensei_Course::get_courses_page_url()); ?>">
 
-                    <?php  _e( 'Start a Course!', 'woothemes-sensei' ); ?>
+                    <?php  _e('Start a Course!', 'woothemes-sensei'); ?>
 
                 </a>
 
@@ -226,11 +226,11 @@ 
      *
      * @return string $content
      */
-    public function render(){
+    public function render() {
 
         global $wp_query;
 
-        if(!  is_user_logged_in() ) {
+        if ( ! is_user_logged_in()) {
             return '';
         }
 
@@ -249,7 +249,7 @@ 
         Sensei_Templates::get_template('globals/pagination.php');
         echo '</section>';
 
-        $shortcode_output =  ob_get_clean();
+        $shortcode_output = ob_get_clean();
 
         $this->detach_shortcode_hooks();
 
@@ -265,23 +265,23 @@ 
      *
      * @since 1.9.0
      */
-    public function attach_shortcode_hooks(){
+    public function attach_shortcode_hooks() {
 
         // attach the toggle functionality
         // don't show the toggle action if the user specified complete or active for this shortcode
-        if( ! in_array( $this->status, array( 'active', 'complete' ) ) ){
+        if ( ! in_array($this->status, array('active', 'complete'))) {
 
-            add_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
-            add_action( 'wp_footer', array( $this, 'print_course_toggle_actions_inline_script' ), 90 );
+            add_action('sensei_loop_course_before', array($this, 'course_toggle_actions'));
+            add_action('wp_footer', array($this, 'print_course_toggle_actions_inline_script'), 90);
 
         }
 
         // add extra classes to distinguish the course based on user completed or active
-        add_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
+        add_filter('sensei_course_loop_content_class', array($this, 'course_status_class_tagging'), 20, 2);
 
         // attach progress meter below course
-        add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
-        add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
+        add_action('sensei_course_content_inside_after', array($this, 'attach_course_progress'));
+        add_action('sensei_course_content_inside_after', array($this, 'attach_course_buttons'));
 
     }
 
@@ -290,13 +290,13 @@ 
      *
      * @since 1.9.0
      */
-    public function detach_shortcode_hooks(){
+    public function detach_shortcode_hooks() {
 
         //remove all hooks after the output is generated
-        remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
-        remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
-        remove_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
-        remove_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
+        remove_action('sensei_course_content_inside_after', array($this, 'attach_course_progress'));
+        remove_action('sensei_course_content_inside_after', array($this, 'attach_course_buttons'));
+        remove_filter('sensei_course_loop_content_class', array($this, 'course_status_class_tagging'), 20, 2);
+        remove_action('sensei_loop_course_before', array($this, 'course_toggle_actions'));
     }
 
     /**
@@ -304,10 +304,10 @@ 
      *
      * @param $course
      */
-    public function attach_course_progress( $course_id ){
+    public function attach_course_progress($course_id) {
 
-        $percentage = Sensei()->course->get_completion_percentage( $course_id, get_current_user_id() );
-        echo Sensei()->course->get_progress_meter( $percentage );
+        $percentage = Sensei()->course->get_completion_percentage($course_id, get_current_user_id());
+        echo Sensei()->course->get_progress_meter($percentage);
 
     }// attach_course_progress
 
@@ -319,9 +319,9 @@ 
      *
      * @param integer $course_id
      */
-    public function attach_course_buttons( $course_id ){
+    public function attach_course_buttons($course_id) {
 
-        Sensei()->course->the_course_action_buttons( get_post( $course_id ) );
+        Sensei()->course->the_course_action_buttons(get_post($course_id));
 
     }// attach_course_buttons
 
@@ -334,9 +334,9 @@ 
      * @param WP_Post $course
      * @return array $classes
      */
-    public function course_status_class_tagging($classes, $course){
+    public function course_status_class_tagging($classes, $course) {
 
-        if ( Sensei_Utils::user_completed_course( $course, get_current_user_id() ) ) {
+        if (Sensei_Utils::user_completed_course($course, get_current_user_id())) {
 
             $classes[] = 'user-completed';
 
@@ -353,7 +353,7 @@ 
     /**
      * Output the course toggle functionality
      */
-    public function course_toggle_actions(){ ?>
+    public function course_toggle_actions() { ?>
 
         <section id="user-course-status-toggle">
             <a id="sensei-user-courses-active-action" href=""><?php _e('Active Courses'); ?></a>

includes/shortcodes/class-sensei-legacy-shortcodes.php

Spacing +79 added lines, -79 removed lines

@@ -22,14 +22,14 @@ 
      *
      * @since 1.9.0
      */
-    public static function init(){
+    public static function init() {
 
-        add_shortcode( 'allcourses',      array( __CLASS__, 'all_courses' ) );
-        add_shortcode( 'newcourses',      array( __CLASS__,'new_courses' ) );
-        add_shortcode( 'featuredcourses', array( __CLASS__,'featured_courses') );
-        add_shortcode( 'freecourses',     array( __CLASS__,'free_courses') );
-        add_shortcode( 'paidcourses',     array( __CLASS__,'paid_courses') );
-        add_shortcode( 'usercourses',     array( __CLASS__,'user_courses' ) );
+        add_shortcode('allcourses', array(__CLASS__, 'all_courses'));
+        add_shortcode('newcourses', array(__CLASS__, 'new_courses'));
+        add_shortcode('featuredcourses', array(__CLASS__, 'featured_courses'));
+        add_shortcode('freecourses', array(__CLASS__, 'free_courses'));
+        add_shortcode('paidcourses', array(__CLASS__, 'paid_courses'));
+        add_shortcode('usercourses', array(__CLASS__, 'user_courses'));
 
     }
     /**
@@ -42,9 +42,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function all_courses( $atts, $content = null ) {
+    public static function all_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( '', 'allcourses' ); // all courses but no title
+        return self::generate_shortcode_courses('', 'allcourses'); // all courses but no title
 
     } // all_courses()
 
@@ -56,9 +56,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function paid_courses( $atts, $content = null ) {
+    public static function paid_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'Paid Courses', 'paidcourses' );
+        return self::generate_shortcode_courses('Paid Courses', 'paidcourses');
 
     } // End paid_courses()
 
@@ -71,9 +71,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function featured_courses( $atts, $content = null ) {
+    public static function featured_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'Featured Courses', 'featuredcourses' );
+        return self::generate_shortcode_courses('Featured Courses', 'featuredcourses');
 
     } // End featured_courses()
 
@@ -85,9 +85,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function free_courses( $atts, $content = null ) {
+    public static function free_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'Free Courses', 'freecourses' );
+        return self::generate_shortcode_courses('Free Courses', 'freecourses');
 
     } // End free_courses()
 
@@ -99,9 +99,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function new_courses( $atts, $content = null ) {
+    public static function new_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'New Courses', 'newcourses' );
+        return self::generate_shortcode_courses('New Courses', 'newcourses');
 
     } // End new_courses()
 
@@ -114,7 +114,7 @@ 
      * @param $shortcode_specific_override
      * @return string
      */
-    public static function generate_shortcode_courses( $title , $shortcode_specific_override  ){
+    public static function generate_shortcode_courses($title, $shortcode_specific_override) {
 
         global  $shortcode_override, $posts_array;
 
@@ -122,9 +122,9 @@ 
 
         // do not show this short code if there is a shortcode int he url and
         // this specific shortcode is not the one requested in the ur.
-        $specific_shortcode_requested = isset( $_GET['action'] ) ?  sanitize_text_field(  $_GET['action']  ) : '';
-        if( ! empty( $specific_shortcode_requested) &&
-            $specific_shortcode_requested != $shortcode_override ){
+        $specific_shortcode_requested = isset($_GET['action']) ? sanitize_text_field($_GET['action']) : '';
+        if ( ! empty($specific_shortcode_requested) &&
+            $specific_shortcode_requested != $shortcode_override) {
 
             return '';
 
@@ -136,15 +136,15 @@ 
         $courses = ob_get_clean();
 
         $content = '';
-        if( count( $posts_array ) > 0 ){
+        if (count($posts_array) > 0) {
 
-            $before = empty($title)? '' : '<header class="archive-header"><h2>'. $title .'</h2></header>';
+            $before = empty($title) ? '' : '<header class="archive-header"><h2>'.$title.'</h2></header>';
             $before .= '<section id="main-course" class="course-container">';
 
             $after = '</section>';
 
             //assemble
-            $content = $before . $courses . $after;
+            $content = $before.$courses.$after;
 
         }
 
@@ -161,19 +161,19 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function user_courses( $atts, $content = null ) {
+    public static function user_courses($atts, $content = null) {
         global $shortcode_override;
-        extract( shortcode_atts( array(	'amount' => 0 ), $atts ) );
+        extract(shortcode_atts(array('amount' => 0), $atts));
 
         $shortcode_override = 'usercourses';
 
         ob_start();
 
-        if( is_user_logged_in() ){
+        if (is_user_logged_in()) {
 
-            Sensei_Templates::get_template( 'user/my-courses.php' );
+            Sensei_Templates::get_template('user/my-courses.php');
 
-        }else{
+        } else {
 
             Sensei()->frontend->sensei_login_form();
 
@@ -189,54 +189,54 @@ 
      * loop-course.php for the old shortcodes.
      * @since 1.9.0
      */
-    public static function initialise_legacy_course_loop(){
+    public static function initialise_legacy_course_loop() {
 
         global  $post, $wp_query, $shortcode_override, $course_excludes;
 
         // Handle Query Type
         $query_type = '';
 
-        if ( isset( $_GET[ 'action' ] ) && ( '' != esc_html( $_GET[ 'action' ] ) ) ) {
-            $query_type = esc_html( $_GET[ 'action' ] );
+        if (isset($_GET['action']) && ('' != esc_html($_GET['action']))) {
+            $query_type = esc_html($_GET['action']);
         } // End If Statement
 
-        if ( '' != $shortcode_override ) {
+        if ('' != $shortcode_override) {
             $query_type = $shortcode_override;
         } // End If Statement
 
-        if ( !is_array( $course_excludes ) ) { $course_excludes = array(); }
+        if ( ! is_array($course_excludes)) { $course_excludes = array(); }
 
         // Check that query returns results
         // Handle Pagination
-        $paged = $wp_query->get( 'paged' );
-        $paged = empty( $paged ) ? 1 : $paged;
+        $paged = $wp_query->get('paged');
+        $paged = empty($paged) ? 1 : $paged;
 
 
         // Check for pagination settings
-        if ( isset( Sensei()->settings->settings[ 'course_archive_amount' ] ) && ( 0 < absint( Sensei()->settings->settings[ 'course_archive_amount' ] ) ) ) {
+        if (isset(Sensei()->settings->settings['course_archive_amount']) && (0 < absint(Sensei()->settings->settings['course_archive_amount']))) {
 
-            $amount = absint( Sensei()->settings->settings[ 'course_archive_amount' ] );
+            $amount = absint(Sensei()->settings->settings['course_archive_amount']);
 
         } else {
 
-            $amount = $wp_query->get( 'posts_per_page' );
+            $amount = $wp_query->get('posts_per_page');
 
         } // End If Statement
 
         // This is not a paginated page (or it's simply the first page of a paginated page/post)
 
         global $posts_array;
-        $course_includes   = array();
+        $course_includes = array();
 
-        $query_args = Sensei()->course->get_archive_query_args( $shortcode_override, $amount, $course_includes, $course_excludes );
-        $course_query = new WP_Query( $query_args );
+        $query_args = Sensei()->course->get_archive_query_args($shortcode_override, $amount, $course_includes, $course_excludes);
+        $course_query = new WP_Query($query_args);
         $posts_array = $course_query->get_posts();
 
         // output the courses
-        if( ! empty( $posts_array ) ) {
+        if ( ! empty($posts_array)) {
 
             //output all courses for current query
-            self::loop_courses( $course_query, $amount );
+            self::loop_courses($course_query, $amount);
 
         }
 
@@ -249,31 +249,31 @@ 
      *
      * @param WP_Query $course_query
      */
-    public static function loop_courses( $course_query, $amount ){
+    public static function loop_courses($course_query, $amount) {
 
         global $shortcode_override, $posts_array, $post, $wp_query, $shortcode_override, $course_excludes, $course_includes;
 
-        if ( count( $course_query->get_posts() ) > 0 ) {
+        if (count($course_query->get_posts()) > 0) {
 
-            do_action( 'sensei_course_archive_header', $shortcode_override );
+            do_action('sensei_course_archive_header', $shortcode_override);
 
-            foreach ( $course_query->get_posts() as $course){
+            foreach ($course_query->get_posts() as $course) {
 
                 // Make sure the other loops dont include the same post twice!
-                array_push( $course_excludes, $course->ID );
+                array_push($course_excludes, $course->ID);
 
                 // output the course markup
-                self::the_course( $course->ID );
+                self::the_course($course->ID);
 
             } // End For Loop
 
             // More and Prev links
-            $posts_array_query = new WP_Query(Sensei()->course->course_query( $shortcode_override, $amount, $course_includes, $course_excludes ) );
+            $posts_array_query = new WP_Query(Sensei()->course->course_query($shortcode_override, $amount, $course_includes, $course_excludes));
             $posts_array       = $posts_array_query->get_posts();
             $max_pages = $course_query->found_posts / $amount;
-            if ( '' != $shortcode_override && ( $max_pages > $course_query->get( 'paged' ) ) ) {
+            if ('' != $shortcode_override && ($max_pages > $course_query->get('paged'))) {
 
-                switch( $shortcode_override ){
+                switch ($shortcode_override) {
                     case 'paidcourses':
                         $filter = 'paid';
                         break;
@@ -289,22 +289,22 @@ 
                 }
 
                 $quer_args = array();
-                $quer_args[ 'paged' ] = '2';
-                if( !empty( $filter ) ){
-                    $quer_args[ 'course_filter' ] = $filter;
+                $quer_args['paged'] = '2';
+                if ( ! empty($filter)) {
+                    $quer_args['course_filter'] = $filter;
                 }
 
-                $course_pagination_link = get_post_type_archive_link( 'course' );
-                $more_link_text = esc_html( Sensei()->settings->settings[ 'course_archive_more_link_text' ] );
-                $more_link_url =  esc_url( add_query_arg( $quer_args, $course_pagination_link ) );
+                $course_pagination_link = get_post_type_archive_link('course');
+                $more_link_text = esc_html(Sensei()->settings->settings['course_archive_more_link_text']);
+                $more_link_url = esc_url(add_query_arg($quer_args, $course_pagination_link));
 
                 // next/more
                 $html  = '<div class="navigation"><div class="nav-next">';
-                $html .= '<a href="' . $more_link_url . '">';
+                $html .= '<a href="'.$more_link_url.'">';
                 $html .= $more_link_text;
                 $html .= '<span class="meta-nav"></span></a></div>';
 
-                echo apply_filters( 'course_archive_next_link', $html );
+                echo apply_filters('course_archive_next_link', $html);
 
             } // End If Statement
 
@@ -316,23 +316,23 @@ 
      *
      * @param $course_id
      */
-    public static function the_course( $course_id ){
+    public static function the_course($course_id) {
 
         // Get meta data
-        $course =  get_post( $course_id );
-        $user_info = get_userdata( absint( $course->post_author ) );
-        $author_link = get_author_posts_url( absint( $course->post_author ) );
+        $course = get_post($course_id);
+        $user_info = get_userdata(absint($course->post_author));
+        $author_link = get_author_posts_url(absint($course->post_author));
         $author_display_name = $user_info->display_name;
         $author_id = $course->post_author;
-        $category_output = get_the_term_list( $course_id, 'course-category', '', ', ', '' );
-        $preview_lesson_count = intval( Sensei()->course->course_lesson_preview_count( $course_id ) );
-        $is_user_taking_course = Sensei_Utils::user_started_course( $course_id, get_current_user_id() );
+        $category_output = get_the_term_list($course_id, 'course-category', '', ', ', '');
+        $preview_lesson_count = intval(Sensei()->course->course_lesson_preview_count($course_id));
+        $is_user_taking_course = Sensei_Utils::user_started_course($course_id, get_current_user_id());
         ?>
 
-        <article class="<?php echo esc_attr( join( ' ', get_post_class( array( 'course', 'post' ), $course_id ) ) ); ?>">
+        <article class="<?php echo esc_attr(join(' ', get_post_class(array('course', 'post'), $course_id))); ?>">
             <?php
             // so that legacy shortcodes work with the party plugins that wants to hook in
-            do_action('sensei_course_content_before',$course->ID );
+            do_action('sensei_course_content_before', $course->ID);
             ?>
             <div class="course-content">
 
@@ -348,19 +348,19 @@ 
 
                     <p class="sensei-course-meta">
 
-                        <?php if ( isset( Sensei()->settings->settings[ 'course_author' ] ) && ( Sensei()->settings->settings[ 'course_author' ] ) ) { ?>
-                            <span class="course-author"><?php _e( 'by ', 'woothemes-sensei' ); ?><a href="<?php echo $author_link; ?>" title="<?php echo esc_attr( $author_display_name ); ?>"><?php echo esc_html( $author_display_name   ); ?></a></span>
+                        <?php if (isset(Sensei()->settings->settings['course_author']) && (Sensei()->settings->settings['course_author'])) { ?>
+                            <span class="course-author"><?php _e('by ', 'woothemes-sensei'); ?><a href="<?php echo $author_link; ?>" title="<?php echo esc_attr($author_display_name); ?>"><?php echo esc_html($author_display_name); ?></a></span>
                         <?php } // End If Statement ?>
 
                         <span class="course-lesson-count">
-                                    <?php echo Sensei()->course->course_lesson_count( $course_id ) . '&nbsp;' .  __( 'Lessons', 'woothemes-sensei' ); ?>
+                                    <?php echo Sensei()->course->course_lesson_count($course_id).'&nbsp;'.__('Lessons', 'woothemes-sensei'); ?>
                                 </span>
 
-                        <?php if ( '' != $category_output ) { ?>
-                            <span class="course-category"><?php echo sprintf( __( 'in %s', 'woothemes-sensei' ), $category_output ); ?></span>
+                        <?php if ('' != $category_output) { ?>
+                            <span class="course-category"><?php echo sprintf(__('in %s', 'woothemes-sensei'), $category_output); ?></span>
                         <?php } // End If Statement ?>
 
-                        <?php sensei_simple_course_price( $course_id ); ?>
+                        <?php sensei_simple_course_price($course_id); ?>
 
                     </p>
 
@@ -368,10 +368,10 @@ 
 
                     </p>
 
-                    <?php if ( 0 < $preview_lesson_count && !$is_user_taking_course ) {
-                        $preview_lessons = sprintf( __( '(%d preview lessons)', 'woothemes-sensei' ), $preview_lesson_count ); ?>
+                    <?php if (0 < $preview_lesson_count && ! $is_user_taking_course) {
+                        $preview_lessons = sprintf(__('(%d preview lessons)', 'woothemes-sensei'), $preview_lesson_count); ?>
                         <p class="sensei-free-lessons">
-                            <a href="<?php echo get_permalink( $course_id ); ?>"><?php _e( 'Preview this course', 'woothemes-sensei' ) ?>
+                            <a href="<?php echo get_permalink($course_id); ?>"><?php _e('Preview this course', 'woothemes-sensei') ?>
                             </a> - <?php echo $preview_lessons; ?>
                         </p>
                     <?php } ?>