woothemes / sensei

templates/single-course.php

Spacing +4 added lines, -4 removed lines

@@ -11,9 +11,9 @@ 
  */
 ?>
 
-<?php  get_sensei_header();  ?>
+<?php  get_sensei_header(); ?>
 
-<article <?php post_class( array( 'course', 'post' ) ); ?>>
+<article <?php post_class(array('course', 'post')); ?>>
 
     <?php
 
@@ -32,7 +32,7 @@ 
      * @hooked Sensei()->message->send_message_link        -  35
      * @hooked Sensei_Course::the_course_video             -  40
      */
-    do_action( 'sensei_single_course_content_inside_before', get_the_ID() );
+    do_action('sensei_single_course_content_inside_before', get_the_ID());
 
     ?>
 
@@ -52,7 +52,7 @@ 
      * @param integer $course_id
      *
      */
-    do_action( 'sensei_single_course_content_inside_after', get_the_ID() );
+    do_action('sensei_single_course_content_inside_after', get_the_ID());
 
     ?>
 </article><!-- .post .single-course -->

widgets/widget-woothemes-sensei-lesson-component.php

Spacing +50 added lines, -50 removed lines

@@ -1,5 +1,5 @@ 
 <?php

-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.

+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.

 

 /**

  * Sensei Lesson Component Widget

@@ -26,22 +26,22 @@ 
 	public function __construct() {

 		/* Widget variable settings. */

 		$this->woo_widget_cssclass = 'widget_sensei_lesson_component';

-		$this->woo_widget_description = __( 'This widget will output a list of the latest Lessons.', 'woothemes-sensei' );

+		$this->woo_widget_description = __('This widget will output a list of the latest Lessons.', 'woothemes-sensei');

 		$this->woo_widget_idbase = 'sensei_lesson_component';

-		$this->woo_widget_title = __( 'Sensei - Lesson Component', 'woothemes-sensei' );

+		$this->woo_widget_title = __('Sensei - Lesson Component', 'woothemes-sensei');

 

 		$this->woo_widget_componentslist = array(

-												'newlessons' => __( 'New Lessons', 'woothemes-sensei' ),

+												'newlessons' => __('New Lessons', 'woothemes-sensei'),

 												);

 

 		/* Widget settings. */

-		$widget_ops = array( 'classname' => $this->woo_widget_cssclass, 'description' => $this->woo_widget_description );

+		$widget_ops = array('classname' => $this->woo_widget_cssclass, 'description' => $this->woo_widget_description);

 

 		/* Widget control settings. */

-		$control_ops = array( 'width' => 250, 'height' => 350, 'id_base' => $this->woo_widget_idbase );

+		$control_ops = array('width' => 250, 'height' => 350, 'id_base' => $this->woo_widget_idbase);

 

 		/* Create the widget. */

-		parent::__construct( $this->woo_widget_idbase, $this->woo_widget_title, $widget_ops, $control_ops );

+		parent::__construct($this->woo_widget_idbase, $this->woo_widget_title, $widget_ops, $control_ops);

 	} // End __construct()

 

 	/**

@@ -51,35 +51,35 @@ 
 	 * @param  array $instance Widget settings for this instance.

 	 * @return void

 	 */

-	public function widget( $args, $instance ) {

+	public function widget($args, $instance) {

 

-        $before_widget = $args[ 'before_widget' ];

-        $before_title  = $args[ 'before_title' ];

-        $after_title   = $args[ 'after_title' ];

-        $after_widget  = $args[ 'after_widget' ];

+        $before_widget = $args['before_widget'];

+        $before_title  = $args['before_title'];

+        $after_title   = $args['after_title'];

+        $after_widget  = $args['after_widget'];

 

-		if ( in_array( $instance['component'], array_keys( $this->woo_widget_componentslist ) ) && ( 'activecourses' == $instance['component'] || 'completedcourses' == $instance['component'] ) && !is_user_logged_in() ) {

+		if (in_array($instance['component'], array_keys($this->woo_widget_componentslist)) && ('activecourses' == $instance['component'] || 'completedcourses' == $instance['component']) && ! is_user_logged_in()) {

 			// No Output

 		} else {

 			/* Our variables from the widget settings. */

-			$title = apply_filters('widget_title', $instance['title'], $instance, $this->id_base );

+			$title = apply_filters('widget_title', $instance['title'], $instance, $this->id_base);

 

 			/* Before widget (defined by themes). */

 			echo $before_widget;

 

 			/* Display the widget title if one was input (before and after defined by themes). */

-			if ( $title ) { echo $before_title . $title . $after_title; }

+			if ($title) { echo $before_title.$title.$after_title; }

 

 			/* Widget content. */

 			// Add actions for plugins/themes to hook onto.

-			do_action( $this->woo_widget_cssclass . '_top' );

+			do_action($this->woo_widget_cssclass.'_top');

 

-			if ( in_array( $instance['component'], array_keys( $this->woo_widget_componentslist ) ) ) {

-				$this->load_component( $instance );

+			if (in_array($instance['component'], array_keys($this->woo_widget_componentslist))) {

+				$this->load_component($instance);

 			}

 

 			// Add actions for plugins/themes to hook onto.

-			do_action( $this->woo_widget_cssclass . '_bottom' );

+			do_action($this->woo_widget_cssclass.'_bottom');

 

 			/* After widget (defined by themes). */

 			echo $after_widget;

@@ -94,17 +94,17 @@ 
 	 * @param  array $old_instance Previous settings.

 	 * @return array               Updated settings.

 	 */

-	public function update ( $new_instance, $old_instance ) {

+	public function update($new_instance, $old_instance) {

 		$instance = $old_instance;

 

 		/* Strip tags for title and name to remove HTML (important for text inputs). */

-		$instance['title'] = strip_tags( $new_instance['title'] );

+		$instance['title'] = strip_tags($new_instance['title']);

 

 		/* The select box is returning a text value, so we escape it. */

-		$instance['component'] = esc_attr( $new_instance['component'] );

+		$instance['component'] = esc_attr($new_instance['component']);

 

 		/* The select box is returning a text value, so we escape it. */

-		$instance['limit'] = esc_attr( $new_instance['limit'] );

+		$instance['limit'] = esc_attr($new_instance['limit']);

 

 

 		return $instance;

@@ -117,7 +117,7 @@ 
 	 * @param  array $instance The settings for this instance.

 	 * @return void

 	 */

-    public function form( $instance ) {

+    public function form($instance) {

 

 		/* Set up some default widget settings. */

 		/* Make sure all keys are added here, even with empty string values. */

@@ -127,26 +127,26 @@ 
 						'limit' => 3

 					);

 

-		$instance = wp_parse_args( (array) $instance, $defaults );

+		$instance = wp_parse_args((array) $instance, $defaults);

 ?>

 		<!-- Widget Title: Text Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>"><?php _e( 'Title (optional):', 'woothemes-sensei' ); ?></label>

-			<input type="text" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>"  value="<?php echo esc_attr( $instance['title'] ); ?>" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" />

+			<label for="<?php echo esc_attr($this->get_field_id('title')); ?>"><?php _e('Title (optional):', 'woothemes-sensei'); ?></label>

+			<input type="text" name="<?php echo esc_attr($this->get_field_name('title')); ?>"  value="<?php echo esc_attr($instance['title']); ?>" class="widefat" id="<?php echo esc_attr($this->get_field_id('title')); ?>" />

 		</p>

 		<!-- Widget Component: Select Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'component' ) ); ?>"><?php _e( 'Component:', 'woothemes-sensei' ); ?></label>

-			<select name="<?php echo esc_attr( $this->get_field_name( 'component' ) ); ?>" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'component' ) ); ?>">

-			<?php foreach ( $this->woo_widget_componentslist as $k => $v ) { ?>

-				<option value="<?php echo esc_attr( $k ); ?>"<?php selected( $instance['component'], $k ); ?>><?php echo $v; ?></option>

+			<label for="<?php echo esc_attr($this->get_field_id('component')); ?>"><?php _e('Component:', 'woothemes-sensei'); ?></label>

+			<select name="<?php echo esc_attr($this->get_field_name('component')); ?>" class="widefat" id="<?php echo esc_attr($this->get_field_id('component')); ?>">

+			<?php foreach ($this->woo_widget_componentslist as $k => $v) { ?>

+				<option value="<?php echo esc_attr($k); ?>"<?php selected($instance['component'], $k); ?>><?php echo $v; ?></option>

 			<?php } ?>

 			</select>

 		</p>

 		<!-- Widget Limit: Text Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'limit' ) ); ?>"><?php _e( 'Number of Lessons (optional):', 'woothemes-sensei' ); ?></label>

-			<input type="text" name="<?php echo esc_attr( $this->get_field_name( 'limit' ) ); ?>"  value="<?php echo esc_attr( $instance['limit'] ); ?>" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'limit' ) ); ?>" />

+			<label for="<?php echo esc_attr($this->get_field_id('limit')); ?>"><?php _e('Number of Lessons (optional):', 'woothemes-sensei'); ?></label>

+			<input type="text" name="<?php echo esc_attr($this->get_field_name('limit')); ?>"  value="<?php echo esc_attr($instance['limit']); ?>" class="widefat" id="<?php echo esc_attr($this->get_field_id('limit')); ?>" />

 		</p>

 

 <?php

@@ -158,7 +158,7 @@ 
 	 * @since  5.0.8

 	 * @return void

 	 */

-	protected function load_component ( $instance ) {

+	protected function load_component($instance) {

 

 		global  $current_user;

 		// Get User Meta

@@ -169,41 +169,41 @@ 
 

 		$posts_array = array();

 

-		$post_args = array(	'post_type' 		=> 'lesson',

-							'posts_per_page' 		=> intval( $instance[ 'limit' ] ),

+		$post_args = array('post_type' 		=> 'lesson',

+							'posts_per_page' 		=> intval($instance['limit']),

 							'orderby'         	=> 'menu_order date',

     						'order'           	=> 'DESC',

     						'post_status'       => 'publish',

 							'suppress_filters' 	=> 0

 							);

-		$posts_array = get_posts( $post_args );

+		$posts_array = get_posts($post_args);

 

-		if ( count( $posts_array ) > 0 ) { ?>

+		if (count($posts_array) > 0) { ?>

 			<ul>

-			<?php foreach ($posts_array as $post_item){

-		    	$post_id = absint( $post_item->ID );

+			<?php foreach ($posts_array as $post_item) {

+		    	$post_id = absint($post_item->ID);

 		    	$post_title = $post_item->post_title;

-		    	$user_info = get_userdata( absint( $post_item->post_author ) );

-		    	$author_link = get_author_posts_url( absint( $post_item->post_author ) );

+		    	$user_info = get_userdata(absint($post_item->post_author));

+		    	$author_link = get_author_posts_url(absint($post_item->post_author));

 		    	$author_display_name = $user_info->display_name;

 		    	$author_id = $post_item->post_author;

-		    	$lesson_course_id = get_post_meta( $post_id, '_lesson_course', true );

+		    	$lesson_course_id = get_post_meta($post_id, '_lesson_course', true);

 		    ?>

 		    	<li class="fix">

-		    		<?php do_action( 'sensei_lesson_image', $post_id, '100', '100', false, true ); ?>

-		    		<a href="<?php echo esc_url( get_permalink( $post_id ) ); ?>" title="<?php echo esc_attr( $post_title ); ?>"><?php echo $post_title; ?></a>

+		    		<?php do_action('sensei_lesson_image', $post_id, '100', '100', false, true); ?>

+		    		<a href="<?php echo esc_url(get_permalink($post_id)); ?>" title="<?php echo esc_attr($post_title); ?>"><?php echo $post_title; ?></a>

 		    		<br />

-		    		<?php if ( isset( Sensei()->settings->settings[ 'lesson_author' ] ) && ( Sensei()->settings->settings[ 'lesson_author' ] ) ) { ?>

-    					<span class="course-author"><?php _e( 'by ', 'woothemes-sensei' ); ?><a href="<?php echo esc_url( $author_link ); ?>" title="<?php echo esc_attr( $author_display_name ); ?>"><?php echo esc_html( $author_display_name ); ?></a></span>

+		    		<?php if (isset(Sensei()->settings->settings['lesson_author']) && (Sensei()->settings->settings['lesson_author'])) { ?>

+    					<span class="course-author"><?php _e('by ', 'woothemes-sensei'); ?><a href="<?php echo esc_url($author_link); ?>" title="<?php echo esc_attr($author_display_name); ?>"><?php echo esc_html($author_display_name); ?></a></span>

     					<br />

     				<?php } // End If Statement ?>

-    				<?php if ( 0 < $lesson_course_id ) { ?>

-                        <span class="lesson-course"><?php echo ' ' . sprintf( __( 'Part of: %s', 'woothemes-sensei' ), '<a href="' . esc_url( get_permalink( $lesson_course_id ) ) . '" title="' . esc_attr( __( 'View course', 'woothemes-sensei' ) ) . '"><em>' . get_the_title( $lesson_course_id ) . '</em></a>' ); ?></span>

+    				<?php if (0 < $lesson_course_id) { ?>

+                        <span class="lesson-course"><?php echo ' '.sprintf(__('Part of: %s', 'woothemes-sensei'), '<a href="'.esc_url(get_permalink($lesson_course_id)).'" title="'.esc_attr(__('View course', 'woothemes-sensei')).'"><em>'.get_the_title($lesson_course_id).'</em></a>'); ?></span>

                     <?php } ?>

     				<br />

 		    	</li>

 		    <?php } // End For Loop ?>

-		    <?php echo '<li class="my-account fix"><a class="button" href="'. esc_url( get_post_type_archive_link( 'lesson' ) ) .'">'.__('More Lessons', 'woothemes-sensei').'</a></li>'; ?>

+		    <?php echo '<li class="my-account fix"><a class="button" href="'.esc_url(get_post_type_archive_link('lesson')).'">'.__('More Lessons', 'woothemes-sensei').'</a></li>'; ?>

 		</ul>

 		<?php } // End If Statement

 	} // End load_component()

widgets/widget-woothemes-sensei-category-courses.php

Spacing +46 added lines, -46 removed lines

@@ -1,5 +1,5 @@ 
 <?php

-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.

+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.

 

 /**

  * Sensei Category Courses Widget

@@ -26,18 +26,18 @@ 
 	public function __construct() {

 		/* Widget variable settings. */

 		$this->woo_widget_cssclass = 'widget_sensei_category_courses';

-		$this->woo_widget_description = __( 'This widget will output a list of Courses for a specific category.', 'woothemes-sensei' );

+		$this->woo_widget_description = __('This widget will output a list of Courses for a specific category.', 'woothemes-sensei');

 		$this->woo_widget_idbase = 'sensei_category_courses';

-		$this->woo_widget_title = __( 'Sensei - Category Courses', 'woothemes-sensei' );

+		$this->woo_widget_title = __('Sensei - Category Courses', 'woothemes-sensei');

 

 		/* Widget settings. */

-		$widget_ops = array( 'classname' => $this->woo_widget_cssclass, 'description' => $this->woo_widget_description );

+		$widget_ops = array('classname' => $this->woo_widget_cssclass, 'description' => $this->woo_widget_description);

 

 		/* Widget control settings. */

-		$control_ops = array( 'width' => 250, 'height' => 350, 'id_base' => $this->woo_widget_idbase );

+		$control_ops = array('width' => 250, 'height' => 350, 'id_base' => $this->woo_widget_idbase);

 

 		/* Create the widget. */

-		parent::__construct( $this->woo_widget_idbase, $this->woo_widget_title, $widget_ops, $control_ops );

+		parent::__construct($this->woo_widget_idbase, $this->woo_widget_title, $widget_ops, $control_ops);

 

 	} // End __construct()

 

@@ -48,32 +48,32 @@ 
 	 * @param  array $instance Widget settings for this instance.

 	 * @return void

 	 */

-	public function widget( $args, $instance ) {

+	public function widget($args, $instance) {

 

-        $before_widget = $args[ 'before_widget' ];

-        $before_title  = $args[ 'before_title' ];

-        $after_title   = $args[ 'after_title' ];

-        $after_widget  = $args[ 'after_widget' ];

+        $before_widget = $args['before_widget'];

+        $before_title  = $args['before_title'];

+        $after_title   = $args['after_title'];

+        $after_widget  = $args['after_widget'];

 

 		/* Our variables from the widget settings. */

-		$title = apply_filters('widget_title', $instance['title'], $instance, $this->id_base );

+		$title = apply_filters('widget_title', $instance['title'], $instance, $this->id_base);

 

 		/* Before widget (defined by themes). */

 		echo $before_widget;

 

 		/* Display the widget title if one was input (before and after defined by themes). */

-		if ( $title ) { echo $before_title . $title . $after_title; }

+		if ($title) { echo $before_title.$title.$after_title; }

 

 		/* Widget content. */

 		// Add actions for plugins/themes to hook onto.

-		do_action( $this->woo_widget_cssclass . '_top' );

+		do_action($this->woo_widget_cssclass.'_top');

 

-		if ( 0 < intval( $instance['course_category'] ) ) {

-			$this->load_component( $instance );

+		if (0 < intval($instance['course_category'])) {

+			$this->load_component($instance);

 		} // End If Statement

 

 		// Add actions for plugins/themes to hook onto.

-		do_action( $this->woo_widget_cssclass . '_bottom' );

+		do_action($this->woo_widget_cssclass.'_bottom');

 

 		/* After widget (defined by themes). */

 		echo $after_widget;

@@ -87,17 +87,17 @@ 
 	 * @param  array $old_instance Previous settings.

 	 * @return array               Updated settings.

 	 */

-	public function update ( $new_instance, $old_instance ) {

+	public function update($new_instance, $old_instance) {

 		$instance = $old_instance;

 

 		/* Strip tags for title and name to remove HTML (important for text inputs). */

-		$instance['title'] = strip_tags( $new_instance['title'] );

+		$instance['title'] = strip_tags($new_instance['title']);

 

 		/* The select box is returning a text value, so we escape it. */

-		$instance['course_category'] = esc_attr( $new_instance['course_category'] );

+		$instance['course_category'] = esc_attr($new_instance['course_category']);

 

 		/* Strip tags for limit to remove HTML (important for text inputs). */

-		$instance['limit'] = strip_tags( $new_instance['limit'] );

+		$instance['limit'] = strip_tags($new_instance['limit']);

 

 

 		return $instance;

@@ -110,7 +110,7 @@ 
 	 * @param  array $instance The settings for this instance.

 	 * @return void

 	 */

-    public function form( $instance ) {

+    public function form($instance) {

 

 		/* Set up some default widget settings. */

 		/* Make sure all keys are added here, even with empty string values. */

@@ -120,25 +120,25 @@ 
 						'limit' => 3

 					);

 

-		$instance = wp_parse_args( (array) $instance, $defaults );

+		$instance = wp_parse_args((array) $instance, $defaults);

 ?>

 		<!-- Widget Title: Text Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>"><?php _e( 'Title (optional):', 'woothemes-sensei' ); ?></label>

-			<input type="text" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>"  value="<?php echo esc_attr( $instance['title'] ); ?>" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" />

+			<label for="<?php echo esc_attr($this->get_field_id('title')); ?>"><?php _e('Title (optional):', 'woothemes-sensei'); ?></label>

+			<input type="text" name="<?php echo esc_attr($this->get_field_name('title')); ?>"  value="<?php echo esc_attr($instance['title']); ?>" class="widefat" id="<?php echo esc_attr($this->get_field_id('title')); ?>" />

 		</p>

 		<!-- Widget Course Category: Select Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'course_category' ) ); ?>"><?php _e( 'Course Category:', 'woothemes-sensei' ); ?></label>

+			<label for="<?php echo esc_attr($this->get_field_id('course_category')); ?>"><?php _e('Course Category:', 'woothemes-sensei'); ?></label>

 			<?php

-			$cat_args = array( 'hierarchical' => true, 'show_option_none' => __( 'Select Category:', 'woothemes-sensei' ), 'taxonomy' => 'course-category', 'orderby' => 'name', 'selected' => intval( $instance['course_category'] ), 'id' => $this->get_field_id( 'course_category' ), 'name' => $this->get_field_name( 'course_category' ), 'class' => 'widefat' );

+			$cat_args = array('hierarchical' => true, 'show_option_none' => __('Select Category:', 'woothemes-sensei'), 'taxonomy' => 'course-category', 'orderby' => 'name', 'selected' => intval($instance['course_category']), 'id' => $this->get_field_id('course_category'), 'name' => $this->get_field_name('course_category'), 'class' => 'widefat');

 			wp_dropdown_categories(apply_filters('widget_course_categories_dropdown_args', $cat_args));

 			?>

 		</p>

 		<!-- Widget Limit: Text Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'limit' ) ); ?>"><?php _e( 'Number of Courses (optional):', 'woothemes-sensei' ); ?></label>

-			<input type="text" name="<?php echo esc_attr( $this->get_field_name( 'limit' ) ); ?>"  value="<?php echo esc_attr( $instance['limit'] ); ?>" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'limit' ) ); ?>" />

+			<label for="<?php echo esc_attr($this->get_field_id('limit')); ?>"><?php _e('Number of Courses (optional):', 'woothemes-sensei'); ?></label>

+			<input type="text" name="<?php echo esc_attr($this->get_field_name('limit')); ?>"  value="<?php echo esc_attr($instance['limit']); ?>" class="widefat" id="<?php echo esc_attr($this->get_field_id('limit')); ?>" />

 		</p>

 

 <?php

@@ -150,49 +150,49 @@ 
 	 * @since  1.1.0

 	 * @return void

 	 */

-	protected function load_component ( $instance ) {

+	protected function load_component($instance) {

 

 

 		$posts_array = array();

-		$post_args = array(	'post_type' 		=> 'course',

-							'posts_per_page' 		=> intval( $instance[ 'limit' ] ),

+		$post_args = array('post_type' 		=> 'course',

+							'posts_per_page' 		=> intval($instance['limit']),

 							'orderby'         	=> 'menu_order date',

     						'order'           	=> 'ASC',

     						'post_status'       => 'publish',

 							'suppress_filters' 	=> 0,

 							);

 

-		$post_args[ 'tax_query' ] = array(

+		$post_args['tax_query'] = array(

 											array(

 												'taxonomy' => 'course-category',

 												'field' => 'id',

-												'terms' => intval( $instance['course_category'] )

+												'terms' => intval($instance['course_category'])

 											)

 										);

 

-		$posts_array = get_posts( $post_args );

+		$posts_array = get_posts($post_args);

 

-		if ( count( $posts_array ) > 0 ) { ?>

+		if (count($posts_array) > 0) { ?>

 			<ul>

-			<?php foreach ($posts_array as $post_item){

-		    	$post_id = absint( $post_item->ID );

+			<?php foreach ($posts_array as $post_item) {

+		    	$post_id = absint($post_item->ID);

 		    	$post_title = $post_item->post_title;

-		    	$user_info = get_userdata( absint( $post_item->post_author ) );

-		    	$author_link = get_author_posts_url( absint( $post_item->post_author ) );

+		    	$user_info = get_userdata(absint($post_item->post_author));

+		    	$author_link = get_author_posts_url(absint($post_item->post_author));

 		    	$author_display_name = $user_info->display_name;

 		    	$author_id = $post_item->post_author;

 		    ?>

 		    	<li class="fix">

-		    		<?php do_action( 'sensei_course_image', $post_id ); ?>

-		    		<a href="<?php echo esc_url( get_permalink( $post_id ) ); ?>" title="<?php echo esc_attr( $post_title ); ?>"><?php echo $post_title; ?></a>

+		    		<?php do_action('sensei_course_image', $post_id); ?>

+		    		<a href="<?php echo esc_url(get_permalink($post_id)); ?>" title="<?php echo esc_attr($post_title); ?>"><?php echo $post_title; ?></a>

 		    		<br />

-		    		<?php if ( isset( Sensei()->settings->settings[ 'course_author' ] ) && ( Sensei()->settings->settings[ 'course_author' ] ) ) { ?>

-    					<span class="course-author"><?php _e( 'by ', 'woothemes-sensei' ); ?><a href="<?php echo esc_url( $author_link ); ?>" title="<?php echo esc_attr( $author_display_name ); ?>"><?php echo esc_html( $author_display_name ); ?></a></span>

+		    		<?php if (isset(Sensei()->settings->settings['course_author']) && (Sensei()->settings->settings['course_author'])) { ?>

+    					<span class="course-author"><?php _e('by ', 'woothemes-sensei'); ?><a href="<?php echo esc_url($author_link); ?>" title="<?php echo esc_attr($author_display_name); ?>"><?php echo esc_html($author_display_name); ?></a></span>

     					<br />

     				<?php } // End If Statement ?>

-    				<span class="course-lesson-count"><?php echo Sensei()->course->course_lesson_count( $post_id ) . '&nbsp;' .  __( 'Lessons', 'woothemes-sensei' ); ?></span>

+    				<span class="course-lesson-count"><?php echo Sensei()->course->course_lesson_count($post_id).'&nbsp;'.__('Lessons', 'woothemes-sensei'); ?></span>

     				<br />

-    				<?php sensei_simple_course_price( $post_id ); ?>

+    				<?php sensei_simple_course_price($post_id); ?>

 		    	</li>

 		    <?php } // End For Loop ?>

 			</ul>

widgets/widget-woothemes-sensei-course-categories.php

Spacing +37 added lines, -37 removed lines

@@ -1,5 +1,5 @@ 
 <?php

-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly.

+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly.

 

 /**

  * Sensei Course Categories Widget

@@ -26,18 +26,18 @@ 
 	public function __construct() {

 		/* Widget variable settings. */

 		$this->woo_widget_cssclass = 'widget_sensei_course_categories';

-		$this->woo_widget_description = __( 'This widget will output a list of Course Categories.', 'woothemes-sensei' );

+		$this->woo_widget_description = __('This widget will output a list of Course Categories.', 'woothemes-sensei');

 		$this->woo_widget_idbase = 'sensei_course_categories';

-		$this->woo_widget_title = __( 'Sensei - Course Categories', 'woothemes-sensei' );

+		$this->woo_widget_title = __('Sensei - Course Categories', 'woothemes-sensei');

 

 		/* Widget settings. */

-		$widget_ops = array( 'classname' => $this->woo_widget_cssclass, 'description' => $this->woo_widget_description );

+		$widget_ops = array('classname' => $this->woo_widget_cssclass, 'description' => $this->woo_widget_description);

 

 		/* Widget control settings. */

-		$control_ops = array( 'width' => 250, 'height' => 350, 'id_base' => $this->woo_widget_idbase );

+		$control_ops = array('width' => 250, 'height' => 350, 'id_base' => $this->woo_widget_idbase);

 

 		/* Create the widget. */

-		parent::__construct( $this->woo_widget_idbase, $this->woo_widget_title, $widget_ops, $control_ops );

+		parent::__construct($this->woo_widget_idbase, $this->woo_widget_title, $widget_ops, $control_ops);

 	} // End __construct()

 

 	/**

@@ -47,30 +47,30 @@ 
 	 * @param  array $instance Widget settings for this instance.

 	 * @return void

 	 */

-	public function widget( $args, $instance ) {

+	public function widget($args, $instance) {

 

-        $before_widget = $args[ 'before_widget' ];

-        $before_title  = $args[ 'before_title' ];

-        $after_title   = $args[ 'after_title' ];

-        $after_widget  = $args[ 'after_widget' ];

+        $before_widget = $args['before_widget'];

+        $before_title  = $args['before_title'];

+        $after_title   = $args['after_title'];

+        $after_widget  = $args['after_widget'];

 

         /* Our variables from the widget settings. */

-        $title = apply_filters('widget_title', $instance['title'], $instance, $this->id_base );

+        $title = apply_filters('widget_title', $instance['title'], $instance, $this->id_base);

 

         /* Before widget (defined by themes). */

         echo $before_widget;

 

         /* Display the widget title if one was input (before and after defined by themes). */

-        if ( $title ) { echo $before_title . $title . $after_title; }

+        if ($title) { echo $before_title.$title.$after_title; }

 

         /* Widget content. */

         // Add actions for plugins/themes to hook onto.

-        do_action( $this->woo_widget_cssclass . '_top' );

+        do_action($this->woo_widget_cssclass.'_top');

 

-        $this->load_component( $instance );

+        $this->load_component($instance);

 

         // Add actions for plugins/themes to hook onto.

-        do_action( $this->woo_widget_cssclass . '_bottom' );

+        do_action($this->woo_widget_cssclass.'_bottom');

 

         /* After widget (defined by themes). */

         echo $after_widget;

@@ -84,14 +84,14 @@ 
 	 * @param  array $old_instance Previous settings.

 	 * @return array               Updated settings.

 	 */

-	public function update ( $new_instance, $old_instance ) {

+	public function update($new_instance, $old_instance) {

 		$instance = $old_instance;

 

 		/* Strip tags for title and name to remove HTML (important for text inputs). */

-		$instance['title'] = strip_tags( $new_instance['title'] );

+		$instance['title'] = strip_tags($new_instance['title']);

 

 		/* The select box is returning a text value, so we escape it. */

-		$instance['limit'] = esc_attr( $new_instance['limit'] );

+		$instance['limit'] = esc_attr($new_instance['limit']);

 

 		/* The check box is returning a boolean value. */

 		$instance['count'] = $new_instance['count'];

@@ -107,7 +107,7 @@ 
 	 * @param  array $instance The settings for this instance.

 	 * @return void

 	 */

-    public function form( $instance ) {

+    public function form($instance) {

 

 		/* Set up some default widget settings. */

 		/* Make sure all keys are added here, even with empty string values. */

@@ -118,27 +118,27 @@ 
 						'hierarchical' => false

 					);

 

-		$instance = wp_parse_args( (array) $instance, $defaults );

+		$instance = wp_parse_args((array) $instance, $defaults);

 ?>

 		<!-- Widget Title: Text Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>"><?php _e( 'Title (optional):', 'woothemes-sensei' ); ?></label>

-			<input type="text" name="<?php echo esc_attr( $this->get_field_name( 'title' ) ); ?>"  value="<?php echo esc_attr( $instance['title'] ); ?>" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'title' ) ); ?>" />

+			<label for="<?php echo esc_attr($this->get_field_id('title')); ?>"><?php _e('Title (optional):', 'woothemes-sensei'); ?></label>

+			<input type="text" name="<?php echo esc_attr($this->get_field_name('title')); ?>"  value="<?php echo esc_attr($instance['title']); ?>" class="widefat" id="<?php echo esc_attr($this->get_field_id('title')); ?>" />

 		</p>

 		<!-- Widget Limit: Text Input -->

 		<p>

-			<label for="<?php echo esc_attr( $this->get_field_id( 'limit' ) ); ?>"><?php _e( 'Number of Categories (optional):', 'woothemes-sensei' ); ?></label>

-			<input type="text" name="<?php echo esc_attr( $this->get_field_name( 'limit' ) ); ?>"  value="<?php echo esc_attr( $instance['limit'] ); ?>" class="widefat" id="<?php echo esc_attr( $this->get_field_id( 'limit' ) ); ?>" />

+			<label for="<?php echo esc_attr($this->get_field_id('limit')); ?>"><?php _e('Number of Categories (optional):', 'woothemes-sensei'); ?></label>

+			<input type="text" name="<?php echo esc_attr($this->get_field_name('limit')); ?>"  value="<?php echo esc_attr($instance['limit']); ?>" class="widefat" id="<?php echo esc_attr($this->get_field_id('limit')); ?>" />

 		</p>

 		<!-- Widget Show Count: Checkbox Input -->

 		<p>

-			<input type="checkbox" class="checkbox" id="<?php echo esc_attr( $this->get_field_id('count') ); ?>" name="<?php echo esc_attr( $this->get_field_name('count') ); ?>"<?php checked( $instance['count'], 'on' ); ?> />

-			<label for="<?php echo esc_attr( $this->get_field_id('count') ); ?>"><?php _e( 'Show post counts', 'woothemes-sensei' ); ?></label><br />

+			<input type="checkbox" class="checkbox" id="<?php echo esc_attr($this->get_field_id('count')); ?>" name="<?php echo esc_attr($this->get_field_name('count')); ?>"<?php checked($instance['count'], 'on'); ?> />

+			<label for="<?php echo esc_attr($this->get_field_id('count')); ?>"><?php _e('Show post counts', 'woothemes-sensei'); ?></label><br />

 		</p>

 		<!-- Widget Show Hierarchy: Checkbox Input -->

 		<p>

-			<input type="checkbox" class="checkbox" id="<?php echo esc_attr( $this->get_field_id('hierarchical') ); ?>" name="<?php echo esc_attr( $this->get_field_name('hierarchical') ); ?>"<?php checked( $instance['hierarchical'], 'on' ); ?> />

-			<label for="<?php echo esc_attr( $this->get_field_id('hierarchical') ); ?>"><?php _e( 'Show hierarchy', 'woothemes-sensei' ); ?></label></p>

+			<input type="checkbox" class="checkbox" id="<?php echo esc_attr($this->get_field_id('hierarchical')); ?>" name="<?php echo esc_attr($this->get_field_name('hierarchical')); ?>"<?php checked($instance['hierarchical'], 'on'); ?> />

+			<label for="<?php echo esc_attr($this->get_field_id('hierarchical')); ?>"><?php _e('Show hierarchy', 'woothemes-sensei'); ?></label></p>

 		</p>

 <?php

 	} // End form()

@@ -149,18 +149,18 @@ 
 	 * @since  1.1.0

 	 * @return void

 	 */

-	protected function load_component ( $instance ) {

+	protected function load_component($instance) {

 

-		$limit = intval( $instance['limit'] );

-		$count = isset($instance['count']) ? (bool) $instance['count'] :false;

-		$hierarchical = isset( $instance['hierarchical'] ) ? (bool) $instance['hierarchical'] : false;

+		$limit = intval($instance['limit']);

+		$count = isset($instance['count']) ? (bool) $instance['count'] : false;

+		$hierarchical = isset($instance['hierarchical']) ? (bool) $instance['hierarchical'] : false;

 

-		$cat_args = array( 'title_li' => '', 'taxonomy' => 'course-category', 'orderby' => 'name', 'show_count' => $count, 'hierarchical' => $hierarchical);

-		if ( 0 < $limit ) {

-			$cat_args[ 'number' ] = $limit;

+		$cat_args = array('title_li' => '', 'taxonomy' => 'course-category', 'orderby' => 'name', 'show_count' => $count, 'hierarchical' => $hierarchical);

+		if (0 < $limit) {

+			$cat_args['number'] = $limit;

 		} // End If Statement

 		echo '<ul>';

-			wp_list_categories( apply_filters('widget_course_categories_args', $cat_args) );

+			wp_list_categories(apply_filters('widget_course_categories_args', $cat_args));

 		echo '</ul>';

 	} // End load_component()

 } // End Class
\ No newline at end of file

includes/class-sensei-learner-profiles.php

Spacing +41 added lines, -41 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 learner profiles in Sensei.
@@ -19,20 +19,20 @@ 
 	 * Constructor.
 	 * @since  1.4.0
 	 */
-	public function __construct () {
+	public function __construct() {
 
 		// Setup learner profile URL base
-		$this->profile_url_base = apply_filters( 'sensei_learner_profiles_url_base', __( 'learner', 'woothemes-sensei') );
+		$this->profile_url_base = apply_filters('sensei_learner_profiles_url_base', __('learner', 'woothemes-sensei'));
 
 		// Setup permalink structure for learner profiles
-		add_action( 'init', array( $this, 'setup_permastruct' ) );
-		add_filter( 'wp_title', array( $this, 'page_title' ), 10, 2 );
+		add_action('init', array($this, 'setup_permastruct'));
+		add_filter('wp_title', array($this, 'page_title'), 10, 2);
 
 		// Set heading for courses section of learner profiles
-		add_action( 'sensei_learner_profile_info', array( $this, 'learner_profile_courses_heading' ), 30, 1 );
+		add_action('sensei_learner_profile_info', array($this, 'learner_profile_courses_heading'), 30, 1);
 
 		// Add class to body tag
-		add_filter( 'body_class', array( $this, 'learner_profile_body_class' ), 10, 1 );
+		add_filter('body_class', array($this, 'learner_profile_body_class'), 10, 1);
 	} // End __construct()
 
 	/**
@@ -42,11 +42,11 @@ 
 	 */
 	public function setup_permastruct() {
 
-        if( isset( Sensei()->settings->settings[ 'learner_profile_enable' ] )
-            && Sensei()->settings->settings[ 'learner_profile_enable' ] ) {
+        if (isset(Sensei()->settings->settings['learner_profile_enable'])
+            && Sensei()->settings->settings['learner_profile_enable']) {
 
-			add_rewrite_rule( '^' . $this->profile_url_base . '/([^/]*)/?', 'index.php?learner_profile=$matches[1]', 'top' );
-			add_rewrite_tag( '%learner_profile%', '([^&]+)' );
+			add_rewrite_rule('^'.$this->profile_url_base.'/([^/]*)/?', 'index.php?learner_profile=$matches[1]', 'top');
+			add_rewrite_tag('%learner_profile%', '([^&]+)');
 
 		}
 	}
@@ -57,14 +57,14 @@ 
 	 * @param  string $sep   Seeparator string
 	 * @return string        Modified title
 	 */
-	public function page_title( $title, $sep = null ) {
+	public function page_title($title, $sep = null) {
 		global $wp_query;
-		if( isset( $wp_query->query_vars['learner_profile'] ) ) {
-			$learner_user = get_user_by( 'login', $wp_query->query_vars['learner_profile'] );
+		if (isset($wp_query->query_vars['learner_profile'])) {
+			$learner_user = get_user_by('login', $wp_query->query_vars['learner_profile']);
 
-            $name = Sensei_Learner::get_full_name( $learner_user->ID );
+            $name = Sensei_Learner::get_full_name($learner_user->ID);
 
-			$title = apply_filters( 'sensei_learner_profile_courses_heading', sprintf( __( 'Courses %s is taking', 'woothemes-sensei' ), $name ) ) . ' ' . $sep . ' ';
+			$title = apply_filters('sensei_learner_profile_courses_heading', sprintf(__('Courses %s is taking', 'woothemes-sensei'), $name)).' '.$sep.' ';
 		}
 		return $title;
 	}
@@ -75,23 +75,23 @@ 
 	 * @param  integer $user_id ID of user
 	 * @return string           The learner profile permalink
 	 */
-	public function get_permalink( $user_id = 0 ) {
+	public function get_permalink($user_id = 0) {
 		$user = false;
-		if( $user_id == 0 ) {
+		if ($user_id == 0) {
 			global $current_user;
 			wp_get_current_user();
 			$user = $current_user;
 		} else {
-			$user = get_userdata( $user_id );
+			$user = get_userdata($user_id);
 		}
 
 		$permalink = '';
 
-		if( $user ) {
-			if ( get_option('permalink_structure') ) {
-				$permalink = trailingslashit( get_site_url() ) . $this->profile_url_base . '/' . $user->user_nicename;
+		if ($user) {
+			if (get_option('permalink_structure')) {
+				$permalink = trailingslashit(get_site_url()).$this->profile_url_base.'/'.$user->user_nicename;
 			} else {
-				$permalink = trailingslashit( get_site_url() ) . '?learner_profile=' . $user->user_nicename;
+				$permalink = trailingslashit(get_site_url()).'?learner_profile='.$user->user_nicename;
 			}
 		}
 
@@ -104,13 +104,13 @@ 
 	 * @return void
 	 */
 	public function content() {
-		global $wp_query,  $learner_user, $current_user;
+		global $wp_query, $learner_user, $current_user;
 
-		if( isset( Sensei()->settings->settings[ 'learner_profile_enable' ] ) && Sensei()->settings->settings[ 'learner_profile_enable' ] ) {
+		if (isset(Sensei()->settings->settings['learner_profile_enable']) && Sensei()->settings->settings['learner_profile_enable']) {
 
-			if( isset( $wp_query->query_vars['learner_profile'] ) ) {
+			if (isset($wp_query->query_vars['learner_profile'])) {
 
-                Sensei_Templates::get_template( 'learner-profile/learner-info.php' );
+                Sensei_Templates::get_template('learner-profile/learner-info.php');
 
 			}
 		}
@@ -122,14 +122,14 @@ 
 	 * @param  object $user Queried user object
 	 * @return void
 	 */
-	public function learner_profile_courses_heading( $user ) {
-		if( strlen( $user->first_name ) > 0 ) {
+	public function learner_profile_courses_heading($user) {
+		if (strlen($user->first_name) > 0) {
 			$name = $user->first_name;
 		} else {
 			$name = $user->display_name;
 		}
-		$name = apply_filters( 'sensei_learner_profile_courses_heading_name', $name );
-		echo '<h2>' . apply_filters( 'sensei_learner_profile_courses_heading', sprintf( __( 'Courses %s is taking', 'woothemes-sensei' ), $name ) ) . '</h2>';
+		$name = apply_filters('sensei_learner_profile_courses_heading_name', $name);
+		echo '<h2>'.apply_filters('sensei_learner_profile_courses_heading', sprintf(__('Courses %s is taking', 'woothemes-sensei'), $name)).'</h2>';
 	}
 
 	/**
@@ -138,14 +138,14 @@ 
 	 * @param  object $user Queried user object
 	 * @return void
 	 */
-	public static function user_info( $user ) {
+	public static function user_info($user) {
 
         /**
          * This hooke fires inside the Sensei_Learner_Profiles::user_info function.
          * just before the htmls is generated.
          * @since 1.0.0
          */
-        do_action( 'sensei_learner_profile_info', $user );
+        do_action('sensei_learner_profile_info', $user);
 
         /**
          * This filter runs inside the Sensei_Learner_Profiles::user_info function.
@@ -155,7 +155,7 @@ 
          *
          * @param false|string `<img>` $user_avatar
          */
-		$learner_avatar = apply_filters( 'sensei_learner_profile_info_avatar', get_avatar( $user->ID, 120 ), $user->ID );
+		$learner_avatar = apply_filters('sensei_learner_profile_info_avatar', get_avatar($user->ID, 120), $user->ID);
 
         /**
          * This filter runs inside the Sensei_Learner_Profiles::user_info function.
@@ -165,7 +165,7 @@ 
          * @param string $user_display_name
          * @param string $user_id
          */
-		$learner_name = apply_filters( 'sensei_learner_profile_info_name', $user->display_name, $user->ID );
+		$learner_name = apply_filters('sensei_learner_profile_info_name', $user->display_name, $user->ID);
 
         /**
          * This filter runs inside the Sensei_Learner_Profiles::user_info function.
@@ -177,7 +177,7 @@ 
          * @param string $user_description
          * @param string $user_id
          */
-		$learner_bio = apply_filters( 'sensei_learner_profile_info_bio', $user->description, $user->ID );
+		$learner_bio = apply_filters('sensei_learner_profile_info_bio', $user->description, $user->ID);
 		?>
 
 		<div id="learner-info">
@@ -188,7 +188,7 @@ 
 
 				<h2><?php echo $learner_name; ?></h2>
 
-				<div class="description"><?php echo wpautop( $learner_bio ); ?></div>
+				<div class="description"><?php echo wpautop($learner_bio); ?></div>
 
 			</div>
 
@@ -202,9 +202,9 @@ 
 	 * @param  array $classes Existing classes
 	 * @return array          Modified classes
 	 */
-	public function learner_profile_body_class( $classes ) {
+	public function learner_profile_body_class($classes) {
 		global $wp_query;
-		if( isset( $wp_query->query_vars['learner_profile'] ) ) {
+		if (isset($wp_query->query_vars['learner_profile'])) {
 			$classes[] = 'learner-profile';
 		}
 		return $classes;
@@ -215,9 +215,9 @@ 
      *
      * @since 1.9.0
      */
-    public static function deprecate_sensei_learner_profile_content_hook(){
+    public static function deprecate_sensei_learner_profile_content_hook() {
 
-        sensei_do_deprecated_action( 'sensei_learner_profile_content', '1.9.0', 'sensei_learner_profile_content_before' );
+        sensei_do_deprecated_action('sensei_learner_profile_content', '1.9.0', 'sensei_learner_profile_content_before');
 
     }
 

includes/class-sensei-grading-main.php

Spacing +130 added lines, -130 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Admin Grading Overview Data Table in Sensei.
@@ -21,7 +21,7 @@ 
 	 * Constructor
 	 * @since  1.3.0
 	 */
-	public function __construct ( $args = null ) {
+	public function __construct($args = null) {
 
 		$defaults = array(
 			'course_id' => 0,
@@ -29,24 +29,24 @@ 
 			'user_id' => false,
 			'view' => 'ungraded',
 		);
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$this->course_id = intval( $args['course_id'] );
-		$this->lesson_id = intval( $args['lesson_id'] );
-		if ( !empty($args['user_id']) ) {
-			$this->user_id = intval( $args['user_id'] );
+		$this->course_id = intval($args['course_id']);
+		$this->lesson_id = intval($args['lesson_id']);
+		if ( ! empty($args['user_id'])) {
+			$this->user_id = intval($args['user_id']);
 		}
 
-		if( !empty( $args['view'] ) && in_array( $args['view'], array( 'in-progress', 'graded', 'ungraded', 'all' ) ) ) {
+		if ( ! empty($args['view']) && in_array($args['view'], array('in-progress', 'graded', 'ungraded', 'all'))) {
 			$this->view = $args['view'];
 		}
 
 		// Load Parent token into constructor
-		parent::__construct( 'grading_main' );
+		parent::__construct('grading_main');
 
 		// 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'));
 	} // End __construct()
 
 	/**
@@ -56,16 +56,16 @@ 
 	 */
 	function get_columns() {
 		$columns = array(
-			'title' => __( 'Learner', 'woothemes-sensei' ),
-			'course' => __( 'Course', 'woothemes-sensei' ),
-			'lesson' => __( 'Lesson', 'woothemes-sensei' ),
-			'updated' => __( 'Updated', 'woothemes-sensei' ),
-			'user_status' => __( 'Status', 'woothemes-sensei' ),
-			'user_grade' => __( 'Grade', 'woothemes-sensei' ),
+			'title' => __('Learner', 'woothemes-sensei'),
+			'course' => __('Course', 'woothemes-sensei'),
+			'lesson' => __('Lesson', 'woothemes-sensei'),
+			'updated' => __('Updated', 'woothemes-sensei'),
+			'user_status' => __('Status', 'woothemes-sensei'),
+			'user_grade' => __('Grade', 'woothemes-sensei'),
 			'action' => '',
 		);
 
-		$columns = apply_filters( 'sensei_grading_default_columns', $columns, $this );
+		$columns = apply_filters('sensei_grading_default_columns', $columns, $this);
 		return $columns;
 	}
 
@@ -76,14 +76,14 @@ 
 	 */
 	function get_sortable_columns() {
 		$columns = array(
-			'title' => array( 'title', false ),
-			'course' => array( 'course', false ),
-			'lesson' => array( 'lesson', false ),
-			'updated' => array( 'updated', false ),
-			'user_status' => array( 'user_status', false ),
-			'user_grade' => array( 'user_grade', false ),
+			'title' => array('title', false),
+			'course' => array('course', false),
+			'lesson' => array('lesson', false),
+			'updated' => array('updated', false),
+			'user_status' => array('user_status', false),
+			'user_grade' => array('user_grade', false),
 		);
-		$columns = apply_filters( 'sensei_grading_default_columns_sortable', $columns, $this );
+		$columns = apply_filters('sensei_grading_default_columns_sortable', $columns, $this);
 		return $columns;
 	}
 
@@ -97,47 +97,47 @@ 
 
 		// 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 = 'DESC';
-		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;
 
 		// Searching users on statuses requires sub-selecting the statuses by user_ids
-		if ( $this->search ) {
+		if ($this->search) {
 			$user_args = array(
-				'search' => '*' . $this->search . '*',
+				'search' => '*'.$this->search.'*',
 				'fields' => 'ID',
 			);
 			// Filter for extending
-			$user_args = apply_filters( 'sensei_grading_search_users', $user_args );
-			if ( !empty( $user_args ) ) {
-				$learners_search = new WP_User_Query( $user_args );
+			$user_args = apply_filters('sensei_grading_search_users', $user_args);
+			if ( ! empty($user_args)) {
+				$learners_search = new WP_User_Query($user_args);
 				// Store for reuse on counts
 				$this->user_ids = $learners_search->get_results();
 			}
 		} // End If Statement
 
-		$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
 
 		$activity_args = array(
@@ -149,27 +149,27 @@ 
 			'status' => 'any',
 		);
 
-		if( $this->lesson_id ) {
+		if ($this->lesson_id) {
 			$activity_args['post_id'] = $this->lesson_id;
 		}
-		elseif( $this->course_id ) {
+		elseif ($this->course_id) {
 			// Currently not possible to restrict to a single Course, as that requires WP_Comment to support multiple
 			// post_ids (i.e. every lesson within the Course), WP 4.1 ( https://core.trac.wordpress.org/changeset/29808 )
-			if ( version_compare($wp_version, '4.1', '>=') ) {
-				$activity_args['post__in'] = Sensei()->course->course_lessons( $this->course_id, 'any', 'ids' );
+			if (version_compare($wp_version, '4.1', '>=')) {
+				$activity_args['post__in'] = Sensei()->course->course_lessons($this->course_id, 'any', 'ids');
 			}
 		}
 		// Sub select to group of learners
-		if ( $this->user_ids ) {
+		if ($this->user_ids) {
 			$activity_args['user_id'] = (array) $this->user_ids;
 		}
 		// Restrict to a single Learner
-		if( $this->user_id ) {
+		if ($this->user_id) {
 			$activity_args['user_id'] = $this->user_id;
 		}
 
 
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'in-progress' :
 				$activity_args['status'] = 'in-progress';
 				break;
@@ -179,7 +179,7 @@ 
 				break;
 
 			case 'graded' :
-				$activity_args['status'] = array( 'graded', 'passed', 'failed' );
+				$activity_args['status'] = array('graded', 'passed', 'failed');
 				break;
 
 			case 'all' :
@@ -188,31 +188,31 @@ 
 				break;
 		} // End switch
 
-		$activity_args = apply_filters( 'sensei_grading_filter_statuses', $activity_args );
+		$activity_args = apply_filters('sensei_grading_filter_statuses', $activity_args);
 
 		// WP_Comment_Query doesn't support SQL_CALC_FOUND_ROWS, so instead do this twice
-		$total_statuses = Sensei_Utils::sensei_check_for_activity( array_merge( $activity_args, array('count' => true, 'offset' => 0, 'number' => 0) ) );
+		$total_statuses = 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 ( $total_statuses < $activity_args['offset'] ) {
-			$new_paged = floor( $total_statuses / $activity_args['number'] );
+		if ($total_statuses < $activity_args['offset']) {
+			$new_paged = floor($total_statuses / $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);
 		}
 		$this->total_items = $total_statuses;
 		$this->items = $statuses;
 
 		$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
-		) );
+		));
 	}
 
 	/**
@@ -220,73 +220,73 @@ 
 	 * @since  1.7.0
 	 * @param object $item The current item
 	 */
-	protected function get_row_data( $item ) {
+	protected function get_row_data($item) {
 		global $wp_version;
 
 		$grade = '';
-		if( 'complete' == $item->comment_approved ) {
-			$status_html = '<span class="graded">' . __( 'Completed', 'woothemes-sensei' ) . '</span>';
-			$grade =  __( 'No Grade', 'woothemes-sensei' );
+		if ('complete' == $item->comment_approved) {
+			$status_html = '<span class="graded">'.__('Completed', 'woothemes-sensei').'</span>';
+			$grade = __('No Grade', 'woothemes-sensei');
 		}
-		elseif( 'graded' == $item->comment_approved ) {
-			$status_html = '<span class="graded">' .  __( 'Graded', 'woothemes-sensei' )  . '</span>';
-			$grade = get_comment_meta( $item->comment_ID, 'grade', true) . '%';
+		elseif ('graded' == $item->comment_approved) {
+			$status_html = '<span class="graded">'.__('Graded', 'woothemes-sensei').'</span>';
+			$grade = get_comment_meta($item->comment_ID, 'grade', true).'%';
 		}
-		elseif( 'passed' == $item->comment_approved ) {
-			$status_html = '<span class="passed">' .  __( 'Passed', 'woothemes-sensei' )  . '</span>';
-			$grade = get_comment_meta( $item->comment_ID, 'grade', true) . '%';
+		elseif ('passed' == $item->comment_approved) {
+			$status_html = '<span class="passed">'.__('Passed', 'woothemes-sensei').'</span>';
+			$grade = get_comment_meta($item->comment_ID, 'grade', true).'%';
 		}
-		elseif( 'failed' == $item->comment_approved ) {
-			$status_html = '<span class="failed">' .  __( 'Failed', 'woothemes-sensei' )  . '</span>';
-			$grade = get_comment_meta( $item->comment_ID, 'grade', true) . '%';
+		elseif ('failed' == $item->comment_approved) {
+			$status_html = '<span class="failed">'.__('Failed', 'woothemes-sensei').'</span>';
+			$grade = get_comment_meta($item->comment_ID, 'grade', true).'%';
 		}
-		elseif( 'ungraded' == $item->comment_approved ) {
-			$status_html = '<span class="ungraded">' .  __( 'Ungraded', 'woothemes-sensei' )  . '</span>';
-			$grade = __( 'N/A', 'woothemes-sensei' );
+		elseif ('ungraded' == $item->comment_approved) {
+			$status_html = '<span class="ungraded">'.__('Ungraded', 'woothemes-sensei').'</span>';
+			$grade = __('N/A', 'woothemes-sensei');
 		}
 		else {
-			$status_html = '<span class="in-progress">' . __( 'In Progress', 'woothemes-sensei' ) . '</span>';
-			$grade = __( 'N/A', 'woothemes-sensei' );
+			$status_html = '<span class="in-progress">'.__('In Progress', 'woothemes-sensei').'</span>';
+			$grade = __('N/A', 'woothemes-sensei');
 		}
 
-        $title = Sensei_Learner::get_full_name( $item->user_id );
+        $title = Sensei_Learner::get_full_name($item->user_id);
 
 		// QuizID to be deprecated
-		$quiz_id = get_post_meta( $item->comment_post_ID, '_lesson_quiz', true );
-		$quiz_link = esc_url( add_query_arg( array( 'page' => $this->page_slug, 'user' => $item->user_id, 'quiz_id' => $quiz_id ), admin_url( 'admin.php' ) ) );
+		$quiz_id = get_post_meta($item->comment_post_ID, '_lesson_quiz', true);
+		$quiz_link = esc_url(add_query_arg(array('page' => $this->page_slug, 'user' => $item->user_id, 'quiz_id' => $quiz_id), admin_url('admin.php')));
 
 		$grade_link = '';
-		switch( $item->comment_approved ) {
+		switch ($item->comment_approved) {
 			case 'ungraded':
-				$grade_link = '<a class="button-primary button" href="' . $quiz_link . '">' . __('Grade quiz', 'woothemes-sensei' ) . '</a>';
+				$grade_link = '<a class="button-primary button" href="'.$quiz_link.'">'.__('Grade quiz', 'woothemes-sensei').'</a>';
 				break;
 
 			case 'graded':
 			case 'passed':
 			case 'failed':
-				$grade_link = '<a class="button-secondary button" href="' . $quiz_link . '">' . __('Review grade', 'woothemes-sensei' ) . '</a>';
+				$grade_link = '<a class="button-secondary button" href="'.$quiz_link.'">'.__('Review grade', 'woothemes-sensei').'</a>';
 				break;
 		}
 
-		$course_id = get_post_meta( $item->comment_post_ID, '_lesson_course', true );
+		$course_id = get_post_meta($item->comment_post_ID, '_lesson_course', true);
 		$course_title = '';
-		if ( !empty($course_id) && version_compare($wp_version, '4.1', '>=') ) {
-			$course_title = '<a href="' . esc_url( add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) ) ) . '">' . get_the_title( $course_id ) . '</a>';
+		if ( ! empty($course_id) && version_compare($wp_version, '4.1', '>=')) {
+			$course_title = '<a href="'.esc_url(add_query_arg(array('page' => $this->page_slug, 'course_id' => $course_id), admin_url('admin.php'))).'">'.get_the_title($course_id).'</a>';
 		}
-		else if ( !empty($course_id) ) {
-			$course_title = get_the_title( $course_id );
+		else if ( ! empty($course_id)) {
+			$course_title = get_the_title($course_id);
 		}
-		$lesson_title = '<a href="' . add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $item->comment_post_ID ), admin_url( 'admin.php' ) ) . '">' . get_the_title( $item->comment_post_ID ) . '</a>';
+		$lesson_title = '<a href="'.add_query_arg(array('page' => $this->page_slug, 'lesson_id' => $item->comment_post_ID), admin_url('admin.php')).'">'.get_the_title($item->comment_post_ID).'</a>';
 
-		$column_data = apply_filters( 'sensei_grading_main_column_data', array(
-				'title' => '<strong><a class="row-title" href="' . esc_url( add_query_arg( array( 'page' => $this->page_slug, 'user_id' => $item->user_id ), admin_url( 'admin.php' ) ) ) . '"">' . $title . '</a></strong>',
+		$column_data = apply_filters('sensei_grading_main_column_data', array(
+				'title' => '<strong><a class="row-title" href="'.esc_url(add_query_arg(array('page' => $this->page_slug, 'user_id' => $item->user_id), admin_url('admin.php'))).'"">'.$title.'</a></strong>',
 				'course' => $course_title,
 				'lesson' => $lesson_title,
 				'updated' => $item->comment_date,
 				'user_status' => $status_html,
 				'user_grade' => $grade,
 				'action' => $grade_link,
-			), $item, $course_id );
+			), $item, $course_id);
 
 		return $column_data;
 	}
@@ -299,7 +299,7 @@ 
 	 */
 	public function no_items() {
 
-        _e( 'No submissions found.', 'woothemes-sensei' );
+        _e('No submissions found.', 'woothemes-sensei');
 
 	} // End no_items()
 
@@ -312,35 +312,35 @@ 
 		global  $wp_version;
 
 		echo '<div class="grading-selects">';
-		do_action( 'sensei_grading_before_dropdown_filters' );
+		do_action('sensei_grading_before_dropdown_filters');
 
-		echo '<div class="select-box">' . "\n";
+		echo '<div class="select-box">'."\n";
 
-			echo '<select id="grading-course-options" name="grading_course" class="chosen_select widefat">' . "\n";
+			echo '<select id="grading-course-options" name="grading_course" class="chosen_select widefat">'."\n";
 
-				echo Sensei()->grading->courses_drop_down_html( $this->course_id );
+				echo Sensei()->grading->courses_drop_down_html($this->course_id);
 
-			echo '</select>' . "\n";
+			echo '</select>'."\n";
 
-		echo '</div>' . "\n";
+		echo '</div>'."\n";
 
-		echo '<div class="select-box">' . "\n";
+		echo '<div class="select-box">'."\n";
 
-			echo '<select id="grading-lesson-options" data-placeholder="&larr; ' . __( 'Select a course', 'woothemes-sensei' ) . '" name="grading_lesson" class="chosen_select widefat">' . "\n";
+			echo '<select id="grading-lesson-options" data-placeholder="&larr; '.__('Select a course', 'woothemes-sensei').'" name="grading_lesson" class="chosen_select widefat">'."\n";
 
-				echo Sensei()->grading->lessons_drop_down_html( $this->course_id, $this->lesson_id );
+				echo Sensei()->grading->lessons_drop_down_html($this->course_id, $this->lesson_id);
 
-			echo '</select>' . "\n";
+			echo '</select>'."\n";
 
-		echo '</div>' . "\n";
+		echo '</div>'."\n";
 
-		if( $this->course_id && $this->lesson_id ) {
+		if ($this->course_id && $this->lesson_id) {
 
-			echo '<div class="select-box reset-filter">' . "\n";
+			echo '<div class="select-box reset-filter">'."\n";
 
-				echo '<a class="button-secondary" href="' . esc_url( remove_query_arg( array( 'lesson_id', 'course_id' ) ) ) . '">' . __( 'Reset filter', 'woothemes-sensei' ) . '</a>' . "\n";
+				echo '<a class="button-secondary" href="'.esc_url(remove_query_arg(array('lesson_id', 'course_id'))).'">'.__('Reset filter', 'woothemes-sensei').'</a>'."\n";
 
-			echo '</div>' . "\n";
+			echo '</div>'."\n";
 
 		}
 
@@ -355,26 +355,26 @@ 
 		$query_args = array(
 			'page' => $this->page_slug,
 		);
-		if( $this->course_id ) {
+		if ($this->course_id) {
 			// Currently not possible to restrict to a single Course, as that requires WP_Comment to support multiple
 			// post_ids (i.e. every lesson within the Course), WP 4.1 ( https://core.trac.wordpress.org/changeset/29808 )
 			$query_args['course_id'] = $this->course_id;
-			if ( version_compare($wp_version, '4.1', '>=') ) {
-				$count_args['post__in'] = Sensei()->course->course_lessons( $this->course_id, 'any', 'ids' );
+			if (version_compare($wp_version, '4.1', '>=')) {
+				$count_args['post__in'] = Sensei()->course->course_lessons($this->course_id, 'any', 'ids');
 			}
 		}
-		if( $this->lesson_id ) {
+		if ($this->lesson_id) {
 			$query_args['lesson_id'] = $this->lesson_id;
 			// Restrict to a single lesson
 			$count_args['post_id'] = $this->lesson_id;
 		}
-		if( $this->search ) {
+		if ($this->search) {
 			$query_args['s'] = $this->search;
 		}
-		if ( !empty($this->user_ids) ) {
+		if ( ! empty($this->user_ids)) {
 			$count_args['user_id'] = $this->user_ids;
 		}
-		if( !empty($this->user_id) ) {
+		if ( ! empty($this->user_id)) {
 			$query_args['user_id'] = $this->user_id;
 			$count_args['user_id'] = $this->user_id;
 		}
@@ -382,7 +382,7 @@ 
 		$all_lessons_count = $ungraded_lessons_count = $graded_lessons_count = $inprogress_lessons_count = 0;
 		$all_class = $ungraded_class = $graded_class = $inprogress_class = '';
 
-		switch( $this->view ) :
+		switch ($this->view) :
 			case 'all':
 				$all_class = 'current';
 				break;
@@ -398,7 +398,7 @@ 
 				break;
 		endswitch;
 
-		$counts = Sensei()->grading->count_statuses( apply_filters( 'sensei_grading_count_statues', $count_args ) );
+		$counts = Sensei()->grading->count_statuses(apply_filters('sensei_grading_count_statues', $count_args));
 
 		$inprogress_lessons_count = $counts['in-progress'];
 		$ungraded_lessons_count = $counts['ungraded'];
@@ -414,19 +414,19 @@ 
 		$inprogress_args['view'] = 'in-progress';
 
 		$format = '<a class="%s" href="%s">%s <span class="count">(%s)</span></a>';
-		$menu['all'] = sprintf( $format, $all_class, esc_url( add_query_arg( $all_args, admin_url( 'admin.php' ) ) ), __( 'All', 'woothemes-sensei' ), number_format( (int) $all_lessons_count ) );
-		$menu['ungraded'] = sprintf( $format, $ungraded_class, esc_url( add_query_arg( $ungraded_args, admin_url( 'admin.php' ) ) ), __( 'Ungraded', 'woothemes-sensei' ), number_format( (int) $ungraded_lessons_count ) );
-		$menu['graded'] = sprintf( $format, $graded_class, esc_url( add_query_arg( $graded_args, admin_url( 'admin.php' ) ) ), __( 'Graded', 'woothemes-sensei' ), number_format( (int) $graded_lessons_count ) );
-		$menu['in-progress'] = sprintf( $format, $inprogress_class, esc_url( add_query_arg( $inprogress_args, admin_url( 'admin.php' ) ) ), __( 'In Progress', 'woothemes-sensei' ), number_format( (int) $inprogress_lessons_count ) );
-
-		$menu = apply_filters( 'sensei_grading_sub_menu', $menu );
-		if ( !empty($menu) ) {
-			echo '<ul class="subsubsub">' . "\n";
-			foreach ( $menu as $class => $item ) {
-				$menu[ $class ] = "\t<li class='$class'>$item";
+		$menu['all'] = sprintf($format, $all_class, esc_url(add_query_arg($all_args, admin_url('admin.php'))), __('All', 'woothemes-sensei'), number_format((int) $all_lessons_count));
+		$menu['ungraded'] = sprintf($format, $ungraded_class, esc_url(add_query_arg($ungraded_args, admin_url('admin.php'))), __('Ungraded', 'woothemes-sensei'), number_format((int) $ungraded_lessons_count));
+		$menu['graded'] = sprintf($format, $graded_class, esc_url(add_query_arg($graded_args, admin_url('admin.php'))), __('Graded', 'woothemes-sensei'), number_format((int) $graded_lessons_count));
+		$menu['in-progress'] = sprintf($format, $inprogress_class, esc_url(add_query_arg($inprogress_args, admin_url('admin.php'))), __('In Progress', 'woothemes-sensei'), number_format((int) $inprogress_lessons_count));
+
+		$menu = apply_filters('sensei_grading_sub_menu', $menu);
+		if ( ! empty($menu)) {
+			echo '<ul class="subsubsub">'."\n";
+			foreach ($menu as $class => $item) {
+				$menu[$class] = "\t<li class='$class'>$item";
 			}
-			echo implode( " |</li>\n", $menu ) . "</li>\n";
-			echo '</ul>' . "\n";
+			echo implode(" |</li>\n", $menu)."</li>\n";
+			echo '</ul>'."\n";
 		}
 
 	} // End data_table_header()
@@ -447,4 +447,4 @@ 
  * @ignore only for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Grading_Main extends Sensei_Grading_Main{}
+class WooThemes_Sensei_Grading_Main extends Sensei_Grading_Main {}

includes/class-sensei-analysis-lesson-list-table.php

Spacing +95 added lines, -95 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Admin Analysis Lesson Data Table in Sensei.
@@ -18,18 +18,18 @@ 
 	 * Constructor
 	 * @since  1.2.0
 	 */
-	public function __construct ( $lesson_id = 0 ) {
-		$this->lesson_id = intval( $lesson_id );
-		$this->course_id = intval( get_post_meta( $this->lesson_id, '_lesson_course', true ) );
+	public function __construct($lesson_id = 0) {
+		$this->lesson_id = intval($lesson_id);
+		$this->course_id = intval(get_post_meta($this->lesson_id, '_lesson_course', true));
 
 		// Load Parent token into constructor
-		parent::__construct( 'analysis_lesson' );
+		parent::__construct('analysis_lesson');
 
 		// 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()
 
 	/**
@@ -39,13 +39,13 @@ 
 	 */
 	function get_columns() {
 		$columns = array(
-			'title' => __( 'Learner', 'woothemes-sensei' ),
-			'started' => __( 'Date Started', 'woothemes-sensei' ),
-			'completed' => __( 'Date Completed', 'woothemes-sensei' ),
-			'status' => __( 'Status', 'woothemes-sensei' ),
-			'grade' => __( 'Grade', 'woothemes-sensei' ),
+			'title' => __('Learner', 'woothemes-sensei'),
+			'started' => __('Date Started', 'woothemes-sensei'),
+			'completed' => __('Date Completed', 'woothemes-sensei'),
+			'status' => __('Status', 'woothemes-sensei'),
+			'grade' => __('Grade', 'woothemes-sensei'),
 		);
-		$columns = apply_filters( 'sensei_analysis_lesson_columns', $columns, $this );
+		$columns = apply_filters('sensei_analysis_lesson_columns', $columns, $this);
 		return $columns;
 	}
 
@@ -56,13 +56,13 @@ 
 	 */
 	function get_sortable_columns() {
 		$columns = array(
-			'title' => array( 'title', false ),
-			'started' => array( 'started', false ),
-			'completed' => array( 'completed', false ),
-			'status' => array( 'status', false ),
-			'grade' => array( 'grade', false ),
+			'title' => array('title', false),
+			'started' => array('started', false),
+			'completed' => array('completed', false),
+			'status' => array('status', false),
+			'grade' => array('grade', false),
 		);
-		$columns = apply_filters( 'sensei_analysis_lesson_columns_sortable', $columns, $this );
+		$columns = apply_filters('sensei_analysis_lesson_columns_sortable', $columns, $this);
 		return $columns;
 	}
 
@@ -76,32 +76,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(
@@ -110,19 +110,19 @@ 
 			'orderby' => $orderby,
 			'order' => $order,
 		);
-		if ( $this->search ) {
+		if ($this->search) {
 			$args['search'] = $this->search;
 		} // End If Statement
 
-		$this->items = $this->get_lesson_statuses( $args );
+		$this->items = $this->get_lesson_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
-		) );
+		));
 	}
 
 	/**
@@ -130,7 +130,7 @@ 
 	 * @since  1.7.0
 	 * @return data
 	 */
-	public function generate_report( $report ) {
+	public function generate_report($report) {
 
 		$data = array();
 
@@ -138,22 +138,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;
 
@@ -161,23 +161,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_lesson_statuses( $args );
+		$this->items = $this->get_lesson_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,64 +189,64 @@ 
 	 * @since  1.7.0
 	 * @param object $item The current item
 	 */
-	protected function get_row_data( $item ) {
+	protected function get_row_data($item) {
 
-		$user_start_date = get_comment_meta( $item->comment_ID, 'start', true );
+		$user_start_date = get_comment_meta($item->comment_ID, 'start', true);
 		$user_end_date = $item->comment_date;
 		$status_class = $grade = '';
 
-		if( 'complete' == $item->comment_approved ) {
-			$status =  __( 'Completed', 'woothemes-sensei' );
+		if ('complete' == $item->comment_approved) {
+			$status = __('Completed', 'woothemes-sensei');
 			$status_class = 'graded';
 
-			$grade =  __( 'No Grade', 'woothemes-sensei' );
+			$grade = __('No Grade', 'woothemes-sensei');
 		}
-		elseif( 'graded' == $item->comment_approved ) {
-			$status = __( 'Graded', 'woothemes-sensei' ) ;
+		elseif ('graded' == $item->comment_approved) {
+			$status = __('Graded', 'woothemes-sensei');
 			$status_class = 'graded';
 
-			$grade = get_comment_meta( $item->comment_ID, 'grade', true);
+			$grade = get_comment_meta($item->comment_ID, 'grade', true);
 		}
-		elseif( 'passed' == $item->comment_approved ) {
-			$status =  __( 'Passed', 'woothemes-sensei' );
+		elseif ('passed' == $item->comment_approved) {
+			$status = __('Passed', 'woothemes-sensei');
 			$status_class = 'graded';
 
-			$grade = get_comment_meta( $item->comment_ID, 'grade', true);
+			$grade = get_comment_meta($item->comment_ID, 'grade', true);
 		}
-		elseif( 'failed' == $item->comment_approved ) {
-			$status = __( 'Failed', 'woothemes-sensei' );
+		elseif ('failed' == $item->comment_approved) {
+			$status = __('Failed', 'woothemes-sensei');
 			$status_class = 'failed';
 
-			$grade = get_comment_meta( $item->comment_ID, 'grade', true);
+			$grade = get_comment_meta($item->comment_ID, 'grade', true);
 		}
-		elseif( 'ungraded' == $item->comment_approved ) {
-			$status =  __( 'Ungraded', 'woothemes-sensei' );
+		elseif ('ungraded' == $item->comment_approved) {
+			$status = __('Ungraded', 'woothemes-sensei');
 			$status_class = 'ungraded';
 
 		}
 		else {
-			$status =  __( 'In Progress', 'woothemes-sensei' );
+			$status = __('In Progress', 'woothemes-sensei');
 			$user_end_date = '';
 		}
 
 		// Output users data
-        $user_name = Sensei_Learner::get_full_name( $item->user_id );
+        $user_name = Sensei_Learner::get_full_name($item->user_id);
 
-        if ( !$this->csv_output ) {
-			$url = add_query_arg( array( 'page' => $this->page_slug, 'user_id' => $item->user_id, 'course_id' => $this->course_id ), admin_url( 'admin.php' ) );
+        if ( ! $this->csv_output) {
+			$url = add_query_arg(array('page' => $this->page_slug, 'user_id' => $item->user_id, 'course_id' => $this->course_id), admin_url('admin.php'));
 
-			$user_name = '<strong><a class="row-title" href="' . esc_url( $url ) . '">' . $user_name . '</a></strong>';
-			$status = sprintf( '<span class="%s">%s</span>', $item->comment_approved, $status );
-			if ( is_numeric($grade) ) {
+			$user_name = '<strong><a class="row-title" href="'.esc_url($url).'">'.$user_name.'</a></strong>';
+			$status = sprintf('<span class="%s">%s</span>', $item->comment_approved, $status);
+			if (is_numeric($grade)) {
 				$grade .= '%';
 			}
 		} // End If Statement
-		$column_data = apply_filters( 'sensei_analysis_lesson_column_data', array( 'title' => $user_name,
+		$column_data = apply_filters('sensei_analysis_lesson_column_data', array('title' => $user_name,
 										'started' => $user_start_date,
 										'completed' => $user_end_date,
 										'status' => $status,
 										'grade' => $grade,
-									), $item, $this );
+									), $item, $this);
 
 		return $column_data;
 	}
@@ -256,7 +256,7 @@ 
 	 * @since  1.7.0
 	 * @return array statuses
 	 */
-	private function get_lesson_statuses( $args ) {
+	private function get_lesson_statuses($args) {
 
 		$activity_args = array( 
 				'post_id' => $this->lesson_id,
@@ -269,34 +269,34 @@ 
 			);
 
 		// Searching users on statuses requires sub-selecting the statuses by user_ids
-		if ( $this->search ) {
+		if ($this->search) {
 			$user_args = array(
-				'search' => '*' . $this->search . '*',
+				'search' => '*'.$this->search.'*',
 				'fields' => 'ID',
 			);
 			// Filter for extending
-			$user_args = apply_filters( 'sensei_analysis_lesson_search_users', $user_args );
-			if ( !empty( $user_args ) ) {
-				$learners_search = new WP_User_Query( $user_args );
+			$user_args = apply_filters('sensei_analysis_lesson_search_users', $user_args);
+			if ( ! empty($user_args)) {
+				$learners_search = new WP_User_Query($user_args);
 				// Store for reuse on counts
 				$activity_args['user_id'] = (array) $learners_search->get_results();
 			}
 		} // End If Statement
 
-		$activity_args = apply_filters( 'sensei_analysis_lesson_filter_statuses', $activity_args );
+		$activity_args = apply_filters('sensei_analysis_lesson_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'] ) {
-			$new_paged = floor( $total_statuses / $activity_args['number'] );
+		if ($this->total_items < $activity_args['offset']) {
+			$new_paged = floor($total_statuses / $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;
 	} // End get_lesson_statuses()
@@ -308,7 +308,7 @@ 
 	 * @return void
 	 */
 	public function no_items() {
-		 _e( 'No learners found.', 'woothemes-sensei' );
+		 _e('No learners found.', 'woothemes-sensei');
 	} // End no_items()
 
 	/**
@@ -317,7 +317,7 @@ 
 	 * @return void
 	 */
 	public function data_table_header() {
-		echo '<strong>' . __( 'Learners taking this Lesson', 'woothemes-sensei' ) . '</strong>';
+		echo '<strong>'.__('Learners taking this Lesson', 'woothemes-sensei').'</strong>';
 	} // End data_table_header()
 
 	/**
@@ -326,10 +326,10 @@ 
 	 * @return void
 	 */
 	public function data_table_footer() {
-		$lesson = get_post( $this->lesson_id );
-		$report = sanitize_title( $lesson->post_title ) . '-learners-overview';
-		$url = add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $this->lesson_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>';
+		$lesson = get_post($this->lesson_id);
+		$report = sanitize_title($lesson->post_title).'-learners-overview';
+		$url = add_query_arg(array('page' => $this->page_slug, 'lesson_id' => $this->lesson_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>';
 	} // End data_table_footer()
 
 	/**
@@ -337,9 +337,9 @@ 
 	 * @since  1.7.0
 	 * @return string $text
 	 */
-	public function search_button( $text = '' ) {
+	public function search_button($text = '') {
 
-        $text =  __( 'Search Learners', 'woothemes-sensei' );
+        $text = __('Search Learners', 'woothemes-sensei');
 
         return $text;
 

includes/class-sensei-learner.php

Spacing +10 added lines, -10 removed lines

@@ -8,7 +8,7 @@ 
  *
  * @since 1.9.0
  */
-class Sensei_Learner{
+class Sensei_Learner {
 
     /**
      * Get the students full name
@@ -19,25 +19,25 @@ 
      * @param $user_id
      * @return bool|mixed|void
      */
-    public static function get_full_name( $user_id ){
+    public static function get_full_name($user_id) {
 
         $full_name = '';
 
-        if( empty( $user_id ) || ! ( 0 < intval( $user_id ) )
-            || !( get_userdata( $user_id ) ) ){
+        if (empty($user_id) || ! (0 < intval($user_id))
+            || ! (get_userdata($user_id))) {
             return false;
         }
 
         // get the user details
-        $user = get_user_by( 'id', $user_id );
+        $user = get_user_by('id', $user_id);
 
-        if( ! empty( $user->first_name  ) && ! empty( $user->last_name  )  ){
+        if ( ! empty($user->first_name) && ! empty($user->last_name)) {
 
-            $full_name = trim( $user->first_name   ) . ' ' . trim( $user->last_name  );
+            $full_name = trim($user->first_name).' '.trim($user->last_name);
 
-        }else{
+        } else {
 
-            $full_name =  $user->display_name;
+            $full_name = $user->display_name;
 
         }
 
@@ -48,7 +48,7 @@ 
          * @param $full_name
          * @param $user_id
          */
-        return apply_filters( 'sensei_learner_full_name' , $full_name , $user_id );
+        return apply_filters('sensei_learner_full_name', $full_name, $user_id);
 
     }// end get_full_name
 

includes/class-sensei-analysis-course-list-table.php

Spacing +195 added lines, -195 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Admin Analysis Course Data Table in Sensei.
@@ -20,27 +20,27 @@ 
 	 * Constructor
 	 * @since  1.2.0
 	 */
-	public function __construct ( $course_id = 0, $user_id = 0 ) {
-		$this->course_id = intval( $course_id );
-		$this->user_id = intval( $user_id );
+	public function __construct($course_id = 0, $user_id = 0) {
+		$this->course_id = intval($course_id);
+		$this->user_id = intval($user_id);
 
-		if( isset( $_GET['view'] ) && in_array( $_GET['view'], array( 'user', 'lesson' ) ) ) {
+		if (isset($_GET['view']) && in_array($_GET['view'], array('user', 'lesson'))) {
 			$this->view = $_GET['view'];
 		}
 
 		// Viewing a single Learner always sets the view to Lessons
-		if( $this->user_id ) {
+		if ($this->user_id) {
 			$this->view = 'lesson';
 		}
 
 		// Load Parent token into constructor
-		parent::__construct( 'analysis_course' );
+		parent::__construct('analysis_course');
 
 		// 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()
 
@@ -51,45 +51,45 @@ 
 	 */
 	function get_columns() {
 
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'user' :
 				$columns = array(
-					'title' => __( 'Learner', 'woothemes-sensei' ),
-					'started' => __( 'Date Started', 'woothemes-sensei' ),
-					'completed' => __( 'Date Completed', 'woothemes-sensei' ),
-					'user_status' => __( 'Status', 'woothemes-sensei' ),
-					'percent' => __( 'Percent Complete', 'woothemes-sensei' ),
+					'title' => __('Learner', 'woothemes-sensei'),
+					'started' => __('Date Started', 'woothemes-sensei'),
+					'completed' => __('Date Completed', 'woothemes-sensei'),
+					'user_status' => __('Status', 'woothemes-sensei'),
+					'percent' => __('Percent Complete', 'woothemes-sensei'),
 				);
 				break;
 
 			case 'lesson' :
 			default:
-				if ( $this->user_id ) {
+				if ($this->user_id) {
 
 					$columns = array(
-						'title' => __( 'Lesson', 'woothemes-sensei' ),
-						'started' => __( 'Date Started', 'woothemes-sensei' ),
-						'completed' => __( 'Date Completed', 'woothemes-sensei' ),
-						'user_status' => __( 'Status', 'woothemes-sensei' ),
-						'grade' => __( 'Grade', 'woothemes-sensei' ),
+						'title' => __('Lesson', 'woothemes-sensei'),
+						'started' => __('Date Started', 'woothemes-sensei'),
+						'completed' => __('Date Completed', 'woothemes-sensei'),
+						'user_status' => __('Status', 'woothemes-sensei'),
+						'grade' => __('Grade', 'woothemes-sensei'),
 					);
 
 				} else {
 
 					$columns = array(
-						'title' => __( 'Lesson', 'woothemes-sensei' ),
-						'num_learners' => __( 'Learners', 'woothemes-sensei' ),
-						'completions' => __( 'Completed', 'woothemes-sensei' ),
-						'average_grade' => __( 'Average Grade', 'woothemes-sensei' ),
+						'title' => __('Lesson', 'woothemes-sensei'),
+						'num_learners' => __('Learners', 'woothemes-sensei'),
+						'completions' => __('Completed', 'woothemes-sensei'),
+						'average_grade' => __('Average Grade', 'woothemes-sensei'),
 					);
 
 				}
 				break;
 		}
 		// Backwards compatible
-		$columns = apply_filters( 'sensei_analysis_course_' . $this->view . '_columns', $columns, $this );
+		$columns = apply_filters('sensei_analysis_course_'.$this->view.'_columns', $columns, $this);
 		// Moving forward, single filter with args
-		$columns = apply_filters( 'sensei_analysis_course_columns', $columns, $this );
+		$columns = apply_filters('sensei_analysis_course_columns', $columns, $this);
 		return $columns;
 	}
 
@@ -100,46 +100,46 @@ 
 	 */
 	function get_sortable_columns() {
 
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'user' :
 				$columns = array(
-					'title' => array( 'title', false ),
-					'started' => array( 'started', false ),
-					'completed' => array( 'completed', false ),
-					'user_status' => array( 'user_status', false ),
+					'title' => array('title', false),
+					'started' => array('started', false),
+					'completed' => array('completed', false),
+					'user_status' => array('user_status', false),
 //					'grade' => array( 'grade', false ),
-					'percent' => array( 'percent', false )
+					'percent' => array('percent', false)
 				);
 				break;
 
 			case 'lesson' :
 			default:
-				if ( $this->user_id ) {
+				if ($this->user_id) {
 
 					$columns = array(
-						'title' => array( 'title', false ),
-						'started' => array( 'started', false ),
-						'completed' => array( 'completed', false ),
-						'user_status' => array( 'user_status', false ),
-						'grade' => array( 'grade', false ),
+						'title' => array('title', false),
+						'started' => array('started', false),
+						'completed' => array('completed', false),
+						'user_status' => array('user_status', false),
+						'grade' => array('grade', false),
 					);
 
 				} else {
 
 					$columns = array(
-						'title' => array( 'title', false ),
-						'num_learners' => array( 'num_learners', false ),
-						'completions' => array( 'completions', false ),
-						'average_grade' => array( 'average_grade', false )
+						'title' => array('title', false),
+						'num_learners' => array('num_learners', false),
+						'completions' => array('completions', false),
+						'average_grade' => array('average_grade', false)
 					);
 
 				}
 				break;
 		}
 		// Backwards compatible
-		$columns = apply_filters( 'sensei_analysis_course_' . $this->view . '_columns_sortable', $columns, $this );
+		$columns = apply_filters('sensei_analysis_course_'.$this->view.'_columns_sortable', $columns, $this);
 		// Moving forward, single filter with args
-		$columns = apply_filters( 'sensei_analysis_course_columns_sortable', $columns, $this );
+		$columns = apply_filters('sensei_analysis_course_columns_sortable', $columns, $this);
 		return $columns;
 	}
 
@@ -153,32 +153,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(
@@ -187,28 +187,28 @@ 
 			'orderby' => $orderby,
 			'order' => $order,
 		);
-		if ( $this->search ) {
+		if ($this->search) {
 			$args['search'] = $this->search;
 		} // End If Statement
 
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'user' :
-				$this->items = $this->get_course_statuses( $args );
+				$this->items = $this->get_course_statuses($args);
 				break;
 
 			case 'lesson':
 			default:
-				$this->items = $this->get_lessons( $args );
+				$this->items = $this->get_lessons($args);
 				break;
 		}
 
 		$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
-		) );
+		));
 	}
 
 	/**
@@ -216,7 +216,7 @@ 
 	 * @since  1.7.0
 	 * @return data
 	 */
-	public function generate_report( $report ) {
+	public function generate_report($report) {
 
 		$data = array();
 
@@ -224,22 +224,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;
 
@@ -247,32 +247,32 @@ 
 			'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;
 
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'user' :
-				$this->items = $this->get_course_statuses( $args );
+				$this->items = $this->get_course_statuses($args);
 				break;
 
 			case 'lesson':
 			default:
-				$this->items = $this->get_lessons( $args );
+				$this->items = $this->get_lessons($args);
 				break;
 		}
 
 		// 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;
@@ -284,37 +284,37 @@ 
 	 * @since  1.7.0
 	 * @param object $item The current item
 	 */
-	protected function get_row_data( $item ) {
+	protected function get_row_data($item) {
 
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'user' :
-				$user_start_date = get_comment_meta( $item->comment_ID, 'start', true );
+				$user_start_date = get_comment_meta($item->comment_ID, 'start', true);
 				$user_end_date = $item->comment_date;
 
-				if( 'complete' == $item->comment_approved ) {
+				if ('complete' == $item->comment_approved) {
 
-					$status =  __( 'Completed', 'woothemes-sensei' );
+					$status = __('Completed', 'woothemes-sensei');
 					$status_class = 'graded';
 
 				} else {
 
-					$status =  __( 'In Progress', 'woothemes-sensei' );
+					$status = __('In Progress', 'woothemes-sensei');
 					$status_class = 'in-progress';
 					$user_end_date = '';
 
 				}
-				$course_percent = get_comment_meta( $item->comment_ID, 'percent', true );
+				$course_percent = get_comment_meta($item->comment_ID, 'percent', true);
 
 				// Output users data
-				$user_name = Sensei_Learner::get_full_name( $item->user_id );
+				$user_name = Sensei_Learner::get_full_name($item->user_id);
 
-				if ( !$this->csv_output ) {
+				if ( ! $this->csv_output) {
 
-					$url = add_query_arg( array( 'page' => $this->page_slug, 'user_id' => $item->user_id, 'course_id' => $this->course_id ), admin_url( 'admin.php' ) );
+					$url = add_query_arg(array('page' => $this->page_slug, 'user_id' => $item->user_id, 'course_id' => $this->course_id), admin_url('admin.php'));
 
-					$user_name = '<strong><a class="row-title" href="' . esc_url( $url ) . '">' . $user_name . '</a></strong>';
-					$status = sprintf( '<span class="%s">%s</span>', $status_class, $status );
-					if ( is_numeric($course_percent) ) {
+					$user_name = '<strong><a class="row-title" href="'.esc_url($url).'">'.$user_name.'</a></strong>';
+					$status = sprintf('<span class="%s">%s</span>', $status_class, $status);
+					if (is_numeric($course_percent)) {
 
 						$course_percent .= '%';
 
@@ -322,19 +322,19 @@ 
 
 				} // End If Statement
 
-				$column_data = apply_filters( 'sensei_analysis_course_column_data', array( 'title' => $user_name,
+				$column_data = apply_filters('sensei_analysis_course_column_data', array('title' => $user_name,
 												'started' => $user_start_date,
 												'completed' => $user_end_date,
 												'user_status' => $status,
 												'percent' => $course_percent,
-											), $item, $this );
+											), $item, $this);
 				break;
 
 			case 'lesson':
 			default:
 				// Displaying lessons for this Course for a specific User
-				if ( $this->user_id ) {
-					$status = __( 'Not started', 'woothemes-sensei' );
+				if ($this->user_id) {
+					$status = __('Not started', 'woothemes-sensei');
 					$user_start_date = $user_end_date = $status_class = $grade = '';
 
 					$lesson_args = array(
@@ -343,66 +343,66 @@ 
 							'type' => 'sensei_lesson_status',
 							'status' => 'any',
 						);
-					$lesson_status = Sensei_Utils::sensei_check_for_activity( apply_filters( 'sensei_analysis_course_user_lesson', $lesson_args, $item, $this->user_id ), true );
+					$lesson_status = Sensei_Utils::sensei_check_for_activity(apply_filters('sensei_analysis_course_user_lesson', $lesson_args, $item, $this->user_id), true);
 
-					if ( !empty($lesson_status) ) {
-						$user_start_date = get_comment_meta( $lesson_status->comment_ID, 'start', true );
+					if ( ! empty($lesson_status)) {
+						$user_start_date = get_comment_meta($lesson_status->comment_ID, 'start', true);
 						$user_end_date = $lesson_status->comment_date;
 
-						if( 'complete' == $lesson_status->comment_approved ) {
-							$status = __( 'Completed', 'woothemes-sensei' );
+						if ('complete' == $lesson_status->comment_approved) {
+							$status = __('Completed', 'woothemes-sensei');
 							$status_class = 'graded';
 
-							$grade = __( 'No Grade', 'woothemes-sensei' );
+							$grade = __('No Grade', 'woothemes-sensei');
 						}
-						elseif( 'graded' == $lesson_status->comment_approved ) {
-							$status =  __( 'Graded', 'woothemes-sensei' );
+						elseif ('graded' == $lesson_status->comment_approved) {
+							$status = __('Graded', 'woothemes-sensei');
 							$status_class = 'graded';
 
-							$grade = get_comment_meta( $lesson_status->comment_ID, 'grade', true);
+							$grade = get_comment_meta($lesson_status->comment_ID, 'grade', true);
 						}
-						elseif( 'passed' == $lesson_status->comment_approved ) {
-							$status =  __( 'Passed', 'woothemes-sensei' );
+						elseif ('passed' == $lesson_status->comment_approved) {
+							$status = __('Passed', 'woothemes-sensei');
 							$status_class = 'graded';
 
-							$grade = get_comment_meta( $lesson_status->comment_ID, 'grade', true);
+							$grade = get_comment_meta($lesson_status->comment_ID, 'grade', true);
 						}
-						elseif( 'failed' == $lesson_status->comment_approved ) {
-							$status =  __( 'Failed', 'woothemes-sensei' );
+						elseif ('failed' == $lesson_status->comment_approved) {
+							$status = __('Failed', 'woothemes-sensei');
 							$status_class = 'failed';
 
-							$grade = get_comment_meta( $lesson_status->comment_ID, 'grade', true);
+							$grade = get_comment_meta($lesson_status->comment_ID, 'grade', true);
 						}
-						elseif( 'ungraded' == $lesson_status->comment_approved ) {
-							$status =  __( 'Ungraded', 'woothemes-sensei' );
+						elseif ('ungraded' == $lesson_status->comment_approved) {
+							$status = __('Ungraded', 'woothemes-sensei');
 							$status_class = 'ungraded';
 
 						}
-						elseif( 'in-progress' == $lesson_status->comment_approved ) {
-							$status =  __( 'In Progress', 'woothemes-sensei' );
+						elseif ('in-progress' == $lesson_status->comment_approved) {
+							$status = __('In Progress', 'woothemes-sensei');
 							$user_end_date = '';
 						}
 					} // END lesson_status
 
 					// Output users data
-					if ( $this->csv_output ) {
-						$lesson_title = apply_filters( 'the_title', $item->post_title, $item->ID );
+					if ($this->csv_output) {
+						$lesson_title = apply_filters('the_title', $item->post_title, $item->ID);
 					}
 					else {
-						$url = add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $item->ID ), admin_url( 'admin.php' ) );
-						$lesson_title = '<strong><a class="row-title" href="' . esc_url( $url ) . '">' . apply_filters( 'the_title', $item->post_title, $item->ID ) . '</a></strong>';
+						$url = add_query_arg(array('page' => $this->page_slug, 'lesson_id' => $item->ID), admin_url('admin.php'));
+						$lesson_title = '<strong><a class="row-title" href="'.esc_url($url).'">'.apply_filters('the_title', $item->post_title, $item->ID).'</a></strong>';
 
-						$status = sprintf( '<span class="%s">%s</span>', $status_class, $status );
-						if ( is_numeric($grade) ) {
+						$status = sprintf('<span class="%s">%s</span>', $status_class, $status);
+						if (is_numeric($grade)) {
 							$grade .= '%';
 						}
 					} // End If Statement
-					$column_data = apply_filters( 'sensei_analysis_course_column_data', array( 'title' => $lesson_title,
+					$column_data = apply_filters('sensei_analysis_course_column_data', array('title' => $lesson_title,
 													'started' => $user_start_date,
 													'completed' => $user_end_date,
 													'user_status' => $status,
 													'grade' => $grade,
-												), $item, $this );
+												), $item, $this);
 				}
 				// Display lessons for this Course regardless of users
 				else {
@@ -412,51 +412,51 @@ 
 							'type' => 'sensei_lesson_status',
 							'status' => 'any',
 						);
-					$lesson_students = Sensei_Utils::sensei_check_for_activity( apply_filters( 'sensei_analysis_lesson_learners', $lesson_args, $item ) );
+					$lesson_students = Sensei_Utils::sensei_check_for_activity(apply_filters('sensei_analysis_lesson_learners', $lesson_args, $item));
 
 					// Get Course Completions
 					$lesson_args = array(
 							'post_id' => $item->ID,
 							'type' => 'sensei_lesson_status',
-							'status' => array( 'complete', 'graded', 'passed', 'failed' ),
+							'status' => array('complete', 'graded', 'passed', 'failed'),
 							'count' => true,
 						);
-					$lesson_completions = Sensei_Utils::sensei_check_for_activity( apply_filters( 'sensei_analysis_lesson_completions', $lesson_args, $item ) );
+					$lesson_completions = Sensei_Utils::sensei_check_for_activity(apply_filters('sensei_analysis_lesson_completions', $lesson_args, $item));
 
 					$lesson_average_grade = __('n/a', 'woothemes-sensei');
-					if ( false != get_post_meta($item->ID, '_quiz_has_questions', true) ) {
+					if (false != get_post_meta($item->ID, '_quiz_has_questions', true)) {
 						// Get Percent Complete
 						$grade_args = array(
 								'post_id' => $item->ID,
 								'type' => 'sensei_lesson_status',
-								'status' => array( 'graded', 'passed', 'failed' ),
+								'status' => array('graded', 'passed', 'failed'),
 								'meta_key' => 'grade',
 							);
-						add_filter( 'comments_clauses', array( 'WooThemes_Sensei_Utils', 'comment_total_sum_meta_value_filter' ) );
-						$lesson_grades = Sensei_Utils::sensei_check_for_activity( apply_filters( 'sensei_analysis_lesson_grades', $grade_args, $item ), true );
-						remove_filter( 'comments_clauses', array( 'WooThemes_Sensei_Utils', 'comment_total_sum_meta_value_filter' ) );
+						add_filter('comments_clauses', array('WooThemes_Sensei_Utils', 'comment_total_sum_meta_value_filter'));
+						$lesson_grades = Sensei_Utils::sensei_check_for_activity(apply_filters('sensei_analysis_lesson_grades', $grade_args, $item), true);
+						remove_filter('comments_clauses', array('WooThemes_Sensei_Utils', 'comment_total_sum_meta_value_filter'));
 
-						$grade_count = !empty( $lesson_grades->total ) ? $lesson_grades->total : 1;
-						$grade_total = !empty( $lesson_grades->meta_sum ) ? doubleval( $lesson_grades->meta_sum ) : 0;
-						$lesson_average_grade = abs( round( doubleval( $grade_total / $grade_count ), 2 ) );
+						$grade_count = ! empty($lesson_grades->total) ? $lesson_grades->total : 1;
+						$grade_total = ! empty($lesson_grades->meta_sum) ? doubleval($lesson_grades->meta_sum) : 0;
+						$lesson_average_grade = abs(round(doubleval($grade_total / $grade_count), 2));
 					}
 					// Output lesson data
-					if ( $this->csv_output ) {
-						$lesson_title = apply_filters( 'the_title', $item->post_title, $item->ID );
+					if ($this->csv_output) {
+						$lesson_title = apply_filters('the_title', $item->post_title, $item->ID);
 					}
 					else {
-						$url = add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $item->ID ), admin_url( 'admin.php' ) );
-						$lesson_title = '<strong><a class="row-title" href="' . esc_url( $url ) . '">' . apply_filters( 'the_title', $item->post_title, $item->ID ) . '</a></strong>';
+						$url = add_query_arg(array('page' => $this->page_slug, 'lesson_id' => $item->ID), admin_url('admin.php'));
+						$lesson_title = '<strong><a class="row-title" href="'.esc_url($url).'">'.apply_filters('the_title', $item->post_title, $item->ID).'</a></strong>';
 
-						if ( is_numeric( $lesson_average_grade ) ) {
+						if (is_numeric($lesson_average_grade)) {
 							$lesson_average_grade .= '%';
 						}
 					} // End If Statement
-					$column_data = apply_filters( 'sensei_analysis_course_column_data', array( 'title' => $lesson_title,
+					$column_data = apply_filters('sensei_analysis_course_column_data', array('title' => $lesson_title,
 													'num_learners' => $lesson_students,
 													'completions' => $lesson_completions,
 													'average_grade' => $lesson_average_grade,
-												), $item, $this );
+												), $item, $this);
 				} // END if
 				break;
 		} // END switch
@@ -469,7 +469,7 @@ 
 	 * @since  1.7.0
 	 * @return array statuses
 	 */
-	private function get_course_statuses( $args ) {
+	private function get_course_statuses($args) {
 
 		$activity_args = array(
 				'post_id' => $this->course_id,
@@ -482,34 +482,34 @@ 
 			);
 
 		// Searching users on statuses requires sub-selecting the statuses by user_ids
-		if ( $this->search ) {
+		if ($this->search) {
 			$user_args = array(
-				'search' => '*' . $this->search . '*',
+				'search' => '*'.$this->search.'*',
 				'fields' => 'ID',
 			);
 			// Filter for extending
-			$user_args = apply_filters( 'sensei_analysis_course_search_users', $user_args );
-			if ( !empty( $user_args ) ) {
-				$learners_search = new WP_User_Query( $user_args );
+			$user_args = apply_filters('sensei_analysis_course_search_users', $user_args);
+			if ( ! empty($user_args)) {
+				$learners_search = new WP_User_Query($user_args);
 				// Store for reuse on counts
 				$activity_args['user_id'] = (array) $learners_search->get_results();
 			}
 		} // End If Statement
 
-		$activity_args = apply_filters( 'sensei_analysis_course_filter_statuses', $activity_args );
+		$activity_args = apply_filters('sensei_analysis_course_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'] ) {
-			$new_paged = floor( $this->total_items / $activity_args['number'] );
+		if ($this->total_items < $activity_args['offset']) {
+			$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;
 	} // End get_course_statuses()
@@ -519,32 +519,32 @@ 
 	 * @since  1.7.0
 	 * @return array statuses
 	 */
-	private function get_lessons( $args ) {
+	private function get_lessons($args) {
 
-		$lessons_args = array( 'post_type'         => 'lesson',
+		$lessons_args = array('post_type'         => 'lesson',
 							'posts_per_page'      => $args['number'],
 							'offset'              => $args['offset'],
-							'meta_key'            => '_order_' . $this->course_id,
+							'meta_key'            => '_order_'.$this->course_id,
 //							'orderby'             => $args['orderby'],
 							'order'               => $args['order'],
 							'meta_query'          => array(
 								array(
 									'key' => '_lesson_course',
-									'value' => intval( $this->course_id ),
+									'value' => intval($this->course_id),
 								),
 							),
 							'post_status'         => array('publish', 'private'),
 							'suppress_filters'    => 0
 							);
-		if ( $this->search ) {
+		if ($this->search) {
 			$lessons_args['s'] = $this->search;
 		}
-		if ( $this->csv_output ) {
+		if ($this->csv_output) {
 			$lessons_args['posts_per_page'] = '-1';
 		}
 
 		// Using WP_Query as get_posts() doesn't support 'found_posts'
-		$lessons_query = new WP_Query( apply_filters( 'sensei_analysis_course_filter_lessons', $lessons_args ) );
+		$lessons_query = new WP_Query(apply_filters('sensei_analysis_course_filter_lessons', $lessons_args));
 		$this->total_items = $lessons_query->found_posts;
 		return $lessons_query->posts;
 	} // End get_lessons()
@@ -556,17 +556,17 @@ 
 	 * @return void
 	 */
 	public function no_items() {
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'user' :
-				$text = __( 'No learners found.', 'woothemes-sensei' );
+				$text = __('No learners found.', 'woothemes-sensei');
 				break;
 
 			case 'lesson':
 			default:
-				$text = __( 'No lessons found.', 'woothemes-sensei' );
+				$text = __('No lessons found.', 'woothemes-sensei');
 				break;
 		}
-		echo apply_filters( 'sensei_analysis_course_no_items_text', $text );
+		echo apply_filters('sensei_analysis_course_no_items_text', $text);
 	} // End no_items()
 
 	/**
@@ -575,25 +575,25 @@ 
 	 * @return void
 	 */
 	public function data_table_header() {
-		if ( $this->user_id ) {
-			$learners_text = __( 'Other Learners taking this Course', 'woothemes-sensei' );
+		if ($this->user_id) {
+			$learners_text = __('Other Learners taking this Course', 'woothemes-sensei');
 		}
 		else {
-			$learners_text = __( 'Learners taking this Course', 'woothemes-sensei' );
+			$learners_text = __('Learners taking this Course', 'woothemes-sensei');
 		}
-		$lessons_text = __( 'Lessons in this Course', 'woothemes-sensei' );
+		$lessons_text = __('Lessons in this Course', 'woothemes-sensei');
 
 		$url_args = array(
 			'page' => $this->page_slug,
 			'course_id' => $this->course_id,
 		);
-		$learners_url = esc_url( add_query_arg( array_merge( $url_args, array( 'view' => 'user' ) ), admin_url( 'admin.php' ) ) );
-		$lessons_url = esc_url( add_query_arg( array_merge( $url_args, array( 'view' => 'lesson' ) ), admin_url( 'admin.php' ) ) );
+		$learners_url = esc_url(add_query_arg(array_merge($url_args, array('view' => 'user')), admin_url('admin.php')));
+		$lessons_url = esc_url(add_query_arg(array_merge($url_args, array('view' => 'lesson')), admin_url('admin.php')));
 
 		$learners_class = $lessons_class = '';
 
 		$menu = array();
-		switch( $this->view ) {
+		switch ($this->view) {
 			case 'user' :
 				$learners_class = 'current';
 				break;
@@ -603,17 +603,17 @@ 
 				$lessons_class = 'current';
 				break;
 		}
-		$menu['lesson'] = sprintf( '<a href="%s" class="%s">%s</a>', $lessons_url, $lessons_class, $lessons_text );
-		$menu['user'] = sprintf( '<a href="%s" class="%s">%s</a>', $learners_url, $learners_class, $learners_text );
-
-		$menu = apply_filters( 'sensei_analysis_course_sub_menu', $menu );
-		if ( !empty($menu) ) {
-			echo '<ul class="subsubsub">' . "\n";
-			foreach ( $menu as $class => $item ) {
-				$menu[ $class ] = "\t<li class='$class'>$item";
+		$menu['lesson'] = sprintf('<a href="%s" class="%s">%s</a>', $lessons_url, $lessons_class, $lessons_text);
+		$menu['user'] = sprintf('<a href="%s" class="%s">%s</a>', $learners_url, $learners_class, $learners_text);
+
+		$menu = apply_filters('sensei_analysis_course_sub_menu', $menu);
+		if ( ! empty($menu)) {
+			echo '<ul class="subsubsub">'."\n";
+			foreach ($menu as $class => $item) {
+				$menu[$class] = "\t<li class='$class'>$item";
 			}
-			echo implode( " |</li>\n", $menu ) . "</li>\n";
-			echo '</ul>' . "\n";
+			echo implode(" |</li>\n", $menu)."</li>\n";
+			echo '</ul>'."\n";
 		}
 	} // End data_table_header()
 
@@ -624,19 +624,19 @@ 
 	 */
 	public function data_table_footer() {
 
-		$course = get_post( $this->course_id );
-		$report = sanitize_title( $course->post_title ) . '-' . $this->view . 's-overview';
-		if ( $this->user_id ) {
-            $user_name = Sensei_Learner::get_full_name( $this->user_id );
-			$report = sanitize_title( $user_name  ) . '-' . $report;
+		$course = get_post($this->course_id);
+		$report = sanitize_title($course->post_title).'-'.$this->view.'s-overview';
+		if ($this->user_id) {
+            $user_name = Sensei_Learner::get_full_name($this->user_id);
+			$report = sanitize_title($user_name).'-'.$report;
 		}
 
-		$url_args = array( 'page' => $this->page_slug, 'course_id' => $this->course_id, 'view' => $this->view, 'sensei_report_download' => $report );
-		if ( $this->user_id ) {
+		$url_args = array('page' => $this->page_slug, 'course_id' => $this->course_id, 'view' => $this->view, 'sensei_report_download' => $report);
+		if ($this->user_id) {
 			$url_args['user_id'] = $this->user_id;
 		}
-		$url =  add_query_arg( $url_args, 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>';
+		$url = add_query_arg($url_args, 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>';
 	} // End data_table_footer()
 
 	/**
@@ -644,15 +644,15 @@ 
 	 * @since  1.7.0
 	 * @return string $text
 	 */
-	public function search_button( $text = '' ) {
-		switch( $this->view ) {
+	public function search_button($text = '') {
+		switch ($this->view) {
 			case 'user':
-				$text = __( 'Search Learners', 'woothemes-sensei' );
+				$text = __('Search Learners', 'woothemes-sensei');
 			break;
 
 			case 'lesson':
 			default:
-				$text = __( 'Search Lessons', 'woothemes-sensei' );
+				$text = __('Search Lessons', 'woothemes-sensei');
 			break;
 		} // End Switch Statement