woothemes / sensei

includes/class-sensei-analysis-user-profile-list-table.php

Spacing +78 added lines, -78 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Analysis User Profile Data Table in Sensei.
@@ -18,17 +18,17 @@ 
 	 * @since  1.2.0
 	 * @return  void
 	 */
-	public function __construct ( $user_id = 0 ) {
-		$this->user_id = intval( $user_id );
+	public function __construct($user_id = 0) {
+		$this->user_id = intval($user_id);
 
 		// Load Parent token into constructor
-		parent::__construct( 'analysis_user_profile' );
+		parent::__construct('analysis_user_profile');
 
 		// Actions
-		add_action( 'sensei_before_list_table', array( $this, 'data_table_header' ) );
-		add_action( 'sensei_after_list_table', array( $this, 'data_table_footer' ) );
+		add_action('sensei_before_list_table', array($this, 'data_table_header'));
+		add_action('sensei_after_list_table', array($this, 'data_table_footer'));
 
-		add_filter( 'sensei_list_table_search_button_text', array( $this, 'search_button' ) );
+		add_filter('sensei_list_table_search_button_text', array($this, 'search_button'));
 	} // End __construct()
 
 	/**
@@ -38,14 +38,14 @@ 
 	 */
 	function get_columns() {
 		$columns = array(
-			'title' => __( 'Course', 'woothemes-sensei' ),
-			'started' => __( 'Date Started', 'woothemes-sensei' ),
-			'completed' => __( 'Date Completed', 'woothemes-sensei' ),
-			'status' => __( 'Status', 'woothemes-sensei' ),
+			'title' => __('Course', 'woothemes-sensei'),
+			'started' => __('Date Started', 'woothemes-sensei'),
+			'completed' => __('Date Completed', 'woothemes-sensei'),
+			'status' => __('Status', 'woothemes-sensei'),
 //			'grade' => __( 'Grade', 'woothemes-sensei' ),
-			'percent' => __( 'Percent Complete', 'woothemes-sensei' ),
+			'percent' => __('Percent Complete', 'woothemes-sensei'),
 		);
-		$columns = apply_filters( 'sensei_analysis_user_profile_columns', $columns );
+		$columns = apply_filters('sensei_analysis_user_profile_columns', $columns);
 		return $columns;
 	}
 
@@ -56,14 +56,14 @@ 
 	 */
 	function get_sortable_columns() {
 		$columns = array(
-			'title' => array( 'title', false ),
-			'started' => array( 'started', false ),
-			'completed' => array( 'completed', false ),
-			'status' => array( 'status', false ),
+			'title' => array('title', false),
+			'started' => array('started', false),
+			'completed' => array('completed', false),
+			'status' => array('status', false),
 //			'grade' => array( 'grade', false ),
-			'percent' => array( 'percent', false )
+			'percent' => array('percent', false)
 		);
-		$columns = apply_filters( 'sensei_analysis_user_profile_columns_sortable', $columns );
+		$columns = apply_filters('sensei_analysis_user_profile_columns_sortable', $columns);
 		return $columns;
 	}
 
@@ -77,32 +77,32 @@ 
 
 		// Handle orderby (needs work)
 		$orderby = '';
-		if ( !empty( $_GET['orderby'] ) ) {
-			if ( array_key_exists( esc_html( $_GET['orderby'] ), $this->get_sortable_columns() ) ) {
-				$orderby = esc_html( $_GET['orderby'] );
+		if ( ! empty($_GET['orderby'])) {
+			if (array_key_exists(esc_html($_GET['orderby']), $this->get_sortable_columns())) {
+				$orderby = esc_html($_GET['orderby']);
 			} // End If Statement
 		}
 
 		// Handle order
 		$order = 'ASC';
-		if ( !empty( $_GET['order'] ) ) {
-			$order = ( 'ASC' == strtoupper($_GET['order']) ) ? 'ASC' : 'DESC';
+		if ( ! empty($_GET['order'])) {
+			$order = ('ASC' == strtoupper($_GET['order'])) ? 'ASC' : 'DESC';
 		}
 
 		// Handle search, need 4.1 version of WP to be able to restrict statuses to known post_ids
 		$search = false;
-		if ( !empty( $_GET['s'] ) ) {
-			$search = esc_html( $_GET['s'] );
+		if ( ! empty($_GET['s'])) {
+			$search = esc_html($_GET['s']);
 		} // End If Statement
 		$this->search = $search;
 
-		$per_page = $this->get_items_per_page( 'sensei_comments_per_page' );
-		$per_page = apply_filters( 'sensei_comments_per_page', $per_page, 'sensei_comments' );
+		$per_page = $this->get_items_per_page('sensei_comments_per_page');
+		$per_page = apply_filters('sensei_comments_per_page', $per_page, 'sensei_comments');
 
 		$paged = $this->get_pagenum();
 		$offset = 0;
-		if ( !empty($paged) ) {
-			$offset = $per_page * ( $paged - 1 );
+		if ( ! empty($paged)) {
+			$offset = $per_page * ($paged - 1);
 		} // End If Statement
 
 		$args = array(
@@ -111,19 +111,19 @@ 
 			'orderby' => $orderby,
 			'order' => $order,
 		);
-		if ( $this->search ) {
+		if ($this->search) {
 			$args['search'] = $this->search;
 		} // End If Statement
 
-		$this->items = $this->get_course_statuses( $args );
+		$this->items = $this->get_course_statuses($args);
 
 		$total_items = $this->total_items;
-		$total_pages = ceil( $total_items / $per_page );
-		$this->set_pagination_args( array(
+		$total_pages = ceil($total_items / $per_page);
+		$this->set_pagination_args(array(
 			'total_items' => $total_items,
 			'total_pages' => $total_pages,
 			'per_page' => $per_page
-		) );
+		));
 	}
 
 	/**
@@ -131,7 +131,7 @@ 
 	 * @since  1.7.0
 	 * @return data
 	 */
-	public function generate_report( $report ) {
+	public function generate_report($report) {
 
 		$data = array();
 
@@ -139,22 +139,22 @@ 
 
 		// Handle orderby
 		$orderby = '';
-		if ( !empty( $_GET['orderby'] ) ) {
-			if ( array_key_exists( esc_html( $_GET['orderby'] ), $this->get_sortable_columns() ) ) {
-				$orderby = esc_html( $_GET['orderby'] );
+		if ( ! empty($_GET['orderby'])) {
+			if (array_key_exists(esc_html($_GET['orderby']), $this->get_sortable_columns())) {
+				$orderby = esc_html($_GET['orderby']);
 			} // End If Statement
 		}
 
 		// Handle order
 		$order = 'ASC';
-		if ( !empty( $_GET['order'] ) ) {
-			$order = ( 'ASC' == strtoupper($_GET['order']) ) ? 'ASC' : 'DESC';
+		if ( ! empty($_GET['order'])) {
+			$order = ('ASC' == strtoupper($_GET['order'])) ? 'ASC' : 'DESC';
 		}
 
 		// Handle search
 		$search = false;
-		if ( !empty( $_GET['s'] ) ) {
-			$search = esc_html( $_GET['s'] );
+		if ( ! empty($_GET['s'])) {
+			$search = esc_html($_GET['s']);
 		} // End If Statement
 		$this->search = $search;
 
@@ -162,23 +162,23 @@ 
 			'orderby' => $orderby,
 			'order' => $order,
 		);
-		if ( $this->search ) {
+		if ($this->search) {
 			$args['search'] = $this->search;
 		} // End If Statement
 
 		// Start the csv with the column headings
 		$column_headers = array();
 		$columns = $this->get_columns();
-		foreach( $columns AS $key => $title ) {
+		foreach ($columns AS $key => $title) {
 			$column_headers[] = $title;
 		}
 		$data[] = $column_headers;
 
-		$this->items = $this->get_course_statuses( $args );
+		$this->items = $this->get_course_statuses($args);
 
 		// Process each row
-		foreach( $this->items AS $item) {
-			$data[] = $this->get_row_data( $item );
+		foreach ($this->items AS $item) {
+			$data[] = $this->get_row_data($item);
 		}
 
 		return $data;
@@ -189,43 +189,43 @@ 
 	 * @since  1.7.0
 	 * @param object $item The current item
 	 */
-	protected function get_row_data( $item ) {
+	protected function get_row_data($item) {
 
-		$course_title =  get_the_title( $item->comment_post_ID );
-		$course_percent = get_comment_meta( $item->comment_ID, 'percent', true );
-		$course_start_date = get_comment_meta( $item->comment_ID, 'start', true );
+		$course_title = get_the_title($item->comment_post_ID);
+		$course_percent = get_comment_meta($item->comment_ID, 'percent', true);
+		$course_start_date = get_comment_meta($item->comment_ID, 'start', true);
 		$course_end_date = '';
 
-		if( 'complete' == $item->comment_approved ) {
+		if ('complete' == $item->comment_approved) {
 
-            $status =  __( 'Completed', 'woothemes-sensei' );
+            $status = __('Completed', 'woothemes-sensei');
 			$status_class = 'graded';
 
 			$course_end_date = $item->comment_date;
 
 		} else {
 
-			$status =  __( 'In Progress', 'woothemes-sensei' );
+			$status = __('In Progress', 'woothemes-sensei');
 			$status_class = 'in-progress';
 
 		}
 
 		// Output users data
-		if ( !$this->csv_output ) {
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'user_id' => $this->user_id, 'course_id' => $item->comment_post_ID ), admin_url( 'admin.php' ) );
+		if ( ! $this->csv_output) {
+			$url = add_query_arg(array('page' => $this->page_slug, 'user_id' => $this->user_id, 'course_id' => $item->comment_post_ID), admin_url('admin.php'));
 
-			$course_title = '<strong><a class="row-title" href="' . esc_url( $url ) . '">' . $course_title . '</a></strong>';
-			$status = sprintf( '<span class="%s">%s</span>', $status_class, $status );
-			if ( is_numeric($course_percent) ) {
+			$course_title = '<strong><a class="row-title" href="'.esc_url($url).'">'.$course_title.'</a></strong>';
+			$status = sprintf('<span class="%s">%s</span>', $status_class, $status);
+			if (is_numeric($course_percent)) {
 				$course_percent .= '%';
 			}
 		} // End If Statement
-		$column_data = apply_filters( 'sensei_analysis_user_profile_column_data', array( 'title' => $course_title,
+		$column_data = apply_filters('sensei_analysis_user_profile_column_data', array('title' => $course_title,
 										'started' => $course_start_date,
 										'completed' => $course_end_date,
 										'status' => $status,
 										'percent' => $course_percent,
-									), $item );
+									), $item);
 
 		return $column_data;
 	}
@@ -235,7 +235,7 @@ 
 	 * @since  1.7.0
 	 * @return array statuses
 	 */
-	private function get_course_statuses( $args ) {
+	private function get_course_statuses($args) {
 
 		$activity_args = array( 
 				'user_id' => $this->user_id,
@@ -247,23 +247,23 @@ 
 				'status' => 'any',
 			);
 
-		$activity_args = apply_filters( 'sensei_analysis_user_profile_filter_statuses', $activity_args );
+		$activity_args = apply_filters('sensei_analysis_user_profile_filter_statuses', $activity_args);
 
 		// WP_Comment_Query doesn't support SQL_CALC_FOUND_ROWS, so instead do this twice
-		$this->total_items = Sensei_Utils::sensei_check_for_activity( array_merge( $activity_args, array('count' => true, 'offset' => 0, 'number' => 0) ) );
+		$this->total_items = Sensei_Utils::sensei_check_for_activity(array_merge($activity_args, array('count' => true, 'offset' => 0, 'number' => 0)));
 
 		// Ensure we change our range to fit (in case a search threw off the pagination) - Should this be added to all views?
-		if ( $this->total_items < $activity_args['offset'] ) {
+		if ($this->total_items < $activity_args['offset']) {
 
-			$new_paged = floor( $this->total_items / $activity_args['number'] );
+			$new_paged = floor($this->total_items / $activity_args['number']);
 			$activity_args['offset'] = $new_paged * $activity_args['number'];
 
 		}
-		$statuses = Sensei_Utils::sensei_check_for_activity( $activity_args, true );
+		$statuses = Sensei_Utils::sensei_check_for_activity($activity_args, true);
 
 		// Need to always return an array, even with only 1 item
-		if ( !is_array($statuses) ) {
-			$statuses = array( $statuses );
+		if ( ! is_array($statuses)) {
+			$statuses = array($statuses);
 		}
 
 		return $statuses;
@@ -276,7 +276,7 @@ 
 	 * @return void
 	 */
 	public function no_items() {
-		echo  __( 'No courses found.', 'woothemes-sensei' );
+		echo  __('No courses found.', 'woothemes-sensei');
 	} // End no_items()
 
 	/**
@@ -285,7 +285,7 @@ 
 	 * @return void
 	 */
 	public function data_table_header() {
-		echo '<strong>' . __( 'Courses', 'woothemes-sensei' ) . '</strong>';
+		echo '<strong>'.__('Courses', 'woothemes-sensei').'</strong>';
 	}
 
 	/**
@@ -294,10 +294,10 @@ 
 	 * @return void
 	 */
 	public function data_table_footer() {
-		$user = get_user_by( 'id', $this->user_id );
-		$report = sanitize_title( $user->display_name ) . '-course-overview';
-		$url = add_query_arg( array( 'page' => $this->page_slug, 'user_id' => $this->user_id, 'sensei_report_download' => $report ), admin_url( 'admin.php' ) );
-		echo '<a class="button button-primary" href="' . esc_url( wp_nonce_url( $url, 'sensei_csv_download-' . $report, '_sdl_nonce' ) ) . '">' . __( 'Export all rows (CSV)', 'woothemes-sensei' ) . '</a>';
+		$user = get_user_by('id', $this->user_id);
+		$report = sanitize_title($user->display_name).'-course-overview';
+		$url = add_query_arg(array('page' => $this->page_slug, 'user_id' => $this->user_id, 'sensei_report_download' => $report), admin_url('admin.php'));
+		echo '<a class="button button-primary" href="'.esc_url(wp_nonce_url($url, 'sensei_csv_download-'.$report, '_sdl_nonce')).'">'.__('Export all rows (CSV)', 'woothemes-sensei').'</a>';
 	}
 
 	/**
@@ -305,8 +305,8 @@ 
 	 * @since  1.7.0
 	 * @return string
 	 */
-	public function search_button( $text = '' ) {
-		return __( 'Search Courses', 'woothemes-sensei' );
+	public function search_button($text = '') {
+		return __('Search Courses', 'woothemes-sensei');
 	}
 
 } // End Class

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

Spacing +23 added lines, -23 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
-if ( ! class_exists( 'WooThemes_Sensei_Email_New_Message_Reply' ) ) :
+if ( ! class_exists('WooThemes_Sensei_Email_New_Message_Reply')) :
 
 /**
  * Teacher New Message
@@ -57,8 +57,8 @@ 
 	 */
 	function __construct() {
 		$this->template = 'new-message-reply';
-		$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] You have a new message', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
-		$this->heading = apply_filters( 'sensei_email_heading', __( 'You have received a reply to your private message', 'woothemes-sensei' ), $this->template );
+		$this->subject = apply_filters('sensei_email_subject', sprintf(__('[%1$s] You have a new message', 'woothemes-sensei'), get_bloginfo('name')), $this->template);
+		$this->heading = apply_filters('sensei_email_heading', __('You have received a reply to your private message', 'woothemes-sensei'), $this->template);
 	}
 
 	/**
@@ -69,35 +69,35 @@ 
      *
 	 * @return void
 	 */
-	function trigger ( $comment, $message ) {
+	function trigger($comment, $message) {
 
 		global  $sensei_email_data;
 
 		$this->comment = $comment;
 		$this->message = $message;
 
-		$this->commenter = get_userdata( $comment->user_id );
+		$this->commenter = get_userdata($comment->user_id);
 
-		$original_sender = get_post_meta( $this->message->ID, '_sender', true );
-		$this->original_sender = get_user_by( 'login', $original_sender );
+		$original_sender = get_post_meta($this->message->ID, '_sender', true);
+		$this->original_sender = get_user_by('login', $original_sender);
 
-		$original_receiver = get_post_meta( $this->message->ID, '_receiver', true );
-		$this->original_receiver = get_user_by( 'login', $original_receiver );
+		$original_receiver = get_post_meta($this->message->ID, '_receiver', true);
+		$this->original_receiver = get_user_by('login', $original_receiver);
 
-		$content_type = get_post_meta( $this->message->ID, '_posttype', true );
-		$content_id = get_post_meta( $this->message->ID, '_post', true );
-		$content_title = get_the_title( $content_id );
+		$content_type = get_post_meta($this->message->ID, '_posttype', true);
+		$content_id = get_post_meta($this->message->ID, '_post', true);
+		$content_title = get_the_title($content_id);
 
-		$comment_link = get_comment_link( $comment );
+		$comment_link = get_comment_link($comment);
 
         // setup the post type parameter
-        $content_type = get_post_type( $content_id );
-        if( !$content_type ){
-            $content_type ='';
+        $content_type = get_post_type($content_id);
+        if ( ! $content_type) {
+            $content_type = '';
         }
 
         // Construct data array
-        $sensei_email_data = apply_filters( 'sensei_email_data', array(
+        $sensei_email_data = apply_filters('sensei_email_data', array(
             'template'			=> $this->template,
             $content_type.'_id' => $content_id,
 			'heading'			=> $this->heading,
@@ -106,17 +106,17 @@ 
 			'comment_link'		=> $comment_link,
 			'content_title'		=> $content_title,
 			'content_type'		=> $content_type,
-		), $this->template );
+		), $this->template);
 
 		// Set recipient
-		if( $this->commenter->user_login == $original_sender ) {
-			$this->recipient = stripslashes( $this->original_receiver->user_email );
+		if ($this->commenter->user_login == $original_sender) {
+			$this->recipient = stripslashes($this->original_receiver->user_email);
 		} else {
-			$this->recipient = stripslashes( $this->original_sender->user_email );
+			$this->recipient = stripslashes($this->original_sender->user_email);
 		}
 
 		// Send mail
-		Sensei()->emails->send( $this->recipient, $this->subject, Sensei()->emails->get_content( $this->template ) );
+		Sensei()->emails->send($this->recipient, $this->subject, Sensei()->emails->get_content($this->template));
 	}
 }
 

includes/emails/class-woothemes-sensei-email-learner-graded-quiz.php

Spacing +19 added lines, -19 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
-if ( ! class_exists( 'WooThemes_Sensei_Email_Learner_Graded_Quiz' ) ) :
+if ( ! class_exists('WooThemes_Sensei_Email_Learner_Graded_Quiz')) :
 
 /**
  * Learner Graded Quiz
@@ -29,8 +29,8 @@ 
 	 */
 	function __construct() {
 		$this->template = 'learner-graded-quiz';
-		$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] Your quiz has been graded', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
-		$this->heading = apply_filters( 'sensei_email_heading', __( 'Your quiz has been graded', 'woothemes-sensei' ), $this->template );
+		$this->subject = apply_filters('sensei_email_subject', sprintf(__('[%1$s] Your quiz has been graded', 'woothemes-sensei'), get_bloginfo('name')), $this->template);
+		$this->heading = apply_filters('sensei_email_heading', __('Your quiz has been graded', 'woothemes-sensei'), $this->template);
 	}
 
 	/**
@@ -43,48 +43,48 @@ 
      *
 	 * @return void
 	 */
-	function trigger ( $user_id = 0, $quiz_id = 0, $grade = 0, $passmark = 0 ) {
+	function trigger($user_id = 0, $quiz_id = 0, $grade = 0, $passmark = 0) {
 
 		global  $sensei_email_data;
 
 		// Get learner user object
-		$this->user = new WP_User( $user_id );
+		$this->user = new WP_User($user_id);
 
 		// Get passed flag
-		$passed = __( 'failed', 'woothemes-sensei' );
-		if( $grade >= $passmark ) {
-			$passed = __( 'passed', 'woothemes-sensei' );
+		$passed = __('failed', 'woothemes-sensei');
+		if ($grade >= $passmark) {
+			$passed = __('passed', 'woothemes-sensei');
 		}
 
 		// Get grade tye (auto/manual)
-		$grade_type = get_post_meta( $quiz_id, '_quiz_grade_type', true );
+		$grade_type = get_post_meta($quiz_id, '_quiz_grade_type', true);
 
-		if( 'auto' == $grade_type ) {
-			$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] You have completed a quiz', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
-			$this->heading = apply_filters( 'sensei_email_heading', __( 'You have completed a quiz', 'woothemes-sensei' ), $this->template );
+		if ('auto' == $grade_type) {
+			$this->subject = apply_filters('sensei_email_subject', sprintf(__('[%1$s] You have completed a quiz', 'woothemes-sensei'), get_bloginfo('name')), $this->template);
+			$this->heading = apply_filters('sensei_email_heading', __('You have completed a quiz', 'woothemes-sensei'), $this->template);
 		}
 
-		$lesson_id = get_post_meta( $quiz_id, '_quiz_lesson', true );
+		$lesson_id = get_post_meta($quiz_id, '_quiz_lesson', true);
 
 		// Construct data array
-		$sensei_email_data = apply_filters( 'sensei_email_data', array(
+		$sensei_email_data = apply_filters('sensei_email_data', array(
 			'template'			=> $this->template,
 			'heading'			=> $this->heading,
 			'user_id'			=> $user_id,
-			'user_name'         => stripslashes( $this->user->display_name ),
+			'user_name'         => stripslashes($this->user->display_name),
 			'lesson_id'			=> $lesson_id,
 			'quiz_id'			=> $quiz_id,
 			'grade'				=> $grade,
 			'passmark'			=> $passmark,
 			'passed'			=> $passed,
 			'grade_type'		=> $grade_type,
-		), $this->template );
+		), $this->template);
 
 		// Set recipient (learner)
-		$this->recipient = stripslashes( $this->user->user_email );
+		$this->recipient = stripslashes($this->user->user_email);
 
 		// Send mail
-		Sensei()->emails->send( $this->recipient, $this->subject, Sensei()->emails->get_content( $this->template ) );
+		Sensei()->emails->send($this->recipient, $this->subject, Sensei()->emails->get_content($this->template));
 	}
 }
 

includes/emails/class-sensei-email-teacher-new-course-assignment.php

Spacing +13 added lines, -13 removed lines

@@ -3,9 +3,9 @@ 
  * This email will be sent to a teacher when a course is assigned to them.
  */
 
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
-if (  class_exists('Sensei_Email_Teacher_New_Course_Assignment') ){
+if (class_exists('Sensei_Email_Teacher_New_Course_Assignment')) {
     return;
 }
 
@@ -36,8 +36,8 @@ 
 	function __construct() {
 
         $this->template = 'teacher-new-course-assignment';
-		$this->subject = apply_filters( 'sensei_email_subject', sprintf( __( '[%1$s] You have been assigned to a course', 'woothemes-sensei' ), get_bloginfo( 'name' ) ), $this->template );
-		$this->heading = apply_filters( 'sensei_email_heading', __( 'Course assigned to you', 'woothemes-sensei' ), $this->template );
+		$this->subject = apply_filters('sensei_email_subject', sprintf(__('[%1$s] You have been assigned to a course', 'woothemes-sensei'), get_bloginfo('name')), $this->template);
+		$this->heading = apply_filters('sensei_email_heading', __('Course assigned to you', 'woothemes-sensei'), $this->template);
         return;
 	}
 
@@ -49,30 +49,30 @@ 
      * @param $course_id
 	 * @return void
 	 */
-	function trigger( $teacher_id = 0, $course_id = 0 ) {
+	function trigger($teacher_id = 0, $course_id = 0) {
 		global $sensei_email_data;
 
-		$this->teacher = new WP_User( $teacher_id );
-        $this->recipient = stripslashes( $this->teacher->user_email );
-        $this->subject = __( 'New course assigned to you', 'woothemes-sensei' );
+		$this->teacher = new WP_User($teacher_id);
+        $this->recipient = stripslashes($this->teacher->user_email);
+        $this->subject = __('New course assigned to you', 'woothemes-sensei');
 
         //course edit link
-        $course_edit_link = admin_url('post.php?post=' . $course_id . '&action=edit' );
+        $course_edit_link = admin_url('post.php?post='.$course_id.'&action=edit');
 
         // Course name
-        $course = get_post( $course_id);
+        $course = get_post($course_id);
 		// Construct data array
-		$sensei_email_data = apply_filters( 'sensei_email_data', array(
+		$sensei_email_data = apply_filters('sensei_email_data', array(
 			'template'			=> $this->template,
 			'heading'			=> $this->heading,
 			'teacher_id'		=> $teacher_id,
 			'course_id'			=> $course_id,
             'course_name'			=> $course->post_title,
             'course_edit_link' => $course_edit_link,
-		), $this->template );
+		), $this->template);
 
 		// Send mail
-		Sensei()->emails->send( $this->recipient, $this->subject, Sensei()->emails->get_content( $this->template ) );
+		Sensei()->emails->send($this->recipient, $this->subject, Sensei()->emails->get_content($this->template));
 	}
 }
 

includes/class-sensei-teacher.php

Spacing +283 added lines, -283 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Teacher class
@@ -40,58 +40,58 @@ 
      * @since 1.8.0
      * @access public
      */
-    public function __construct ( ) {
+    public function __construct( ) {
 
-        add_action( 'add_meta_boxes', array( $this , 'add_teacher_meta_boxes' ) , 10, 2 );
-        add_action( 'save_post',  array( $this, 'save_teacher_meta_box' ) );
-        add_filter( 'parse_query', array( $this, 'limit_teacher_edit_screen_post_types' ));
-        add_filter( 'pre_get_posts', array( $this, 'course_analysis_teacher_access_limit' ) );
-        add_filter( 'wp_count_posts', array( $this, 'list_table_counts' ), 10, 3 );
+        add_action('add_meta_boxes', array($this, 'add_teacher_meta_boxes'), 10, 2);
+        add_action('save_post', array($this, 'save_teacher_meta_box'));
+        add_filter('parse_query', array($this, 'limit_teacher_edit_screen_post_types'));
+        add_filter('pre_get_posts', array($this, 'course_analysis_teacher_access_limit'));
+        add_filter('wp_count_posts', array($this, 'list_table_counts'), 10, 3);
 
-        add_action( 'pre_get_posts', array( $this, 'filter_queries' ) );
+        add_action('pre_get_posts', array($this, 'filter_queries'));
 
         //filter the quiz submissions
-        add_filter( 'sensei_check_for_activity' , array( $this, 'filter_grading_activity_queries') );
+        add_filter('sensei_check_for_activity', array($this, 'filter_grading_activity_queries'));
 
         //grading totals count only those belonging to the teacher
-        add_filter('sensei_count_statuses_args', array( $this, 'limit_grading_totals' ) );
+        add_filter('sensei_count_statuses_args', array($this, 'limit_grading_totals'));
 
         // show the courses owned by a user on his author archive page
-        add_filter( 'pre_get_posts', array( $this, 'add_courses_to_author_archive' ) );
+        add_filter('pre_get_posts', array($this, 'add_courses_to_author_archive'));
 
         // notify admin when a teacher creates a course
-        add_action( 'transition_post_status',array( $this, 'notify_admin_teacher_course_creation' ), 10, 3 );
+        add_action('transition_post_status', array($this, 'notify_admin_teacher_course_creation'), 10, 3);
 
         // limit the analysis view to only the users taking courses belong to this teacher
-        add_filter( 'sensei_analysis_overview_filter_users',array( $this, 'limit_analysis_learners' ) , 5, 1 );
+        add_filter('sensei_analysis_overview_filter_users', array($this, 'limit_analysis_learners'), 5, 1);
 
         // give teacher access to question post type
-        add_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20, 2 );
+        add_filter('sensei_lesson_quiz_questions', array($this, 'allow_teacher_access_to_questions'), 20, 2);
 
         // Teacher column on the courses list on the admin edit screen
-        add_filter('manage_edit-course_columns' , array( $this, 'course_column_heading'), 10,1 );
-        add_filter('manage_course_posts_custom_column' , array( $this, 'course_column_data'), 10,2 );
+        add_filter('manage_edit-course_columns', array($this, 'course_column_heading'), 10, 1);
+        add_filter('manage_course_posts_custom_column', array($this, 'course_column_data'), 10, 2);
 
         //admin edit messages query limit teacher
-        add_filter( 'pre_get_posts', array( $this, 'limit_edit_messages_query' ) );
+        add_filter('pre_get_posts', array($this, 'limit_edit_messages_query'));
 
         //add filter by teacher on courses list
-        add_action( 'restrict_manage_posts', array( $this, 'course_teacher_filter_options' ) );
-        add_filter( 'request', array( $this, 'teacher_filter_query_modify' ) );
+        add_action('restrict_manage_posts', array($this, 'course_teacher_filter_options'));
+        add_filter('request', array($this, 'teacher_filter_query_modify'));
 
         // Handle media library restrictions
-        add_filter( 'request', array( $this, 'restrict_media_library' ), 10, 1 );
-        add_filter( 'ajax_query_attachments_args', array( $this, 'restrict_media_library_modal' ), 10, 1 );
+        add_filter('request', array($this, 'restrict_media_library'), 10, 1);
+        add_filter('ajax_query_attachments_args', array($this, 'restrict_media_library_modal'), 10, 1);
 
         // update lesson owner to course teacher when saved
-        add_action( 'save_post',  array( $this, 'update_lesson_teacher' ) );
+        add_action('save_post', array($this, 'update_lesson_teacher'));
 
         // If a Teacher logs in, redirect to /wp-admin/
-        add_filter( 'wp_login', array( $this, 'teacher_login_redirect') , 10, 2 );
+        add_filter('wp_login', array($this, 'teacher_login_redirect'), 10, 2);
 
 
-        add_action( 'admin_menu', array( $this, 'restrict_posts_menu_page'), 10);
-        add_filter('pre_get_comments',  array ($this, 'restrict_comment_moderation'), 10, 1);
+        add_action('admin_menu', array($this, 'restrict_posts_menu_page'), 10);
+        add_filter('pre_get_comments', array($this, 'restrict_comment_moderation'), 10, 1);
 
 
     } // end __constructor()
@@ -106,15 +106,15 @@ 
      * @access public
      * @return void
      */
-    public function create_role ( ) {
+    public function create_role( ) {
 
         // check if the role exists
-        $this->teacher_role = get_role( 'teacher' );
+        $this->teacher_role = get_role('teacher');
 
         // if the the teacher is not a valid WordPress role create it
-       if ( ! is_a( $this->teacher_role, 'WP_Role' ) ) {
+       if ( ! is_a($this->teacher_role, 'WP_Role')) {
            // create the role
-           $this->teacher_role = add_role( 'teacher', __( 'Teacher', 'woothemes-sensei' ) );
+           $this->teacher_role = add_role('teacher', __('Teacher', 'woothemes-sensei'));
        }
 
        // add the capabilities before returning
@@ -128,10 +128,10 @@ 
      * @since 1.8.0
      * @access protected
      */
-    protected function add_capabilities ( ) {
+    protected function add_capabilities( ) {
 
         // if this is not a valid WP_Role object exit without adding anything
-        if(  ! is_a( $this->teacher_role, 'WP_Role' ) || empty( $this->teacher_role ) ) {
+        if ( ! is_a($this->teacher_role, 'WP_Role') || empty($this->teacher_role)) {
             return;
         }
 
@@ -142,7 +142,7 @@ 
          * @param array $capabilities
          * keys: (string) $cap_name => (bool) $grant
          */
-        $caps = apply_filters( 'sensei_teacher_role_capabilities', array(
+        $caps = apply_filters('sensei_teacher_role_capabilities', array(
             // General access rules
             'read' => true,
             'manage_sensei_grades' => true,
@@ -199,7 +199,7 @@ 
 
         ));
 
-        foreach ( $caps as $cap => $grant ) {
+        foreach ($caps as $cap => $grant) {
 
             // load the capability on to the teacher role
             $this->teacher_role->add_cap($cap, $grant);
@@ -219,12 +219,12 @@ 
      * @parameter WP_Post $post
      * @return void
      */
-    public function add_teacher_meta_boxes ( $post ) {
+    public function add_teacher_meta_boxes($post) {
 
-        if( !current_user_can('manage_options') ){
+        if ( ! current_user_can('manage_options')) {
             return;
         }
-        add_meta_box( 'sensei-teacher',  __( 'Teacher' , 'woothemes-sensei'),  array( $this , 'teacher_meta_box_content' ),
+        add_meta_box('sensei-teacher', __('Teacher', 'woothemes-sensei'), array($this, 'teacher_meta_box_content'),
             'course',
             'side',
             'core'
@@ -241,7 +241,7 @@ 
      * @access public
      * @parameters
      */
-    public function teacher_meta_box_content ( $post ) {
+    public function teacher_meta_box_content($post) {
 
         // get the current author
         $current_author = $post->post_author;
@@ -252,12 +252,12 @@ 
     ?>
         <select name="sensei-course-teacher-author" class="sensei course teacher">
 
-            <?php foreach ( $users as $user_id ) { ?>
+            <?php foreach ($users as $user_id) { ?>
 
                     <?php
                         $user = get_user_by('id', $user_id);
                     ?>
-                    <option <?php selected(  $current_author , $user_id , true ); ?> value="<?php echo $user_id; ?>" >
+                    <option <?php selected($current_author, $user_id, true); ?> value="<?php echo $user_id; ?>" >
                         <?php echo  $user->display_name; ?>
                     </option>
 
@@ -279,7 +279,7 @@ 
      * @parameters
      * @return array $users user id array
      */
-    public function get_teachers_and_authors ( ){
+    public function get_teachers_and_authors( ) {
 
         $author_query_args = array(
             'blog_id'      => $GLOBALS['blog_id'],
@@ -287,7 +287,7 @@ 
             'who'          => 'authors'
         );
 
-        $authors = get_users( $author_query_args );
+        $authors = get_users($author_query_args);
 
         $teacher_query_args = array(
             'blog_id'      => $GLOBALS['blog_id'],
@@ -295,9 +295,9 @@ 
             'role'         => 'teacher',
         );
 
-        $teachers = get_users( $teacher_query_args );
+        $teachers = get_users($teacher_query_args);
 
-        return  array_unique( array_merge( $teachers, $authors ) );
+        return  array_unique(array_merge($teachers, $authors));
 
     }// end get_teachers_and_authors
 
@@ -313,43 +313,43 @@ 
      * @parameters
      * @return array $users user id array
      */
-    public function save_teacher_meta_box ( $course_id ){
+    public function save_teacher_meta_box($course_id) {
 
         // check if this is a post from saving the teacher, if not exit early
-        if(! isset( $_POST[ 'sensei-course-teacher-author' ] ) || ! isset( $_POST['post_ID'] )  ){
+        if ( ! isset($_POST['sensei-course-teacher-author']) || ! isset($_POST['post_ID'])) {
             return;
         }
 
         //don't fire this hook again
-        remove_action('save_post', array( $this, 'save_teacher_meta_box' ) );
+        remove_action('save_post', array($this, 'save_teacher_meta_box'));
 
         // get the current post object
-        $post = get_post( $course_id );
+        $post = get_post($course_id);
 
         // get the current teacher/author
-        $current_author = absint( $post->post_author );
-        $new_author = absint( $_POST[ 'sensei-course-teacher-author' ] );
+        $current_author = absint($post->post_author);
+        $new_author = absint($_POST['sensei-course-teacher-author']);
 
         // loop through all post lessons to update their authors as well
-        $this->update_course_lessons_author( $course_id , $new_author );
+        $this->update_course_lessons_author($course_id, $new_author);
 
         // do not do any processing if the selected author is the same as the current author
-        if( $current_author == $new_author ){
+        if ($current_author == $new_author) {
             return;
         }
 
         // save the course  author
         $post_updates = array(
-            'ID' => $post->ID ,
+            'ID' => $post->ID,
             'post_author' => $new_author
         );
-        wp_update_post( $post_updates );
+        wp_update_post($post_updates);
 
         // ensure the the modules are update so that then new teacher has access to them
-        Sensei_Teacher::update_course_modules_author( $course_id, $new_author );
+        Sensei_Teacher::update_course_modules_author($course_id, $new_author);
 
         // notify the new teacher
-        $this->teacher_course_assigned_notification( $new_author, $course_id );
+        $this->teacher_course_assigned_notification($new_author, $course_id);
 
     } // end save_teacher_meta_box
 
@@ -363,56 +363,56 @@ 
      * @param $new_teacher_id
      * @return void
      */
-    public static function update_course_modules_author( $course_id ,$new_teacher_id ){
+    public static function update_course_modules_author($course_id, $new_teacher_id) {
 
-        if( empty( $course_id ) || empty( $new_teacher_id ) ){
+        if (empty($course_id) || empty($new_teacher_id)) {
             return;
         }
 
-        $terms_selected_on_course = wp_get_object_terms( $course_id, 'module' );
+        $terms_selected_on_course = wp_get_object_terms($course_id, 'module');
 
-        if( empty( $terms_selected_on_course ) ){
+        if (empty($terms_selected_on_course)) {
             return;
         }
 
-        foreach( $terms_selected_on_course as $term ){
+        foreach ($terms_selected_on_course as $term) {
 
-            $term_author = Sensei_Core_Modules::get_term_author( $term->slug );
-            if( $new_teacher_id != $term_author->ID  ){
+            $term_author = Sensei_Core_Modules::get_term_author($term->slug);
+            if ($new_teacher_id != $term_author->ID) {
 
                 $new_term = '';
 
                 //if the new teacher is admin first check to see if the term with this name already exists
-                if( user_can( $new_teacher_id, 'manage_options' ) ){
+                if (user_can($new_teacher_id, 'manage_options')) {
 
-                    $slug_without_teacher_id = str_ireplace(' ', '-', trim( $term->name ) );
-                    $term_args = array( 'slug'=> $slug_without_teacher_id, 'hide_empty' => false, );
-                    $existing_admin_terms = get_terms( 'module', $term_args );
-                    if( !empty( $existing_admin_terms ) ){
+                    $slug_without_teacher_id = str_ireplace(' ', '-', trim($term->name));
+                    $term_args = array('slug'=> $slug_without_teacher_id, 'hide_empty' => false,);
+                    $existing_admin_terms = get_terms('module', $term_args);
+                    if ( ! empty($existing_admin_terms)) {
                         // insert it even if it exists
-                        $new_term = get_term( $existing_admin_terms[0]->term_id, 'module', ARRAY_A );
+                        $new_term = get_term($existing_admin_terms[0]->term_id, 'module', ARRAY_A);
                     }
                 }
 
-                if( empty ( $new_term ) ){
+                if (empty ($new_term)) {
 
                    //setup the new slug
-                   $new_author_term_slug =  $new_teacher_id . '-' . str_ireplace(' ', '-', trim( $term->name ) );
+                   $new_author_term_slug = $new_teacher_id.'-'.str_ireplace(' ', '-', trim($term->name));
 
                    // create new term and set it
-                   $new_term = wp_insert_term( $term->name,'module', array('slug'=> $new_author_term_slug )  );
+                   $new_term = wp_insert_term($term->name, 'module', array('slug'=> $new_author_term_slug));
 
                 }
 
 
 
                 // if term exists
-                if( is_wp_error( $new_term ) && isset( $new_term->errors['term_exists'] ) ){
+                if (is_wp_error($new_term) && isset($new_term->errors['term_exists'])) {
 
-                    $existing_term = get_term_by( 'slug', $new_author_term_slug, 'module');
+                    $existing_term = get_term_by('slug', $new_author_term_slug, 'module');
                     $term_id = $existing_term->term_id;
 
-                }else{
+                } else {
 
                     // for a new term simply get the term from the returned value
                     $term_id = $new_term['term_id'];
@@ -420,21 +420,21 @@ 
                 } // end if term exist
 
                 // set the terms selected on the course
-                wp_set_object_terms( $course_id, $term_id , 'module', true );
+                wp_set_object_terms($course_id, $term_id, 'module', true);
 
                 // remove old term
-                wp_remove_object_terms( $course_id, $term->term_id, 'module' );
+                wp_remove_object_terms($course_id, $term->term_id, 'module');
 
                 // update the lessons within the current module term
-                $lessons = Sensei()->course->course_lessons( $course_id );
-                foreach( $lessons as $lesson  ){
+                $lessons = Sensei()->course->course_lessons($course_id);
+                foreach ($lessons as $lesson) {
 
-                    if( has_term( $term->slug, 'module', $lesson ) ){
+                    if (has_term($term->slug, 'module', $lesson)) {
 
                         // add the new term, the false at the end says to replace all terms on this module
                         // with the new term.
-                        wp_set_object_terms( $lesson->ID, $term_id , 'module', false );
-                        update_post_meta( $lesson->ID, '_order_module_' . intval( $term_id ), 0 );
+                        wp_set_object_terms($lesson->ID, $term_id, 'module', false);
+                        update_post_meta($lesson->ID, '_order_module_'.intval($term_id), 0);
                     }
 
                 }// end for each
@@ -454,50 +454,50 @@ 
      * @parameters
      * @return array $users user id array
      */
-    public function update_course_lessons_author ( $course_id, $new_author  ){
+    public function update_course_lessons_author($course_id, $new_author) {
 
 
-        if( empty( $course_id ) || empty( $new_author ) ){
+        if (empty($course_id) || empty($new_author)) {
             return false;
         }
 
         //get a list of course lessons
-        $lessons = Sensei()->course->course_lessons( $course_id );
+        $lessons = Sensei()->course->course_lessons($course_id);
 
-        if( empty( $lessons )  ||  ! is_array( $lessons )  ){
+        if (empty($lessons) || ! is_array($lessons)) {
             return false;
         }
 
         // update each lesson and quiz author
-        foreach( $lessons as $lesson ){
+        foreach ($lessons as $lesson) {
 
             // don't update if the author is tha same as the new author
-            if( $new_author == $lesson->post_author ){
+            if ($new_author == $lesson->post_author) {
                 continue;
             }
 
             // update lesson author
-            wp_update_post( array(
+            wp_update_post(array(
                 'ID'=> $lesson->ID,
                 'post_author' => $new_author
-                ) );
+                ));
 
             // update quiz author
             //get the lessons quiz
-            $lesson_quizzes = Sensei()->lesson->lesson_quizzes( $lesson->ID );
-            if( is_array( $lesson_quizzes ) ){
-                foreach ( $lesson_quizzes as $quiz_id ) {
+            $lesson_quizzes = Sensei()->lesson->lesson_quizzes($lesson->ID);
+            if (is_array($lesson_quizzes)) {
+                foreach ($lesson_quizzes as $quiz_id) {
                     // update quiz with new author
-                    wp_update_post( array(
+                    wp_update_post(array(
                         'ID'           => $quiz_id,
                         'post_author' =>  $new_author
-                    ) );
+                    ));
                 }
-            }else{
-                wp_update_post( array(
+            } else {
+                wp_update_post(array(
                     'ID'           => $lesson_quizzes,
                     'post_author' =>  $new_author
-                ) );
+                ));
             }
 
         } // end for each lessons
@@ -518,30 +518,30 @@ 
      * @parameters $query
      * @return array $users user id array
      */
-    public function course_analysis_teacher_access_limit ( $query ) {
+    public function course_analysis_teacher_access_limit($query) {
 
-        if( ! is_admin() || ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+        if ( ! is_admin() || (defined('DOING_AJAX') && DOING_AJAX)) {
             return $query;
         }
 
-        if ( ! function_exists( 'get_current_screen' ) ) {
+        if ( ! function_exists('get_current_screen')) {
             return $query;
         }
 
         $screen = get_current_screen();
-        $sensei_post_types = array('course', 'lesson', 'question' );
+        $sensei_post_types = array('course', 'lesson', 'question');
 
         // exit early for the following conditions
-        $limit_screen_ids = array( 'sensei_page_sensei_analysis', 'course_page_module-order' );
+        $limit_screen_ids = array('sensei_page_sensei_analysis', 'course_page_module-order');
 
-        if( ! $this->is_admin_teacher() || empty( $screen ) || ! in_array( $screen->id ,$limit_screen_ids )
-            || ! in_array( $query->query['post_type'], $sensei_post_types ) ){
+        if ( ! $this->is_admin_teacher() || empty($screen) || ! in_array($screen->id, $limit_screen_ids)
+            || ! in_array($query->query['post_type'], $sensei_post_types)) {
             return $query;
         }
 
         global $current_user;
         // set the query author to the current user to only show those those posts
-        $query->set( 'author', $current_user->ID );
+        $query->set('author', $current_user->ID);
         return $query;
 
     }// end course_analysis_teacher_access_limit
@@ -557,14 +557,14 @@ 
      * @parameters array $wp_query
      * @return bool $is_admin_teacher
      */
-    public function is_admin_teacher ( ){
+    public function is_admin_teacher( ) {
 
-        if( ! is_user_logged_in()){
+        if ( ! is_user_logged_in()) {
             return false;
         }
         $is_admin_teacher = false;
 
-        if( is_admin() && Sensei_Teacher::is_a_teacher( get_current_user_id() )  ){
+        if (is_admin() && Sensei_Teacher::is_a_teacher(get_current_user_id())) {
 
             $is_admin_teacher = true;
 
@@ -584,14 +584,14 @@ 
      * @param  string $perm   User permission level
      * @return object         Modified status counts
      */
-    public function list_table_counts( $counts, $type, $perm ) {
+    public function list_table_counts($counts, $type, $perm) {
         global $current_user;
 
-        if( ! in_array( $type, array( 'course', 'lesson', 'question' ) ) ) {
+        if ( ! in_array($type, array('course', 'lesson', 'question'))) {
             return $counts;
         }
 
-        if( ! $this->is_admin_teacher() ) {
+        if ( ! $this->is_admin_teacher()) {
             return $counts;
         }
 
@@ -605,10 +605,10 @@ 
         $stati = get_post_stati();
 
         // Update count object
-        foreach( $stati as $status ) {
+        foreach ($stati as $status) {
             $args['post_status'] = $status;
-            $posts = get_posts( $args );
-            $counts->$status = count( $posts );
+            $posts = get_posts($args);
+            $counts->$status = count($posts);
         }
 
         return $counts;
@@ -622,22 +622,22 @@ 
      * @since 1.8.0
      *
      */
-    public function filter_queries ( $query ) {
+    public function filter_queries($query) {
         global $current_user;
 
-        if( ! $this->is_admin_teacher() ) {
+        if ( ! $this->is_admin_teacher()) {
             return;
         }
 
-        if ( ! function_exists( 'get_current_screen' ) ) {
+        if ( ! function_exists('get_current_screen')) {
             return;
         }
 
         $screen = get_current_screen();
-        if( empty( $screen ) ) {
+        if (empty($screen)) {
             return $query;
         }
-        switch( $screen->id ) {
+        switch ($screen->id) {
             case 'sensei_page_sensei_grading':
             case 'sensei_page_sensei_analysis':
             case 'sensei_page_sensei_learners':
@@ -656,7 +656,7 @@ 
              * @param string $screen_id
              *
              */
-            $query->set( 'author', apply_filters( 'sensei_filter_queries_set_author', $current_user->ID, $screen->id ) );
+            $query->set('author', apply_filters('sensei_filter_queries_set_author', $current_user->ID, $screen->id));
             break;
         }
     }
@@ -671,31 +671,31 @@ 
      *
      * @return array $comments
      */
-    public function filter_grading_activity_queries( $comments ){
+    public function filter_grading_activity_queries($comments) {
 
-        if( !is_admin() || ! $this->is_admin_teacher() || is_numeric( $comments ) || ! is_array( $comments ) ){
-            return $comments ;
+        if ( ! is_admin() || ! $this->is_admin_teacher() || is_numeric($comments) || ! is_array($comments)) {
+            return $comments;
         }
 
         //check if we're on the grading screen
         $screen = get_current_screen();
 
-        if( empty( $screen ) || 'sensei_page_sensei_grading' != $screen->id ){
+        if (empty($screen) || 'sensei_page_sensei_grading' != $screen->id) {
             return $comments;
         }
 
         // get the course and determine if the current teacher is the owner
         // if not remove it from the list of comments to be returned
-        foreach( $comments as $key => $comment){
-            $lesson = get_post( $comment->comment_post_ID );
-            $course_id = Sensei()->lesson->get_course_id( $lesson->ID );
-            $course = get_post( $course_id );
-            if( ! isset( $course->post_author ) || intval( $course->post_author) != intval( get_current_user_id() ) ){
+        foreach ($comments as $key => $comment) {
+            $lesson = get_post($comment->comment_post_ID);
+            $course_id = Sensei()->lesson->get_course_id($lesson->ID);
+            $course = get_post($course_id);
+            if ( ! isset($course->post_author) || intval($course->post_author) != intval(get_current_user_id())) {
                 //remove this as the teacher should see this.
-                unset( $comments[ $key ] );
+                unset($comments[$key]);
             }
         }
-        return $comments ;
+        return $comments;
 
     }// end function filter grading
 
@@ -711,34 +711,34 @@ 
      *
      * @return array  $args
      */
-    public function limit_grading_totals( $args ){
+    public function limit_grading_totals($args) {
 
-        if( !is_admin() || ! $this->is_admin_teacher() || ! is_array( $args ) ){
-            return $args ;
+        if ( ! is_admin() || ! $this->is_admin_teacher() || ! is_array($args)) {
+            return $args;
         }
 
         //get the teachers courses
         // the query is already filtered to only the teacher
-        $courses =  Sensei()->course->get_all_courses();
+        $courses = Sensei()->course->get_all_courses();
 
-        if( empty(  $courses ) || ! is_array( $courses ) ){
+        if (empty($courses) || ! is_array($courses)) {
             return $args;
         }
 
         //setup the lessons quizzes  to limit the grading totals to
         $quiz_scope = array();
-        foreach( $courses as $course ){
+        foreach ($courses as $course) {
 
-            $course_lessons = Sensei()->course->course_lessons( $course->ID );
+            $course_lessons = Sensei()->course->course_lessons($course->ID);
 
-            if( ! empty( $course_lessons ) && is_array( $course_lessons  ) ){
+            if ( ! empty($course_lessons) && is_array($course_lessons)) {
 
-                foreach(  $course_lessons as $lesson ){
+                foreach ($course_lessons as $lesson) {
 
-                    $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson->ID );
-                    if( !empty( $quiz_id ) ) {
+                    $quiz_id = Sensei()->lesson->lesson_quizzes($lesson->ID);
+                    if ( ! empty($quiz_id)) {
 
-                        array_push( $quiz_scope, $quiz_id );
+                        array_push($quiz_scope, $quiz_id);
 
                     }
 
@@ -761,41 +761,41 @@ 
      * @param WP_Query $query
      * @return WP_Query $query
      */
-    public function add_courses_to_author_archive( $query ) {
+    public function add_courses_to_author_archive($query) {
 
-        if ( is_admin() || ! $query->is_author() ){
+        if (is_admin() || ! $query->is_author()) {
             return $query;
         }
 
         // this should only apply to users with the teacher role
-        $current_page_user = get_user_by('login', $query->get('author_name') );
-        if( ! $current_page_user || ! in_array('teacher', $current_page_user->roles ) )     {
+        $current_page_user = get_user_by('login', $query->get('author_name'));
+        if ( ! $current_page_user || ! in_array('teacher', $current_page_user->roles)) {
 
             return $query;
 
         }
 
         // Change post types depending on what is set already
-        $current_post_types = $query->get( 'post_type' );
-        if( empty( $current_post_types  ) ){
+        $current_post_types = $query->get('post_type');
+        if (empty($current_post_types)) {
 
             // if empty it means post by default, so add post so that it also includes that for now
-            $new_post_types = array( 'post', 'course' );
+            $new_post_types = array('post', 'course');
 
-        } elseif( is_array( $current_post_types  ) ) {
+        } elseif (is_array($current_post_types)) {
 
             // merge the post types instead of overwriting it
-            $new_post_types = array_merge( $current_post_types, array( 'course' ) );
+            $new_post_types = array_merge($current_post_types, array('course'));
 
-        }else{
+        } else {
 
             // in this instance it is probably just one post type in string format
-            $new_post_types =  array( $current_post_types , 'course');
+            $new_post_types = array($current_post_types, 'course');
 
         }
 
         // change the query before returning it
-        $query->set('post_type', $new_post_types );
+        $query->set('post_type', $new_post_types);
 
         /**
          * Change the query on the teacher author archive template
@@ -803,7 +803,7 @@ 
          * @since 1.8.4
          * @param WP_Query $query
          */
-        return apply_filters( 'sensei_teacher_archive_query', $query );
+        return apply_filters('sensei_teacher_archive_query', $query);
 
     }
 
@@ -816,21 +816,21 @@ 
      * @param $course_id
      * @return bool
      */
-    public function teacher_course_assigned_notification( $teacher_id, $course_id ){
+    public function teacher_course_assigned_notification($teacher_id, $course_id) {
 
-        if( 'course' != get_post_type( $course_id ) || ! get_userdata( $teacher_id ) ){
+        if ('course' != get_post_type($course_id) || ! get_userdata($teacher_id)) {
             return false;
         }
 
         // if new user is the same as the current logged user, they don't need an email
-        if( $teacher_id == get_current_user_id() ){
+        if ($teacher_id == get_current_user_id()) {
             return true;
         }
 
         // load the email class
         include('emails/class-sensei-email-teacher-new-course-assignment.php');
         $email = new Sensei_Email_Teacher_New_Course_Assignment();
-        $email->trigger( $teacher_id, $course_id );
+        $email->trigger($teacher_id, $course_id);
 
         return true;
     } // end  teacher_course_assigned_notification
@@ -844,12 +844,12 @@ 
      * @param int $course_id
      * @return bool
      */
-    public function notify_admin_teacher_course_creation( $new_status, $old_status, $post ){
+    public function notify_admin_teacher_course_creation($new_status, $old_status, $post) {
 
         $course_id = $post->ID;
 
-        if( 'publish'== $old_status || 'course' != get_post_type( $course_id ) || 'auto-draft' == get_post_status( $course_id )
-            || 'trash' == get_post_status( $course_id ) || 'draft' == get_post_status( $course_id ) ) {
+        if ('publish' == $old_status || 'course' != get_post_type($course_id) || 'auto-draft' == get_post_status($course_id)
+            || 'trash' == get_post_status($course_id) || 'draft' == get_post_status($course_id)) {
 
             return false;
 
@@ -863,19 +863,19 @@ 
          *
          * @param bool $on default true
          */
-        if( ! apply_filters('sensei_notify_admin_new_course_creation', true ) ){
+        if ( ! apply_filters('sensei_notify_admin_new_course_creation', true)) {
             return false;
         }
 
         // setting up the data needed by the email template
         global $sensei_email_data;
         $template = 'admin-teacher-new-course-created';
-        $course = get_post( $course_id );
-        $teacher = new WP_User( $course->post_author );
+        $course = get_post($course_id);
+        $teacher = new WP_User($course->post_author);
         $recipient = get_option('admin_email', true);
 
         // don't send if the course is created by admin
-        if( $recipient == $teacher->user_email || current_user_can( 'manage_options' )){
+        if ($recipient == $teacher->user_email || current_user_can('manage_options')) {
             return false;
         }
 
@@ -885,7 +885,7 @@ 
          * @since 1.8.0
          * @param string $template
          */
-        $heading = apply_filters( 'sensei_email_heading', __( 'New course created.', 'woothemes-sensei' ), $template );
+        $heading = apply_filters('sensei_email_heading', __('New course created.', 'woothemes-sensei'), $template);
 
         /**
          * Filter the email subject for the the
@@ -896,11 +896,11 @@ 
          * @param string $template
          */
         $subject = apply_filters('sensei_email_subject',
-                                '['. get_bloginfo( 'name', 'display' ) .'] '. __( 'New course created by', 'woothemes-sensei' ) . ' ' . $teacher->display_name ,
-                                $template );
+                                '['.get_bloginfo('name', 'display').'] '.__('New course created by', 'woothemes-sensei').' '.$teacher->display_name,
+                                $template);
 
         //course edit link
-        $course_edit_link = admin_url('post.php?post=' . $course_id . '&action=edit' );
+        $course_edit_link = admin_url('post.php?post='.$course_id.'&action=edit');
 
         // Construct data array
         $email_data = array(
@@ -919,10 +919,10 @@ 
          * @param array $email_data
          * @param string $template
          */
-        $sensei_email_data = apply_filters( 'sensei_email_data', $email_data , $template );
+        $sensei_email_data = apply_filters('sensei_email_data', $email_data, $template);
 
         // Send mail
-        Sensei()->emails->send( $recipient, $subject , Sensei()->emails->get_content( $template ) );
+        Sensei()->emails->send($recipient, $subject, Sensei()->emails->get_content($template));
 
     }// end notify admin of course creation
 
@@ -933,10 +933,10 @@ 
      * @param array $args WP_User_Query arguments
      * @return array $learners_query_results
      */
-    public function limit_analysis_learners( $args ){
+    public function limit_analysis_learners($args) {
 
         // show default for none teachers
-        if( ! Sensei()->teacher->is_admin_teacher() ) {
+        if ( ! Sensei()->teacher->is_admin_teacher()) {
                 return $args;
         }
 
@@ -945,31 +945,31 @@ 
         $teacher_courses = Sensei()->course->get_all_courses();
 
         // if the user has no courses they should see no users
-        if( empty( $teacher_courses ) ||  ! is_array( $teacher_courses ) ){
+        if (empty($teacher_courses) || ! is_array($teacher_courses)) {
             // tell the query to return 0 students
-            $args[ 'include'] = array( 0 );
+            $args['include'] = array(0);
             return $args;
 
         }
 
         $learner_ids_for_teacher_courses = array();
-        foreach( $teacher_courses as $course ){
+        foreach ($teacher_courses as $course) {
 
             $course_learner_ids = array();
-            $activity_comments =  Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $course->ID, 'type' => 'sensei_course_status', 'field' => 'user_id' ), true );
+            $activity_comments = Sensei_Utils::sensei_check_for_activity(array('post_id' => $course->ID, 'type' => 'sensei_course_status', 'field' => 'user_id'), true);
 
-            if( empty( $activity_comments ) ||  ( is_array( $activity_comments  ) && ! ( count( $activity_comments ) > 0 ) ) ){
+            if (empty($activity_comments) || (is_array($activity_comments) && ! (count($activity_comments) > 0))) {
                 continue; // skip to the next course as there are no users on this course
             }
 
             // it could be an array of comments or a single comment
-            if( is_array( $activity_comments ) ){
+            if (is_array($activity_comments)) {
 
-                foreach( $activity_comments as $comment ){
+                foreach ($activity_comments as $comment) {
 
-                    $user = get_userdata( $comment->user_id );
+                    $user = get_userdata($comment->user_id);
 
-                    if( empty( $user ) ){
+                    if (empty($user)) {
                         // next comment in this array
                         continue;
                     }
@@ -977,26 +977,26 @@ 
                     $course_learner_ids[] = $user->ID;
                 }
 
-            }else{
+            } else {
 
-                $user = get_userdata( $activity_comments->user_id );
+                $user = get_userdata($activity_comments->user_id);
                 $course_learner_ids[] = $user->ID;
 
             }
 
             // add learners on this course to the all courses learner list
-            $learner_ids_for_teacher_courses = array_merge( $learner_ids_for_teacher_courses, $course_learner_ids );
+            $learner_ids_for_teacher_courses = array_merge($learner_ids_for_teacher_courses, $course_learner_ids);
 
         }
 
         // if there are no students taking the courses by this teacher don't show them any of the other users
-        if( empty( $learner_ids_for_teacher_courses ) ){
+        if (empty($learner_ids_for_teacher_courses)) {
 
-            $args[ 'include'] = array( 0 );
+            $args['include'] = array(0);
 
-        }else{
+        } else {
 
-            $args[ 'include'] = $learner_ids_for_teacher_courses;
+            $args['include'] = $learner_ids_for_teacher_courses;
 
         }
 
@@ -1013,35 +1013,35 @@ 
      * @param $questions
      * @return mixed
      */
-    public function allow_teacher_access_to_questions( $questions, $quiz_id ){
+    public function allow_teacher_access_to_questions($questions, $quiz_id) {
 
-        if( ! $this->is_admin_teacher() ){
+        if ( ! $this->is_admin_teacher()) {
             return $questions;
         }
 
         $screen = get_current_screen();
 
         // don't run this filter within this functions call to Sensei()->lesson->lesson_quiz_questions
-        remove_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20 );
+        remove_filter('sensei_lesson_quiz_questions', array($this, 'allow_teacher_access_to_questions'), 20);
 
-        if( ! empty( $screen ) && 'lesson'== $screen->post_type ){
+        if ( ! empty($screen) && 'lesson' == $screen->post_type) {
 
             $admin_user = get_user_by('email', get_bloginfo('admin_email'));
-            if( ! empty($admin_user) ){
+            if ( ! empty($admin_user)) {
 
                 $current_teacher_id = get_current_user_id();
 
                 // set current user to admin so teacher can view all questions
-                wp_set_current_user( $admin_user->ID  );
-                $questions = Sensei()->lesson->lesson_quiz_questions( $quiz_id  );
+                wp_set_current_user($admin_user->ID);
+                $questions = Sensei()->lesson->lesson_quiz_questions($quiz_id);
 
                 // set the teacher as the current use again
-                wp_set_current_user( $current_teacher_id );
+                wp_set_current_user($current_teacher_id);
             }
 
         }
         // attach the filter again for other funtion calls to Sensei()->lesson->lesson_quiz_questions
-        add_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20,2 );
+        add_filter('sensei_lesson_quiz_questions', array($this, 'allow_teacher_access_to_questions'), 20, 2);
 
         return $questions;
     }
@@ -1053,30 +1053,30 @@ 
      * @param $wp_query
      * @return mixed
      */
-    public function give_access_to_all_questions( $wp_query ){
+    public function give_access_to_all_questions($wp_query) {
 
-        if( ! $this->is_admin_teacher() || !function_exists( 'get_current_screen') || 'question' != $wp_query->get('post_type') ){
+        if ( ! $this->is_admin_teacher() || ! function_exists('get_current_screen') || 'question' != $wp_query->get('post_type')) {
 
             return $wp_query;
         }
 
         $screen = get_current_screen();
-        if( ( isset($screen->id) && 'lesson' == $screen->id )
-            || ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ){
+        if ((isset($screen->id) && 'lesson' == $screen->id)
+            || (defined('DOING_AJAX') && DOING_AJAX)) {
 
             $admin_user = get_user_by('email', get_bloginfo('admin_email'));
-            if( ! empty($admin_user) ){
+            if ( ! empty($admin_user)) {
 
                 $current_teacher_id = get_current_user_id();
 
                 // set current user to admin so teacher can view all questions
-                wp_set_current_user( $admin_user->ID  );
+                wp_set_current_user($admin_user->ID);
 
                 //run new query as admin
-                $wp_query = new WP_Query( $wp_query->query );
+                $wp_query = new WP_Query($wp_query->query);
 
                 //set the teache as current use again
-                wp_set_current_user( $current_teacher_id );
+                wp_set_current_user($current_teacher_id);
 
             }
         }
@@ -1093,7 +1093,7 @@ 
      */
     public function course_column_heading($columns) {
 
-        if( $this->is_admin_teacher() ){
+        if ($this->is_admin_teacher()) {
             return $columns;
         }
         $new_columns = array(
@@ -1110,20 +1110,20 @@ 
      * @param $column
      * @param $course_id
      */
-    public function course_column_data( $column, $course_id  ){
+    public function course_column_data($column, $course_id) {
 
-        if( $this->is_admin_teacher() || 'teacher' != $column  ){
+        if ($this->is_admin_teacher() || 'teacher' != $column) {
             return;
         }
 
-        $course = get_post( $course_id );
-        $teacher = get_userdata( $course->post_author );
+        $course = get_post($course_id);
+        $teacher = get_userdata($course->post_author);
 
-        if( !$teacher ){
+        if ( ! $teacher) {
             return;
         }
 
-        echo '<a href="'. get_edit_user_link( $teacher->ID ) .'" >'. $teacher->display_name.'</a>';
+        echo '<a href="'.get_edit_user_link($teacher->ID).'" >'.$teacher->display_name.'</a>';
 
     }// end course_column_ data
 
@@ -1138,31 +1138,31 @@ 
      *
      * @return array $teachers_courses
      */
-    public function get_teacher_courses( $teacher_id, $return_ids_only= false){
+    public function get_teacher_courses($teacher_id, $return_ids_only = false) {
 
         $teachers_courses = array();
 
-        if( empty( $teacher_id  ) ){
+        if (empty($teacher_id)) {
             $teacher_id = get_current_user_id();
         }
 
         $all_courses = Sensei()->course->get_all_courses();
 
-        if( empty( $all_courses ) ){
+        if (empty($all_courses)) {
             return $all_courses;
         }
 
-        foreach( $all_courses as $course ){
+        foreach ($all_courses as $course) {
 
-            if( $course->post_author != $teacher_id  ){
+            if ($course->post_author != $teacher_id) {
                 continue;
             }
 
-            if( $return_ids_only ){
+            if ($return_ids_only) {
 
                 $teachers_courses[] = $course->ID;
 
-            }else{
+            } else {
 
                 $teachers_courses[] = $course;
 
@@ -1182,21 +1182,21 @@ 
      * @param $query
      * @return mixed
      */
-    public function limit_edit_messages_query( $query ){
-        if( ! $this->is_admin_teacher() || 'sensei_message' != $query->get('post_type') ){
+    public function limit_edit_messages_query($query) {
+        if ( ! $this->is_admin_teacher() || 'sensei_message' != $query->get('post_type')) {
             return $query;
         }
 
         $teacher = wp_get_current_user();
 
-        $query->set( 'meta_key', '_receiver' );
+        $query->set('meta_key', '_receiver');
         $meta_query_args = array(
             'key'     => '_receiver',
-            'value'   => $teacher->get('user_login') ,
+            'value'   => $teacher->get('user_login'),
             'compare' => '='
         );
 
-        $query->set('meta_query', $meta_query_args  );
+        $query->set('meta_query', $meta_query_args);
 
         return $query;
     }
@@ -1212,7 +1212,7 @@ 
     public function course_teacher_filter_options() {
         global $typenow;
 
-        if( ! is_admin() || 'course' != $typenow || ! current_user_can('manage_sensei') ) {
+        if ( ! is_admin() || 'course' != $typenow || ! current_user_can('manage_sensei')) {
             return;
         }
 
@@ -1222,31 +1222,31 @@ 
         // get roles with the course edit capability
         // and then get the users with those roles
         $users_who_can_edit_courses = array();
-        foreach( $roles as $role_item ){
+        foreach ($roles as $role_item) {
 
-            $role = get_role( strtolower( $role_item['name'] ) );
+            $role = get_role(strtolower($role_item['name']));
 
-            if( is_a( $role, 'WP_Role' ) && $role->has_cap('edit_courses') ){
+            if (is_a($role, 'WP_Role') && $role->has_cap('edit_courses')) {
 
-                $user_query_args = array( 'role' => $role->name, 'fields' => array( 'ID', 'display_name' ) );
-                $role_users_who_can_edit_courses = get_users( $user_query_args );
+                $user_query_args = array('role' => $role->name, 'fields' => array('ID', 'display_name'));
+                $role_users_who_can_edit_courses = get_users($user_query_args);
 
                 // add user from the current $user_role to all users
-                $users_who_can_edit_courses = array_merge( $users_who_can_edit_courses, $role_users_who_can_edit_courses );
+                $users_who_can_edit_courses = array_merge($users_who_can_edit_courses, $role_users_who_can_edit_courses);
 
             }
 
         }
 
         // Create the select element with the given users who can edit course
-        $selected = isset( $_GET['course_teacher'] ) ? $_GET['course_teacher'] : '';
+        $selected = isset($_GET['course_teacher']) ? $_GET['course_teacher'] : '';
         $course_options = '';
-        foreach( $users_who_can_edit_courses as $user ) {
-            $course_options .= '<option value="' . esc_attr( $user->ID ) . '" ' . selected( $selected, $user->ID, false ) . '>' .  $user->display_name . '</option>';
+        foreach ($users_who_can_edit_courses as $user) {
+            $course_options .= '<option value="'.esc_attr($user->ID).'" '.selected($selected, $user->ID, false).'>'.$user->display_name.'</option>';
         }
 
         $output = '<select name="course_teacher" id="dropdown_course_teachers">';
-        $output .= '<option value="">'.__( 'Show all teachers', 'woothemes-sensei' ).'</option>';
+        $output .= '<option value="">'.__('Show all teachers', 'woothemes-sensei').'</option>';
         $output .= $course_options;
         $output .= '</select>';
 
@@ -1261,15 +1261,15 @@ 
      * @param $query
      * @return $query
      */
-    public function teacher_filter_query_modify( $query ){
+    public function teacher_filter_query_modify($query) {
         global $typenow;
 
-        if( ! is_admin() && 'course' != $typenow  || ! current_user_can('manage_sensei')  ) {
+        if ( ! is_admin() && 'course' != $typenow || ! current_user_can('manage_sensei')) {
             return $query;
         }
-        $course_teacher = isset( $_GET['course_teacher'] ) ? $_GET['course_teacher'] : '';
+        $course_teacher = isset($_GET['course_teacher']) ? $_GET['course_teacher'] : '';
 
-        if( empty( $course_teacher ) ) {
+        if (empty($course_teacher)) {
             return $query;
         }
 
@@ -1282,23 +1282,23 @@ 
      * @param  array $request Default request arguments
      * @return array          Modified request arguments
      */
-    public function restrict_media_library( $request = array() ) {
+    public function restrict_media_library($request = array()) {
 
-        if( ! is_admin() ) {
+        if ( ! is_admin()) {
             return $request;
         }
 
-        if( ! $this->is_admin_teacher() ) {
+        if ( ! $this->is_admin_teacher()) {
             return $request;
         }
 
         $screen = get_current_screen();
 
-        if( in_array( $screen->id, array( 'upload', 'course', 'lesson', 'question' ) ) ) {
-            $teacher = intval( get_current_user_id() );
+        if (in_array($screen->id, array('upload', 'course', 'lesson', 'question'))) {
+            $teacher = intval(get_current_user_id());
 
-            if( $teacher ) {
-                $request['author__in'] = array( $teacher );
+            if ($teacher) {
+                $request['author__in'] = array($teacher);
             }
         }
 
@@ -1310,20 +1310,20 @@ 
      * @param  array $query Default query arguments
      * @return array        Modified query arguments
      */
-    public function restrict_media_library_modal( $query = array() ) {
+    public function restrict_media_library_modal($query = array()) {
 
-        if( ! is_admin() ) {
+        if ( ! is_admin()) {
             return $query;
         }
 
-        if( ! $this->is_admin_teacher() ) {
+        if ( ! $this->is_admin_teacher()) {
             return $query;
         }
 
-        $teacher = intval( get_current_user_id() );
+        $teacher = intval(get_current_user_id());
 
-        if( $teacher ) {
-            $query['author__in'] = array( $teacher );
+        if ($teacher) {
+            $query['author__in'] = array($teacher);
         }
 
         return $query;
@@ -1336,28 +1336,28 @@ 
      *
      * @param int $lesson_id
      */
-    public function update_lesson_teacher( $lesson_id ){
+    public function update_lesson_teacher($lesson_id) {
 
-        if( 'lesson'!= get_post_type() ){
+        if ('lesson' != get_post_type()) {
             return;
         }
 
         // this should only run once per request cycle
-        remove_action( 'save_post',  array( $this, 'update_lesson_teacher' ) );
+        remove_action('save_post', array($this, 'update_lesson_teacher'));
 
-        $course_id = Sensei()->lesson->get_course_id( $lesson_id );
+        $course_id = Sensei()->lesson->get_course_id($lesson_id);
 
-        if(  empty( $course_id ) || ! $course_id ){
+        if (empty($course_id) || ! $course_id) {
             return;
         }
 
-        $course = get_post( $course_id );
+        $course = get_post($course_id);
 
-        $lesson_update_args= array(
-            'ID' => $lesson_id ,
+        $lesson_update_args = array(
+            'ID' => $lesson_id,
             'post_author' => $course->post_author
         );
-        wp_update_post( $lesson_update_args );
+        wp_update_post($lesson_update_args);
 
     } // end update_lesson_teacher
 
@@ -1371,21 +1371,21 @@ 
      * @parameters array $wp_query
      * @return WP_Query $wp_query
      */
-    public function limit_teacher_edit_screen_post_types( $wp_query ) {
+    public function limit_teacher_edit_screen_post_types($wp_query) {
         global $current_user;
 
         //exit early
-        if( ! $this->is_admin_teacher() ){
+        if ( ! $this->is_admin_teacher()) {
             return $wp_query;
         }
 
-        if ( ! function_exists( 'get_current_screen' ) ) {
+        if ( ! function_exists('get_current_screen')) {
             return $wp_query;
         }
 
         $screen = get_current_screen();
 
-        if( empty( $screen ) ){
+        if (empty($screen)) {
             return $wp_query;
         }
 
@@ -1398,10 +1398,10 @@ 
             'lesson_page_lesson-order',
         );
 
-        if(  in_array($screen->id  , $limit_screens ) ) {
+        if (in_array($screen->id, $limit_screens)) {
 
             // set the query author to the current user to only show those those posts
-            $wp_query->set( 'author', $current_user->ID );
+            $wp_query->set('author', $current_user->ID);
         }
 
         return $wp_query;
@@ -1421,7 +1421,7 @@ 
      * @return void
      */
 
-    public function teacher_login_redirect( $user_login, $user  ) {
+    public function teacher_login_redirect($user_login, $user) {
 
         if (user_can($user, 'edit_courses')) {
 
@@ -1471,9 +1471,9 @@ 
          * @param bool $restrict default true
          */
 
-        $restrict = apply_filters('sensei_restrict_posts_menu_page', true );
+        $restrict = apply_filters('sensei_restrict_posts_menu_page', true);
 
-        if ( in_array( 'teacher', (array) $user->roles ) && !current_user_can('delete_posts') && $restrict) {
+        if (in_array('teacher', (array) $user->roles) && ! current_user_can('delete_posts') && $restrict) {
 
             remove_menu_page('edit.php');
 
@@ -1503,11 +1503,11 @@ 
      * @return WP_Comment_Query  $clauses
      */
 
-    public function restrict_comment_moderation ( $clauses ) {
+    public function restrict_comment_moderation($clauses) {
 
         global $pagenow;
 
-        if( self::is_a_teacher( get_current_user_id() ) && $pagenow == "edit-comments.php") {
+        if (self::is_a_teacher(get_current_user_id()) && $pagenow == "edit-comments.php") {
 
             $clauses->query_vars['post_author'] = get_current_user_id();
 
@@ -1524,15 +1524,15 @@ 
      *
      * @return bool
      */
-    public static function is_a_teacher( $user_id ){
+    public static function is_a_teacher($user_id) {
 
         $user = get_user_by('id', $user_id);
 
-        if( isset( $user->roles ) && in_array(  'teacher',  $user->roles )   ){
+        if (isset($user->roles) && in_array('teacher', $user->roles)) {
 
             return true;
 
-        }else{
+        } else {
 
             return false;
 
@@ -1545,14 +1545,14 @@ 
      *
      * @since 1.9.0
      */
-    public static function archive_title(){
+    public static function archive_title() {
 
-        $author = get_user_by( 'id', get_query_var( 'author' ) );
+        $author = get_user_by('id', get_query_var('author'));
         $author_name = $author->display_name;
         ?>
             <h2 class="teacher-archive-title">
 
-                <?php echo sprintf( __( 'All courses by %s', 'woothemes-sensei') , $author_name ); ?>
+                <?php echo sprintf(__('All courses by %s', 'woothemes-sensei'), $author_name); ?>
 
             </h2>
         <?php
@@ -1564,9 +1564,9 @@ 
      *
      * @since 1.9.0
      */
-    public static function remove_course_meta_on_teacher_archive(){
+    public static function remove_course_meta_on_teacher_archive() {
 
-        remove_action('sensei_course_content_inside_before', array( Sensei()->course, 'the_course_meta' ) );
+        remove_action('sensei_course_content_inside_before', array(Sensei()->course, 'the_course_meta'));
 
     }
 

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