woothemes / sensei

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

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( $active_ids, $completed_ids );
+            } else {
+                $included_courses = Sensei_Utils::array_zip_merge($active_ids, $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 '';
         }
 
@@ -242,7 +242,7 @@ 
         Sensei_Messages::the_my_messages_link();
         Sensei_Templates::get_template('loop-course.php');
         Sensei_Templates::get_template('globals/pagination.php');
-        $shortcode_output =  ob_get_clean();
+        $shortcode_output = ob_get_clean();
 
         $this->detach_shortcode_hooks();
 
@@ -258,23 +258,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'));
 
     }
 
@@ -283,13 +283,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'));
     }
 
     /**
@@ -297,10 +297,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
 
@@ -312,9 +312,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
 
@@ -327,9 +327,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';
 
@@ -346,7 +346,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>

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( $active_ids, $completed_ids );
-            }
+			}else{
+				$included_courses = Sensei_Utils::array_zip_merge( $active_ids, $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');

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.

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 = WooThemes_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 = WooThemes_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 = WooThemes_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 = WooThemes_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

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

templates/single-quiz.php

Spacing +13 added lines, -13 removed lines

@@ -11,7 +11,7 @@ 
  */
 ?>
 
-<?php  get_sensei_header();  ?>
+<?php  get_sensei_header(); ?>
 
 <article <?php post_class(); ?>>
 
@@ -27,15 +27,15 @@ 
          * @param $quiz_id
          *
          */
-        do_action( 'sensei_single_quiz_content_inside_before', get_the_ID() );
+        do_action('sensei_single_quiz_content_inside_before', get_the_ID());
 
     ?>
 
     <section class="entry quiz-questions">
 
-        <?php if ( sensei_quiz_has_questions() ): ?>
+        <?php if (sensei_quiz_has_questions()): ?>
 
-            <form method="POST" action="<?php echo esc_url_raw( get_permalink() ); ?>" enctype="multipart/form-data">
+            <form method="POST" action="<?php echo esc_url_raw(get_permalink()); ?>" enctype="multipart/form-data">
 
                 <?php
 
@@ -46,7 +46,7 @@ 
                      *
                      * @param string $the_quiz_id
                      */
-                    do_action( 'sensei_single_quiz_questions_before', get_the_id() );
+                    do_action('sensei_single_quiz_questions_before', get_the_id());
 
                 ?>
 
@@ -54,9 +54,9 @@ 
 
                 <ol id="sensei-quiz-list">
 
-                <?php while ( sensei_quiz_has_questions() ): sensei_setup_the_question(); ?>
+                <?php while (sensei_quiz_has_questions()): sensei_setup_the_question(); ?>
 
-                    <li class="<?php sensei_the_question_class();?>">
+                    <li class="<?php sensei_the_question_class(); ?>">
 
                         <?php
 
@@ -71,7 +71,7 @@ 
                              * @since 1.9.0
                              * @param string $the_question_id
                              */
-                            do_action( 'sensei_quiz_question_inside_before', sensei_get_the_question_id() );
+                            do_action('sensei_quiz_question_inside_before', sensei_get_the_question_id());
 
                         ?>
 
@@ -86,7 +86,7 @@ 
                              *
                              * @param string $the_question_id
                              */
-                            do_action( 'sensei_quiz_question_inside_after', sensei_get_the_question_id() );
+                            do_action('sensei_quiz_question_inside_after', sensei_get_the_question_id());
 
                         ?>
 
@@ -103,21 +103,21 @@ 
                      *
                      * @param string $the_quiz_id
                      */
-                    do_action( 'sensei_single_quiz_questions_after', get_the_id() );
+                    do_action('sensei_single_quiz_questions_after', get_the_id());
 
                 ?>
 
             </form>
         <?php else:  ?>
 
-            <div class="sensei-message alert"> <?php _e( 'There are no questions for this Quiz yet. Check back soon.', 'woothemes-sensei' ); ?></div>
+            <div class="sensei-message alert"> <?php _e('There are no questions for this Quiz yet. Check back soon.', 'woothemes-sensei'); ?></div>
 
         <?php endif; // End If have questions ?>
 
 
         <?php
             $quiz_lesson = Sensei()->quiz->data->quiz_lesson;
-            do_action( 'sensei_quiz_back_link', $quiz_lesson  );
+            do_action('sensei_quiz_back_link', $quiz_lesson);
         ?>
 
     </section>
@@ -132,7 +132,7 @@ 
      * @hooked
      *
      */
-    do_action( 'sensei_single_quiz_content_inside_after' );
+    do_action('sensei_single_quiz_content_inside_after');
 
     ?>
 

Braces +5 added lines, -2 removed lines

@@ -108,9 +108,12 @@
                 ?>
 
             </form>
-        <?php else:  ?>
+        <?php else {
+	:  ?>
 
-            <div class="sensei-message alert"> <?php _e( 'There are no questions for this Quiz yet. Check back soon.', 'woothemes-sensei' ); ?></div>
+            <div class="sensei-message alert"> <?php _e( 'There are no questions for this Quiz yet. Check back soon.', 'woothemes-sensei' );
+}
+?></div>
 
         <?php endif; // End If have questions ?>
 

Indentation +74 added lines, -74 removed lines

@@ -1,14 +1,14 @@ 
 <?php
 /**
- * The Template for displaying all Quiz Questions.
- *
- * Override this template by copying it to yourtheme/sensei/single-quiz.php
- *
- * @author 		Automattic
- * @package 	Sensei
- * @category    Templates
- * @version     1.9.0
- */
+	 * The Template for displaying all Quiz Questions.
+	 *
+	 * Override this template by copying it to yourtheme/sensei/single-quiz.php
+	 *
+	 * @author 		Automattic
+	 * @package 	Sensei
+	 * @category    Templates
+	 * @version     1.9.0
+	 */
 ?>
 
 <?php  get_sensei_header();  ?>
@@ -17,19 +17,19 @@ 
 
     <?php
 
-        /**
-         * Hook inside the single quiz post above the content
-         *
-         * @since 1.9.0
-         *
-         * @hooked Sensei_Quiz::the_title               - 20
-         * @hooked Sensei_Quiz::the_user_status_message - 40
-         * @param $quiz_id
-         *
-         */
-        do_action( 'sensei_single_quiz_content_inside_before', get_the_ID() );
+		/**
+		 * Hook inside the single quiz post above the content
+		 *
+		 * @since 1.9.0
+		 *
+		 * @hooked Sensei_Quiz::the_title               - 20
+		 * @hooked Sensei_Quiz::the_user_status_message - 40
+		 * @param $quiz_id
+		 *
+		 */
+		do_action( 'sensei_single_quiz_content_inside_before', get_the_ID() );
 
-    ?>
+	?>
 
     <section class="entry quiz-questions">
 
@@ -39,16 +39,16 @@ 
 
                 <?php
 
-                    /**
-                     * Action inside before the question content on single-quiz page
-                     *
-                     * @hooked WooThemes_Sensei_Quiz::the_user_status_message  - 10
-                     *
-                     * @param string $the_quiz_id
-                     */
-                    do_action( 'sensei_single_quiz_questions_before', get_the_id() );
+					/**
+					 * Action inside before the question content on single-quiz page
+					 *
+					 * @hooked WooThemes_Sensei_Quiz::the_user_status_message  - 10
+					 *
+					 * @param string $the_quiz_id
+					 */
+					do_action( 'sensei_single_quiz_questions_before', get_the_id() );
 
-                ?>
+				?>
 
 
 
@@ -60,35 +60,35 @@ 
 
                         <?php
 
-                            /**
-                             * Action inside before the question content on single-quiz page
-                             *
-                             * @hooked WooThemes_Sensei_Question::the_question_title        - 10
-                             * @hooked WooThemes_Sensei_Question::the_question_description  - 20
-                             * @hooked WooThemes_Sensei_Question::the_question_media        - 30
-                             * @hooked WooThemes_Sensei_Question::the_question_hidden_field - 40
-                             *
-                             * @since 1.9.0
-                             * @param string $the_question_id
-                             */
-                            do_action( 'sensei_quiz_question_inside_before', sensei_get_the_question_id() );
-
-                        ?>
+							/**
+							 * Action inside before the question content on single-quiz page
+							 *
+							 * @hooked WooThemes_Sensei_Question::the_question_title        - 10
+							 * @hooked WooThemes_Sensei_Question::the_question_description  - 20
+							 * @hooked WooThemes_Sensei_Question::the_question_media        - 30
+							 * @hooked WooThemes_Sensei_Question::the_question_hidden_field - 40
+							 *
+							 * @since 1.9.0
+							 * @param string $the_question_id
+							 */
+							do_action( 'sensei_quiz_question_inside_before', sensei_get_the_question_id() );
+
+						?>
 
                         <?php sensei_the_question_content(); ?>
 
                         <?php
 
-                            /**
-                             * Action inside before the question content on single-quiz page
-                             *
-                             * @hooked WooThemes_Sensei_Question::answer_feedback_notes
-                             *
-                             * @param string $the_question_id
-                             */
-                            do_action( 'sensei_quiz_question_inside_after', sensei_get_the_question_id() );
+							/**
+							 * Action inside before the question content on single-quiz page
+							 *
+							 * @hooked WooThemes_Sensei_Question::answer_feedback_notes
+							 *
+							 * @param string $the_question_id
+							 */
+							do_action( 'sensei_quiz_question_inside_after', sensei_get_the_question_id() );
 
-                        ?>
+						?>
 
                     </li>
 
@@ -98,14 +98,14 @@ 
 
                 <?php
 
-                    /**
-                     * Action inside before the question content on single-quiz page
-                     *
-                     * @param string $the_quiz_id
-                     */
-                    do_action( 'sensei_single_quiz_questions_after', get_the_id() );
+					/**
+					 * Action inside before the question content on single-quiz page
+					 *
+					 * @param string $the_quiz_id
+					 */
+					do_action( 'sensei_single_quiz_questions_after', get_the_id() );
 
-                ?>
+				?>
 
             </form>
         <?php else:  ?>
@@ -116,25 +116,25 @@ 
 
 
         <?php
-            $quiz_lesson = Sensei()->quiz->data->quiz_lesson;
-            do_action( 'sensei_quiz_back_link', $quiz_lesson  );
-        ?>
+			$quiz_lesson = Sensei()->quiz->data->quiz_lesson;
+			do_action( 'sensei_quiz_back_link', $quiz_lesson  );
+		?>
 
     </section>
 
     <?php
 
-    /**
-     * Hook inside the single quiz post above the content
-     *
-     * @since 1.9.0
-     *
-     * @hooked
-     *
-     */
-    do_action( 'sensei_single_quiz_content_inside_after' );
-
-    ?>
+	/**
+	 * Hook inside the single quiz post above the content
+	 *
+	 * @since 1.9.0
+	 *
+	 * @hooked
+	 *
+	 */
+	do_action( 'sensei_single_quiz_content_inside_after' );
+
+	?>
 
 </article><!-- .quiz -->
 

templates/course-results.php

Spacing +10 added lines, -10 removed lines

@@ -11,7 +11,7 @@ 
  */
 ?>
 
-<?php  get_sensei_header();  ?>
+<?php  get_sensei_header(); ?>
 
 <?php
 /**
@@ -21,15 +21,15 @@ 
  *
  * @hooked Sensei_Course_Results::deprecate_sensei_course_results_content_hook() - 20
  */
-do_action( 'sensei_course_results_content_before' );
+do_action('sensei_course_results_content_before');
 ?>
 
 <?php
 global $course;
-$course = get_page_by_path( $wp_query->query_vars['course_results'], OBJECT, 'course' );
+$course = get_page_by_path($wp_query->query_vars['course_results'], OBJECT, 'course');
 ?>
 
-<article <?php post_class( array( 'course', 'post','course-results' ) ); ?> >
+<article <?php post_class(array('course', 'post', 'course-results')); ?> >
 
     <section class="entry fix">
 
@@ -41,7 +41,7 @@ 
          *
          * @hooked Sensei_Course_Results::fire_sensei_message_hook() - 20
          */
-        do_action( 'sensei_course_results_content_inside_before' );
+        do_action('sensei_course_results_content_inside_before');
         ?>
 
         <header>
@@ -52,12 +52,12 @@ 
 
         </header>
 
-        <?php if ( is_user_logged_in() ):?>
+        <?php if (is_user_logged_in()):?>
 
             <section class="course-results-lessons">
                 <?php
-                $started_course = WooThemes_Sensei_Utils::user_started_course( $course->ID, get_current_user_id() );
-                if( $started_course ) {
+                $started_course = WooThemes_Sensei_Utils::user_started_course($course->ID, get_current_user_id());
+                if ($started_course) {
 
                     sensei_the_course_results_lessons();
 
@@ -75,7 +75,7 @@ 
          *
          * @hooked Sensei()->course_results->course_info - 20
          */
-        do_action( 'sensei_course_results_content_inside_after' );
+        do_action('sensei_course_results_content_inside_after');
         ?>
 
     </section>
@@ -89,7 +89,7 @@ 
  * @since 1.9.0
  *
  */
-do_action( 'sensei_course_results_content_after' );
+do_action('sensei_course_results_content_after');
 ?>
 
 

Indentation +43 added lines, -43 removed lines

@@ -1,26 +1,26 @@ 
 <?php
 /**
- * The Template for displaying course archives, including the course page template.
- *
- * Override this template by copying it to yourtheme/sensei/archive-course.php
- *
- * @author 		Automattic
- * @package 	Sensei
- * @category    Templates
- * @version     1.9.0
- */
+		 * The Template for displaying course archives, including the course page template.
+		 *
+		 * Override this template by copying it to yourtheme/sensei/archive-course.php
+		 *
+		 * @author 		Automattic
+		 * @package 	Sensei
+		 * @category    Templates
+		 * @version     1.9.0
+		 */
 ?>
 
 <?php  get_sensei_header();  ?>
 
 <?php
 /**
- * This hook fire inside learner-profile.php before the content
- *
- * @since 1.9.0
- *
- * @hooked Sensei_Course_Results::deprecate_sensei_course_results_content_hook() - 20
- */
+		 * This hook fire inside learner-profile.php before the content
+		 *
+		 * @since 1.9.0
+		 *
+		 * @hooked Sensei_Course_Results::deprecate_sensei_course_results_content_hook() - 20
+		 */
 do_action( 'sensei_course_results_content_before' );
 ?>
 
@@ -34,15 +34,15 @@ 
     <section class="entry fix">
 
         <?php
-        /**
-         * This hook fire inside learner-profile.php inside directly before the content
-         *
-         * @since 1.9.0
-         *
-         * @hooked Sensei_Course_Results::fire_sensei_message_hook() - 20
-         */
-        do_action( 'sensei_course_results_content_inside_before' );
-        ?>
+		/**
+		 * This hook fire inside learner-profile.php inside directly before the content
+		 *
+		 * @since 1.9.0
+		 *
+		 * @hooked Sensei_Course_Results::fire_sensei_message_hook() - 20
+		 */
+		do_action( 'sensei_course_results_content_inside_before' );
+		?>
 
         <header>
 
@@ -56,27 +56,27 @@ 
 
             <section class="course-results-lessons">
                 <?php
-                $started_course = WooThemes_Sensei_Utils::user_started_course( $course->ID, get_current_user_id() );
-                if( $started_course ) {
+				$started_course = WooThemes_Sensei_Utils::user_started_course( $course->ID, get_current_user_id() );
+				if( $started_course ) {
 
-                    sensei_the_course_results_lessons();
+					sensei_the_course_results_lessons();
 
-                }
-                ?>
+				}
+				?>
             </section>
 
         <?php endif; ?>
 
         <?php
-        /**
-         * This hook fire inside learner-profile.php inside directly after the content
-         *
-         * @since 1.9.0
-         *
-         * @hooked Sensei()->course_results->course_info - 20
-         */
-        do_action( 'sensei_course_results_content_inside_after' );
-        ?>
+		/**
+		 * This hook fire inside learner-profile.php inside directly after the content
+		 *
+		 * @since 1.9.0
+		 *
+		 * @hooked Sensei()->course_results->course_info - 20
+		 */
+		do_action( 'sensei_course_results_content_inside_after' );
+		?>
 
     </section>
 
@@ -84,11 +84,11 @@ 
 
 <?php
 /**
- * This hook fire inside course-results.php before the content
- *
- * @since 1.9.0
- *
- */
+		 * This hook fire inside course-results.php before the content
+		 *
+		 * @since 1.9.0
+		 *
+		 */
 do_action( 'sensei_course_results_content_after' );
 ?>