woothemes / sensei

includes/shortcodes/class-sensei-legacy-shortcodes.php

Doc Comments +2 added lines, -2 removed lines

@@ -103,8 +103,8 @@
      *
      * @since 1.9.0
      *
-     * @param $title
-     * @param $shortcode_specific_override
+     * @param string $title
+     * @param string $shortcode_specific_override
      * @return string
      */
     public static function generate_shortcode_courses( $title , $shortcode_specific_override  ){

Braces +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
 
             Sensei_Templates::get_template( 'user/my-courses.php' );
 
-        }else{
+        } else{
 
             Sensei()->frontend->sensei_login_form();
 

Indentation +249 added lines, -249 removed lines

@@ -10,323 +10,323 @@ 
  */
 class Sensei_Legacy_Shortcodes {
 
-    /**
-     * Add the legacy shortcodes to WordPress
-     *
-     * @since 1.9.0
-     */
-    public static function init(){
-
-        add_shortcode( 'allcourses',      array( __CLASS__, 'all_courses' ) );
-        add_shortcode( 'newcourses',      array( __CLASS__,'new_courses' ) );
-        add_shortcode( 'featuredcourses', array( __CLASS__,'featured_courses') );
-        add_shortcode( 'freecourses',     array( __CLASS__,'free_courses') );
-        add_shortcode( 'paidcourses',     array( __CLASS__,'paid_courses') );
-        add_shortcode( 'usercourses',     array( __CLASS__,'user_courses' ) );
-
-    }
-    /**
-     * all_courses shortcode output function.
-     *
-     * The function should only be called indirectly through do_shortcode()
-     *
-     * @access public
-     * @param mixed $atts
-     * @param mixed $content (default: null)
-     * @return string
-     */
-    public static function all_courses( $atts, $content = null ) {
-
-        return self::generate_shortcode_courses( '', 'allcourses' ); // all courses but no title
-
-    } // all_courses()
-
-    /**
-     * paid_courses function.
-     *
-     * @access public
-     * @param mixed $atts
-     * @param mixed $content (default: null)
-     * @return string
-     */
-    public static function paid_courses( $atts, $content = null ) {
-
-        return self::generate_shortcode_courses( 'Paid Courses', 'paidcourses' );
-
-    } // End paid_courses()
-
-
-    /**
-     * featured_courses function.
-     *
-     * @access public
-     * @param mixed $atts
-     * @param mixed $content (default: null)
-     * @return string
-     */
-    public static function featured_courses( $atts, $content = null ) {
-
-        return self::generate_shortcode_courses( 'Featured Courses', 'featuredcourses' );
+	/**
+	 * Add the legacy shortcodes to WordPress
+	 *
+	 * @since 1.9.0
+	 */
+	public static function init(){
+
+		add_shortcode( 'allcourses',      array( __CLASS__, 'all_courses' ) );
+		add_shortcode( 'newcourses',      array( __CLASS__,'new_courses' ) );
+		add_shortcode( 'featuredcourses', array( __CLASS__,'featured_courses') );
+		add_shortcode( 'freecourses',     array( __CLASS__,'free_courses') );
+		add_shortcode( 'paidcourses',     array( __CLASS__,'paid_courses') );
+		add_shortcode( 'usercourses',     array( __CLASS__,'user_courses' ) );
+
+	}
+	/**
+	 * all_courses shortcode output function.
+	 *
+	 * The function should only be called indirectly through do_shortcode()
+	 *
+	 * @access public
+	 * @param mixed $atts
+	 * @param mixed $content (default: null)
+	 * @return string
+	 */
+	public static function all_courses( $atts, $content = null ) {
+
+		return self::generate_shortcode_courses( '', 'allcourses' ); // all courses but no title
+
+	} // all_courses()
+
+	/**
+	 * paid_courses function.
+	 *
+	 * @access public
+	 * @param mixed $atts
+	 * @param mixed $content (default: null)
+	 * @return string
+	 */
+	public static function paid_courses( $atts, $content = null ) {
+
+		return self::generate_shortcode_courses( 'Paid Courses', 'paidcourses' );
+
+	} // End paid_courses()
+
+
+	/**
+	 * featured_courses function.
+	 *
+	 * @access public
+	 * @param mixed $atts
+	 * @param mixed $content (default: null)
+	 * @return string
+	 */
+	public static function featured_courses( $atts, $content = null ) {
+
+		return self::generate_shortcode_courses( 'Featured Courses', 'featuredcourses' );
 
-    } // End featured_courses()
+	} // End featured_courses()
 
-    /**
-     * shortcode_free_courses function.
-     *
-     * @access public
-     * @param mixed $atts
-     * @param mixed $content (default: null)
-     * @return string
-     */
-    public static function free_courses( $atts, $content = null ) {
+	/**
+	 * shortcode_free_courses function.
+	 *
+	 * @access public
+	 * @param mixed $atts
+	 * @param mixed $content (default: null)
+	 * @return string
+	 */
+	public static function free_courses( $atts, $content = null ) {
 
-        return self::generate_shortcode_courses( 'Free Courses', 'freecourses' );
+		return self::generate_shortcode_courses( 'Free Courses', 'freecourses' );
 
-    } // End free_courses()
+	} // End free_courses()
 
-    /**
-     * shortcode_new_courses function.
-     *
-     * @access public
-     * @param mixed $atts
-     * @param mixed $content (default: null)
-     * @return string
-     */
-    public static function new_courses( $atts, $content = null ) {
+	/**
+	 * shortcode_new_courses function.
+	 *
+	 * @access public
+	 * @param mixed $atts
+	 * @param mixed $content (default: null)
+	 * @return string
+	 */
+	public static function new_courses( $atts, $content = null ) {
 
-        return self::generate_shortcode_courses( 'New Courses', 'newcourses' );
+		return self::generate_shortcode_courses( 'New Courses', 'newcourses' );
 
-    } // End new_courses()
+	} // End new_courses()
 
-    /**
-     * Generate courses adding a title.
-     *
-     * @since 1.9.0
-     *
-     * @param $title
-     * @param $shortcode_specific_override
-     * @return string
-     */
-    public static function generate_shortcode_courses( $title , $shortcode_specific_override  ){
+	/**
+	 * Generate courses adding a title.
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param $title
+	 * @param $shortcode_specific_override
+	 * @return string
+	 */
+	public static function generate_shortcode_courses( $title , $shortcode_specific_override  ){
 
-        global  $shortcode_override, $posts_array;
+		global  $shortcode_override, $posts_array;
 
-        $shortcode_override = $shortcode_specific_override;
+		$shortcode_override = $shortcode_specific_override;
 
-        // do not show this short code if there is a shortcode int he url and
-        // this specific shortcode is not the one requested in the ur.
-        $specific_shortcode_requested = isset( $_GET['action'] ) ?  sanitize_text_field(  $_GET['action']  ) : '';
-        if( ! empty( $specific_shortcode_requested) &&
-            $specific_shortcode_requested != $shortcode_override ){
+		// do not show this short code if there is a shortcode int he url and
+		// this specific shortcode is not the one requested in the ur.
+		$specific_shortcode_requested = isset( $_GET['action'] ) ?  sanitize_text_field(  $_GET['action']  ) : '';
+		if( ! empty( $specific_shortcode_requested) &&
+			$specific_shortcode_requested != $shortcode_override ){
 
-            return '';
+			return '';
 
-        }
+		}
 
-        // loop and get all courses html
-        ob_start();
-        self::initialise_legacy_course_loop();
-        $courses = ob_get_clean();
+		// loop and get all courses html
+		ob_start();
+		self::initialise_legacy_course_loop();
+		$courses = ob_get_clean();
 
-        $content = '';
-        if( count( $posts_array ) > 0 ){
+		$content = '';
+		if( count( $posts_array ) > 0 ){
 
-            $before = empty($title)? '' : '<header class="archive-header"><h2>'. $title .'</h2></header>';
-            $before .= '<section id="main-course" class="course-container">';
+			$before = empty($title)? '' : '<header class="archive-header"><h2>'. $title .'</h2></header>';
+			$before .= '<section id="main-course" class="course-container">';
 
-            $after = '</section>';
+			$after = '</section>';
 
-            //assemble
-            $content = $before . $courses . $after;
+			//assemble
+			$content = $before . $courses . $after;
 
-        }
+		}
 
-        return $content;
+		return $content;
 
-    }// end generate_shortcode_courses
+	}// end generate_shortcode_courses
 
 
-    /**
-     * user_courses function.
-     *
-     * @access public
-     * @param mixed $atts
-     * @param mixed $content (default: null)
-     * @return string
-     */
-    public static function user_courses( $atts, $content = null ) {
-        global $shortcode_override;
-        extract( shortcode_atts( array(	'amount' => 0 ), $atts ) );
+	/**
+	 * user_courses function.
+	 *
+	 * @access public
+	 * @param mixed $atts
+	 * @param mixed $content (default: null)
+	 * @return string
+	 */
+	public static function user_courses( $atts, $content = null ) {
+		global $shortcode_override;
+		extract( shortcode_atts( array(	'amount' => 0 ), $atts ) );
 
-        $shortcode_override = 'usercourses';
+		$shortcode_override = 'usercourses';
 
-        ob_start();
+		ob_start();
 
-        if( is_user_logged_in() ){
+		if( is_user_logged_in() ){
 
-            Sensei_Templates::get_template( 'user/my-courses.php' );
+			Sensei_Templates::get_template( 'user/my-courses.php' );
 
-        }else{
+		}else{
 
-            Sensei()->frontend->sensei_login_form();
+			Sensei()->frontend->sensei_login_form();
 
-        }
+		}
 
-        $content = ob_get_clean();
-        return $content;
+		$content = ob_get_clean();
+		return $content;
 
-    } // End user_courses()
+	} // End user_courses()
 
-    /**
-     * This function is simply to honor the legacy
-     * loop-course.php for the old shortcodes.
-     * @since 1.9.0
-     */
-    public static function initialise_legacy_course_loop(){
+	/**
+	 * This function is simply to honor the legacy
+	 * loop-course.php for the old shortcodes.
+	 * @since 1.9.0
+	 */
+	public static function initialise_legacy_course_loop(){
 
-        global  $post, $wp_query, $shortcode_override, $course_excludes;
+		global  $post, $wp_query, $shortcode_override, $course_excludes;
 
-        // Handle Query Type
-        $query_type = '';
+		// Handle Query Type
+		$query_type = '';
 
-        if ( isset( $_GET[ 'action' ] ) && ( '' != esc_html( $_GET[ 'action' ] ) ) ) {
-            $query_type = esc_html( $_GET[ 'action' ] );
-        } // End If Statement
+		if ( isset( $_GET[ 'action' ] ) && ( '' != esc_html( $_GET[ 'action' ] ) ) ) {
+			$query_type = esc_html( $_GET[ 'action' ] );
+		} // End If Statement
 
-        if ( '' != $shortcode_override ) {
-            $query_type = $shortcode_override;
-        } // End If Statement
+		if ( '' != $shortcode_override ) {
+			$query_type = $shortcode_override;
+		} // End If Statement
 
-        if ( !is_array( $course_excludes ) ) { $course_excludes = array(); }
+		if ( !is_array( $course_excludes ) ) { $course_excludes = array(); }
 
-        // Check that query returns results
-        // Handle Pagination
-        $paged = $wp_query->get( 'paged' );
-        $paged = empty( $paged ) ? 1 : $paged;
+		// Check that query returns results
+		// Handle Pagination
+		$paged = $wp_query->get( 'paged' );
+		$paged = empty( $paged ) ? 1 : $paged;
 
 
-        // Check for pagination settings
-        if ( isset( Sensei()->settings->settings[ 'course_archive_amount' ] ) && ( 0 < absint( Sensei()->settings->settings[ 'course_archive_amount' ] ) ) ) {
+		// Check for pagination settings
+		if ( isset( Sensei()->settings->settings[ 'course_archive_amount' ] ) && ( 0 < absint( Sensei()->settings->settings[ 'course_archive_amount' ] ) ) ) {
 
-            $amount = absint( Sensei()->settings->settings[ 'course_archive_amount' ] );
+			$amount = absint( Sensei()->settings->settings[ 'course_archive_amount' ] );
 
-        } else {
+		} else {
 
-            $amount = $wp_query->get( 'posts_per_page' );
+			$amount = $wp_query->get( 'posts_per_page' );
 
-        } // End If Statement
+		} // End If Statement
 
-        // This is not a paginated page (or it's simply the first page of a paginated page/post)
+		// This is not a paginated page (or it's simply the first page of a paginated page/post)
 
-        global $posts_array;
-        $course_includes   = array();
+		global $posts_array;
+		$course_includes   = array();
 
-        $query_args = Sensei()->course->get_archive_query_args( $shortcode_override, $amount, $course_includes, $course_excludes );
-        $course_query = new WP_Query( $query_args );
-        $posts_array = $course_query->get_posts();
+		$query_args = Sensei()->course->get_archive_query_args( $shortcode_override, $amount, $course_includes, $course_excludes );
+		$course_query = new WP_Query( $query_args );
+		$posts_array = $course_query->get_posts();
 
-        // output the courses
-        if( ! empty( $posts_array ) ) {
+		// output the courses
+		if( ! empty( $posts_array ) ) {
 
-            //output all courses for current query
-            self::loop_courses( $course_query, $amount );
+			//output all courses for current query
+			self::loop_courses( $course_query, $amount );
 
-        }
+		}
 
-    }
+	}
 
-    /**
-     * Loop through courses in the query and output the infomration needed
-     *
-     * @since 1.9.0
-     *
-     * @param WP_Query $course_query
-     */
-    public static function loop_courses( $course_query, $amount ){
+	/**
+	 * Loop through courses in the query and output the infomration needed
+	 *
+	 * @since 1.9.0
+	 *
+	 * @param WP_Query $course_query
+	 */
+	public static function loop_courses( $course_query, $amount ){
 
-        global $shortcode_override, $posts_array, $post, $wp_query, $shortcode_override, $course_excludes, $course_includes;
+		global $shortcode_override, $posts_array, $post, $wp_query, $shortcode_override, $course_excludes, $course_includes;
 
-        if ( count( $course_query->get_posts() ) > 0 ) {
+		if ( count( $course_query->get_posts() ) > 0 ) {
 
-            do_action( 'sensei_course_archive_header', $shortcode_override );
+			do_action( 'sensei_course_archive_header', $shortcode_override );
 
-            foreach ( $course_query->get_posts() as $course){
+			foreach ( $course_query->get_posts() as $course){
 
-                // Make sure the other loops dont include the same post twice!
-                array_push( $course_excludes, $course->ID );
+				// Make sure the other loops dont include the same post twice!
+				array_push( $course_excludes, $course->ID );
 
-                // output the course markup
-                self::the_course( $course->ID );
+				// output the course markup
+				self::the_course( $course->ID );
 
-            } // End For Loop
+			} // End For Loop
 
-            // More and Prev links
-            $posts_array_query = new WP_Query(Sensei()->course->course_query( $shortcode_override, $amount, $course_includes, $course_excludes ) );
-            $posts_array       = $posts_array_query->get_posts();
-            $max_pages = $course_query->found_posts / $amount;
-            if ( '' != $shortcode_override && ( $max_pages > $course_query->get( 'paged' ) ) ) {
+			// More and Prev links
+			$posts_array_query = new WP_Query(Sensei()->course->course_query( $shortcode_override, $amount, $course_includes, $course_excludes ) );
+			$posts_array       = $posts_array_query->get_posts();
+			$max_pages = $course_query->found_posts / $amount;
+			if ( '' != $shortcode_override && ( $max_pages > $course_query->get( 'paged' ) ) ) {
 
-                switch( $shortcode_override ){
-                    case 'paidcourses':
-                        $filter = 'paid';
-                        break;
-                    case 'featuredcourses':
-                        $filter = 'featured';
-                        break;
-                    case 'freecourses':
-                        $filter = 'free';
-                        break;
-                    default:
-                        $filter = '';
-                        break;
-                }
+				switch( $shortcode_override ){
+					case 'paidcourses':
+						$filter = 'paid';
+						break;
+					case 'featuredcourses':
+						$filter = 'featured';
+						break;
+					case 'freecourses':
+						$filter = 'free';
+						break;
+					default:
+						$filter = '';
+						break;
+				}
 
-                $quer_args = array();
-                $quer_args[ 'paged' ] = '2';
-                if( !empty( $filter ) ){
-                    $quer_args[ 'course_filter' ] = $filter;
-                }
+				$quer_args = array();
+				$quer_args[ 'paged' ] = '2';
+				if( !empty( $filter ) ){
+					$quer_args[ 'course_filter' ] = $filter;
+				}
 
-                $course_pagination_link = get_post_type_archive_link( 'course' );
-                $more_link_text = esc_html( Sensei()->settings->settings[ 'course_archive_more_link_text' ] );
-                $more_link_url =  esc_url( add_query_arg( $quer_args, $course_pagination_link ) );
+				$course_pagination_link = get_post_type_archive_link( 'course' );
+				$more_link_text = esc_html( Sensei()->settings->settings[ 'course_archive_more_link_text' ] );
+				$more_link_url =  esc_url( add_query_arg( $quer_args, $course_pagination_link ) );
 
-                // next/more
-                $html  = '<div class="navigation"><div class="nav-next">';
-                $html .= '<a href="' . $more_link_url . '">';
-                $html .= $more_link_text;
-                $html .= '<span class="meta-nav"></span></a></div>';
+				// next/more
+				$html  = '<div class="navigation"><div class="nav-next">';
+				$html .= '<a href="' . $more_link_url . '">';
+				$html .= $more_link_text;
+				$html .= '<span class="meta-nav"></span></a></div>';
 
-                echo apply_filters( 'course_archive_next_link', $html );
+				echo apply_filters( 'course_archive_next_link', $html );
 
-            } // End If Statement
+			} // End If Statement
 
-        } // End If Statement
-    }
+		} // End If Statement
+	}
 
-    /**
-     * Print a single course markup
-     *
-     * @param $course_id
-     */
-    public static function the_course( $course_id ){
+	/**
+	 * Print a single course markup
+	 *
+	 * @param $course_id
+	 */
+	public static function the_course( $course_id ){
 
-        // Get meta data
-        $course =  get_post( $course_id );
-        $user_info = get_userdata( absint( $course->post_author ) );
-        $author_link = get_author_posts_url( absint( $course->post_author ) );
-        $author_display_name = $user_info->display_name;
-        $author_id = $course->post_author;
-        $category_output = get_the_term_list( $course_id, 'course-category', '', ', ', '' );
-        $preview_lesson_count = intval( Sensei()->course->course_lesson_preview_count( $course_id ) );
-        $is_user_taking_course = Sensei_Utils::user_started_course( $course_id, get_current_user_id() );
-        ?>
+		// Get meta data
+		$course =  get_post( $course_id );
+		$user_info = get_userdata( absint( $course->post_author ) );
+		$author_link = get_author_posts_url( absint( $course->post_author ) );
+		$author_display_name = $user_info->display_name;
+		$author_id = $course->post_author;
+		$category_output = get_the_term_list( $course_id, 'course-category', '', ', ', '' );
+		$preview_lesson_count = intval( Sensei()->course->course_lesson_preview_count( $course_id ) );
+		$is_user_taking_course = Sensei_Utils::user_started_course( $course_id, get_current_user_id() );
+		?>
 
         <article class="<?php echo esc_attr( join( ' ', get_post_class( array( 'course', 'post' ), $course_id ) ) ); ?>">
             <?php
-            // so that legacy shortcodes work with thir party plugins that wants to hook in
-            do_action('sensei_course_content_before',$course );
-            ?>
+			// so that legacy shortcodes work with thir party plugins that wants to hook in
+			do_action('sensei_course_content_before',$course );
+			?>
             <div class="course-content">
 
                 <?php Sensei()->course->course_image($course_id); ?>
@@ -362,7 +362,7 @@ 
                     </p>
 
                     <?php if ( 0 < $preview_lesson_count && !$is_user_taking_course ) {
-                        $preview_lessons = sprintf( __( '(%d preview lessons)', 'woothemes-sensei' ), $preview_lesson_count ); ?>
+						$preview_lessons = sprintf( __( '(%d preview lessons)', 'woothemes-sensei' ), $preview_lesson_count ); ?>
                         <p class="sensei-free-lessons">
                             <a href="<?php echo get_permalink( $course_id ); ?>"><?php _e( 'Preview this course', 'woothemes-sensei' ) ?>
                             </a> - <?php echo $preview_lessons; ?>
@@ -373,15 +373,15 @@ 
 
             </div>
             <?php
-            // so that legacy shortcodes work with thir party plugins that wants to hook in
-            do_action('sensei_course_content_after', $course);
-            ?>
+			// so that legacy shortcodes work with thir party plugins that wants to hook in
+			do_action('sensei_course_content_after', $course);
+			?>
 
         </article>
 
         <?php
 
 
-    } // end the_course
+	} // end the_course
 
 }// end class legacy shortcodes
\ No newline at end of file

Spacing +79 added lines, -79 removed lines

@@ -15,14 +15,14 @@ 
      *
      * @since 1.9.0
      */
-    public static function init(){
+    public static function init() {
 
-        add_shortcode( 'allcourses',      array( __CLASS__, 'all_courses' ) );
-        add_shortcode( 'newcourses',      array( __CLASS__,'new_courses' ) );
-        add_shortcode( 'featuredcourses', array( __CLASS__,'featured_courses') );
-        add_shortcode( 'freecourses',     array( __CLASS__,'free_courses') );
-        add_shortcode( 'paidcourses',     array( __CLASS__,'paid_courses') );
-        add_shortcode( 'usercourses',     array( __CLASS__,'user_courses' ) );
+        add_shortcode('allcourses', array(__CLASS__, 'all_courses'));
+        add_shortcode('newcourses', array(__CLASS__, 'new_courses'));
+        add_shortcode('featuredcourses', array(__CLASS__, 'featured_courses'));
+        add_shortcode('freecourses', array(__CLASS__, 'free_courses'));
+        add_shortcode('paidcourses', array(__CLASS__, 'paid_courses'));
+        add_shortcode('usercourses', array(__CLASS__, 'user_courses'));
 
     }
     /**
@@ -35,9 +35,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function all_courses( $atts, $content = null ) {
+    public static function all_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( '', 'allcourses' ); // all courses but no title
+        return self::generate_shortcode_courses('', 'allcourses'); // all courses but no title
 
     } // all_courses()
 
@@ -49,9 +49,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function paid_courses( $atts, $content = null ) {
+    public static function paid_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'Paid Courses', 'paidcourses' );
+        return self::generate_shortcode_courses('Paid Courses', 'paidcourses');
 
     } // End paid_courses()
 
@@ -64,9 +64,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function featured_courses( $atts, $content = null ) {
+    public static function featured_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'Featured Courses', 'featuredcourses' );
+        return self::generate_shortcode_courses('Featured Courses', 'featuredcourses');
 
     } // End featured_courses()
 
@@ -78,9 +78,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function free_courses( $atts, $content = null ) {
+    public static function free_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'Free Courses', 'freecourses' );
+        return self::generate_shortcode_courses('Free Courses', 'freecourses');
 
     } // End free_courses()
 
@@ -92,9 +92,9 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function new_courses( $atts, $content = null ) {
+    public static function new_courses($atts, $content = null) {
 
-        return self::generate_shortcode_courses( 'New Courses', 'newcourses' );
+        return self::generate_shortcode_courses('New Courses', 'newcourses');
 
     } // End new_courses()
 
@@ -107,7 +107,7 @@ 
      * @param $shortcode_specific_override
      * @return string
      */
-    public static function generate_shortcode_courses( $title , $shortcode_specific_override  ){
+    public static function generate_shortcode_courses($title, $shortcode_specific_override) {
 
         global  $shortcode_override, $posts_array;
 
@@ -115,9 +115,9 @@ 
 
         // do not show this short code if there is a shortcode int he url and
         // this specific shortcode is not the one requested in the ur.
-        $specific_shortcode_requested = isset( $_GET['action'] ) ?  sanitize_text_field(  $_GET['action']  ) : '';
-        if( ! empty( $specific_shortcode_requested) &&
-            $specific_shortcode_requested != $shortcode_override ){
+        $specific_shortcode_requested = isset($_GET['action']) ? sanitize_text_field($_GET['action']) : '';
+        if ( ! empty($specific_shortcode_requested) &&
+            $specific_shortcode_requested != $shortcode_override) {
 
             return '';
 
@@ -129,15 +129,15 @@ 
         $courses = ob_get_clean();
 
         $content = '';
-        if( count( $posts_array ) > 0 ){
+        if (count($posts_array) > 0) {
 
-            $before = empty($title)? '' : '<header class="archive-header"><h2>'. $title .'</h2></header>';
+            $before = empty($title) ? '' : '<header class="archive-header"><h2>'.$title.'</h2></header>';
             $before .= '<section id="main-course" class="course-container">';
 
             $after = '</section>';
 
             //assemble
-            $content = $before . $courses . $after;
+            $content = $before.$courses.$after;
 
         }
 
@@ -154,19 +154,19 @@ 
      * @param mixed $content (default: null)
      * @return string
      */
-    public static function user_courses( $atts, $content = null ) {
+    public static function user_courses($atts, $content = null) {
         global $shortcode_override;
-        extract( shortcode_atts( array(	'amount' => 0 ), $atts ) );
+        extract(shortcode_atts(array('amount' => 0), $atts));
 
         $shortcode_override = 'usercourses';
 
         ob_start();
 
-        if( is_user_logged_in() ){
+        if (is_user_logged_in()) {
 
-            Sensei_Templates::get_template( 'user/my-courses.php' );
+            Sensei_Templates::get_template('user/my-courses.php');
 
-        }else{
+        } else {
 
             Sensei()->frontend->sensei_login_form();
 
@@ -182,54 +182,54 @@ 
      * loop-course.php for the old shortcodes.
      * @since 1.9.0
      */
-    public static function initialise_legacy_course_loop(){
+    public static function initialise_legacy_course_loop() {
 
         global  $post, $wp_query, $shortcode_override, $course_excludes;
 
         // Handle Query Type
         $query_type = '';
 
-        if ( isset( $_GET[ 'action' ] ) && ( '' != esc_html( $_GET[ 'action' ] ) ) ) {
-            $query_type = esc_html( $_GET[ 'action' ] );
+        if (isset($_GET['action']) && ('' != esc_html($_GET['action']))) {
+            $query_type = esc_html($_GET['action']);
         } // End If Statement
 
-        if ( '' != $shortcode_override ) {
+        if ('' != $shortcode_override) {
             $query_type = $shortcode_override;
         } // End If Statement
 
-        if ( !is_array( $course_excludes ) ) { $course_excludes = array(); }
+        if ( ! is_array($course_excludes)) { $course_excludes = array(); }
 
         // Check that query returns results
         // Handle Pagination
-        $paged = $wp_query->get( 'paged' );
-        $paged = empty( $paged ) ? 1 : $paged;
+        $paged = $wp_query->get('paged');
+        $paged = empty($paged) ? 1 : $paged;
 
 
         // Check for pagination settings
-        if ( isset( Sensei()->settings->settings[ 'course_archive_amount' ] ) && ( 0 < absint( Sensei()->settings->settings[ 'course_archive_amount' ] ) ) ) {
+        if (isset(Sensei()->settings->settings['course_archive_amount']) && (0 < absint(Sensei()->settings->settings['course_archive_amount']))) {
 
-            $amount = absint( Sensei()->settings->settings[ 'course_archive_amount' ] );
+            $amount = absint(Sensei()->settings->settings['course_archive_amount']);
 
         } else {
 
-            $amount = $wp_query->get( 'posts_per_page' );
+            $amount = $wp_query->get('posts_per_page');
 
         } // End If Statement
 
         // This is not a paginated page (or it's simply the first page of a paginated page/post)
 
         global $posts_array;
-        $course_includes   = array();
+        $course_includes = array();
 
-        $query_args = Sensei()->course->get_archive_query_args( $shortcode_override, $amount, $course_includes, $course_excludes );
-        $course_query = new WP_Query( $query_args );
+        $query_args = Sensei()->course->get_archive_query_args($shortcode_override, $amount, $course_includes, $course_excludes);
+        $course_query = new WP_Query($query_args);
         $posts_array = $course_query->get_posts();
 
         // output the courses
-        if( ! empty( $posts_array ) ) {
+        if ( ! empty($posts_array)) {
 
             //output all courses for current query
-            self::loop_courses( $course_query, $amount );
+            self::loop_courses($course_query, $amount);
 
         }
 
@@ -242,31 +242,31 @@ 
      *
      * @param WP_Query $course_query
      */
-    public static function loop_courses( $course_query, $amount ){
+    public static function loop_courses($course_query, $amount) {
 
         global $shortcode_override, $posts_array, $post, $wp_query, $shortcode_override, $course_excludes, $course_includes;
 
-        if ( count( $course_query->get_posts() ) > 0 ) {
+        if (count($course_query->get_posts()) > 0) {
 
-            do_action( 'sensei_course_archive_header', $shortcode_override );
+            do_action('sensei_course_archive_header', $shortcode_override);
 
-            foreach ( $course_query->get_posts() as $course){
+            foreach ($course_query->get_posts() as $course) {
 
                 // Make sure the other loops dont include the same post twice!
-                array_push( $course_excludes, $course->ID );
+                array_push($course_excludes, $course->ID);
 
                 // output the course markup
-                self::the_course( $course->ID );
+                self::the_course($course->ID);
 
             } // End For Loop
 
             // More and Prev links
-            $posts_array_query = new WP_Query(Sensei()->course->course_query( $shortcode_override, $amount, $course_includes, $course_excludes ) );
+            $posts_array_query = new WP_Query(Sensei()->course->course_query($shortcode_override, $amount, $course_includes, $course_excludes));
             $posts_array       = $posts_array_query->get_posts();
             $max_pages = $course_query->found_posts / $amount;
-            if ( '' != $shortcode_override && ( $max_pages > $course_query->get( 'paged' ) ) ) {
+            if ('' != $shortcode_override && ($max_pages > $course_query->get('paged'))) {
 
-                switch( $shortcode_override ){
+                switch ($shortcode_override) {
                     case 'paidcourses':
                         $filter = 'paid';
                         break;
@@ -282,22 +282,22 @@ 
                 }
 
                 $quer_args = array();
-                $quer_args[ 'paged' ] = '2';
-                if( !empty( $filter ) ){
-                    $quer_args[ 'course_filter' ] = $filter;
+                $quer_args['paged'] = '2';
+                if ( ! empty($filter)) {
+                    $quer_args['course_filter'] = $filter;
                 }
 
-                $course_pagination_link = get_post_type_archive_link( 'course' );
-                $more_link_text = esc_html( Sensei()->settings->settings[ 'course_archive_more_link_text' ] );
-                $more_link_url =  esc_url( add_query_arg( $quer_args, $course_pagination_link ) );
+                $course_pagination_link = get_post_type_archive_link('course');
+                $more_link_text = esc_html(Sensei()->settings->settings['course_archive_more_link_text']);
+                $more_link_url = esc_url(add_query_arg($quer_args, $course_pagination_link));
 
                 // next/more
                 $html  = '<div class="navigation"><div class="nav-next">';
-                $html .= '<a href="' . $more_link_url . '">';
+                $html .= '<a href="'.$more_link_url.'">';
                 $html .= $more_link_text;
                 $html .= '<span class="meta-nav"></span></a></div>';
 
-                echo apply_filters( 'course_archive_next_link', $html );
+                echo apply_filters('course_archive_next_link', $html);
 
             } // End If Statement
 
@@ -309,23 +309,23 @@ 
      *
      * @param $course_id
      */
-    public static function the_course( $course_id ){
+    public static function the_course($course_id) {
 
         // Get meta data
-        $course =  get_post( $course_id );
-        $user_info = get_userdata( absint( $course->post_author ) );
-        $author_link = get_author_posts_url( absint( $course->post_author ) );
+        $course = get_post($course_id);
+        $user_info = get_userdata(absint($course->post_author));
+        $author_link = get_author_posts_url(absint($course->post_author));
         $author_display_name = $user_info->display_name;
         $author_id = $course->post_author;
-        $category_output = get_the_term_list( $course_id, 'course-category', '', ', ', '' );
-        $preview_lesson_count = intval( Sensei()->course->course_lesson_preview_count( $course_id ) );
-        $is_user_taking_course = Sensei_Utils::user_started_course( $course_id, get_current_user_id() );
+        $category_output = get_the_term_list($course_id, 'course-category', '', ', ', '');
+        $preview_lesson_count = intval(Sensei()->course->course_lesson_preview_count($course_id));
+        $is_user_taking_course = Sensei_Utils::user_started_course($course_id, get_current_user_id());
         ?>
 
-        <article class="<?php echo esc_attr( join( ' ', get_post_class( array( 'course', 'post' ), $course_id ) ) ); ?>">
+        <article class="<?php echo esc_attr(join(' ', get_post_class(array('course', 'post'), $course_id))); ?>">
             <?php
             // so that legacy shortcodes work with thir party plugins that wants to hook in
-            do_action('sensei_course_content_before',$course );
+            do_action('sensei_course_content_before', $course);
             ?>
             <div class="course-content">
 
@@ -341,19 +341,19 @@ 
 
                     <p class="sensei-course-meta">
 
-                        <?php if ( isset( Sensei()->settings->settings[ 'course_author' ] ) && ( Sensei()->settings->settings[ 'course_author' ] ) ) { ?>
-                            <span class="course-author"><?php _e( 'by ', 'woothemes-sensei' ); ?><a href="<?php echo $author_link; ?>" title="<?php echo esc_attr( $author_display_name ); ?>"><?php echo esc_html( $author_display_name   ); ?></a></span>
+                        <?php if (isset(Sensei()->settings->settings['course_author']) && (Sensei()->settings->settings['course_author'])) { ?>
+                            <span class="course-author"><?php _e('by ', 'woothemes-sensei'); ?><a href="<?php echo $author_link; ?>" title="<?php echo esc_attr($author_display_name); ?>"><?php echo esc_html($author_display_name); ?></a></span>
                         <?php } // End If Statement ?>
 
                         <span class="course-lesson-count">
-                                    <?php echo Sensei()->course->course_lesson_count( $course_id ) . '&nbsp;' .  __( 'Lessons', 'woothemes-sensei' ); ?>
+                                    <?php echo Sensei()->course->course_lesson_count($course_id).'&nbsp;'.__('Lessons', 'woothemes-sensei'); ?>
                                 </span>
 
-                        <?php if ( '' != $category_output ) { ?>
-                            <span class="course-category"><?php echo sprintf( __( 'in %s', 'woothemes-sensei' ), $category_output ); ?></span>
+                        <?php if ('' != $category_output) { ?>
+                            <span class="course-category"><?php echo sprintf(__('in %s', 'woothemes-sensei'), $category_output); ?></span>
                         <?php } // End If Statement ?>
 
-                        <?php sensei_simple_course_price( $course_id ); ?>
+                        <?php sensei_simple_course_price($course_id); ?>
 
                     </p>
 
@@ -361,10 +361,10 @@ 
 
                     </p>
 
-                    <?php if ( 0 < $preview_lesson_count && !$is_user_taking_course ) {
-                        $preview_lessons = sprintf( __( '(%d preview lessons)', 'woothemes-sensei' ), $preview_lesson_count ); ?>
+                    <?php if (0 < $preview_lesson_count && ! $is_user_taking_course) {
+                        $preview_lessons = sprintf(__('(%d preview lessons)', 'woothemes-sensei'), $preview_lesson_count); ?>
                         <p class="sensei-free-lessons">
-                            <a href="<?php echo get_permalink( $course_id ); ?>"><?php _e( 'Preview this course', 'woothemes-sensei' ) ?>
+                            <a href="<?php echo get_permalink($course_id); ?>"><?php _e('Preview this course', 'woothemes-sensei') ?>
                             </a> - <?php echo $preview_lessons; ?>
                         </p>
                     <?php } ?>

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

Doc Comments +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
      *
      * @since 1.9.0
      *
-     * @param array $category_ids
+     * @param array $categories
      * @return array
      */
     public function generate_term_ids( $categories = array() ){

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
 

Spacing +23 added lines, -23 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 /**
  *
  * Renders the [sensei_course_categories] shortcode. Show the list or course categories as links to their archive
@@ -37,22 +37,22 @@ 
      * @param string $content
      * @param string $shortcode the shortcode that was called for this instance
      */
-    public function __construct( $attributes, $content, $shortcode ){
+    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;
+        $hide_empty = isset($attributes['hide_empty']) ? $attributes['hide_empty'] : 'false';
+        $this->hide_empty = 'true' == $hide_empty ? true : false;
 
         $this->setup_course_categories();
 
@@ -63,7 +63,7 @@ 
      *
      * @return mixed
      */
-    public function setup_course_categories(){
+    public function setup_course_categories() {
 
         $args = array(
             'orderby'       => $this->orderby,
@@ -85,11 +85,11 @@ 
      *
      * @return string $content
      */
-    public function render(){
+    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');
 
         }
 
@@ -97,10 +97,10 @@ 
 
         //set the wp_query to the current messages query
         $terms_html .= '<ul class="sensei course-categories">';
-        foreach( $this->sensei_course_taxonomy_terms as $category ){
+        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>';
@@ -117,20 +117,20 @@ 
      * @param array $category_ids
      * @return array
      */
-    public function generate_term_ids( $categories = array() ){
+    public function generate_term_ids($categories = array()) {
 
         $cat_ids = array();
 
-        if ( is_array($categories) ) {
+        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');
 
                     // if the slug didn't work try the name
-                    if (!$term) {
+                    if ( ! $term) {
 
                         $term = get_term_by('name', $cat, 'course-category');
 

Braces +4 added lines, -1 removed lines

@@ -1,5 +1,8 @@
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * Sensei Analysis User Profile List Table Class

includes/shortcodes/class-sensei-shortcode-user-courses.php

Doc Comments +1 added lines, -1 removed lines

@@ -325,7 +325,7 @@
      *
      * @param array $classes
      * @param WP_Post $course
-     * @return array $classes
+     * @return string[] $classes
      */
     public function course_status_class_tagging($classes, $course){
 

Braces +8 added lines, -5 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 /**
  * This class is loaded int WP by the shortcode loader class.
  *
@@ -75,7 +78,7 @@ 
 
             $this->order =  'ASC';
 
-        }else{
+        } else{
 
             // for everything else use the value passed or the default DESC
             $this->order = isset( $attributes['order']  ) ? $attributes['order'] : 'ASC';
@@ -121,11 +124,11 @@ 
             $included_courses =  $completed_ids;
 
 
-        }elseif( 'active'==$this->status ){
+        } elseif( 'active'==$this->status ){
 
             $included_courses =  $active_ids;
 
-        }else{ // all courses
+        } else{ // all courses
 
             if( empty( $completed_ids ) ){
 
@@ -142,7 +145,7 @@ 
 
                 $included_courses = array('-1000'); // don't show any courses
 
-            }else{
+            } else{
                 $included_courses = Sensei_Utils::array_zip_merge( $active_ids, $completed_ids );
             }
 

Indentation +233 added lines, -233 removed lines

@@ -23,165 +23,165 @@ 
  */
 class Sensei_Shortcode_User_Courses implements Sensei_Shortcode_Interface {
 
-    /**
-     * @var WP_Query to help setup the query needed by the render method.
-     */
-    protected $query;
+	/**
+	 * @var WP_Query to help setup the query needed by the render method.
+	 */
+	protected $query;
 
-    /**
-     * @var string number of items to show on the current page
-     * Default: -1.
-     */
-    protected $number;
+	/**
+	 * @var string number of items to show on the current page
+	 * Default: -1.
+	 */
+	protected $number;
 
-    /**
-     * @var string ordery by course field
-     * Default: date
-     */
-    protected $orderby;
+	/**
+	 * @var string ordery by course field
+	 * Default: date
+	 */
+	protected $orderby;
 
-    /**
-     * @var string ASC or DESC
-     * Default: 'DESC'
-     */
-    protected  $order;
+	/**
+	 * @var string ASC or DESC
+	 * Default: 'DESC'
+	 */
+	protected  $order;
 
-    /**
-     * @var status can be completed or active. If none is specified all will be shown
-     */
-    protected $status;
+	/**
+	 * @var status can be completed or active. If none is specified all will be shown
+	 */
+	protected $status;
 
-    /**
-     * 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 ){
 
-        if(!  is_user_logged_in() ) {
-            return;
-        }
+		if(!  is_user_logged_in() ) {
+			return;
+		}
 
-        // set up all argument need for constructing the course query
-        $this->number = isset( $attributes['number'] ) ? $attributes['number'] : '10';
-        $this->orderby = isset( $attributes['orderby'] ) ? $attributes['orderby'] : 'title';
-        $this->status = isset( $attributes['status'] ) ? $attributes['status'] : 'all';
+		// set up all argument need for constructing the course query
+		$this->number = isset( $attributes['number'] ) ? $attributes['number'] : '10';
+		$this->orderby = isset( $attributes['orderby'] ) ? $attributes['orderby'] : 'title';
+		$this->status = isset( $attributes['status'] ) ? $attributes['status'] : 'all';
 
-        // set the default for menu_order to be ASC
-        if( 'menu_order' == $this->orderby && !isset( $attributes['order']  ) ){
+		// set the default for menu_order to be ASC
+		if( 'menu_order' == $this->orderby && !isset( $attributes['order']  ) ){
 
-            $this->order =  'ASC';
+			$this->order =  'ASC';
 
-        }else{
+		}else{
 
-            // for everything else use the value passed or the default DESC
-            $this->order = isset( $attributes['order']  ) ? $attributes['order'] : 'ASC';
+			// for everything else use the value passed or the default DESC
+			$this->order = isset( $attributes['order']  ) ? $attributes['order'] : 'ASC';
 
-        }
+		}
 
-        // setup the course query that will be used when rendering
-        $this->setup_course_query();
-    }
+		// setup the course query that will be used when rendering
+		$this->setup_course_query();
+	}
 
-    /**
-     * Sets up the object course query
-     * that will be used in the render method.
-     *
-     * @since 1.9.0
-     */
-    protected function setup_course_query(){
+	/**
+	 * Sets up the object course query
+	 * that will be used in the render method.
+	 *
+	 * @since 1.9.0
+	 */
+	protected function setup_course_query(){
 
-        $status_query = array( 'user_id' => get_current_user_id(), 'type' => 'sensei_course_status' );
-        $user_courses_logs = Sensei_Utils::sensei_check_for_activity( $status_query , true );
-        if ( !is_array($user_courses_logs) ) {
+		$status_query = array( 'user_id' => get_current_user_id(), 'type' => 'sensei_course_status' );
+		$user_courses_logs = Sensei_Utils::sensei_check_for_activity( $status_query , true );
+		if ( !is_array($user_courses_logs) ) {
 
-            $user_courses_logs = array( $user_courses_logs );
+			$user_courses_logs = array( $user_courses_logs );
 
-        }
+		}
 
-        $completed_ids = $active_ids = array();
-        foreach( $user_courses_logs as $course_status ) {
+		$completed_ids = $active_ids = array();
+		foreach( $user_courses_logs as $course_status ) {
 
-            if ( Sensei_Utils::user_completed_course( $course_status, get_current_user_id() ) ) {
+			if ( Sensei_Utils::user_completed_course( $course_status, get_current_user_id() ) ) {
 
-                $completed_ids[] = $course_status->comment_post_ID;
+				$completed_ids[] = $course_status->comment_post_ID;
 
-            } else {
+			} else {
 
-                $active_ids[] = $course_status->comment_post_ID;
+				$active_ids[] = $course_status->comment_post_ID;
 
-            }
-        }
+			}
+		}
 
-        if( 'completed' == $this->status ){
+		if( 'completed' == $this->status ){
 
-            $included_courses =  $completed_ids;
+			$included_courses =  $completed_ids;
 
 
-        }elseif( 'active'==$this->status ){
+		}elseif( 'active'==$this->status ){
 
-            $included_courses =  $active_ids;
+			$included_courses =  $active_ids;
 
-        }else{ // all courses
+		}else{ // all courses
 
-            if( empty( $completed_ids ) ){
+			if( empty( $completed_ids ) ){
 
-                add_action( 'sensei_loop_course_inside_before', array( $this, 'completed_no_course_message_output' ) );
-            }
+				add_action( 'sensei_loop_course_inside_before', array( $this, 'completed_no_course_message_output' ) );
+			}
 
-            if( empty( $active_ids ) ){
+			if( empty( $active_ids ) ){
 
-                add_action( 'sensei_loop_course_inside_before', array( $this, 'active_no_course_message_output' ) );
+				add_action( 'sensei_loop_course_inside_before', array( $this, 'active_no_course_message_output' ) );
 
-            }
+			}
 
-            if( empty( $completed_ids ) &&  empty( $active_ids )  ){
+			if( empty( $completed_ids ) &&  empty( $active_ids )  ){
 
-                $included_courses = array('-1000'); // don't show any courses
+				$included_courses = array('-1000'); // don't show any courses
 
-            }else{
-                $included_courses = Sensei_Utils::array_zip_merge( (array)$active_ids, (array)$completed_ids );
-            }
+			}else{
+				$included_courses = Sensei_Utils::array_zip_merge( (array)$active_ids, (array)$completed_ids );
+			}
+
+
+		}
+
+		// temporary work around to hide pagination on the courses page
+		// this is in place until we can load the course for each tab via ajax
+		// if the shortcode is not active or in active and the active and completed
+		// tabs show up.
+		$number_of_posts = $this->number;
+		if( 'active' != $this->status && 'complete' != $this->status  ){
+			$number_of_posts = 1000;
+		}
+
+		// course query parameters
+		$query_var_paged = get_query_var('paged');
+		$query_args = array(
+			'post_type'        => 'course',
+			'post_status'      => 'publish',
+			'orderby'          => $this->orderby,
+			'order'            => $this->order,
+			'paged' => empty( $query_var_paged )? 1 : $query_var_paged,
+			'posts_per_page'   => $number_of_posts,
+			'post__in'         => $included_courses,
+		);
+
+		$this->query = new WP_Query( $query_args );
 
+	}// end setup _course_query
 
-        }
-
-        // temporary work around to hide pagination on the courses page
-        // this is in place until we can load the course for each tab via ajax
-        // if the shortcode is not active or in active and the active and completed
-        // tabs show up.
-        $number_of_posts = $this->number;
-        if( 'active' != $this->status && 'complete' != $this->status  ){
-            $number_of_posts = 1000;
-        }
-
-        // course query parameters
-        $query_var_paged = get_query_var('paged');
-        $query_args = array(
-            'post_type'        => 'course',
-            'post_status'      => 'publish',
-            'orderby'          => $this->orderby,
-            'order'            => $this->order,
-            'paged' => empty( $query_var_paged )? 1 : $query_var_paged,
-            'posts_per_page'   => $number_of_posts,
-            'post__in'         => $included_courses,
-        );
-
-        $this->query = new WP_Query( $query_args );
-
-    }// end setup _course_query
-
-    /**
-     * Output the message that tells the user they have
-     * no completed courses.
-     *
-     * @since 1.9.0
-     */
-    public function completed_no_course_message_output(){
-        ?>
+	/**
+	 * Output the message that tells the user they have
+	 * no completed courses.
+	 *
+	 * @since 1.9.0
+	 */
+	public function completed_no_course_message_output(){
+		?>
         <li class="user-completed">
             <div class="sensei-message info">
 
@@ -190,16 +190,16 @@ 
             </div>
         </li>
         <?php
-    }
+	}
 
-    /**
-     * Output the message that tells the user they have
-     * no active courses.
-     *
-     * @since 1.9.0
-     */
-    public function active_no_course_message_output(){
-        ?>
+	/**
+	 * Output the message that tells the user they have
+	 * no active courses.
+	 *
+	 * @since 1.9.0
+	 */
+	public function active_no_course_message_output(){
+		?>
 
         <li class="user-active">
             <div class="sensei-message info">
@@ -215,141 +215,141 @@ 
             </div>
         </li>
         <?php
-    }
+	}
 
-    /**
-     * 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(){
 
-        global $wp_query;
+		global $wp_query;
 
-        if(!  is_user_logged_in() ) {
-            return '';
-        }
+		if(!  is_user_logged_in() ) {
+			return '';
+		}
 
-        // keep a reference to old query
-        $current_global_query = $wp_query;
+		// keep a reference to old query
+		$current_global_query = $wp_query;
 
-        // assign the query setup in $this-> setup_course_query
-        $wp_query = $this->query;
+		// assign the query setup in $this-> setup_course_query
+		$wp_query = $this->query;
 
-        $this->attach_shortcode_hooks();
+		$this->attach_shortcode_hooks();
 
-        ob_start();
-        echo '<section id="sensei-user-courses">';
-        Sensei_Messages::the_my_messages_link();
-        Sensei_Templates::get_template('loop-course.php');
-        Sensei_Templates::get_template('globals/pagination.php');
-        echo '</section>';
+		ob_start();
+		echo '<section id="sensei-user-courses">';
+		Sensei_Messages::the_my_messages_link();
+		Sensei_Templates::get_template('loop-course.php');
+		Sensei_Templates::get_template('globals/pagination.php');
+		echo '</section>';
 
-        $shortcode_output =  ob_get_clean();
+		$shortcode_output =  ob_get_clean();
 
-        $this->detach_shortcode_hooks();
+		$this->detach_shortcode_hooks();
 
-        //restore old query
-        $wp_query = $current_global_query;
+		//restore old query
+		$wp_query = $current_global_query;
 
-        return $shortcode_output;
+		return $shortcode_output;
 
-    }// end render
+	}// end render
 
-    /**
-     * Add hooks for the shortcode
-     *
-     * @since 1.9.0
-     */
-    public function attach_shortcode_hooks(){
+	/**
+	 * Add hooks for the shortcode
+	 *
+	 * @since 1.9.0
+	 */
+	public function attach_shortcode_hooks(){
 
-        // attach the toggle functionality
-        // don't show the toggle action if the user specified complete or active for this shortcode
-        if( ! in_array( $this->status, array( 'active', 'complete' ) ) ){
+		// attach the toggle functionality
+		// don't show the toggle action if the user specified complete or active for this shortcode
+		if( ! in_array( $this->status, array( 'active', 'complete' ) ) ){
 
-            add_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
-            add_action( 'wp_footer', array( $this, 'print_course_toggle_actions_inline_script' ), 90 );
+			add_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
+			add_action( 'wp_footer', array( $this, 'print_course_toggle_actions_inline_script' ), 90 );
 
-        }
+		}
 
-        // add extra classes to distinguish the course based on user completed or active
-        add_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
+		// add extra classes to distinguish the course based on user completed or active
+		add_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
 
-        // attach progress meter below course
-        add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
-        add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
+		// attach progress meter below course
+		add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
+		add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
 
-    }
+	}
 
-    /**
-     * Remove hooks for the shortcode
-     *
-     * @since 1.9.0
-     */
-    public function detach_shortcode_hooks(){
+	/**
+	 * Remove hooks for the shortcode
+	 *
+	 * @since 1.9.0
+	 */
+	public function detach_shortcode_hooks(){
 
-        //remove all hooks after the output is generated
-        remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
-        remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
-        remove_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
-        remove_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
-    }
+		//remove all hooks after the output is generated
+		remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
+		remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
+		remove_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
+		remove_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
+	}
 
-    /**
-     * Hooks into sensei_course_content_inside_after
-     *
-     * @param $course
-     */
-    public function attach_course_progress( $course ){
+	/**
+	 * Hooks into sensei_course_content_inside_after
+	 *
+	 * @param $course
+	 */
+	public function attach_course_progress( $course ){
 
-        $percentage = Sensei()->course->get_completion_percentage( $course->ID, get_current_user_id() );
-        echo Sensei()->course->get_progress_meter( $percentage );
+		$percentage = Sensei()->course->get_completion_percentage( $course->ID, get_current_user_id() );
+		echo Sensei()->course->get_progress_meter( $percentage );
 
-    }// attach_course_progress
+	}// attach_course_progress
 
 
-    /**
-     * Hooked into sensei_course_content_inside_after
-     *
-     * Prints out the course action buttons
-     *
-     * @param $course
-     */
-    public function attach_course_buttons( $course ){
+	/**
+	 * Hooked into sensei_course_content_inside_after
+	 *
+	 * Prints out the course action buttons
+	 *
+	 * @param $course
+	 */
+	public function attach_course_buttons( $course ){
 
-        Sensei()->course->the_course_action_buttons( $course );
+		Sensei()->course->the_course_action_buttons( $course );
 
-    }// attach_course_buttons
+	}// attach_course_buttons
 
-    /**
-     * Add a the user status class for the given course.
-     *
-     * @since 1.9
-     *
-     * @param array $classes
-     * @param WP_Post $course
-     * @return array $classes
-     */
-    public function course_status_class_tagging($classes, $course){
+	/**
+	 * Add a the user status class for the given course.
+	 *
+	 * @since 1.9
+	 *
+	 * @param array $classes
+	 * @param WP_Post $course
+	 * @return array $classes
+	 */
+	public function course_status_class_tagging($classes, $course){
 
-        if ( Sensei_Utils::user_completed_course( $course, get_current_user_id() ) ) {
+		if ( Sensei_Utils::user_completed_course( $course, get_current_user_id() ) ) {
 
-            $classes[] = 'user-completed';
+			$classes[] = 'user-completed';
 
-        } else {
+		} else {
 
-            $classes[] = 'user-active';
+			$classes[] = 'user-active';
 
-        }
+		}
 
-        return $classes;
+		return $classes;
 
-    }// end course_status_class_tagging
+	}// end course_status_class_tagging
 
-    /**
-     * Output the course toggle functionality
-     */
-    public function course_toggle_actions(){ ?>
+	/**
+	 * Output the course toggle functionality
+	 */
+	public function course_toggle_actions(){ ?>
 
         <section id="user-course-status-toggle">
             <a id="sensei-user-courses-active-action" href=""><?php _e('Active Courses'); ?></a>
@@ -359,12 +359,12 @@ 
 
     <?php }
 
-    /**
-     * Load the javascript for the toggle functionality
-     *
-     * @since 1.9.0
-     */
-    function print_course_toggle_actions_inline_script() { ?>
+	/**
+	 * Load the javascript for the toggle functionality
+	 *
+	 * @since 1.9.0
+	 */
+	function print_course_toggle_actions_inline_script() { ?>
 
         <script type="text/javascript">
             var buttonContainer = jQuery('#user-course-status-toggle');

Spacing +61 added lines, -61 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined('ABSPATH')) exit; // Exit if accessed directly
 /**
  * This class is loaded int WP by the shortcode loader class.
  *
@@ -59,26 +59,26 @@ 
      * @param string $content
      * @param string $shortcode the shortcode that was called for this instance
      */
-    public function __construct( $attributes, $content, $shortcode ){
+    public function __construct($attributes, $content, $shortcode) {
 
-        if(!  is_user_logged_in() ) {
+        if ( ! is_user_logged_in()) {
             return;
         }
 
         // set up all argument need for constructing the course query
-        $this->number = isset( $attributes['number'] ) ? $attributes['number'] : '10';
-        $this->orderby = isset( $attributes['orderby'] ) ? $attributes['orderby'] : 'title';
-        $this->status = isset( $attributes['status'] ) ? $attributes['status'] : 'all';
+        $this->number = isset($attributes['number']) ? $attributes['number'] : '10';
+        $this->orderby = isset($attributes['orderby']) ? $attributes['orderby'] : 'title';
+        $this->status = isset($attributes['status']) ? $attributes['status'] : 'all';
 
         // set the default for menu_order to be ASC
-        if( 'menu_order' == $this->orderby && !isset( $attributes['order']  ) ){
+        if ('menu_order' == $this->orderby && ! isset($attributes['order'])) {
 
-            $this->order =  'ASC';
+            $this->order = 'ASC';
 
-        }else{
+        } else {
 
             // for everything else use the value passed or the default DESC
-            $this->order = isset( $attributes['order']  ) ? $attributes['order'] : 'ASC';
+            $this->order = isset($attributes['order']) ? $attributes['order'] : 'ASC';
 
         }
 
@@ -92,20 +92,20 @@ 
      *
      * @since 1.9.0
      */
-    protected function setup_course_query(){
+    protected function setup_course_query() {
 
-        $status_query = array( 'user_id' => get_current_user_id(), 'type' => 'sensei_course_status' );
-        $user_courses_logs = Sensei_Utils::sensei_check_for_activity( $status_query , true );
-        if ( !is_array($user_courses_logs) ) {
+        $status_query = array('user_id' => get_current_user_id(), 'type' => 'sensei_course_status');
+        $user_courses_logs = Sensei_Utils::sensei_check_for_activity($status_query, true);
+        if ( ! is_array($user_courses_logs)) {
 
-            $user_courses_logs = array( $user_courses_logs );
+            $user_courses_logs = array($user_courses_logs);
 
         }
 
         $completed_ids = $active_ids = array();
-        foreach( $user_courses_logs as $course_status ) {
+        foreach ($user_courses_logs as $course_status) {
 
-            if ( Sensei_Utils::user_completed_course( $course_status, get_current_user_id() ) ) {
+            if (Sensei_Utils::user_completed_course($course_status, get_current_user_id())) {
 
                 $completed_ids[] = $course_status->comment_post_ID;
 
@@ -116,34 +116,34 @@ 
             }
         }
 
-        if( 'completed' == $this->status ){
+        if ('completed' == $this->status) {
 
-            $included_courses =  $completed_ids;
+            $included_courses = $completed_ids;
 
 
-        }elseif( 'active'==$this->status ){
+        }elseif ('active' == $this->status) {
 
-            $included_courses =  $active_ids;
+            $included_courses = $active_ids;
 
-        }else{ // all courses
+        } else { // all courses
 
-            if( empty( $completed_ids ) ){
+            if (empty($completed_ids)) {
 
-                add_action( 'sensei_loop_course_inside_before', array( $this, 'completed_no_course_message_output' ) );
+                add_action('sensei_loop_course_inside_before', array($this, 'completed_no_course_message_output'));
             }
 
-            if( empty( $active_ids ) ){
+            if (empty($active_ids)) {
 
-                add_action( 'sensei_loop_course_inside_before', array( $this, 'active_no_course_message_output' ) );
+                add_action('sensei_loop_course_inside_before', array($this, 'active_no_course_message_output'));
 
             }
 
-            if( empty( $completed_ids ) &&  empty( $active_ids )  ){
+            if (empty($completed_ids) && empty($active_ids)) {
 
                 $included_courses = array('-1000'); // don't show any courses
 
-            }else{
-                $included_courses = Sensei_Utils::array_zip_merge( (array)$active_ids, (array)$completed_ids );
+            } else {
+                $included_courses = Sensei_Utils::array_zip_merge((array) $active_ids, (array) $completed_ids);
             }
 
 
@@ -154,7 +154,7 @@ 
         // if the shortcode is not active or in active and the active and completed
         // tabs show up.
         $number_of_posts = $this->number;
-        if( 'active' != $this->status && 'complete' != $this->status  ){
+        if ('active' != $this->status && 'complete' != $this->status) {
             $number_of_posts = 1000;
         }
 
@@ -165,12 +165,12 @@ 
             'post_status'      => 'publish',
             'orderby'          => $this->orderby,
             'order'            => $this->order,
-            'paged' => empty( $query_var_paged )? 1 : $query_var_paged,
+            'paged' => empty($query_var_paged) ? 1 : $query_var_paged,
             'posts_per_page'   => $number_of_posts,
             'post__in'         => $included_courses,
         );
 
-        $this->query = new WP_Query( $query_args );
+        $this->query = new WP_Query($query_args);
 
     }// end setup _course_query
 
@@ -180,12 +180,12 @@ 
      *
      * @since 1.9.0
      */
-    public function completed_no_course_message_output(){
+    public function completed_no_course_message_output() {
         ?>
         <li class="user-completed">
             <div class="sensei-message info">
 
-                <?php _e( 'You have not completed any courses yet.', 'woothemes-sensei' ); ?>
+                <?php _e('You have not completed any courses yet.', 'woothemes-sensei'); ?>
 
             </div>
         </li>
@@ -198,17 +198,17 @@ 
      *
      * @since 1.9.0
      */
-    public function active_no_course_message_output(){
+    public function active_no_course_message_output() {
         ?>
 
         <li class="user-active">
             <div class="sensei-message info">
 
-                <?php _e( 'You have no active courses.', 'woothemes-sensei' ); ?>
+                <?php _e('You have no active courses.', 'woothemes-sensei'); ?>
 
-                <a href="<?php esc_attr_e( Sensei_Course::get_courses_page_url() ); ?>">
+                <a href="<?php esc_attr_e(Sensei_Course::get_courses_page_url()); ?>">
 
-                    <?php  _e( 'Start a Course!', 'woothemes-sensei' ); ?>
+                    <?php  _e('Start a Course!', 'woothemes-sensei'); ?>
 
                 </a>
 
@@ -222,11 +222,11 @@ 
      *
      * @return string $content
      */
-    public function render(){
+    public function render() {
 
         global $wp_query;
 
-        if(!  is_user_logged_in() ) {
+        if ( ! is_user_logged_in()) {
             return '';
         }
 
@@ -245,7 +245,7 @@ 
         Sensei_Templates::get_template('globals/pagination.php');
         echo '</section>';
 
-        $shortcode_output =  ob_get_clean();
+        $shortcode_output = ob_get_clean();
 
         $this->detach_shortcode_hooks();
 
@@ -261,23 +261,23 @@ 
      *
      * @since 1.9.0
      */
-    public function attach_shortcode_hooks(){
+    public function attach_shortcode_hooks() {
 
         // attach the toggle functionality
         // don't show the toggle action if the user specified complete or active for this shortcode
-        if( ! in_array( $this->status, array( 'active', 'complete' ) ) ){
+        if ( ! in_array($this->status, array('active', 'complete'))) {
 
-            add_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
-            add_action( 'wp_footer', array( $this, 'print_course_toggle_actions_inline_script' ), 90 );
+            add_action('sensei_loop_course_before', array($this, 'course_toggle_actions'));
+            add_action('wp_footer', array($this, 'print_course_toggle_actions_inline_script'), 90);
 
         }
 
         // add extra classes to distinguish the course based on user completed or active
-        add_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
+        add_filter('sensei_course_loop_content_class', array($this, 'course_status_class_tagging'), 20, 2);
 
         // attach progress meter below course
-        add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
-        add_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
+        add_action('sensei_course_content_inside_after', array($this, 'attach_course_progress'));
+        add_action('sensei_course_content_inside_after', array($this, 'attach_course_buttons'));
 
     }
 
@@ -286,13 +286,13 @@ 
      *
      * @since 1.9.0
      */
-    public function detach_shortcode_hooks(){
+    public function detach_shortcode_hooks() {
 
         //remove all hooks after the output is generated
-        remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_progress' ) );
-        remove_action( 'sensei_course_content_inside_after', array( $this, 'attach_course_buttons' ) );
-        remove_filter( 'sensei_course_loop_content_class', array( $this, 'course_status_class_tagging' ), 20, 2 );
-        remove_action( 'sensei_loop_course_before', array( $this, 'course_toggle_actions' ) );
+        remove_action('sensei_course_content_inside_after', array($this, 'attach_course_progress'));
+        remove_action('sensei_course_content_inside_after', array($this, 'attach_course_buttons'));
+        remove_filter('sensei_course_loop_content_class', array($this, 'course_status_class_tagging'), 20, 2);
+        remove_action('sensei_loop_course_before', array($this, 'course_toggle_actions'));
     }
 
     /**
@@ -300,10 +300,10 @@ 
      *
      * @param $course
      */
-    public function attach_course_progress( $course ){
+    public function attach_course_progress($course) {
 
-        $percentage = Sensei()->course->get_completion_percentage( $course->ID, get_current_user_id() );
-        echo Sensei()->course->get_progress_meter( $percentage );
+        $percentage = Sensei()->course->get_completion_percentage($course->ID, get_current_user_id());
+        echo Sensei()->course->get_progress_meter($percentage);
 
     }// attach_course_progress
 
@@ -315,9 +315,9 @@ 
      *
      * @param $course
      */
-    public function attach_course_buttons( $course ){
+    public function attach_course_buttons($course) {
 
-        Sensei()->course->the_course_action_buttons( $course );
+        Sensei()->course->the_course_action_buttons($course);
 
     }// attach_course_buttons
 
@@ -330,9 +330,9 @@ 
      * @param WP_Post $course
      * @return array $classes
      */
-    public function course_status_class_tagging($classes, $course){
+    public function course_status_class_tagging($classes, $course) {
 
-        if ( Sensei_Utils::user_completed_course( $course, get_current_user_id() ) ) {
+        if (Sensei_Utils::user_completed_course($course, get_current_user_id())) {
 
             $classes[] = 'user-completed';
 
@@ -349,7 +349,7 @@ 
     /**
      * Output the course toggle functionality
      */
-    public function course_toggle_actions(){ ?>
+    public function course_toggle_actions() { ?>
 
         <section id="user-course-status-toggle">
             <a id="sensei-user-courses-active-action" href=""><?php _e('Active Courses'); ?></a>

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

Doc Comments -2 removed lines

@@ -167,8 +167,6 @@
 

 	/**

 	 * Load the desired component, if a method is available for it.

-	 * @param  string $component The component to potentially be loaded.

-     *

 	 * @since  1.0.0

 	 * @return void

 	 */

Indentation +26 added lines, -26 removed lines

@@ -64,11 +64,11 @@ 
 		remove_filter( 'pre_get_posts', 'sensei_course_archive_filter', 10, 1 );

 

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

-            && ( 'activecourses' == $instance['component'] || 'completedcourses' == $instance['component'] )

-            && !is_user_logged_in() ) {

+			&& ( 'activecourses' == $instance['component'] || 'completedcourses' == $instance['component'] )

+			&& !is_user_logged_in() ) {

 

 			// No Output

-            return;

+			return;

 

 		} else {

 			/* Our variables from the widget settings. */

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+	public function form( $instance ) {

 

 		/* Set up some default widget settings. */

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

@@ -168,7 +168,7 @@ 
 	/**

 	 * Load the desired component, if a method is available for it.

 	 * @param  string $component The component to potentially be loaded.

-     *

+	 *

 	 * @since  1.0.0

 	 * @return void

 	 */

@@ -207,24 +207,24 @@ 
 

 		}

 

-        if ( ! empty( $course_ids ) ) {

+		if ( ! empty( $course_ids ) ) {

 

-            $posts_array = Sensei()->course->course_query( intval( $instance['limit'] ), esc_attr( $instance['component'] ), $course_ids );

+			$posts_array = Sensei()->course->course_query( intval( $instance['limit'] ), esc_attr( $instance['component'] ), $course_ids );

 

 		} else {

 

-            if ( 'activecourses' == esc_attr( $instance['component'] ) || 'completedcourses' == esc_attr( $instance['component'] ) ) {

+			if ( 'activecourses' == esc_attr( $instance['component'] ) || 'completedcourses' == esc_attr( $instance['component'] ) ) {

 				$posts_array = array();

 

-            } else {

+			} else {

 

-                $course_args = array(

-                    'post_type' => 'course',

-                    'orderby'         	=> 'date',

-                    'order'           	=> 'DESC',

-                    'post_status'      	=> 'publish',

-                    'posts_per_page' => $instance['limit'],

-                );

+				$course_args = array(

+					'post_type' => 'course',

+					'orderby'         	=> 'date',

+					'order'           	=> 'DESC',

+					'post_status'      	=> 'publish',

+					'posts_per_page' => $instance['limit'],

+				);

 

 				$posts_array = get_posts( $course_args );

 			}

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

 			<ul>

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

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

-		    	$post_title = $post_item->post_title;

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

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

-		    	$author_display_name = $user_info->display_name;

-		    	$author_id = $post_item->post_author;

-		    ?>

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

+				$post_title = $post_item->post_title;

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

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

+				$author_display_name = $user_info->display_name;

+				$author_id = $post_item->post_author;

+			?>

 		    	<li class="fix">

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

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

@@ -255,9 +255,9 @@ 
 		    	</li>

 		    <?php } // End For Loop ?>

 		    <?php if ( 'activecourses' == esc_attr( $instance['component'] ) || 'completedcourses' == esc_attr( $instance['component'] ) ) {

-		    	$my_account_page_id = intval( Sensei()->settings->settings[ 'my_course_page' ] );

-		    	echo '<li class="my-account fix"><a href="'. esc_url( get_permalink( $my_account_page_id ) ) .'">'.__('My Courses', 'woothemes-sensei').' <span class="meta-nav"></span></a></li>';

-		    } // End If Statement ?>

+				$my_account_page_id = intval( Sensei()->settings->settings[ 'my_course_page' ] );

+				echo '<li class="my-account fix"><a href="'. esc_url( get_permalink( $my_account_page_id ) ) .'">'.__('My Courses', 'woothemes-sensei').' <span class="meta-nav"></span></a></li>';

+			} // End If Statement ?>

 		</ul>

 		<?php } else {

 			// No posts returned. This means the user either has no active or no completed courses.

Braces +4 added lines, -1 removed lines

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

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

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

 

 /**

  * Sensei Course Component Widget

Spacing +72 added lines, -72 removed lines

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

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

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

 

 /**

  * Sensei Course Component Widget

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

 		/* Widget variable settings. */

 		$this->woo_widget_cssclass = 'widget_sensei_course_component';

-		$this->woo_widget_description = __( 'This widget will output a list of Courses - New, Featured, Free, Paid, Active, Completed.', 'woothemes-sensei' );

+		$this->woo_widget_description = __('This widget will output a list of Courses - New, Featured, Free, Paid, Active, Completed.', 'woothemes-sensei');

 		$this->woo_widget_idbase = 'sensei_course_component';

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

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

 

 		$this->woo_widget_componentslist = array(

-												'usercourses' => __( 'New Courses', 'woothemes-sensei' ),

-												'featuredcourses' => __( 'Featured Courses', 'woothemes-sensei' ),

-												'activecourses' => __( 'My Active Courses', 'woothemes-sensei' ),

-												'completedcourses' => __( 'My Completed Courses', 'woothemes-sensei' ),

+												'usercourses' => __('New Courses', 'woothemes-sensei'),

+												'featuredcourses' => __('Featured Courses', 'woothemes-sensei'),

+												'activecourses' => __('My Active Courses', 'woothemes-sensei'),

+												'completedcourses' => __('My Completed Courses', 'woothemes-sensei'),

 												);

 

 		// Add support for the WooCommerce shelf.

-		if ( Sensei_WC::is_woocommerce_active() ) {

-			$this->woo_widget_componentslist['freecourses'] = __( 'Free Courses', 'woothemes-sensei' );

-			$this->woo_widget_componentslist['paidcourses'] = __( 'Paid Courses', 'woothemes-sensei' );

+		if (Sensei_WC::is_woocommerce_active()) {

+			$this->woo_widget_componentslist['freecourses'] = __('Free Courses', 'woothemes-sensei');

+			$this->woo_widget_componentslist['paidcourses'] = __('Paid Courses', 'woothemes-sensei');

 		}

 		/* Widget settings. */

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

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

 

 		/* Widget control settings. */

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

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

 

 		/* Create the widget. */

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

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

 	} // End __construct()

 

 	/**

@@ -59,44 +59,44 @@ 
 	 * @param  array $instance Widget settings for this instance.

 	 * @return void

 	 */

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

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

 

-		remove_filter( 'pre_get_posts', 'sensei_course_archive_filter', 10, 1 );

+		remove_filter('pre_get_posts', 'sensei_course_archive_filter', 10, 1);

 

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

-            && ( 'activecourses' == $instance['component'] || 'completedcourses' == $instance['component'] )

-            && !is_user_logged_in() ) {

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

+            && ('activecourses' == $instance['component'] || 'completedcourses' == $instance['component'])

+            && ! is_user_logged_in()) {

 

 			// No Output

             return;

 

 		} else {

 			/* Our variables from the widget settings. */

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

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

 

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

 			echo $args['before_widget'];

 

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

-			if ( $title ) { echo $args['before_title'] . $title . $args['after_title']; }

+			if ($title) { echo $args['before_title'].$title.$args['after_title']; }

 

 			/* Widget content. */

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

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

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

 

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

-				$this->load_component( $instance );

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

+				$this->load_component($instance);

 			}

 

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

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

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

 

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

 			echo $args['after_widget'];

 

 		} // End If Statement

 

-		add_filter( 'pre_get_posts', 'sensei_course_archive_filter', 10, 1 );

+		add_filter('pre_get_posts', 'sensei_course_archive_filter', 10, 1);

 

 	} // End widget()

 

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

 	 * @return array               Updated settings.

 	 */

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

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

 		$instance = $old_instance;

 

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

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

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

 

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

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

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

 

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

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

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

 

 

 		return $instance;

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+    public function form($instance) {

 

 		/* Set up some default widget settings. */

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

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

 					);

 

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

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

 ?>

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

 		<p>

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

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

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

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

 		</p>

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

 		<p>

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

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

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

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

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

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

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

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

 			<?php } ?>

 			</select>

 		</p>

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

 		<p>

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

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

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

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

 		</p>

 

 <?php

@@ -172,28 +172,28 @@ 
 	 * @since  1.0.0

 	 * @return void

 	 */

-	protected function load_component ( $instance ) {

+	protected function load_component($instance) {

 		global  $current_user;

 

 		get_currentuserinfo();

 

 		$course_ids = array();

-		if ( 'activecourses' == esc_attr( $instance['component'] ) ) {

-			$courses = Sensei_Utils::sensei_check_for_activity( array( 'user_id' => $current_user->ID, 'type' => 'sensei_course_status', 'status' => 'in-progress' ), true );

+		if ('activecourses' == esc_attr($instance['component'])) {

+			$courses = Sensei_Utils::sensei_check_for_activity(array('user_id' => $current_user->ID, 'type' => 'sensei_course_status', 'status' => 'in-progress'), true);

 			// Need to always return an array, even with only 1 item

-			if ( !is_array($courses) ) {

-				$courses = array( $courses );

+			if ( ! is_array($courses)) {

+				$courses = array($courses);

 			}

-			foreach( $courses AS $course ) {

+			foreach ($courses AS $course) {

 				$course_ids[] = $course->comment_post_ID;

 			}

-		} elseif( 'completedcourses' == esc_attr( $instance['component'] ) ) {

-			$courses = Sensei_Utils::sensei_check_for_activity( array( 'user_id' => $current_user->ID, 'type' => 'sensei_course_status', 'status' => 'complete' ), true );

+		} elseif ('completedcourses' == esc_attr($instance['component'])) {

+			$courses = Sensei_Utils::sensei_check_for_activity(array('user_id' => $current_user->ID, 'type' => 'sensei_course_status', 'status' => 'complete'), true);

 			// Need to always return an array, even with only 1 item

-			if ( !is_array($courses) ) {

-				$courses = array( $courses );

+			if ( ! is_array($courses)) {

+				$courses = array($courses);

 			}

-			foreach( $courses AS $course ) {

+			foreach ($courses AS $course) {

 				$course_ids[] = $course->comment_post_ID;

 			}

 		} // End If Statement

@@ -201,19 +201,19 @@ 
 		$posts_array = array();

 

 		// course_query() is buggy, it doesn't honour the 1st arg if includes are provided, so instead slice the includes

-		if ( !empty($instance['limit']) && intval( $instance['limit'] ) >= 1 && intval( $instance['limit'] ) < count($course_ids) ) {

+		if ( ! empty($instance['limit']) && intval($instance['limit']) >= 1 && intval($instance['limit']) < count($course_ids)) {

 

-			$course_ids = array_slice( $course_ids, 0, intval( $instance['limit'] ) ); // This does mean the order by is effectively ignored

+			$course_ids = array_slice($course_ids, 0, intval($instance['limit'])); // This does mean the order by is effectively ignored

 

 		}

 

-        if ( ! empty( $course_ids ) ) {

+        if ( ! empty($course_ids)) {

 

-            $posts_array = Sensei()->course->course_query( intval( $instance['limit'] ), esc_attr( $instance['component'] ), $course_ids );

+            $posts_array = Sensei()->course->course_query(intval($instance['limit']), esc_attr($instance['component']), $course_ids);

 

 		} else {

 

-            if ( 'activecourses' == esc_attr( $instance['component'] ) || 'completedcourses' == esc_attr( $instance['component'] ) ) {

+            if ('activecourses' == esc_attr($instance['component']) || 'completedcourses' == esc_attr($instance['component'])) {

 				$posts_array = array();

 

             } else {

@@ -226,43 +226,43 @@ 
                     'posts_per_page' => $instance['limit'],

                 );

 

-				$posts_array = get_posts( $course_args );

+				$posts_array = get_posts($course_args);

 			}

 

 		} // End If Statement

 

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

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

 			<ul>

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

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

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

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

 		    	$post_title = $post_item->post_title;

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

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

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

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

 		    	$author_display_name = $user_info->display_name;

 		    	$author_id = $post_item->post_author;

 		    ?>

 		    	<li class="fix">

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

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

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

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

 		    		<br />

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

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

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

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

     					<br />

     				<?php } // End If Statement ?>

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

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

     				<br />

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

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

 		    	</li>

 		    <?php } // End For Loop ?>

-		    <?php if ( 'activecourses' == esc_attr( $instance['component'] ) || 'completedcourses' == esc_attr( $instance['component'] ) ) {

-		    	$my_account_page_id = intval( Sensei()->settings->settings[ 'my_course_page' ] );

-		    	echo '<li class="my-account fix"><a href="'. esc_url( get_permalink( $my_account_page_id ) ) .'">'.__('My Courses', 'woothemes-sensei').' <span class="meta-nav"></span></a></li>';

+		    <?php if ('activecourses' == esc_attr($instance['component']) || 'completedcourses' == esc_attr($instance['component'])) {

+		    	$my_account_page_id = intval(Sensei()->settings->settings['my_course_page']);

+		    	echo '<li class="my-account fix"><a href="'.esc_url(get_permalink($my_account_page_id)).'">'.__('My Courses', 'woothemes-sensei').' <span class="meta-nav"></span></a></li>';

 		    } // End If Statement ?>

 		</ul>

 		<?php } else {

 			// No posts returned. This means the user either has no active or no completed courses.

-			$course_status = substr( esc_attr( $instance['component'] ) , 0, -7);

-			echo sprintf( __( 'You have no %1s courses.', 'woothemes-sensei' ), $course_status );

+			$course_status = substr(esc_attr($instance['component']), 0, -7);

+			echo sprintf(__('You have no %1s courses.', 'woothemes-sensei'), $course_status);

 		} // End If Statement

 	} // End load_component()

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

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

Doc Comments -1 removed lines

@@ -151,7 +151,6 @@
 

 	/**

 	 * Load the desired component, if a method is available for it.

-	 * @param  string $component The component to potentially be loaded.

 	 * @since  5.0.8

 	 * @return void

 	 */

Indentation +11 added lines, -11 removed lines

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+	public function form( $instance ) {

 

 		/* Set up some default widget settings. */

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

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

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

 							'orderby'         	=> 'menu_order date',

-    						'order'           	=> 'DESC',

-    						'post_status'       => 'publish',

+							'order'           	=> 'DESC',

+							'post_status'       => 'publish',

 							'suppress_filters' 	=> 0

 							);

 		$posts_array = get_posts( $post_args );

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

 			<ul>

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

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

-		    	$post_title = $post_item->post_title;

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

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

-		    	$author_display_name = $user_info->display_name;

-		    	$author_id = $post_item->post_author;

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

-		    ?>

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

+				$post_title = $post_item->post_title;

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

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

+				$author_display_name = $user_info->display_name;

+				$author_id = $post_item->post_author;

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

+			?>

 		    	<li class="fix">

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

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

Spacing +47 added lines, -47 removed lines

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

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

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

 

 /**

  * Sensei Lesson Component Widget

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

 		/* Widget variable settings. */

 		$this->woo_widget_cssclass = 'widget_sensei_lesson_component';

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

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

 		$this->woo_widget_idbase = 'sensei_lesson_component';

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

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

 

 		$this->woo_widget_componentslist = array(

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

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

 												);

 

 		/* Widget settings. */

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

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

 

 		/* Widget control settings. */

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

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

 

 		/* Create the widget. */

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

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

 	} // End __construct()

 

 	/**

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

 	 * @return void

 	 */

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

-		extract( $args, EXTR_SKIP );

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

+		extract($args, EXTR_SKIP);

 

 

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

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

 			// No Output

 		} else {

 			/* Our variables from the widget settings. */

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

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

 

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

 			echo $before_widget;

 

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

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

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

 

 			/* Widget content. */

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

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

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

 

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

-				$this->load_component( $instance );

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

+				$this->load_component($instance);

 			}

 

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

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

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

 

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

 			echo $after_widget;

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

 	 * @return array               Updated settings.

 	 */

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

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

 		$instance = $old_instance;

 

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

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

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

 

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

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

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

 

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

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

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

 

 

 		return $instance;

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+    public function form($instance) {

 

 		/* Set up some default widget settings. */

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

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

 					);

 

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

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

 ?>

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

 		<p>

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

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

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

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

 		</p>

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

 		<p>

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

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

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

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

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

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

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

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

 			<?php } ?>

 			</select>

 		</p>

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

 		<p>

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

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

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

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

 		</p>

 

 <?php

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

 	 * @return void

 	 */

-	protected function load_component ( $instance ) {

+	protected function load_component($instance) {

 		global  $current_user;

 		// Get User Meta

 		get_currentuserinfo();

@@ -165,41 +165,41 @@ 
 

 		$posts_array = array();

 

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

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

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

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

 							'orderby'         	=> 'menu_order date',

     						'order'           	=> 'DESC',

     						'post_status'       => 'publish',

 							'suppress_filters' 	=> 0

 							);

-		$posts_array = get_posts( $post_args );

+		$posts_array = get_posts($post_args);

 

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

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

 			<ul>

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

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

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

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

 		    	$post_title = $post_item->post_title;

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

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

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

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

 		    	$author_display_name = $user_info->display_name;

 		    	$author_id = $post_item->post_author;

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

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

 		    ?>

 		    	<li class="fix">

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

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

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

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

 		    		<br />

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

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

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

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

     					<br />

     				<?php } // End If Statement ?>

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

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

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

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

                     <?php } ?>

     				<br />

 		    	</li>

 		    <?php } // End For Loop ?>

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

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

 		</ul>

 		<?php } // End If Statement

 	} // End load_component()

Braces +4 added lines, -1 removed lines

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

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

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

 

 /**

  * Sensei Course Component Widget

uninstall.php

Spacing +3 added lines, -3 removed lines

@@ -10,8 +10,8 @@
  * @author WooThemes
  * @since 1.0.0
  */
-if( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) exit();
+if ( ! defined('WP_UNINSTALL_PLUGIN')) exit();
 
 $token = 'woothemes-sensei';
-delete_option( 'skip_install_sensei_pages' );
-delete_option( 'sensei_installed' );
\ No newline at end of file
+delete_option('skip_install_sensei_pages');
+delete_option('sensei_installed');
\ No newline at end of file

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  * @author WooThemes
  * @since 1.0.0
  */
-if( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) exit();
+if( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) {
+	exit();
+}
 
 $token = 'woothemes-sensei';
 delete_option( 'skip_install_sensei_pages' );

Indentation +10 added lines, -10 removed lines

@@ -1,15 +1,15 @@
 <?php
 /**
- * WooThemes Sensei Uninstall
- *
- * Uninstalls the plugin and associated data.
- *
- * @package WordPress
- * @subpackage Sensei
- * @category Core
- * @author WooThemes
- * @since 1.0.0
- */
+	 * WooThemes Sensei Uninstall
+	 *
+	 * Uninstalls the plugin and associated data.
+	 *
+	 * @package WordPress
+	 * @subpackage Sensei
+	 * @category Core
+	 * @author WooThemes
+	 * @since 1.0.0
+	 */
 if( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) exit();
 
 $token = 'woothemes-sensei';

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

Indentation +1 added lines, -1 removed lines

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+	public function form( $instance ) {

 

 		/* Set up some default widget settings. */

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

Spacing +34 added lines, -34 removed lines

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

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

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

 

 /**

  * Sensei Course Categories Widget

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

 		/* Widget variable settings. */

 		$this->woo_widget_cssclass = 'widget_sensei_course_categories';

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

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

 		$this->woo_widget_idbase = 'sensei_course_categories';

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

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

 

 		/* Widget settings. */

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

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

 

 		/* Widget control settings. */

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

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

 

 		/* Create the widget. */

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

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

 	} // End __construct()

 

 	/**

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

 	 * @return void

 	 */

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

-		extract( $args, EXTR_SKIP );

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

+		extract($args, EXTR_SKIP);

 

 			/* Our variables from the widget settings. */

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

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

 

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

 			echo $before_widget;

 

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

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

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

 

 			/* Widget content. */

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

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

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

 

-			$this->load_component( $instance );

+			$this->load_component($instance);

 

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

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

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

 

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

 			echo $after_widget;

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

 	 * @return array               Updated settings.

 	 */

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

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

 		$instance = $old_instance;

 

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

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

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

 

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

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

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

 

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

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

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+    public function form($instance) {

 

 		/* Set up some default widget settings. */

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

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

 					);

 

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

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

 ?>

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

 		<p>

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

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

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

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

 		</p>

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

 		<p>

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

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

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

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

 		</p>

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

 		<p>

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

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

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

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

 		</p>

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

 		<p>

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

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

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

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

 		</p>

 <?php

 	} // End form()

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

 	 * @return void

 	 */

-	protected function load_component ( $instance ) {

+	protected function load_component($instance) {

 

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

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

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

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

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

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

 

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

-		if ( 0 < $limit ) {

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

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

+		if (0 < $limit) {

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

 		} // End If Statement

 		echo '<ul>';

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

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

 		echo '</ul>';

 	} // End load_component()

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

Braces +4 added lines, -1 removed lines

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

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

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

 

 /**

  * Sensei Course Component Widget

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

Indentation +10 added lines, -10 removed lines

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+	public function form( $instance ) {

 

 		/* Set up some default widget settings. */

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

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

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

 							'orderby'         	=> 'menu_order date',

-    						'order'           	=> 'ASC',

-    						'post_status'       => 'publish',

+							'order'           	=> 'ASC',

+							'post_status'       => 'publish',

 							'suppress_filters' 	=> 0,

 							);

 

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

 			<ul>

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

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

-		    	$post_title = $post_item->post_title;

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

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

-		    	$author_display_name = $user_info->display_name;

-		    	$author_id = $post_item->post_author;

-		    ?>

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

+				$post_title = $post_item->post_title;

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

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

+				$author_display_name = $user_info->display_name;

+				$author_id = $post_item->post_author;

+			?>

 		    	<li class="fix">

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

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

Spacing +43 added lines, -43 removed lines

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

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

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

 

 /**

  * Sensei Category Courses Widget

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

 		/* Widget variable settings. */

 		$this->woo_widget_cssclass = 'widget_sensei_category_courses';

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

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

 		$this->woo_widget_idbase = 'sensei_category_courses';

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

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

 

 		/* Widget settings. */

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

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

 

 		/* Widget control settings. */

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

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

 

 		/* Create the widget. */

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

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

 

 	} // End __construct()

 

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

 	 * @return void

 	 */

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

-		extract( $args, EXTR_SKIP );

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

+		extract($args, EXTR_SKIP);

 

 		/* Our variables from the widget settings. */

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

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

 

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

 		echo $before_widget;

 

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

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

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

 

 		/* Widget content. */

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

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

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

 

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

-			$this->load_component( $instance );

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

+			$this->load_component($instance);

 		} // End If Statement

 

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

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

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

 

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

 		echo $after_widget;

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

 	 * @return array               Updated settings.

 	 */

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

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

 		$instance = $old_instance;

 

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

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

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

 

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

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

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

 

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

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

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

 

 

 		return $instance;

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

 	 * @return void

 	 */

-    public function form( $instance ) {

+    public function form($instance) {

 

 		/* Set up some default widget settings. */

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

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

 					);

 

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

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

 ?>

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

 		<p>

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

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

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

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

 		</p>

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

 		<p>

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

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

 			<?php

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

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

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

 			?>

 		</p>

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

 		<p>

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

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

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

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

 		</p>

 

 <?php

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

 	 * @return void

 	 */

-	protected function load_component ( $instance ) {

+	protected function load_component($instance) {

 

 

 		$posts_array = array();

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

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

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

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

 							'orderby'         	=> 'menu_order date',

     						'order'           	=> 'ASC',

     						'post_status'       => 'publish',

 							'suppress_filters' 	=> 0,

 							);

 

-		$post_args[ 'tax_query' ] = array(

+		$post_args['tax_query'] = array(

 											array(

 												'taxonomy' => 'course-category',

 												'field' => 'id',

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

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

 											)

 										);

 

-		$posts_array = get_posts( $post_args );

+		$posts_array = get_posts($post_args);

 

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

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

 			<ul>

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

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

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

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

 		    	$post_title = $post_item->post_title;

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

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

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

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

 		    	$author_display_name = $user_info->display_name;

 		    	$author_id = $post_item->post_author;

 		    ?>

 		    	<li class="fix">

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

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

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

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

 		    		<br />

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

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

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

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

     					<br />

     				<?php } // End If Statement ?>

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

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

     				<br />

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

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

 		    	</li>

 		    <?php } // End For Loop ?>

 			</ul>

Braces +4 added lines, -1 removed lines

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

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

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

 

 /**

  * Sensei Course Component Widget

templates/content-lesson.php

Indentation +51 added lines, -51 removed lines

@@ -1,17 +1,17 @@ 
 <?php
 if ( ! defined( 'ABSPATH' ) ) exit;
 /**
- * Content-lesson.php template file
- *
- * responsible for content on archive like pages. Only shows the lesson excerpt.
- *
- * For single lesson content please see single-lesson.php
- *
- * @author 		Automattic
- * @package 	Sensei
- * @category    Templates
- * @version     1.9.0
- */
+	 * Content-lesson.php template file
+	 *
+	 * responsible for content on archive like pages. Only shows the lesson excerpt.
+	 *
+	 * For single lesson content please see single-lesson.php
+	 *
+	 * @author 		Automattic
+	 * @package 	Sensei
+	 * @category    Templates
+	 * @version     1.9.0
+	 */
 ?>
 
 <article <?php post_class( get_the_ID() ); ?> >
@@ -19,32 +19,32 @@ 
     <section class="lesson-content">
 
         <?php
-        /**
-         * sensei_content_lesson_before
-         * action that runs before the sensei {post_type} content. It runs inside the sensei
-         * content.php template. This applies to the specific post type that you've targeted.
-         *
-         * @since 1.9.0
-         * @param string $post_id
-         */
-        do_action( 'sensei_content_lesson_before', get_the_ID() );
-        ?>
+		/**
+		 * sensei_content_lesson_before
+		 * action that runs before the sensei {post_type} content. It runs inside the sensei
+		 * content.php template. This applies to the specific post type that you've targeted.
+		 *
+		 * @since 1.9.0
+		 * @param string $post_id
+		 */
+		do_action( 'sensei_content_lesson_before', get_the_ID() );
+		?>
 
         <section class="entry">
 
             <?php
-            /**
-             * Fires just before the post content in the content-lesson.php file.
-             *
-             * @since 1.9.0
-             *
-             * @hooked Sensei()->modules->module_archive_description -  11
-             * @hooked Sensei_Lesson::the_lesson_meta                -  20
-             *
-             * @param string $post_id
-             */
-            do_action('sensei_content_lesson_inside_before', get_the_ID());
-            ?>
+			/**
+			 * Fires just before the post content in the content-lesson.php file.
+			 *
+			 * @since 1.9.0
+			 *
+			 * @hooked Sensei()->modules->module_archive_description -  11
+			 * @hooked Sensei_Lesson::the_lesson_meta                -  20
+			 *
+			 * @param string $post_id
+			 */
+			do_action('sensei_content_lesson_inside_before', get_the_ID());
+			?>
 
             <p class="lesson-excerpt">
 
@@ -53,28 +53,28 @@ 
             </p>
 
             <?php
-            /**
-             * Fires just after the post content in the lesson-content.php file.
-             *
-             * @since 1.9.0
-             *
-             * @param string $post_id
-             */
-            do_action('sensei_content_lesson_inside_after', get_the_ID());
-            ?>
+			/**
+			 * Fires just after the post content in the lesson-content.php file.
+			 *
+			 * @since 1.9.0
+			 *
+			 * @param string $post_id
+			 */
+			do_action('sensei_content_lesson_inside_after', get_the_ID());
+			?>
 
         </section> <!-- section .entry -->
 
         <?php
-        /**
-         * This action runs after the sensei lesson content. It runs inside the sensei
-         * lesson-content.php template.
-         *
-         * @since 1.9.0
-         * @param string $post_id
-         */
-        do_action( 'sensei_content_lesson_after', get_the_ID() );
-        ?>
+		/**
+		 * This action runs after the sensei lesson content. It runs inside the sensei
+		 * lesson-content.php template.
+		 *
+		 * @since 1.9.0
+		 * @param string $post_id
+		 */
+		do_action( 'sensei_content_lesson_after', get_the_ID() );
+		?>
 
     </section> <!-- article .lesson-content -->
 

Spacing +5 added lines, -5 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 /**
  * Content-lesson.php template file
  *
@@ -14,7 +14,7 @@ 
  */
 ?>
 
-<article <?php post_class( get_the_ID() ); ?> >
+<article <?php post_class(get_the_ID()); ?> >
 
     <section class="lesson-content">
 
@@ -27,7 +27,7 @@ 
          * @since 1.9.0
          * @param string $post_id
          */
-        do_action( 'sensei_content_lesson_before', get_the_ID() );
+        do_action('sensei_content_lesson_before', get_the_ID());
         ?>
 
         <section class="entry">
@@ -73,9 +73,9 @@ 
          * @since 1.9.0
          * @param string $post_id
          */
-        do_action( 'sensei_content_lesson_after', get_the_ID() );
+        do_action('sensei_content_lesson_after', get_the_ID());
         ?>
 
     </section> <!-- article .lesson-content -->
 
-</article> <!-- article .(<?php esc_attr_e( join( ' ', get_post_class( array( 'lesson', 'post' ) ) ) ); ?>  -->
\ No newline at end of file
+</article> <!-- article .(<?php esc_attr_e(join(' ', get_post_class(array('lesson', 'post')))); ?>  -->
\ No newline at end of file

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 /**
  * The Template for displaying the my course page data.
  *