woothemes / sensei

includes/shortcodes/class-sensei-shortcode-teachers.php

Indentation +191 added lines, -191 removed lines

@@ -17,270 +17,270 @@
  */
 class Sensei_Shortcode_Teachers implements Sensei_Shortcode_Interface {
 
-    /**
-     * @var WP_User_Query keeps a reference to the user query created
-     */
-    protected $user_query;
+	/**
+	 * @var WP_User_Query keeps a reference to the user query created
+	 */
+	protected $user_query;
 
-    /**
-     * @var which user id's to include
-     */
-    protected $include;
+	/**
+	 * @var which user id's to include
+	 */
+	protected $include;
 
-    /**
-     * @var which user id's to exclude
-     */
-    protected $exclude;
+	/**
+	 * @var which user id's to exclude
+	 */
+	protected $exclude;
 
-    /**
-     * Setup the shortcode object
-     *
-     * @since 1.9.0
-     * @param array $attributes
-     * @param string $content
-     * @param string $shortcode the shortcode that was called for this instance
-     */
-    public function __construct( $attributes, $content, $shortcode ){
+	/**
+	 * Setup the shortcode object
+	 *
+	 * @since 1.9.0
+	 * @param array $attributes
+	 * @param string $content
+	 * @param string $shortcode the shortcode that was called for this instance
+	 */
+	public function __construct( $attributes, $content, $shortcode ){
 
-        $include = isset( $attributes['include'] ) ? explode( ',', $attributes['include'] ) : '';
-        $exclude = isset( $attributes['exclude'] ) ? explode( ',', $attributes['exclude'] ) : '';
+		$include = isset( $attributes['include'] ) ? explode( ',', $attributes['include'] ) : '';
+		$exclude = isset( $attributes['exclude'] ) ? explode( ',', $attributes['exclude'] ) : '';
 
-        // convert teacher usernames given to the id
-        $this->include = $this->convert_usernames_to_ids( $include );
-        $this->exclude = $this->convert_usernames_to_ids( $exclude );
+		// convert teacher usernames given to the id
+		$this->include = $this->convert_usernames_to_ids( $include );
+		$this->exclude = $this->convert_usernames_to_ids( $exclude );
 
-        $this->setup_teacher_query();
+		$this->setup_teacher_query();
 
-    }
+	}
 
-    /**
-     *
-     * Setup the user query that will be used in the render method
-     *
-     * @since 1.9.0
-     */
-    protected function setup_teacher_query(){
+	/**
+	 *
+	 * Setup the user query that will be used in the render method
+	 *
+	 * @since 1.9.0
+	 */
+	protected function setup_teacher_query(){
 
-        $user_query_args = array(
-            'role' => 'teacher',
-        );
+		$user_query_args = array(
+			'role' => 'teacher',
+		);
 
-        $this->user_query = new WP_User_Query( $user_query_args );
+		$this->user_query = new WP_User_Query( $user_query_args );
 
-    }// end setup _course_query
+	}// end setup _course_query
 
-    /**
-     * Rendering the shortcode this class is responsible for.
-     *
-     * @return string $content
-     */
-    public function render(){
+	/**
+	 * Rendering the shortcode this class is responsible for.
+	 *
+	 * @return string $content
+	 */
+	public function render(){
 
-        $all_users = $this->user_query->get_results();
-        // if the user has specified more users add them as well.
-        if( ! empty( $this->include ) ){
+		$all_users = $this->user_query->get_results();
+		// if the user has specified more users add them as well.
+		if( ! empty( $this->include ) ){
 
-            $included_users_query = new WP_User_Query( array( 'include' => $this->include ) );
-            $included_users = $included_users_query->get_results();
-            if( ! empty( $included_users ) ){
+			$included_users_query = new WP_User_Query( array( 'include' => $this->include ) );
+			$included_users = $included_users_query->get_results();
+			if( ! empty( $included_users ) ){
 
-                $merged_users = array_merge( $all_users, $included_users );
-                $all_users = $this->users_unique( $merged_users );
-                $all_users = $this->users_sort( $all_users );
+				$merged_users = array_merge( $all_users, $included_users );
+				$all_users = $this->users_unique( $merged_users );
+				$all_users = $this->users_sort( $all_users );
 
-            }
+			}
 
-        }
+		}
 
-        // exclude the users not wanted
-        if( ! empty( $this->exclude ) ){
+		// exclude the users not wanted
+		if( ! empty( $this->exclude ) ){
 
-            $all_users = $this->exclude_users( $all_users, $this->exclude );
+			$all_users = $this->exclude_users( $all_users, $this->exclude );
 
-        }
+		}
 
-        if( ! count( $all_users )> 0  ){
-            return '';
-        }
+		if( ! count( $all_users )> 0  ){
+			return '';
+		}
 
 
-        $users_output = '';
+		$users_output = '';
 
-        foreach ( $all_users as $user ) {
+		foreach ( $all_users as $user ) {
 
-            $user_display_name = $this->get_user_public_name( $user );
+			$user_display_name = $this->get_user_public_name( $user );
 
-            /**
-             * Sensei teachers shortcode list item filter
-             *
-             * @since 1.9.0
-             *
-             * @param string $teacher_li the html for the teacher li
-             * @param WP_User $user
-             */
-            $users_output .= apply_filters( 'sensei_teachers_shortcode_list_item', '<li class="teacher"><a href="'. get_author_posts_url( $user->ID ) . '">'. $user_display_name .  '<a/></li>', $user );
+			/**
+			 * Sensei teachers shortcode list item filter
+			 *
+			 * @since 1.9.0
+			 *
+			 * @param string $teacher_li the html for the teacher li
+			 * @param WP_User $user
+			 */
+			$users_output .= apply_filters( 'sensei_teachers_shortcode_list_item', '<li class="teacher"><a href="'. get_author_posts_url( $user->ID ) . '">'. $user_display_name .  '<a/></li>', $user );
 
-        }
+		}
 
-        return '<ul class="sensei-teachers">' . $users_output . '</ul>';
+		return '<ul class="sensei-teachers">' . $users_output . '</ul>';
 
-    }// end render
+	}// end render
 
-    /**
-     * remove duplicate user objects from and array of users
-     *
-     * @since 1.9.0
-     *
-     * @param array $users{
-     *   @type WP_User
-     * }
-     *
-     * @return array $unique_users {
-     *   @type WP_User
-     * }
-     */
-    public  function users_unique( $users ){
+	/**
+	 * remove duplicate user objects from and array of users
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param array $users{
+	 *   @type WP_User
+	 * }
+	 *
+	 * @return array $unique_users {
+	 *   @type WP_User
+	 * }
+	 */
+	public  function users_unique( $users ){
 
-        $array_unique_users_ids = array();
-        foreach( $users as $index => $user ){
+		$array_unique_users_ids = array();
+		foreach( $users as $index => $user ){
 
-            if(  in_array( $user->ID,  $array_unique_users_ids)  ){
+			if(  in_array( $user->ID,  $array_unique_users_ids)  ){
 
-                // exclude this user as it is already in the list
-                unset( $users[ $index ] );
+				// exclude this user as it is already in the list
+				unset( $users[ $index ] );
 
-            }else{
+			}else{
 
-                // add teh user to the list of users
-                $array_unique_users_ids[] = $user->ID;
+				// add teh user to the list of users
+				$array_unique_users_ids[] = $user->ID;
 
-            }
+			}
 
-        }
+		}
 
-        return $users;
+		return $users;
 
-    }// end users_unique
+	}// end users_unique
 
-    /**
-     * Exclude users based ont he ids given.
-     *
-     * @since 1.9.0
-     *
-     * @param array $users
-     * @param array $exclude_ids
-     * @return array
-     */
-    public function exclude_users( $users, $exclude_ids ){
+	/**
+	 * Exclude users based ont he ids given.
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param array $users
+	 * @param array $exclude_ids
+	 * @return array
+	 */
+	public function exclude_users( $users, $exclude_ids ){
 
-        foreach( $users as $index => $user ){
+		foreach( $users as $index => $user ){
 
-            if( in_array( $user->ID, $exclude_ids )  ){
+			if( in_array( $user->ID, $exclude_ids )  ){
 
-                // remove the user from the list
-                unset( $users[ $index ] );
+				// remove the user from the list
+				unset( $users[ $index ] );
 
-            }
+			}
 
-        }
+		}
 
-        return $users;
+		return $users;
 
-    }// end exclude_users
+	}// end exclude_users
 
-    /**
-     * Convert mixed array of user id and user names to only be an array of user_ids
-     *
-     * @param array $users
-     * @return array $users_ids
-     */
-    public function convert_usernames_to_ids( $users ){
+	/**
+	 * Convert mixed array of user id and user names to only be an array of user_ids
+	 *
+	 * @param array $users
+	 * @return array $users_ids
+	 */
+	public function convert_usernames_to_ids( $users ){
 
-        // backup
-        $users_ids = array();
+		// backup
+		$users_ids = array();
 
-        if ( is_array($users) ) {
+		if ( is_array($users) ) {
 
-            foreach ($users as $user_id_or_username) {
+			foreach ($users as $user_id_or_username) {
 
-                if (!is_numeric($user_id_or_username)) {
+				if (!is_numeric($user_id_or_username)) {
 
-                    $user_name = $user_id_or_username;
-                    $user = get_user_by('login', $user_name);
+					$user_name = $user_id_or_username;
+					$user = get_user_by('login', $user_name);
 
-                    if (is_a($user, 'WP_User')) {
-                        $users_ids[] = $user->ID;
-                    }
+					if (is_a($user, 'WP_User')) {
+						$users_ids[] = $user->ID;
+					}
 
-                } else {
+				} else {
 
-                    $user_id = $user_id_or_username;
-                    $users_ids[] = $user_id;
+					$user_id = $user_id_or_username;
+					$users_ids[] = $user_id;
 
-                }
+				}
 
-            }
-        }
+			}
+		}
 
-        return $users_ids;
-    }
+		return $users_ids;
+	}
 
-    /**
-     * Returns the first name and last name or the display name of a user.
-     *
-     * @since 1.9.0
-     *
-     * @param $user
-     * @return string $user_public_name
-     */
-    public function get_user_public_name( $user ){
+	/**
+	 * Returns the first name and last name or the display name of a user.
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param $user
+	 * @return string $user_public_name
+	 */
+	public function get_user_public_name( $user ){
 
-        if (!empty($user->first_name) && !empty($user->last_name)) {
+		if (!empty($user->first_name) && !empty($user->last_name)) {
 
-            $user_public_name = $user->first_name . ' ' . $user->last_name;
+			$user_public_name = $user->first_name . ' ' . $user->last_name;
 
-        }
+		}
 
-        else {
+		else {
 
-            $user_public_name = $user->display_name;
+			$user_public_name = $user->display_name;
 
-        }
+		}
 
-        return $user_public_name;
-    }
+		return $user_public_name;
+	}
 
-    /**
-     *
-     * Sort user objects by user display
-     *
-     * @since 1.9.0
-     *
-     * @param $users
-     * @return  array $sorted_users
-     */
-    public function users_sort( $users ){
+	/**
+	 *
+	 * Sort user objects by user display
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param $users
+	 * @return  array $sorted_users
+	 */
+	public function users_sort( $users ){
 
-        $sorted_users = $users;
+		$sorted_users = $users;
 
-        uasort( $sorted_users, array( $this, 'custom_user_sort' ) );
+		uasort( $sorted_users, array( $this, 'custom_user_sort' ) );
 
-        return $sorted_users;
-    }
+		return $sorted_users;
+	}
 
-    /**
-     * Used in the uasort function to sort users by title
-     *
-     * @since 1.9.0
-     *
-     * @param $user_1
-     * @param $user_2
-     * @return int
-     */
-    public function custom_user_sort($user_1, $user_2){
+	/**
+	 * Used in the uasort function to sort users by title
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param $user_1
+	 * @param $user_2
+	 * @return int
+	 */
+	public function custom_user_sort($user_1, $user_2){
 
-        return strcasecmp( $this->get_user_public_name( $user_1 ), $this->get_user_public_name( $user_2 )  );
+		return strcasecmp( $this->get_user_public_name( $user_1 ), $this->get_user_public_name( $user_2 )  );
 
-    }// end custom_user_sort
+	}// end custom_user_sort
 
 }// end class
\ No newline at end of file

includes/shortcodes/class-sensei-shortcode-course-categories.php

Indentation +92 added lines, -92 removed lines

@@ -24,135 +24,135 @@
  */
 class Sensei_Shortcode_Course_Categories implements Sensei_Shortcode_Interface {
 
-    /**
-     * @var array list of taxonomy terms.
-     */
-    protected $sensei_course_taxonomy_terms;
+	/**
+	 * @var array list of taxonomy terms.
+	 */
+	protected $sensei_course_taxonomy_terms;
 
-    /**
-     * Setup the shortcode object
-     *
-     * @since 1.9.0
-     * @param array $attributes
-     * @param string $content
-     * @param string $shortcode the shortcode that was called for this instance
-     */
-    public function __construct( $attributes, $content, $shortcode ){
+	/**
+	 * Setup the shortcode object
+	 *
+	 * @since 1.9.0
+	 * @param array $attributes
+	 * @param string $content
+	 * @param string $shortcode the shortcode that was called for this instance
+	 */
+	public function __construct( $attributes, $content, $shortcode ){
 
-        $this->orderby = isset( $attributes['orderby'] ) ? $attributes['orderby'] : 'name';
-        $this->order = isset( $attributes['order'] ) ? $attributes['order'] : 'ASC';
-        $this->number = isset( $attributes['number'] ) ? $attributes['number'] : '100';
-        $this->parent = isset( $attributes['parent'] ) ? $attributes['parent'] : '';
+		$this->orderby = isset( $attributes['orderby'] ) ? $attributes['orderby'] : 'name';
+		$this->order = isset( $attributes['order'] ) ? $attributes['order'] : 'ASC';
+		$this->number = isset( $attributes['number'] ) ? $attributes['number'] : '100';
+		$this->parent = isset( $attributes['parent'] ) ? $attributes['parent'] : '';
 
-        $include = isset( $attributes['include'] ) ? explode( ',' , $attributes['include'] ) : '';
-        $this->include = $this->generate_term_ids( $include );
+		$include = isset( $attributes['include'] ) ? explode( ',' , $attributes['include'] ) : '';
+		$this->include = $this->generate_term_ids( $include );
 
-        $exclude = isset( $attributes['exclude'] ) ? explode( ',' , $attributes['exclude'] ) : '';
-        $this->exclude = $this->generate_term_ids( $exclude );
+		$exclude = isset( $attributes['exclude'] ) ? explode( ',' , $attributes['exclude'] ) : '';
+		$this->exclude = $this->generate_term_ids( $exclude );
 
-        // make sure we handle string true/false values correctly with respective defaults
-        $hide_empty = isset( $attributes['hide_empty'] ) ? $attributes['hide_empty'] : 'false';
-        $this->hide_empty = 'true' == $hide_empty ? true: false;
+		// make sure we handle string true/false values correctly with respective defaults
+		$hide_empty = isset( $attributes['hide_empty'] ) ? $attributes['hide_empty'] : 'false';
+		$this->hide_empty = 'true' == $hide_empty ? true: false;
 
-        $this->setup_course_categories();
+		$this->setup_course_categories();
 
-    }
+	}
 
-    /**
-     * create the messages query .
-     *
-     * @return mixed
-     */
-    public function setup_course_categories(){
+	/**
+	 * create the messages query .
+	 *
+	 * @return mixed
+	 */
+	public function setup_course_categories(){
 
-        $args = array(
-            'orderby'       => $this->orderby,
-            'order'         => $this->order,
-            'exclude'       => $this->exclude,
-            'include'       => $this->include,
-            'number'        => $this->number,
-            'parent'        => $this->parent,
-            'hide_empty'    => $this->hide_empty,
-            'fields'        => 'all',
-        );
+		$args = array(
+			'orderby'       => $this->orderby,
+			'order'         => $this->order,
+			'exclude'       => $this->exclude,
+			'include'       => $this->include,
+			'number'        => $this->number,
+			'parent'        => $this->parent,
+			'hide_empty'    => $this->hide_empty,
+			'fields'        => 'all',
+		);
 
-        $this->sensei_course_taxonomy_terms = get_terms('course-category', $args);
+		$this->sensei_course_taxonomy_terms = get_terms('course-category', $args);
 
-    }
+	}
 
-    /**
-     * Rendering the shortcode this class is responsible for.
-     *
-     * @return string $content
-     */
-    public function render(){
+	/**
+	 * Rendering the shortcode this class is responsible for.
+	 *
+	 * @return string $content
+	 */
+	public function render(){
 
-        if( empty(  $this->sensei_course_taxonomy_terms  ) ){
+		if( empty(  $this->sensei_course_taxonomy_terms  ) ){
 
-            return __( 'No course categories found.', 'woothemes-sensei' );
+			return __( 'No course categories found.', 'woothemes-sensei' );
 
-        }
+		}
 
-        $terms_html = '';
+		$terms_html = '';
 
-        //set the wp_query to the current messages query
-        $terms_html .= '<ul class="sensei course-categories">';
-        foreach( $this->sensei_course_taxonomy_terms as $category ){
+		//set the wp_query to the current messages query
+		$terms_html .= '<ul class="sensei course-categories">';
+		foreach( $this->sensei_course_taxonomy_terms as $category ){
 
-            $category_link = '<a href="'. get_term_link( $category ) . '">' . $category->name  . '</a>';
-            $terms_html .=  '<li class="sensei course-category" >' . $category_link . '</li>';
+			$category_link = '<a href="'. get_term_link( $category ) . '">' . $category->name  . '</a>';
+			$terms_html .=  '<li class="sensei course-category" >' . $category_link . '</li>';
 
-        }
-        $terms_html .= '<ul>';
+		}
+		$terms_html .= '<ul>';
 
-        return $terms_html;
+		return $terms_html;
 
-    }// end render
+	}// end render
 
-    /**
-     * Convert an array of mixed ids, slugs or names to only the id's of those terms
-     *
-     * @since 1.9.0
-     *
-     * @param array $category_ids
-     * @return array
-     */
-    public function generate_term_ids( $categories = array() ){
+	/**
+	 * Convert an array of mixed ids, slugs or names to only the id's of those terms
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param array $category_ids
+	 * @return array
+	 */
+	public function generate_term_ids( $categories = array() ){
 
-        $cat_ids = array();
+		$cat_ids = array();
 
-        if ( is_array($categories) ) {
-            foreach ($categories as $cat) {
+		if ( is_array($categories) ) {
+			foreach ($categories as $cat) {
 
-                if (!is_numeric($cat)) {
+				if (!is_numeric($cat)) {
 
-                    // try the slug
-                    $term = get_term_by('slug', $cat, 'course-category');
+					// try the slug
+					$term = get_term_by('slug', $cat, 'course-category');
 
-                    // if the slug didn't work try the name
-                    if (!$term) {
+					// if the slug didn't work try the name
+					if (!$term) {
 
-                        $term = get_term_by('name', $cat, 'course-category');
+						$term = get_term_by('name', $cat, 'course-category');
 
-                    }
+					}
 
-                    if ($term) {
-                        $cat_ids[] = $term->term_id;
-                    }
+					if ($term) {
+						$cat_ids[] = $term->term_id;
+					}
 
-                } else {
+				} else {
 
-                    $cat_ids[] = $cat;
+					$cat_ids[] = $cat;
 
-                }
+				}
 
-            }
+			}
 
-        }
+		}
 
-        return $cat_ids;
+		return $cat_ids;
 
-    }// end generate_term_ids
+	}// end generate_term_ids
 
 }// end class
 

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

Indentation +19 added lines, -19 removed lines

@@ -162,10 +162,10 @@ 
 			'order' => $order,
 		);
 
-        // Handle search
-        if ( isset( $_GET['s'] ) && !empty( $_GET['s'] ) ) {
-            $args['search'] = esc_html( $_GET['s'] );
-        }
+		// Handle search
+		if ( isset( $_GET['s'] ) && !empty( $_GET['s'] ) ) {
+			$args['search'] = esc_html( $_GET['s'] );
+		}
 
 		switch ( $this->type ) {
 			case 'courses':
@@ -222,10 +222,10 @@ 
 		);
 
 
-        // Handle search
-        if ( isset( $_GET['s'] ) && !empty( $_GET['s'] ) ) {
-            $args['search'] = esc_html( $_GET['s'] );
-        }
+		// Handle search
+		if ( isset( $_GET['s'] ) && !empty( $_GET['s'] ) ) {
+			$args['search'] = esc_html( $_GET['s'] );
+		}
 
 
 		// Start the csv with the column headings
@@ -263,7 +263,7 @@ 
 	 * Generates the overall array for a single item in the display
 	 * @since  1.7.0
 	 * @param object $item The current item
-     * @return array $column_data;
+	 * @return array $column_data;
 	 */
 	protected function get_row_data( $item ) {
 
@@ -422,8 +422,8 @@ 
 
 				// Output the users data
 				if ( $this->csv_output ) {
-                    $user_name = Sensei()->learners->get_learner_full_name( $item->ID );
-                }
+					$user_name = Sensei()->learners->get_learner_full_name( $item->ID );
+				}
 				else {
 					$url = add_query_arg( array( 'page' => $this->page_slug, 'user_id' => $item->ID ), admin_url( 'admin.php' ) );
 					$user_name = '<strong><a class="row-title" href="' . esc_url( $url ) . '">' . $item->display_name . '</a></strong>';
@@ -517,17 +517,17 @@ 
 		// This stops the full meta data of each user being loaded
 		$args['fields'] = array( 'ID', 'user_login', 'user_email', 'user_registered', 'display_name' );
 
-        /**
-         * Filter the WP_User_Query arguments
-         * @since 1.6.0
-         * @param $args
-         */
-        $args = apply_filters( 'sensei_analysis_overview_filter_users', $args );
+		/**
+		 * Filter the WP_User_Query arguments
+		 * @since 1.6.0
+		 * @param $args
+		 */
+		$args = apply_filters( 'sensei_analysis_overview_filter_users', $args );
 		$wp_user_search = new WP_User_Query( $args );
-        $learners = $wp_user_search->get_results();
+		$learners = $wp_user_search->get_results();
 		$this->total_items = $wp_user_search->get_total();
 
-        return $learners;
+		return $learners;
 
 	} // End get_learners()
 

includes/class-sensei-learners.php

Indentation +53 added lines, -53 removed lines

@@ -79,17 +79,17 @@ 
 
 		// Load Learners JS
 		wp_enqueue_script( 'sensei-learners-general',
-            Sensei()->plugin_url . 'assets/js/learners-general' . $suffix . '.js',
-                            array('jquery','select2','sensei-chosen-ajax' ), Sensei()->version, true );
+			Sensei()->plugin_url . 'assets/js/learners-general' . $suffix . '.js',
+							array('jquery','select2','sensei-chosen-ajax' ), Sensei()->version, true );
 
 		$data = array(
 			'remove_generic_confirm' => __( 'Are you sure you want to remove this user?', 'woothemes-sensei' ),
 			'remove_from_lesson_confirm' => __( 'Are you sure you want to remove the user from this lesson?', 'woothemes-sensei' ),
 			'remove_from_course_confirm' => __( 'Are you sure you want to remove the user from this course?', 'woothemes-sensei' ),
-            'remove_from_purchased_course_confirm' => __( 'Are you sure you want to remove the user from this course? This order associate with this will also be set to canceled.', 'woothemes-sensei' ),
+			'remove_from_purchased_course_confirm' => __( 'Are you sure you want to remove the user from this course? This order associate with this will also be set to canceled.', 'woothemes-sensei' ),
 			'remove_user_from_post_nonce' => wp_create_nonce( 'remove_user_from_post_nonce' ),
-            'search_users_nonce' => wp_create_nonce( 'search-users' ),
-            'selectplaceholder'=> __( 'Select Learner', 'woothemes-sensei' )
+			'search_users_nonce' => wp_create_nonce( 'search-users' ),
+			'selectplaceholder'=> __( 'Select Learner', 'woothemes-sensei' )
 		);
 
 		wp_localize_script( 'sensei-learners-general', 'woo_learners_general_data', $data );
@@ -274,7 +274,7 @@ 
 			$user_id = intval( $action_data['user_id'] );
 			$post_id = intval( $action_data['post_id'] );
 			$post_type = sanitize_text_field( $action_data['post_type'] );
-            $order_id = sanitize_text_field( $action_data['order_id'] );
+			$order_id = sanitize_text_field( $action_data['order_id'] );
 
 			$user = get_userdata( $user_id );
 
@@ -283,15 +283,15 @@ 
 				case 'course':
 					$removed = WooThemes_Sensei_Utils::sensei_remove_user_from_course( $post_id, $user_id );
 
-                    if( ! empty( $order_id ) && Sensei_WC::is_woocommerce_active()  ){
+					if( ! empty( $order_id ) && Sensei_WC::is_woocommerce_active()  ){
 
-                        $order = new WC_Order($order_id);
+						$order = new WC_Order($order_id);
 
-                        if (!empty($order)) {
-                            $order->update_status( 'cancelled' );
-                        }
+						if (!empty($order)) {
+							$order->update_status( 'cancelled' );
+						}
 
-                    }
+					}
 
 				break;
 
@@ -336,19 +336,19 @@ 
 
 		if ( $users ) {
 			foreach ( $users as $user ) {
-                $full_name = Sensei()->learners->get_learner_full_name( $user->ID );
+				$full_name = Sensei()->learners->get_learner_full_name( $user->ID );
 
-                if( trim($user->display_name ) == trim( $full_name ) ){
+				if( trim($user->display_name ) == trim( $full_name ) ){
 
-                    $name = $full_name;
+					$name = $full_name;
 
-                }else{
+				}else{
 
-                    $name = $full_name . ' ['. $user->display_name .']';
+					$name = $full_name . ' ['. $user->display_name .']';
 
-                }
+				}
 
-                $found_users[ $user->ID ] = $name  . ' (#' . $user->ID . ' – ' . sanitize_email( $user->user_email ) . ')';
+				$found_users[ $user->ID ] = $name  . ' (#' . $user->ID . ' – ' . sanitize_email( $user->user_email ) . ')';
 			}
 		}
 
@@ -392,7 +392,7 @@ 
 			break;
 
 			case 'lesson':
-                $lesson_id = absint( $_POST['add_lesson_id'] );
+				$lesson_id = absint( $_POST['add_lesson_id'] );
 				$complete = false;
 				if( isset( $_POST['add_complete_lesson'] ) && 'yes' == $_POST['add_complete_lesson'] ) {
 					$complete = true;
@@ -452,49 +452,49 @@ 
 		}
 	}
 
-    /**
-     * Return the full name and surname or the display name of the user.
-     *
-     * The user must have both name and surname otherwise display name will be returned.
-     *
-     * @since 1.8.0
-     *
-     * @param int $user_id | bool false for an invalid $user_id
-     *
-     * @return string $full_name
-     */
-    public function get_learner_full_name( $user_id ){
+	/**
+	 * Return the full name and surname or the display name of the user.
+	 *
+	 * The user must have both name and surname otherwise display name will be returned.
+	 *
+	 * @since 1.8.0
+	 *
+	 * @param int $user_id | bool false for an invalid $user_id
+	 *
+	 * @return string $full_name
+	 */
+	public function get_learner_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_learner_full_name
+	} // end get_learner_full_name
 
 } // End Class
 

includes/class-sensei-grading-user-quiz.php

Indentation +8 added lines, -8 removed lines

@@ -41,9 +41,9 @@ 
 
 	/**
 	 * Display output to the admin view
-     *
-     * This view is shown when grading a quiz for a single user in admin under grading
-     *
+	 *
+	 * This view is shown when grading a quiz for a single user in admin under grading
+	 *
 	 * @since  1.3.0
 	 * @return html
 	 */
@@ -57,8 +57,8 @@ 
 		$user_quiz_grade_total = 0;
 		$quiz_grade_total = 0;
 		$quiz_grade = 0;
-        $lesson_id = $this->lesson_id;
-        $user_id = $this->user_id;
+		$lesson_id = $this->lesson_id;
+		$user_id = $this->user_id;
 
 		?><form name="<?php esc_attr_e( 'quiz_' . $this->quiz_id ); ?>" action="" method="post">
 			<?php wp_nonce_field( 'sensei_manual_grading', '_wp_sensei_manual_grading_nonce' ); ?>
@@ -206,11 +206,11 @@ 
 						<p class="user-answer"><?php
 							foreach ( $user_answer_content as $_user_answer ) {
 
-                                if( 'multi-line' == Sensei()->question->get_question_type( $question->ID ) ){
+								if( 'multi-line' == Sensei()->question->get_question_type( $question->ID ) ){
 
-                                    $_user_answer = htmlspecialchars_decode( nl2br( esc_html($_user_answer) ) );
+									$_user_answer = htmlspecialchars_decode( nl2br( esc_html($_user_answer) ) );
 
-                                }
+								}
 
 								echo apply_filters( 'sensei_answer_text', $_user_answer ) . "<br>";
 							}

includes/class-sensei-grading-main.php

Indentation +2 added lines, -2 removed lines

@@ -253,7 +253,7 @@ 
 			$grade = __( 'N/A', 'woothemes-sensei' );
 		}
 
-        $title = Sensei()->learners->get_learner_full_name( $item->user_id );
+		$title = Sensei()->learners->get_learner_full_name( $item->user_id );
 
 		// QuizID to be deprecated
 		$quiz_id = get_post_meta( $item->comment_post_ID, '_lesson_quiz', true );
@@ -303,7 +303,7 @@ 
 	 */
 	public function no_items() {
 
-        _e( 'No submissions found.', 'woothemes-sensei' );
+		_e( 'No submissions found.', 'woothemes-sensei' );
 
 	} // End no_items()
 

includes/class-sensei-autoloader.php

Indentation +116 added lines, -116 removed lines

@@ -13,123 +13,123 @@
  */
 class Sensei_Autoloader {
 
-    /**
-     * @var path to the includes directory within Sensei.
-     */
-    private $include_path = 'includes';
-
-    /**
-     * @var array $class_file_map. List of classes mapped to their files
-     */
-    private $class_file_map = array();
-
-    /**
-     * Constructor
-     * @since 1.9.0
-     */
-    public function __construct(){
-
-        // make sure we do not override an existing autoload function
-        if( function_exists('__autoload') ){
-           spl_autoload_register( '__autoload' );
-        }
-
-        // setup a relative path for the current autoload instance
-        $this->include_path = trailingslashit( untrailingslashit( dirname( __FILE__ ) ) );
-
-        //setup the class file map
-        $this->initialize_class_file_map();
-
-        // add Sensei custom auto loader
-        spl_autoload_register( array( $this, 'autoload' )  );
-
-    }
-
-    /**
-     * Generate a list of Sensei class and map them the their respective
-     * files within the includes directory
-     *
-     * @since 1.9.0
-     */
-    public function initialize_class_file_map(){
-
-        $this->class_file_map = array(
-
-            /**
-             * Main Sensei class
-             */
-            'Sensei_Main' => 'class-sensei.php',
-
-            /**
-             * Admin
-             */
-            'Sensei_Welcome'=> 'admin/class-sensei-welcome.php' ,
-
-            /**
-             * Shortcodes
-             */
-            'Sensei_Shortcode_Loader'              => 'shortcodes/class-sensei-shortcode-loader.php',
-            'Sensei_Shortcode_Interface'           => 'shortcodes/interface-sensei-shortcode.php',
-            'Sensei_Shortcode_Featured_Courses'    => 'shortcodes/class-sensei-shortcode-featured-courses.php',
-            'Sensei_Shortcode_User_Courses'        => 'shortcodes/class-sensei-shortcode-user-courses.php',
-            'Sensei_Shortcode_Courses'             => 'shortcodes/class-sensei-shortcode-courses.php',
-            'Sensei_Shortcode_Teachers'            => 'shortcodes/class-sensei-shortcode-teachers.php',
-            'Sensei_Shortcode_User_Messages'       => 'shortcodes/class-sensei-shortcode-user-messages.php',
-            'Sensei_Shortcode_Course_Page'         => 'shortcodes/class-sensei-shortcode-course-page.php',
-            'Sensei_Shortcode_Lesson_Page'         => 'shortcodes/class-sensei-shortcode-lesson-page.php',
-            'Sensei_Shortcode_Course_Categories'   => 'shortcodes/class-sensei-shortcode-course-categories.php',
-            'Sensei_Shortcode_Unpurchased_Courses' => 'shortcodes/class-sensei-shortcode-unpurchased-courses.php',
-            'Sensei_Legacy_Shortcodes'             => 'shortcodes/class-sensei-legacy-shortcodes.php',
-
-            /**
-             * WooCommerce
-             */
-            'Sensei_WC' => 'class-sensei-wc.php',
-
-        );
-    }
-
-    /**
-     * Autoload all sensei files as the class names are used.
-     */
-    public function autoload( $class ){
-
-        // only handle classes with the word `sensei` in it
-        if( ! is_numeric( strpos ( strtolower( $class ), 'sensei') ) ){
-
-            return;
-
-        }
-
-        // exit if we didn't provide mapping for this class
-        if( isset( $this->class_file_map[ $class ] ) ){
-
-            $file_location = $this->include_path . $this->class_file_map[ $class ];
-            require_once( $file_location);
-            return;
-
-        }
-
-        // check for file in the main includes directory
-        $class_file_path = $this->include_path . 'class-'.str_replace( '_','-', strtolower( $class ) ) . '.php';
-        if( file_exists( $class_file_path ) ){
-
-            require_once( $class_file_path );
-            return;
-        }
-
-        // lastly check legacy types
-        $stripped_woothemes_from_class = str_replace( 'woothemes_','', strtolower( $class ) ); // remove woothemes
-        $legacy_class_file_path = $this->include_path . 'class-'.str_replace( '_','-', strtolower( $stripped_woothemes_from_class ) ) . '.php';
-        if( file_exists( $legacy_class_file_path ) ){
-
-            require_once( $legacy_class_file_path );
-            return;
-        }
-
-        return;
+	/**
+	 * @var path to the includes directory within Sensei.
+	 */
+	private $include_path = 'includes';
+
+	/**
+	 * @var array $class_file_map. List of classes mapped to their files
+	 */
+	private $class_file_map = array();
+
+	/**
+	 * Constructor
+	 * @since 1.9.0
+	 */
+	public function __construct(){
+
+		// make sure we do not override an existing autoload function
+		if( function_exists('__autoload') ){
+		   spl_autoload_register( '__autoload' );
+		}
+
+		// setup a relative path for the current autoload instance
+		$this->include_path = trailingslashit( untrailingslashit( dirname( __FILE__ ) ) );
+
+		//setup the class file map
+		$this->initialize_class_file_map();
+
+		// add Sensei custom auto loader
+		spl_autoload_register( array( $this, 'autoload' )  );
+
+	}
+
+	/**
+	 * Generate a list of Sensei class and map them the their respective
+	 * files within the includes directory
+	 *
+	 * @since 1.9.0
+	 */
+	public function initialize_class_file_map(){
+
+		$this->class_file_map = array(
+
+			/**
+			 * Main Sensei class
+			 */
+			'Sensei_Main' => 'class-sensei.php',
+
+			/**
+			 * Admin
+			 */
+			'Sensei_Welcome'=> 'admin/class-sensei-welcome.php' ,
+
+			/**
+			 * Shortcodes
+			 */
+			'Sensei_Shortcode_Loader'              => 'shortcodes/class-sensei-shortcode-loader.php',
+			'Sensei_Shortcode_Interface'           => 'shortcodes/interface-sensei-shortcode.php',
+			'Sensei_Shortcode_Featured_Courses'    => 'shortcodes/class-sensei-shortcode-featured-courses.php',
+			'Sensei_Shortcode_User_Courses'        => 'shortcodes/class-sensei-shortcode-user-courses.php',
+			'Sensei_Shortcode_Courses'             => 'shortcodes/class-sensei-shortcode-courses.php',
+			'Sensei_Shortcode_Teachers'            => 'shortcodes/class-sensei-shortcode-teachers.php',
+			'Sensei_Shortcode_User_Messages'       => 'shortcodes/class-sensei-shortcode-user-messages.php',
+			'Sensei_Shortcode_Course_Page'         => 'shortcodes/class-sensei-shortcode-course-page.php',
+			'Sensei_Shortcode_Lesson_Page'         => 'shortcodes/class-sensei-shortcode-lesson-page.php',
+			'Sensei_Shortcode_Course_Categories'   => 'shortcodes/class-sensei-shortcode-course-categories.php',
+			'Sensei_Shortcode_Unpurchased_Courses' => 'shortcodes/class-sensei-shortcode-unpurchased-courses.php',
+			'Sensei_Legacy_Shortcodes'             => 'shortcodes/class-sensei-legacy-shortcodes.php',
+
+			/**
+			 * WooCommerce
+			 */
+			'Sensei_WC' => 'class-sensei-wc.php',
+
+		);
+	}
+
+	/**
+	 * Autoload all sensei files as the class names are used.
+	 */
+	public function autoload( $class ){
+
+		// only handle classes with the word `sensei` in it
+		if( ! is_numeric( strpos ( strtolower( $class ), 'sensei') ) ){
+
+			return;
+
+		}
+
+		// exit if we didn't provide mapping for this class
+		if( isset( $this->class_file_map[ $class ] ) ){
+
+			$file_location = $this->include_path . $this->class_file_map[ $class ];
+			require_once( $file_location);
+			return;
+
+		}
+
+		// check for file in the main includes directory
+		$class_file_path = $this->include_path . 'class-'.str_replace( '_','-', strtolower( $class ) ) . '.php';
+		if( file_exists( $class_file_path ) ){
+
+			require_once( $class_file_path );
+			return;
+		}
+
+		// lastly check legacy types
+		$stripped_woothemes_from_class = str_replace( 'woothemes_','', strtolower( $class ) ); // remove woothemes
+		$legacy_class_file_path = $this->include_path . 'class-'.str_replace( '_','-', strtolower( $stripped_woothemes_from_class ) ) . '.php';
+		if( file_exists( $legacy_class_file_path ) ){
+
+			require_once( $legacy_class_file_path );
+			return;
+		}
+
+		return;
 
-    }// end autoload
+	}// end autoload
 
 }
 new Sensei_Autoloader();

includes/class-sensei-emails.php

Indentation +42 added lines, -42 removed lines

@@ -51,7 +51,7 @@ 
 		add_action( 'sensei_course_status_updated', array( $this, 'learner_completed_course' ), 10, 4 );
 		add_action( 'sensei_course_status_updated', array( $this, 'teacher_completed_course' ), 10, 4 );
 		add_action( 'sensei_user_course_start', array( $this, 'teacher_started_course' ), 10, 2 );
-        add_action( 'sensei_user_lesson_end', array( $this, 'teacher_completed_lesson' ), 10, 2 );
+		add_action( 'sensei_user_lesson_end', array( $this, 'teacher_completed_lesson' ), 10, 2 );
 		add_action( 'sensei_user_quiz_submitted', array( $this, 'teacher_quiz_submitted' ), 10, 5 );
 		add_action( 'sensei_new_private_message', array( $this, 'teacher_new_message' ), 10, 1 );
 		add_action( 'sensei_private_message_reply', array( $this, 'new_message_reply' ), 10, 2 );
@@ -68,9 +68,9 @@ 
 		$this->emails['learner-graded-quiz'] = include( 'emails/class-woothemes-sensei-email-learner-graded-quiz.php' );
 		$this->emails['learner-completed-course'] = include( 'emails/class-woothemes-sensei-email-learner-completed-course.php' );
 		$this->emails['teacher-completed-course'] = include( 'emails/class-woothemes-sensei-email-teacher-completed-course.php' );
-        $this->emails['teacher-started-course'] = include( 'emails/class-woothemes-sensei-email-teacher-started-course.php' );
-        $this->emails['teacher-completed-lesson'] = include( 'emails/class-woothemes-sensei-email-teacher-completed-lesson.php' );
-        $this->emails['teacher-quiz-submitted'] = include( 'emails/class-woothemes-sensei-email-teacher-quiz-submitted.php' );
+		$this->emails['teacher-started-course'] = include( 'emails/class-woothemes-sensei-email-teacher-started-course.php' );
+		$this->emails['teacher-completed-lesson'] = include( 'emails/class-woothemes-sensei-email-teacher-completed-lesson.php' );
+		$this->emails['teacher-quiz-submitted'] = include( 'emails/class-woothemes-sensei-email-teacher-quiz-submitted.php' );
 		$this->emails['teacher-new-message'] = include( 'emails/class-woothemes-sensei-email-teacher-new-message.php' );
 		$this->emails['new-message-reply'] = include( 'emails/class-woothemes-sensei-email-new-message-reply.php' );
 		$this->emails = apply_filters( 'sensei_email_classes', $this->emails );
@@ -177,20 +177,20 @@ 
 		add_filter( 'wp_mail_from_name', array( $this, 'get_from_name' ) );
 		add_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
 
-        // Send
-        $send_email = true;
+		// Send
+		$send_email = true;
 
-        /**
-         * Filter Sensei's ability to send out emails.
-         *
-         * @since 1.8.0
-         * @param bool $send_email default true
-         */
-        if( apply_filters('sensei_send_emails', $send_email,$to, $subject, $message )  ){
+		/**
+		 * Filter Sensei's ability to send out emails.
+		 *
+		 * @since 1.8.0
+		 * @param bool $send_email default true
+		 */
+		if( apply_filters('sensei_send_emails', $send_email,$to, $subject, $message )  ){
 
-            wp_mail( $to, $subject, $message, $headers, $attachments );
+			wp_mail( $to, $subject, $message, $headers, $attachments );
 
-        }
+		}
 
 		// Unhook filters
 		remove_filter( 'wp_mail_from', array( $this, 'get_from_address' ) );
@@ -332,33 +332,33 @@ 
 		}
 	}
 
-    /**
-     * teacher_completed_lesson()
-     *
-     * Send email to teacher on student completing lesson
-     *
-     * @access public
-     * @return void
-     * @since 1.9.0
-     */
-    function teacher_completed_lesson( $learner_id = 0, $lesson_id = 0 ) {
-
-
-        $send = false;
-
-        if( isset( Sensei()->settings->settings[ 'email_teachers' ] ) ) {
-            if( in_array( 'teacher-completed-lesson', (array) Sensei()->settings->settings[ 'email_teachers' ]) ) {
-                $send = true;
-            }
-        } else {
-            $send = true;
-        }
-
-        if( $send ) {
-            $email = $this->emails['teacher-completed-lesson'];
-            $email->trigger( $learner_id, $lesson_id );
-        }
-    }
+	/**
+	 * teacher_completed_lesson()
+	 *
+	 * Send email to teacher on student completing lesson
+	 *
+	 * @access public
+	 * @return void
+	 * @since 1.9.0
+	 */
+	function teacher_completed_lesson( $learner_id = 0, $lesson_id = 0 ) {
+
+
+		$send = false;
+
+		if( isset( Sensei()->settings->settings[ 'email_teachers' ] ) ) {
+			if( in_array( 'teacher-completed-lesson', (array) Sensei()->settings->settings[ 'email_teachers' ]) ) {
+				$send = true;
+			}
+		} else {
+			$send = true;
+		}
+
+		if( $send ) {
+			$email = $this->emails['teacher-completed-lesson'];
+			$email->trigger( $learner_id, $lesson_id );
+		}
+	}
 
 	/**
 	 * Send email to teacher on quiz submission

includes/class-sensei-posttypes.php

Indentation +220 added lines, -220 removed lines

@@ -84,90 +84,90 @@ 
 	/**
 	 * Setup the "course" post type, it's admin menu item and the appropriate labels and permissions.
 	 * @since  1.0.0
-     * @uses  Sensei()
+	 * @uses  Sensei()
 	 * @return void
 	 */
 	public function setup_course_post_type () {
 
 		$args = array(
-		    'labels'              => $this->create_post_type_labels( 'course', $this->labels['course']['singular'], $this->labels['course']['plural'], $this->labels['course']['menu'] ),
-		    'public'              => true,
-		    'publicly_queryable'  => true,
-		    'show_ui'             => true,
-		    'show_in_menu'        => true,
-		    'show_in_admin_bar'   => true,
-		    'query_var'           => true,
-		    'rewrite'             => array(
-                'slug' => esc_attr( apply_filters( 'sensei_course_slug', _x( 'course', 'post type single url base', 'woothemes-sensei' ) ) ) ,
-                'with_front' => true,
-                'feeds' => true,
-                'pages' => true
-            ),
-		    'map_meta_cap'        => true,
-		    'capability_type'     => 'course',
-            'has_archive'         => $this->get_course_post_type_archive_slug(),
-		    'hierarchical'        => false,
-		    'menu_position'       => 51,
-		    'supports'            => array( 'title', 'editor', 'excerpt', 'thumbnail' )
+			'labels'              => $this->create_post_type_labels( 'course', $this->labels['course']['singular'], $this->labels['course']['plural'], $this->labels['course']['menu'] ),
+			'public'              => true,
+			'publicly_queryable'  => true,
+			'show_ui'             => true,
+			'show_in_menu'        => true,
+			'show_in_admin_bar'   => true,
+			'query_var'           => true,
+			'rewrite'             => array(
+				'slug' => esc_attr( apply_filters( 'sensei_course_slug', _x( 'course', 'post type single url base', 'woothemes-sensei' ) ) ) ,
+				'with_front' => true,
+				'feeds' => true,
+				'pages' => true
+			),
+			'map_meta_cap'        => true,
+			'capability_type'     => 'course',
+			'has_archive'         => $this->get_course_post_type_archive_slug(),
+			'hierarchical'        => false,
+			'menu_position'       => 51,
+			'supports'            => array( 'title', 'editor', 'excerpt', 'thumbnail' )
 		);
 
-        /**
-         * Filter the arguments passed in when registering the Sensei Course post type.
-         *
-         * @since 1.9.0
-         * @param array $args
-         */
+		/**
+		 * Filter the arguments passed in when registering the Sensei Course post type.
+		 *
+		 * @since 1.9.0
+		 * @param array $args
+		 */
 		register_post_type( 'course', apply_filters( 'sensei_register_post_type_course', $args ) );
 
 	} // End setup_course_post_type()
 
-    /**
-     * Figure out of the course post type has an archive and what it should be.
-     *
-     * This function should return 'courses' or the page_uri for the course page setting.
-     *
-     * For backward compatibility  sake ( pre 1.9 )If the course page set in settings
-     * still has any of the old shortcodes: [newcourses][featuredcourses][freecourses][paidcourses] the
-     * page slug will not be returned. For any other pages without it the page URI will be returned.
-     *
-     *
-     * @sine 1.9.0
-     *
-     *
-     * @return false|string
-     */
-    public function get_course_post_type_archive_slug(){
+	/**
+	 * Figure out of the course post type has an archive and what it should be.
+	 *
+	 * This function should return 'courses' or the page_uri for the course page setting.
+	 *
+	 * For backward compatibility  sake ( pre 1.9 )If the course page set in settings
+	 * still has any of the old shortcodes: [newcourses][featuredcourses][freecourses][paidcourses] the
+	 * page slug will not be returned. For any other pages without it the page URI will be returned.
+	 *
+	 *
+	 * @sine 1.9.0
+	 *
+	 *
+	 * @return false|string
+	 */
+	public function get_course_post_type_archive_slug(){
 
-        $settings_course_page = get_post( Sensei()->settings->get( 'course_page' ) );
+		$settings_course_page = get_post( Sensei()->settings->get( 'course_page' ) );
 
-        // for a valid post that doesn't have any of the old short codes set the archive the same
-        // as the page URI
-        if( is_a( $settings_course_page, 'WP_Post') && ! $this->has_old_shortcodes( $settings_course_page->post_content ) ){
+		// for a valid post that doesn't have any of the old short codes set the archive the same
+		// as the page URI
+		if( is_a( $settings_course_page, 'WP_Post') && ! $this->has_old_shortcodes( $settings_course_page->post_content ) ){
 
-             return get_page_uri( $settings_course_page->ID );
+			 return get_page_uri( $settings_course_page->ID );
 
-        }else{
+		}else{
 
-            return 'courses';
+			return 'courses';
 
-        }
+		}
 
-    }// end course_post_type_determine_archive
+	}// end course_post_type_determine_archive
 
-    /**
-     * Check if given content has any of these old shortcodes:
-     * [newcourses][featuredcourses][freecourses][paidcourses]
-     *
-     * @since 1.9.0
-     */
-    public function has_old_shortcodes( $content ){
+	/**
+	 * Check if given content has any of these old shortcodes:
+	 * [newcourses][featuredcourses][freecourses][paidcourses]
+	 *
+	 * @since 1.9.0
+	 */
+	public function has_old_shortcodes( $content ){
 
-        return  ( has_shortcode( $content, 'newcourses')
-        || has_shortcode( $content, 'featuredcourses')
-        || has_shortcode( $content, 'freecourses')
-        || has_shortcode( $content, 'paidcourses') );
+		return  ( has_shortcode( $content, 'newcourses')
+		|| has_shortcode( $content, 'featuredcourses')
+		|| has_shortcode( $content, 'freecourses')
+		|| has_shortcode( $content, 'paidcourses') );
 
-    }// end has old shortcodes
+	}// end has old shortcodes
 
 
 	/**
@@ -189,32 +189,32 @@ 
 		} // End If Statement
 
 		$args = array(
-		    'labels' => $this->create_post_type_labels( 'lesson', $this->labels['lesson']['singular'], $this->labels['lesson']['plural'], $this->labels['lesson']['menu'] ),
-		    'public' => true,
-		    'publicly_queryable' => true,
-		    'show_ui' => true,
-		    'show_in_menu' => true,
-		    'query_var' => true,
-		    'rewrite' => array(
-                'slug' => esc_attr( apply_filters( 'sensei_lesson_slug', _x( 'lesson', 'post type single slug', 'woothemes-sensei' ) ) ) ,
-                'with_front' =>  true,
-                'feeds' => true,
-                'pages' => true
-            ),
-		    'map_meta_cap' => true,
-		    'capability_type' => 'lesson',
-		    'has_archive' => true,
-		    'hierarchical' => false,
-		    'menu_position' => 52,
-		    'supports' => $supports_array
+			'labels' => $this->create_post_type_labels( 'lesson', $this->labels['lesson']['singular'], $this->labels['lesson']['plural'], $this->labels['lesson']['menu'] ),
+			'public' => true,
+			'publicly_queryable' => true,
+			'show_ui' => true,
+			'show_in_menu' => true,
+			'query_var' => true,
+			'rewrite' => array(
+				'slug' => esc_attr( apply_filters( 'sensei_lesson_slug', _x( 'lesson', 'post type single slug', 'woothemes-sensei' ) ) ) ,
+				'with_front' =>  true,
+				'feeds' => true,
+				'pages' => true
+			),
+			'map_meta_cap' => true,
+			'capability_type' => 'lesson',
+			'has_archive' => true,
+			'hierarchical' => false,
+			'menu_position' => 52,
+			'supports' => $supports_array
 		);
 
-        /**
-         * Filter the arguments passed in when registering the Sensei Lesson post type.
-         *
-         * @since 1.9.0
-         * @param array $args
-         */
+		/**
+		 * Filter the arguments passed in when registering the Sensei Lesson post type.
+		 *
+		 * @since 1.9.0
+		 * @param array $args
+		 */
 		register_post_type( 'lesson', apply_filters( 'sensei_register_post_type_lesson', $args ) );
 
 	} // End setup_lesson_post_type()
@@ -222,46 +222,46 @@ 
 	/**
 	 * Setup the "quiz" post type, it's admin menu item and the appropriate labels and permissions.
 	 * @since  1.0.0
-     * @uses  Sensei()
+	 * @uses  Sensei()
 	 * @return void
 	 */
 	public function setup_quiz_post_type () {
 
 
 		$args = array(
-		    'labels' => $this->create_post_type_labels(
-                'quiz',
-                $this->labels['quiz']['singular'],
-                $this->labels['quiz']['plural'],
-                $this->labels['quiz']['menu']
-            ),
-		    'public' => true,
-		    'publicly_queryable' => true,
-		    'show_ui' => true,
-		    'show_in_menu' => false,
-		    'show_in_nav_menus' => false,
-		    'query_var' => true,
-		    'exclude_from_search' => true,
-		    'rewrite' => array(
-                'slug' => esc_attr( apply_filters( 'sensei_quiz_slug', _x( 'quiz', 'post type single slug', 'woothemes-sensei' ) ) ) ,
-                'with_front' =>  true,
-                'feeds' => true,
-                'pages' => true
-            ),
-		    'map_meta_cap' => true,
-		    'capability_type' => 'quiz',
-		    'has_archive' => false,
-		    'hierarchical' => false,
-		    'menu_position' => 20, // Below "Pages"
-		    'supports' => array( '' )
+			'labels' => $this->create_post_type_labels(
+				'quiz',
+				$this->labels['quiz']['singular'],
+				$this->labels['quiz']['plural'],
+				$this->labels['quiz']['menu']
+			),
+			'public' => true,
+			'publicly_queryable' => true,
+			'show_ui' => true,
+			'show_in_menu' => false,
+			'show_in_nav_menus' => false,
+			'query_var' => true,
+			'exclude_from_search' => true,
+			'rewrite' => array(
+				'slug' => esc_attr( apply_filters( 'sensei_quiz_slug', _x( 'quiz', 'post type single slug', 'woothemes-sensei' ) ) ) ,
+				'with_front' =>  true,
+				'feeds' => true,
+				'pages' => true
+			),
+			'map_meta_cap' => true,
+			'capability_type' => 'quiz',
+			'has_archive' => false,
+			'hierarchical' => false,
+			'menu_position' => 20, // Below "Pages"
+			'supports' => array( '' )
 		);
 
-        /**
-         * Filter the arguments passed in when registering the Sensei Quiz post type.
-         *
-         * @since 1.9.0
-         * @param array $args
-         */
+		/**
+		 * Filter the arguments passed in when registering the Sensei Quiz post type.
+		 *
+		 * @since 1.9.0
+		 * @param array $args
+		 */
 		register_post_type( 'quiz', apply_filters( 'sensei_register_post_type_quiz', $args ) );
 
 	} // End setup_quiz_post_type()
@@ -275,34 +275,34 @@ 
 	public function setup_question_post_type () {
 
 		$args = array(
-		    'labels' => $this->create_post_type_labels( 'question', $this->labels['question']['singular'], $this->labels['question']['plural'], $this->labels['question']['menu'] ),
-		    'public' => false,
-		    'publicly_queryable' => true,
-		    'show_ui' => true,
-		    'show_in_menu' => true,
-		    'show_in_nav_menus' => false,
-		    'query_var' => true,
-		    'exclude_from_search' => true,
-		    'rewrite' => array(
-                'slug' => esc_attr( apply_filters( 'sensei_question_slug', _x( 'question', 'post type single slug', 'woothemes-sensei' ) ) ) ,
-                'with_front' =>  true,
-                'feeds' => true,
-                'pages' => true
-            ),
-		    'map_meta_cap' => true,
-		    'capability_type' => 'question',
-		    'has_archive' => true,
-		    'hierarchical' => false,
-		    'menu_position' => 51,
-		    'supports' => array( 'title' )
+			'labels' => $this->create_post_type_labels( 'question', $this->labels['question']['singular'], $this->labels['question']['plural'], $this->labels['question']['menu'] ),
+			'public' => false,
+			'publicly_queryable' => true,
+			'show_ui' => true,
+			'show_in_menu' => true,
+			'show_in_nav_menus' => false,
+			'query_var' => true,
+			'exclude_from_search' => true,
+			'rewrite' => array(
+				'slug' => esc_attr( apply_filters( 'sensei_question_slug', _x( 'question', 'post type single slug', 'woothemes-sensei' ) ) ) ,
+				'with_front' =>  true,
+				'feeds' => true,
+				'pages' => true
+			),
+			'map_meta_cap' => true,
+			'capability_type' => 'question',
+			'has_archive' => true,
+			'hierarchical' => false,
+			'menu_position' => 51,
+			'supports' => array( 'title' )
 		);
 
-        /**
-         * Filter the arguments passed in when registering the Sensei Question post type.
-         *
-         * @since 1.9.0
-         * @param array $args
-         */
+		/**
+		 * Filter the arguments passed in when registering the Sensei Question post type.
+		 *
+		 * @since 1.9.0
+		 * @param array $args
+		 */
 		register_post_type( 'question', apply_filters('sensei_register_post_type_question', $args ) );
 
 	} // End setup_question_post_type()
@@ -315,26 +315,26 @@ 
 	public function setup_multiple_question_post_type () {
 
 		$args = array(
-		    'labels' => $this->create_post_type_labels( 'multiple_question', $this->labels['multiple_question']['singular'], $this->labels['multiple_question']['plural'], $this->labels['multiple_question']['menu'] ),
-		    'public' => false,
-		    'publicly_queryable' => false,
-		    'show_ui' => false,
-		    'show_in_menu' => false,
-		    'show_in_nav_menus' => false,
-		    'query_var' => false,
-		    'exclude_from_search' => true,
-		    'rewrite' => array(
-                'slug' => esc_attr( apply_filters( 'sensei_multiple_question_slug', _x( 'multiple_question', 'post type single slug', 'woothemes-sensei' ) ) ) ,
-                'with_front' =>  false,
-                'feeds' => false,
-                'pages' => false
-            ),
-		    'map_meta_cap' => true,
-		    'capability_type' => 'question',
-		    'has_archive' => false,
-		    'hierarchical' => false,
-		    'menu_position' => 51,
-		    'supports' => array( 'title', 'custom-fields' )
+			'labels' => $this->create_post_type_labels( 'multiple_question', $this->labels['multiple_question']['singular'], $this->labels['multiple_question']['plural'], $this->labels['multiple_question']['menu'] ),
+			'public' => false,
+			'publicly_queryable' => false,
+			'show_ui' => false,
+			'show_in_menu' => false,
+			'show_in_nav_menus' => false,
+			'query_var' => false,
+			'exclude_from_search' => true,
+			'rewrite' => array(
+				'slug' => esc_attr( apply_filters( 'sensei_multiple_question_slug', _x( 'multiple_question', 'post type single slug', 'woothemes-sensei' ) ) ) ,
+				'with_front' =>  false,
+				'feeds' => false,
+				'pages' => false
+			),
+			'map_meta_cap' => true,
+			'capability_type' => 'question',
+			'has_archive' => false,
+			'hierarchical' => false,
+			'menu_position' => 51,
+			'supports' => array( 'title', 'custom-fields' )
 		);
 
 		register_post_type( 'multiple_question', $args );
@@ -351,34 +351,34 @@ 
 		if( ! isset( Sensei()->settings->settings['messages_disable'] ) || ! Sensei()->settings->settings['messages_disable'] ) {
 
 			$args = array(
-			    'labels' => $this->create_post_type_labels( 'sensei_message', $this->labels['sensei_message']['singular'], $this->labels['sensei_message']['plural'], $this->labels['sensei_message']['menu'] ),
-			    'public' => true,
-			    'publicly_queryable' => true,
-			    'show_ui' => true,
-			    'show_in_menu' => 'sensei',
-			    'show_in_nav_menus' => true,
-			    'query_var' => true,
-			    'exclude_from_search' => true,
-			    'rewrite' => array(
-                    'slug' => esc_attr( apply_filters( 'sensei_messages_slug', _x( 'messages', 'post type single slug', 'woothemes-sensei' ) ) ) ,
-                    'with_front' =>  false,
-                    'feeds' => false,
-                    'pages' => true
-                ),
-			    'map_meta_cap' => true,
-			    'capability_type' => 'question',
-			    'has_archive' => true,
-			    'hierarchical' => false,
-			    'menu_position' => 50,
-			    'supports' => array( 'title', 'editor', 'comments' ),
+				'labels' => $this->create_post_type_labels( 'sensei_message', $this->labels['sensei_message']['singular'], $this->labels['sensei_message']['plural'], $this->labels['sensei_message']['menu'] ),
+				'public' => true,
+				'publicly_queryable' => true,
+				'show_ui' => true,
+				'show_in_menu' => 'sensei',
+				'show_in_nav_menus' => true,
+				'query_var' => true,
+				'exclude_from_search' => true,
+				'rewrite' => array(
+					'slug' => esc_attr( apply_filters( 'sensei_messages_slug', _x( 'messages', 'post type single slug', 'woothemes-sensei' ) ) ) ,
+					'with_front' =>  false,
+					'feeds' => false,
+					'pages' => true
+				),
+				'map_meta_cap' => true,
+				'capability_type' => 'question',
+				'has_archive' => true,
+				'hierarchical' => false,
+				'menu_position' => 50,
+				'supports' => array( 'title', 'editor', 'comments' ),
 			);
 
-            /**
-             * Filter the arguments passed in when registering the Sensei sensei_message post type.
-             *
-             * @since 1.9.0
-             * @param array $args
-             */
+			/**
+			 * Filter the arguments passed in when registering the Sensei sensei_message post type.
+			 *
+			 * @since 1.9.0
+			 * @param array $args
+			 */
 			register_post_type( 'sensei_message', apply_filters('sensei_register_post_type_sensei_message',  $args ) );
 		}
 	} // End setup_sensei_message_post_type()
@@ -411,11 +411,11 @@ 
 			'show_ui' => true,
 			'query_var' => true,
 			'show_in_nav_menus' => true,
-            'capabilities' => array(
-                'manage_terms' => 'manage_categories',
-                'edit_terms'   => 'edit_courses',
-                'delete_terms' => 'manage_categories',
-                'assign_terms' => 'edit_courses',),
+			'capabilities' => array(
+				'manage_terms' => 'manage_categories',
+				'edit_terms'   => 'edit_courses',
+				'delete_terms' => 'manage_categories',
+				'assign_terms' => 'edit_courses',),
 			'rewrite' => array( 'slug' => esc_attr( apply_filters( 'sensei_course_category_slug', _x( 'course-category', 'taxonomy archive slug', 'woothemes-sensei' ) ) ) )
 		);
 
@@ -451,7 +451,7 @@ 
 			'show_ui' => true, /* TO DO - future releases */
 			'query_var' => true,
 			'show_in_nav_menus' => false,
-            'public' => false,
+			'public' => false,
 			'rewrite' => array( 'slug' => esc_attr( apply_filters( 'sensei_quiz_type_slug', _x( 'quiz-type', 'taxonomy archive slug', 'woothemes-sensei' ) ) ) )
 		);
 
@@ -523,11 +523,11 @@ 
 			'query_var' => false,
 			'show_in_nav_menus' => false,
 			'show_admin_column' => true,
-            'capabilities' => array(
-                'manage_terms' => 'manage_categories',
-                'edit_terms'   => 'edit_questions',
-                'delete_terms' => 'manage_categories',
-                'assign_terms' => 'edit_questions',),
+			'capabilities' => array(
+				'manage_terms' => 'manage_categories',
+				'edit_terms'   => 'edit_questions',
+				'delete_terms' => 'manage_categories',
+				'assign_terms' => 'edit_questions',),
 			'rewrite' => array( 'slug' => esc_attr( apply_filters( 'sensei_question_category_slug', _x( 'question-category', 'taxonomy archive slug', 'woothemes-sensei' ) ) ) )
 		);
 
@@ -561,11 +561,11 @@ 
 			'show_ui' => true,
 			'query_var' => true,
 			'show_in_nav_menus' => true,
-            'capabilities' => array(
-                'manage_terms' => 'manage_categories',
-                'edit_terms'   => 'edit_lessons',
-                'delete_terms' => 'manage_categories',
-                'assign_terms' => 'edit_lessons',),
+			'capabilities' => array(
+				'manage_terms' => 'manage_categories',
+				'edit_terms'   => 'edit_lessons',
+				'delete_terms' => 'manage_categories',
+				'assign_terms' => 'edit_lessons',),
 			'rewrite' => array( 'slug' => esc_attr( apply_filters( 'sensei_lesson_tag_slug', _x( 'lesson-tag', 'taxonomy archive slug', 'woothemes-sensei' ) ) ) )
 		);
 
@@ -600,19 +600,19 @@ 
 	 */
 	private function create_post_type_labels ( $token, $singular, $plural, $menu ) {
 		$labels = array(
-		    'name' => sprintf( _x( '%s', 'post type general name', 'woothemes-sensei' ), $plural ),
-		    'singular_name' => sprintf( _x( '%s', 'post type singular name', 'woothemes-sensei' ), $singular ),
-		    'add_new' => __( 'Add New', 'woothemes-sensei' ),
-		    'add_new_item' => sprintf( __( 'Add New %s', 'woothemes-sensei' ), $singular ),
-		    'edit_item' => sprintf( __( 'Edit %s', 'woothemes-sensei' ), $singular ),
-		    'new_item' => sprintf( __( 'New %s', 'woothemes-sensei' ), $singular ),
-		    'all_items' => sprintf( __( 'All %s', 'woothemes-sensei' ), $plural ),
-		    'view_item' => sprintf( __( 'View %s', 'woothemes-sensei' ), $singular ),
-		    'search_items' => sprintf( __( 'Search %s', 'woothemes-sensei' ), $plural ),
-		    'not_found' =>  sprintf( __( 'No %s found', 'woothemes-sensei' ), mb_strtolower( $plural, 'UTF-8') ),
-		    'not_found_in_trash' => sprintf( __( 'No %s found in Trash', 'woothemes-sensei' ), mb_strtolower( $plural, 'UTF-8') ),
-		    'parent_item_colon' => '',
-		    'menu_name' => sprintf( __( '%s', 'woothemes-sensei' ), $menu )
+			'name' => sprintf( _x( '%s', 'post type general name', 'woothemes-sensei' ), $plural ),
+			'singular_name' => sprintf( _x( '%s', 'post type singular name', 'woothemes-sensei' ), $singular ),
+			'add_new' => __( 'Add New', 'woothemes-sensei' ),
+			'add_new_item' => sprintf( __( 'Add New %s', 'woothemes-sensei' ), $singular ),
+			'edit_item' => sprintf( __( 'Edit %s', 'woothemes-sensei' ), $singular ),
+			'new_item' => sprintf( __( 'New %s', 'woothemes-sensei' ), $singular ),
+			'all_items' => sprintf( __( 'All %s', 'woothemes-sensei' ), $plural ),
+			'view_item' => sprintf( __( 'View %s', 'woothemes-sensei' ), $singular ),
+			'search_items' => sprintf( __( 'Search %s', 'woothemes-sensei' ), $plural ),
+			'not_found' =>  sprintf( __( 'No %s found', 'woothemes-sensei' ), mb_strtolower( $plural, 'UTF-8') ),
+			'not_found_in_trash' => sprintf( __( 'No %s found in Trash', 'woothemes-sensei' ), mb_strtolower( $plural, 'UTF-8') ),
+			'parent_item_colon' => '',
+			'menu_name' => sprintf( __( '%s', 'woothemes-sensei' ), $menu )
 		  );
 
 		return $labels;