woothemes / sensei

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

Indentation +15 added lines, -15 removed lines

@@ -53,10 +53,10 @@ 
 	 */

 	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() ) {

 			// No Output

@@ -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. */

@@ -172,8 +172,8 @@ 
 		$post_args = array(	'post_type' 		=> 'lesson',

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

 							'orderby'         	=> 'menu_order date',

-    						'order'           	=> 'DESC',

-    						'post_status'       => 'publish',

+							'order'           	=> 'DESC',

+							'post_status'       => 'publish',

 							'suppress_filters' 	=> 0

 							);

 		$posts_array = get_posts( $post_args );

@@ -181,14 +181,14 @@ 
 		if ( count( $posts_array ) > 0 ) { ?>

 			<ul>

 			<?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 ) );

-		    	$author_display_name = $user_info->display_name;

-		    	$author_id = $post_item->post_author;

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

-		    ?>

+				$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 ) );

+				$author_display_name = $user_info->display_name;

+				$author_id = $post_item->post_author;

+				$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>

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

Indentation +14 added lines, -14 removed lines

@@ -50,10 +50,10 @@ 
 	 */

 	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 );

@@ -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. */

@@ -157,8 +157,8 @@ 
 		$post_args = array(	'post_type' 		=> 'course',

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

 							'orderby'         	=> 'menu_order date',

-    						'order'           	=> 'ASC',

-    						'post_status'       => 'publish',

+							'order'           	=> 'ASC',

+							'post_status'       => 'publish',

 							'suppress_filters' 	=> 0,

 							);

 

@@ -175,13 +175,13 @@ 
 		if ( count( $posts_array ) > 0 ) { ?>

 			<ul>

 			<?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 ) );

-		    	$author_display_name = $user_info->display_name;

-		    	$author_id = $post_item->post_author;

-		    ?>

+				$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 ) );

+				$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>

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

Indentation +19 added lines, -19 removed lines

@@ -49,31 +49,31 @@ 
 	 */

 	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 );

+		/* Our variables from the widget settings. */

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

 

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

-        echo $before_widget;

+		/* 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; }

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

+		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' );

+		/* Widget content. */

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

+		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' );

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

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

 

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

-        echo $after_widget;

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

+		echo $after_widget;

 

 	} // End widget()

 

@@ -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. */

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

Indentation +49 added lines, -49 removed lines

@@ -10,10 +10,10 @@ 
  * @since 1.4.0
  */
 class Sensei_Learner_Profiles {
-    /**
-     * @var string
-     */
-    private $profile_url_base;
+	/**
+	 * @var string
+	 */
+	private $profile_url_base;
 
 	/**
 	 * Constructor.
@@ -42,8 +42,8 @@ 
 	 */
 	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%', '([^&]+)' );
@@ -62,7 +62,7 @@ 
 		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 . ' ';
 		}
@@ -110,7 +110,7 @@ 
 
 			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' );
 
 			}
 		}
@@ -140,43 +140,43 @@ 
 	 */
 	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 );
-
-        /**
-         * This filter runs inside the Sensei_Learner_Profiles::user_info function.
-         * Here you can change the user avatar.
-         *
-         * @since 1.0.0
-         *
-         * @param false|string `<img>` $user_avatar
-         */
+		/**
+		 * 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 );
+
+		/**
+		 * This filter runs inside the Sensei_Learner_Profiles::user_info function.
+		 * Here you can change the user avatar.
+		 *
+		 * @since 1.0.0
+		 *
+		 * @param false|string `<img>` $user_avatar
+		 */
 		$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.
-         * Here you can change the learner profile user display name.
-         * @since 1.0.0
-         *
-         * @param string $user_display_name
-         * @param string $user_id
-         */
+		/**
+		 * This filter runs inside the Sensei_Learner_Profiles::user_info function.
+		 * Here you can change the learner profile user display name.
+		 * @since 1.0.0
+		 *
+		 * @param string $user_display_name
+		 * @param string $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.
-         * With this filter can change the users description on the learner user info
-         * output.
-         *
-         * @since 1.0.0
-         *
-         * @param string $user_description
-         * @param string $user_id
-         */
+		/**
+		 * This filter runs inside the Sensei_Learner_Profiles::user_info function.
+		 * With this filter can change the users description on the learner user info
+		 * output.
+		 *
+		 * @since 1.0.0
+		 *
+		 * @param string $user_description
+		 * @param string $user_id
+		 */
 		$learner_bio = apply_filters( 'sensei_learner_profile_info_bio', $user->description, $user->ID );
 		?>
 
@@ -210,16 +210,16 @@ 
 		return $classes;
 	}
 
-    /**
-     * Deprecate the deprecate_sensei_learner_profile_content hook
-     *
-     * @since 1.9.0
-     */
-    public static function deprecate_sensei_learner_profile_content_hook(){
+	/**
+	 * Deprecate the deprecate_sensei_learner_profile_content hook
+	 *
+	 * @since 1.9.0
+	 */
+	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' );
 
-    }
+	}
 
 
 } // End Class

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

Indentation +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 class Sensei_Grading_Main extends WooThemes_Sensei_List_Table {
 
-    public $user_id;
+	public $user_id;
 	public $course_id;
 	public $lesson_id;
 	public $view;
@@ -249,7 +249,7 @@ 
 			$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 );
@@ -299,7 +299,7 @@ 
 	 */
 	public function no_items() {
 
-        _e( 'No submissions found.', 'woothemes-sensei' );
+		_e( 'No submissions found.', 'woothemes-sensei' );
 
 	} // End no_items()
 

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

Indentation +4 added lines, -4 removed lines

@@ -230,9 +230,9 @@ 
 		}
 
 		// 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' ) );
 
 			$user_name = '<strong><a class="row-title" href="' . esc_url( $url ) . '">' . $user_name . '</a></strong>';
@@ -339,9 +339,9 @@ 
 	 */
 	public function search_button( $text = '' ) {
 
-        $text =  __( 'Search Learners', 'woothemes-sensei' );
+		$text =  __( 'Search Learners', 'woothemes-sensei' );
 
-        return $text;
+		return $text;
 
 	}
 } // End Class

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/theme-integrations/twentyeleven.php

Indentation +22 added lines, -22 removed lines

@@ -1,36 +1,36 @@
 <?php
 /**
- * Class Sensei_Twentyeleven
- *
- * Responsible for wrapping twenty eleven theme Sensei content
- * with the correct markup
- *
- * @package Views
- * @subpackage Theme-Integration
- * @author Automattic
- *
- * @since 1.9.0
+	 * Class Sensei_Twentyeleven
+	 *
+	 * Responsible for wrapping twenty eleven theme Sensei content
+	 * with the correct markup
+	 *
+	 * @package Views
+	 * @subpackage Theme-Integration
+	 * @author Automattic
+	 *
+	 * @since 1.9.0
 */
 Class Sensei_Twentyeleven {
 
-    /**
-     * Output opening wrappers
-     * @since 1.9.0
-     */
-    public function wrapper_start(){
-    ?>
+	/**
+	 * Output opening wrappers
+	 * @since 1.9.0
+	 */
+	public function wrapper_start(){
+	?>
 
         <div id="primary">
             <div id="content" role="main">
 
     <?php }
 
-    /**
-     * Output closing wrappers
-     *
-     * @since 1.9.0
-     */
-    public function wrapper_end(){ ?>
+	/**
+	 * Output closing wrappers
+	 *
+	 * @since 1.9.0
+	 */
+	public function wrapper_end(){ ?>
 
             </div>
         </div>

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-utils.php

Spacing +531 added lines, -531 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 
 /**
  * Sensei Utilities Class
@@ -18,9 +18,9 @@ 
 	 * @since   1.0.0
 	 * @return  string The URL to the placeholder thumbnail image.
 	 */
-	public static function get_placeholder_image () {
+	public static function get_placeholder_image() {
 
-		return esc_url( apply_filters( 'sensei_placeholder_thumbnail', Sensei()->plugin_url . 'assets/images/placeholder.png' ) );
+		return esc_url(apply_filters('sensei_placeholder_thumbnail', Sensei()->plugin_url.'assets/images/placeholder.png'));
 	} // End get_placeholder_image()
 
 	/**
@@ -32,7 +32,7 @@ 
 	 * @static
 	 * @return bool
 	 */
-	public static function sensei_is_woocommerce_present () {
+	public static function sensei_is_woocommerce_present() {
 
         return Sensei_WC::is_woocommerce_present();
 
@@ -47,7 +47,7 @@ 
 	 * @static
 	 * @return boolean
 	 */
-	public static function sensei_is_woocommerce_activated () {
+	public static function sensei_is_woocommerce_activated() {
 
 		return  Sensei_WC::is_woocommerce_active();
 
@@ -60,70 +60,70 @@ 
 	 * @param  array $args (default: array())
 	 * @return bool | int
 	 */
-	public static function sensei_log_activity ( $args = array() ) {
+	public static function sensei_log_activity($args = array()) {
 		global $wpdb;
 
 		// Args, minimum data required for WP
 		$data = array(
-					'comment_post_ID' => intval( $args['post_id'] ),
+					'comment_post_ID' => intval($args['post_id']),
 					'comment_author' => '', // Not needed
 					'comment_author_email' => '', // Not needed
 					'comment_author_url' => '', // Not needed
-					'comment_content' => !empty($args['data']) ? esc_html( $args['data'] ) : '',
-					'comment_type' => esc_attr( $args['type'] ),
-					'user_id' => intval( $args['user_id'] ),
-					'comment_approved' => !empty($args['status']) ? esc_html( $args['status'] ) : 'log', // 'log' == 'sensei_user_answer'
+					'comment_content' => ! empty($args['data']) ? esc_html($args['data']) : '',
+					'comment_type' => esc_attr($args['type']),
+					'user_id' => intval($args['user_id']),
+					'comment_approved' => ! empty($args['status']) ? esc_html($args['status']) : 'log', // 'log' == 'sensei_user_answer'
 				);
 		// Allow extra data
-		if ( !empty($args['username']) ) {
-			$data['comment_author'] = sanitize_user( $args['username'] );
+		if ( ! empty($args['username'])) {
+			$data['comment_author'] = sanitize_user($args['username']);
 		}
-		if ( !empty($args['user_email']) ) {
-			$data['comment_author_email'] = sanitize_email( $args['user_email'] );
+		if ( ! empty($args['user_email'])) {
+			$data['comment_author_email'] = sanitize_email($args['user_email']);
 		}
-		if ( !empty($args['user_url']) ) {
-			$data['comment_author_url'] = esc_url( $args['user_url'] );
+		if ( ! empty($args['user_url'])) {
+			$data['comment_author_url'] = esc_url($args['user_url']);
 		}
-		if ( !empty($args['parent']) ) {
+		if ( ! empty($args['parent'])) {
 			$data['comment_parent'] = $args['parent'];
 		}
 		// Sanity check
-		if ( empty($args['user_id']) ) {
-			_deprecated_argument( __FUNCTION__, '1.0', __('At no point should user_id be equal to 0.', 'woothemes-sensei') );
+		if (empty($args['user_id'])) {
+			_deprecated_argument(__FUNCTION__, '1.0', __('At no point should user_id be equal to 0.', 'woothemes-sensei'));
 			return false;
 		}
 
-		do_action( 'sensei_log_activity_before', $args, $data );
+		do_action('sensei_log_activity_before', $args, $data);
 
 		$flush_cache = false;
 
 		// Custom Logic
 		// Check if comment exists first
-		$comment_id = $wpdb->get_var( $wpdb->prepare( "SELECT comment_ID FROM $wpdb->comments WHERE comment_post_ID = %d AND user_id = %d AND comment_type = %s ", $args['post_id'], $args['user_id'], $args['type'] ) );
-		if ( ! $comment_id ) {
+		$comment_id = $wpdb->get_var($wpdb->prepare("SELECT comment_ID FROM $wpdb->comments WHERE comment_post_ID = %d AND user_id = %d AND comment_type = %s ", $args['post_id'], $args['user_id'], $args['type']));
+		if ( ! $comment_id) {
 			// Add the comment
-			$comment_id = wp_insert_comment( $data );
+			$comment_id = wp_insert_comment($data);
 
 			$flush_cache = true;
-		} elseif ( isset( $args['action'] ) && 'update' == $args['action'] ) {
+		} elseif (isset($args['action']) && 'update' == $args['action']) {
 			// Update the comment if an update was requested
 			$data['comment_ID'] = $comment_id;
 			// By default update the timestamp of the comment
-			if ( empty($args['keep_time']) ) {
+			if (empty($args['keep_time'])) {
 				$data['comment_date'] = current_time('mysql');
 			}
-			wp_update_comment( $data );
+			wp_update_comment($data);
 			$flush_cache = true;
 		} // End If Statement
 
 		// Manually Flush the Cache
-		if ( $flush_cache ) {
+		if ($flush_cache) {
 			wp_cache_flush();
 		}
 
-		do_action( 'sensei_log_activity_after', $args, $data,  $comment_id );
+		do_action('sensei_log_activity_after', $args, $data, $comment_id);
 
-		if ( 0 < $comment_id ) {
+		if (0 < $comment_id) {
 			// Return the ID so that it can be used for meta data storage
 			return $comment_id;
 		} else {
@@ -140,41 +140,41 @@ 
 	 * @param  bool $return_comments (default: false)
 	 * @return mixed | int
 	 */
-	public static function sensei_check_for_activity ( $args = array(), $return_comments = false ) {
+	public static function sensei_check_for_activity($args = array(), $return_comments = false) {
 
 		global  $wp_version;
-		if ( !$return_comments ) {
+		if ( ! $return_comments) {
 			$args['count'] = true;
 		}
 
 		// Are we only retrieving a single entry, or not care about the order...
-		if ( isset( $args['count'] ) || isset( $args['post_id'] ) ){
+		if (isset($args['count']) || isset($args['post_id'])) {
 
 			// ...then we don't need to ask the db to order the results, this overrides WP default behaviour
-			if ( version_compare( $wp_version, '4.1', '>=' ) ) {
+			if (version_compare($wp_version, '4.1', '>=')) {
 				$args['order'] = false;
 				$args['orderby'] = false;
 			}
 		}
 
 		// A user ID of 0 is in valid, so shortcut this
-		if ( isset( $args['user_id'] ) && 0 == intval ( $args['user_id'] ) ) {
-			_deprecated_argument( __FUNCTION__, '1.0', __('At no point should user_id be equal to 0.', 'woothemes-sensei') );
+		if (isset($args['user_id']) && 0 == intval($args['user_id'])) {
+			_deprecated_argument(__FUNCTION__, '1.0', __('At no point should user_id be equal to 0.', 'woothemes-sensei'));
 			return false;
 		}
 		// Check for legacy code
-		if ( isset($args['type']) && in_array($args['type'], array('sensei_course_start', 'sensei_course_end', 'sensei_lesson_start', 'sensei_lesson_end', 'sensei_quiz_asked', 'sensei_user_grade', 'sensei_quiz_grade', 'sense_answer_notes') ) ) {
-			_deprecated_argument( __FUNCTION__, '1.7', sprintf( __('Sensei activity type %s is no longer used.', 'woothemes-sensei'), $args['type'] ) );
+		if (isset($args['type']) && in_array($args['type'], array('sensei_course_start', 'sensei_course_end', 'sensei_lesson_start', 'sensei_lesson_end', 'sensei_quiz_asked', 'sensei_user_grade', 'sensei_quiz_grade', 'sense_answer_notes'))) {
+			_deprecated_argument(__FUNCTION__, '1.7', sprintf(__('Sensei activity type %s is no longer used.', 'woothemes-sensei'), $args['type']));
 			return false;
 		}
 		// Are we checking for specific comment_approved statuses?
-		if ( isset($args['status']) ) {
+		if (isset($args['status'])) {
 			// Temporarily store as a custom status if requesting an array...
-			if ( is_array( $args['status'] ) && version_compare($wp_version, '4.1', '<') ) {
+			if (is_array($args['status']) && version_compare($wp_version, '4.1', '<')) {
 				// Encode now, decode later
-				$args['status'] = implode( ",", $args['status'] );
+				$args['status'] = implode(",", $args['status']);
 				// ...use a filter to switch the encoding back
-				add_filter( 'comments_clauses', array( __CLASS__, 'comment_multiple_status_filter' ) );
+				add_filter('comments_clauses', array(__CLASS__, 'comment_multiple_status_filter'));
 			}
 		}
 		else {
@@ -182,8 +182,8 @@ 
 		}
 
 		// Take into account WP < 4.1 will automatically add ' comment_approved = 1 OR comment_approved = 0 '
-		if ( ( is_array( $args['status'] ) || 'any' == $args['status'] ) && version_compare($wp_version, '4.1', '<') ) {
-			add_filter( 'comments_clauses', array( __CLASS__, 'comment_any_status_filter' ) );
+		if ((is_array($args['status']) || 'any' == $args['status']) && version_compare($wp_version, '4.1', '<')) {
+			add_filter('comments_clauses', array(__CLASS__, 'comment_any_status_filter'));
 		}
 
         //Get the comments
@@ -194,14 +194,14 @@ 
          *
          * @param int|array $comments
          */
-        $comments = apply_filters('sensei_check_for_activity', get_comments( $args ) );
+        $comments = apply_filters('sensei_check_for_activity', get_comments($args));
 
-		remove_filter( 'comments_clauses', array( __CLASS__, 'comment_multiple_status_filter' ) );
-		remove_filter( 'comments_clauses', array( __CLASS__, 'comment_any_status_filter' ) );
+		remove_filter('comments_clauses', array(__CLASS__, 'comment_multiple_status_filter'));
+		remove_filter('comments_clauses', array(__CLASS__, 'comment_any_status_filter'));
 		// Return comments
-		if ( $return_comments ) {
+		if ($return_comments) {
 			// Could check for array of 1 and just return the 1 item?
-			if ( is_array($comments) && 1 == count($comments) ) {
+			if (is_array($comments) && 1 == count($comments)) {
 				$comments = array_shift($comments);
 			}
 
@@ -219,31 +219,31 @@ 
 	 * @param  array $args (default: array())
 	 * @return array
 	 */
-	public static function sensei_activity_ids ( $args = array() ) {
+	public static function sensei_activity_ids($args = array()) {
 
 
-		$comments = Sensei_Utils::sensei_check_for_activity( $args, true );
+		$comments = Sensei_Utils::sensei_check_for_activity($args, true);
 		// Need to always use an array, even with only 1 item
-		if ( !is_array($comments) ) {
-			$comments = array( $comments );
+		if ( ! is_array($comments)) {
+			$comments = array($comments);
 		}
 
 		$post_ids = array();
 		// Count comments
-		if ( is_array( $comments ) && ( 0 < intval( count( $comments ) ) ) ) {
-			foreach ( $comments as $key => $value  ) {
+		if (is_array($comments) && (0 < intval(count($comments)))) {
+			foreach ($comments as $key => $value) {
 				// Add matches to id array
-				if ( isset( $args['field'] ) && 'comment' == $args['field'] ) {
-					array_push( $post_ids, $value->comment_ID );
-				} elseif( isset( $args['field'] ) && 'user_id' == $args['field'] ) {
-					array_push( $post_ids, $value->user_id );
+				if (isset($args['field']) && 'comment' == $args['field']) {
+					array_push($post_ids, $value->comment_ID);
+				} elseif (isset($args['field']) && 'user_id' == $args['field']) {
+					array_push($post_ids, $value->user_id);
 				} else {
-					array_push( $post_ids, $value->comment_post_ID );
+					array_push($post_ids, $value->comment_post_ID);
 				} // End If Statement
 			} // End For Loop
 			// Reset array indexes
-			$post_ids = array_unique( $post_ids );
-			$post_ids = array_values( $post_ids );
+			$post_ids = array_unique($post_ids);
+			$post_ids = array_values($post_ids);
 		} // End If Statement
 
 		return $post_ids;
@@ -257,20 +257,20 @@ 
 	 * @param  array $args (default: array())
 	 * @return boolean
 	 */
-	public static function sensei_delete_activities ( $args = array() ) {
+	public static function sensei_delete_activities($args = array()) {
 
 		$dataset_changes = false;
 
 		// If activity exists remove activity from log
-		$comments = Sensei_Utils::sensei_check_for_activity( array( 'post_id' => intval( $args['post_id'] ), 'user_id' => intval( $args['user_id'] ), 'type' => esc_attr( $args['type'] ) ), true );
-		if( $comments ) {
+		$comments = Sensei_Utils::sensei_check_for_activity(array('post_id' => intval($args['post_id']), 'user_id' => intval($args['user_id']), 'type' => esc_attr($args['type'])), true);
+		if ($comments) {
 			// Need to always return an array, even with only 1 item
-			if ( !is_array( $comments ) ) {
-				$comments = array( $comments );
+			if ( ! is_array($comments)) {
+				$comments = array($comments);
 			}
-			foreach ( $comments as $key => $value  ) {
-				if ( isset( $value->comment_ID ) && 0 < $value->comment_ID ) {
-					$dataset_changes = wp_delete_comment( intval( $value->comment_ID ), true );
+			foreach ($comments as $key => $value) {
+				if (isset($value->comment_ID) && 0 < $value->comment_ID) {
+					$dataset_changes = wp_delete_comment(intval($value->comment_ID), true);
 				} // End If Statement
 			} // End For Loop
 			// Manually flush the cache
@@ -286,25 +286,25 @@ 
      * @param  integer $user_id User ID
      * @return boolean
      */
-    public static function delete_all_user_activity( $user_id = 0 ) {
+    public static function delete_all_user_activity($user_id = 0) {
 
     	$dataset_changes = false;
 
-    	if( $user_id ) {
+    	if ($user_id) {
 
-			$activities = Sensei_Utils::sensei_check_for_activity( array( 'user_id' => $user_id ), true );
+			$activities = Sensei_Utils::sensei_check_for_activity(array('user_id' => $user_id), true);
 
-			if( $activities ) {
+			if ($activities) {
 
 				// Need to always return an array, even with only 1 item
-				if ( ! is_array( $activities ) ) {
-					$activities = array( $activities );
+				if ( ! is_array($activities)) {
+					$activities = array($activities);
 				}
 
-				foreach( $activities as $activity ) {
-					if( '' == $activity->comment_type ) continue;
-					if( strpos( 'sensei_', $activity->comment_type ) != 0 ) continue;
-					$dataset_changes = wp_delete_comment( intval( $activity->comment_ID ), true );
+				foreach ($activities as $activity) {
+					if ('' == $activity->comment_type) continue;
+					if (strpos('sensei_', $activity->comment_type) != 0) continue;
+					$dataset_changes = wp_delete_comment(intval($activity->comment_ID), true);
 					wp_cache_flush();
 				}
 			}
@@ -321,14 +321,14 @@ 
 	 * @param  array $args (default: array())
 	 * @return string
 	 */
-	public static function sensei_get_activity_value ( $args = array() ) {
+	public static function sensei_get_activity_value($args = array()) {
 
 
 		$activity_value = false;
-		if ( !empty($args['field']) ) {
-			$comment = Sensei_Utils::sensei_check_for_activity( $args, true );
+		if ( ! empty($args['field'])) {
+			$comment = Sensei_Utils::sensei_check_for_activity($args, true);
 
-			if ( isset( $comment->{$args['field']} ) && '' != $comment->{$args['field']} ) {
+			if (isset($comment->{$args['field']} ) && '' != $comment->{$args['field']} ) {
 				$activity_value = $comment->{$args['field']};
 			} // End If Statement
 		}
@@ -346,22 +346,22 @@ 
      * @param  int $product_id
      * @return bool
      */
-    public static function sensei_customer_bought_product ( $customer_email, $user_id, $product_id ) {
+    public static function sensei_customer_bought_product($customer_email, $user_id, $product_id) {
 
         $emails = array();
 
-        if ( $user_id ) {
-            $user = get_user_by( 'id', intval( $user_id ) );
+        if ($user_id) {
+            $user = get_user_by('id', intval($user_id));
             $emails[] = $user->user_email;
         }
 
-        if ( is_email( $customer_email ) )
+        if (is_email($customer_email))
             $emails[] = $customer_email;
 
-        if ( sizeof( $emails ) == 0 )
+        if (sizeof($emails) == 0)
             return false;
 
-        return Sensei_WC::has_customer_bought_product( $user_id, $product_id );
+        return Sensei_WC::has_customer_bought_product($user_id, $product_id);
 
     } // End sensei_customer_bought_product()
 
@@ -372,9 +372,9 @@ 
 	 * @param  string $input_name Name for text area form element
 	 * @return void
 	 */
-	public static function sensei_text_editor( $content = '', $editor_id = 'senseitexteditor', $input_name = '' ) {
+	public static function sensei_text_editor($content = '', $editor_id = 'senseitexteditor', $input_name = '') {
 
-		if( ! $input_name ) $input_name = $editor_id;
+		if ( ! $input_name) $input_name = $editor_id;
 
 		$buttons = 'bold,italic,underline,strikethrough,blockquote,bullist,numlist,justifyleft,justifycenter,justifyright,undo,redo,pastetext';
 
@@ -392,7 +392,7 @@ 
 			'quicktags' => false
 		);
 
-		wp_editor( $content, $editor_id, $settings );
+		wp_editor($content, $editor_id, $settings);
 
 	} // End sensei_text_editor()
 
@@ -402,57 +402,57 @@ 
      * @param int $user_id
 	 * @return boolean            Whether the answers were saved or not
 	 */
-	public static function sensei_save_quiz_answers( $submitted = array(), $user_id = 0 ) {
+	public static function sensei_save_quiz_answers($submitted = array(), $user_id = 0) {
 
-		if( intval( $user_id ) == 0 ) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$answers_saved = false;
 
-		if( $submitted && intval( $user_id ) > 0 ) {
+		if ($submitted && intval($user_id) > 0) {
 
-			foreach( $submitted as $question_id => $answer ) {
+			foreach ($submitted as $question_id => $answer) {
 
 				// Get question type
-				$question_type = Sensei()->question->get_question_type( $question_id );
+				$question_type = Sensei()->question->get_question_type($question_id);
 
 				// Sanitise answer
-				if( 0 == get_magic_quotes_gpc() ) {
-					$answer = wp_unslash( $answer );
+				if (0 == get_magic_quotes_gpc()) {
+					$answer = wp_unslash($answer);
 				}
-				switch( $question_type ) {
-					case 'multi-line': $answer = nl2br( $answer ); break;
+				switch ($question_type) {
+					case 'multi-line': $answer = nl2br($answer); break;
 					case 'single-line': break;
 					case 'gap-fill': break;
-					default: $answer = maybe_serialize( $answer ); break;
+					default: $answer = maybe_serialize($answer); break;
 				}
 				$args = array(
 							'post_id' => $question_id,
-							'data' => base64_encode( $answer ),
+							'data' => base64_encode($answer),
 							'type' => 'sensei_user_answer', /* FIELD SIZE 20 */
 							'user_id' => $user_id,
 							'action' => 'update'
 						);
-				$answers_saved = Sensei_Utils::sensei_log_activity( $args );
+				$answers_saved = Sensei_Utils::sensei_log_activity($args);
 			}
 
 			// Handle file upload questions
-			if( isset( $_FILES ) ) {
-				foreach( $_FILES as $field => $file ) {
-					if( strpos( $field, 'file_upload_' ) !== false ) {
-						$question_id = str_replace( 'file_upload_', '', $field );
-						if( $file && $question_id ) {
-							$attachment_id = self::upload_file( $file );
-							if( $attachment_id ) {
+			if (isset($_FILES)) {
+				foreach ($_FILES as $field => $file) {
+					if (strpos($field, 'file_upload_') !== false) {
+						$question_id = str_replace('file_upload_', '', $field);
+						if ($file && $question_id) {
+							$attachment_id = self::upload_file($file);
+							if ($attachment_id) {
 								$args = array(
 									'post_id' => $question_id,
-									'data' => base64_encode( $attachment_id ),
+									'data' => base64_encode($attachment_id),
 									'type' => 'sensei_user_answer', /* FIELD SIZE 20 */
 									'user_id' => $user_id,
 									'action' => 'update'
 								);
-								$answers_saved = Sensei_Utils::sensei_log_activity( $args );
+								$answers_saved = Sensei_Utils::sensei_log_activity($args);
 							}
 						}
 					}
@@ -464,9 +464,9 @@ 
 
 	} // End sensei_save_quiz_answers()
 
-	public static function upload_file( $file = array() ) {
+	public static function upload_file($file = array()) {
 
-		require_once( ABSPATH . 'wp-admin/includes/admin.php' );
+		require_once(ABSPATH.'wp-admin/includes/admin.php');
 
         /**
          * Filter the data array for the Sensei wp_handle_upload function call
@@ -481,11 +481,11 @@ 
          *     @type string test_form set to false by default
          * }
          */
-        $file_upload_args = apply_filters( 'sensei_file_upload_args', array('test_form' => false ) );
+        $file_upload_args = apply_filters('sensei_file_upload_args', array('test_form' => false));
 
-        $file_return = wp_handle_upload( $file, $file_upload_args );
+        $file_return = wp_handle_upload($file, $file_upload_args);
 
-        if( isset( $file_return['error'] ) || isset( $file_return['upload_error_handler'] ) ) {
+        if (isset($file_return['error']) || isset($file_return['upload_error_handler'])) {
             return false;
         } else {
 
@@ -493,19 +493,19 @@ 
 
             $attachment = array(
                 'post_mime_type' => $file_return['type'],
-                'post_title' => preg_replace( '/\.[^.]+$/', '', basename( $filename ) ),
+                'post_title' => preg_replace('/\.[^.]+$/', '', basename($filename)),
                 'post_content' => '',
                 'post_status' => 'inherit',
                 'guid' => $file_return['url']
             );
 
-            $attachment_id = wp_insert_attachment( $attachment, $filename );
+            $attachment_id = wp_insert_attachment($attachment, $filename);
 
-            require_once(ABSPATH . 'wp-admin/includes/image.php');
-            $attachment_data = wp_generate_attachment_metadata( $attachment_id, $filename );
-            wp_update_attachment_metadata( $attachment_id, $attachment_data );
+            require_once(ABSPATH.'wp-admin/includes/image.php');
+            $attachment_data = wp_generate_attachment_metadata($attachment_id, $filename);
+            wp_update_attachment_metadata($attachment_id, $attachment_data);
 
-            if( 0 < intval( $attachment_id ) ) {
+            if (0 < intval($attachment_id)) {
             	return $attachment_id;
             }
         }
@@ -531,9 +531,9 @@ 
      *
 	 * @return int $quiz_grade total sum of all question grades
 	 */
-	public static function sensei_grade_quiz_auto( $quiz_id = 0, $submitted = array(), $total_questions = 0, $quiz_grade_type = 'auto' ) {
+	public static function sensei_grade_quiz_auto($quiz_id = 0, $submitted = array(), $total_questions = 0, $quiz_grade_type = 'auto') {
 
-        return Sensei_Grading::grade_quiz_auto( $quiz_id, $submitted, $total_questions, $quiz_grade_type );
+        return Sensei_Grading::grade_quiz_auto($quiz_id, $submitted, $total_questions, $quiz_grade_type);
 
 	} // End sensei_grade_quiz_auto()
 
@@ -545,20 +545,20 @@ 
      * @param  string $quiz_grade_type default 'auto'
 	 * @return boolean
 	 */
-	public static function sensei_grade_quiz( $quiz_id = 0, $grade = 0, $user_id = 0, $quiz_grade_type = 'auto' ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_grade_quiz($quiz_id = 0, $grade = 0, $user_id = 0, $quiz_grade_type = 'auto') {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$activity_logged = false;
-		if( intval( $quiz_id ) > 0 && intval( $user_id ) > 0 ) {
-			$lesson_id = get_post_meta( $quiz_id, '_quiz_lesson', true );
-			$user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
-			$activity_logged = update_comment_meta( $user_lesson_status->comment_ID, 'grade', $grade );
+		if (intval($quiz_id) > 0 && intval($user_id) > 0) {
+			$lesson_id = get_post_meta($quiz_id, '_quiz_lesson', true);
+			$user_lesson_status = Sensei_Utils::user_lesson_status($lesson_id, $user_id);
+			$activity_logged = update_comment_meta($user_lesson_status->comment_ID, 'grade', $grade);
 
-			$quiz_passmark = absint( get_post_meta( $quiz_id, '_quiz_passmark', true ) );
+			$quiz_passmark = absint(get_post_meta($quiz_id, '_quiz_passmark', true));
 
-			do_action( 'sensei_user_quiz_grade', $user_id, $quiz_id, $grade, $quiz_passmark, $quiz_grade_type );
+			do_action('sensei_user_quiz_grade', $user_id, $quiz_id, $grade, $quiz_passmark, $quiz_grade_type);
 		}
 
 		return $activity_logged;
@@ -578,9 +578,9 @@ 
      *
 	 * @return int $question_grade
 	 */
-	public static function sensei_grade_question_auto( $question_id = 0, $question_type = '', $answer = '', $user_id = 0 ) {
+	public static function sensei_grade_question_auto($question_id = 0, $question_type = '', $answer = '', $user_id = 0) {
 
-       return  WooThemes_Sensei_Grading::grade_question_auto( $question_id, $question_type, $answer, $user_id  );
+       return  WooThemes_Sensei_Grading::grade_question_auto($question_id, $question_type, $answer, $user_id);
 
 	} // end sensei_grade_question_auto
 
@@ -591,20 +591,20 @@ 
      * @param int $user_id
 	 * @return boolean
 	 */
-	public static function sensei_grade_question( $question_id = 0, $grade = 0, $user_id = 0 ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_grade_question($question_id = 0, $grade = 0, $user_id = 0) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$activity_logged = false;
-		if( intval( $question_id ) > 0 && intval( $user_id ) > 0 ) {
+		if (intval($question_id) > 0 && intval($user_id) > 0) {
 
-			$user_answer_id = Sensei_Utils::sensei_get_activity_value( array( 'post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID' ) );
-			$activity_logged = update_comment_meta( $user_answer_id, 'user_grade', $grade );
+			$user_answer_id = Sensei_Utils::sensei_get_activity_value(array('post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID'));
+			$activity_logged = update_comment_meta($user_answer_id, 'user_grade', $grade);
 
-			$answer_notes = get_post_meta( $question_id, '_answer_feedback', true );
-			if ( !empty($answer_notes) ) {
-				update_comment_meta( $user_answer_id, 'answer_note', base64_encode( $answer_notes ) );
+			$answer_notes = get_post_meta($question_id, '_answer_feedback', true);
+			if ( ! empty($answer_notes)) {
+				update_comment_meta($user_answer_id, 'answer_note', base64_encode($answer_notes));
 			}
 
 		}
@@ -612,15 +612,15 @@ 
 		return $activity_logged;
 	}
 
-	public static function sensei_delete_question_grade( $question_id = 0, $user_id = 0 ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_delete_question_grade($question_id = 0, $user_id = 0) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$activity_logged = false;
-		if( intval( $question_id ) > 0 ) {
-			$user_answer_id = Sensei_Utils::sensei_get_activity_value( array( 'post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID' ) );
-			$activity_logged = delete_comment_meta( $user_answer_id, 'user_grade' );
+		if (intval($question_id) > 0) {
+			$user_answer_id = Sensei_Utils::sensei_get_activity_value(array('post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID'));
+			$activity_logged = delete_comment_meta($user_answer_id, 'user_grade');
 		}
 
 		return $activity_logged;
@@ -638,9 +638,9 @@ 
      *
      * @return mixed boolean or comment_ID
      */
-    public static function user_start_lesson(  $user_id = 0, $lesson_id = 0, $complete = false ) {
+    public static function user_start_lesson($user_id = 0, $lesson_id = 0, $complete = false) {
 
-        return self::sensei_start_lesson( $lesson_id, $user_id, $complete );
+        return self::sensei_start_lesson($lesson_id, $user_id, $complete);
 
     }// end user_start_lesson()
 
@@ -657,22 +657,22 @@ 
      *
      * @return mixed boolean or comment_ID
 	 */
-	public static function sensei_start_lesson( $lesson_id = 0, $user_id = 0, $complete = false ) {
+	public static function sensei_start_lesson($lesson_id = 0, $user_id = 0, $complete = false) {
 
 
-		if( intval( $user_id ) == 0 ) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$activity_logged = false;
 
-		if( intval( $lesson_id ) > 0 ) {
+		if (intval($lesson_id) > 0) {
 
-			$course_id = get_post_meta( $lesson_id, '_lesson_course', true );
-			if( $course_id ) {
-				$is_user_taking_course = Sensei_Utils::user_started_course( $course_id, $user_id );
-				if( ! $is_user_taking_course ) {
-					Sensei_Utils::user_start_course( $user_id, $course_id );
+			$course_id = get_post_meta($lesson_id, '_lesson_course', true);
+			if ($course_id) {
+				$is_user_taking_course = Sensei_Utils::user_started_course($course_id, $user_id);
+				if ( ! $is_user_taking_course) {
+					Sensei_Utils::user_start_course($user_id, $course_id);
 				}
 			}
 
@@ -680,12 +680,12 @@ 
 			$status = 'in-progress';
 
 			// Note: When this action runs the lesson status may not yet exist
-			do_action( 'sensei_user_lesson_start', $user_id, $lesson_id );
+			do_action('sensei_user_lesson_start', $user_id, $lesson_id);
 
-			if( $complete ) {
+			if ($complete) {
 
-				$has_questions = get_post_meta( $lesson_id, '_quiz_has_questions', true );
-				if ( $has_questions ) {
+				$has_questions = get_post_meta($lesson_id, '_quiz_has_questions', true);
+				if ($has_questions) {
 					$status = 'passed'; // Force a pass
 					$metadata['grade'] = 0;
 				}
@@ -695,31 +695,31 @@ 
 			}
 
 			// Check if user is already taking the lesson
-			$activity_logged = Sensei_Utils::user_started_lesson( $lesson_id, $user_id );
-			if( ! $activity_logged ) {
+			$activity_logged = Sensei_Utils::user_started_lesson($lesson_id, $user_id);
+			if ( ! $activity_logged) {
 
 				$metadata['start'] = current_time('mysql');
-				$activity_logged = Sensei_Utils::update_lesson_status( $user_id, $lesson_id, $status, $metadata );
+				$activity_logged = Sensei_Utils::update_lesson_status($user_id, $lesson_id, $status, $metadata);
 
             } else {
 
                 // if users is already taking the lesson  and the status changes to complete update it
                 $current_user_activity = get_comment($activity_logged);
-                if( $status=='complete' &&
-                    $status != $current_user_activity->comment_approved  ){
+                if ($status == 'complete' &&
+                    $status != $current_user_activity->comment_approved) {
 
                     $comment = array();
                     $comment['comment_ID'] = $activity_logged;
                     $comment['comment_approved'] = $status;
-                    wp_update_comment( $comment );
+                    wp_update_comment($comment);
 
                 }
 
             }
 
-			if ( $complete ) {
+			if ($complete) {
 				// Run this *after* the lesson status has been created/updated
-				do_action( 'sensei_user_lesson_end', $user_id, $lesson_id );
+				do_action('sensei_user_lesson_end', $user_id, $lesson_id);
 			}
 
 		}
@@ -734,20 +734,20 @@ 
 	 * @param int $user_id
 	 * @return boolean
 	 */
-	public static function sensei_remove_user_from_lesson( $lesson_id = 0, $user_id = 0, $from_course = false ) {
+	public static function sensei_remove_user_from_lesson($lesson_id = 0, $user_id = 0, $from_course = false) {
 
 
-		if( ! $lesson_id ) return false;
+		if ( ! $lesson_id) return false;
 
-		if( intval( $user_id ) == 0 ) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		// Process quiz
-		$lesson_quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
+		$lesson_quiz_id = Sensei()->lesson->lesson_quizzes($lesson_id);
 
 		// Delete quiz answers, this auto deletes the corresponding meta data, such as the question/answer grade
-		Sensei_Utils::sensei_delete_quiz_answers( $lesson_quiz_id, $user_id );
+		Sensei_Utils::sensei_delete_quiz_answers($lesson_quiz_id, $user_id);
 
 		// Delete lesson status
 		$args = array(
@@ -756,10 +756,10 @@ 
 			'user_id' => $user_id,
 		);
 		// This auto deletes the corresponding meta data, such as the quiz grade, and questions asked
-		Sensei_Utils::sensei_delete_activities( $args );
+		Sensei_Utils::sensei_delete_activities($args);
 
-		if( ! $from_course ) {
-			do_action( 'sensei_user_lesson_reset', $user_id, $lesson_id );
+		if ( ! $from_course) {
+			do_action('sensei_user_lesson_reset', $user_id, $lesson_id);
 		}
 
 		return true;
@@ -772,19 +772,19 @@ 
 	 * @param int $user_id
 	 * @return boolean
 	 */
-	public static function sensei_remove_user_from_course( $course_id = 0, $user_id = 0 ) {
+	public static function sensei_remove_user_from_course($course_id = 0, $user_id = 0) {
 
 
-		if( ! $course_id ) return false;
+		if ( ! $course_id) return false;
 
-		if( intval( $user_id ) == 0 ) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
-		$lesson_ids = Sensei()->course->course_lessons( $course_id, 'any', 'ids' );
+		$lesson_ids = Sensei()->course->course_lessons($course_id, 'any', 'ids');
 
-		foreach( $lesson_ids as $lesson_id ) {
-			Sensei_Utils::sensei_remove_user_from_lesson( $lesson_id, $user_id, true );
+		foreach ($lesson_ids as $lesson_id) {
+			Sensei_Utils::sensei_remove_user_from_lesson($lesson_id, $user_id, true);
 		}
 
 		// Delete course status
@@ -794,36 +794,36 @@ 
 			'user_id' => $user_id,
 		);
 
-		Sensei_Utils::sensei_delete_activities( $args );
+		Sensei_Utils::sensei_delete_activities($args);
 
-		do_action( 'sensei_user_course_reset', $user_id, $course_id );
+		do_action('sensei_user_course_reset', $user_id, $course_id);
 
 		return true;
 	}
 
-	public static function sensei_get_quiz_questions( $quiz_id = 0 ) {
+	public static function sensei_get_quiz_questions($quiz_id = 0) {
 
 
 		$questions = array();
 
-		if( intval( $quiz_id ) > 0 ) {
-			$questions = Sensei()->lesson->lesson_quiz_questions( $quiz_id );
-			$questions = Sensei_Utils::array_sort_reorder( $questions );
+		if (intval($quiz_id) > 0) {
+			$questions = Sensei()->lesson->lesson_quiz_questions($quiz_id);
+			$questions = Sensei_Utils::array_sort_reorder($questions);
 		}
 
 		return $questions;
 	}
 
-	public static function sensei_get_quiz_total( $quiz_id = 0 ) {
+	public static function sensei_get_quiz_total($quiz_id = 0) {
 
 
 		$quiz_total = 0;
 
-		if( $quiz_id > 0 ) {
-			$questions = Sensei_Utils::sensei_get_quiz_questions( $quiz_id );
+		if ($quiz_id > 0) {
+			$questions = Sensei_Utils::sensei_get_quiz_questions($quiz_id);
 			$question_grade = 0;
-			foreach( $questions as $question ) {
-				$question_grade = Sensei()->question->get_question_grade( $question->ID );
+			foreach ($questions as $question) {
+				$question_grade = Sensei()->question->get_question_grade($question->ID);
 				$quiz_total += $question_grade;
 			}
 		}
@@ -838,20 +838,20 @@ 
 	 * @param int $user_id
 	 * @return string
 	 */
-	public static function sensei_get_user_question_grade( $question = 0, $user_id = 0 ) {
+	public static function sensei_get_user_question_grade($question = 0, $user_id = 0) {
 		$question_grade = false;
-		if( $question ) {
-			if ( is_object( $question ) ) {
+		if ($question) {
+			if (is_object($question)) {
 				$user_answer_id = $question->comment_ID;
 			}
 			else {
-				if( intval( $user_id ) == 0 ) {
+				if (intval($user_id) == 0) {
 					$user_id = get_current_user_id();
 				}
-				$user_answer_id = Sensei_Utils::sensei_get_activity_value( array( 'post_id' => intval($question), 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID' ) );
+				$user_answer_id = Sensei_Utils::sensei_get_activity_value(array('post_id' => intval($question), 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID'));
 			}
-			if ( $user_answer_id ) {
-				$question_grade = get_comment_meta( $user_answer_id, 'user_grade', true );
+			if ($user_answer_id) {
+				$question_grade = get_comment_meta($user_answer_id, 'user_grade', true);
 			}
 		}
 
@@ -866,52 +866,52 @@ 
 	 * @param int $user_id
 	 * @return string
 	 */
-	public static function sensei_get_user_question_answer_notes( $question = 0, $user_id = 0 ) {
+	public static function sensei_get_user_question_answer_notes($question = 0, $user_id = 0) {
 		$answer_notes = false;
-		if( $question ) {
-			if ( is_object( $question ) ) {
+		if ($question) {
+			if (is_object($question)) {
 				$user_answer_id = $question->comment_ID;
 			}
 			else {
-				if( intval( $user_id ) == 0 ) {
+				if (intval($user_id) == 0) {
 					$user_id = get_current_user_id();
 				}
-				$user_answer_id = Sensei_Utils::sensei_get_activity_value( array( 'post_id' => intval($question), 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID' ) );
+				$user_answer_id = Sensei_Utils::sensei_get_activity_value(array('post_id' => intval($question), 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID'));
 			}
-			if ( $user_answer_id ) {
-				$answer_notes = base64_decode( get_comment_meta( $user_answer_id, 'answer_note', true ) );
+			if ($user_answer_id) {
+				$answer_notes = base64_decode(get_comment_meta($user_answer_id, 'answer_note', true));
 			}
 		}
 
 		return $answer_notes;
 	}
 
-	public static function sensei_delete_quiz_answers( $quiz_id = 0, $user_id = 0 ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_delete_quiz_answers($quiz_id = 0, $user_id = 0) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$delete_answers = false;
-		if( intval( $quiz_id ) > 0 ) {
-			$questions = Sensei_Utils::sensei_get_quiz_questions( $quiz_id );
-			foreach( $questions as $question ) {
-				$delete_answers = Sensei_Utils::sensei_delete_activities( array( 'post_id' => $question->ID, 'user_id' => $user_id, 'type' => 'sensei_user_answer' ) );
+		if (intval($quiz_id) > 0) {
+			$questions = Sensei_Utils::sensei_get_quiz_questions($quiz_id);
+			foreach ($questions as $question) {
+				$delete_answers = Sensei_Utils::sensei_delete_activities(array('post_id' => $question->ID, 'user_id' => $user_id, 'type' => 'sensei_user_answer'));
 			}
 		}
 
 		return $delete_answers;
 	}
 
-	public static function sensei_delete_quiz_grade( $quiz_id = 0, $user_id = 0 ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_delete_quiz_grade($quiz_id = 0, $user_id = 0) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$delete_grade = false;
-		if( intval( $quiz_id ) > 0 ) {
-			$lesson_id = get_post_meta( $quiz_id, '_quiz_lesson', true );
-			$user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
-			$delete_grade = delete_comment_meta( $user_lesson_status->comment_ID, 'grade' );
+		if (intval($quiz_id) > 0) {
+			$lesson_id = get_post_meta($quiz_id, '_quiz_lesson', true);
+			$user_lesson_status = Sensei_Utils::user_lesson_status($lesson_id, $user_id);
+			$delete_grade = delete_comment_meta($user_lesson_status->comment_ID, 'grade');
 		}
 
 		return $delete_grade;
@@ -924,20 +924,20 @@ 
      * @param string $notes
 	 * @return boolean
 	 */
-	public static function sensei_add_answer_notes( $question_id = 0, $user_id = 0, $notes = '' ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_add_answer_notes($question_id = 0, $user_id = 0, $notes = '') {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$activity_logged = false;
 
-		if( intval( $question_id ) > 0 ) {
-			$notes = base64_encode( $notes );
+		if (intval($question_id) > 0) {
+			$notes = base64_encode($notes);
 
 			// Don't store empty values, no point
-			if ( !empty($notes) ) {
-				$user_lesson_id = Sensei_Utils::sensei_get_activity_value( array( 'post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID' ) );
-				$activity_logged = update_comment_meta( $user_lesson_id, 'answer_note', $notes );
+			if ( ! empty($notes)) {
+				$user_lesson_id = Sensei_Utils::sensei_get_activity_value(array('post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer', 'field' => 'comment_ID'));
+				$activity_logged = update_comment_meta($user_lesson_id, 'answer_note', $notes);
 			}
 			else {
 				$activity_logged = true;
@@ -953,16 +953,16 @@ 
 	 * @param  array $return_array data to be ordered
 	 * @return array $return_array ordered data
 	 */
-	public static function array_sort_reorder( $return_array ) {
-		if ( isset( $_GET['orderby'] ) && '' != esc_html( $_GET['orderby'] ) ) {
+	public static function array_sort_reorder($return_array) {
+		if (isset($_GET['orderby']) && '' != esc_html($_GET['orderby'])) {
 			$sort_key = '';
 			// if ( array_key_exists( esc_html( $_GET['orderby'] ), $this->sortable_columns ) ) {
 			// 	$sort_key = esc_html( $_GET['orderby'] );
 			// } // End If Statement
-			if ( '' != $sort_key ) {
-					Sensei_Utils::sort_array_by_key($return_array,$sort_key);
-				if ( isset( $_GET['order'] ) && 'desc' == esc_html( $_GET['order'] ) ) {
-					$return_array = array_reverse( $return_array, true );
+			if ('' != $sort_key) {
+					Sensei_Utils::sort_array_by_key($return_array, $sort_key);
+				if (isset($_GET['order']) && 'desc' == esc_html($_GET['order'])) {
+					$return_array = array_reverse($return_array, true);
 				} // End If Statement
 			} // End If Statement
 			return $return_array;
@@ -978,15 +978,15 @@ 
 	 * @param  $key string column name in array
 	 * @return void
 	 */
-	public static function sort_array_by_key( $array, $key ) {
+	public static function sort_array_by_key($array, $key) {
 	    $sorter = array();
 	    $ret = array();
-	    reset( $array );
-	    foreach ( $array as $ii => $va ) {
+	    reset($array);
+	    foreach ($array as $ii => $va) {
 	        $sorter[$ii] = $va[$key];
 	    } // End For Loop
-	    asort( $sorter );
-	    foreach ( $sorter as $ii => $va ) {
+	    asort($sorter);
+	    foreach ($sorter as $ii => $va) {
 	        $ret[$ii] = $array[$ii];
 	    } // End For Loop
 	    $array = $ret;
@@ -998,10 +998,10 @@ 
 	 * @param  integer $quiz_id
 	 * @return array of quiz questions
 	 */
-	public static function lesson_quiz_questions( $quiz_id = 0 ) {
+	public static function lesson_quiz_questions($quiz_id = 0) {
 		$questions_array = array();
-		if ( 0 < $quiz_id ) {
-			$question_args = array( 'post_type'         => 'question',
+		if (0 < $quiz_id) {
+			$question_args = array('post_type'         => 'question',
                                     'posts_per_page'       => -1,
                                     'orderby'           => 'ID',
                                     'order'             => 'ASC',
@@ -1014,7 +1014,7 @@ 
                                     'post_status'       => 'any',
                                     'suppress_filters'  => 0
                                 );
-            $questions_array = get_posts( $question_args );
+            $questions_array = get_posts($question_args);
         } // End If Statement
         return $questions_array;
 	} // End lesson_quiz_questions()
@@ -1024,25 +1024,25 @@ 
 	 * @param  integer $course_id ID of course
 	 * @return integer            Pass mark for course
 	 */
-	public static function sensei_course_pass_grade( $course_id = 0 ) {
+	public static function sensei_course_pass_grade($course_id = 0) {
 
 
 		$course_passmark = 0;
 
-		if( $course_id > 0 ) {
-			$lessons = Sensei()->course->course_lessons( $course_id );
+		if ($course_id > 0) {
+			$lessons = Sensei()->course->course_lessons($course_id);
 			$lesson_count = 0;
 			$total_passmark = 0;
-			foreach( $lessons as $lesson ) {
+			foreach ($lessons as $lesson) {
 
 				// Get Quiz ID
-				$quiz_id = Sensei()->lesson->lesson_quizzes( $lesson->ID );
+				$quiz_id = Sensei()->lesson->lesson_quizzes($lesson->ID);
 
 				// Check for a pass being required
-				$pass_required = get_post_meta( $quiz_id, '_pass_required', true );
-				if ( $pass_required ) {
+				$pass_required = get_post_meta($quiz_id, '_pass_required', true);
+				if ($pass_required) {
 					// Get quiz passmark
-					$quiz_passmark = absint( get_post_meta( $quiz_id, '_quiz_passmark', true ) );
+					$quiz_passmark = absint(get_post_meta($quiz_id, '_quiz_passmark', true));
 
 					// Add up total passmark
 					$total_passmark += $quiz_passmark;
@@ -1051,12 +1051,12 @@ 
 				}
 			}
 			// Might be a case of no required lessons
-			if ( $lesson_count ) {
-				$course_passmark = ( $total_passmark / $lesson_count );
+			if ($lesson_count) {
+				$course_passmark = ($total_passmark / $lesson_count);
 			}
 		}
 
-		return Sensei_Utils::round( $course_passmark );
+		return Sensei_Utils::round($course_passmark);
 	}
 
 	/**
@@ -1065,27 +1065,27 @@ 
 	 * @param  integer $user_id   ID of user
 	 * @return integer            User's total grade
 	 */
-	public static function sensei_course_user_grade( $course_id = 0, $user_id = 0 ) {
+	public static function sensei_course_user_grade($course_id = 0, $user_id = 0) {
 
 
-		if( intval( $user_id ) == 0 ) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$total_grade = 0;
 
-		if( $course_id > 0 && $user_id > 0 ) {
-			$lessons = Sensei()->course->course_lessons( $course_id );
+		if ($course_id > 0 && $user_id > 0) {
+			$lessons = Sensei()->course->course_lessons($course_id);
 			$lesson_count = 0;
 			$total_grade = 0;
-			foreach( $lessons as $lesson ) {
+			foreach ($lessons as $lesson) {
 
 				// Check for lesson having questions, thus a quiz, thus having a grade
-				$has_questions = get_post_meta( $lesson->ID, '_quiz_has_questions', true );
-				if ( $has_questions ) {
-					$user_lesson_status = Sensei_Utils::user_lesson_status( $lesson->ID, $user_id );
+				$has_questions = get_post_meta($lesson->ID, '_quiz_has_questions', true);
+				if ($has_questions) {
+					$user_lesson_status = Sensei_Utils::user_lesson_status($lesson->ID, $user_id);
 					// Get user quiz grade
-					$quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
+					$quiz_grade = get_comment_meta($user_lesson_status->comment_ID, 'grade', true);
 
 					// Add up total grade
 					$total_grade += $quiz_grade;
@@ -1095,13 +1095,13 @@ 
 			}
 
 			// Might be a case of no lessons with quizzes
-			if ( $lesson_count ) {
-				$total_grade = ( $total_grade / $lesson_count );
+			if ($lesson_count) {
+				$total_grade = ($total_grade / $lesson_count);
 			}
 
 		}
 
-		return Sensei_Utils::round( $total_grade );
+		return Sensei_Utils::round($total_grade);
 	}
 
 	/**
@@ -1110,18 +1110,18 @@ 
 	 * @param  integer $user_id   ID of user
 	 * @return boolean
 	 */
-	public static function sensei_user_passed_course( $course_id = 0, $user_id = 0 ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_user_passed_course($course_id = 0, $user_id = 0) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$pass = false;
 
-		if( $course_id > 0 && $user_id > 0 ) {
-			$passmark = Sensei_Utils::sensei_course_pass_grade( $course_id );
-			$user_grade = Sensei_Utils::sensei_course_user_grade( $course_id, $user_id );
+		if ($course_id > 0 && $user_id > 0) {
+			$passmark = Sensei_Utils::sensei_course_pass_grade($course_id);
+			$user_grade = Sensei_Utils::sensei_course_user_grade($course_id, $user_id);
 
-			if( $user_grade >= $passmark ) {
+			if ($user_grade >= $passmark) {
 				$pass = true;
 			}
 		}
@@ -1136,38 +1136,38 @@ 
 	 * @param  integer $user_id   ID of user
 	 * @return array              Status code and message
 	 */
-	public static function sensei_user_course_status_message( $course_id = 0, $user_id = 0 ) {
-		if( intval( $user_id ) == 0 ) {
+	public static function sensei_user_course_status_message($course_id = 0, $user_id = 0) {
+		if (intval($user_id) == 0) {
 			$user_id = get_current_user_id();
 		}
 
 		$status = 'not_started';
 		$box_class = 'info';
-		$message = __( 'You have not started this course yet.', 'woothemes-sensei' );
+		$message = __('You have not started this course yet.', 'woothemes-sensei');
 
-		if( $course_id > 0 && $user_id > 0 ) {
+		if ($course_id > 0 && $user_id > 0) {
 
-			$started_course = Sensei_Utils::user_started_course( $course_id, $user_id );
+			$started_course = Sensei_Utils::user_started_course($course_id, $user_id);
 
-			if( $started_course ) {
-				$passmark = Sensei_Utils::sensei_course_pass_grade( $course_id ); // This happens inside sensei_user_passed_course()!
-				$user_grade = Sensei_Utils::sensei_course_user_grade( $course_id, $user_id ); // This happens inside sensei_user_passed_course()!
-				if( $user_grade >= $passmark ) {
+			if ($started_course) {
+				$passmark = Sensei_Utils::sensei_course_pass_grade($course_id); // This happens inside sensei_user_passed_course()!
+				$user_grade = Sensei_Utils::sensei_course_user_grade($course_id, $user_id); // This happens inside sensei_user_passed_course()!
+				if ($user_grade >= $passmark) {
 					$status = 'passed';
 					$box_class = 'tick';
-					$message = sprintf( __( 'You have passed this course with a grade of %1$d%%.', 'woothemes-sensei' ), $user_grade );
+					$message = sprintf(__('You have passed this course with a grade of %1$d%%.', 'woothemes-sensei'), $user_grade);
 				} else {
 					$status = 'failed';
 					$box_class = 'alert';
-					$message = sprintf( __( 'You require %1$d%% to pass this course. Your grade is %2$s%%.', 'woothemes-sensei' ), $passmark, $user_grade );
+					$message = sprintf(__('You require %1$d%% to pass this course. Your grade is %2$s%%.', 'woothemes-sensei'), $passmark, $user_grade);
 				}
 			}
 
 		}
 
-		$message = apply_filters( 'sensei_user_course_status_' . $status, $message );
+		$message = apply_filters('sensei_user_course_status_'.$status, $message);
 
-		return array( 'status' => $status, 'box_class' => $box_class, 'message' => $message );
+		return array('status' => $status, 'box_class' => $box_class, 'message' => $message);
 	}
 
 	/**
@@ -1177,94 +1177,94 @@ 
      * @param  bool $is_lesson
 	 * @return array              Status code and message
 	 */
-	public static function sensei_user_quiz_status_message( $lesson_id = 0, $user_id = 0, $is_lesson = false ) {
+	public static function sensei_user_quiz_status_message($lesson_id = 0, $user_id = 0, $is_lesson = false) {
 		global  $current_user;
-		if( intval( $user_id ) == 0 ) {
+		if (intval($user_id) == 0) {
 			$user_id = $current_user->ID;
 		}
 
 		$status = 'not_started';
 		$box_class = 'info';
-		$message = __( "You have not taken this lesson's quiz yet", 'woothemes-sensei' );
+		$message = __("You have not taken this lesson's quiz yet", 'woothemes-sensei');
 		$extra = '';
 
-		if( $lesson_id > 0 && $user_id > 0 ) {
+		if ($lesson_id > 0 && $user_id > 0) {
 
 			// Prerequisite lesson
-			$prerequisite = get_post_meta( $lesson_id, '_lesson_prerequisite', true );
+			$prerequisite = get_post_meta($lesson_id, '_lesson_prerequisite', true);
 
 			// Course ID
-			$course_id = absint( get_post_meta( $lesson_id, '_lesson_course', true ) );
+			$course_id = absint(get_post_meta($lesson_id, '_lesson_course', true));
 
 			// Has user started course
-			$started_course = Sensei_Utils::user_started_course( $course_id, $user_id );
+			$started_course = Sensei_Utils::user_started_course($course_id, $user_id);
 
 			// Has user completed lesson
-			$user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
-			$lesson_complete = Sensei_Utils::user_completed_lesson( $user_lesson_status );
+			$user_lesson_status = Sensei_Utils::user_lesson_status($lesson_id, $user_id);
+			$lesson_complete = Sensei_Utils::user_completed_lesson($user_lesson_status);
 
 			// Quiz ID
-			$quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
+			$quiz_id = Sensei()->lesson->lesson_quizzes($lesson_id);
 
 			// Quiz grade
 			$quiz_grade = 0;
-			if ( $user_lesson_status ) {
-				$quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
+			if ($user_lesson_status) {
+				$quiz_grade = get_comment_meta($user_lesson_status->comment_ID, 'grade', true);
 			}
 
 			// Quiz passmark
-			$quiz_passmark = absint( get_post_meta( $quiz_id, '_quiz_passmark', true ) );
+			$quiz_passmark = absint(get_post_meta($quiz_id, '_quiz_passmark', true));
 			$quiz_passmark_float = (float) $quiz_passmark;
 
 			// Pass required
-			$pass_required = get_post_meta( $quiz_id, '_pass_required', true );
+			$pass_required = get_post_meta($quiz_id, '_pass_required', true);
 
 			// Quiz questions
-			$has_quiz_questions = get_post_meta( $lesson_id, '_quiz_has_questions', true );
+			$has_quiz_questions = get_post_meta($lesson_id, '_quiz_has_questions', true);
 
-			if ( ! $started_course ) {
+			if ( ! $started_course) {
 
 				$status = 'not_started_course';
 				$box_class = 'info';
-				$message = sprintf( __( 'Please sign up for %1$sthe course%2$s before taking this quiz', 'woothemes-sensei' ), '<a href="' . esc_url( get_permalink( $course_id ) ) . '" title="' . esc_attr( __( 'Sign Up', 'woothemes-sensei' ) ) . '">', '</a>' );
+				$message = sprintf(__('Please sign up for %1$sthe course%2$s before taking this quiz', 'woothemes-sensei'), '<a href="'.esc_url(get_permalink($course_id)).'" title="'.esc_attr(__('Sign Up', 'woothemes-sensei')).'">', '</a>');
 
-			} elseif ( ! is_user_logged_in() ) {
+			} elseif ( ! is_user_logged_in()) {
 
 				$status = 'login_required';
 				$box_class = 'info';
-				$message = __( 'You must be logged in to take this quiz', 'woothemes-sensei' );
+				$message = __('You must be logged in to take this quiz', 'woothemes-sensei');
 
 			}
 			// Lesson/Quiz is marked as complete thus passing any quiz restrictions
-			elseif ( $lesson_complete ) {
+			elseif ($lesson_complete) {
 
 				$status = 'passed';
 				$box_class = 'tick';
 				// Lesson status will be "complete" (has no Quiz)
-				if ( ! $has_quiz_questions ) {
-					$message = sprintf( __( 'Congratulations! You have passed this lesson.', 'woothemes-sensei' ) );
+				if ( ! $has_quiz_questions) {
+					$message = sprintf(__('Congratulations! You have passed this lesson.', 'woothemes-sensei'));
 				}
 				// Lesson status will be "graded" (no passmark required so might have failed all the questions)
-				elseif ( empty( $quiz_grade ) ) {
-					$message = sprintf( __( 'Congratulations! You have completed this lesson.', 'woothemes-sensei' ) );
+				elseif (empty($quiz_grade)) {
+					$message = sprintf(__('Congratulations! You have completed this lesson.', 'woothemes-sensei'));
 				}
 				// Lesson status will be "passed" (passmark reached)
-				elseif ( ! empty( $quiz_grade ) && abs( $quiz_grade ) >= 0 ) {
-					if( $is_lesson ) {
-						$message = sprintf( __( 'Congratulations! You have passed this lesson\'s quiz achieving %s%%', 'woothemes-sensei' ), Sensei_Utils::round( $quiz_grade ) );
+				elseif ( ! empty($quiz_grade) && abs($quiz_grade) >= 0) {
+					if ($is_lesson) {
+						$message = sprintf(__('Congratulations! You have passed this lesson\'s quiz achieving %s%%', 'woothemes-sensei'), Sensei_Utils::round($quiz_grade));
 					} else {
-						$message = sprintf( __( 'Congratulations! You have passed this quiz achieving %s%%', 'woothemes-sensei' ),  Sensei_Utils::round( $quiz_grade ) );
+						$message = sprintf(__('Congratulations! You have passed this quiz achieving %s%%', 'woothemes-sensei'), Sensei_Utils::round($quiz_grade));
 					}
 				}
 
                 // add next lesson button
-                $nav_id_array = sensei_get_prev_next_lessons( $lesson_id );
-                $next_lesson_id = absint( $nav_id_array['next_lesson'] );
+                $nav_id_array = sensei_get_prev_next_lessons($lesson_id);
+                $next_lesson_id = absint($nav_id_array['next_lesson']);
 
                 // Output HTML
-                if ( ( 0 < $next_lesson_id ) ) {
-                    $message .= ' ' . '<a class="next-lesson" href="' . esc_url( get_permalink( $next_lesson_id ) )
-                                . '" rel="next"><span class="meta-nav"></span>'. __( 'Next Lesson' ,'woothemes-sensei')
+                if ((0 < $next_lesson_id)) {
+                    $message .= ' '.'<a class="next-lesson" href="'.esc_url(get_permalink($next_lesson_id))
+                                . '" rel="next"><span class="meta-nav"></span>'.__('Next Lesson', 'woothemes-sensei')
                                 .'</a>';
 
                 }
@@ -1273,33 +1273,33 @@ 
             // Lesson/Quiz not complete
 			else {
 				// Lesson/Quiz isn't "complete" instead it's ungraded (previously this "state" meant that it *was* complete)
-				if ( isset( $user_lesson_status->comment_approved ) && 'ungraded' == $user_lesson_status->comment_approved ) {
+				if (isset($user_lesson_status->comment_approved) && 'ungraded' == $user_lesson_status->comment_approved) {
 					$status = 'complete';
 					$box_class = 'info';
-					if( $is_lesson ) {
-						$message = sprintf( __( 'You have completed this lesson\'s quiz and it will be graded soon. %1$sView the lesson quiz%2$s', 'woothemes-sensei' ), '<a href="' . esc_url( get_permalink( $quiz_id ) ) . '" title="' . esc_attr( get_the_title( $quiz_id ) ) . '">', '</a>' );
+					if ($is_lesson) {
+						$message = sprintf(__('You have completed this lesson\'s quiz and it will be graded soon. %1$sView the lesson quiz%2$s', 'woothemes-sensei'), '<a href="'.esc_url(get_permalink($quiz_id)).'" title="'.esc_attr(get_the_title($quiz_id)).'">', '</a>');
 					} else {
-						$message = sprintf( __( 'You have completed this quiz and it will be graded soon. You require %1$s%% to pass.', 'woothemes-sensei' ),  Sensei_Utils::round( $quiz_passmark ) );
+						$message = sprintf(__('You have completed this quiz and it will be graded soon. You require %1$s%% to pass.', 'woothemes-sensei'), Sensei_Utils::round($quiz_passmark));
 					}
 				}
 				// Lesson status must be "failed"
-				elseif ( isset( $user_lesson_status->comment_approved ) && 'failed' == $user_lesson_status->comment_approved ) {
+				elseif (isset($user_lesson_status->comment_approved) && 'failed' == $user_lesson_status->comment_approved) {
 					$status = 'failed';
 					$box_class = 'alert';
-					if( $is_lesson ) {
-						$message = sprintf( __( 'You require %1$d%% to pass this lesson\'s quiz. Your grade is %2$s%%', 'woothemes-sensei' ),  Sensei_Utils::round( $quiz_passmark ),  Sensei_Utils::round( $quiz_grade ) );
+					if ($is_lesson) {
+						$message = sprintf(__('You require %1$d%% to pass this lesson\'s quiz. Your grade is %2$s%%', 'woothemes-sensei'), Sensei_Utils::round($quiz_passmark), Sensei_Utils::round($quiz_grade));
 					} else {
-						$message = sprintf( __( 'You require %1$d%% to pass this quiz. Your grade is %2$s%%', 'woothemes-sensei' ),  Sensei_Utils::round( $quiz_passmark ),  Sensei_Utils::round( $quiz_grade ) );
+						$message = sprintf(__('You require %1$d%% to pass this quiz. Your grade is %2$s%%', 'woothemes-sensei'), Sensei_Utils::round($quiz_passmark), Sensei_Utils::round($quiz_grade));
 					}
 				}
 				// Lesson/Quiz requires a pass
-				elseif( $pass_required ) {
+				elseif ($pass_required) {
 					$status = 'not_started';
 					$box_class = 'info';
-					if( $is_lesson ) {
-						$message = sprintf( __( 'You require %1$d%% to pass this lesson\'s quiz.', 'woothemes-sensei' ),  Sensei_Utils::round( $quiz_passmark ) );
+					if ($is_lesson) {
+						$message = sprintf(__('You require %1$d%% to pass this lesson\'s quiz.', 'woothemes-sensei'), Sensei_Utils::round($quiz_passmark));
 					} else {
-						$message = sprintf( __( 'You require %1$d%% to pass this quiz.', 'woothemes-sensei' ),  Sensei_Utils::round( $quiz_passmark ) );
+						$message = sprintf(__('You require %1$d%% to pass this quiz.', 'woothemes-sensei'), Sensei_Utils::round($quiz_passmark));
 					}
 				}
 			}
@@ -1307,15 +1307,15 @@ 
 		}
 
 		// Legacy filter
-		$message = apply_filters( 'sensei_user_quiz_status_' . $status, $message );
+		$message = apply_filters('sensei_user_quiz_status_'.$status, $message);
 
-		if( $is_lesson && ! in_array( $status, array( 'login_required', 'not_started_course' ) ) ) {
-            $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
-			$extra = '<p><a class="button" href="' . esc_url( get_permalink( $quiz_id ) ) . '" title="' .  __( 'View the lesson quiz', 'woothemes-sensei' ) . '">' .  __( 'View the lesson quiz', 'woothemes-sensei' )  . '</a></p>';
+		if ($is_lesson && ! in_array($status, array('login_required', 'not_started_course'))) {
+            $quiz_id = Sensei()->lesson->lesson_quizzes($lesson_id);
+			$extra = '<p><a class="button" href="'.esc_url(get_permalink($quiz_id)).'" title="'.__('View the lesson quiz', 'woothemes-sensei').'">'.__('View the lesson quiz', 'woothemes-sensei').'</a></p>';
 		}
 
 		// Filter of all messages
-		return apply_filters( 'sensei_user_quiz_status', array( 'status' => $status, 'box_class' => $box_class, 'message' => $message, 'extra' => $extra ), $lesson_id, $user_id, $is_lesson );
+		return apply_filters('sensei_user_quiz_status', array('status' => $status, 'box_class' => $box_class, 'message' => $message, 'extra' => $extra), $lesson_id, $user_id, $is_lesson);
 	}
 
 	/**
@@ -1325,14 +1325,14 @@ 
 	 * @param  integer $course_id Course ID
 	 * @return mixed boolean or comment_ID
 	 */
-	public static function user_start_course( $user_id = 0, $course_id = 0 ) {
+	public static function user_start_course($user_id = 0, $course_id = 0) {
 
 		$activity_logged = false;
 
-		if( $user_id && $course_id ) {
+		if ($user_id && $course_id) {
 			// Check if user is already on the Course
-			$activity_logged = Sensei_Utils::user_started_course( $course_id, $user_id );
-			if ( ! $activity_logged ) {
+			$activity_logged = Sensei_Utils::user_started_course($course_id, $user_id);
+			if ( ! $activity_logged) {
 
 				// Add user to course
 				$course_metadata = array(
@@ -1341,11 +1341,11 @@ 
 					'complete' => 0,
 				);
 
-				$activity_logged = Sensei_Utils::update_course_status( $user_id, $course_id, $course_status = 'in-progress', $course_metadata );
+				$activity_logged = Sensei_Utils::update_course_status($user_id, $course_id, $course_status = 'in-progress', $course_metadata);
 
 				// Allow further actions
-				if ( $activity_logged ) {
-					do_action( 'sensei_user_course_start', $user_id, $course_id );
+				if ($activity_logged) {
+					do_action('sensei_user_course_start', $user_id, $course_id);
 				}
 			}
 		}
@@ -1361,19 +1361,19 @@ 
 	 * @param int $user_id
 	 * @return mixed false or comment_ID
 	 */
-	public static function user_started_course( $course_id = 0, $user_id = 0 ) {
+	public static function user_started_course($course_id = 0, $user_id = 0) {
 
-		if( $course_id ) {
-			if( ! $user_id ) {
+		if ($course_id) {
+			if ( ! $user_id) {
 				$user_id = get_current_user_id();
 			}
 
-            if( ! $user_id > 0 ){
+            if ( ! $user_id > 0) {
                 return false;
             }
 
-			$user_course_status_id = Sensei_Utils::sensei_get_activity_value( array( 'post_id' => $course_id, 'user_id' => $user_id, 'type' => 'sensei_course_status', 'field' => 'comment_ID' ) );
-			if( $user_course_status_id ) {
+			$user_course_status_id = Sensei_Utils::sensei_get_activity_value(array('post_id' => $course_id, 'user_id' => $user_id, 'type' => 'sensei_course_status', 'field' => 'comment_ID'));
+			if ($user_course_status_id) {
 				return $user_course_status_id;
 			}
 		}
@@ -1388,17 +1388,17 @@ 
 	 * @param  integer $user_id   User ID
 	 * @return int
 	 */
-	public static function user_complete_course( $course_id = 0, $user_id = 0 ) {
+	public static function user_complete_course($course_id = 0, $user_id = 0) {
 		global  $wp_version;
 
-		if( $course_id ) {
-			if( ! $user_id ) {
+		if ($course_id) {
+			if ( ! $user_id) {
 				$user_id = get_current_user_id();
 			}
 
 			$course_status = 'in-progress';
 			$course_metadata = array();
-			$course_completion = Sensei()->settings->settings[ 'course_completion' ];
+			$course_completion = Sensei()->settings->settings['course_completion'];
 			$lessons_completed = $total_lessons = 0;
 			$lesson_status_args = array(
 					'user_id' => $user_id,
@@ -1407,8 +1407,8 @@ 
 				);
 
 			// Grab all of this Courses' lessons, looping through each...
-			$lesson_ids = Sensei()->course->course_lessons( $course_id, 'any', 'ids' );
-			$total_lessons = count( $lesson_ids );
+			$lesson_ids = Sensei()->course->course_lessons($course_id, 'any', 'ids');
+			$total_lessons = count($lesson_ids);
 				// ...if course completion not set to 'passed', and all lessons are complete or graded,
 				// ......then all lessons are 'passed'
 				// ...else if course completion is set to 'passed', check if each lesson has questions...
@@ -1421,29 +1421,29 @@ 
 			// The below checks if a lesson is fully completed, though maybe should be Utils::user_completed_lesson()
 			$all_lesson_statuses = array();
 			// In WordPress 4.1 get_comments() allows a single query to cover multiple comment_post_IDs
-			if ( version_compare($wp_version, '4.1', '>=') ) {
+			if (version_compare($wp_version, '4.1', '>=')) {
 				$lesson_status_args['post__in'] = $lesson_ids;
-				$all_lesson_statuses = Sensei_Utils::sensei_check_for_activity( $lesson_status_args, true );
+				$all_lesson_statuses = Sensei_Utils::sensei_check_for_activity($lesson_status_args, true);
 				// Need to always return an array, even with only 1 item
-				if ( !is_array($all_lesson_statuses) ) {
-					$all_lesson_statuses = array( $all_lesson_statuses );
+				if ( ! is_array($all_lesson_statuses)) {
+					$all_lesson_statuses = array($all_lesson_statuses);
 				}
 			}
 			// ...otherwise check each one
 			else {
-				foreach( $lesson_ids as $lesson_id ) {
+				foreach ($lesson_ids as $lesson_id) {
 					$lesson_status_args['post_id'] = $lesson_id;
-					$each_lesson_status = Sensei_Utils::sensei_check_for_activity( $lesson_status_args, true );
+					$each_lesson_status = Sensei_Utils::sensei_check_for_activity($lesson_status_args, true);
 					// Check for valid return before using
-					if ( !empty($each_lesson_status->comment_approved) ) {
+					if ( ! empty($each_lesson_status->comment_approved)) {
 						$all_lesson_statuses[] = $each_lesson_status;
 					}
 				}
 			}
-			foreach( $all_lesson_statuses as $lesson_status ) {
+			foreach ($all_lesson_statuses as $lesson_status) {
 				// If lessons are complete without needing quizzes to be passed
-				if ( 'passed' != $course_completion ) {
-					switch ( $lesson_status->comment_approved ) {
+				if ('passed' != $course_completion) {
+					switch ($lesson_status->comment_approved) {
 						// A user cannot 'complete' a course if a lesson...
 						case 'in-progress': // ...is still in progress
 						case 'ungraded': // ...hasn't yet been graded
@@ -1455,7 +1455,7 @@ 
 					}
 				}
 				else {
-					switch ( $lesson_status->comment_approved ) {
+					switch ($lesson_status->comment_approved) {
 						case 'complete': // Lesson has no quiz/questions
 						case 'graded': // Lesson has quiz, but it's not important what the grade was
 						case 'passed': // Lesson has quiz and the user passed
@@ -1469,20 +1469,20 @@ 
 					}
 				}
 			} // Each lesson
-			if ( $lessons_completed == $total_lessons ) {
+			if ($lessons_completed == $total_lessons) {
 				$course_status = 'complete';
 			}
 
 			// Update meta data on how many lessons have been completed
 			$course_metadata['complete'] = $lessons_completed;
 			// update the overall percentage of the course lessons complete (or graded) compared to 'in-progress' regardless of the above
-			$course_metadata['percent'] = abs( round( ( doubleval( $lessons_completed ) * 100 ) / ( $total_lessons ), 0 ) );
+			$course_metadata['percent'] = abs(round((doubleval($lessons_completed) * 100) / ($total_lessons), 0));
 
-			$activity_logged = Sensei_Utils::update_course_status( $user_id, $course_id, $course_status, $course_metadata );
+			$activity_logged = Sensei_Utils::update_course_status($user_id, $course_id, $course_status, $course_metadata);
 
 			// Allow further actions
-			if ( 'complete' == $course_status ) {
-				do_action( 'sensei_user_course_end', $user_id, $course_id );
+			if ('complete' == $course_status) {
+				do_action('sensei_user_course_end', $user_id, $course_id);
 			}
 			return $activity_logged;
 		}
@@ -1498,31 +1498,31 @@ 
 	 * @param int $user_id
 	 * @return boolean
 	 */
-	public static function user_completed_course( $course , $user_id = 0 ) {
+	public static function user_completed_course($course, $user_id = 0) {
 
-		if( $course ) {
-			if ( is_object( $course ) && is_a( $course,'WP_Comment') ) {
+		if ($course) {
+			if (is_object($course) && is_a($course, 'WP_Comment')) {
 				$user_course_status = $course->comment_approved;
 			}
-			elseif ( !is_numeric( $course ) && ! is_a( $course,'WP_Post') ) {
+			elseif ( ! is_numeric($course) && ! is_a($course, 'WP_Post')) {
 				$user_course_status = $course;
 			}
 			else {
-				if( ! $user_id ) {
+				if ( ! $user_id) {
 					$user_id = get_current_user_id();
 				}
 
-                if( is_a( $course, 'WP_Post' ) ){
-                    $course =   $course->ID;
+                if (is_a($course, 'WP_Post')) {
+                    $course = $course->ID;
                 }
 
-				$user_course_status = Sensei_Utils::user_course_status( $course , $user_id );
-				if( isset( $user_course_status->comment_approved ) ){
+				$user_course_status = Sensei_Utils::user_course_status($course, $user_id);
+				if (isset($user_course_status->comment_approved)) {
                     $user_course_status = $user_course_status->comment_approved;
                 }
 
 			}
-			if( $user_course_status && 'complete' == $user_course_status ) {
+			if ($user_course_status && 'complete' == $user_course_status) {
 				return true;
 			}
 		}
@@ -1537,10 +1537,10 @@ 
 	 * @param int $user_id
 	 * @return mixed false or comment_ID
 	 */
-	public static function user_started_lesson( $lesson_id = 0, $user_id = 0 ) {
+	public static function user_started_lesson($lesson_id = 0, $user_id = 0) {
 
-		if( $lesson_id ) {
-			if( ! $user_id ) {
+		if ($lesson_id) {
+			if ( ! $user_id) {
 				$user_id = get_current_user_id();
 			}
 
@@ -1550,8 +1550,8 @@ 
                 'type' => 'sensei_lesson_status',
                 'field' => 'comment_ID' );
 
-			$user_lesson_status_id = Sensei_Utils::sensei_get_activity_value( $activity_args );
-			if( $user_lesson_status_id ) {
+			$user_lesson_status_id = Sensei_Utils::sensei_get_activity_value($activity_args);
+			if ($user_lesson_status_id) {
 				return $user_lesson_status_id;
 			}
 		}
@@ -1566,29 +1566,29 @@ 
 	 * @param int $user_id
 	 * @return boolean
 	 */
-	public static function user_completed_lesson( $lesson = 0, $user_id = 0 ) {
+	public static function user_completed_lesson($lesson = 0, $user_id = 0) {
 
-		if( $lesson ) {
+		if ($lesson) {
 			$lesson_id = 0;
-			if ( is_object( $lesson ) ) {
+			if (is_object($lesson)) {
 				$user_lesson_status = $lesson->comment_approved;
 				$lesson_id = $lesson->comment_post_ID;
 			}
-			elseif ( ! is_numeric( $lesson ) ) {
+			elseif ( ! is_numeric($lesson)) {
 				$user_lesson_status = $lesson;
 			}
 			else {
-				if( ! $user_id ) {
+				if ( ! $user_id) {
 					$user_id = get_current_user_id();
 				}
 
                 // the user is not logged in
-                if( ! $user_id > 0 ){
+                if ( ! $user_id > 0) {
                     return false;
                 }
-				$_user_lesson_status = Sensei_Utils::user_lesson_status( $lesson, $user_id );
+				$_user_lesson_status = Sensei_Utils::user_lesson_status($lesson, $user_id);
 
-				if ( $_user_lesson_status ) {
+				if ($_user_lesson_status) {
 					$user_lesson_status = $_user_lesson_status->comment_approved;
 				}
 				else {
@@ -1596,12 +1596,12 @@ 
 				}
 				$lesson_id = $lesson;
 			}
-			if ( 'in-progress' != $user_lesson_status ) {
+			if ('in-progress' != $user_lesson_status) {
 				// Check for Passed or Completed Setting
 				// Should we be checking for the Course completion setting? Surely that should only affect the Course completion, not bypass each Lesson setting
 //				$course_completion = Sensei()->settings->settings[ 'course_completion' ];
 //				if ( 'passed' == $course_completion ) {
-					switch( $user_lesson_status ) {
+					switch ($user_lesson_status) {
 						case 'complete':
 						case 'graded':
 						case 'passed':
@@ -1610,13 +1610,13 @@ 
 
 						case 'failed':
 							// This may be 'completed' depending on...
-							if ( $lesson_id ) {
+							if ($lesson_id) {
 								// Get Quiz ID, this won't be needed once all Quiz meta fields are stored on the Lesson
-								$lesson_quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
-								if ( $lesson_quiz_id ) {
+								$lesson_quiz_id = Sensei()->lesson->lesson_quizzes($lesson_id);
+								if ($lesson_quiz_id) {
 									// ...the quiz pass setting
-									$pass_required = get_post_meta( $lesson_quiz_id, '_pass_required', true );
-									if ( empty($pass_required) ) {
+									$pass_required = get_post_meta($lesson_quiz_id, '_pass_required', true);
+									if (empty($pass_required)) {
 										// We just require the user to have done the quiz, not to have passed
 										return true;
 									}
@@ -1639,15 +1639,15 @@ 
 	 * @param int $user_id
 	 * @return object
 	 */
-	public static function user_course_status( $course_id = 0, $user_id = 0 ) {
+	public static function user_course_status($course_id = 0, $user_id = 0) {
 
 
-		if( $course_id ) {
-			if( ! $user_id ) {
+		if ($course_id) {
+			if ( ! $user_id) {
 				$user_id = get_current_user_id();
 			}
 
-			$user_course_status = Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $course_id, 'user_id' => $user_id, 'type' => 'sensei_course_status' ), true );
+			$user_course_status = Sensei_Utils::sensei_check_for_activity(array('post_id' => $course_id, 'user_id' => $user_id, 'type' => 'sensei_course_status'), true);
 			return $user_course_status;
 		}
 
@@ -1662,27 +1662,27 @@ 
 	 * @param int $user_id
 	 * @return object | bool
 	 */
-	public static function user_lesson_status( $lesson_id = 0, $user_id = 0 ) {
+	public static function user_lesson_status($lesson_id = 0, $user_id = 0) {
 
-        if( ! $user_id ) {
+        if ( ! $user_id) {
             $user_id = get_current_user_id();
         }
 
-		if( $lesson_id > 0 && $user_id > 0 ) {
+		if ($lesson_id > 0 && $user_id > 0) {
 
-			$user_lesson_status = Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $lesson_id, 'user_id' => $user_id, 'type' => 'sensei_lesson_status' ), true );
+			$user_lesson_status = Sensei_Utils::sensei_check_for_activity(array('post_id' => $lesson_id, 'user_id' => $user_id, 'type' => 'sensei_lesson_status'), true);
 			return $user_lesson_status;
 		}
 
 		return false;
 	}
 
-	public static function is_preview_lesson( $lesson_id ) {
+	public static function is_preview_lesson($lesson_id) {
 		$is_preview = false;
 
-		if( 'lesson' == get_post_type( $lesson_id ) ) {
-			$lesson_preview = get_post_meta( $lesson_id, '_lesson_preview', true );
-			if ( isset( $lesson_preview ) && '' != $lesson_preview ) {
+		if ('lesson' == get_post_type($lesson_id)) {
+			$lesson_preview = get_post_meta($lesson_id, '_lesson_preview', true);
+			if (isset($lesson_preview) && '' != $lesson_preview) {
 				$is_preview = true;
 			}
 		}
@@ -1690,22 +1690,22 @@ 
 		return $is_preview;
 	}
 
-	public static function user_passed_quiz( $quiz_id = 0, $user_id = 0 ) {
+	public static function user_passed_quiz($quiz_id = 0, $user_id = 0) {
 
-		if( ! $quiz_id  ) return false;
+		if ( ! $quiz_id) return false;
 
-		if( ! $user_id ) {
+		if ( ! $user_id) {
 			$user_id = get_current_user_id();
 		}
-		$lesson_id = get_post_meta( $quiz_id, '_quiz_lesson', true );
+		$lesson_id = get_post_meta($quiz_id, '_quiz_lesson', true);
 
 		// Quiz Grade
-		$lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
-		$quiz_grade = get_comment_meta( $lesson_status->comment_ID, 'grade', true );
+		$lesson_status = Sensei_Utils::user_lesson_status($lesson_id, $user_id);
+		$quiz_grade = get_comment_meta($lesson_status->comment_ID, 'grade', true);
 
 		// Check if Grade is greater than or equal to pass percentage
-		$quiz_passmark = abs( round( doubleval( get_post_meta( $quiz_id, '_quiz_passmark', true ) ), 2 ) );
-		if ( $quiz_passmark <= intval( $quiz_grade ) ) {
+		$quiz_passmark = abs(round(doubleval(get_post_meta($quiz_id, '_quiz_passmark', true)), 2));
+		if ($quiz_passmark <= intval($quiz_grade)) {
 			return true;
 		}
 
@@ -1725,9 +1725,9 @@ 
      *
 	 * @return mixed false or comment_ID
 	 */
-	public static function update_lesson_status( $user_id, $lesson_id, $status = 'in-progress', $metadata = array() ) {
+	public static function update_lesson_status($user_id, $lesson_id, $status = 'in-progress', $metadata = array()) {
 		$comment_id = false;
-		if ( !empty($status) ) {
+		if ( ! empty($status)) {
 			$args = array(
 					'user_id'   => $user_id,
 					'post_id'   => $lesson_id,
@@ -1736,18 +1736,18 @@ 
 					'action'    => 'update', // Update the existing status...
 					'keep_time' => true, // ...but don't change the existing timestamp
 				);
-			if( 'in-progress' == $status ) {
-				unset( $args['keep_time'] ); // Keep updating what's happened
+			if ('in-progress' == $status) {
+				unset($args['keep_time']); // Keep updating what's happened
 			}
 
-			$comment_id = Sensei_Utils::sensei_log_activity( $args );
-			if ( $comment_id && !empty($metadata) ) {
-				foreach( $metadata as $key => $value ) {
-					update_comment_meta( $comment_id, $key, $value );
+			$comment_id = Sensei_Utils::sensei_log_activity($args);
+			if ($comment_id && ! empty($metadata)) {
+				foreach ($metadata as $key => $value) {
+					update_comment_meta($comment_id, $key, $value);
 				}
 			}
 
-			do_action( 'sensei_lesson_status_updated', $status, $user_id, $lesson_id, $comment_id );
+			do_action('sensei_lesson_status_updated', $status, $user_id, $lesson_id, $comment_id);
 		}
 		return $comment_id;
 	}
@@ -1763,9 +1763,9 @@ 
 	 * @param array $metadata
 	 * @return mixed false or comment_ID
 	 */
-	public static function update_course_status( $user_id, $course_id, $status = 'in-progress', $metadata = array() ) {
+	public static function update_course_status($user_id, $course_id, $status = 'in-progress', $metadata = array()) {
 		$comment_id = false;
-		if ( !empty($status) ) {
+		if ( ! empty($status)) {
 			$args = array(
 					'user_id'   => $user_id,
 					'post_id'   => $course_id,
@@ -1774,17 +1774,17 @@ 
 					'action'    => 'update', // Update the existing status...
 					'keep_time' => true, // ...but don't change the existing timestamp
 				);
-			if( 'in-progress' == $status ) {
-				unset( $args['keep_time'] ); // Keep updating what's happened
+			if ('in-progress' == $status) {
+				unset($args['keep_time']); // Keep updating what's happened
 			}
 
-			$comment_id = Sensei_Utils::sensei_log_activity( $args );
-			if ( $comment_id && !empty($metadata) ) {
-				foreach( $metadata as $key => $value ) {
-					update_comment_meta( $comment_id, $key, $value );
+			$comment_id = Sensei_Utils::sensei_log_activity($args);
+			if ($comment_id && ! empty($metadata)) {
+				foreach ($metadata as $key => $value) {
+					update_comment_meta($comment_id, $key, $value);
 				}
 			}
-			do_action( 'sensei_course_status_updated', $status, $user_id, $course_id, $comment_id );
+			do_action('sensei_course_status_updated', $status, $user_id, $course_id, $comment_id);
 		}
 		return $comment_id;
 	}
@@ -1796,9 +1796,9 @@ 
 	 * @param  array $pieces (default: array())
 	 * @return array
 	 */
-	public static function single_comment_filter( $pieces ) {
-		unset( $pieces['orderby'] );
-		unset( $pieces['order'] );
+	public static function single_comment_filter($pieces) {
+		unset($pieces['orderby']);
+		unset($pieces['order']);
 
 		return $pieces;
 	}
@@ -1810,9 +1810,9 @@ 
 	 * @param  array $pieces (default: array())
 	 * @return array
 	 */
-	public static function comment_any_status_filter( $pieces ) {
+	public static function comment_any_status_filter($pieces) {
 
-		$pieces['where'] = str_replace( array( "( comment_approved = '0' OR comment_approved = '1' ) AND", "comment_approved = 'any' AND" ), '', $pieces['where'] );
+		$pieces['where'] = str_replace(array("( comment_approved = '0' OR comment_approved = '1' ) AND", "comment_approved = 'any' AND"), '', $pieces['where']);
 
 		return $pieces;
 	}
@@ -1824,12 +1824,12 @@ 
 	 * @param  array $pieces (default: array())
 	 * @return array
 	 */
-	public static function comment_multiple_status_filter( $pieces ) {
+	public static function comment_multiple_status_filter($pieces) {
 
-		preg_match( "/^comment_approved = '([a-z\-\,]+)'/", $pieces['where'], $placeholder );
-		if ( !empty($placeholder[1]) ) {
-			$statuses = explode( ',', $placeholder[1] );
-			$pieces['where'] = str_replace( "comment_approved = '" . $placeholder[1] . "'", "comment_approved IN ('". implode( "', '", $statuses ) . "')", $pieces['where'] );
+		preg_match("/^comment_approved = '([a-z\-\,]+)'/", $pieces['where'], $placeholder);
+		if ( ! empty($placeholder[1])) {
+			$statuses = explode(',', $placeholder[1]);
+			$pieces['where'] = str_replace("comment_approved = '".$placeholder[1]."'", "comment_approved IN ('".implode("', '", $statuses)."')", $pieces['where']);
 		}
 
 		return $pieces;
@@ -1841,12 +1841,12 @@ 
      * @param array $pieces
 	 * @return array $pieces
 	 */
-	public static function comment_total_sum_meta_value_filter( $pieces ) {
+	public static function comment_total_sum_meta_value_filter($pieces) {
 		global $wpdb, $wp_version;
 
 		$pieces['fields'] = " COUNT(*) AS total, SUM($wpdb->commentmeta.meta_value) AS meta_sum ";
-		unset( $pieces['groupby'] );
-		if ( version_compare($wp_version, '4.1', '>=') ) {
+		unset($pieces['groupby']);
+		if (version_compare($wp_version, '4.1', '>=')) {
 			$args['order'] = false;
 			$args['orderby'] = false;
 		}
@@ -1861,12 +1861,12 @@ 
 	 * @param  array $pieces (default: array())
 	 * @return array
 	 */
-	public static function get_posts_count_only_filter( $pieces ) {
+	public static function get_posts_count_only_filter($pieces) {
 		global $wp_version;
 
 		$pieces['fields'] = " COUNT(*) AS total ";
-		unset( $pieces['groupby'] );
-		if ( version_compare($wp_version, '4.1', '>=') ) {
+		unset($pieces['groupby']);
+		if (version_compare($wp_version, '4.1', '>=')) {
 			$args['order'] = false;
 			$args['orderby'] = false;
 		}
@@ -1885,9 +1885,9 @@ 
      *
      * @return bool $success
      */
-    public static function add_user_data( $data_key, $post_id , $value = '' , $user_id = 0  ){
+    public static function add_user_data($data_key, $post_id, $value = '', $user_id = 0) {
 
-        return self::update_user_data( $data_key, $post_id, $value , $user_id );
+        return self::update_user_data($data_key, $post_id, $value, $user_id);
 
     }// end add_user_data
 
@@ -1906,18 +1906,18 @@ 
      *
      * @return bool $success
      */
-    public static function update_user_data( $data_key, $post_id, $value = '' , $user_id = 0  ){
+    public static function update_user_data($data_key, $post_id, $value = '', $user_id = 0) {
 
-        if( ! ( $user_id > 0 ) ){
+        if ( ! ($user_id > 0)) {
             $user_id = get_current_user_id();
         }
 
-        $supported_post_types = array( 'course', 'lesson' );
-        $post_type = get_post_type( $post_id );
-        if( empty( $post_id ) || empty( $data_key )
-            || ! is_int( $post_id ) || ! ( intval( $post_id ) > 0 ) || ! ( intval( $user_id ) > 0 )
-            || !get_userdata( $user_id )
-            || ! in_array( $post_type, $supported_post_types )  ){
+        $supported_post_types = array('course', 'lesson');
+        $post_type = get_post_type($post_id);
+        if (empty($post_id) || empty($data_key)
+            || ! is_int($post_id) || ! (intval($post_id) > 0) || ! (intval($user_id) > 0)
+            || ! get_userdata($user_id)
+            || ! in_array($post_type, $supported_post_types)) {
 
             return false;
         }
@@ -1925,20 +1925,20 @@ 
         // check if there and existing Sensei status on this post type if not create it
         // and get the  activity ID
         $status_function = 'user_'.$post_type.'_status';
-        $sensei_user_status = self::$status_function( $post_id ,$user_id  );
-        if( ! isset( $sensei_user_status->comment_ID ) ){
+        $sensei_user_status = self::$status_function($post_id, $user_id);
+        if ( ! isset($sensei_user_status->comment_ID)) {
 
             $start_function = 'user_start_'.$post_type;
-            $sensei_user_activity_id = self::$start_function( $user_id, $post_id );
+            $sensei_user_activity_id = self::$start_function($user_id, $post_id);
 
-        }else{
+        } else {
 
             $sensei_user_activity_id = $sensei_user_status->comment_ID;
 
         }
 
         // store the data
-        $success = update_comment_meta( $sensei_user_activity_id, $data_key, $value );
+        $success = update_comment_meta($sensei_user_activity_id, $data_key, $value);
 
        return $success;
 
@@ -1957,20 +1957,20 @@ 
      *
      * @return mixed $user_data_value
      */
-    public static function get_user_data( $data_key, $post_id, $user_id = 0  ){
+    public static function get_user_data($data_key, $post_id, $user_id = 0) {
 
         $user_data_value = true;
 
-        if( ! ( $user_id > 0 ) ){
+        if ( ! ($user_id > 0)) {
             $user_id = get_current_user_id();
         }
 
-        $supported_post_types = array( 'course', 'lesson' );
-        $post_type = get_post_type( $post_id );
-        if( empty( $post_id ) || empty( $data_key )
-            || ! ( intval( $post_id ) > 0 ) || ! ( intval( $user_id ) > 0 )
-            || ! get_userdata( $user_id )
-            || !in_array( $post_type, $supported_post_types )  ){
+        $supported_post_types = array('course', 'lesson');
+        $post_type = get_post_type($post_id);
+        if (empty($post_id) || empty($data_key)
+            || ! (intval($post_id) > 0) || ! (intval($user_id) > 0)
+            || ! get_userdata($user_id)
+            || ! in_array($post_type, $supported_post_types)) {
 
             return false;
         }
@@ -1978,13 +1978,13 @@ 
         // check if there and existing Sensei status on this post type if not create it
         // and get the  activity ID
         $status_function = 'user_'.$post_type.'_status';
-        $sensei_user_status = self::$status_function( $post_id ,$user_id  );
-        if( ! isset( $sensei_user_status->comment_ID ) ){
+        $sensei_user_status = self::$status_function($post_id, $user_id);
+        if ( ! isset($sensei_user_status->comment_ID)) {
             return false;
         }
 
         $sensei_user_activity_id = $sensei_user_status->comment_ID;
-        $user_data_value = get_comment_meta( $sensei_user_activity_id , $data_key, true );
+        $user_data_value = get_comment_meta($sensei_user_activity_id, $data_key, true);
 
         return $user_data_value;
 
@@ -1999,19 +1999,19 @@ 
      *
      * @return bool $deleted
      */
-    public static function delete_user_data( $data_key, $post_id , $user_id ){
+    public static function delete_user_data($data_key, $post_id, $user_id) {
         $deleted = true;
 
-        if( ! ( $user_id > 0 ) ){
+        if ( ! ($user_id > 0)) {
             $user_id = get_current_user_id();
         }
 
-        $supported_post_types = array( 'course', 'lesson' );
-        $post_type = get_post_type( $post_id );
-        if( empty( $post_id ) || empty( $data_key )
-            || ! is_int( $post_id ) || ! ( intval( $post_id ) > 0 ) || ! ( intval( $user_id ) > 0 )
-            || ! get_userdata( $user_id )
-            || !in_array( $post_type, $supported_post_types )  ){
+        $supported_post_types = array('course', 'lesson');
+        $post_type = get_post_type($post_id);
+        if (empty($post_id) || empty($data_key)
+            || ! is_int($post_id) || ! (intval($post_id) > 0) || ! (intval($user_id) > 0)
+            || ! get_userdata($user_id)
+            || ! in_array($post_type, $supported_post_types)) {
 
             return false;
         }
@@ -2019,13 +2019,13 @@ 
         // check if there and existing Sensei status on this post type if not create it
         // and get the  activity ID
         $status_function = 'user_'.$post_type.'_status';
-        $sensei_user_status = self::$status_function( $post_id ,$user_id  );
-        if( ! isset( $sensei_user_status->comment_ID ) ){
+        $sensei_user_status = self::$status_function($post_id, $user_id);
+        if ( ! isset($sensei_user_status->comment_ID)) {
             return false;
         }
 
         $sensei_user_activity_id = $sensei_user_status->comment_ID;
-        $deleted = delete_comment_meta( $sensei_user_activity_id , $data_key );
+        $deleted = delete_comment_meta($sensei_user_activity_id, $data_key);
 
         return $deleted;
 
@@ -2050,62 +2050,62 @@ 
      *
      * @return string $drop_down_element
      */
-    public static function generate_drop_down( $selected_value, $options = array() , $attributes = array(), $enable_none_option = true ) {
+    public static function generate_drop_down($selected_value, $options = array(), $attributes = array(), $enable_none_option = true) {
 
         $drop_down_element = '';
 
         // setup the basic attributes
-        if( !isset( $attributes['name'] ) || empty( $attributes['name']  ) ) {
+        if ( ! isset($attributes['name']) || empty($attributes['name'])) {
 
             $attributes['name'] = 'sensei-options';
 
         }
 
-        if( !isset( $attributes['id'] ) || empty( $attributes['id']  ) ) {
+        if ( ! isset($attributes['id']) || empty($attributes['id'])) {
 
             $attributes['id'] = 'sensei-options';
 
         }
 
-        if( !isset( $attributes['class'] ) || empty( $attributes['class']  ) ) {
+        if ( ! isset($attributes['class']) || empty($attributes['class'])) {
 
-            $attributes['class'] ='chosen_select widefat';
+            $attributes['class'] = 'chosen_select widefat';
 
         }
 
         // create element attributes
         $combined_attributes = '';
-        foreach( $attributes as $attribute => $value ){
+        foreach ($attributes as $attribute => $value) {
 
-            $combined_attributes .= $attribute . '="'.$value.'"' . ' ';
+            $combined_attributes .= $attribute.'="'.$value.'"'.' ';
 
         }// end for each
 
 
         // create the select element
-        $drop_down_element .= '<select '. $combined_attributes . ' >' . "\n";
+        $drop_down_element .= '<select '.$combined_attributes.' >'."\n";
 
         // show the none option if the client requested
-        if( $enable_none_option ) {
-            $drop_down_element .= '<option value="">' . __('None', 'woothemes-sensei') . '</option>';
+        if ($enable_none_option) {
+            $drop_down_element .= '<option value="">'.__('None', 'woothemes-sensei').'</option>';
         }
 
-        if ( count( $options ) > 0 ) {
+        if (count($options) > 0) {
 
-            foreach ($options as $value => $option ){
+            foreach ($options as $value => $option) {
 
                 $element = '';
-                $element.= '<option value="' . esc_attr( $value ) . '"';
-                $element .= selected( $value, $selected_value, false ) . '>';
-                $element .= esc_html(  $option ) . '</option>' . "\n";
+                $element .= '<option value="'.esc_attr($value).'"';
+                $element .= selected($value, $selected_value, false).'>';
+                $element .= esc_html($option).'</option>'."\n";
 
                 // add the element to the select html
-                $drop_down_element.= $element;
+                $drop_down_element .= $element;
             } // End For Loop
 
         } // End If Statement
 
-        $drop_down_element .= '</select>' . "\n";
+        $drop_down_element .= '</select>'."\n";
 
         return $drop_down_element;
 
@@ -2125,14 +2125,14 @@ 
      *
      * @return double $val
      */
-    public static function round( $val, $precision = 0, $mode = PHP_ROUND_HALF_UP, $context = ''  ){
+    public static function round($val, $precision = 0, $mode = PHP_ROUND_HALF_UP, $context = '') {
 
         /**å
          * Change the precision for the Sensei_Utils::round function.
          * the precision given will be passed into the php round function
          * @since 1.8.5
          */
-        $precision = apply_filters( 'sensei_round_precision', $precision , $val, $context, $mode );
+        $precision = apply_filters('sensei_round_precision', $precision, $val, $context, $mode);
 
         /**
          * Change the mode for the Sensei_Utils::round function.
@@ -2142,15 +2142,15 @@ 
          *
          * @since 1.8.5
          */
-        $mode = apply_filters( 'sensei_round_mode', $mode , $val, $context, $precision   );
+        $mode = apply_filters('sensei_round_mode', $mode, $val, $context, $precision);
 
-        if ( version_compare(PHP_VERSION, '5.3.0') >= 0 ) {
+        if (version_compare(PHP_VERSION, '5.3.0') >= 0) {
 
-            return round( $val, $precision, $mode );
+            return round($val, $precision, $mode);
 
-        }else{
+        } else {
 
-            return round( $val, $precision );
+            return round($val, $precision);
 
         }
 
@@ -2162,15 +2162,15 @@ 
      * @since 1.9.0
      * @return string $url
      */
-    public static function get_current_url(){
+    public static function get_current_url() {
 
         global $wp;
-        $current_url = trailingslashit( home_url( $wp->request ) );
-        if ( isset( $_GET ) ) {
+        $current_url = trailingslashit(home_url($wp->request));
+        if (isset($_GET)) {
 
-            foreach ($_GET as $param => $val ) {
+            foreach ($_GET as $param => $val) {
 
-                $current_url = add_query_arg( $param, $val , $current_url );
+                $current_url = add_query_arg($param, $val, $current_url);
 
             }
         }
@@ -2200,27 +2200,27 @@ 
      * @param array $array_b
      * @return array $merged_array
      */
-    public static function array_zip_merge( $array_a, $array_b ){
+    public static function array_zip_merge($array_a, $array_b) {
 
-        if( ! isset( $array_a[0]  ) || ! isset( $array_b[0] )  ){
+        if ( ! isset($array_a[0]) || ! isset($array_b[0])) {
             trigger_error('array_zip_merge requires both arrays to be indexed arrays ');
         }
 
         $merged_array = array();
-        $total_elements = count( $array_a )  + count( $array_b );
+        $total_elements = count($array_a) + count($array_b);
 
         // Zip arrays
-        for ( $i = 0; $i < $total_elements; $i++) {
+        for ($i = 0; $i < $total_elements; $i++) {
 
             // if has an element at current index push a on top
-            if( isset( $array_a[ $i ] ) ){
-                $merged_array[] = $array_a[ $i ]  ;
+            if (isset($array_a[$i])) {
+                $merged_array[] = $array_a[$i];
             }
 
             // next if $array_b has an element at current index push a on top of the element
             // from a if there was one, if not the element before that.
-            if( isset( $array_b[ $i ] ) ){
-                $merged_array[] = $array_b[ $i ]  ;
+            if (isset($array_b[$i])) {
+                $merged_array[] = $array_b[$i];
             }
 
         }
@@ -2235,4 +2235,4 @@ 
  * @ignore only for backward compatibility
  * @since 1.9.0
  */
-class WooThemes_Sensei_Utils extends Sensei_Utils{}
\ No newline at end of file
+class WooThemes_Sensei_Utils extends Sensei_Utils {}
\ No newline at end of file