woothemes / sensei

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

includes/class-sensei-learner.php

Indentation +31 added lines, -31 removed lines

@@ -13,46 +13,46 @@
  */
 class Sensei_Student{
 
-    /**
-     * Get the students full name
-     *
-     * This function replaces Sensei_Learner_Managment->get_learner_full_name
-     * @since 1.9.0
-     *
-     * @param $user_id
-     * @return bool|mixed|void
-     */
-    public static function get_full_name( $user_id ){
+	/**
+	 * Get the students full name
+	 *
+	 * This function replaces Sensei_Learner_Managment->get_learner_full_name
+	 * @since 1.9.0
+	 *
+	 * @param $user_id
+	 * @return bool|mixed|void
+	 */
+	public static function get_full_name( $user_id ){
 
-        $full_name = '';
+		$full_name = '';
 
-        if( empty( $user_id ) || ! ( 0 < intval( $user_id ) )
-            || !( get_userdata( $user_id ) ) ){
-            return false;
-        }
+		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 );
+		// get the user details
+		$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;
 
-        }
+		}
 
-        /**
-         * Filter the user full name from the get_learner_full_name function.
-         *
-         * @since 1.8.0
-         * @param $full_name
-         * @param $user_id
-         */
-        return apply_filters( 'sensei_learner_full_name' , $full_name , $user_id );
+		/**
+		 * Filter the user full name from the get_learner_full_name function.
+		 *
+		 * @since 1.8.0
+		 * @param $full_name
+		 * @param $user_id
+		 */
+		return apply_filters( 'sensei_learner_full_name' , $full_name , $user_id );
 
-    }// end get_full_name
+	}// end get_full_name
 
 }
\ No newline at end of file

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

Indentation +1 added lines, -1 removed lines

@@ -627,7 +627,7 @@
 		$course = get_post( $this->course_id );
 		$report = sanitize_title( $course->post_title ) . '-' . $this->view . 's-overview';
 		if ( $this->user_id ) {
-            $user_name = Sensei_Learner::get_full_name( $this->user_id );
+			$user_name = Sensei_Learner::get_full_name( $this->user_id );
 			$report = sanitize_title( $user_name  ) . '-' . $report;
 		}
 

includes/class-sensei-frontend.php

Indentation +310 added lines, -310 removed lines

@@ -69,10 +69,10 @@ 
 		// Only show course & lesson excerpts in search results
 		add_filter( 'the_content', array( $this, 'sensei_search_results_excerpt' ) );
 
-        //Use WooCommerce filter to show admin bar to Teachers.
-        add_action( 'init', array( $this, 'sensei_show_admin_bar') );
+		//Use WooCommerce filter to show admin bar to Teachers.
+		add_action( 'init', array( $this, 'sensei_show_admin_bar') );
 
-        // Remove course from active courses if an order is cancelled or refunded
+		// Remove course from active courses if an order is cancelled or refunded
 		add_action( 'woocommerce_order_status_processing_to_cancelled', array( $this, 'remove_active_course' ), 10, 1 );
 		add_action( 'woocommerce_order_status_completed_to_cancelled', array( $this, 'remove_active_course' ), 10, 1 );
 		add_action( 'woocommerce_order_status_on-hold_to_cancelled', array( $this, 'remove_active_course' ), 10, 1 );
@@ -133,13 +133,13 @@ 
 			wp_enqueue_script( Sensei()->token . '-user-dashboard' );
 
 
-            // Course Archive javascript
-            if( is_post_type_archive( 'course' ) ){
+			// Course Archive javascript
+			if( is_post_type_archive( 'course' ) ){
 
-                wp_register_script( 'sensei-course-archive-js', esc_url( Sensei()->plugin_url . 'assets/js/frontend/course-archive' . $suffix . '.js' ), array( 'jquery' ), '1', true );
-                wp_enqueue_script( 'sensei-course-archive-js' );
+				wp_register_script( 'sensei-course-archive-js', esc_url( Sensei()->plugin_url . 'assets/js/frontend/course-archive' . $suffix . '.js' ), array( 'jquery' ), '1', true );
+				wp_enqueue_script( 'sensei-course-archive-js' );
 
-            }
+			}
 
 
 			// Allow additional scripts to be loaded
@@ -180,7 +180,7 @@ 
 	/**
 	 * sensei_get_template_part function.
 	 *
-     * @deprecated sine 1.9.0
+	 * @deprecated sine 1.9.0
 	 * @access public
 	 * @param mixed $slug
 	 * @param string $name (default: '')
@@ -188,15 +188,15 @@ 
 	 */
 	function sensei_get_template_part( $slug, $name = '' ) {
 
-        _deprecated_function( 'class-woothemes-sensei-frontend.php', '1.9.0', 'Sensei_Templates::get_part' );
-        Sensei_Templates::get_part( $slug, $name );
+		_deprecated_function( 'class-woothemes-sensei-frontend.php', '1.9.0', 'Sensei_Templates::get_part' );
+		Sensei_Templates::get_part( $slug, $name );
 
 	} // End sensei_get_template_part()
 
 	/**
 	 * sensei_get_template function.
 	 *
-     * @deprecated since 1.9.0
+	 * @deprecated since 1.9.0
 	 * @access public
 	 * @param mixed $template_name
 	 * @param array $args (default: array())
@@ -206,8 +206,8 @@ 
 	 */
 	function sensei_get_template( $template_name, $args = array(), $template_path = '', $default_path = '' ) {
 
-        _deprecated_function( 'sensei_get_template', '1.9.0', 'Sensei_Templates::get_template' );
-        Sensei_Templates::get_template($template_name, $args, $template_path, $default_path  );
+		_deprecated_function( 'sensei_get_template', '1.9.0', 'Sensei_Templates::get_template' );
+		Sensei_Templates::get_template($template_name, $args, $template_path, $default_path  );
 
 	} // End sensei_get_template()
 
@@ -223,33 +223,33 @@ 
 	 */
 	function sensei_locate_template( $template_name, $template_path = '', $default_path = '' ) {
 
-        _deprecated_function( 'sensei_locate_template', '1.9.0', 'Sensei_Templates::locate_template' );
-        Sensei_Templates::locate_template( $template_name, $template_path, $default_path );
+		_deprecated_function( 'sensei_locate_template', '1.9.0', 'Sensei_Templates::locate_template' );
+		Sensei_Templates::locate_template( $template_name, $template_path, $default_path );
 
 	} // End sensei_locate_template()
 
 
 	/**
 	 * sensei_output_content_wrapper function.
-     *
+	 *
 	 * @access public
 	 * @return void
 	 */
 	function sensei_output_content_wrapper() {
 
-	    // backwards compatibility check for old location under the wrappers directory of the active theme
-        $backwards_compatible_wrapper_location =   array(
-            Sensei()->template_url . 'wrappers/wrapper-start.php',
-            'wrappers/wrapper-start.php'
-        );
+		// backwards compatibility check for old location under the wrappers directory of the active theme
+		$backwards_compatible_wrapper_location =   array(
+			Sensei()->template_url . 'wrappers/wrapper-start.php',
+			'wrappers/wrapper-start.php'
+		);
 
-        $template = locate_template( $backwards_compatible_wrapper_location );
-        if( !empty( $template ) ){
+		$template = locate_template( $backwards_compatible_wrapper_location );
+		if( !empty( $template ) ){
 
-            Sensei_Templates::get_template( 'wrappers/wrapper-start.php' );
-            return;
+			Sensei_Templates::get_template( 'wrappers/wrapper-start.php' );
+			return;
 
-        }
+		}
 
 		Sensei_Templates::get_template( 'globals/wrapper-start.php' );
 
@@ -258,25 +258,25 @@ 
 
 	/**
 	 * sensei_output_content_wrapper_end function.
-     *
+	 *
 	 * @access public
 	 * @return void
 	 */
 	function sensei_output_content_wrapper_end() {
 
-	    // backwards compatibility check for old location under the wrappers directory of the active theme
-        $backwards_compatible_wrapper_location =   array(
-            Sensei()->template_url . 'wrappers/wrapper-end.php',
-            'wrappers/wrapper-end.php'
-        );
+		// backwards compatibility check for old location under the wrappers directory of the active theme
+		$backwards_compatible_wrapper_location =   array(
+			Sensei()->template_url . 'wrappers/wrapper-end.php',
+			'wrappers/wrapper-end.php'
+		);
 
-        $backwards_compatible_template = locate_template( $backwards_compatible_wrapper_location );
-        if( !empty( $backwards_compatible_template ) ){
+		$backwards_compatible_template = locate_template( $backwards_compatible_wrapper_location );
+		if( !empty( $backwards_compatible_template ) ){
 
-            Sensei_Templates::get_template( 'wrappers/wrapper-end.php' );
-            return;
+			Sensei_Templates::get_template( 'wrappers/wrapper-end.php' );
+			return;
 
-        }
+		}
 
 
 		Sensei_Templates::get_template( 'globals/wrapper-end.php' );
@@ -292,55 +292,55 @@ 
 	 */
 	public static function load_content_pagination() {
 
-        if( is_singular('course') ) {
+		if( is_singular('course') ) {
 
-            // backwards compatibility check for old location under the wrappers directory of the active theme
-            $template = locate_template( array( Sensei()->template_url . 'wrappers/pagination-posts.php' ) );
-            if( !empty( $template ) ){
+			// backwards compatibility check for old location under the wrappers directory of the active theme
+			$template = locate_template( array( Sensei()->template_url . 'wrappers/pagination-posts.php' ) );
+			if( !empty( $template ) ){
 
-                Sensei_Templates::get_template( 'wrappers/pagination-posts.php' );
-                return;
+				Sensei_Templates::get_template( 'wrappers/pagination-posts.php' );
+				return;
 
-            }
+			}
 
 			Sensei_Templates::get_template( 'globals/pagination-posts.php' );
 
 		} elseif( is_singular('lesson') ) {
 
-		    // backwards compatibility check for old location under the wrappers directory of the active theme
-		    $template = locate_template( array( Sensei()->template_url . 'wrappers/pagination-lesson.php' ) );
-            if( !empty( $template ) ){
+			// backwards compatibility check for old location under the wrappers directory of the active theme
+			$template = locate_template( array( Sensei()->template_url . 'wrappers/pagination-lesson.php' ) );
+			if( !empty( $template ) ){
 
-                Sensei_Templates::get_template( 'wrappers/pagination-lesson.php' );
-                return;
+				Sensei_Templates::get_template( 'wrappers/pagination-lesson.php' );
+				return;
 
-            }
+			}
 
 			Sensei_Templates::get_template( 'globals/pagination-lesson.php' );
 
 		} elseif( is_singular('quiz') ) {
 
-		    // backwards compatibility check for old location under the wrappers directory of the active theme
-		    $template = locate_template( array( Sensei()->template_url . 'wrappers/pagination-quiz.php' ) );
-            if( !empty( $template ) ){
+			// backwards compatibility check for old location under the wrappers directory of the active theme
+			$template = locate_template( array( Sensei()->template_url . 'wrappers/pagination-quiz.php' ) );
+			if( !empty( $template ) ){
 
-                Sensei_Templates::get_template( 'wrappers/pagination-quiz.php' );
-                return;
+				Sensei_Templates::get_template( 'wrappers/pagination-quiz.php' );
+				return;
 
-            }
+			}
 
 			Sensei_Templates::get_template( 'globals/pagination-quiz.php' );
 
 		} else {
 
-            // backwards compatibility check for old location under the wrappers directory of the active theme
-            $template = locate_template( array( Sensei()->template_url . 'wrappers/pagination.php' ) );
-            if( !empty( $template ) ){
+			// backwards compatibility check for old location under the wrappers directory of the active theme
+			$template = locate_template( array( Sensei()->template_url . 'wrappers/pagination.php' ) );
+			if( !empty( $template ) ){
 
-                Sensei_Templates::get_template( 'wrappers/pagination.php' );
-                return;
+				Sensei_Templates::get_template( 'wrappers/pagination.php' );
+				return;
 
-            }
+			}
 
 			Sensei_Templates::get_template( 'globals/pagination.php' );
 
@@ -397,13 +397,13 @@ 
 
 				case '#senseimymessages':
 					$item->url = $my_messages_url;
-                    // if no archive link exist for sensei_message
-                    // set it back to the place holder
-                    if( ! $item->url ){
+					// if no archive link exist for sensei_message
+					// set it back to the place holder
+					if( ! $item->url ){
 
-                        $item->url = '#senseimymessages';
+						$item->url = '#senseimymessages';
 
-                    }
+					}
 					break;
 
 				case '#senseilearnerprofile':
@@ -469,7 +469,7 @@ 
 
 			// Remove the My Messages link for logged out users or if Private Messages are disabled
 			if( ! get_post_type_archive_link( 'sensei_message' )
-                && '#senseimymessages' == $item->url ) {
+				&& '#senseimymessages' == $item->url ) {
 
 				if ( !is_user_logged_in() || ( isset( Sensei()->settings->settings['messages_disable'] ) && Sensei()->settings->settings['messages_disable'] ) ) {
 
@@ -492,42 +492,42 @@ 
 
 	// add category nicenames in body and post class
 	function sensei_search_results_classes($classes) {
-	    global $post;
-	    // Handle Search Classes for Courses, Lessons, and WC Products
-	    if ( isset( $post->post_type ) && ( ( 'course' == $post->post_type ) || ( 'lesson' == $post->post_type ) || ( 'product' == $post->post_type ) ) ) {
-	    	$classes[] = 'post';
+		global $post;
+		// Handle Search Classes for Courses, Lessons, and WC Products
+		if ( isset( $post->post_type ) && ( ( 'course' == $post->post_type ) || ( 'lesson' == $post->post_type ) || ( 'product' == $post->post_type ) ) ) {
+			$classes[] = 'post';
 		} // End If Statement
-	    return $classes;
+		return $classes;
 	} // End sensei_search_results_classes()
 
-    /**
-     * sensei_single_title output for single page title
-     * @since  1.1.0
-     * @return void
-     * @deprecated
-     */
-    function the_single_title() {
+	/**
+	 * sensei_single_title output for single page title
+	 * @since  1.1.0
+	 * @return void
+	 * @deprecated
+	 */
+	function the_single_title() {
 
-        _deprecated_function(' WooThemes_Sensei_Frontend::the_single_title', '1.9.0');
+		_deprecated_function(' WooThemes_Sensei_Frontend::the_single_title', '1.9.0');
 
-    } // End sensei_single_title()
+	} // End sensei_single_title()
 
 	/**
 	 * sensei_course_image output for course image Please use Sensei()->course->course_image instead.
-     *
-     * @deprecated since 1.9.0
-     * @param $course_id
-     * @param string $width
-     * @param string $height
-     * @param bool|false $return
-     * @return string|void
+	 *
+	 * @deprecated since 1.9.0
+	 * @param $course_id
+	 * @param string $width
+	 * @param string $height
+	 * @param bool|false $return
+	 * @return string|void
 	 */
 	function sensei_course_image( $course_id, $width = '100', $height = '100', $return = false ) {
 
-    	if ( ! $return ) {
+		if ( ! $return ) {
 
 			echo Sensei()->course->course_image( $course_id, $width, $height );
-            return '';
+			return '';
 
 		} // End If Statement
 
@@ -538,31 +538,31 @@ 
 	/**
 	 * sensei_lesson_image output for lesson image
 	 * @since  1.2.0
-     * @deprecated since 1.9.0
-     * @param $lesson_id
-     * @param string $width
-     * @param string $height
-     * @param bool|false $return
-     * @param bool|false $widget
-     * @return string
+	 * @deprecated since 1.9.0
+	 * @param $lesson_id
+	 * @param string $width
+	 * @param string $height
+	 * @param bool|false $return
+	 * @param bool|false $widget
+	 * @return string
 	 */
 	function sensei_lesson_image( $lesson_id, $width = '100', $height = '100', $return = false, $widget = false ) {
 
-        if( ! $return ){
+		if( ! $return ){
 
-            echo Sensei()->lesson->lesson_image( $lesson_id, $width, $height, $widget );
-            return '';
-        }
+			echo Sensei()->lesson->lesson_image( $lesson_id, $width, $height, $widget );
+			return '';
+		}
 
-        return Sensei()->lesson->lesson_image( $lesson_id, $width, $height, $widget );
+		return Sensei()->lesson->lesson_image( $lesson_id, $width, $height, $widget );
 
 	} // End sensei_lesson_image()
 
-    /**
-     * @since 1.0.0
-     * @param WP_Query $query
-     */
-    function sensei_course_archive_pagination( $query ) {
+	/**
+	 * @since 1.0.0
+	 * @param WP_Query $query
+	 */
+	function sensei_course_archive_pagination( $query ) {
 
 		if( ! is_admin() && $query->is_main_query() && isset( $_GET['action'] ) && in_array( $_GET['action'], array( 'newcourses', 'featuredcourses', 'freecourses', 'paidcourses' ) ) ) {
 
@@ -585,50 +585,50 @@ 
 	 *
 	 * @access public
 	 * @since  1.2.0
-     * @deprecated since 1.9.0 use WooThemes_Sensei_Course::archive_header
+	 * @deprecated since 1.9.0 use WooThemes_Sensei_Course::archive_header
 	 * @return void
 	 */
 	function sensei_course_archive_header(  ) {
 
-        trigger_error('This function sensei_course_archive_header has been depricated. Please use: WooThemes_Sensei_Course::course_archive_header ');
-        WooThemes_Sensei_Course::archive_header( '', '<header class="archive-header"><h1>', '</h1></header>' );
+		trigger_error('This function sensei_course_archive_header has been depricated. Please use: WooThemes_Sensei_Course::course_archive_header ');
+		WooThemes_Sensei_Course::archive_header( '', '<header class="archive-header"><h1>', '</h1></header>' );
 
 	} // sensei_course_archive_header()
 
 	/**
 	 * sensei_lesson_archive_header function.
 	 *
-     * @deprecated since 1.9.0
+	 * @deprecated since 1.9.0
 	 * @access public
 	 * @since  1.2.1
 	 * @return void
 	 */
 	public function sensei_lesson_archive_header( ) {
-        _deprecated_function( 'WooThemes_Sensei_Frontend::sensei_lesson_archive_header', '1.9.0', 'WooThemes_Sensei_Lesson::the_archive_header' );
-        Sensei()->lesson->the_archive_header();
+		_deprecated_function( 'WooThemes_Sensei_Frontend::sensei_lesson_archive_header', '1.9.0', 'WooThemes_Sensei_Lesson::the_archive_header' );
+		Sensei()->lesson->the_archive_header();
 	} // sensei_course_archive_header()
 
-    /**
-     * @deprecated since 1.9.0
-     */
+	/**
+	 * @deprecated since 1.9.0
+	 */
 	public function sensei_message_archive_header( ){
-        _deprecated_function('Sensei_Frontend::sensei_message_archive_header','Please use: Sense');
-        Sensei_Messages::the_archive_header();
+		_deprecated_function('Sensei_Frontend::sensei_message_archive_header','Please use: Sense');
+		Sensei_Messages::the_archive_header();
 	} // sensei_message_archive_header()
 
 	/**
 	 * sensei_course_archive_course_title output for course archive page individual course title
 	 * @since  1.2.0
-     * @param WP_Post $post_item
+	 * @param WP_Post $post_item
 	 * @return void
 	 */
 	function sensei_course_archive_course_title( $post_item ) {
 		if ( isset( $post_item->ID ) && ( 0 < $post_item->ID ) ) {
 			$post_id = absint( $post_item->ID );
-    		$post_title = $post_item->post_title;
+			$post_title = $post_item->post_title;
 		} else {
 			$post_id = get_the_ID();
-    		$post_title = get_the_title();
+			$post_title = get_the_title();
 		} // End If Statement
 		?><header><h2><a href="<?php echo get_permalink( $post_id ); ?>" title="<?php echo esc_attr( $post_title ); ?>"><?php echo $post_title; ?></a></h2></header><?php
 	} // End sensei_course_archive_course_title()
@@ -640,7 +640,7 @@ 
 	 */
 	public function sensei_lesson_archive_lesson_title() {
 		$post_id = get_the_ID();
-    	$post_title = get_the_title();
+		$post_title = get_the_title();
 		?><header><h2><a href="<?php echo get_permalink( $post_id ); ?>" title="<?php echo esc_attr( $post_title ); ?>"><?php echo $post_title; ?></a></h2></header><?php
 	} // End sensei_lesson_archive_lesson_title()
 
@@ -657,9 +657,9 @@ 
 
 		if( empty( $id )  ){
 
-            $id = get_the_ID();
+			$id = get_the_ID();
 
-        }
+		}
 
 		$sensei_breadcrumb_prefix = __( 'Back to: ', 'woothemes-sensei' );
 		$separator = apply_filters( 'sensei_breadcrumb_separator', '&gt;' );
@@ -672,31 +672,31 @@ 
 				return;
 			}
 			$html .= '<a href="' . esc_url( get_permalink( $course_id ) ) . '" title="' . __( 'Back to the course', 'woothemes-sensei' ) . '">' . get_the_title( $course_id ) . '</a>';
-    	} // End If Statement
-    	// Quiz
+		} // End If Statement
+		// Quiz
 		if ( is_singular( 'quiz' ) && 0 < intval( $id ) ) {
 			$lesson_id = intval( get_post_meta( $id, '_quiz_lesson', true ) );
 			if( ! $lesson_id ) {
 				return;
 			}
 			 $html .= '<a href="' . esc_url( get_permalink( $lesson_id ) ) . '" title="' .  __( 'Back to the lesson', 'woothemes-sensei' ) . '">' . get_the_title( $lesson_id ) . '</a>';
-    	} // End If Statement
+		} // End If Statement
 
-    	// Allow other plugins to filter html
-    	$html = apply_filters ( 'sensei_breadcrumb_output', $html, $separator );
-    	$html .= '</section>';
+		// Allow other plugins to filter html
+		$html = apply_filters ( 'sensei_breadcrumb_output', $html, $separator );
+		$html .= '</section>';
 
-    	echo $html;
+		echo $html;
 	} // End sensei_breadcrumb()
 
 
-    /**
-     * @deprecated since 1.9.0 use WooThemes_Sensei_Lesson::course_signup_link instead
-     */
+	/**
+	 * @deprecated since 1.9.0 use WooThemes_Sensei_Lesson::course_signup_link instead
+	 */
 	public function sensei_lesson_course_signup_link( ) {
 
-        _deprecated_function('sensei_lesson_course_signup_link', '1.9.0', 'WooThemes_Sensei_Lesson::course_signup_link' );
-        WooThemes_Sensei_Lesson::course_signup_link();
+		_deprecated_function('sensei_lesson_course_signup_link', '1.9.0', 'WooThemes_Sensei_Lesson::course_signup_link' );
+		WooThemes_Sensei_Lesson::course_signup_link();
 	}
 
 	public function lesson_tags_display( $lesson_id = 0 ) {
@@ -715,27 +715,27 @@ 
 					?><section class="lesson-tags">
 		    			<?php printf( __( 'Lesson tags: %1$s', 'woothemes-sensei' ), $tag_list ); ?>
 		    		</section><?php
-		    	}
-	    	}
+				}
+			}
 		}
 	}
 
-    /**
-     * @param WP_Query $query
-     */
+	/**
+	 * @param WP_Query $query
+	 */
 	public function lesson_tag_archive_filter( $query ) {
-    	if( is_tax( 'lesson-tag' ) && $query->is_main_query() ) {
-    		// Limit to lessons only
-    		$query->set( 'post_type', 'lesson' );
+		if( is_tax( 'lesson-tag' ) && $query->is_main_query() ) {
+			// Limit to lessons only
+			$query->set( 'post_type', 'lesson' );
 
-    		// Set order of lessons
-    		$query->set( 'orderby', 'menu_order' );
-    		$query->set( 'order', 'ASC' );
+			// Set order of lessons
+			$query->set( 'orderby', 'menu_order' );
+			$query->set( 'order', 'ASC' );
 
-    	}
-    }
+		}
+	}
 
-    public function lesson_tag_archive_header( $title ) {
+	public function lesson_tag_archive_header( $title ) {
 		if( is_tax( 'lesson-tag' ) ) {
 			$title = sprintf( __( 'Lesson tag: %1$s', 'woothemes-sensei' ), apply_filters( 'sensei_lesson_tag_archive_title', get_queried_object()->name ) );
 		}
@@ -757,13 +757,13 @@ 
 			$sanitized_submit = esc_html( $_POST['quiz_action'] );
 
 			switch ($sanitized_submit) {
-                case 'lesson-complete':
+				case 'lesson-complete':
 
 					Sensei_Utils::sensei_start_lesson( $post->ID, $current_user->ID, $complete = true );
 
 					break;
 
-                case 'lesson-reset':
+				case 'lesson-reset':
 
 					Sensei_Utils::sensei_remove_user_from_lesson( $post->ID, $current_user->ID );
 
@@ -875,13 +875,13 @@ 
 		if ( 0 < intval( $post_id ) ) {
 			$lesson_video_embed = get_post_meta( $post_id, '_lesson_video_embed', true );
 			if ( 'http' == substr( $lesson_video_embed, 0, 4) ) {
-        		// V2 - make width and height a setting for video embed
-        		$lesson_video_embed = wp_oembed_get( esc_url( $lesson_video_embed )/*, array( 'width' => 100 , 'height' => 100)*/ );
-        	} // End If Statement
-        	if ( '' != $lesson_video_embed ) {
-        	?><div class="video"><?php echo html_entity_decode($lesson_video_embed); ?></div><?php
-        	} // End If Statement
-        } // End If Statement
+				// V2 - make width and height a setting for video embed
+				$lesson_video_embed = wp_oembed_get( esc_url( $lesson_video_embed )/*, array( 'width' => 100 , 'height' => 100)*/ );
+			} // End If Statement
+			if ( '' != $lesson_video_embed ) {
+			?><div class="video"><?php echo html_entity_decode($lesson_video_embed); ?></div><?php
+			} // End If Statement
+		} // End If Statement
 	} // End sensei_lesson_video()
 
 	public function sensei_complete_lesson_button() {
@@ -894,8 +894,8 @@ 
 		$pass_required = true;
 		if( $quiz_id ) {
 			// Get quiz pass setting
-	    	$pass_required = get_post_meta( $quiz_id, '_pass_required', true );
-	    }
+			$pass_required = get_post_meta( $quiz_id, '_pass_required', true );
+		}
 		if( ! $quiz_id || ( $quiz_id && ! $pass_required ) ) {
 			?>
 			<form class="lesson_button_form" method="POST" action="<?php echo esc_url( get_permalink() ); ?>">
@@ -948,12 +948,12 @@ 
 		} // End If Statement
 	} // End sensei_reset_lesson_button()
 
-    /**
-     * @deprecated since 1.9.0
-     */
-    public function sensei_lesson_quiz_meta( ) {
+	/**
+	 * @deprecated since 1.9.0
+	 */
+	public function sensei_lesson_quiz_meta( ) {
 
-        Sensei_Lesson::footer_quiz_call_to_action();
+		Sensei_Lesson::footer_quiz_call_to_action();
 
 	} // End sensei_lesson_quiz_meta()
 
@@ -979,38 +979,38 @@ 
         	</p>
         	<p class="course-excerpt"><?php the_excerpt(); ?></p>
         	<?php if ( 0 < $free_lesson_count ) {
-                $free_lessons = sprintf( __( 'You can access %d of this course\'s lessons for free', 'woothemes-sensei' ), $free_lesson_count ); ?>
+				$free_lessons = sprintf( __( 'You can access %d of this course\'s lessons for free', 'woothemes-sensei' ), $free_lesson_count ); ?>
                 <p class="sensei-free-lessons"><a href="<?php echo get_permalink( $post_id ); ?>"><?php _e( 'Preview this course', 'woothemes-sensei' ) ?></a> - <?php echo $free_lessons; ?></p>
             <?php } ?>
 		</section><?php
 	} // End sensei_course_archive_meta()
 
-    /**
-     * @deprecated since 1.9.0
-     */
+	/**
+	 * @deprecated since 1.9.0
+	 */
 	public function sensei_single_main_content() {
-	    _deprecated_function('Woothemes_Sensei_Frontend::sensei_single_main_content', '1.9.0');
+		_deprecated_function('Woothemes_Sensei_Frontend::sensei_single_main_content', '1.9.0');
 	} // End sensei_single_main_content()
 
-    /**
-    * @deprecated since 1.9.0
-    */
+	/**
+	 * @deprecated since 1.9.0
+	 */
 	public function sensei_lesson_archive_main_content() {
-        _deprecated_function('Sensei_Frontend::sensei_lesson_archive_main_content', '1.9.0', 'Please include loop-lesson.php directly');
+		_deprecated_function('Sensei_Frontend::sensei_lesson_archive_main_content', '1.9.0', 'Please include loop-lesson.php directly');
 	} // End sensei_lesson_archive_main_content()
 
-    /**
-    * @deprecated since 1.9.0
-    */
+	/**
+	 * @deprecated since 1.9.0
+	 */
 	public function sensei_message_archive_main_content() {
 		_deprecated_function( 'Sensei_Frontend::sensei_message_archive_main_content', 'This method is no longer needed' );
 	} // End sensei_lesson_archive_main_content()
 
-    /**
-    * @deprecated since 1.9.0
-    */
+	/**
+	 * @deprecated since 1.9.0
+	 */
 	public function sensei_no_permissions_main_content() {
-        _deprecated_function( 'Sensei_Frontend::sensei_no_permissions_main_content', 'This method is no longer needed' );
+		_deprecated_function( 'Sensei_Frontend::sensei_no_permissions_main_content', 'This method is no longer needed' );
 	} // End sensei_no_permissions_main_content()
 
 	public function sensei_course_category_main_content() {
@@ -1058,7 +1058,7 @@ 
 				<div class="col-1">
 					<?php
 					// output the actual form markup
-                    Sensei_Templates::get_template( 'user/login-form.php');
+					Sensei_Templates::get_template( 'user/login-form.php');
 					?>
 				</div>
 
@@ -1139,12 +1139,12 @@ 
 
 		//if this is a paid course
 		if ( Sensei_WC::is_woocommerce_active() ) {
-    	    $wc_post_id = get_post_meta( $course_id, '_course_woocommerce_product', true );
-    	    if ( 0 < $wc_post_id ) {
-    	    	$preview_text = __( ' (Free Preview)', 'woothemes-sensei' );
-    	    } // End If Statement
-    	}
-    	return $preview_text;
+			$wc_post_id = get_post_meta( $course_id, '_course_woocommerce_product', true );
+			if ( 0 < $wc_post_id ) {
+				$preview_text = __( ' (Free Preview)', 'woothemes-sensei' );
+			} // End If Statement
+		}
+		return $preview_text;
 	}
 
 	public function sensei_lesson_preview_title( $title = '', $id = 0 ) {
@@ -1176,8 +1176,8 @@ 
 		$is_user_taking_course = Sensei_Utils::user_started_course( $post->ID, $current_user->ID );
 		// Handle user starting the course
 		if ( isset( $_POST['course_start'] )
-		    && wp_verify_nonce( $_POST[ 'woothemes_sensei_start_course_noonce' ], 'woothemes_sensei_start_course_noonce' )
-		    && !$is_user_taking_course ) {
+			&& wp_verify_nonce( $_POST[ 'woothemes_sensei_start_course_noonce' ], 'woothemes_sensei_start_course_noonce' )
+			&& !$is_user_taking_course ) {
 
 			// Start the course
 			$activity_logged = Sensei_Utils::user_start_course( $current_user->ID, $post->ID );
@@ -1196,35 +1196,35 @@ 
 		} // End If Statement
 	} // End sensei_course_start()
 
-    /**
-     * @deprecated since 1.9.0
-     */
+	/**
+	 * @deprecated since 1.9.0
+	 */
 	public function sensei_course_meta() {
-        _deprecated_function( 'Sensei_Frontend::sensei_course_meta', '1.9.0' , 'Sensei_Course::the_course_meta()' );
-        Sensei()->course->the_course_meta( get_post() );
+		_deprecated_function( 'Sensei_Frontend::sensei_course_meta', '1.9.0' , 'Sensei_Course::the_course_meta()' );
+		Sensei()->course->the_course_meta( get_post() );
 	} // End sensei_course_meta()
 
-    /**
-     * @deprecated since 1.9.0
-     */
+	/**
+	 * @deprecated since 1.9.0
+	 */
 	public function sensei_course_meta_video() {
-        _deprecated_function( 'Sensei_Frontend::sensei_course_meta_video', '1.9.0' , 'Sensei_Course::the_course_video()' );
-        Sensei_Course::the_course_video();
+		_deprecated_function( 'Sensei_Frontend::sensei_course_meta_video', '1.9.0' , 'Sensei_Course::the_course_video()' );
+		Sensei_Course::the_course_video();
 	} // End sensei_course_meta_video()
 
-    /**
-     * This function shows the WooCommerce cart notice if the user has
-     * added the current course to cart. It does not show if the user is already taking
-     * the course.
-     *
-     * @since 1.0.2
-     * @return void;
-     */
-    public function sensei_woocommerce_in_cart_message() {
+	/**
+	 * This function shows the WooCommerce cart notice if the user has
+	 * added the current course to cart. It does not show if the user is already taking
+	 * the course.
+	 *
+	 * @since 1.0.2
+	 * @return void;
+	 */
+	public function sensei_woocommerce_in_cart_message() {
 		global $post, $woocommerce;
 
 		$wc_post_id = absint( get_post_meta( $post->ID, '_course_woocommerce_product', true ) );
-        $user_course_status_id = Sensei_Utils::user_started_course($post->ID , get_current_user_id() );
+		$user_course_status_id = Sensei_Utils::user_started_course($post->ID , get_current_user_id() );
 		if ( 0 < intval( $wc_post_id ) && ! $user_course_status_id ) {
 
 			if ( Sensei_WC::is_product_in_cart( $wc_post_id ) ) {
@@ -1272,7 +1272,7 @@ 
 				$item_id = $item['product_id'];
 			} 
 
-            if ( $item_id > 0 ) {
+			if ( $item_id > 0 ) {
 
 				$user_id = get_post_meta( $order_id, '_customer_user', true );
 
@@ -1352,15 +1352,15 @@ 
 
 					$items = $order->get_items();
 					foreach( $items as $item ) {
-                                            if (isset($item['variation_id']) && $item['variation_id'] > 0) {
-                                                $item_id = $item['variation_id'];
-                                                $product_type = 'variation';
-                                            } else {
-                                                $item_id = $item['product_id'];
-                                            }
+											if (isset($item['variation_id']) && $item['variation_id'] > 0) {
+												$item_id = $item['variation_id'];
+												$product_type = 'variation';
+											} else {
+												$item_id = $item['product_id'];
+											}
 
-                                            $product_ids[] = $item_id;
-                                            }
+											$product_ids[] = $item_id;
+											}
 
 					$order_ids[] = $post_id;
 				}
@@ -1467,32 +1467,32 @@ 
 
 				$items = $order->get_items();
 				foreach( $items as $item ) {
-                    $product = wc_get_product( $item['product_id'] );
+					$product = wc_get_product( $item['product_id'] );
 
-                    // handle product bundles
-                    if( $product->is_type('bundle') ){
+					// handle product bundles
+					if( $product->is_type('bundle') ){
 
-                        $bundled_product = new WC_Product_Bundle( $product->id );
-                        $bundled_items = $bundled_product->get_bundled_items();
+						$bundled_product = new WC_Product_Bundle( $product->id );
+						$bundled_items = $bundled_product->get_bundled_items();
 
-                        foreach( $bundled_items as $bundled_item ){
+						foreach( $bundled_items as $bundled_item ){
 
-                            if( $bundled_item->product_id == $course_product_id ) {
-                                Sensei_Utils::user_start_course( $user_id, $course_id );
-                                return;
-                            }
+							if( $bundled_item->product_id == $course_product_id ) {
+								Sensei_Utils::user_start_course( $user_id, $course_id );
+								return;
+							}
 
-                        }
+						}
 
-                    } else {
+					} else {
 
-                    // handle regular products
-                        if( $item['product_id'] == $course_product_id ) {
-                            Sensei_Utils::user_start_course( $user_id, $course_id );
-                            return;
-                        }
+					// handle regular products
+						if( $item['product_id'] == $course_product_id ) {
+							Sensei_Utils::user_start_course( $user_id, $course_id );
+							return;
+						}
 
-                    }
+					}
 				}
 			}
 
@@ -1523,21 +1523,21 @@ 
 
 		//if not posted from the sensei login form let
 		// WordPress or any other party handle the failed request
-	    if( ! isset( $_REQUEST['form'] ) || 'sensei-login' != $_REQUEST['form']  ){
+		if( ! isset( $_REQUEST['form'] ) || 'sensei-login' != $_REQUEST['form']  ){
 
-	    	return ;
+			return ;
 
-	    }
+		}
 
-    	// Get the reffering page, where did the post submission come from?
-    	$referrer = add_query_arg('login', false, $_SERVER['HTTP_REFERER']);
+		// Get the reffering page, where did the post submission come from?
+		$referrer = add_query_arg('login', false, $_SERVER['HTTP_REFERER']);
 
    		 // if there's a valid referrer, and it's not the default log-in screen
-	    if(!empty($referrer) && !strstr($referrer,'wp-login') && !strstr($referrer,'wp-admin')){
-	        // let's append some information (login=failed) to the URL for the theme to use
-	        wp_redirect( esc_url_raw( add_query_arg('login', 'failed',  $referrer) ) );
-	    	exit;
-    	}
+		if(!empty($referrer) && !strstr($referrer,'wp-login') && !strstr($referrer,'wp-admin')){
+			// let's append some information (login=failed) to the URL for the theme to use
+			wp_redirect( esc_url_raw( add_query_arg('login', 'failed',  $referrer) ) );
+			exit;
+		}
 	}// End sensei_login_fail_redirect_to_front_end_login
 
 	/**
@@ -1548,46 +1548,46 @@ 
 	function sensei_handle_login_request( ) {
 
 		// Check that it is a sensei login request and if it has a valid nonce
-	    if(  isset( $_REQUEST['form'] ) && 'sensei-login' == $_REQUEST['form'] ) {
+		if(  isset( $_REQUEST['form'] ) && 'sensei-login' == $_REQUEST['form'] ) {
 
-	    	// Validate the login request nonce
-		    if( !wp_verify_nonce( $_REQUEST['_wpnonce'], 'sensei-login' ) ){
-		    	return;
-		    }
+			// Validate the login request nonce
+			if( !wp_verify_nonce( $_REQUEST['_wpnonce'], 'sensei-login' ) ){
+				return;
+			}
 
-		    //get the page where the sensei log form is located
-		    $referrer = $_REQUEST['_wp_http_referer'];
-		    //$redirect = $_REQUEST['_sensei_redirect'];
+			//get the page where the sensei log form is located
+			$referrer = $_REQUEST['_wp_http_referer'];
+			//$redirect = $_REQUEST['_sensei_redirect'];
 
-		    if ( ( isset( $_REQUEST['log'] ) && !empty( $_REQUEST['log'] ) )
-		    	 && ( isset( $_REQUEST['pwd'] ) && !empty( $_REQUEST['pwd'] ) ) ){
+			if ( ( isset( $_REQUEST['log'] ) && !empty( $_REQUEST['log'] ) )
+				 && ( isset( $_REQUEST['pwd'] ) && !empty( $_REQUEST['pwd'] ) ) ){
 
-		    	// when the user has entered a password or username do the sensei login
-		    	$creds = array();
+				// when the user has entered a password or username do the sensei login
+				$creds = array();
 
-		    	// check if the requests login is an email address
-		    	if( is_email(  trim( $_REQUEST['log'] ) )  ){
-		    		// query wordpress for the users details
-		    		$user =	get_user_by( 'email', sanitize_email( $_REQUEST['log'] )  );
+				// check if the requests login is an email address
+				if( is_email(  trim( $_REQUEST['log'] ) )  ){
+					// query wordpress for the users details
+					$user =	get_user_by( 'email', sanitize_email( $_REQUEST['log'] )  );
 
-		    		// validate the user object
-		    		if( !$user ){
+					// validate the user object
+					if( !$user ){
 
-		    			// the email doesnt exist
-                        wp_redirect( esc_url_raw( add_query_arg('login', 'failed', $referrer) ) );
-		        		exit;
+						// the email doesnt exist
+						wp_redirect( esc_url_raw( add_query_arg('login', 'failed', $referrer) ) );
+						exit;
 
-		    		}
+					}
 
-		    		//assigne the username to the creds array for further processing
-		    		$creds['user_login'] =  $user->user_login ;
+					//assigne the username to the creds array for further processing
+					$creds['user_login'] =  $user->user_login ;
 
-		    	}else{
+				}else{
 
-		    		// process this as a default username login
-		    		$creds['user_login'] = sanitize_text_field( $_REQUEST['log'] ) ;
+					// process this as a default username login
+					$creds['user_login'] = sanitize_text_field( $_REQUEST['log'] ) ;
 
-		    	}
+				}
 
 				// get setup the rest of the creds array
 				$creds['user_password'] = sanitize_text_field( $_REQUEST['pwd'] );
@@ -1597,42 +1597,42 @@ 
 				$user = wp_signon( $creds, false );
 
 				if ( is_wp_error($user) ){ // on login failure
-                    wp_redirect( esc_url_raw( add_query_arg('login', 'failed', $referrer) ) );
-                    exit;
+					wp_redirect( esc_url_raw( add_query_arg('login', 'failed', $referrer) ) );
+					exit;
 				}else{ // on login success
 
 					/**
-					* change the redirect url programatically
-					*
-					* @since 1.6.1
-					*
-					* @param string $referrer the page where the current url wheresensei login form was posted from
-					*/
+					 * change the redirect url programatically
+					 *
+					 * @since 1.6.1
+					 *
+					 * @param string $referrer the page where the current url wheresensei login form was posted from
+					 */
 
 					$success_redirect_url = apply_filters('sesei_login_success_redirect_url', remove_query_arg( 'login', $referrer ) );
 
 					wp_redirect( esc_url_raw( $success_redirect_url ) );
-		        	exit;
+					exit;
 
 				}	// end is_wp_error($user)
 
-		    }else{ // if username or password is empty
+			}else{ // if username or password is empty
 
-                wp_redirect( esc_url_raw( add_query_arg('login', 'emptyfields', $referrer) ) );
-		        exit;
+				wp_redirect( esc_url_raw( add_query_arg('login', 'emptyfields', $referrer) ) );
+				exit;
 
-		    } // end if username $_REQUEST['log']  and password $_REQUEST['pwd'] is empty
+			} // end if username $_REQUEST['log']  and password $_REQUEST['pwd'] is empty
 
-	    }elseif( ( isset( $_GET['login'] ) ) ) {
-	    	// else if this request is a redircect from a previously faile login request
-	    	$this->login_message_process();
+		}elseif( ( isset( $_GET['login'] ) ) ) {
+			// else if this request is a redircect from a previously faile login request
+			$this->login_message_process();
 
 			//exit the handle login request function
 			return;
-	    }
+		}
 
-	    // if none of the above
-	    return;
+		// if none of the above
+		return;
 
 	} // End  sensei_login_fail_redirect_to_front_end_login
 
@@ -1735,7 +1735,7 @@ 
 	 */
 	public function login_message_process(){
 
-            // setup the message variables
+			// setup the message variables
 			$message = '';
 
 			//only output message if the url contains login=failed and login=emptyfields
@@ -1754,22 +1754,22 @@ 
 	}// end login_message_process
 
 
-    /**
-     * sensei_show_admin_bar(). Use WooCommerce filter
-     * to show admin bar to Teachers as well.
-     *
-     * @return void redirect
-     *
-     */
-    public function sensei_show_admin_bar () {
+	/**
+	 * sensei_show_admin_bar(). Use WooCommerce filter
+	 * to show admin bar to Teachers as well.
+	 *
+	 * @return void redirect
+	 *
+	 */
+	public function sensei_show_admin_bar () {
 
-        if (current_user_can('edit_courses')) {
+		if (current_user_can('edit_courses')) {
 
-            add_filter( 'woocommerce_disable_admin_bar', '__return_false', 10, 1);
+			add_filter( 'woocommerce_disable_admin_bar', '__return_false', 10, 1);
 
-        }
+		}
 
-    }
+	}
 
 } // End Class
 

includes/admin/class-sensei-welcome.php

Indentation +39 added lines, -39 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 if ( ! defined( 'ABSPATH' ) ) {
-    exit; // Exit if accessed directly
+	exit; // Exit if accessed directly
 }
 
 /**
@@ -17,45 +17,45 @@ 
 
 	/**
 	 * Hook in tabs.
-     * @since 1.8.0
+	 * @since 1.8.0
 	 */
 	public function __construct() {
 
-        add_action( 'admin_menu', array( $this, 'admin_menus') );
+		add_action( 'admin_menu', array( $this, 'admin_menus') );
 		add_action( 'admin_head', array( $this, 'admin_head' ) );
 
 	}
 
-    /**
-     * Sends user to the welcome page on first activation.
-     *
-     * Hooked into activated_plugin
-     * @since 1.8.0
-     */
-    public static function redirect( $plugin ) {
+	/**
+	 * Sends user to the welcome page on first activation.
+	 *
+	 * Hooked into activated_plugin
+	 * @since 1.8.0
+	 */
+	public static function redirect( $plugin ) {
 
-        // Bail if activating from network, or bulk, or within an iFrame
-        if ( is_network_admin() || isset( $_GET['activate-multi'] ) || defined( 'IFRAME_REQUEST' ) ) {
+		// Bail if activating from network, or bulk, or within an iFrame
+		if ( is_network_admin() || isset( $_GET['activate-multi'] ) || defined( 'IFRAME_REQUEST' ) ) {
 
-            return;
+			return;
 
-        }
-        // don't run for upgrades and for pages already on the welcome screen
-        if ( ( isset( $_GET['action'] ) && 'upgrade-plugin' == $_GET['action'] )
-            || ( ! empty( $_GET['page'] ) && $_GET['page'] === 'sensei-welcome' ) ) {
+		}
+		// don't run for upgrades and for pages already on the welcome screen
+		if ( ( isset( $_GET['action'] ) && 'upgrade-plugin' == $_GET['action'] )
+			|| ( ! empty( $_GET['page'] ) && $_GET['page'] === 'sensei-welcome' ) ) {
 
-            return;
+			return;
 
-        }
+		}
 
-        wp_redirect( admin_url( 'index.php?page=sensei-welcome' ) );
-        exit;
-    }
+		wp_redirect( admin_url( 'index.php?page=sensei-welcome' ) );
+		exit;
+	}
 
 	/**
 	 * Add admin menus/screens.
-     *
-     * @since 1.8.0
+	 *
+	 * @since 1.8.0
 	 */
 	public function admin_menus() {
 
@@ -65,17 +65,17 @@ 
 
 		$welcome_page_name  = __( 'Sensei Activation', 'woothemes-sensei' );
 		$welcome_page_title = __( 'Welcome to Sensei', 'woothemes-sensei' );
-        add_dashboard_page( $welcome_page_title, $welcome_page_name, 'manage_options', 'sensei-welcome', array( $this, 'welcome_screen' ) );
+		add_dashboard_page( $welcome_page_title, $welcome_page_name, 'manage_options', 'sensei-welcome', array( $this, 'welcome_screen' ) );
 
 	}
 
-    /**
-     * Output the Welcome  screen.
-     *
-     * @since 1.8.0
-     */
-    public function welcome_screen() {
-        ?>
+	/**
+	 * Output the Welcome  screen.
+	 *
+	 * @since 1.8.0
+	 */
+	public function welcome_screen() {
+		?>
         <div class="wrap about-wrap">
 
             <?php  $this->intro(); ?>
@@ -128,13 +128,13 @@ 
             </div>
         </div>
     <?php
-    }
+	}
 
 	/**
 	 * Add styles just for this page, and remove dashboard page links.
 	 */
 	public function admin_head() {
-        // remove the menu page so it is not visible in the admin side bar
+		// remove the menu page so it is not visible in the admin side bar
 		remove_submenu_page( 'index.php', 'sensei-welcome' );
 		?>
 		<style type="text/css">
@@ -258,7 +258,7 @@ 
 
 		// Random tweet - must be kept to 102 chars to "fit"
 		$tweets        = array(
-            'I\'ve just installed Sensei - A premium Learning Management plugin for #WordPress.'
+			'I\'ve just installed Sensei - A premium Learning Management plugin for #WordPress.'
 		);
 		shuffle( $tweets );
 		?>
@@ -280,11 +280,11 @@ 
 
 		<div class="sensei-badge">
             <?php
-                _e('Sensei by WooThemes','woothemes-sensei');
-                echo '<br />';
+				_e('Sensei by WooThemes','woothemes-sensei');
+				echo '<br />';
 
-                printf( __( 'Version %s', 'woothemes-sensei' ), Sensei()->version );
-            ?>
+				printf( __( 'Version %s', 'woothemes-sensei' ), Sensei()->version );
+			?>
         </div>
 
 		<p class="sensei-actions">

includes/class-sensei-quiz.php

Indentation +884 added lines, -884 removed lines

@@ -24,69 +24,69 @@ 
 	 */
 	public function __construct ( $file = __FILE__ ) {
 		$this->file = $file;
-        $this->token = 'quiz';
+		$this->token = 'quiz';
 		$this->meta_fields = array( 'quiz_passmark', 'quiz_lesson', 'quiz_type', 'quiz_grade_type', 'pass_required','enable_quiz_reset' );
 		add_action( 'save_post', array( $this, 'update_author' ));
 
 		// listen to the reset button click
 		add_action( 'template_redirect', array( $this, 'reset_button_click_listener'  ) );
 
-        // fire the complete quiz button submit for grading action
-        add_action( 'sensei_complete_quiz', array( $this, 'user_quiz_submit_listener' ) );
+		// fire the complete quiz button submit for grading action
+		add_action( 'sensei_complete_quiz', array( $this, 'user_quiz_submit_listener' ) );
 
 		// fire the save user answers quiz button click responder
 		add_action( 'sensei_complete_quiz', array( $this, 'user_save_quiz_answers_listener' ) );
 
-        // fire the load global data function
-        add_action( 'sensei_complete_quiz', array( $this, 'load_global_quiz_data' ), 80 );
+		// fire the load global data function
+		add_action( 'sensei_complete_quiz', array( $this, 'load_global_quiz_data' ), 80 );
 
-        add_action( 'template_redirect', array ( $this, 'quiz_has_no_questions') );
+		add_action( 'template_redirect', array ( $this, 'quiz_has_no_questions') );
 
 
-    } // End __construct()
+	} // End __construct()
 
 	/**
-	* Update the quiz author when the lesson post type is save
-	*
-	* @param int $post_id
-	* @return void
-	*/
+	 * Update the quiz author when the lesson post type is save
+	 *
+	 * @param int $post_id
+	 * @return void
+	 */
 	public function update_author( $post_id ){
 
 		// If this isn't a 'lesson' post, don't update it.
-        // if this is a revision don't save it
-	    if ( isset( $_POST['post_type'] ) && 'lesson' != $_POST['post_type']
-            || wp_is_post_revision( $post_id ) ) {
+		// if this is a revision don't save it
+		if ( isset( $_POST['post_type'] ) && 'lesson' != $_POST['post_type']
+			|| wp_is_post_revision( $post_id ) ) {
 
-                return;
+				return;
 
-        }
-	    // get the lesson author id to be use late
-	    $saved_post = get_post( $post_id );
-	    $new_lesson_author_id =  $saved_post->post_author;
+		}
+		// get the lesson author id to be use late
+		$saved_post = get_post( $post_id );
+		$new_lesson_author_id =  $saved_post->post_author;
 
-	    //get the lessons quiz
+		//get the lessons quiz
 		$lesson_quizzes = Sensei()->lesson->lesson_quizzes( $post_id );
-	    foreach ( (array) $lesson_quizzes as $quiz_item ) {
+		foreach ( (array) $lesson_quizzes as $quiz_item ) {
 
-	    	if( ! $quiz_item ) {
-	    		continue;
-	    	}
+			if( ! $quiz_item ) {
+				continue;
+			}
 
-		    // setup the quiz items new author value
+			// setup the quiz items new author value
 			$my_post = array(
-			      'ID'           => $quiz_item,
-			      'post_author' =>  $new_lesson_author_id
+				  'ID'           => $quiz_item,
+				  'post_author' =>  $new_lesson_author_id
 			);
 
-            // remove the action so that it doesn't fire again
-            remove_action( 'save_post', array( $this, 'update_author' ));
+			// remove the action so that it doesn't fire again
+			remove_action( 'save_post', array( $this, 'update_author' ));
 
 			// Update the post into the database
 		  	wp_update_post( $my_post );
-	    }
+		}
 
-	    return;
+		return;
 	}// end update_author
 
 
@@ -117,32 +117,32 @@ 
 	} // end lesson
 
 
-    /**
-     * user_save_quiz_answers_listener
-     *
-     * This function hooks into the quiz page and accepts the answer form save post.
-     * @since 1.7.3
-     * @return bool $saved;
-     */
-    public function user_save_quiz_answers_listener(){
+	/**
+	 * user_save_quiz_answers_listener
+	 *
+	 * This function hooks into the quiz page and accepts the answer form save post.
+	 * @since 1.7.3
+	 * @return bool $saved;
+	 */
+	public function user_save_quiz_answers_listener(){
 
-        if( ! isset( $_POST[ 'quiz_save' ])
-            || !isset( $_POST[ 'sensei_question' ] )
-            || empty( $_POST[ 'sensei_question' ] )
-            ||  ! wp_verify_nonce( $_POST['woothemes_sensei_save_quiz_nonce'], 'woothemes_sensei_save_quiz_nonce'  ) > 1 ) {
-            return;
-        }
+		if( ! isset( $_POST[ 'quiz_save' ])
+			|| !isset( $_POST[ 'sensei_question' ] )
+			|| empty( $_POST[ 'sensei_question' ] )
+			||  ! wp_verify_nonce( $_POST['woothemes_sensei_save_quiz_nonce'], 'woothemes_sensei_save_quiz_nonce'  ) > 1 ) {
+			return;
+		}
 
-        global $post;
-        $lesson_id = $this->get_lesson_id( $post->ID );
-        $quiz_answers = $_POST[ 'sensei_question' ];
-        // call the save function
-        self::save_user_answers( $quiz_answers, $_FILES , $lesson_id  , get_current_user_id() );
+		global $post;
+		$lesson_id = $this->get_lesson_id( $post->ID );
+		$quiz_answers = $_POST[ 'sensei_question' ];
+		// call the save function
+		self::save_user_answers( $quiz_answers, $_FILES , $lesson_id  , get_current_user_id() );
 
-        // remove the hook as it should only fire once per click
-        remove_action( 'sensei_complete_quiz', 'user_save_quiz_answers_listener' );
+		// remove the hook as it should only fire once per click
+		remove_action( 'sensei_complete_quiz', 'user_save_quiz_answers_listener' );
 
-    } // end user_save_quiz_answers_listener
+	} // end user_save_quiz_answers_listener
 
 	/**
 	 * Save the user answers for the given lesson's quiz
@@ -153,7 +153,7 @@ 
 	 * @access public
 	 *
 	 * @param array $quiz_answers
-     * @param array $files from global $_FILES
+	 * @param array $files from global $_FILES
 	 * @param int $lesson_id
 	 * @param int $user_id
 	 *
@@ -161,11 +161,11 @@ 
 	 */
 	public static function save_user_answers( $quiz_answers, $files = array(), $lesson_id , $user_id = 0 ){
 
-        if( ! ( $user_id > 0 ) ){
-            $user_id = get_current_user_id();
-        }
+		if( ! ( $user_id > 0 ) ){
+			$user_id = get_current_user_id();
+		}
 
-        // make sure the parameters are valid before continuing
+		// make sure the parameters are valid before continuing
 		if( empty( $lesson_id ) || empty( $user_id )
 			|| 'lesson' != get_post_type( $lesson_id )
 			||!get_userdata( $user_id )
@@ -176,25 +176,25 @@ 
 		}
 
 
-        // start the lesson before saving the data in case the user has not started the lesson
-        $activity_logged = Sensei_Utils::sensei_start_lesson( $lesson_id, $user_id );
+		// start the lesson before saving the data in case the user has not started the lesson
+		$activity_logged = Sensei_Utils::sensei_start_lesson( $lesson_id, $user_id );
 
 		//prepare the answers
 		$prepared_answers = self::prepare_form_submitted_answers( $quiz_answers , $files );
 
 		// save the user data
-        $answers_saved = Sensei_Utils::add_user_data( 'quiz_answers', $lesson_id, $prepared_answers, $user_id ) ;
+		$answers_saved = Sensei_Utils::add_user_data( 'quiz_answers', $lesson_id, $prepared_answers, $user_id ) ;
 
 		// were the answers saved correctly?
 		if( intval( $answers_saved ) > 0){
 
-            // save transient to make retrieval faster
-            $transient_key = 'sensei_answers_'.$user_id.'_'.$lesson_id;
-            set_transient( $transient_key, $prepared_answers, 10 * DAY_IN_SECONDS );
+			// save transient to make retrieval faster
+			$transient_key = 'sensei_answers_'.$user_id.'_'.$lesson_id;
+			set_transient( $transient_key, $prepared_answers, 10 * DAY_IN_SECONDS );
 
-            // update the message showed to user
-            Sensei()->frontend->messages = '<div class="sensei-message note">' . __( 'Quiz Saved Successfully.', 'woothemes-sensei' )  . '</div>';
-        }
+			// update the message showed to user
+			Sensei()->frontend->messages = '<div class="sensei-message note">' . __( 'Quiz Saved Successfully.', 'woothemes-sensei' )  . '</div>';
+		}
 
 		return $answers_saved;
 
@@ -202,9 +202,9 @@ 
 
 	/**
 	 * Get the user answers for the given lesson's quiz.
-     *
-     * This function returns the data that is stored on the lesson as meta and is not compatible with
-     * retrieving data for quiz answer before sensei 1.7.4
+	 *
+	 * This function returns the data that is stored on the lesson as meta and is not compatible with
+	 * retrieving data for quiz answer before sensei 1.7.4
 	 *
 	 *
 	 * @since 1.7.4
@@ -224,27 +224,27 @@ 
 			return false;
 		}
 
-        // save some time and get the transient cached data
-        $transient_key = 'sensei_answers_'.$user_id.'_'.$lesson_id;
-        $transient_cached_answers = get_transient( $transient_key );
+		// save some time and get the transient cached data
+		$transient_key = 'sensei_answers_'.$user_id.'_'.$lesson_id;
+		$transient_cached_answers = get_transient( $transient_key );
 
-        // return the transient or get the values get the values from the comment meta
-        if( !empty( $transient_cached_answers  ) && false != $transient_cached_answers ){
+		// return the transient or get the values get the values from the comment meta
+		if( !empty( $transient_cached_answers  ) && false != $transient_cached_answers ){
 
-            $encoded_user_answers = $transient_cached_answers;
+			$encoded_user_answers = $transient_cached_answers;
 
-        }else{
+		}else{
 
-            $encoded_user_answers = Sensei_Utils::get_user_data( 'quiz_answers', $lesson_id  , $user_id );
+			$encoded_user_answers = Sensei_Utils::get_user_data( 'quiz_answers', $lesson_id  , $user_id );
 
-        } // end if transient check
+		} // end if transient check
 
 		if( ! is_array( $encoded_user_answers ) ){
 			return false;
 		}
 
-        //set the transient with the new valid data for faster retrieval in future
-        set_transient( $transient_key,  $encoded_user_answers, 10 * DAY_IN_SECONDS);
+		//set the transient with the new valid data for faster retrieval in future
+		set_transient( $transient_key,  $encoded_user_answers, 10 * DAY_IN_SECONDS);
 
 		// decode an unserialize all answers
 		foreach( $encoded_user_answers as $question_id => $encoded_answer ) {
@@ -278,8 +278,8 @@ 
 		$current_quiz_id = $post->ID;
 		$lesson_id = $this->get_lesson_id( $current_quiz_id );
 
-        // reset all user data
-        $this->reset_user_lesson_data( $lesson_id, get_current_user_id() );
+		// reset all user data
+		$this->reset_user_lesson_data( $lesson_id, get_current_user_id() );
 
 		//this function should only run once
 		remove_action( 'template_redirect', array( $this, 'reset_button_click_listener'  ) );
@@ -290,7 +290,7 @@ 
 	 * Complete/ submit  quiz hooked function
 	 *
 	 * This function listens to the complete button submit action and processes the users submitted answers
-     * not that this function submits the given users quiz answers for grading.
+	 * not that this function submits the given users quiz answers for grading.
 	 *
 	 * @since  1.7.4
 	 * @access public
@@ -300,90 +300,90 @@ 
 	 */
 	public function user_quiz_submit_listener() {
 
-        // only respond to valid quiz completion submissions
-        if( ! isset( $_POST[ 'quiz_complete' ])
-            || !isset( $_POST[ 'sensei_question' ] )
-            || empty( $_POST[ 'sensei_question' ] )
-            ||  ! wp_verify_nonce( $_POST['woothemes_sensei_complete_quiz_nonce'], 'woothemes_sensei_complete_quiz_nonce'  ) > 1 ) {
-            return;
-        }
+		// only respond to valid quiz completion submissions
+		if( ! isset( $_POST[ 'quiz_complete' ])
+			|| !isset( $_POST[ 'sensei_question' ] )
+			|| empty( $_POST[ 'sensei_question' ] )
+			||  ! wp_verify_nonce( $_POST['woothemes_sensei_complete_quiz_nonce'], 'woothemes_sensei_complete_quiz_nonce'  ) > 1 ) {
+			return;
+		}
 
-        global $post, $current_user;
-        $lesson_id = $this->get_lesson_id( $post->ID );
-        $quiz_answers = $_POST[ 'sensei_question' ];
+		global $post, $current_user;
+		$lesson_id = $this->get_lesson_id( $post->ID );
+		$quiz_answers = $_POST[ 'sensei_question' ];
 
-        self::submit_answers_for_grading( $quiz_answers, $_FILES ,  $lesson_id  , $current_user->ID );
+		self::submit_answers_for_grading( $quiz_answers, $_FILES ,  $lesson_id  , $current_user->ID );
 
 	} // End sensei_complete_quiz()
 
-    /**
-     * This function set's up the data need for the quiz page
-     *
-     * This function hooks into sensei_complete_quiz and load the global data for the
-     * current quiz.
-     *
-     * @since 1.7.4
-     * @access public
-     *
-     */
-    public function load_global_quiz_data(){
+	/**
+	 * This function set's up the data need for the quiz page
+	 *
+	 * This function hooks into sensei_complete_quiz and load the global data for the
+	 * current quiz.
+	 *
+	 * @since 1.7.4
+	 * @access public
+	 *
+	 */
+	public function load_global_quiz_data(){
 
-        global  $post, $current_user;
-        $this->data = new stdClass();
+		global  $post, $current_user;
+		$this->data = new stdClass();
 
-        // Default grade
-        $grade = 0;
+		// Default grade
+		$grade = 0;
 
-        // Get Quiz Questions
-        $lesson_quiz_questions = Sensei()->lesson->lesson_quiz_questions( $post->ID );
+		// Get Quiz Questions
+		$lesson_quiz_questions = Sensei()->lesson->lesson_quiz_questions( $post->ID );
 
-        $quiz_lesson_id = absint( get_post_meta( $post->ID, '_quiz_lesson', true ) );
+		$quiz_lesson_id = absint( get_post_meta( $post->ID, '_quiz_lesson', true ) );
 
-        // Get quiz grade type
-        $quiz_grade_type = get_post_meta( $post->ID, '_quiz_grade_type', true );
+		// Get quiz grade type
+		$quiz_grade_type = get_post_meta( $post->ID, '_quiz_grade_type', true );
 
-        // Get quiz pass setting
-        $pass_required = get_post_meta( $post->ID, '_pass_required', true );
+		// Get quiz pass setting
+		$pass_required = get_post_meta( $post->ID, '_pass_required', true );
 
-        // Get quiz pass mark
-        $quiz_passmark = abs( round( doubleval( get_post_meta( $post->ID, '_quiz_passmark', true ) ), 2 ) );
+		// Get quiz pass mark
+		$quiz_passmark = abs( round( doubleval( get_post_meta( $post->ID, '_quiz_passmark', true ) ), 2 ) );
 
-        // Get latest quiz answers and grades
-        $lesson_id = Sensei()->quiz->get_lesson_id( $post->ID );
-        $user_quizzes = Sensei()->quiz->get_user_answers( $lesson_id, get_current_user_id() );
-        $user_lesson_status = Sensei_Utils::user_lesson_status( $quiz_lesson_id, $current_user->ID );
-        $user_quiz_grade = 0;
-        if( isset( $user_lesson_status->comment_ID ) ) {
-            $user_quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
-        }
+		// Get latest quiz answers and grades
+		$lesson_id = Sensei()->quiz->get_lesson_id( $post->ID );
+		$user_quizzes = Sensei()->quiz->get_user_answers( $lesson_id, get_current_user_id() );
+		$user_lesson_status = Sensei_Utils::user_lesson_status( $quiz_lesson_id, $current_user->ID );
+		$user_quiz_grade = 0;
+		if( isset( $user_lesson_status->comment_ID ) ) {
+			$user_quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
+		}
 
-        if ( ! is_array($user_quizzes) ) { $user_quizzes = array(); }
+		if ( ! is_array($user_quizzes) ) { $user_quizzes = array(); }
 
-        // Check again that the lesson is complete
-        $user_lesson_end = Sensei_Utils::user_completed_lesson( $user_lesson_status );
-        $user_lesson_complete = false;
-        if ( $user_lesson_end ) {
-            $user_lesson_complete = true;
-        } // End If Statement
+		// Check again that the lesson is complete
+		$user_lesson_end = Sensei_Utils::user_completed_lesson( $user_lesson_status );
+		$user_lesson_complete = false;
+		if ( $user_lesson_end ) {
+			$user_lesson_complete = true;
+		} // End If Statement
 
-        $reset_allowed = get_post_meta( $post->ID, '_enable_quiz_reset', true );
-        //backwards compatibility
-        if( 'on' == $reset_allowed ) {
-            $reset_allowed = 1;
-        }
+		$reset_allowed = get_post_meta( $post->ID, '_enable_quiz_reset', true );
+		//backwards compatibility
+		if( 'on' == $reset_allowed ) {
+			$reset_allowed = 1;
+		}
 
-        // Build frontend data object for backwards compatibility
-        // using this is no longer recommended
-        $this->data->user_quiz_grade = $user_quiz_grade;// Sensei_Quiz::get_user_quiz_grade( $lesson_id, get_current_user_id() );
-        $this->data->quiz_passmark = $quiz_passmark;
-        $this->data->quiz_lesson = $quiz_lesson_id;
-        $this->data->quiz_grade_type = $quiz_grade_type; // get_post_meta( $quiz_id, '_quiz_grade_type', true );
-        $this->data->user_lesson_end = $user_lesson_end;
-        $this->data->user_lesson_complete = $user_lesson_complete; //Sensei_Utils::user_completed_lesson( $lesson_id, get_current_user_id() );
-        $this->data->lesson_quiz_questions = $lesson_quiz_questions;
-        $this->data->reset_quiz_allowed = $reset_allowed; // Sensei_Quiz::is_reset_allowed( $lesson_id );
+		// Build frontend data object for backwards compatibility
+		// using this is no longer recommended
+		$this->data->user_quiz_grade = $user_quiz_grade;// Sensei_Quiz::get_user_quiz_grade( $lesson_id, get_current_user_id() );
+		$this->data->quiz_passmark = $quiz_passmark;
+		$this->data->quiz_lesson = $quiz_lesson_id;
+		$this->data->quiz_grade_type = $quiz_grade_type; // get_post_meta( $quiz_id, '_quiz_grade_type', true );
+		$this->data->user_lesson_end = $user_lesson_end;
+		$this->data->user_lesson_complete = $user_lesson_complete; //Sensei_Utils::user_completed_lesson( $lesson_id, get_current_user_id() );
+		$this->data->lesson_quiz_questions = $lesson_quiz_questions;
+		$this->data->reset_quiz_allowed = $reset_allowed; // Sensei_Quiz::is_reset_allowed( $lesson_id );
 
-    } // end load_global_quiz_data
+	} // end load_global_quiz_data
 
 
 	/**
@@ -413,25 +413,25 @@ 
 		foreach( $unprepared_answers as $question_id => $answer ) {
 
 			//get the current questions 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 );
 			}
 
-            // compress the answer for saving
+			// compress the answer for saving
 			if( 'multi-line' == $question_type ) {
-                $answer = esc_html( $answer );
-            }elseif( 'file-upload' == $question_type  ){
-                $file_key = 'file_upload_' . $question_id;
-                if( isset( $files[ $file_key ] ) ) {
-                        $attachment_id = Sensei_Utils::upload_file(  $files[ $file_key ] );
-                        if( $attachment_id ) {
-                            $answer = $attachment_id;
-                        }
-                    }
-            } // end if
+				$answer = esc_html( $answer );
+			}elseif( 'file-upload' == $question_type  ){
+				$file_key = 'file_upload_' . $question_id;
+				if( isset( $files[ $file_key ] ) ) {
+						$attachment_id = Sensei_Utils::upload_file(  $files[ $file_key ] );
+						if( $attachment_id ) {
+							$answer = $attachment_id;
+						}
+					}
+			} // end if
 
 			$prepared_answers[ $question_id ] =  base64_encode( maybe_serialize( $answer ) );
 
@@ -440,813 +440,813 @@ 
 		return $prepared_answers;
 	} // prepare_form_submitted_answers
 
-    /**
-     * Reset user submitted questions
-     *
-     * This function resets the quiz data for a user that has been submitted fro grading already. It is different to
-     * the save_user_answers as currently the saved and submitted answers are stored differently.
-     *
-     * @since 1.7.4
-     * @access public
-     *
-     * @return bool $reset_success
-     * @param int $user_id
-     * @param int $lesson_id
-     */
-    public function reset_user_lesson_data( $lesson_id , $user_id = 0 ){
+	/**
+	 * Reset user submitted questions
+	 *
+	 * This function resets the quiz data for a user that has been submitted fro grading already. It is different to
+	 * the save_user_answers as currently the saved and submitted answers are stored differently.
+	 *
+	 * @since 1.7.4
+	 * @access public
+	 *
+	 * @return bool $reset_success
+	 * @param int $user_id
+	 * @param int $lesson_id
+	 */
+	public function reset_user_lesson_data( $lesson_id , $user_id = 0 ){
+
+		//make sure the parameters are valid
+		if( empty( $lesson_id ) || empty( $user_id )
+			|| 'lesson' != get_post_type( $lesson_id )
+			|| ! get_userdata( $user_id ) ){
+			return false;
+		}
+
+
+
+		//get the users lesson status to make
+		$user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
+		if( ! isset( $user_lesson_status->comment_ID ) ) {
+			// this user is not taking this lesson so this process is not needed
+			return false;
+		}
+
+		//get the lesson quiz and course
+		$quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
+		$course_id = Sensei()->lesson->get_course_id( $lesson_id );
+
+		// reset the transients
+		$answers_transient_key = 'sensei_answers_'.$user_id.'_'.$lesson_id;
+		$grades_transient_key = 'quiz_grades_'.$user_id.'_'.$lesson_id;
+		$answers_feedback_transient_key = 'sensei_answers_feedback_'.$user_id.'_'.$lesson_id;
+		delete_transient( $answers_transient_key );
+		delete_transient( $grades_transient_key );
+		delete_transient( $answers_feedback_transient_key );
+
+		// reset the quiz answers and feedback notes
+		$deleted_answers = Sensei_Utils::delete_user_data( 'quiz_answers', $lesson_id, $user_id );
+		$deleted_grades = Sensei_Utils::delete_user_data( 'quiz_grades', $lesson_id, $user_id );
+		$deleted_user_feedback = Sensei_Utils::delete_user_data( 'quiz_answers_feedback', $lesson_id, $user_id );
+
+		// Delete quiz answers, this auto deletes the corresponding meta data, such as the question/answer grade
+		Sensei_Utils::sensei_delete_quiz_answers( $quiz_id, $user_id );
+
+		Sensei_Utils::update_lesson_status( $user_id , $lesson_id, 'in-progress', array( 'questions_asked' => '', 'grade' => '' ) );
+
+		// Update course completion
+		Sensei_Utils::update_course_status( $user_id, $course_id );
 
-        //make sure the parameters are valid
-        if( empty( $lesson_id ) || empty( $user_id )
-            || 'lesson' != get_post_type( $lesson_id )
-            || ! get_userdata( $user_id ) ){
-            return false;
-        }
-
-
-
-        //get the users lesson status to make
-        $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
-        if( ! isset( $user_lesson_status->comment_ID ) ) {
-            // this user is not taking this lesson so this process is not needed
-            return false;
-        }
+		// Run any action on quiz/lesson reset (previously this didn't occur on resetting a quiz, see resetting a lesson in sensei_complete_lesson()
+		do_action( 'sensei_user_lesson_reset', $user_id, $lesson_id );
+		Sensei()->frontend->messages = '<div class="sensei-message note">' . __( 'Quiz Reset Successfully.', 'woothemes-sensei' ) . '</div>';
 
-        //get the lesson quiz and course
-        $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
-        $course_id = Sensei()->lesson->get_course_id( $lesson_id );
+		return ( $deleted_answers && $deleted_grades ) ;
 
-        // reset the transients
-        $answers_transient_key = 'sensei_answers_'.$user_id.'_'.$lesson_id;
-        $grades_transient_key = 'quiz_grades_'.$user_id.'_'.$lesson_id;
-        $answers_feedback_transient_key = 'sensei_answers_feedback_'.$user_id.'_'.$lesson_id;
-        delete_transient( $answers_transient_key );
-        delete_transient( $grades_transient_key );
-        delete_transient( $answers_feedback_transient_key );
+	} // end reset_user_lesson_data
 
-        // reset the quiz answers and feedback notes
-        $deleted_answers = Sensei_Utils::delete_user_data( 'quiz_answers', $lesson_id, $user_id );
-        $deleted_grades = Sensei_Utils::delete_user_data( 'quiz_grades', $lesson_id, $user_id );
-        $deleted_user_feedback = Sensei_Utils::delete_user_data( 'quiz_answers_feedback', $lesson_id, $user_id );
+	 /**
+	  * Submit the users quiz answers for grading
+	  *
+	  * This function accepts users answers and stores it but also initiates the grading
+	  * if a quiz can be graded automatically it will, if not the answers can be graded by the teacher.
+	  *
+	  * @since 1.7.4
+	  * @access public
+	  *
+	  * @param array $quiz_answers
+	  * @param array $files from $_FILES
+	  * @param int $user_id
+	  * @param int $lesson_id
+	  *
+	  * @return bool $answers_submitted
+	  */
+	 public static function submit_answers_for_grading( $quiz_answers , $files = array() , $lesson_id , $user_id = 0 ){
 
-        // Delete quiz answers, this auto deletes the corresponding meta data, such as the question/answer grade
-        Sensei_Utils::sensei_delete_quiz_answers( $quiz_id, $user_id );
+		 $answers_submitted = false;
 
-        Sensei_Utils::update_lesson_status( $user_id , $lesson_id, 'in-progress', array( 'questions_asked' => '', 'grade' => '' ) );
+		 // get the user_id if none was passed in use the current logged in user
+		 if( ! intval( $user_id ) > 0 ) {
+			 $user_id = get_current_user_id();
+		 }
 
-        // Update course completion
-        Sensei_Utils::update_course_status( $user_id, $course_id );
+		 // make sure the parameters are valid before continuing
+		 if( empty( $lesson_id ) || empty( $user_id )
+			 || 'lesson' != get_post_type( $lesson_id )
+			 ||!get_userdata( $user_id )
+			 || !is_array( $quiz_answers ) ){
 
-        // Run any action on quiz/lesson reset (previously this didn't occur on resetting a quiz, see resetting a lesson in sensei_complete_lesson()
-        do_action( 'sensei_user_lesson_reset', $user_id, $lesson_id );
-        Sensei()->frontend->messages = '<div class="sensei-message note">' . __( 'Quiz Reset Successfully.', 'woothemes-sensei' ) . '</div>';
+			 return false;
 
-        return ( $deleted_answers && $deleted_grades ) ;
+		 }
 
-    } // end reset_user_lesson_data
+		 // Default grade
+		 $grade = 0;
 
-     /**
-      * Submit the users quiz answers for grading
-      *
-      * This function accepts users answers and stores it but also initiates the grading
-      * if a quiz can be graded automatically it will, if not the answers can be graded by the teacher.
-      *
-      * @since 1.7.4
-      * @access public
-      *
-      * @param array $quiz_answers
-      * @param array $files from $_FILES
-      * @param int $user_id
-      * @param int $lesson_id
-      *
-      * @return bool $answers_submitted
-      */
-     public static function submit_answers_for_grading( $quiz_answers , $files = array() , $lesson_id , $user_id = 0 ){
+		 // Get Quiz ID
+		 $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
 
-         $answers_submitted = false;
+		 // Get quiz grade type
+		 $quiz_grade_type = get_post_meta( $quiz_id, '_quiz_grade_type', true );
 
-         // get the user_id if none was passed in use the current logged in user
-         if( ! intval( $user_id ) > 0 ) {
-             $user_id = get_current_user_id();
-         }
+		 // Get quiz pass setting
+		 $pass_required = get_post_meta( $quiz_id, '_pass_required', true );
 
-         // make sure the parameters are valid before continuing
-         if( empty( $lesson_id ) || empty( $user_id )
-             || 'lesson' != get_post_type( $lesson_id )
-             ||!get_userdata( $user_id )
-             || !is_array( $quiz_answers ) ){
+		 // Get the minimum percentage need to pass this quiz
+		 $quiz_pass_percentage = abs( round( doubleval( get_post_meta( $quiz_id, '_quiz_passmark', true ) ), 2 ) );
 
-             return false;
+		 // Handle Quiz Questions asked
+		 // This is to ensure we save the questions that we've asked this user and that this can't be change unless
+		 // the quiz is reset by admin or user( user: only if the setting is enabled ).
+		 // get the questions asked when when the quiz questions were generated for the user : Sensei_Lesson::lesson_quiz_questions
+		 $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
+		 $questions_asked = get_comment_meta( $user_lesson_status->comment_ID, 'questions_asked', true );
+		 if( empty( $questions_asked ) ){
 
-         }
+			 $questions_asked = array_keys( $quiz_answers );
+			 $questions_asked_string = implode( ',', $questions_asked );
 
-         // Default grade
-         $grade = 0;
+			 // Save questions that were asked in this quiz
+			 update_comment_meta( $user_lesson_status->comment_ID, 'questions_asked', $questions_asked_string );
 
-         // Get Quiz ID
-         $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
+		 }
 
-         // Get quiz grade type
-         $quiz_grade_type = get_post_meta( $quiz_id, '_quiz_grade_type', true );
+		 // Save Quiz Answers for grading, the save function also calls the sensei_start_lesson
+		 self::save_user_answers( $quiz_answers , $files , $lesson_id , $user_id );
 
-         // Get quiz pass setting
-         $pass_required = get_post_meta( $quiz_id, '_pass_required', true );
+		 // Grade quiz
+		 $grade = Sensei_Grading::grade_quiz_auto( $quiz_id, $quiz_answers, 0 , $quiz_grade_type );
 
-         // Get the minimum percentage need to pass this quiz
-         $quiz_pass_percentage = abs( round( doubleval( get_post_meta( $quiz_id, '_quiz_passmark', true ) ), 2 ) );
+		 // Get Lesson Grading Setting
+		 $lesson_metadata = array();
+		 $lesson_status = 'ungraded'; // Default when completing a quiz
 
-         // Handle Quiz Questions asked
-         // This is to ensure we save the questions that we've asked this user and that this can't be change unless
-         // the quiz is reset by admin or user( user: only if the setting is enabled ).
-         // get the questions asked when when the quiz questions were generated for the user : Sensei_Lesson::lesson_quiz_questions
-         $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
-         $questions_asked = get_comment_meta( $user_lesson_status->comment_ID, 'questions_asked', true );
-         if( empty( $questions_asked ) ){
+		 // At this point the answers have been submitted
+		 $answers_submitted = true;
 
-             $questions_asked = array_keys( $quiz_answers );
-             $questions_asked_string = implode( ',', $questions_asked );
-
-             // Save questions that were asked in this quiz
-             update_comment_meta( $user_lesson_status->comment_ID, 'questions_asked', $questions_asked_string );
+		 // if this condition is false the quiz should manually be graded by admin
+		 if ('auto' == $quiz_grade_type && ! is_wp_error( $grade )  ) {
 
-         }
+			 // Quiz has been automatically Graded
+			 if ( 'on' == $pass_required ) {
 
-         // Save Quiz Answers for grading, the save function also calls the sensei_start_lesson
-         self::save_user_answers( $quiz_answers , $files , $lesson_id , $user_id );
+				 // Student has reached the pass mark and lesson is complete
+				 if ( $quiz_pass_percentage <= $grade ) {
 
-         // Grade quiz
-         $grade = Sensei_Grading::grade_quiz_auto( $quiz_id, $quiz_answers, 0 , $quiz_grade_type );
+					 $lesson_status = 'passed';
 
-         // Get Lesson Grading Setting
-         $lesson_metadata = array();
-         $lesson_status = 'ungraded'; // Default when completing a quiz
+				 } else {
 
-         // At this point the answers have been submitted
-         $answers_submitted = true;
+					 $lesson_status = 'failed';
 
-         // if this condition is false the quiz should manually be graded by admin
-         if ('auto' == $quiz_grade_type && ! is_wp_error( $grade )  ) {
+				 } // End If Statement
 
-             // Quiz has been automatically Graded
-             if ( 'on' == $pass_required ) {
+			 } else {
 
-                 // Student has reached the pass mark and lesson is complete
-                 if ( $quiz_pass_percentage <= $grade ) {
+				 // Student only has to partake the quiz
+				 $lesson_status = 'graded';
+
+			 }
+
+			 $lesson_metadata['grade'] = $grade; // Technically already set as part of "WooThemes_Sensei_Utils::sensei_grade_quiz_auto()" above
+
+		 } // end if ! is_wp_error( $grade ...
+
+		 Sensei_Utils::update_lesson_status( $user_id, $lesson_id, $lesson_status, $lesson_metadata );
 
-                     $lesson_status = 'passed';
+		 if( 'passed' == $lesson_status || 'graded' == $lesson_status ){
 
-                 } else {
+			 /**
+			  * Lesson end action hook
+			  *
+			  * This hook is fired after a lesson quiz has been graded and the lesson status is 'passed' OR 'graded'
+			  *
+			  * @param int $user_id
+			  * @param int $lesson_id
+			  */
+			 do_action( 'sensei_user_lesson_end', $user_id, $lesson_id );
 
-                     $lesson_status = 'failed';
+		 }
 
-                 } // End If Statement
+		 /**
+		  * User quiz has been submitted
+		  *
+		  * Fires the end of the submit_answers_for_grading function. It will fire irrespective of the submission
+		  * results.
+		  *
+		  * @param int $user_id
+		  * @param int $quiz_id
+		  * @param string $grade
+		  * @param string $quiz_pass_percentage
+		  * @param string $quiz_grade_type
+		  */
+		 do_action( 'sensei_user_quiz_submitted', $user_id, $quiz_id, $grade, $quiz_pass_percentage, $quiz_grade_type );
 
-             } else {
+		 return $answers_submitted;
 
-                 // Student only has to partake the quiz
-                 $lesson_status = 'graded';
-
-             }
-
-             $lesson_metadata['grade'] = $grade; // Technically already set as part of "WooThemes_Sensei_Utils::sensei_grade_quiz_auto()" above
-
-         } // end if ! is_wp_error( $grade ...
-
-         Sensei_Utils::update_lesson_status( $user_id, $lesson_id, $lesson_status, $lesson_metadata );
+	 }// end submit_answers_for_grading
 
-         if( 'passed' == $lesson_status || 'graded' == $lesson_status ){
+	 /**
+	  * Get the user question answer
+	  *
+	  * This function gets the the users saved answer on given quiz for the given question parameter
+	  * this function allows for a fallback to users still using the question saved data from before 1.7.4
+	  *
+	  * @since 1.7.4
+	  *
+	  * @param int  $lesson_id
+	  * @param int $question_id
+	  * @param int  $user_id ( optional )
+	  *
+	  * @return bool $answers_submitted
+	  */
+	 public function get_user_question_answer( $lesson_id, $question_id, $user_id = 0 ){
 
-             /**
-              * Lesson end action hook
-              *
-              * This hook is fired after a lesson quiz has been graded and the lesson status is 'passed' OR 'graded'
-              *
-              * @param int $user_id
-              * @param int $lesson_id
-              */
-             do_action( 'sensei_user_lesson_end', $user_id, $lesson_id );
+		 // parameter validation
+		 if( empty( $lesson_id ) || empty( $question_id )
+			 || ! ( intval( $lesson_id  ) > 0 )
+			 || ! ( intval( $question_id  ) > 0 )
+			 || 'lesson' != get_post_type( $lesson_id )
+			 || 'question' != get_post_type( $question_id )) {
 
-         }
+			 return false;
+		 }
+
+		 if( ! ( intval( $user_id ) > 0 )   ){
+			 $user_id = get_current_user_id();
+		 }
+
+		 $users_answers = $this->get_user_answers( $lesson_id, $user_id );
+
+		 if( !$users_answers || empty( $users_answers )
+		 ||  ! is_array( $users_answers ) || ! isset( $users_answers[ $question_id ] ) ){
+
+			 //Fallback for pre 1.7.4 data
+			 $comment =  Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer' ), true );
+
+			 if( ! isset( $comment->comment_content ) ){
+				 return false;
+			 }
+
+			 return maybe_unserialize( base64_decode( $comment->comment_content ) );
+		 }
+
+		 return $users_answers[ $question_id ];
+
+	 }// end get_user_question_answer
+
+	 /**
+	  * Saving the users quiz question grades
+	  *
+	  * This function save all the grades for all the question in a given quiz on the lesson
+	  * comment meta. It makes use of transients to save the grades for easier access at a later stage
+	  *
+	  * @since 1.7.4
+	  *
+	  * @param array $quiz_grades{
+	  *      @type int $question_id
+	  *      @type int $question_grade
+	  * }
+	  * @param $lesson_id
+	  * @param $user_id (Optional) will use the current user if not supplied
+	  *
+	  * @return bool
+	  */
+	 public function set_user_grades( $quiz_grades, $lesson_id, $user_id = 0 ){
+
+		 // get the user_id if none was passed in use the current logged in user
+		 if( ! intval( $user_id ) > 0 ) {
+			 $user_id = get_current_user_id();
+		 }
+
+		 // make sure the parameters are valid before continuing
+		 if( empty( $lesson_id ) || empty( $user_id )
+			 || 'lesson' != get_post_type( $lesson_id )
+			 ||!get_userdata( $user_id )
+			 || !is_array( $quiz_grades ) ){
+
+			 return false;
+
+		 }
+
+		 $success = false;
+
+		 // save that data for the user on the lesson comment meta
+		 $comment_meta_id = Sensei_Utils::add_user_data( 'quiz_grades', $lesson_id, $quiz_grades, $user_id   );
+
+		 // were the grades save successfully ?
+		 if( intval( $comment_meta_id ) > 0 ) {
+
+			 $success = true;
+			 // save transient
+			 $transient_key = 'quiz_grades_'. $user_id . '_' . $lesson_id;
+			 set_transient( $transient_key, $quiz_grades, 10 * DAY_IN_SECONDS );
+		 }
+
+		 return $success;
+
+	 }// end set_user_grades
+
+	 /**
+	  * Retrieve the users quiz question grades
+	  *
+	  * This function gets all the grades for all the questions in the given lesson quiz for a specific user.
+	  *
+	  * @since 1.7.4
+	  *
+	  * @param $lesson_id
+	  * @param $user_id (Optional) will use the current user if not supplied
+	  *
+	  * @return array $user_quiz_grades or false if none exists for this users
+	  */
+	 public function get_user_grades( $lesson_id, $user_id = 0 ){
+
+		 $user_grades = array();
+
+		 // get the user_id if none was passed in use the current logged in user
+		 if( ! intval( $user_id ) > 0 ) {
+			 $user_id = get_current_user_id();
+		 }
+
+		 if ( ! intval( $lesson_id ) > 0 || 'lesson' != get_post_type( $lesson_id )
+			 || ! intval( $user_id )  > 0 || !get_userdata( $user_id )  ) {
+			 return false;
+		 }
+
+		 // save some time and get the transient cached data
+		 $transient_key = 'quiz_grades_'. $user_id . '_' . $lesson_id;
+		 $user_grades = get_transient( $transient_key );
+
+		 // get the data if nothing was stored in the transient
+		 if( empty( $user_grades  ) || false != $user_grades ){
+
+			 $user_grades = Sensei_Utils::get_user_data( 'quiz_grades', $lesson_id, $user_id );
+
+			 //set the transient with the new valid data for faster retrieval in future
+			 set_transient( $transient_key,  $user_grades, 10 * DAY_IN_SECONDS );
+
+		 } // end if transient check
+
+		 // if there is no data for this user
+		 if( ! is_array( $user_grades ) ){
+			 return false;
+		 }
+
+		 return $user_grades;
+
+	 }// end  get_user_grades
+
+	 /**
+	  * Get the user question grade
+	  *
+	  * This function gets the grade on a quiz for the given question parameter
+	  * It does NOT do any grading. It simply retrieves the data that was stored during grading.
+	  * this function allows for a fallback to users still using the question saved data from before 1.7.4
+	  *
+	  * @since 1.7.4
+	  *
+	  * @param int  $lesson_id
+	  * @param int $question_id
+	  * @param int  $user_id ( optional )
+	  *
+	  * @return bool $question_grade
+	  */
+	 public function get_user_question_grade( $lesson_id, $question_id, $user_id = 0 ){
+
+		 // parameter validation
+		 if( empty( $lesson_id ) || empty( $question_id )
+			 || ! ( intval( $lesson_id  ) > 0 )
+			 || ! ( intval( $question_id  ) > 0 )
+			 || 'lesson' != get_post_type( $lesson_id )
+			 || 'question' != get_post_type( $question_id )) {
+
+			 return false;
+		 }
+
+		 $all_user_grades = self::get_user_grades( $lesson_id,$user_id );
+
+		 if( ! $all_user_grades || ! isset(  $all_user_grades[ $question_id ] ) ){
+
+			 //fallback to data pre 1.7.4
+			 $args = array(
+				 'post_id' => $question_id,
+				 'user_id' => $user_id,
+				 'type'    => 'sensei_user_answer'
+			 );
+
+			 $question_activity = Sensei_Utils::sensei_check_for_activity( $args , true );
+			 $fall_back_grade = false;
+			 if( isset( $question_activity->comment_ID ) ){
+				 $fall_back_grade = get_comment_meta(  $question_activity->comment_ID , 'user_grade', true );
+			 }
+
+			 return $fall_back_grade;
+
+		 } // end if $all_user_grades...
+
+		 return $all_user_grades[ $question_id ];
+
+	 }// end get_user_question_grade
+
+	 /**
+	  * Save the user's answers feedback
+	  *
+	  * For this function you must supply all three parameters. If will return false one is left out.
+	  * The data will be saved on the lesson ID supplied.
+	  *
+	  * @since 1.7.5
+	  * @access public
+	  *
+	  * @param array $answers_feedback{
+	  *  $type int $question_id
+	  *  $type string $question_feedback
+	  * }
+	  * @param int $lesson_id
+	  * @param int $user_id
+	  *
+	  * @return false or int $feedback_saved
+	  */
+	public function save_user_answers_feedback( $answers_feedback, $lesson_id , $user_id = 0 ){
+
+		// make sure the parameters are valid before continuing
+		if( empty( $lesson_id ) || empty( $user_id )
+			|| 'lesson' != get_post_type( $lesson_id )
+			||!get_userdata( $user_id )
+			|| !is_array( $answers_feedback ) ){
+
+			return false;
+
+		}
+
+
+		// check if the lesson is started before saving, if not start the lesson for the user
+		if ( !( 0 < intval( Sensei_Utils::user_started_lesson( $lesson_id, $user_id) ) ) ) {
+			Sensei_Utils::sensei_start_lesson( $lesson_id, $user_id );
+		}
 
-         /**
-          * User quiz has been submitted
-          *
-          * Fires the end of the submit_answers_for_grading function. It will fire irrespective of the submission
-          * results.
-          *
-          * @param int $user_id
-          * @param int $quiz_id
-          * @param string $grade
-          * @param string $quiz_pass_percentage
-          * @param string $quiz_grade_type
-          */
-         do_action( 'sensei_user_quiz_submitted', $user_id, $quiz_id, $grade, $quiz_pass_percentage, $quiz_grade_type );
+		// encode the feedback
+		$encoded_answers_feedback =  array();
+		foreach( $answers_feedback as $question_id => $feedback ){
+			$encoded_answers_feedback[ $question_id ] = base64_encode( $feedback );
+		}
+
+		// save the user data
+		$feedback_saved = Sensei_Utils::add_user_data( 'quiz_answers_feedback', $lesson_id , $encoded_answers_feedback, $user_id ) ;
+
+		//Were the the question feedback save correctly?
+		if( intval( $feedback_saved ) > 0){
+
+			// save transient to make retrieval faster in future
+			 $transient_key = 'sensei_answers_feedback_'.$user_id.'_'.$lesson_id;
+			 set_transient( $transient_key, $encoded_answers_feedback, 10 * DAY_IN_SECONDS );
 
-         return $answers_submitted;
+		}
+
+		return $feedback_saved;
+
+	} // end save_user_answers_feedback
+
+	 /**
+	  * Get the user's answers feedback.
+	  *
+	  * This function returns the feedback submitted by the teacher/admin
+	  * during grading. Grading occurs manually or automatically.
+	  *
+	  * @since 1.7.5
+	  * @access public
+	  *
+	  * @param int $lesson_id
+	  * @param int $user_id
+	  *
+	  * @return false | array $answers_feedback{
+	  *  $type int $question_id
+	  *  $type string $question_feedback
+	  * }
+	  */
+	 public function get_user_answers_feedback( $lesson_id , $user_id = 0 ){
+
+		 $answers_feedback = array();
 
-     }// end submit_answers_for_grading
+		 // get the user_id if none was passed in use the current logged in user
+		 if( ! intval( $user_id ) > 0 ) {
+			 $user_id = get_current_user_id();
+		 }
 
-     /**
-      * Get the user question answer
-      *
-      * This function gets the the users saved answer on given quiz for the given question parameter
-      * this function allows for a fallback to users still using the question saved data from before 1.7.4
-      *
-      * @since 1.7.4
-      *
-      * @param int  $lesson_id
-      * @param int $question_id
-      * @param int  $user_id ( optional )
-      *
-      * @return bool $answers_submitted
-      */
-     public function get_user_question_answer( $lesson_id, $question_id, $user_id = 0 ){
+		 if ( ! intval( $lesson_id ) > 0 || 'lesson' != get_post_type( $lesson_id )
+			 || ! intval( $user_id )  > 0 || !get_userdata( $user_id )  ) {
+			 return false;
+		 }
+
+		 // first check the transient to save a few split seconds
+		 $transient_key = 'sensei_answers_feedback_'.$user_id.'_'.$lesson_id;
+		 $encoded_feedback = get_transient( $transient_key );
+
+		 // get the data if nothing was stored in the transient
+		 if( empty( $encoded_feedback  ) || !$encoded_feedback ){
+
+			 $encoded_feedback = Sensei_Utils::get_user_data( 'quiz_answers_feedback', $lesson_id, $user_id );
+
+			 //set the transient with the new valid data for faster retrieval in future
+			 set_transient( $transient_key,  $encoded_feedback, 10 * DAY_IN_SECONDS);
+
+		 } // end if transient check
 
-         // parameter validation
-         if( empty( $lesson_id ) || empty( $question_id )
-             || ! ( intval( $lesson_id  ) > 0 )
-             || ! ( intval( $question_id  ) > 0 )
-             || 'lesson' != get_post_type( $lesson_id )
-             || 'question' != get_post_type( $question_id )) {
+		 // if there is no data for this user
+		 if( ! is_array( $encoded_feedback ) ){
+			 return false;
+		 }
 
-             return false;
-         }
-
-         if( ! ( intval( $user_id ) > 0 )   ){
-             $user_id = get_current_user_id();
-         }
-
-         $users_answers = $this->get_user_answers( $lesson_id, $user_id );
-
-         if( !$users_answers || empty( $users_answers )
-         ||  ! is_array( $users_answers ) || ! isset( $users_answers[ $question_id ] ) ){
-
-             //Fallback for pre 1.7.4 data
-             $comment =  Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $question_id, 'user_id' => $user_id, 'type' => 'sensei_user_answer' ), true );
-
-             if( ! isset( $comment->comment_content ) ){
-                 return false;
-             }
-
-             return maybe_unserialize( base64_decode( $comment->comment_content ) );
-         }
-
-         return $users_answers[ $question_id ];
-
-     }// end get_user_question_answer
-
-     /**
-      * Saving the users quiz question grades
-      *
-      * This function save all the grades for all the question in a given quiz on the lesson
-      * comment meta. It makes use of transients to save the grades for easier access at a later stage
-      *
-      * @since 1.7.4
-      *
-      * @param array $quiz_grades{
-      *      @type int $question_id
-      *      @type int $question_grade
-      * }
-      * @param $lesson_id
-      * @param $user_id (Optional) will use the current user if not supplied
-      *
-      * @return bool
-      */
-     public function set_user_grades( $quiz_grades, $lesson_id, $user_id = 0 ){
-
-         // get the user_id if none was passed in use the current logged in user
-         if( ! intval( $user_id ) > 0 ) {
-             $user_id = get_current_user_id();
-         }
-
-         // make sure the parameters are valid before continuing
-         if( empty( $lesson_id ) || empty( $user_id )
-             || 'lesson' != get_post_type( $lesson_id )
-             ||!get_userdata( $user_id )
-             || !is_array( $quiz_grades ) ){
-
-             return false;
-
-         }
-
-         $success = false;
-
-         // save that data for the user on the lesson comment meta
-         $comment_meta_id = Sensei_Utils::add_user_data( 'quiz_grades', $lesson_id, $quiz_grades, $user_id   );
-
-         // were the grades save successfully ?
-         if( intval( $comment_meta_id ) > 0 ) {
-
-             $success = true;
-             // save transient
-             $transient_key = 'quiz_grades_'. $user_id . '_' . $lesson_id;
-             set_transient( $transient_key, $quiz_grades, 10 * DAY_IN_SECONDS );
-         }
-
-         return $success;
-
-     }// end set_user_grades
-
-     /**
-      * Retrieve the users quiz question grades
-      *
-      * This function gets all the grades for all the questions in the given lesson quiz for a specific user.
-      *
-      * @since 1.7.4
-      *
-      * @param $lesson_id
-      * @param $user_id (Optional) will use the current user if not supplied
-      *
-      * @return array $user_quiz_grades or false if none exists for this users
-      */
-     public function get_user_grades( $lesson_id, $user_id = 0 ){
-
-         $user_grades = array();
-
-         // get the user_id if none was passed in use the current logged in user
-         if( ! intval( $user_id ) > 0 ) {
-             $user_id = get_current_user_id();
-         }
-
-         if ( ! intval( $lesson_id ) > 0 || 'lesson' != get_post_type( $lesson_id )
-             || ! intval( $user_id )  > 0 || !get_userdata( $user_id )  ) {
-             return false;
-         }
-
-         // save some time and get the transient cached data
-         $transient_key = 'quiz_grades_'. $user_id . '_' . $lesson_id;
-         $user_grades = get_transient( $transient_key );
-
-         // get the data if nothing was stored in the transient
-         if( empty( $user_grades  ) || false != $user_grades ){
-
-             $user_grades = Sensei_Utils::get_user_data( 'quiz_grades', $lesson_id, $user_id );
-
-             //set the transient with the new valid data for faster retrieval in future
-             set_transient( $transient_key,  $user_grades, 10 * DAY_IN_SECONDS );
-
-         } // end if transient check
-
-         // if there is no data for this user
-         if( ! is_array( $user_grades ) ){
-             return false;
-         }
-
-         return $user_grades;
-
-     }// end  get_user_grades
-
-     /**
-      * Get the user question grade
-      *
-      * This function gets the grade on a quiz for the given question parameter
-      * It does NOT do any grading. It simply retrieves the data that was stored during grading.
-      * this function allows for a fallback to users still using the question saved data from before 1.7.4
-      *
-      * @since 1.7.4
-      *
-      * @param int  $lesson_id
-      * @param int $question_id
-      * @param int  $user_id ( optional )
-      *
-      * @return bool $question_grade
-      */
-     public function get_user_question_grade( $lesson_id, $question_id, $user_id = 0 ){
-
-         // parameter validation
-         if( empty( $lesson_id ) || empty( $question_id )
-             || ! ( intval( $lesson_id  ) > 0 )
-             || ! ( intval( $question_id  ) > 0 )
-             || 'lesson' != get_post_type( $lesson_id )
-             || 'question' != get_post_type( $question_id )) {
-
-             return false;
-         }
-
-         $all_user_grades = self::get_user_grades( $lesson_id,$user_id );
-
-         if( ! $all_user_grades || ! isset(  $all_user_grades[ $question_id ] ) ){
-
-             //fallback to data pre 1.7.4
-             $args = array(
-                 'post_id' => $question_id,
-                 'user_id' => $user_id,
-                 'type'    => 'sensei_user_answer'
-             );
-
-             $question_activity = Sensei_Utils::sensei_check_for_activity( $args , true );
-             $fall_back_grade = false;
-             if( isset( $question_activity->comment_ID ) ){
-                 $fall_back_grade = get_comment_meta(  $question_activity->comment_ID , 'user_grade', true );
-             }
-
-             return $fall_back_grade;
-
-         } // end if $all_user_grades...
-
-         return $all_user_grades[ $question_id ];
-
-     }// end get_user_question_grade
-
-     /**
-      * Save the user's answers feedback
-      *
-      * For this function you must supply all three parameters. If will return false one is left out.
-      * The data will be saved on the lesson ID supplied.
-      *
-      * @since 1.7.5
-      * @access public
-      *
-      * @param array $answers_feedback{
-      *  $type int $question_id
-      *  $type string $question_feedback
-      * }
-      * @param int $lesson_id
-      * @param int $user_id
-      *
-      * @return false or int $feedback_saved
-      */
-    public function save_user_answers_feedback( $answers_feedback, $lesson_id , $user_id = 0 ){
-
-        // make sure the parameters are valid before continuing
-        if( empty( $lesson_id ) || empty( $user_id )
-            || 'lesson' != get_post_type( $lesson_id )
-            ||!get_userdata( $user_id )
-            || !is_array( $answers_feedback ) ){
-
-            return false;
-
-        }
-
-
-        // check if the lesson is started before saving, if not start the lesson for the user
-        if ( !( 0 < intval( Sensei_Utils::user_started_lesson( $lesson_id, $user_id) ) ) ) {
-            Sensei_Utils::sensei_start_lesson( $lesson_id, $user_id );
-        }
-
-        // encode the feedback
-        $encoded_answers_feedback =  array();
-        foreach( $answers_feedback as $question_id => $feedback ){
-            $encoded_answers_feedback[ $question_id ] = base64_encode( $feedback );
-        }
-
-        // save the user data
-        $feedback_saved = Sensei_Utils::add_user_data( 'quiz_answers_feedback', $lesson_id , $encoded_answers_feedback, $user_id ) ;
-
-        //Were the the question feedback save correctly?
-        if( intval( $feedback_saved ) > 0){
-
-            // save transient to make retrieval faster in future
-             $transient_key = 'sensei_answers_feedback_'.$user_id.'_'.$lesson_id;
-             set_transient( $transient_key, $encoded_answers_feedback, 10 * DAY_IN_SECONDS );
-
-        }
-
-        return $feedback_saved;
-
-    } // end save_user_answers_feedback
-
-     /**
-      * Get the user's answers feedback.
-      *
-      * This function returns the feedback submitted by the teacher/admin
-      * during grading. Grading occurs manually or automatically.
-      *
-      * @since 1.7.5
-      * @access public
-      *
-      * @param int $lesson_id
-      * @param int $user_id
-      *
-      * @return false | array $answers_feedback{
-      *  $type int $question_id
-      *  $type string $question_feedback
-      * }
-      */
-     public function get_user_answers_feedback( $lesson_id , $user_id = 0 ){
-
-         $answers_feedback = array();
-
-         // get the user_id if none was passed in use the current logged in user
-         if( ! intval( $user_id ) > 0 ) {
-             $user_id = get_current_user_id();
-         }
-
-         if ( ! intval( $lesson_id ) > 0 || 'lesson' != get_post_type( $lesson_id )
-             || ! intval( $user_id )  > 0 || !get_userdata( $user_id )  ) {
-             return false;
-         }
-
-         // first check the transient to save a few split seconds
-         $transient_key = 'sensei_answers_feedback_'.$user_id.'_'.$lesson_id;
-         $encoded_feedback = get_transient( $transient_key );
-
-         // get the data if nothing was stored in the transient
-         if( empty( $encoded_feedback  ) || !$encoded_feedback ){
-
-             $encoded_feedback = Sensei_Utils::get_user_data( 'quiz_answers_feedback', $lesson_id, $user_id );
-
-             //set the transient with the new valid data for faster retrieval in future
-             set_transient( $transient_key,  $encoded_feedback, 10 * DAY_IN_SECONDS);
-
-         } // end if transient check
-
-         // if there is no data for this user
-         if( ! is_array( $encoded_feedback ) ){
-             return false;
-         }
-
-         foreach( $encoded_feedback as $question_id => $feedback ){
+		 foreach( $encoded_feedback as $question_id => $feedback ){
 
-             $answers_feedback[ $question_id ] = base64_decode( $feedback );
+			 $answers_feedback[ $question_id ] = base64_decode( $feedback );
 
-         }
+		 }
 
-         return $answers_feedback;
+		 return $answers_feedback;
 
-     } // end get_user_answers_feedback
+	 } // end get_user_answers_feedback
 
-     /**
-      * Get the user's answer feedback for a specific question.
-      *
-      * This function gives you a single answer note/feedback string
-      * for the user on the given question.
-      *
-      * @since 1.7.5
-      * @access public
-      *
-      * @param int $lesson_id
-      * @param int $question_id
-      * @param int $user_id
-      *
-      * @return string $feedback or bool if false
-      */
-     public function get_user_question_feedback( $lesson_id, $question_id, $user_id = 0 ){
+	 /**
+	  * Get the user's answer feedback for a specific question.
+	  *
+	  * This function gives you a single answer note/feedback string
+	  * for the user on the given question.
+	  *
+	  * @since 1.7.5
+	  * @access public
+	  *
+	  * @param int $lesson_id
+	  * @param int $question_id
+	  * @param int $user_id
+	  *
+	  * @return string $feedback or bool if false
+	  */
+	 public function get_user_question_feedback( $lesson_id, $question_id, $user_id = 0 ){
 
-         $feedback = false;
+		 $feedback = false;
 
-         // parameter validation
-         if( empty( $lesson_id ) || empty( $question_id )
-             || ! ( intval( $lesson_id  ) > 0 )
-             || ! ( intval( $question_id  ) > 0 )
-             || 'lesson' != get_post_type( $lesson_id )
-             || 'question' != get_post_type( $question_id )) {
+		 // parameter validation
+		 if( empty( $lesson_id ) || empty( $question_id )
+			 || ! ( intval( $lesson_id  ) > 0 )
+			 || ! ( intval( $question_id  ) > 0 )
+			 || 'lesson' != get_post_type( $lesson_id )
+			 || 'question' != get_post_type( $question_id )) {
 
-             return false;
-         }
+			 return false;
+		 }
 
-         // get all the feedback for the user on the given lesson
-         $all_feedback = $this->get_user_answers_feedback( $lesson_id, $user_id );
+		 // get all the feedback for the user on the given lesson
+		 $all_feedback = $this->get_user_answers_feedback( $lesson_id, $user_id );
 
-         if( !$all_feedback || empty( $all_feedback )
-             || ! is_array( $all_feedback ) || ! isset( $all_feedback[ $question_id ] ) ){
+		 if( !$all_feedback || empty( $all_feedback )
+			 || ! is_array( $all_feedback ) || ! isset( $all_feedback[ $question_id ] ) ){
 
-             //fallback to data pre 1.7.4
+			 //fallback to data pre 1.7.4
 
-             // setup the sensei data query
-             $args = array(
-                 'post_id' => $question_id,
-                 'user_id' => $user_id,
-                 'type'    => 'sensei_user_answer'
-             );
-             $question_activity = Sensei_Utils::sensei_check_for_activity( $args , true );
+			 // setup the sensei data query
+			 $args = array(
+				 'post_id' => $question_id,
+				 'user_id' => $user_id,
+				 'type'    => 'sensei_user_answer'
+			 );
+			 $question_activity = Sensei_Utils::sensei_check_for_activity( $args , true );
 
-             // set the default to false and return that if no old data is available.
-             if( isset( $question_activity->comment_ID ) ){
-                 $feedback = base64_decode( get_comment_meta(  $question_activity->comment_ID , 'answer_note', true ) );
-             }
+			 // set the default to false and return that if no old data is available.
+			 if( isset( $question_activity->comment_ID ) ){
+				 $feedback = base64_decode( get_comment_meta(  $question_activity->comment_ID , 'answer_note', true ) );
+			 }
 
-             // finally use the default question feedback
-             if( empty( $feedback ) ){
-                 $feedback = get_post_meta( $question_id, '_answer_feedback', true );
-             }
+			 // finally use the default question feedback
+			 if( empty( $feedback ) ){
+				 $feedback = get_post_meta( $question_id, '_answer_feedback', true );
+			 }
 
-             return $feedback;
+			 return $feedback;
 
-         }
+		 }
 
-         return $all_feedback[ $question_id ];
+		 return $all_feedback[ $question_id ];
 
-     } // end get_user_question_feedback
+	 } // end get_user_question_feedback
 
-     /**
-      * Check if a quiz has no questions, and redirect back to lesson.
-      *
-      * Though a quiz is created for each lesson, it should not be visible
-      * unless it has questions.
-      *
-      * @since 1.9.0
-      * @access public
-      * @param none
-      * @return void
-      */
+	 /**
+	  * Check if a quiz has no questions, and redirect back to lesson.
+	  *
+	  * Though a quiz is created for each lesson, it should not be visible
+	  * unless it has questions.
+	  *
+	  * @since 1.9.0
+	  * @access public
+	  * @param none
+	  * @return void
+	  */
 
-     public function quiz_has_no_questions() {
+	 public function quiz_has_no_questions() {
 
-         if( ! is_singular( 'quiz' ) )  {
-             return;
-         }
+		 if( ! is_singular( 'quiz' ) )  {
+			 return;
+		 }
 
-         global $post;
+		 global $post;
 
-         $lesson_id = $this->get_lesson_id($post->ID);
+		 $lesson_id = $this->get_lesson_id($post->ID);
 
-         $has_questions = get_post_meta( $lesson_id, '_quiz_has_questions', true );
+		 $has_questions = get_post_meta( $lesson_id, '_quiz_has_questions', true );
 
-         $lesson = get_post($lesson_id);
+		 $lesson = get_post($lesson_id);
 
-         if ( is_singular('quiz') && ! $has_questions && $_SERVER['REQUEST_URI'] != "/lesson/$lesson->post_name" ) {
+		 if ( is_singular('quiz') && ! $has_questions && $_SERVER['REQUEST_URI'] != "/lesson/$lesson->post_name" ) {
 
-             wp_redirect(get_permalink($lesson->ID), 301);
-             exit;
+			 wp_redirect(get_permalink($lesson->ID), 301);
+			 exit;
 
-         }
+		 }
 
-     } // end quiz_has_no_questions
+	 } // end quiz_has_no_questions
 
 /**
-  * Deprecate the sensei_single_main_content on the single-quiz template.
-  *
-  * @deprecated since 1.9.0
-  */
+ * Deprecate the sensei_single_main_content on the single-quiz template.
+ *
+ * @deprecated since 1.9.0
+ */
  public static function deprecate_quiz_sensei_single_main_content_hook(){
 
-     sensei_do_deprecated_action('sensei_single_main_content', '1.9.0', 'sensei_single_quiz_content_inside_before or sensei_single_quiz_content_inside_after');
+	 sensei_do_deprecated_action('sensei_single_main_content', '1.9.0', 'sensei_single_quiz_content_inside_before or sensei_single_quiz_content_inside_after');
 
  }
-    /*
+	/*
      * Deprecate the sensei_quiz_single_title on the single-quiz template.
      *
      * @deprecated since 1.9.0
      */
-     public static function deprecate_quiz_sensei_quiz_single_title_hook(){
+	 public static function deprecate_quiz_sensei_quiz_single_title_hook(){
 
-         sensei_do_deprecated_action('sensei_quiz_single_title', '1.9.0', 'sensei_single_quiz_content_inside_before ');
+		 sensei_do_deprecated_action('sensei_quiz_single_title', '1.9.0', 'sensei_single_quiz_content_inside_before ');
 
-     }
+	 }
 
-     /**
-      * Filter the single title and add the Quiz to it.
-      *
-      * @param string $title
-      * @param int $id title post id
-      * @return string $quiz_title
-      */
-     public static function single_quiz_title( $title, $post_id ){
+	 /**
+	  * Filter the single title and add the Quiz to it.
+	  *
+	  * @param string $title
+	  * @param int $id title post id
+	  * @return string $quiz_title
+	  */
+	 public static function single_quiz_title( $title, $post_id ){
 
-         if( 'quiz' == get_post_type( $post_id ) ){
+		 if( 'quiz' == get_post_type( $post_id ) ){
 
-             $title_with_no_quizzes = $title;
+			 $title_with_no_quizzes = $title;
 
-             // if the title has quiz, remove it: legacy titles have the word quiz stored.
-             if( 1 < substr_count( strtoupper( $title_with_no_quizzes ), 'QUIZ' ) ){
+			 // if the title has quiz, remove it: legacy titles have the word quiz stored.
+			 if( 1 < substr_count( strtoupper( $title_with_no_quizzes ), 'QUIZ' ) ){
 
-                 // remove all possible appearances of quiz
-                 $title_with_no_quizzes = str_replace( 'quiz', '', $title  );
-                 $title_with_no_quizzes = str_replace( 'Quiz', '', $title_with_no_quizzes  );
-                 $title_with_no_quizzes = str_replace( 'QUIZ', '', $title_with_no_quizzes  );
+				 // remove all possible appearances of quiz
+				 $title_with_no_quizzes = str_replace( 'quiz', '', $title  );
+				 $title_with_no_quizzes = str_replace( 'Quiz', '', $title_with_no_quizzes  );
+				 $title_with_no_quizzes = str_replace( 'QUIZ', '', $title_with_no_quizzes  );
 
-             }
+			 }
 
-             $title = $title_with_no_quizzes .  ' ' . __( 'Quiz', 'woothemes-sensei' );
-         }
+			 $title = $title_with_no_quizzes .  ' ' . __( 'Quiz', 'woothemes-sensei' );
+		 }
 
-         /**
-          * hook document in class-woothemes-sensei-message.php
-          */
-         return apply_filters( 'sensei_single_title', $title, get_post_type( ) );
+		 /**
+		  * hook document in class-woothemes-sensei-message.php
+		  */
+		 return apply_filters( 'sensei_single_title', $title, get_post_type( ) );
 
-     }
+	 }
 
-     /**
-      * Initialize the quiz question loop on the single quiz template
-      *
-      * The function will create a global quiz loop varialbe.
-      *
-      * @since 1.9.0
-      *
-      */
-     public static function start_quiz_questions_loop(){
+	 /**
+	  * Initialize the quiz question loop on the single quiz template
+	  *
+	  * The function will create a global quiz loop varialbe.
+	  *
+	  * @since 1.9.0
+	  *
+	  */
+	 public static function start_quiz_questions_loop(){
 
-         global $sensei_question_loop;
+		 global $sensei_question_loop;
 
-         //intialize the questions loop object
-         $sensei_question_loop['current'] = -1;
-         $sensei_question_loop['total']   =  0;
-         $sensei_question_loop['questions'] = array();
+		 //intialize the questions loop object
+		 $sensei_question_loop['current'] = -1;
+		 $sensei_question_loop['total']   =  0;
+		 $sensei_question_loop['questions'] = array();
 
 
-         $questions = Sensei()->lesson->lesson_quiz_questions( get_the_ID() );
+		 $questions = Sensei()->lesson->lesson_quiz_questions( get_the_ID() );
 
-         if( count( $questions  ) > 0  ){
+		 if( count( $questions  ) > 0  ){
 
-             $sensei_question_loop['total']   =  count( $questions );
-             $sensei_question_loop['questions'] = $questions;
-             $sensei_question_loop['quiz_id'] = get_the_ID();
+			 $sensei_question_loop['total']   =  count( $questions );
+			 $sensei_question_loop['questions'] = $questions;
+			 $sensei_question_loop['quiz_id'] = get_the_ID();
 
-         }
+		 }
 
-     }// static function
+	 }// static function
 
-     /**
-      * Initialize the quiz question loop on the single quiz template
-      *
-      * The function will create a global quiz loop varialbe.
-      *
-      * @since 1.9.0
-      *
-      */
-     public static function stop_quiz_questions_loop(){
+	 /**
+	  * Initialize the quiz question loop on the single quiz template
+	  *
+	  * The function will create a global quiz loop varialbe.
+	  *
+	  * @since 1.9.0
+	  *
+	  */
+	 public static function stop_quiz_questions_loop(){
 
-         $sensei_question_loop['total']   =  0;
-         $sensei_question_loop['questions'] = array();
-         $sensei_question_loop['quiz_id'] = '';
+		 $sensei_question_loop['total']   =  0;
+		 $sensei_question_loop['questions'] = array();
+		 $sensei_question_loop['quiz_id'] = '';
 
-     }
+	 }
 
-     /**
-      * Output the title for the single quiz page
-      *
-      * @since 1.9.0
-      */
-     public static function the_title(){
-         ?>
+	 /**
+	  * Output the title for the single quiz page
+	  *
+	  * @since 1.9.0
+	  */
+	 public static function the_title(){
+		 ?>
          <header>
 
              <h1>
 
                  <?php
-                 /**
-                  * Filter documented in class-sensei-messages.php the_title
-                  */
-                 echo apply_filters( 'sensei_single_title', get_the_title( get_post() ), get_post_type( get_the_ID() ) );
-                 ?>
+				 /**
+				  * Filter documented in class-sensei-messages.php the_title
+				  */
+				 echo apply_filters( 'sensei_single_title', get_the_title( get_post() ), get_post_type( get_the_ID() ) );
+				 ?>
 
              </h1>
 
          </header>
 
          <?php
-     }//the_title
+	 }//the_title
 
-     /**
-      * Output the sensei quiz status message.
-      *
-      * @param $quiz_id
-      */
-    public static function  the_user_status_message( $quiz_id ){
+	 /**
+	  * Output the sensei quiz status message.
+	  *
+	  * @param $quiz_id
+	  */
+	public static function  the_user_status_message( $quiz_id ){
 
-        $lesson_id =  Sensei()->quiz->get_lesson_id( $quiz_id );
-        $status = Sensei_Utils::sensei_user_quiz_status_message( $lesson_id , get_current_user_id() );
-        echo '<div class="sensei-message ' . $status['box_class'] . '">' . $status['message'] . '</div>';
+		$lesson_id =  Sensei()->quiz->get_lesson_id( $quiz_id );
+		$status = Sensei_Utils::sensei_user_quiz_status_message( $lesson_id , get_current_user_id() );
+		echo '<div class="sensei-message ' . $status['box_class'] . '">' . $status['message'] . '</div>';
 
-    }
+	}
 
-     /**
-      * This functions runs the old sensei_quiz_action_buttons action
-      * for backwards compatiblity sake.
-      *
-      * @since 1.9.0
-      * @deprecated
-      */
-     public static function deprecate_sensei_quiz_action_buttons_hook(){
+	 /**
+	  * This functions runs the old sensei_quiz_action_buttons action
+	  * for backwards compatiblity sake.
+	  *
+	  * @since 1.9.0
+	  * @deprecated
+	  */
+	 public static function deprecate_sensei_quiz_action_buttons_hook(){
 
-         sensei_do_deprecated_action( 'sensei_quiz_action_buttons', '1.9.0', 'sensei_single_quiz_questions_after');
+		 sensei_do_deprecated_action( 'sensei_quiz_action_buttons', '1.9.0', 'sensei_single_quiz_questions_after');
 
-     }
+	 }
 
-     /**
-      * The quiz action buttons needed to ouput quiz
-      * action such as reset complete and save.
-      *
-      * @since 1.3.0
-      */
-     public static function action_buttons() {
+	 /**
+	  * The quiz action buttons needed to ouput quiz
+	  * action such as reset complete and save.
+	  *
+	  * @since 1.3.0
+	  */
+	 public static function action_buttons() {
 
-         global $post, $current_user;
+		 global $post, $current_user;
 
-         $lesson_id = (int) get_post_meta( $post->ID, '_quiz_lesson', true );
-         $lesson_course_id = (int) get_post_meta( $lesson_id, '_lesson_course', true );
-         $lesson_prerequisite = (int) get_post_meta( $lesson_id, '_lesson_prerequisite', true );
-         $show_actions = true;
-         $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $current_user->ID );
+		 $lesson_id = (int) get_post_meta( $post->ID, '_quiz_lesson', true );
+		 $lesson_course_id = (int) get_post_meta( $lesson_id, '_lesson_course', true );
+		 $lesson_prerequisite = (int) get_post_meta( $lesson_id, '_lesson_prerequisite', true );
+		 $show_actions = true;
+		 $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $current_user->ID );
 
-         //setup quiz grade
-         $user_quiz_grade = '';
-         if( ! empty( $user_lesson_status  ) ){
-             $user_quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
-         }
+		 //setup quiz grade
+		 $user_quiz_grade = '';
+		 if( ! empty( $user_lesson_status  ) ){
+			 $user_quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
+		 }
 
 
-         if( intval( $lesson_prerequisite ) > 0 ) {
+		 if( intval( $lesson_prerequisite ) > 0 ) {
 
-             // If the user hasn't completed the prereq then hide the current actions
-             $show_actions = Sensei_Utils::user_completed_lesson( $lesson_prerequisite, $current_user->ID );
+			 // If the user hasn't completed the prereq then hide the current actions
+			 $show_actions = Sensei_Utils::user_completed_lesson( $lesson_prerequisite, $current_user->ID );
 
-         }
-         if ( $show_actions && is_user_logged_in() && Sensei_Utils::user_started_course( $lesson_course_id, $current_user->ID ) ) {
+		 }
+		 if ( $show_actions && is_user_logged_in() && Sensei_Utils::user_started_course( $lesson_course_id, $current_user->ID ) ) {
 
-             // Get Reset Settings
-             $reset_quiz_allowed = get_post_meta( $post->ID, '_enable_quiz_reset', true ); ?>
+			 // Get Reset Settings
+			 $reset_quiz_allowed = get_post_meta( $post->ID, '_enable_quiz_reset', true ); ?>
 
              <!-- Action Nonce's -->
              <input type="hidden" name="woothemes_sensei_complete_quiz_nonce" id="woothemes_sensei_complete_quiz_nonce"
@@ -1273,55 +1273,55 @@ 
 
          <?php }
 
-     } // End sensei_quiz_action_buttons()
-
-     /**
-      * Fetch the quiz grade
-      *
-      * @since 1.9.0
-      *
-      * @param int $lesson_id
-      * @param int $user_id
-      *
-      * @return double $user_quiz_grade
-      */
-     public static function get_user_quiz_grade( $lesson_id, $user_id ){
-
-         // get the quiz grade
-         $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
-         $user_quiz_grade = 0;
-         if( isset( $user_lesson_status->comment_ID ) ) {
-             $user_quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
-         }
-
-         return (double) $user_quiz_grade;
-
-     }
-
-     /**
-      * Check the quiz reset property for a given lesson's quiz.
-      *
-      * The data is stored on the quiz but going forward the quiz post
-      * type will be retired, hence the lesson_id is a require parameter.
-      *
-      * @since 1.9.0
-      *
-      * @param int $lesson_id
-      * @return bool
-      */
-     public static function is_reset_allowed( $lesson_id ){
-
-         $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
-
-         $reset_allowed = get_post_meta( $quiz_id, '_enable_quiz_reset', true );
-         //backwards compatibility
-         if( 'on' == $reset_allowed ) {
-             $reset_allowed = 1;
-         }
-
-         return (bool) $reset_allowed;
-
-     }
+	 } // End sensei_quiz_action_buttons()
+
+	 /**
+	  * Fetch the quiz grade
+	  *
+	  * @since 1.9.0
+	  *
+	  * @param int $lesson_id
+	  * @param int $user_id
+	  *
+	  * @return double $user_quiz_grade
+	  */
+	 public static function get_user_quiz_grade( $lesson_id, $user_id ){
+
+		 // get the quiz grade
+		 $user_lesson_status = Sensei_Utils::user_lesson_status( $lesson_id, $user_id );
+		 $user_quiz_grade = 0;
+		 if( isset( $user_lesson_status->comment_ID ) ) {
+			 $user_quiz_grade = get_comment_meta( $user_lesson_status->comment_ID, 'grade', true );
+		 }
+
+		 return (double) $user_quiz_grade;
+
+	 }
+
+	 /**
+	  * Check the quiz reset property for a given lesson's quiz.
+	  *
+	  * The data is stored on the quiz but going forward the quiz post
+	  * type will be retired, hence the lesson_id is a require parameter.
+	  *
+	  * @since 1.9.0
+	  *
+	  * @param int $lesson_id
+	  * @return bool
+	  */
+	 public static function is_reset_allowed( $lesson_id ){
+
+		 $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
+
+		 $reset_allowed = get_post_meta( $quiz_id, '_enable_quiz_reset', true );
+		 //backwards compatibility
+		 if( 'on' == $reset_allowed ) {
+			 $reset_allowed = 1;
+		 }
+
+		 return (bool) $reset_allowed;
+
+	 }
 
  } // End Class WooThemes_Sensei_Quiz
 

includes/class-sensei-learners-main.php

Indentation +24 added lines, -24 removed lines

@@ -205,13 +205,13 @@ 
 
 	/**
 	 * Generates content for a single row of the table in the user management
-     * screen.
-     *
+	 * screen.
+	 *
 	 * @since  1.7.0
-     *
+	 *
 	 * @param object $item The current item
-     *
-     * @return void
+	 *
+	 * @return void
 	 */
 	protected function get_row_data( $item ) {
 		global $wp_version;
@@ -219,8 +219,8 @@ 
 		switch ( $this->view ) {
 			case 'learners' :
 
-                // in this case the item passed in is actually the users activity on course of lesson
-                $user_activity = $item;
+				// in this case the item passed in is actually the users activity on course of lesson
+				$user_activity = $item;
 				$post_id = false;
 
 				if( $this->lesson_id ) {
@@ -239,30 +239,30 @@ 
 
 				if( 'complete' == $user_activity->comment_approved || 'graded' == $user_activity->comment_approved || 'passed' == $user_activity->comment_approved ) {
 
-                    $status_html = '<span class="graded">' .__( 'Completed', 'woothemes-sensei' ) . '</span>';
+					$status_html = '<span class="graded">' .__( 'Completed', 'woothemes-sensei' ) . '</span>';
 
 				} else {
 
-                    $status_html = '<span class="in-progress">' . __( 'In Progress', 'woothemes-sensei' ) . '</span>';
+					$status_html = '<span class="in-progress">' . __( 'In Progress', 'woothemes-sensei' ) . '</span>';
 
 				}
 
-                $title = Sensei_Learner::get_full_name( $user_activity->user_id );
+				$title = Sensei_Learner::get_full_name( $user_activity->user_id );
 				$a_title = sprintf( __( 'Edit &#8220;%s&#8221;' ), $title );
 
-                /**
-                 * sensei_learners_main_column_data filter
-                 *
-                 * This filter runs on the learner management screen for a specific course.
-                 * It provides the learner row column details.
-                 *
-                 * @param array $columns{
-                 *   type string $title
-                 *   type string $date_started
-                 *   type string $course_status (completed, started etc)
-                 *   type html $action_buttons
-                 * }
-                 */
+				/**
+				 * sensei_learners_main_column_data filter
+				 *
+				 * This filter runs on the learner management screen for a specific course.
+				 * It provides the learner row column details.
+				 *
+				 * @param array $columns{
+				 *   type string $title
+				 *   type string $date_started
+				 *   type string $course_status (completed, started etc)
+				 *   type html $action_buttons
+				 * }
+				 */
 				$column_data = apply_filters( 'sensei_learners_main_column_data', array(
 						'title' => '<strong><a class="row-title" href="' . admin_url( 'user-edit.php?user_id=' . $user_activity->user_id ) . '" title="' . esc_attr( $a_title ) . '">' . $title . '</a></strong>',
 						'date_started' => get_comment_meta( $user_activity->comment_ID, 'start', true),
@@ -292,7 +292,7 @@ 
 
 			case 'courses' :
 			default:
-                $course_learners = Sensei_Utils::sensei_check_for_activity( apply_filters( 'sensei_learners_course_learners', array( 'post_id' => $item->ID, 'type' => 'sensei_course_status', 'status' => 'any' ) ) );
+				$course_learners = Sensei_Utils::sensei_check_for_activity( apply_filters( 'sensei_learners_course_learners', array( 'post_id' => $item->ID, 'type' => 'sensei_course_status', 'status' => 'any' ) ) );
 				$title = get_the_title( $item );
 				$a_title = sprintf( __( 'Edit &#8220;%s&#8221;' ), $title );
 

includes/class-sensei-grading.php

Indentation +327 added lines, -327 removed lines

@@ -20,8 +20,8 @@ 
 	/**
 	 * Constructor
 	 * @since  1.3.0
-     *
-     * @param $file
+	 *
+	 * @param $file
 	 */
 	public function __construct ( $file ) {
 		$this->name = __( 'Grading', 'woothemes-sensei' );
@@ -230,7 +230,7 @@ 
 	/**
 	 * Outputs Grading general headers
 	 * @since  1.3.0
-     * @param array $args
+	 * @param array $args
 	 * @return void
 	 */
 	public function grading_headers( $args = array( 'nav' => 'default' ) ) {
@@ -282,7 +282,7 @@ 
 		}
 		if ( isset( $_GET['user_id'] ) && 0 < intval( $_GET['user_id'] ) ) {
 
-            $user_name = Sensei_Learner::get_full_name( $_GET['user_id'] );
+			$user_name = Sensei_Learner::get_full_name( $_GET['user_id'] );
 			$title .= '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;' . $user_name . '</span>';
 
 		} // End If Statement
@@ -316,7 +316,7 @@ 
 		}
 		if ( isset( $_GET['user'] ) && 0 < intval( $_GET['user'] ) ) {
 
-            $user_name = Sensei_Learner::get_full_name( $_GET['user'] );
+			$user_name = Sensei_Learner::get_full_name( $_GET['user'] );
 			$title .= '&nbsp;&nbsp;<span class="user-title">&gt;&nbsp;&nbsp;' . $user_name . '</span>';
 
 		} // End If Statement
@@ -365,15 +365,15 @@ 
 	public function count_statuses( $args = array() ) {
 		global  $wpdb;
 
-        /**
-         * Filter fires inside Sensei_Grading::count_statuses
-         *
-         * Alter the the post_in array to determine which posts the
-         * comment query should be limited to.
-         * @since 1.8.0
-         * @param array $args
-         */
-        $args = apply_filters( 'sensei_count_statuses_args', $args );
+		/**
+		 * Filter fires inside Sensei_Grading::count_statuses
+		 *
+		 * Alter the the post_in array to determine which posts the
+		 * comment query should be limited to.
+		 * @since 1.8.0
+		 * @param array $args
+		 */
+		$args = apply_filters( 'sensei_count_statuses_args', $args );
 
 		if ( 'course' == $args['type'] ) {
 			$type = 'sensei_course_status';
@@ -385,7 +385,7 @@ 
 
 		$query = "SELECT comment_approved, COUNT( * ) AS total FROM {$wpdb->comments} WHERE comment_type = %s ";
 
-        // Restrict to specific posts
+		// Restrict to specific posts
 		if ( isset( $args['post__in'] ) && !empty( $args['post__in'] ) && is_array( $args['post__in'] ) ) {
 			$query .= ' AND comment_post_ID IN (' . implode( ',', array_map( 'absint', $args['post__in'] ) ) . ')';
 		}
@@ -520,148 +520,148 @@ 
 		return $html;
 	} // End lessons_drop_down_html()
 
-    /**
-     * The process grading function handles admin grading submissions.
-     *
-     * This function is hooked on to admin_init. It simply accepts
-     * the grades as the Grader selected theme and saves the total grade and
-     * individual question grades.
-     *
-     * @return bool
-     */
-    public function admin_process_grading_submission() {
+	/**
+	 * The process grading function handles admin grading submissions.
+	 *
+	 * This function is hooked on to admin_init. It simply accepts
+	 * the grades as the Grader selected theme and saves the total grade and
+	 * individual question grades.
+	 *
+	 * @return bool
+	 */
+	public function admin_process_grading_submission() {
 
-        // NEEDS REFACTOR/OPTIMISING, such as combining the various meta data stored against the sensei_user_answer entry
-        if( ! isset( $_POST['sensei_manual_grade'] )
-            || ! wp_verify_nonce( $_POST['_wp_sensei_manual_grading_nonce'], 'sensei_manual_grading' )
-            || ! isset( $_GET['quiz_id'] )
-            || $_GET['quiz_id'] != $_POST['sensei_manual_grade'] ) {
+		// NEEDS REFACTOR/OPTIMISING, such as combining the various meta data stored against the sensei_user_answer entry
+		if( ! isset( $_POST['sensei_manual_grade'] )
+			|| ! wp_verify_nonce( $_POST['_wp_sensei_manual_grading_nonce'], 'sensei_manual_grading' )
+			|| ! isset( $_GET['quiz_id'] )
+			|| $_GET['quiz_id'] != $_POST['sensei_manual_grade'] ) {
 
-            return false; //exit and do not grade
+			return false; //exit and do not grade
 
-        }
+		}
 
-        $quiz_id = $_GET['quiz_id'];
-        $user_id = $_GET['user'];
+		$quiz_id = $_GET['quiz_id'];
+		$user_id = $_GET['user'];
 
 
-        $questions = Sensei_Utils::sensei_get_quiz_questions( $quiz_id );
-        $quiz_lesson_id =  Sensei()->quiz->get_lesson_id( $quiz_id );
-        $quiz_grade = 0;
-        $count = 0;
-        $quiz_grade_total = $_POST['quiz_grade_total'];
-        $all_question_grades = array();
-        $all_answers_feedback = array();
+		$questions = Sensei_Utils::sensei_get_quiz_questions( $quiz_id );
+		$quiz_lesson_id =  Sensei()->quiz->get_lesson_id( $quiz_id );
+		$quiz_grade = 0;
+		$count = 0;
+		$quiz_grade_total = $_POST['quiz_grade_total'];
+		$all_question_grades = array();
+		$all_answers_feedback = array();
 
-        foreach( $questions as $question ) {
+		foreach( $questions as $question ) {
 
-            ++$count;
-            $question_id = $question->ID;
+			++$count;
+			$question_id = $question->ID;
 
-            if( isset( $_POST[ 'question_' . $question_id ] ) ) {
+			if( isset( $_POST[ 'question_' . $question_id ] ) ) {
 
-                $question_grade = 0;
-                if( $_POST[ 'question_' . $question_id ] == 'right' ) {
+				$question_grade = 0;
+				if( $_POST[ 'question_' . $question_id ] == 'right' ) {
 
-                    $question_grade = $_POST[ 'question_' . $question_id . '_grade' ];
+					$question_grade = $_POST[ 'question_' . $question_id . '_grade' ];
 
-                }
+				}
 
-                // add data to the array that will, after the loop, be stored on the lesson status
-                $all_question_grades[ $question_id ] = $question_grade;
+				// add data to the array that will, after the loop, be stored on the lesson status
+				$all_question_grades[ $question_id ] = $question_grade;
 
-                // tally up the total quiz grade
-                $quiz_grade += $question_grade;
+				// tally up the total quiz grade
+				$quiz_grade += $question_grade;
 
-            } // endif
+			} // endif
 
-            // Question answer feedback / notes
-            $question_feedback = '';
-            if( isset( $_POST[ 'questions_feedback' ][ $question_id ] ) ){
+			// Question answer feedback / notes
+			$question_feedback = '';
+			if( isset( $_POST[ 'questions_feedback' ][ $question_id ] ) ){
 
-                $question_feedback = wp_unslash( $_POST[ 'questions_feedback' ][ $question_id ] );
+				$question_feedback = wp_unslash( $_POST[ 'questions_feedback' ][ $question_id ] );
 
-            }
-            $all_answers_feedback[ $question_id ] = $question_feedback;
+			}
+			$all_answers_feedback[ $question_id ] = $question_feedback;
 
-        } // end for each $questions
+		} // end for each $questions
 
-        //store all question grades on the lesson status
-        Sensei()->quiz->set_user_grades( $all_question_grades, $quiz_lesson_id , $user_id );
+		//store all question grades on the lesson status
+		Sensei()->quiz->set_user_grades( $all_question_grades, $quiz_lesson_id , $user_id );
 
-        //store the feedback from grading
-        Sensei()->quiz->save_user_answers_feedback( $all_answers_feedback, $quiz_lesson_id , $user_id );
+		//store the feedback from grading
+		Sensei()->quiz->save_user_answers_feedback( $all_answers_feedback, $quiz_lesson_id , $user_id );
 
-        // $_POST['all_questions_graded'] is set when all questions have been graded
-        // in the class sensei grading user quiz -> display()
-        if( $_POST['all_questions_graded'] == 'yes' ) {
+		// $_POST['all_questions_graded'] is set when all questions have been graded
+		// in the class sensei grading user quiz -> display()
+		if( $_POST['all_questions_graded'] == 'yes' ) {
 
-            // set the users total quiz grade
+			// set the users total quiz grade
 			if ( 0 < intval( $quiz_grade_total ) ) {
-            $grade = abs( round( ( doubleval( $quiz_grade ) * 100 ) / ( $quiz_grade_total ), 2 ) );
+			$grade = abs( round( ( doubleval( $quiz_grade ) * 100 ) / ( $quiz_grade_total ), 2 ) );
 			}
 			else {
 				$grade = 0;
 			}
-            Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id );
-
-            // Duplicating what Frontend->sensei_complete_quiz() does
-            $pass_required = get_post_meta( $quiz_id, '_pass_required', true );
-            $quiz_passmark = abs( round( doubleval( get_post_meta( $quiz_id, '_quiz_passmark', true ) ), 2 ) );
-            $lesson_metadata = array();
-            if ( $pass_required ) {
-                // Student has reached the pass mark and lesson is complete
-                if ( $quiz_passmark <= $grade ) {
-                    $lesson_status = 'passed';
-                }
-                else {
-                    $lesson_status = 'failed';
-                } // End If Statement
-            }
-            // Student only has to partake the quiz
-            else {
-                $lesson_status = 'graded';
-            }
-            $lesson_metadata['grade'] = $grade; // Technically already set as part of "WooThemes_Sensei_Utils::sensei_grade_quiz()" above
+			Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id );
+
+			// Duplicating what Frontend->sensei_complete_quiz() does
+			$pass_required = get_post_meta( $quiz_id, '_pass_required', true );
+			$quiz_passmark = abs( round( doubleval( get_post_meta( $quiz_id, '_quiz_passmark', true ) ), 2 ) );
+			$lesson_metadata = array();
+			if ( $pass_required ) {
+				// Student has reached the pass mark and lesson is complete
+				if ( $quiz_passmark <= $grade ) {
+					$lesson_status = 'passed';
+				}
+				else {
+					$lesson_status = 'failed';
+				} // End If Statement
+			}
+			// Student only has to partake the quiz
+			else {
+				$lesson_status = 'graded';
+			}
+			$lesson_metadata['grade'] = $grade; // Technically already set as part of "WooThemes_Sensei_Utils::sensei_grade_quiz()" above
 
-            Sensei_Utils::update_lesson_status( $user_id, $quiz_lesson_id, $lesson_status, $lesson_metadata );
+			Sensei_Utils::update_lesson_status( $user_id, $quiz_lesson_id, $lesson_status, $lesson_metadata );
 
-            if(  in_array( $lesson_status, array( 'passed', 'graded'  ) ) ) {
+			if(  in_array( $lesson_status, array( 'passed', 'graded'  ) ) ) {
 
-                /**
-                 * Summary.
-                 *
-                 * Description.
-                 *
-                 * @since 1.7.0
-                 *
-                 * @param int  $user_id
-                 * @param int $quiz_lesson_id
-                 */
-                do_action( 'sensei_user_lesson_end', $user_id, $quiz_lesson_id );
+				/**
+				 * Summary.
+				 *
+				 * Description.
+				 *
+				 * @since 1.7.0
+				 *
+				 * @param int  $user_id
+				 * @param int $quiz_lesson_id
+				 */
+				do_action( 'sensei_user_lesson_end', $user_id, $quiz_lesson_id );
 
-            } // end if in_array
+			} // end if in_array
 
-        }// end if $_POST['all_que...
+		}// end if $_POST['all_que...
 
-        if( isset( $_POST['sensei_grade_next_learner'] ) && strlen( $_POST['sensei_grade_next_learner'] ) > 0 ) {
+		if( isset( $_POST['sensei_grade_next_learner'] ) && strlen( $_POST['sensei_grade_next_learner'] ) > 0 ) {
 
-            $load_url = add_query_arg( array( 'message' => 'graded' ) );
+			$load_url = add_query_arg( array( 'message' => 'graded' ) );
 
-        } elseif ( isset( $_POST['_wp_http_referer'] ) ) {
+		} elseif ( isset( $_POST['_wp_http_referer'] ) ) {
 
-            $load_url = add_query_arg( array( 'message' => 'graded' ), $_POST['_wp_http_referer'] );
+			$load_url = add_query_arg( array( 'message' => 'graded' ), $_POST['_wp_http_referer'] );
 
-        } else {
+		} else {
 
-            $load_url = add_query_arg( array( 'message' => 'graded' ) );
+			$load_url = add_query_arg( array( 'message' => 'graded' ) );
 
-        }
+		}
 
-        wp_safe_redirect( esc_url_raw( $load_url ) );
-        exit;
+		wp_safe_redirect( esc_url_raw( $load_url ) );
+		exit;
 
-    } // end admin_process_grading_submission
+	} // end admin_process_grading_submission
 
 	public function get_redirect_url() {
 		// Parse POST data
@@ -706,283 +706,283 @@ 
 		} // End If Statement
 	} // End sensei_grading_notices()
 
-    /**
-     * Grade quiz automatically
-     *
-     * This function grades each question automatically if there all questions are auto gradable. If not
-     * the quiz will not be auto gradable.
-     *
-     * @since 1.7.4
-     *
-     * @param  integer $quiz_id         ID of quiz
-     * @param  array $submitted questions id ans answers {
-     *          @type int $question_id
-     *          @type mixed $answer
-     * }
-     * @param  integer $total_questions Total questions in quiz (not used)
-     * @param string $quiz_grade_type Optional defaults to auto
-     *
-     * @return int $quiz_grade total sum of all question grades
-     */
-    public static function grade_quiz_auto( $quiz_id = 0, $submitted = array(), $total_questions = 0, $quiz_grade_type = 'auto' ) {
-
-        if( ! ( intval( $quiz_id ) > 0 )  || ! $submitted
-            || $quiz_grade_type != 'auto' ) {
-            return false; // exit early
-        }
-
-
-        $user_id = get_current_user_id();
-        $lesson_id =  Sensei()->quiz->get_lesson_id(  $quiz_id ) ;
-        $quiz_autogradable = true;
-
-        /**
-         * Filter the types of question types that can be automatically graded.
-         *
-         * This filter fires inside the auto grade quiz function and provides you with the default list.
-         *
-         * @param array {
-         *      'multiple-choice',
-         *      'boolean',
-         *      'gap-fill'.
-         * }
-         */
-        $autogradable_question_types = apply_filters( 'sensei_autogradable_question_types', array( 'multiple-choice', 'boolean', 'gap-fill' ) );
-
-        $grade_total = 0;
-        $all_question_grades = array();
-        foreach( $submitted as $question_id => $answer ) {
-
-            // check if the question is autogradable, either by type, or because the grade is 0
-            $question_type = Sensei()->question->get_question_type( $question_id );
+	/**
+	 * Grade quiz automatically
+	 *
+	 * This function grades each question automatically if there all questions are auto gradable. If not
+	 * the quiz will not be auto gradable.
+	 *
+	 * @since 1.7.4
+	 *
+	 * @param  integer $quiz_id         ID of quiz
+	 * @param  array $submitted questions id ans answers {
+	 *          @type int $question_id
+	 *          @type mixed $answer
+	 * }
+	 * @param  integer $total_questions Total questions in quiz (not used)
+	 * @param string $quiz_grade_type Optional defaults to auto
+	 *
+	 * @return int $quiz_grade total sum of all question grades
+	 */
+	public static function grade_quiz_auto( $quiz_id = 0, $submitted = array(), $total_questions = 0, $quiz_grade_type = 'auto' ) {
+
+		if( ! ( intval( $quiz_id ) > 0 )  || ! $submitted
+			|| $quiz_grade_type != 'auto' ) {
+			return false; // exit early
+		}
+
+
+		$user_id = get_current_user_id();
+		$lesson_id =  Sensei()->quiz->get_lesson_id(  $quiz_id ) ;
+		$quiz_autogradable = true;
+
+		/**
+		 * Filter the types of question types that can be automatically graded.
+		 *
+		 * This filter fires inside the auto grade quiz function and provides you with the default list.
+		 *
+		 * @param array {
+		 *      'multiple-choice',
+		 *      'boolean',
+		 *      'gap-fill'.
+		 * }
+		 */
+		$autogradable_question_types = apply_filters( 'sensei_autogradable_question_types', array( 'multiple-choice', 'boolean', 'gap-fill' ) );
+
+		$grade_total = 0;
+		$all_question_grades = array();
+		foreach( $submitted as $question_id => $answer ) {
+
+			// check if the question is autogradable, either by type, or because the grade is 0
+			$question_type = Sensei()->question->get_question_type( $question_id );
 			$achievable_grade = Sensei()->question->get_question_grade( $question_id );
 			// Question has a zero grade, so skip grading
 			if ( 0 == $achievable_grade ) {
 				$all_question_grades[ $question_id ] = $achievable_grade;
 			}
-            elseif ( in_array( $question_type, $autogradable_question_types ) ) {
-                // Get user question grade
-                $question_grade = Sensei_Utils::sensei_grade_question_auto( $question_id, $question_type, $answer, $user_id );
-                $all_question_grades[ $question_id ] = $question_grade;
-                $grade_total += $question_grade;
+			elseif ( in_array( $question_type, $autogradable_question_types ) ) {
+				// Get user question grade
+				$question_grade = Sensei_Utils::sensei_grade_question_auto( $question_id, $question_type, $answer, $user_id );
+				$all_question_grades[ $question_id ] = $question_grade;
+				$grade_total += $question_grade;
 
-            } else {
+			} else {
 
-                // There is a question that cannot be autograded
-                $quiz_autogradable = false;
+				// There is a question that cannot be autograded
+				$quiz_autogradable = false;
 
-            } // end if in_array( $question_type...
+			} // end if in_array( $question_type...
 
-        }// end for each question
+		}// end for each question
 
-        // Only if the whole quiz was autogradable do we set a grade
-        if ( $quiz_autogradable ) {
+		// Only if the whole quiz was autogradable do we set a grade
+		if ( $quiz_autogradable ) {
 
-            $quiz_total = Sensei_Utils::sensei_get_quiz_total( $quiz_id );
+			$quiz_total = Sensei_Utils::sensei_get_quiz_total( $quiz_id );
 			// Check for zero total from grades
 			if ( 0 < $quiz_total ) {
-            $grade = abs( round( ( doubleval( $grade_total ) * 100 ) / ( $quiz_total ), 2 ) );
+			$grade = abs( round( ( doubleval( $grade_total ) * 100 ) / ( $quiz_total ), 2 ) );
 			}
 			else {
 				$grade = 0;
 			}
-            Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id, $quiz_grade_type );
+			Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id, $quiz_grade_type );
 
-        } else {
+		} else {
 
-            $grade = new WP_Error( 'autograde', __( 'This quiz is not able to be automatically graded.', 'woothemes-sensei' ) );
+			$grade = new WP_Error( 'autograde', __( 'This quiz is not able to be automatically graded.', 'woothemes-sensei' ) );
 
-        }
-
-        // store the auto gradable grades. If the quiz is not auto gradable the grades can be use as the default
-        // when doing manual grading.
-        Sensei()->quiz-> set_user_grades( $all_question_grades, $lesson_id, $user_id );
+		}
 
-        return $grade;
+		// store the auto gradable grades. If the quiz is not auto gradable the grades can be use as the default
+		// when doing manual grading.
+		Sensei()->quiz-> set_user_grades( $all_question_grades, $lesson_id, $user_id );
 
-    } // End grade_quiz_auto()
+		return $grade;
 
-    /**
-     * Grade question automatically
-     *
-     * This function checks the question typ and then grades it accordingly.
-     *
-     * @since 1.7.4
-     *
-     * @param integer $question_id
-     * @param string $question_type of the standard Sensei question types
-     * @param string $answer
-     * @param int $user_id
-     *
-     * @return int $question_grade
-     */
-    public static function grade_question_auto( $question_id = 0, $question_type = '', $answer = '', $user_id = 0 ) {
-
-        if( intval( $user_id ) == 0 ) {
-
-            $user_id = get_current_user_id();
-
-        }
-
-        if( ! ( intval( $question_id ) > 0 ) ) {
-
-            return false;
-
-        }
-
-
-        Sensei()->question->get_question_type( $question_id );
-
-        /**
-         * Applying a grade before the auto grading takes place.
-         *
-         * This filter is applied just before the question is auto graded. It fires in the context of a single question
-         * in the sensei_grade_question_auto function. It fires irrespective of the question type. If you return a value
-         * other than false the auto grade functionality will be ignored and your supplied grade will be user for this question.
-         *
-         * @param int $question_grade default false
-         * @param int $question_id
-         * @param string $question_type one of the Sensei question type.
-         * @param string $answer user supplied question answer
-         */
-        $question_grade = apply_filters( 'sensei_pre_grade_question_auto', false, $question_id, $question_type, $answer );
+	} // End grade_quiz_auto()
 
-        if ( false !== $question_grade ) {
+	/**
+	 * Grade question automatically
+	 *
+	 * This function checks the question typ and then grades it accordingly.
+	 *
+	 * @since 1.7.4
+	 *
+	 * @param integer $question_id
+	 * @param string $question_type of the standard Sensei question types
+	 * @param string $answer
+	 * @param int $user_id
+	 *
+	 * @return int $question_grade
+	 */
+	public static function grade_question_auto( $question_id = 0, $question_type = '', $answer = '', $user_id = 0 ) {
+
+		if( intval( $user_id ) == 0 ) {
+
+			$user_id = get_current_user_id();
+
+		}
+
+		if( ! ( intval( $question_id ) > 0 ) ) {
+
+			return false;
+
+		}
+
+
+		Sensei()->question->get_question_type( $question_id );
+
+		/**
+		 * Applying a grade before the auto grading takes place.
+		 *
+		 * This filter is applied just before the question is auto graded. It fires in the context of a single question
+		 * in the sensei_grade_question_auto function. It fires irrespective of the question type. If you return a value
+		 * other than false the auto grade functionality will be ignored and your supplied grade will be user for this question.
+		 *
+		 * @param int $question_grade default false
+		 * @param int $question_id
+		 * @param string $question_type one of the Sensei question type.
+		 * @param string $answer user supplied question answer
+		 */
+		$question_grade = apply_filters( 'sensei_pre_grade_question_auto', false, $question_id, $question_type, $answer );
 
-            return $question_grade;
+		if ( false !== $question_grade ) {
 
-        }
+			return $question_grade;
 
-        // auto grading core
-        if( in_array( $question_type ,  array( 'multiple-choice'  , 'boolean'  ) )   ){
+		}
 
-            $right_answer = (array) get_post_meta( $question_id, '_question_right_answer', true );
+		// auto grading core
+		if( in_array( $question_type ,  array( 'multiple-choice'  , 'boolean'  ) )   ){
 
-            if( 0 == get_magic_quotes_gpc() ) {
-                $answer = wp_unslash( $answer );
-            }
-            $answer = (array) $answer;
-            if ( is_array( $right_answer ) && count( $right_answer ) == count( $answer ) ) {
-                // Loop through all answers ensure none are 'missing'
-                $all_correct = true;
-                foreach ( $answer as $check_answer ) {
-                    if ( !in_array( $check_answer, $right_answer ) ) {
-                        $all_correct = false;
-                    }
-                }
-                // If all correct then grade
-                if ( $all_correct ) {
-                    $question_grade = Sensei()->question->get_question_grade( $question_id );
-                }
-            }
+			$right_answer = (array) get_post_meta( $question_id, '_question_right_answer', true );
 
-        } elseif( 'gap-fill' == $question_type ){
+			if( 0 == get_magic_quotes_gpc() ) {
+				$answer = wp_unslash( $answer );
+			}
+			$answer = (array) $answer;
+			if ( is_array( $right_answer ) && count( $right_answer ) == count( $answer ) ) {
+				// Loop through all answers ensure none are 'missing'
+				$all_correct = true;
+				foreach ( $answer as $check_answer ) {
+					if ( !in_array( $check_answer, $right_answer ) ) {
+						$all_correct = false;
+					}
+				}
+				// If all correct then grade
+				if ( $all_correct ) {
+					$question_grade = Sensei()->question->get_question_grade( $question_id );
+				}
+			}
 
-            $question_grade = self::grade_gap_fill_question( $question_id ,$answer );
+		} elseif( 'gap-fill' == $question_type ){
 
-        } else{
+			$question_grade = self::grade_gap_fill_question( $question_id ,$answer );
 
-            /**
-             * Grading questions that are not auto gradable.
-             *
-             * This filter is applied the context of ta single question within the sensei_grade_question_auto function.
-             * It fires for all other questions types. It does not apply to 'multiple-choice'  , 'boolean' and gap-fill.
-             *
-             * @param int $question_grade default zero
-             * @param int $question_id
-             * @param string $question_type one of the Sensei question type.
-             * @param string $answer user supplied question answer
-             */
-            $question_grade = ( int ) apply_filters( 'sensei_grade_question_auto', $question_grade, $question_id, $question_type, $answer );
+		} else{
 
-        } // end if $question_type
+			/**
+			 * Grading questions that are not auto gradable.
+			 *
+			 * This filter is applied the context of ta single question within the sensei_grade_question_auto function.
+			 * It fires for all other questions types. It does not apply to 'multiple-choice'  , 'boolean' and gap-fill.
+			 *
+			 * @param int $question_grade default zero
+			 * @param int $question_id
+			 * @param string $question_type one of the Sensei question type.
+			 * @param string $answer user supplied question answer
+			 */
+			$question_grade = ( int ) apply_filters( 'sensei_grade_question_auto', $question_grade, $question_id, $question_type, $answer );
 
-        return $question_grade;
-    } // end grade_question_auto
+		} // end if $question_type
 
-    /**
-     * Grading logic specifically for the gap fill questions
-     *
-     * @since 1.9.0
-     * @param $question_id
-     * @param $user_answer
-     *
-     * @return bool | int false or the grade given to the user answer
-     */
-    public static function grade_gap_fill_question( $question_id, $user_answer ){
+		return $question_grade;
+	} // end grade_question_auto
 
-        $right_answer = get_post_meta( $question_id, '_question_right_answer', true );
-        $gapfill_array = explode( '||', $right_answer );
+	/**
+	 * Grading logic specifically for the gap fill questions
+	 *
+	 * @since 1.9.0
+	 * @param $question_id
+	 * @param $user_answer
+	 *
+	 * @return bool | int false or the grade given to the user answer
+	 */
+	public static function grade_gap_fill_question( $question_id, $user_answer ){
 
-        if( 0 == get_magic_quotes_gpc() ) { // deprecated from PHP 5.4 but we still support PHP 5.2
-            $user_answer = wp_unslash( $user_answer );
-        }
+		$right_answer = get_post_meta( $question_id, '_question_right_answer', true );
+		$gapfill_array = explode( '||', $right_answer );
 
-        /**
-         * case sensitive grading filter
-         *
-         * alter the value simply use this code in your plugin or the themes functions.php
-         * add_filter( 'sensei_gap_fill_case_sensitive_grading','__return_true' );
-         *
-         * @param bool $do_case_sensitive_comparison default false.
-         *
-         * @since 1.9.0
-         */
-        $do_case_sensitive_comparison = apply_filters('sensei_gap_fill_case_sensitive_grading', false );
+		if( 0 == get_magic_quotes_gpc() ) { // deprecated from PHP 5.4 but we still support PHP 5.2
+			$user_answer = wp_unslash( $user_answer );
+		}
+
+		/**
+		 * case sensitive grading filter
+		 *
+		 * alter the value simply use this code in your plugin or the themes functions.php
+		 * add_filter( 'sensei_gap_fill_case_sensitive_grading','__return_true' );
+		 *
+		 * @param bool $do_case_sensitive_comparison default false.
+		 *
+		 * @since 1.9.0
+		 */
+		$do_case_sensitive_comparison = apply_filters('sensei_gap_fill_case_sensitive_grading', false );
 
-        if( $do_case_sensitive_comparison ){
+		if( $do_case_sensitive_comparison ){
 
-            // Case Sensitive Check that the 'gap' is "exactly" equal to the given answer
-            if ( trim(($gapfill_array[1])) == trim( $user_answer ) ) {
+			// Case Sensitive Check that the 'gap' is "exactly" equal to the given answer
+			if ( trim(($gapfill_array[1])) == trim( $user_answer ) ) {
 
-                return Sensei()->question->get_question_grade( $question_id );
+				return Sensei()->question->get_question_grade( $question_id );
 
-            } else if (@preg_match('/' . $gapfill_array[1] . '/i', null) !== FALSE) {
+			} else if (@preg_match('/' . $gapfill_array[1] . '/i', null) !== FALSE) {
 
-                if (preg_match('/' . $gapfill_array[1] . '/i', $user_answer)) {
+				if (preg_match('/' . $gapfill_array[1] . '/i', $user_answer)) {
 
-                    return Sensei()->question->get_question_grade($question_id);
+					return Sensei()->question->get_question_grade($question_id);
 
-                }else{
+				}else{
 
-                    return false;
+					return false;
 
-                }
+				}
 
-            }else{
+			}else{
 
-                return false;
+				return false;
 
-            }
+			}
 
-        }else{
+		}else{
 
-            // Case Sensitive Check that the 'gap' is "exactly" equal to the given answer
-            if ( trim(strtolower($gapfill_array[1])) == trim(strtolower( $user_answer )) ) {
+			// Case Sensitive Check that the 'gap' is "exactly" equal to the given answer
+			if ( trim(strtolower($gapfill_array[1])) == trim(strtolower( $user_answer )) ) {
 
-               return Sensei()->question->get_question_grade( $question_id );
+			   return Sensei()->question->get_question_grade( $question_id );
 
-            } else if (@preg_match('/' . $gapfill_array[1] . '/i', null) !== FALSE) {
+			} else if (@preg_match('/' . $gapfill_array[1] . '/i', null) !== FALSE) {
 
-                if (preg_match('/' . $gapfill_array[1] . '/i', $user_answer)) {
+				if (preg_match('/' . $gapfill_array[1] . '/i', $user_answer)) {
 
-                    return  Sensei()->question->get_question_grade( $question_id );
+					return  Sensei()->question->get_question_grade( $question_id );
 
-                }else{
+				}else{
 
-                    return false;
+					return false;
 
-                }
+				}
 
-            }else{
+			}else{
 
-                return false;
+				return false;
 
-            }
+			}
 
-        }
+		}
 
-    }
+	}
 
 } // End Class
 

includes/class-sensei-teacher.php

Indentation +1339 added lines, -1339 removed lines

@@ -12,251 +12,251 @@ 
  */
 class Sensei_Teacher {
 
-    /**
-     * $teacher_role
-     *
-     * Keeps a reference to the teacher role object
-     *
-     * @access protected
-     * @since 1.8.0
-     */
-    protected $teacher_role;
-
-    /**
-     * $token
-     *
-     * Keeps a reference to the global sensei token
-     *
-     * @access protected
-     * @since 1.8.0
-     */
-    public  $token;
-
-    /**
-     * Sensei_Teacher::__constructor
-     *
-     * Constructor Function
-     *
-     * @since 1.8.0
-     * @access public
-     */
-    public function __construct ( ) {
-
-        add_action( 'add_meta_boxes', array( $this , 'add_teacher_meta_boxes' ) , 10, 2 );
-        add_action( 'save_post',  array( $this, 'save_teacher_meta_box' ) );
-        add_filter( 'parse_query', array( $this, 'limit_teacher_edit_screen_post_types' ));
-        add_filter( 'pre_get_posts', array( $this, 'course_analysis_teacher_access_limit' ) );
-        add_filter( 'wp_count_posts', array( $this, 'list_table_counts' ), 10, 3 );
-
-        add_action( 'pre_get_posts', array( $this, 'filter_queries' ) );
-
-        //filter the quiz submissions
-        add_filter( 'sensei_check_for_activity' , array( $this, 'filter_grading_activity_queries') );
-
-        //grading totals count only those belonging to the teacher
-        add_filter('sensei_count_statuses_args', array( $this, 'limit_grading_totals' ) );
-
-        // show the courses owned by a user on his author archive page
-        add_filter( 'pre_get_posts', array( $this, 'add_courses_to_author_archive' ) );
-
-        // notify admin when a teacher creates a course
-        add_action( 'transition_post_status',array( $this, 'notify_admin_teacher_course_creation' ), 10, 3 );
-
-        // limit the analysis view to only the users taking courses belong to this teacher
-        add_filter( 'sensei_analysis_overview_filter_users',array( $this, 'limit_analysis_learners' ) , 5, 1 );
-
-        // give teacher access to question post type
-        add_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20, 2 );
-
-        // Teacher column on the courses list on the admin edit screen
-        add_filter('manage_edit-course_columns' , array( $this, 'course_column_heading'), 10,1 );
-        add_filter('manage_course_posts_custom_column' , array( $this, 'course_column_data'), 10,2 );
-
-        //admin edit messages query limit teacher
-        add_filter( 'pre_get_posts', array( $this, 'limit_edit_messages_query' ) );
-
-        //add filter by teacher on courses list
-        add_action( 'restrict_manage_posts', array( $this, 'course_teacher_filter_options' ) );
-        add_filter( 'request', array( $this, 'teacher_filter_query_modify' ) );
-
-        // Handle media library restrictions
-        add_filter( 'request', array( $this, 'restrict_media_library' ), 10, 1 );
-        add_filter( 'ajax_query_attachments_args', array( $this, 'restrict_media_library_modal' ), 10, 1 );
-
-        // update lesson owner to course teacher when saved
-        add_action( 'save_post',  array( $this, 'update_lesson_teacher' ) );
-
-        // If a Teacher logs in, redirect to /wp-admin/
-        add_filter( 'wp_login', array( $this, 'teacher_login_redirect') , 10, 2 );
-
-
-        add_action( 'admin_menu', array( $this, 'restrict_posts_menu_page'), 10);
-        add_filter('pre_get_comments',  array ($this, 'restrict_comment_moderation'), 10, 1);
-
-
-    } // end __constructor()
-
-    /**
-     * Sensei_Teacher::create_teacher_role
-     *
-     * This function checks if the role exist, if not it creates it.
-     * for the teacher role
-     *
-     * @since 1.8.0
-     * @access public
-     * @return void
-     */
-    public function create_role ( ) {
-
-        // check if the role exists
-        $this->teacher_role = get_role( 'teacher' );
-
-        // if the the teacher is not a valid WordPress role create it
-       if ( ! is_a( $this->teacher_role, 'WP_Role' ) ) {
-           // create the role
-           $this->teacher_role = add_role( 'teacher', __( 'Teacher', 'woothemes-sensei' ) );
-       }
-
-       // add the capabilities before returning
-        $this->add_capabilities();
-
-    }// end create_teacher_role
-
-    /**
-     * Sensei_Teacher::add_capabilities
-     *
-     * @since 1.8.0
-     * @access protected
-     */
-    protected function add_capabilities ( ) {
-
-        // if this is not a valid WP_Role object exit without adding anything
-        if(  ! is_a( $this->teacher_role, 'WP_Role' ) || empty( $this->teacher_role ) ) {
-            return;
-        }
-
-        /**
-         * Sensei teachers capabilities array filter
-         *
-         * These capabilities will be applied to the teacher role
-         * @param array $capabilities
-         * keys: (string) $cap_name => (bool) $grant
-         */
-        $caps = apply_filters( 'sensei_teacher_role_capabilities', array(
-            // General access rules
-            'read' => true,
-            'manage_sensei_grades' => true,
-            'moderate_comments'=> true,
-            'upload_files'	=> true,
-            'edit_files'	=> true,
-
-            //Lessons
-            'publish_lessons'	 => true,
-            'manage_lesson_categories'	 => true,
-            'edit_lessons'	 => true,
-            'edit_published_lessons'  => true,
-            'edit_private_lessons' => true,
-            'read_private_lessons' => true,
-            'delete_published_lessons' => true,
-
-            // Courses
-            'create_courses' => true,
-            'publish_courses'	 => false,
-            'manage_course_categories'	 => true,
-            'edit_courses'	 => true,
-            'edit_published_courses'  => true,
-            'edit_private_courses' => true,
-            'read_private_courses' => true,
-            'delete_published_courses' => true,
-
-            // Quiz
-            'publish_quizzes'	 => true,
-            'edit_quizzes'	 => true,
-            'edit_published_quizzes'  => true,
-            'edit_private_quizzes' => true,
-            'read_private_quizzes' => true,
-
-            // Questions
-            'publish_questions'	 => true,
-            'edit_questions'	 => true,
-            'edit_published_questions'  => true,
-            'edit_private_questions' => true,
-            'read_private_questions' => true,
-
-            //messages
-            'publish_sensei_messages'	 => true,
-            'edit_sensei_messages'	 => true,
-            'edit_published_sensei_messages'  => true,
-            'edit_private_sensei_messages' => true,
-            'read_private_sensei_messages' => true,
-
-            // Comments -
-            // Necessary cap so Teachers can moderate comments
-            // on their own lessons. We restrict access to other
-            // post types in $this->restrict_posts_menu_page()
-
-            'edit_posts' => true,
-
-        ));
-
-        foreach ( $caps as $cap => $grant ) {
-
-            // load the capability on to the teacher role
-            $this->teacher_role->add_cap($cap, $grant);
-
-        } // end for each
-
-    }// end add_cap
-
-    /**
-     * Sensei_Teacher::teacher_meta_box
-     *
-     * Add the teacher metabox to the course post type edit screen
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameter string $post_type
-     * @parameter WP_Post $post
-     * @return void
-     */
-    public function add_teacher_meta_boxes ( $post ) {
-
-        if( !current_user_can('manage_options') ){
-            return;
-        }
-        add_meta_box( 'sensei-teacher',  __( 'Teacher' , 'woothemes-sensei'),  array( $this , 'teacher_meta_box_content' ),
-            'course',
-            'side',
-            'core'
-        );
-
-    } // end teacher_meta_box()
-
-    /**
-     * Sensei_Teacher::teacher_meta_box_content
-     *
-     * Render the teacher meta box markup
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameters
-     */
-    public function teacher_meta_box_content ( $post ) {
-
-        // get the current author
-        $current_author = $post->post_author;
-
-        //get the users authorised to author courses
-        $users = $this->get_teachers_and_authors();
-
-    ?>
+	/**
+	 * $teacher_role
+	 *
+	 * Keeps a reference to the teacher role object
+	 *
+	 * @access protected
+	 * @since 1.8.0
+	 */
+	protected $teacher_role;
+
+	/**
+	 * $token
+	 *
+	 * Keeps a reference to the global sensei token
+	 *
+	 * @access protected
+	 * @since 1.8.0
+	 */
+	public  $token;
+
+	/**
+	 * Sensei_Teacher::__constructor
+	 *
+	 * Constructor Function
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 */
+	public function __construct ( ) {
+
+		add_action( 'add_meta_boxes', array( $this , 'add_teacher_meta_boxes' ) , 10, 2 );
+		add_action( 'save_post',  array( $this, 'save_teacher_meta_box' ) );
+		add_filter( 'parse_query', array( $this, 'limit_teacher_edit_screen_post_types' ));
+		add_filter( 'pre_get_posts', array( $this, 'course_analysis_teacher_access_limit' ) );
+		add_filter( 'wp_count_posts', array( $this, 'list_table_counts' ), 10, 3 );
+
+		add_action( 'pre_get_posts', array( $this, 'filter_queries' ) );
+
+		//filter the quiz submissions
+		add_filter( 'sensei_check_for_activity' , array( $this, 'filter_grading_activity_queries') );
+
+		//grading totals count only those belonging to the teacher
+		add_filter('sensei_count_statuses_args', array( $this, 'limit_grading_totals' ) );
+
+		// show the courses owned by a user on his author archive page
+		add_filter( 'pre_get_posts', array( $this, 'add_courses_to_author_archive' ) );
+
+		// notify admin when a teacher creates a course
+		add_action( 'transition_post_status',array( $this, 'notify_admin_teacher_course_creation' ), 10, 3 );
+
+		// limit the analysis view to only the users taking courses belong to this teacher
+		add_filter( 'sensei_analysis_overview_filter_users',array( $this, 'limit_analysis_learners' ) , 5, 1 );
+
+		// give teacher access to question post type
+		add_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20, 2 );
+
+		// Teacher column on the courses list on the admin edit screen
+		add_filter('manage_edit-course_columns' , array( $this, 'course_column_heading'), 10,1 );
+		add_filter('manage_course_posts_custom_column' , array( $this, 'course_column_data'), 10,2 );
+
+		//admin edit messages query limit teacher
+		add_filter( 'pre_get_posts', array( $this, 'limit_edit_messages_query' ) );
+
+		//add filter by teacher on courses list
+		add_action( 'restrict_manage_posts', array( $this, 'course_teacher_filter_options' ) );
+		add_filter( 'request', array( $this, 'teacher_filter_query_modify' ) );
+
+		// Handle media library restrictions
+		add_filter( 'request', array( $this, 'restrict_media_library' ), 10, 1 );
+		add_filter( 'ajax_query_attachments_args', array( $this, 'restrict_media_library_modal' ), 10, 1 );
+
+		// update lesson owner to course teacher when saved
+		add_action( 'save_post',  array( $this, 'update_lesson_teacher' ) );
+
+		// If a Teacher logs in, redirect to /wp-admin/
+		add_filter( 'wp_login', array( $this, 'teacher_login_redirect') , 10, 2 );
+
+
+		add_action( 'admin_menu', array( $this, 'restrict_posts_menu_page'), 10);
+		add_filter('pre_get_comments',  array ($this, 'restrict_comment_moderation'), 10, 1);
+
+
+	} // end __constructor()
+
+	/**
+	 * Sensei_Teacher::create_teacher_role
+	 *
+	 * This function checks if the role exist, if not it creates it.
+	 * for the teacher role
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @return void
+	 */
+	public function create_role ( ) {
+
+		// check if the role exists
+		$this->teacher_role = get_role( 'teacher' );
+
+		// if the the teacher is not a valid WordPress role create it
+	   if ( ! is_a( $this->teacher_role, 'WP_Role' ) ) {
+		   // create the role
+		   $this->teacher_role = add_role( 'teacher', __( 'Teacher', 'woothemes-sensei' ) );
+	   }
+
+	   // add the capabilities before returning
+		$this->add_capabilities();
+
+	}// end create_teacher_role
+
+	/**
+	 * Sensei_Teacher::add_capabilities
+	 *
+	 * @since 1.8.0
+	 * @access protected
+	 */
+	protected function add_capabilities ( ) {
+
+		// if this is not a valid WP_Role object exit without adding anything
+		if(  ! is_a( $this->teacher_role, 'WP_Role' ) || empty( $this->teacher_role ) ) {
+			return;
+		}
+
+		/**
+		 * Sensei teachers capabilities array filter
+		 *
+		 * These capabilities will be applied to the teacher role
+		 * @param array $capabilities
+		 * keys: (string) $cap_name => (bool) $grant
+		 */
+		$caps = apply_filters( 'sensei_teacher_role_capabilities', array(
+			// General access rules
+			'read' => true,
+			'manage_sensei_grades' => true,
+			'moderate_comments'=> true,
+			'upload_files'	=> true,
+			'edit_files'	=> true,
+
+			//Lessons
+			'publish_lessons'	 => true,
+			'manage_lesson_categories'	 => true,
+			'edit_lessons'	 => true,
+			'edit_published_lessons'  => true,
+			'edit_private_lessons' => true,
+			'read_private_lessons' => true,
+			'delete_published_lessons' => true,
+
+			// Courses
+			'create_courses' => true,
+			'publish_courses'	 => false,
+			'manage_course_categories'	 => true,
+			'edit_courses'	 => true,
+			'edit_published_courses'  => true,
+			'edit_private_courses' => true,
+			'read_private_courses' => true,
+			'delete_published_courses' => true,
+
+			// Quiz
+			'publish_quizzes'	 => true,
+			'edit_quizzes'	 => true,
+			'edit_published_quizzes'  => true,
+			'edit_private_quizzes' => true,
+			'read_private_quizzes' => true,
+
+			// Questions
+			'publish_questions'	 => true,
+			'edit_questions'	 => true,
+			'edit_published_questions'  => true,
+			'edit_private_questions' => true,
+			'read_private_questions' => true,
+
+			//messages
+			'publish_sensei_messages'	 => true,
+			'edit_sensei_messages'	 => true,
+			'edit_published_sensei_messages'  => true,
+			'edit_private_sensei_messages' => true,
+			'read_private_sensei_messages' => true,
+
+			// Comments -
+			// Necessary cap so Teachers can moderate comments
+			// on their own lessons. We restrict access to other
+			// post types in $this->restrict_posts_menu_page()
+
+			'edit_posts' => true,
+
+		));
+
+		foreach ( $caps as $cap => $grant ) {
+
+			// load the capability on to the teacher role
+			$this->teacher_role->add_cap($cap, $grant);
+
+		} // end for each
+
+	}// end add_cap
+
+	/**
+	 * Sensei_Teacher::teacher_meta_box
+	 *
+	 * Add the teacher metabox to the course post type edit screen
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameter string $post_type
+	 * @parameter WP_Post $post
+	 * @return void
+	 */
+	public function add_teacher_meta_boxes ( $post ) {
+
+		if( !current_user_can('manage_options') ){
+			return;
+		}
+		add_meta_box( 'sensei-teacher',  __( 'Teacher' , 'woothemes-sensei'),  array( $this , 'teacher_meta_box_content' ),
+			'course',
+			'side',
+			'core'
+		);
+
+	} // end teacher_meta_box()
+
+	/**
+	 * Sensei_Teacher::teacher_meta_box_content
+	 *
+	 * Render the teacher meta box markup
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameters
+	 */
+	public function teacher_meta_box_content ( $post ) {
+
+		// get the current author
+		$current_author = $post->post_author;
+
+		//get the users authorised to author courses
+		$users = $this->get_teachers_and_authors();
+
+	?>
         <select name="sensei-course-teacher-author" class="sensei course teacher">
 
             <?php foreach ( $users as $user_id ) { ?>
 
                     <?php
-                        $user = get_user_by('id', $user_id);
-                    ?>
+						$user = get_user_by('id', $user_id);
+					?>
                     <option <?php selected(  $current_author , $user_id , true ); ?> value="<?php echo $user_id; ?>" >
                         <?php echo  $user->display_name; ?>
                     </option>
@@ -267,1289 +267,1289 @@ 
 
         <?php
 
-    } // end render_teacher_meta_box()
-
-    /**
-     * Sensei_Teacher::get_teachers_and_authors
-     *
-     * Get a list of users who can author courses, lessons and quizes.
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameters
-     * @return array $users user id array
-     */
-    public function get_teachers_and_authors ( ){
-
-        $author_query_args = array(
-            'blog_id'      => $GLOBALS['blog_id'],
-            'fields'       => 'any',
-            'who'          => 'authors'
-        );
-
-        $authors = get_users( $author_query_args );
-
-        $teacher_query_args = array(
-            'blog_id'      => $GLOBALS['blog_id'],
-            'fields'       => 'any',
-            'role'         => 'teacher',
-        );
-
-        $teachers = get_users( $teacher_query_args );
-
-        return  array_unique( array_merge( $teachers, $authors ) );
-
-    }// end get_teachers_and_authors
-
-    /**
-     * Sensei_Teacher::save_teacher_meta_box
-     *
-     * Save the new teacher / author to course and all lessons
-     *
-     * Hooked into admin_init
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameters
-     * @return array $users user id array
-     */
-    public function save_teacher_meta_box ( $course_id ){
+	} // end render_teacher_meta_box()
+
+	/**
+	 * Sensei_Teacher::get_teachers_and_authors
+	 *
+	 * Get a list of users who can author courses, lessons and quizes.
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameters
+	 * @return array $users user id array
+	 */
+	public function get_teachers_and_authors ( ){
+
+		$author_query_args = array(
+			'blog_id'      => $GLOBALS['blog_id'],
+			'fields'       => 'any',
+			'who'          => 'authors'
+		);
+
+		$authors = get_users( $author_query_args );
+
+		$teacher_query_args = array(
+			'blog_id'      => $GLOBALS['blog_id'],
+			'fields'       => 'any',
+			'role'         => 'teacher',
+		);
+
+		$teachers = get_users( $teacher_query_args );
+
+		return  array_unique( array_merge( $teachers, $authors ) );
+
+	}// end get_teachers_and_authors
+
+	/**
+	 * Sensei_Teacher::save_teacher_meta_box
+	 *
+	 * Save the new teacher / author to course and all lessons
+	 *
+	 * Hooked into admin_init
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameters
+	 * @return array $users user id array
+	 */
+	public function save_teacher_meta_box ( $course_id ){
 
-        // check if this is a post from saving the teacher, if not exit early
-        if(! isset( $_POST[ 'sensei-course-teacher-author' ] ) || ! isset( $_POST['post_ID'] )  ){
-            return;
-        }
+		// check if this is a post from saving the teacher, if not exit early
+		if(! isset( $_POST[ 'sensei-course-teacher-author' ] ) || ! isset( $_POST['post_ID'] )  ){
+			return;
+		}
 
-        //don't fire this hook again
-        remove_action('save_post', array( $this, 'save_teacher_meta_box' ) );
+		//don't fire this hook again
+		remove_action('save_post', array( $this, 'save_teacher_meta_box' ) );
 
-        // get the current post object
-        $post = get_post( $course_id );
+		// get the current post object
+		$post = get_post( $course_id );
 
-        // get the current teacher/author
-        $current_author = absint( $post->post_author );
-        $new_author = absint( $_POST[ 'sensei-course-teacher-author' ] );
+		// get the current teacher/author
+		$current_author = absint( $post->post_author );
+		$new_author = absint( $_POST[ 'sensei-course-teacher-author' ] );
 
-        // loop through all post lessons to update their authors as well
-        $this->update_course_lessons_author( $course_id , $new_author );
+		// loop through all post lessons to update their authors as well
+		$this->update_course_lessons_author( $course_id , $new_author );
 
-        // do not do any processing if the selected author is the same as the current author
-        if( $current_author == $new_author ){
-            return;
-        }
+		// do not do any processing if the selected author is the same as the current author
+		if( $current_author == $new_author ){
+			return;
+		}
 
-        // save the course  author
-        $post_updates = array(
-            'ID' => $post->ID ,
-            'post_author' => $new_author
-        );
-        wp_update_post( $post_updates );
+		// save the course  author
+		$post_updates = array(
+			'ID' => $post->ID ,
+			'post_author' => $new_author
+		);
+		wp_update_post( $post_updates );
 
-        // ensure the the modules are update so that then new teacher has access to them
-        Sensei_Teacher::update_course_modules_author( $course_id, $new_author );
+		// ensure the the modules are update so that then new teacher has access to them
+		Sensei_Teacher::update_course_modules_author( $course_id, $new_author );
 
-        // notify the new teacher
-        $this->teacher_course_assigned_notification( $new_author, $course_id );
+		// notify the new teacher
+		$this->teacher_course_assigned_notification( $new_author, $course_id );
 
-    } // end save_teacher_meta_box
+	} // end save_teacher_meta_box
 
-    /**
-     * Update all the course terms set(selected) on the given course. Moving course term ownership to
-     * the new author. Making sure the course terms are maintained.
-     *
-     * This function also checks if terms are shared, with other courses
-     *
-     * @param $course_id
-     * @param $new_teacher_id
-     * @return void
-     */
-    public static function update_course_modules_author( $course_id ,$new_teacher_id ){
+	/**
+	 * Update all the course terms set(selected) on the given course. Moving course term ownership to
+	 * the new author. Making sure the course terms are maintained.
+	 *
+	 * This function also checks if terms are shared, with other courses
+	 *
+	 * @param $course_id
+	 * @param $new_teacher_id
+	 * @return void
+	 */
+	public static function update_course_modules_author( $course_id ,$new_teacher_id ){
 
-        if( empty( $course_id ) || empty( $new_teacher_id ) ){
-            return;
-        }
+		if( empty( $course_id ) || empty( $new_teacher_id ) ){
+			return;
+		}
 
-        $terms_selected_on_course = wp_get_object_terms( $course_id, 'module' );
+		$terms_selected_on_course = wp_get_object_terms( $course_id, 'module' );
 
-        if( empty( $terms_selected_on_course ) ){
-            return;
-        }
+		if( empty( $terms_selected_on_course ) ){
+			return;
+		}
 
-        foreach( $terms_selected_on_course as $term ){
+		foreach( $terms_selected_on_course as $term ){
 
-            $term_author = Sensei_Core_Modules::get_term_author( $term->slug );
-            if( $new_teacher_id != $term_author->ID  ){
+			$term_author = Sensei_Core_Modules::get_term_author( $term->slug );
+			if( $new_teacher_id != $term_author->ID  ){
 
-                $new_term = '';
+				$new_term = '';
 
-                //if the new teacher is admin first check to see if the term with this name already exists
-                if( user_can( $new_teacher_id, 'manage_options' ) ){
+				//if the new teacher is admin first check to see if the term with this name already exists
+				if( user_can( $new_teacher_id, 'manage_options' ) ){
 
-                    $slug_without_teacher_id = str_ireplace(' ', '-', trim( $term->name ) );
-                    $term_args = array( 'slug'=> $slug_without_teacher_id, 'hide_empty' => false, );
-                    $existing_admin_terms = get_terms( 'module', $term_args );
-                    if( !empty( $existing_admin_terms ) ){
-                        // insert it even if it exists
-                        $new_term = get_term( $existing_admin_terms[0]->term_id, 'module', ARRAY_A );
-                    }
-                }
+					$slug_without_teacher_id = str_ireplace(' ', '-', trim( $term->name ) );
+					$term_args = array( 'slug'=> $slug_without_teacher_id, 'hide_empty' => false, );
+					$existing_admin_terms = get_terms( 'module', $term_args );
+					if( !empty( $existing_admin_terms ) ){
+						// insert it even if it exists
+						$new_term = get_term( $existing_admin_terms[0]->term_id, 'module', ARRAY_A );
+					}
+				}
 
-                if( empty ( $new_term ) ){
+				if( empty ( $new_term ) ){
 
-                   //setup the new slug
-                   $new_author_term_slug =  $new_teacher_id . '-' . str_ireplace(' ', '-', trim( $term->name ) );
-
-                   // create new term and set it
-                   $new_term = wp_insert_term( $term->name,'module', array('slug'=> $new_author_term_slug )  );
-
-                }
+				   //setup the new slug
+				   $new_author_term_slug =  $new_teacher_id . '-' . str_ireplace(' ', '-', trim( $term->name ) );
+
+				   // create new term and set it
+				   $new_term = wp_insert_term( $term->name,'module', array('slug'=> $new_author_term_slug )  );
+
+				}
 
 
 
-                // if term exists
-                if( is_wp_error( $new_term ) && isset( $new_term->errors['term_exists'] ) ){
+				// if term exists
+				if( is_wp_error( $new_term ) && isset( $new_term->errors['term_exists'] ) ){
 
-                    $existing_term = get_term_by( 'slug', $new_author_term_slug, 'module');
-                    $term_id = $existing_term->term_id;
+					$existing_term = get_term_by( 'slug', $new_author_term_slug, 'module');
+					$term_id = $existing_term->term_id;
 
-                }else{
+				}else{
 
-                    // for a new term simply get the term from the returned value
-                    $term_id = $new_term['term_id'];
+					// for a new term simply get the term from the returned value
+					$term_id = $new_term['term_id'];
 
-                } // end if term exist
+				} // end if term exist
 
-                // set the terms selected on the course
-                wp_set_object_terms( $course_id, $term_id , 'module', true );
+				// set the terms selected on the course
+				wp_set_object_terms( $course_id, $term_id , 'module', true );
 
-                // remove old term
-                wp_remove_object_terms( $course_id, $term->term_id, 'module' );
+				// remove old term
+				wp_remove_object_terms( $course_id, $term->term_id, 'module' );
 
-                // update the lessons within the current module term
-                $lessons = Sensei()->course->course_lessons( $course_id );
-                foreach( $lessons as $lesson  ){
+				// update the lessons within the current module term
+				$lessons = Sensei()->course->course_lessons( $course_id );
+				foreach( $lessons as $lesson  ){
 
-                    if( has_term( $term->slug, 'module', $lesson ) ){
+					if( has_term( $term->slug, 'module', $lesson ) ){
 
-                        // add the new term, the false at the end says to replace all terms on this module
-                        // with the new term.
-                        wp_set_object_terms( $lesson->ID, $term_id , 'module', false );
-                        update_post_meta( $lesson->ID, '_order_module_' . intval( $term_id ), 0 );
-                    }
+						// add the new term, the false at the end says to replace all terms on this module
+						// with the new term.
+						wp_set_object_terms( $lesson->ID, $term_id , 'module', false );
+						update_post_meta( $lesson->ID, '_order_module_' . intval( $term_id ), 0 );
+					}
 
-                }// end for each
+				}// end for each
 
-            }
-        }
+			}
+		}
 
-    }// end update_course_module_terms_author
+	}// end update_course_module_terms_author
 
-    /**
-     * Sensei_Teacher::update_course_lessons_author
-     *
-     * Update all course lessons and their quiz with a new author
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameters
-     * @return array $users user id array
-     */
-    public function update_course_lessons_author ( $course_id, $new_author  ){
+	/**
+	 * Sensei_Teacher::update_course_lessons_author
+	 *
+	 * Update all course lessons and their quiz with a new author
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameters
+	 * @return array $users user id array
+	 */
+	public function update_course_lessons_author ( $course_id, $new_author  ){
 
 
-        if( empty( $course_id ) || empty( $new_author ) ){
-            return false;
-        }
+		if( empty( $course_id ) || empty( $new_author ) ){
+			return false;
+		}
 
-        //get a list of course lessons
-        $lessons = Sensei()->course->course_lessons( $course_id );
+		//get a list of course lessons
+		$lessons = Sensei()->course->course_lessons( $course_id );
 
-        if( empty( $lessons )  ||  ! is_array( $lessons )  ){
-            return false;
-        }
+		if( empty( $lessons )  ||  ! is_array( $lessons )  ){
+			return false;
+		}
 
-        // update each lesson and quiz author
-        foreach( $lessons as $lesson ){
+		// update each lesson and quiz author
+		foreach( $lessons as $lesson ){
 
-            // don't update if the author is tha same as the new author
-            if( $new_author == $lesson->post_author ){
-                continue;
-            }
+			// don't update if the author is tha same as the new author
+			if( $new_author == $lesson->post_author ){
+				continue;
+			}
 
-            // update lesson author
-            wp_update_post( array(
-                'ID'=> $lesson->ID,
-                'post_author' => $new_author
-                ) );
+			// update lesson author
+			wp_update_post( array(
+				'ID'=> $lesson->ID,
+				'post_author' => $new_author
+				) );
 
-            // update quiz author
-            //get the lessons quiz
-            $lesson_quizzes = Sensei()->lesson->lesson_quizzes( $lesson->ID );
-            if( is_array( $lesson_quizzes ) ){
-                foreach ( $lesson_quizzes as $quiz_id ) {
-                    // update quiz with new author
-                    wp_update_post( array(
-                        'ID'           => $quiz_id,
-                        'post_author' =>  $new_author
-                    ) );
-                }
-            }else{
-                wp_update_post( array(
-                    'ID'           => $lesson_quizzes,
-                    'post_author' =>  $new_author
-                ) );
-            }
+			// update quiz author
+			//get the lessons quiz
+			$lesson_quizzes = Sensei()->lesson->lesson_quizzes( $lesson->ID );
+			if( is_array( $lesson_quizzes ) ){
+				foreach ( $lesson_quizzes as $quiz_id ) {
+					// update quiz with new author
+					wp_update_post( array(
+						'ID'           => $quiz_id,
+						'post_author' =>  $new_author
+					) );
+				}
+			}else{
+				wp_update_post( array(
+					'ID'           => $lesson_quizzes,
+					'post_author' =>  $new_author
+				) );
+			}
 
-        } // end for each lessons
+		} // end for each lessons
 
-        return true;
+		return true;
 
-    }// end update_course_lessons_author
-
-
-
-    /**
-     * Sensei_Teacher::course_analysis_teacher_access_limit
-     *
-     * Alter the query so that users can only see their courses on the analysis page
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameters $query
-     * @return array $users user id array
-     */
-    public function course_analysis_teacher_access_limit ( $query ) {
-
-        if( ! is_admin() || ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
-            return $query;
-        }
-
-        if ( ! function_exists( 'get_current_screen' ) ) {
-            return $query;
-        }
-
-        $screen = get_current_screen();
-        $sensei_post_types = array('course', 'lesson', 'question' );
-
-        // exit early for the following conditions
-        $limit_screen_ids = array( 'sensei_page_sensei_analysis', 'course_page_module-order' );
-
-        if( ! $this->is_admin_teacher() || empty( $screen ) || ! in_array( $screen->id ,$limit_screen_ids )
-            || ! in_array( $query->query['post_type'], $sensei_post_types ) ){
-            return $query;
-        }
-
-        global $current_user;
-        // set the query author to the current user to only show those those posts
-        $query->set( 'author', $current_user->ID );
-        return $query;
-
-    }// end course_analysis_teacher_access_limit
-
-
-    /**
-     * Sensei_Teacher::limit_teacher_edit_screen_post_types
-     *
-     * Determine if we're in admin and the current logged in use is a teacher
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameters array $wp_query
-     * @return bool $is_admin_teacher
-     */
-    public function is_admin_teacher ( ){
-
-        if( ! is_user_logged_in()){
-            return false;
-        }
-        $is_admin_teacher = false;
-
-        if( is_admin() && Sensei_Teacher::is_a_teacher( get_current_user_id() )  ){
-
-            $is_admin_teacher = true;
-
-        }
-
-        return $is_admin_teacher;
-
-    } // end is_admin_teacher
-
-    /**
-     * Show correct post counts on list table for Sensei post types
-     *
-     * @since 1.8.0
-     *
-     * @param  object $counts Default status counts
-     * @param  string $type   Current post type
-     * @param  string $perm   User permission level
-     * @return object         Modified status counts
-     */
-    public function list_table_counts( $counts, $type, $perm ) {
-        global $current_user;
-
-        if( ! in_array( $type, array( 'course', 'lesson', 'question' ) ) ) {
-            return $counts;
-        }
-
-        if( ! $this->is_admin_teacher() ) {
-            return $counts;
-        }
-
-        $args = array(
-            'post_type' => $type,
-            'author' => $current_user->ID,
-            'posts_per_page' => -1
-        );
-
-         // Get all available statuses
-        $stati = get_post_stati();
-
-        // Update count object
-        foreach( $stati as $status ) {
-            $args['post_status'] = $status;
-            $posts = get_posts( $args );
-            $counts->$status = count( $posts );
-        }
-
-        return $counts;
-    }
-
-    /**
-     * Filter the post queries to show
-     * only lesson /course and users that belong
-     * to the current logged teacher.
-     *
-     * @since 1.8.0
-     *
-     */
-    public function filter_queries ( $query ) {
-        global $current_user;
-
-        if( ! $this->is_admin_teacher() ) {
-            return;
-        }
-
-        if ( ! function_exists( 'get_current_screen' ) ) {
-            return;
-        }
-
-        $screen = get_current_screen();
-        if( empty( $screen ) ) {
-            return $query;
-        }
-        switch( $screen->id ) {
-            case 'sensei_page_sensei_grading':
-            case 'sensei_page_sensei_analysis':
-            case 'sensei_page_sensei_learners':
-            case 'lesson':
-            case 'course':
-            case 'question':
-            case 'lesson_page_module-order':
-
-            /**
-             * sensei_filter_queries_set_author
-             * Filter the author Sensei set for queries
-             *
-             * @since 1.8.0
-             *
-             * @param int $user_id
-             * @param string $screen_id
-             *
-             */
-            $query->set( 'author', apply_filters( 'sensei_filter_queries_set_author', $current_user->ID, $screen->id ) );
-            break;
-        }
-    }
-
-    /**
-     * Limit grading quizzes to only those within courses belonging to the current teacher
-     * . This excludes the admin user.
-     *
-     * @since 1.8.0
-     * @hooked into the_comments
-     * @param array  $comments
-     *
-     * @return array $comments
-     */
-    public function filter_grading_activity_queries( $comments ){
-
-        if( !is_admin() || ! $this->is_admin_teacher() || is_numeric( $comments ) || ! is_array( $comments ) ){
-            return $comments ;
-        }
-
-        //check if we're on the grading screen
-        $screen = get_current_screen();
-
-        if( empty( $screen ) || 'sensei_page_sensei_grading' != $screen->id ){
-            return $comments;
-        }
-
-        // get the course and determine if the current teacher is the owner
-        // if not remove it from the list of comments to be returned
-        foreach( $comments as $key => $comment){
-            $lesson = get_post( $comment->comment_post_ID );
-            $course_id = Sensei()->lesson->get_course_id( $lesson->ID );
-            $course = get_post( $course_id );
-            if( ! isset( $course->post_author ) || intval( $course->post_author) != intval( get_current_user_id() ) ){
-                //remove this as the teacher should see this.
-                unset( $comments[ $key ] );
-            }
-        }
-        return $comments ;
-
-    }// end function filter grading
-
-    /**
-     * Limit the grading screen totals to only show lessons in the course
-     * belonging to the currently logged in teacher. This only applies to
-     * the teacher role.
-     *
-     * @since 1.8.0
-     *
-     * @hooked into sensei_count_statuses_args
-     * @param array $args
-     *
-     * @return array  $args
-     */
-    public function limit_grading_totals( $args ){
-
-        if( !is_admin() || ! $this->is_admin_teacher() || ! is_array( $args ) ){
-            return $args ;
-        }
-
-        //get the teachers courses
-        // the query is already filtered to only the teacher
-        $courses =  Sensei()->course->get_all_courses();
-
-        if( empty(  $courses ) || ! is_array( $courses ) ){
-            return $args;
-        }
-
-        //setup the lessons quizzes  to limit the grading totals to
-        $quiz_scope = array();
-        foreach( $courses as $course ){
-
-            $course_lessons = Sensei()->course->course_lessons( $course->ID );
-
-            if( ! empty( $course_lessons ) && is_array( $course_lessons  ) ){
-
-                foreach(  $course_lessons as $lesson ){
-
-                    $quiz_id = Sensei()->lesson->lesson_quizzes( $lesson->ID );
-                    if( !empty( $quiz_id ) ) {
-
-                        array_push( $quiz_scope, $quiz_id );
-
-                    }
-
-                }
-
-            }
-
-        }
-
-        $args['post__in'] = $quiz_scope;
-
-        return $args;
-    }
-
-    /**
-     * It ensures that the author archive shows course by the current user.
-     *
-     * This function is hooked into the pre_get_posts filter
-     *
-     * @param WP_Query $query
-     * @return WP_Query $query
-     */
-    public function add_courses_to_author_archive( $query ) {
-
-        if ( is_admin() || ! $query->is_author() ){
-            return $query;
-        }
-
-        // this should only apply to users with the teacher role
-        $current_page_user = get_user_by('login', $query->get('author_name') );
-        if( ! $current_page_user || ! in_array('teacher', $current_page_user->roles ) )     {
-
-            return $query;
-
-        }
-
-        // Change post types depending on what is set already
-        $current_post_types = $query->get( 'post_type' );
-        if( empty( $current_post_types  ) ){
-
-            // if empty it means post by default, so add post so that it also includes that for now
-            $new_post_types = array( 'post', 'course' );
-
-        } elseif( is_array( $current_post_types  ) ) {
-
-            // merge the post types instead of overwriting it
-            $new_post_types = array_merge( $current_post_types, array( 'course' ) );
-
-        }else{
-
-            // in this instance it is probably just one post type in string format
-            $new_post_types =  array( $current_post_types , 'course');
-
-        }
-
-        // change the query before returning it
-        $query->set('post_type', $new_post_types );
-
-        /**
-         * Change the query on the teacher author archive template
-         *
-         * @since 1.8.4
-         * @param WP_Query $query
-         */
-        return apply_filters( 'sensei_teacher_archive_query', $query );
-
-    }
-
-    /**
-     * Notify teacher when someone assigns a course to their account.
-     *
-     * @since 1.8.0
-     *
-     * @param $teacher_id
-     * @param $course_id
-     * @return bool
-     */
-    public function teacher_course_assigned_notification( $teacher_id, $course_id ){
-
-        if( 'course' != get_post_type( $course_id ) || ! get_userdata( $teacher_id ) ){
-            return false;
-        }
-
-        // if new user is the same as the current logged user, they don't need an email
-        if( $teacher_id == get_current_user_id() ){
-            return true;
-        }
-
-        // load the email class
-        include('emails/class-sensei-email-teacher-new-course-assignment.php');
-        $email = new Sensei_Email_Teacher_New_Course_Assignment();
-        $email->trigger( $teacher_id, $course_id );
-
-        return true;
-    } // end  teacher_course_assigned_notification
-
-    /**
-     * Email the admin when a teacher creates a new course
-     *
-     * This function hooks into wp_insert_post
-     *
-     * @since 1.8.0
-     * @param int $course_id
-     * @return bool
-     */
-    public function notify_admin_teacher_course_creation( $new_status, $old_status, $post ){
-
-        $course_id = $post->ID;
-
-        if( 'course' != get_post_type( $course_id ) || 'auto-draft' == get_post_status( $course_id )
-            || 'trash' == get_post_status( $course_id ) || 'draft' == get_post_status( $course_id ) ) {
-
-            return false;
-
-        }
-
-        /**
-         * Filter the option to send admin notification emails when teachers creation
-         * course.
-         *
-         * @since 1.8.0
-         *
-         * @param bool $on default true
-         */
-        if( ! apply_filters('sensei_notify_admin_new_course_creation', true ) ){
-            return false;
-        }
-
-        // setting up the data needed by the email template
-        global $sensei_email_data;
-        $template = 'admin-teacher-new-course-created';
-        $course = get_post( $course_id );
-        $teacher = new WP_User( $course->post_author );
-        $recipient = get_option('admin_email', true);
-
-        // don't send if the course is created by admin
-        if( $recipient == $teacher->user_email ){
-            return;
-        }
-
-        /**
-         * Filter the email Header for the admin-teacher-new-course-created template
-         *
-         * @since 1.8.0
-         * @param string $template
-         */
-        $heading = apply_filters( 'sensei_email_heading', __( 'New course created.', 'woothemes-sensei' ), $template );
-
-        /**
-         * Filter the email subject for the the
-         * admin-teacher-new-course-created template
-         *
-         * @since 1.8.0
-         * @param string $subject default New course assigned to you
-         * @param string $template
-         */
-        $subject = apply_filters('sensei_email_subject',
-                                '['. get_bloginfo( 'name', 'display' ) .'] '. __( 'New course created by', 'woothemes-sensei' ) . ' ' . $teacher->display_name ,
-                                $template );
-
-        //course edit link
-        $course_edit_link = admin_url('post.php?post=' . $course_id . '&action=edit' );
-
-        // Construct data array
-        $email_data = array(
-            'template'			=> $template,
-            'heading' =>  $heading,
-            'teacher'		=> $teacher,
-            'course_id'			=> $course_id,
-            'course_name'			=> $course->post_title,
-            'course_edit_link' => $course_edit_link,
-        );
-
-        /**
-         * Filter the sensei email data for the admin-teacher-new-course-created template
-         *
-         * @since 1.8.0
-         * @param array $email_data
-         * @param string $template
-         */
-        $sensei_email_data = apply_filters( 'sensei_email_data', $email_data , $template );
-
-        // Send mail
-        Sensei()->emails->send( $recipient, $subject , Sensei()->emails->get_content( $template ) );
-
-    }// end notify admin of course creation
-
-    /**
-     * Limit the analysis view to only the users taking courses belong to this teacher
-     *
-     * Hooked into sensei_analysis_get_learners
-     * @param array $args WP_User_Query arguments
-     * @return array $learners_query_results
-     */
-    public function limit_analysis_learners( $args ){
-
-        // show default for none teachers
-        if( ! Sensei()->teacher->is_admin_teacher() ) {
-                return $args;
-        }
-
-        // for teachers all courses only return those which belong to the teacher
-        // as they don't have access to course belonging to other users
-        $teacher_courses = Sensei()->course->get_all_courses();
+	}// end update_course_lessons_author
+
+
+
+	/**
+	 * Sensei_Teacher::course_analysis_teacher_access_limit
+	 *
+	 * Alter the query so that users can only see their courses on the analysis page
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameters $query
+	 * @return array $users user id array
+	 */
+	public function course_analysis_teacher_access_limit ( $query ) {
+
+		if( ! is_admin() || ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ) {
+			return $query;
+		}
+
+		if ( ! function_exists( 'get_current_screen' ) ) {
+			return $query;
+		}
+
+		$screen = get_current_screen();
+		$sensei_post_types = array('course', 'lesson', 'question' );
+
+		// exit early for the following conditions
+		$limit_screen_ids = array( 'sensei_page_sensei_analysis', 'course_page_module-order' );
+
+		if( ! $this->is_admin_teacher() || empty( $screen ) || ! in_array( $screen->id ,$limit_screen_ids )
+			|| ! in_array( $query->query['post_type'], $sensei_post_types ) ){
+			return $query;
+		}
+
+		global $current_user;
+		// set the query author to the current user to only show those those posts
+		$query->set( 'author', $current_user->ID );
+		return $query;
+
+	}// end course_analysis_teacher_access_limit
+
+
+	/**
+	 * Sensei_Teacher::limit_teacher_edit_screen_post_types
+	 *
+	 * Determine if we're in admin and the current logged in use is a teacher
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameters array $wp_query
+	 * @return bool $is_admin_teacher
+	 */
+	public function is_admin_teacher ( ){
+
+		if( ! is_user_logged_in()){
+			return false;
+		}
+		$is_admin_teacher = false;
+
+		if( is_admin() && Sensei_Teacher::is_a_teacher( get_current_user_id() )  ){
+
+			$is_admin_teacher = true;
+
+		}
+
+		return $is_admin_teacher;
+
+	} // end is_admin_teacher
+
+	/**
+	 * Show correct post counts on list table for Sensei post types
+	 *
+	 * @since 1.8.0
+	 *
+	 * @param  object $counts Default status counts
+	 * @param  string $type   Current post type
+	 * @param  string $perm   User permission level
+	 * @return object         Modified status counts
+	 */
+	public function list_table_counts( $counts, $type, $perm ) {
+		global $current_user;
+
+		if( ! in_array( $type, array( 'course', 'lesson', 'question' ) ) ) {
+			return $counts;
+		}
+
+		if( ! $this->is_admin_teacher() ) {
+			return $counts;
+		}
+
+		$args = array(
+			'post_type' => $type,
+			'author' => $current_user->ID,
+			'posts_per_page' => -1
+		);
+
+		 // Get all available statuses
+		$stati = get_post_stati();
+
+		// Update count object
+		foreach( $stati as $status ) {
+			$args['post_status'] = $status;
+			$posts = get_posts( $args );
+			$counts->$status = count( $posts );
+		}
+
+		return $counts;
+	}
+
+	/**
+	 * Filter the post queries to show
+	 * only lesson /course and users that belong
+	 * to the current logged teacher.
+	 *
+	 * @since 1.8.0
+	 *
+	 */
+	public function filter_queries ( $query ) {
+		global $current_user;
+
+		if( ! $this->is_admin_teacher() ) {
+			return;
+		}
+
+		if ( ! function_exists( 'get_current_screen' ) ) {
+			return;
+		}
+
+		$screen = get_current_screen();
+		if( empty( $screen ) ) {
+			return $query;
+		}
+		switch( $screen->id ) {
+			case 'sensei_page_sensei_grading':
+			case 'sensei_page_sensei_analysis':
+			case 'sensei_page_sensei_learners':
+			case 'lesson':
+			case 'course':
+			case 'question':
+			case 'lesson_page_module-order':
+
+			/**
+			 * sensei_filter_queries_set_author
+			 * Filter the author Sensei set for queries
+			 *
+			 * @since 1.8.0
+			 *
+			 * @param int $user_id
+			 * @param string $screen_id
+			 *
+			 */
+			$query->set( 'author', apply_filters( 'sensei_filter_queries_set_author', $current_user->ID, $screen->id ) );
+			break;
+		}
+	}
+
+	/**
+	 * Limit grading quizzes to only those within courses belonging to the current teacher
+	 * . This excludes the admin user.
+	 *
+	 * @since 1.8.0
+	 * @hooked into the_comments
+	 * @param array  $comments
+	 *
+	 * @return array $comments
+	 */
+	public function filter_grading_activity_queries( $comments ){
+
+		if( !is_admin() || ! $this->is_admin_teacher() || is_numeric( $comments ) || ! is_array( $comments ) ){
+			return $comments ;
+		}
+
+		//check if we're on the grading screen
+		$screen = get_current_screen();
+
+		if( empty( $screen ) || 'sensei_page_sensei_grading' != $screen->id ){
+			return $comments;
+		}
+
+		// get the course and determine if the current teacher is the owner
+		// if not remove it from the list of comments to be returned
+		foreach( $comments as $key => $comment){
+			$lesson = get_post( $comment->comment_post_ID );
+			$course_id = Sensei()->lesson->get_course_id( $lesson->ID );
+			$course = get_post( $course_id );
+			if( ! isset( $course->post_author ) || intval( $course->post_author) != intval( get_current_user_id() ) ){
+				//remove this as the teacher should see this.
+				unset( $comments[ $key ] );
+			}
+		}
+		return $comments ;
+
+	}// end function filter grading
+
+	/**
+	 * Limit the grading screen totals to only show lessons in the course
+	 * belonging to the currently logged in teacher. This only applies to
+	 * the teacher role.
+	 *
+	 * @since 1.8.0
+	 *
+	 * @hooked into sensei_count_statuses_args
+	 * @param array $args
+	 *
+	 * @return array  $args
+	 */
+	public function limit_grading_totals( $args ){
+
+		if( !is_admin() || ! $this->is_admin_teacher() || ! is_array( $args ) ){
+			return $args ;
+		}
+
+		//get the teachers courses
+		// the query is already filtered to only the teacher
+		$courses =  Sensei()->course->get_all_courses();
+
+		if( empty(  $courses ) || ! is_array( $courses ) ){
+			return $args;
+		}
+
+		//setup the lessons quizzes  to limit the grading totals to
+		$quiz_scope = array();
+		foreach( $courses as $course ){
+
+			$course_lessons = Sensei()->course->course_lessons( $course->ID );
+
+			if( ! empty( $course_lessons ) && is_array( $course_lessons  ) ){
+
+				foreach(  $course_lessons as $lesson ){
+
+					$quiz_id = Sensei()->lesson->lesson_quizzes( $lesson->ID );
+					if( !empty( $quiz_id ) ) {
+
+						array_push( $quiz_scope, $quiz_id );
+
+					}
+
+				}
+
+			}
+
+		}
+
+		$args['post__in'] = $quiz_scope;
+
+		return $args;
+	}
+
+	/**
+	 * It ensures that the author archive shows course by the current user.
+	 *
+	 * This function is hooked into the pre_get_posts filter
+	 *
+	 * @param WP_Query $query
+	 * @return WP_Query $query
+	 */
+	public function add_courses_to_author_archive( $query ) {
+
+		if ( is_admin() || ! $query->is_author() ){
+			return $query;
+		}
+
+		// this should only apply to users with the teacher role
+		$current_page_user = get_user_by('login', $query->get('author_name') );
+		if( ! $current_page_user || ! in_array('teacher', $current_page_user->roles ) )     {
+
+			return $query;
+
+		}
+
+		// Change post types depending on what is set already
+		$current_post_types = $query->get( 'post_type' );
+		if( empty( $current_post_types  ) ){
+
+			// if empty it means post by default, so add post so that it also includes that for now
+			$new_post_types = array( 'post', 'course' );
+
+		} elseif( is_array( $current_post_types  ) ) {
+
+			// merge the post types instead of overwriting it
+			$new_post_types = array_merge( $current_post_types, array( 'course' ) );
+
+		}else{
+
+			// in this instance it is probably just one post type in string format
+			$new_post_types =  array( $current_post_types , 'course');
+
+		}
+
+		// change the query before returning it
+		$query->set('post_type', $new_post_types );
+
+		/**
+		 * Change the query on the teacher author archive template
+		 *
+		 * @since 1.8.4
+		 * @param WP_Query $query
+		 */
+		return apply_filters( 'sensei_teacher_archive_query', $query );
+
+	}
+
+	/**
+	 * Notify teacher when someone assigns a course to their account.
+	 *
+	 * @since 1.8.0
+	 *
+	 * @param $teacher_id
+	 * @param $course_id
+	 * @return bool
+	 */
+	public function teacher_course_assigned_notification( $teacher_id, $course_id ){
+
+		if( 'course' != get_post_type( $course_id ) || ! get_userdata( $teacher_id ) ){
+			return false;
+		}
+
+		// if new user is the same as the current logged user, they don't need an email
+		if( $teacher_id == get_current_user_id() ){
+			return true;
+		}
+
+		// load the email class
+		include('emails/class-sensei-email-teacher-new-course-assignment.php');
+		$email = new Sensei_Email_Teacher_New_Course_Assignment();
+		$email->trigger( $teacher_id, $course_id );
+
+		return true;
+	} // end  teacher_course_assigned_notification
+
+	/**
+	 * Email the admin when a teacher creates a new course
+	 *
+	 * This function hooks into wp_insert_post
+	 *
+	 * @since 1.8.0
+	 * @param int $course_id
+	 * @return bool
+	 */
+	public function notify_admin_teacher_course_creation( $new_status, $old_status, $post ){
+
+		$course_id = $post->ID;
+
+		if( 'course' != get_post_type( $course_id ) || 'auto-draft' == get_post_status( $course_id )
+			|| 'trash' == get_post_status( $course_id ) || 'draft' == get_post_status( $course_id ) ) {
+
+			return false;
+
+		}
+
+		/**
+		 * Filter the option to send admin notification emails when teachers creation
+		 * course.
+		 *
+		 * @since 1.8.0
+		 *
+		 * @param bool $on default true
+		 */
+		if( ! apply_filters('sensei_notify_admin_new_course_creation', true ) ){
+			return false;
+		}
+
+		// setting up the data needed by the email template
+		global $sensei_email_data;
+		$template = 'admin-teacher-new-course-created';
+		$course = get_post( $course_id );
+		$teacher = new WP_User( $course->post_author );
+		$recipient = get_option('admin_email', true);
+
+		// don't send if the course is created by admin
+		if( $recipient == $teacher->user_email ){
+			return;
+		}
+
+		/**
+		 * Filter the email Header for the admin-teacher-new-course-created template
+		 *
+		 * @since 1.8.0
+		 * @param string $template
+		 */
+		$heading = apply_filters( 'sensei_email_heading', __( 'New course created.', 'woothemes-sensei' ), $template );
+
+		/**
+		 * Filter the email subject for the the
+		 * admin-teacher-new-course-created template
+		 *
+		 * @since 1.8.0
+		 * @param string $subject default New course assigned to you
+		 * @param string $template
+		 */
+		$subject = apply_filters('sensei_email_subject',
+								'['. get_bloginfo( 'name', 'display' ) .'] '. __( 'New course created by', 'woothemes-sensei' ) . ' ' . $teacher->display_name ,
+								$template );
+
+		//course edit link
+		$course_edit_link = admin_url('post.php?post=' . $course_id . '&action=edit' );
+
+		// Construct data array
+		$email_data = array(
+			'template'			=> $template,
+			'heading' =>  $heading,
+			'teacher'		=> $teacher,
+			'course_id'			=> $course_id,
+			'course_name'			=> $course->post_title,
+			'course_edit_link' => $course_edit_link,
+		);
+
+		/**
+		 * Filter the sensei email data for the admin-teacher-new-course-created template
+		 *
+		 * @since 1.8.0
+		 * @param array $email_data
+		 * @param string $template
+		 */
+		$sensei_email_data = apply_filters( 'sensei_email_data', $email_data , $template );
+
+		// Send mail
+		Sensei()->emails->send( $recipient, $subject , Sensei()->emails->get_content( $template ) );
+
+	}// end notify admin of course creation
+
+	/**
+	 * Limit the analysis view to only the users taking courses belong to this teacher
+	 *
+	 * Hooked into sensei_analysis_get_learners
+	 * @param array $args WP_User_Query arguments
+	 * @return array $learners_query_results
+	 */
+	public function limit_analysis_learners( $args ){
+
+		// show default for none teachers
+		if( ! Sensei()->teacher->is_admin_teacher() ) {
+				return $args;
+		}
+
+		// for teachers all courses only return those which belong to the teacher
+		// as they don't have access to course belonging to other users
+		$teacher_courses = Sensei()->course->get_all_courses();
 
-        // if the user has no courses they should see no users
-        if( empty( $teacher_courses ) ||  ! is_array( $teacher_courses ) ){
-            // tell the query to return 0 students
-            $args[ 'include'] = array( 0 );
-            return $args;
-
-        }
-
-        $learner_ids_for_teacher_courses = array();
-        foreach( $teacher_courses as $course ){
-
-            $course_learner_ids = array();
-            $activity_comments =  Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $course->ID, 'type' => 'sensei_course_status', 'field' => 'user_id' ), true );
-
-            if( empty( $activity_comments ) ||  ( is_array( $activity_comments  ) && ! ( count( $activity_comments ) > 0 ) ) ){
-                continue; // skip to the next course as there are no users on this course
-            }
-
-            // it could be an array of comments or a single comment
-            if( is_array( $activity_comments ) ){
+		// if the user has no courses they should see no users
+		if( empty( $teacher_courses ) ||  ! is_array( $teacher_courses ) ){
+			// tell the query to return 0 students
+			$args[ 'include'] = array( 0 );
+			return $args;
+
+		}
+
+		$learner_ids_for_teacher_courses = array();
+		foreach( $teacher_courses as $course ){
+
+			$course_learner_ids = array();
+			$activity_comments =  Sensei_Utils::sensei_check_for_activity( array( 'post_id' => $course->ID, 'type' => 'sensei_course_status', 'field' => 'user_id' ), true );
+
+			if( empty( $activity_comments ) ||  ( is_array( $activity_comments  ) && ! ( count( $activity_comments ) > 0 ) ) ){
+				continue; // skip to the next course as there are no users on this course
+			}
+
+			// it could be an array of comments or a single comment
+			if( is_array( $activity_comments ) ){
 
-                foreach( $activity_comments as $comment ){
+				foreach( $activity_comments as $comment ){
 
-                    $user = get_userdata( $comment->user_id );
+					$user = get_userdata( $comment->user_id );
 
-                    if( empty( $user ) ){
-                        // next comment in this array
-                        continue;
-                    }
+					if( empty( $user ) ){
+						// next comment in this array
+						continue;
+					}
 
-                    $course_learner_ids[] = $user->ID;
-                }
+					$course_learner_ids[] = $user->ID;
+				}
 
-            }else{
+			}else{
 
-                $user = get_userdata( $activity_comments->user_id );
-                $course_learner_ids[] = $user->ID;
+				$user = get_userdata( $activity_comments->user_id );
+				$course_learner_ids[] = $user->ID;
 
-            }
+			}
 
-            // add learners on this course to the all courses learner list
-            $learner_ids_for_teacher_courses = array_merge( $learner_ids_for_teacher_courses, $course_learner_ids );
+			// add learners on this course to the all courses learner list
+			$learner_ids_for_teacher_courses = array_merge( $learner_ids_for_teacher_courses, $course_learner_ids );
 
-        }
+		}
 
-        // if there are no students taking the courses by this teacher don't show them any of the other users
-        if( empty( $learner_ids_for_teacher_courses ) ){
+		// if there are no students taking the courses by this teacher don't show them any of the other users
+		if( empty( $learner_ids_for_teacher_courses ) ){
 
-            $args[ 'include'] = array( 0 );
+			$args[ 'include'] = array( 0 );
 
-        }else{
+		}else{
 
-            $args[ 'include'] = $learner_ids_for_teacher_courses;
+			$args[ 'include'] = $learner_ids_for_teacher_courses;
 
-        }
+		}
 
-        // return the WP_Use_Query arguments
-        return $args;
+		// return the WP_Use_Query arguments
+		return $args;
 
-    }// end limit_analysis_learners
+	}// end limit_analysis_learners
 
-    /**
-     * Give teacher full admin access to the question post type
-     * in certain cases.
-     *
-     * @since 1.8.0
-     * @param $questions
-     * @return mixed
-     */
-    public function allow_teacher_access_to_questions( $questions, $quiz_id ){
+	/**
+	 * Give teacher full admin access to the question post type
+	 * in certain cases.
+	 *
+	 * @since 1.8.0
+	 * @param $questions
+	 * @return mixed
+	 */
+	public function allow_teacher_access_to_questions( $questions, $quiz_id ){
 
-        if( ! $this->is_admin_teacher() ){
-            return $questions;
-        }
+		if( ! $this->is_admin_teacher() ){
+			return $questions;
+		}
 
-        $screen = get_current_screen();
+		$screen = get_current_screen();
 
-        // don't run this filter within this functions call to Sensei()->lesson->lesson_quiz_questions
-        remove_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20 );
+		// don't run this filter within this functions call to Sensei()->lesson->lesson_quiz_questions
+		remove_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20 );
 
-        if( ! empty( $screen ) && 'lesson'== $screen->post_type ){
+		if( ! empty( $screen ) && 'lesson'== $screen->post_type ){
 
-            $admin_user = get_user_by('email', get_bloginfo('admin_email'));
-            if( ! empty($admin_user) ){
+			$admin_user = get_user_by('email', get_bloginfo('admin_email'));
+			if( ! empty($admin_user) ){
 
-                $current_teacher_id = get_current_user_id();
+				$current_teacher_id = get_current_user_id();
 
-                // set current user to admin so teacher can view all questions
-                wp_set_current_user( $admin_user->ID  );
-                $questions = Sensei()->lesson->lesson_quiz_questions( $quiz_id  );
+				// set current user to admin so teacher can view all questions
+				wp_set_current_user( $admin_user->ID  );
+				$questions = Sensei()->lesson->lesson_quiz_questions( $quiz_id  );
 
-                // set the teacher as the current use again
-                wp_set_current_user( $current_teacher_id );
-            }
+				// set the teacher as the current use again
+				wp_set_current_user( $current_teacher_id );
+			}
 
-        }
-        // attach the filter again for other funtion calls to Sensei()->lesson->lesson_quiz_questions
-        add_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20,2 );
+		}
+		// attach the filter again for other funtion calls to Sensei()->lesson->lesson_quiz_questions
+		add_filter( 'sensei_lesson_quiz_questions', array( $this, 'allow_teacher_access_to_questions' ), 20,2 );
 
-        return $questions;
-    }
+		return $questions;
+	}
 
-    /**
-     * Give the teacher role access to questions from the question bank
-     *
-     * @since 1.8.0
-     * @param $wp_query
-     * @return mixed
-     */
-    public function give_access_to_all_questions( $wp_query ){
+	/**
+	 * Give the teacher role access to questions from the question bank
+	 *
+	 * @since 1.8.0
+	 * @param $wp_query
+	 * @return mixed
+	 */
+	public function give_access_to_all_questions( $wp_query ){
 
-        if( ! $this->is_admin_teacher() || !function_exists( 'get_current_screen') || 'question' != $wp_query->get('post_type') ){
+		if( ! $this->is_admin_teacher() || !function_exists( 'get_current_screen') || 'question' != $wp_query->get('post_type') ){
 
-            return $wp_query;
-        }
+			return $wp_query;
+		}
 
-        $screen = get_current_screen();
-        if( ( isset($screen->id) && 'lesson' == $screen->id )
-            || ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ){
+		$screen = get_current_screen();
+		if( ( isset($screen->id) && 'lesson' == $screen->id )
+			|| ( defined( 'DOING_AJAX' ) && DOING_AJAX ) ){
 
-            $admin_user = get_user_by('email', get_bloginfo('admin_email'));
-            if( ! empty($admin_user) ){
+			$admin_user = get_user_by('email', get_bloginfo('admin_email'));
+			if( ! empty($admin_user) ){
 
-                $current_teacher_id = get_current_user_id();
+				$current_teacher_id = get_current_user_id();
 
-                // set current user to admin so teacher can view all questions
-                wp_set_current_user( $admin_user->ID  );
+				// set current user to admin so teacher can view all questions
+				wp_set_current_user( $admin_user->ID  );
 
-                //run new query as admin
-                $wp_query = new WP_Query( $wp_query->query );
+				//run new query as admin
+				$wp_query = new WP_Query( $wp_query->query );
 
-                //set the teache as current use again
-                wp_set_current_user( $current_teacher_id );
+				//set the teache as current use again
+				wp_set_current_user( $current_teacher_id );
 
-            }
-        }
+			}
+		}
 
-        return $wp_query;
-    }// end give_access_to_all_questions
+		return $wp_query;
+	}// end give_access_to_all_questions
 
-    /**
-     * Add new column heading to the course admin edit list
-     *
-     * @since 1.8.0
-     * @param $columns
-     * @return array
-     */
-    public function course_column_heading($columns) {
+	/**
+	 * Add new column heading to the course admin edit list
+	 *
+	 * @since 1.8.0
+	 * @param $columns
+	 * @return array
+	 */
+	public function course_column_heading($columns) {
 
-        if( $this->is_admin_teacher() ){
-            return $columns;
-        }
-        $new_columns = array(
-            'teacher' => __('Teacher', 'woothemes-sensei'),
-        );
-        return array_merge($columns, $new_columns);
+		if( $this->is_admin_teacher() ){
+			return $columns;
+		}
+		$new_columns = array(
+			'teacher' => __('Teacher', 'woothemes-sensei'),
+		);
+		return array_merge($columns, $new_columns);
 
-    }// end teacher column add
+	}// end teacher column add
 
-    /**
-     * Print out  teacher column data
-     *
-     * @since 1.8.0
-     * @param $column
-     * @param $course_id
-     */
-    public function course_column_data( $column, $course_id  ){
+	/**
+	 * Print out  teacher column data
+	 *
+	 * @since 1.8.0
+	 * @param $column
+	 * @param $course_id
+	 */
+	public function course_column_data( $column, $course_id  ){
 
-        if( $this->is_admin_teacher() || 'teacher' != $column  ){
-            return;
-        }
+		if( $this->is_admin_teacher() || 'teacher' != $column  ){
+			return;
+		}
 
-        $course = get_post( $course_id );
-        $teacher = get_userdata( $course->post_author );
+		$course = get_post( $course_id );
+		$teacher = get_userdata( $course->post_author );
 
-        if( !$teacher ){
-            return;
-        }
+		if( !$teacher ){
+			return;
+		}
 
-        echo '<a href="'. get_edit_user_link( $teacher->ID ) .'" >'. $teacher->display_name.'</a>';
+		echo '<a href="'. get_edit_user_link( $teacher->ID ) .'" >'. $teacher->display_name.'</a>';
 
-    }// end course_column_ data
+	}// end course_column_ data
 
-    /**
-     * Return only courses belonging to the given teacher.
-     *
-     *
-     * @since 1.8.0
-     *
-     * @param int $teacher_id
-     * @param bool $return_ids_only
-     *
-     * @return array $teachers_courses
-     */
-    public function get_teacher_courses( $teacher_id, $return_ids_only= false){
+	/**
+	 * Return only courses belonging to the given teacher.
+	 *
+	 *
+	 * @since 1.8.0
+	 *
+	 * @param int $teacher_id
+	 * @param bool $return_ids_only
+	 *
+	 * @return array $teachers_courses
+	 */
+	public function get_teacher_courses( $teacher_id, $return_ids_only= false){
 
-        $teachers_courses = array();
+		$teachers_courses = array();
 
-        if( empty( $teacher_id  ) ){
-            $teacher_id = get_current_user_id();
-        }
+		if( empty( $teacher_id  ) ){
+			$teacher_id = get_current_user_id();
+		}
 
-        $all_courses = Sensei()->course->get_all_courses();
+		$all_courses = Sensei()->course->get_all_courses();
 
-        if( empty( $all_courses ) ){
-            return $all_courses;
-        }
+		if( empty( $all_courses ) ){
+			return $all_courses;
+		}
 
-        foreach( $all_courses as $course ){
+		foreach( $all_courses as $course ){
 
-            if( $course->post_author != $teacher_id  ){
-                continue;
-            }
+			if( $course->post_author != $teacher_id  ){
+				continue;
+			}
 
-            if( $return_ids_only ){
+			if( $return_ids_only ){
 
-                $teachers_courses[] = $course->ID;
+				$teachers_courses[] = $course->ID;
 
-            }else{
+			}else{
 
-                $teachers_courses[] = $course;
+				$teachers_courses[] = $course;
 
-            }
+			}
 
-        }
+		}
 
-        return $teachers_courses;
+		return $teachers_courses;
 
-    }
+	}
 
-    /**
-     * Limit the message display to only those sent to the current teacher
-     *
-     * @since 1.8.0
-     *
-     * @param $query
-     * @return mixed
-     */
-    public function limit_edit_messages_query( $query ){
-        if( ! $this->is_admin_teacher() || 'sensei_message' != $query->get('post_type') ){
-            return $query;
-        }
-
-        $teacher = wp_get_current_user();
-
-        $query->set( 'meta_key', '_receiver' );
-        $meta_query_args = array(
-            'key'     => '_receiver',
-            'value'   => $teacher->get('user_login') ,
-            'compare' => '='
-        );
+	/**
+	 * Limit the message display to only those sent to the current teacher
+	 *
+	 * @since 1.8.0
+	 *
+	 * @param $query
+	 * @return mixed
+	 */
+	public function limit_edit_messages_query( $query ){
+		if( ! $this->is_admin_teacher() || 'sensei_message' != $query->get('post_type') ){
+			return $query;
+		}
+
+		$teacher = wp_get_current_user();
+
+		$query->set( 'meta_key', '_receiver' );
+		$meta_query_args = array(
+			'key'     => '_receiver',
+			'value'   => $teacher->get('user_login') ,
+			'compare' => '='
+		);
 
-        $query->set('meta_query', $meta_query_args  );
+		$query->set('meta_query', $meta_query_args  );
 
-        return $query;
-    }
+		return $query;
+	}
 
 
-    /**
-     * Add options to filter courses by teacher
-     *
-     * @since 1.8.0
-     *
-     * @return void
-     */
-    public function course_teacher_filter_options() {
-        global $typenow;
+	/**
+	 * Add options to filter courses by teacher
+	 *
+	 * @since 1.8.0
+	 *
+	 * @return void
+	 */
+	public function course_teacher_filter_options() {
+		global $typenow;
 
-        if( ! is_admin() || 'course' != $typenow || ! current_user_can('manage_sensei') ) {
-            return;
-        }
+		if( ! is_admin() || 'course' != $typenow || ! current_user_can('manage_sensei') ) {
+			return;
+		}
 
-        // get all roles
-        $roles = get_editable_roles();
+		// get all roles
+		$roles = get_editable_roles();
 
-        // get roles with the course edit capability
-        // and then get the users with those roles
-        $users_who_can_edit_courses = array();
-        foreach( $roles as $role_item ){
+		// get roles with the course edit capability
+		// and then get the users with those roles
+		$users_who_can_edit_courses = array();
+		foreach( $roles as $role_item ){
 
-            $role = get_role( strtolower( $role_item['name'] ) );
+			$role = get_role( strtolower( $role_item['name'] ) );
 
-            if( is_a( $role, 'WP_Role' ) && $role->has_cap('edit_courses') ){
+			if( is_a( $role, 'WP_Role' ) && $role->has_cap('edit_courses') ){
 
-                $user_query_args = array( 'role' => $role->name, 'fields' => array( 'ID', 'display_name' ) );
-                $role_users_who_can_edit_courses = get_users( $user_query_args );
+				$user_query_args = array( 'role' => $role->name, 'fields' => array( 'ID', 'display_name' ) );
+				$role_users_who_can_edit_courses = get_users( $user_query_args );
 
-                // add user from the current $user_role to all users
-                $users_who_can_edit_courses = array_merge( $users_who_can_edit_courses, $role_users_who_can_edit_courses );
+				// add user from the current $user_role to all users
+				$users_who_can_edit_courses = array_merge( $users_who_can_edit_courses, $role_users_who_can_edit_courses );
 
-            }
+			}
 
-        }
+		}
 
-        // Create the select element with the given users who can edit course
-        $selected = isset( $_GET['course_teacher'] ) ? $_GET['course_teacher'] : '';
-        $course_options = '';
-        foreach( $users_who_can_edit_courses as $user ) {
-            $course_options .= '<option value="' . esc_attr( $user->ID ) . '" ' . selected( $selected, $user->ID, false ) . '>' .  $user->display_name . '</option>';
-        }
+		// Create the select element with the given users who can edit course
+		$selected = isset( $_GET['course_teacher'] ) ? $_GET['course_teacher'] : '';
+		$course_options = '';
+		foreach( $users_who_can_edit_courses as $user ) {
+			$course_options .= '<option value="' . esc_attr( $user->ID ) . '" ' . selected( $selected, $user->ID, false ) . '>' .  $user->display_name . '</option>';
+		}
 
-        $output = '<select name="course_teacher" id="dropdown_course_teachers">';
-        $output .= '<option value="">'.__( 'Show all teachers', 'woothemes-sensei' ).'</option>';
-        $output .= $course_options;
-        $output .= '</select>';
+		$output = '<select name="course_teacher" id="dropdown_course_teachers">';
+		$output .= '<option value="">'.__( 'Show all teachers', 'woothemes-sensei' ).'</option>';
+		$output .= $course_options;
+		$output .= '</select>';
 
-        echo $output;
-    }
+		echo $output;
+	}
 
-    /**
-     * Modify the main query on the admin course list screen
-     *
-     * @since 1.8.0
-     *
-     * @param $query
-     * @return $query
-     */
-    public function teacher_filter_query_modify( $query ){
-        global $typenow;
+	/**
+	 * Modify the main query on the admin course list screen
+	 *
+	 * @since 1.8.0
+	 *
+	 * @param $query
+	 * @return $query
+	 */
+	public function teacher_filter_query_modify( $query ){
+		global $typenow;
 
-        if( ! is_admin() && 'course' != $typenow  || ! current_user_can('manage_sensei')  ) {
-            return $query;
-        }
-        $course_teacher = isset( $_GET['course_teacher'] ) ? $_GET['course_teacher'] : '';
-
-        if( empty( $course_teacher ) ) {
-            return $query;
-        }
-
-        $query['author'] = $course_teacher;
-        return $query;
-    }
-
-    /**
-     * Only show current teacher's media in the media library
-     * @param  array $request Default request arguments
-     * @return array          Modified request arguments
-     */
-    public function restrict_media_library( $request = array() ) {
-
-        if( ! is_admin() ) {
-            return $request;
-        }
-
-        if( ! $this->is_admin_teacher() ) {
-            return $request;
-        }
-
-        $screen = get_current_screen();
-
-        if( in_array( $screen->id, array( 'upload', 'course', 'lesson', 'question' ) ) ) {
-            $teacher = intval( get_current_user_id() );
-
-            if( $teacher ) {
-                $request['author__in'] = array( $teacher );
-            }
-        }
-
-        return $request;
-    } // End restrict_media_library()
-
-    /**
-     * Only show current teacher's media in the media library modal on the course/lesson/quesion edit screen
-     * @param  array $query Default query arguments
-     * @return array        Modified query arguments
-     */
-    public function restrict_media_library_modal( $query = array() ) {
-
-        if( ! is_admin() ) {
-            return $query;
-        }
-
-        if( ! $this->is_admin_teacher() ) {
-            return $query;
-        }
-
-        $teacher = intval( get_current_user_id() );
-
-        if( $teacher ) {
-            $query['author__in'] = array( $teacher );
-        }
-
-        return $query;
-    } // End restrict_media_library_modal()
-
-    /**
-     * When saving the lesson, update the teacher if the lesson belongs to a course
-     *
-     * @since 1.8.0
-     *
-     * @param int $lesson_id
-     */
-    public function update_lesson_teacher( $lesson_id ){
-
-        if( 'lesson'!= get_post_type() ){
-            return;
-        }
+		if( ! is_admin() && 'course' != $typenow  || ! current_user_can('manage_sensei')  ) {
+			return $query;
+		}
+		$course_teacher = isset( $_GET['course_teacher'] ) ? $_GET['course_teacher'] : '';
+
+		if( empty( $course_teacher ) ) {
+			return $query;
+		}
+
+		$query['author'] = $course_teacher;
+		return $query;
+	}
+
+	/**
+	 * Only show current teacher's media in the media library
+	 * @param  array $request Default request arguments
+	 * @return array          Modified request arguments
+	 */
+	public function restrict_media_library( $request = array() ) {
+
+		if( ! is_admin() ) {
+			return $request;
+		}
+
+		if( ! $this->is_admin_teacher() ) {
+			return $request;
+		}
+
+		$screen = get_current_screen();
+
+		if( in_array( $screen->id, array( 'upload', 'course', 'lesson', 'question' ) ) ) {
+			$teacher = intval( get_current_user_id() );
+
+			if( $teacher ) {
+				$request['author__in'] = array( $teacher );
+			}
+		}
+
+		return $request;
+	} // End restrict_media_library()
+
+	/**
+	 * Only show current teacher's media in the media library modal on the course/lesson/quesion edit screen
+	 * @param  array $query Default query arguments
+	 * @return array        Modified query arguments
+	 */
+	public function restrict_media_library_modal( $query = array() ) {
+
+		if( ! is_admin() ) {
+			return $query;
+		}
+
+		if( ! $this->is_admin_teacher() ) {
+			return $query;
+		}
+
+		$teacher = intval( get_current_user_id() );
+
+		if( $teacher ) {
+			$query['author__in'] = array( $teacher );
+		}
+
+		return $query;
+	} // End restrict_media_library_modal()
+
+	/**
+	 * When saving the lesson, update the teacher if the lesson belongs to a course
+	 *
+	 * @since 1.8.0
+	 *
+	 * @param int $lesson_id
+	 */
+	public function update_lesson_teacher( $lesson_id ){
+
+		if( 'lesson'!= get_post_type() ){
+			return;
+		}
 
-        // this should only run once per request cycle
-        remove_action( 'save_post',  array( $this, 'update_lesson_teacher' ) );
+		// this should only run once per request cycle
+		remove_action( 'save_post',  array( $this, 'update_lesson_teacher' ) );
 
-        $course_id = Sensei()->lesson->get_course_id( $lesson_id );
+		$course_id = Sensei()->lesson->get_course_id( $lesson_id );
 
-        if(  empty( $course_id ) || ! $course_id ){
-            return;
-        }
+		if(  empty( $course_id ) || ! $course_id ){
+			return;
+		}
 
-        $course = get_post( $course_id );
+		$course = get_post( $course_id );
 
-        $lesson_update_args= array(
-            'ID' => $lesson_id ,
-            'post_author' => $course->post_author
-        );
-        wp_update_post( $lesson_update_args );
+		$lesson_update_args= array(
+			'ID' => $lesson_id ,
+			'post_author' => $course->post_author
+		);
+		wp_update_post( $lesson_update_args );
 
-    } // end update_lesson_teacher
+	} // end update_lesson_teacher
 
-    /**
-     * Sensei_Teacher::limit_teacher_edit_screen_post_types
-     *
-     * Limit teachers to only see their courses, lessons and questions
-     *
-     * @since 1.8.0
-     * @access public
-     * @parameters array $wp_query
-     * @return WP_Query $wp_query
-     */
-    public function limit_teacher_edit_screen_post_types( $wp_query ) {
-        global $current_user;
+	/**
+	 * Sensei_Teacher::limit_teacher_edit_screen_post_types
+	 *
+	 * Limit teachers to only see their courses, lessons and questions
+	 *
+	 * @since 1.8.0
+	 * @access public
+	 * @parameters array $wp_query
+	 * @return WP_Query $wp_query
+	 */
+	public function limit_teacher_edit_screen_post_types( $wp_query ) {
+		global $current_user;
 
-        //exit early
-        if( ! $this->is_admin_teacher() ){
-            return $wp_query;
-        }
+		//exit early
+		if( ! $this->is_admin_teacher() ){
+			return $wp_query;
+		}
 
-        if ( ! function_exists( 'get_current_screen' ) ) {
-            return $wp_query;
-        }
+		if ( ! function_exists( 'get_current_screen' ) ) {
+			return $wp_query;
+		}
 
-        $screen = get_current_screen();
+		$screen = get_current_screen();
 
-        if( empty( $screen ) ){
-            return $wp_query;
-        }
+		if( empty( $screen ) ){
+			return $wp_query;
+		}
 
-        // for any of these conditions limit what the teacher will see
-        $limit_screens = array(
-            'edit-lesson',
-            'edit-course',
-            'edit-question',
-            'course_page_course-order',
-            'lesson_page_lesson-order',
-        );
+		// for any of these conditions limit what the teacher will see
+		$limit_screens = array(
+			'edit-lesson',
+			'edit-course',
+			'edit-question',
+			'course_page_course-order',
+			'lesson_page_lesson-order',
+		);
 
-        if(  in_array($screen->id  , $limit_screens ) ) {
+		if(  in_array($screen->id  , $limit_screens ) ) {
 
-            // set the query author to the current user to only show those those posts
-            $wp_query->set( 'author', $current_user->ID );
-        }
+			// set the query author to the current user to only show those those posts
+			$wp_query->set( 'author', $current_user->ID );
+		}
 
-        return $wp_query;
+		return $wp_query;
 
-    } // end limit_teacher_edit_screen_post_types()
+	} // end limit_teacher_edit_screen_post_types()
 
 
-    /**
-     * Sensei_Teacher::teacher_login_redirect
-     *
-     * Redirect teachers to /wp-admin/ after login
-     *
-     * @since 1.8.7
-     * @access public
-     * @param string $user_login
-     * @param object $user
-     * @return void
-     */
+	/**
+	 * Sensei_Teacher::teacher_login_redirect
+	 *
+	 * Redirect teachers to /wp-admin/ after login
+	 *
+	 * @since 1.8.7
+	 * @access public
+	 * @param string $user_login
+	 * @param object $user
+	 * @return void
+	 */
 
-    public function teacher_login_redirect( $user_login, $user  ) {
+	public function teacher_login_redirect( $user_login, $user  ) {
 
-        if (user_can($user, 'edit_courses')) {
+		if (user_can($user, 'edit_courses')) {
 
-            if (isset($_POST['redirect_to'])) {
+			if (isset($_POST['redirect_to'])) {
 
-                wp_redirect($_POST['redirect_to'], 303);
+				wp_redirect($_POST['redirect_to'], 303);
 
-                exit;
+				exit;
 
-            } else {
+			} else {
 
-                wp_redirect(admin_url(), 303);
+				wp_redirect(admin_url(), 303);
 
-                exit;
+				exit;
 
-            }
-        }
+			}
+		}
 
-    } // end teacher_login_redirect()
+	} // end teacher_login_redirect()
 
 
 
-    /**
-     * Sensei_Teacher::restrict_posts_menu_page()
-     *
-     * Remove the Posts menu page for teachers and restrict access to it.
-     * We have to do this because we give teachers the 'edit_posts' cap
-     * so they can 'moderate_comments' as well.
-     *
-     * @since 1.8.7
-     * @access public
-     * @parameters void
-     * @return void
-     */
+	/**
+	 * Sensei_Teacher::restrict_posts_menu_page()
+	 *
+	 * Remove the Posts menu page for teachers and restrict access to it.
+	 * We have to do this because we give teachers the 'edit_posts' cap
+	 * so they can 'moderate_comments' as well.
+	 *
+	 * @since 1.8.7
+	 * @access public
+	 * @parameters void
+	 * @return void
+	 */
 
-    public function restrict_posts_menu_page() {
+	public function restrict_posts_menu_page() {
 
-        global $pagenow, $typenow;
+		global $pagenow, $typenow;
 
-        $user = wp_get_current_user();
+		$user = wp_get_current_user();
 
-        /**
-         * Filter the option to hide the Posts menu page.
-         *
-         * @since 1.8.7
-         *
-         * @param bool $restrict default true
-         */
+		/**
+		 * Filter the option to hide the Posts menu page.
+		 *
+		 * @since 1.8.7
+		 *
+		 * @param bool $restrict default true
+		 */
 
-        $restrict = apply_filters('sensei_restrict_posts_menu_page', true );
+		$restrict = apply_filters('sensei_restrict_posts_menu_page', true );
 
-        if ( in_array( 'teacher', (array) $user->roles ) && !current_user_can('delete_posts') && $restrict) {
+		if ( in_array( 'teacher', (array) $user->roles ) && !current_user_can('delete_posts') && $restrict) {
 
-            remove_menu_page('edit.php');
+			remove_menu_page('edit.php');
 
-            if ($pagenow == "edit.php" || $pagenow == "post-new.php") {
+			if ($pagenow == "edit.php" || $pagenow == "post-new.php") {
 
-                if ($typenow == '' || $typenow == 'post' || $typenow == 'page') {
+				if ($typenow == '' || $typenow == 'post' || $typenow == 'page') {
 
-                    wp_die('You do not have sufficient permissions to access this page.');
+					wp_die('You do not have sufficient permissions to access this page.');
 
-                }
+				}
 
-            }
+			}
 
-        }
+		}
 
-    } // end restrict_posts_menu_page()
+	} // end restrict_posts_menu_page()
 
-    /**
-     * Sensei_Teacher::restrict_comment_moderation()
-     *
-     * Restrict commendation moderation for teachers
-     * so they can only moderate comments made to posts they own.
-     *
-     * @since 1.8.7
-     * @access public
-     * @parameters obj $clauses
-     * @return WP_Comment_Query  $clauses
-     */
+	/**
+	 * Sensei_Teacher::restrict_comment_moderation()
+	 *
+	 * Restrict commendation moderation for teachers
+	 * so they can only moderate comments made to posts they own.
+	 *
+	 * @since 1.8.7
+	 * @access public
+	 * @parameters obj $clauses
+	 * @return WP_Comment_Query  $clauses
+	 */
 
-    public function restrict_comment_moderation ( $clauses ) {
+	public function restrict_comment_moderation ( $clauses ) {
 
-        global $pagenow;
+		global $pagenow;
 
-        if( self::is_a_teacher( get_current_user_id() ) && $pagenow == "edit-comments.php") {
+		if( self::is_a_teacher( get_current_user_id() ) && $pagenow == "edit-comments.php") {
 
-            $clauses->query_vars['post_author'] = get_current_user_id();
+			$clauses->query_vars['post_author'] = get_current_user_id();
 
-        }
+		}
 
-        return $clauses;
+		return $clauses;
 
-    }   // end restrict_comment_moderation()
+	}   // end restrict_comment_moderation()
 
-    /**
-     * Determine if a user is a teacher by ID
-     *
-     * @param int $user_id
-     *
-     * @return bool
-     */
-    public static function is_a_teacher( $user_id ){
+	/**
+	 * Determine if a user is a teacher by ID
+	 *
+	 * @param int $user_id
+	 *
+	 * @return bool
+	 */
+	public static function is_a_teacher( $user_id ){
 
-        $user = get_user_by('id', $user_id);
+		$user = get_user_by('id', $user_id);
 
-        if( isset( $user->roles ) && in_array(  'teacher',  $user->roles )   ){
+		if( isset( $user->roles ) && in_array(  'teacher',  $user->roles )   ){
 
-            return true;
+			return true;
 
-        }else{
+		}else{
 
-            return false;
+			return false;
 
-        }
+		}
 
-    }// end is_a_teacher
+	}// end is_a_teacher
 
-    /**
-     * The archive title on the teacher archive filter
-     *
-     * @since 1.9.0
-     */
-    public static function archive_title(){
+	/**
+	 * The archive title on the teacher archive filter
+	 *
+	 * @since 1.9.0
+	 */
+	public static function archive_title(){
 
-        $author = get_user_by( 'id', get_query_var( 'author' ) );
-        $author_name = $author->display_name;
-        ?>
+		$author = get_user_by( 'id', get_query_var( 'author' ) );
+		$author_name = $author->display_name;
+		?>
             <h2 class="teacher-archive-title">
 
                 <?php echo sprintf( __( 'All courses by %s', 'woothemes-sensei') , $author_name ); ?>
@@ -1557,17 +1557,17 @@ 
             </h2>
         <?php
 
-    }// archive title
+	}// archive title
 
-    /**
-     * Removing course meta on the teacher archive page
-     *
-     * @since 1.9.0
-     */
-    public static function remove_course_meta_on_teacher_archive(){
+	/**
+	 * Removing course meta on the teacher archive page
+	 *
+	 * @since 1.9.0
+	 */
+	public static function remove_course_meta_on_teacher_archive(){
 
-        remove_action('sensei_course_content_inside_before', array( Sensei()->course, 'the_course_meta' ) );
+		remove_action('sensei_course_content_inside_before', array( Sensei()->course, 'the_course_meta' ) );
 
-    }
+	}
 
 } // End Class