woothemes / sensei

apigen/hook-docs.php

Doc Comments +3 added lines

@@ -10,6 +10,9 @@
 	private static $found_files            = array();
 	private static $custom_hooks_found     = '';
 
+	/**
+	 * @param string $pattern
+	 */
 	private static function get_files( $pattern, $flags = 0, $path = '' ) {
 
 	    if ( ! $path && ( $dir = dirname( $pattern ) ) != '.' ) {

Braces +3 added lines, -2 removed lines

@@ -28,8 +28,9 @@
 			    $found_files = array();
 		   		$retrieved_files = (array) self::get_files( $pattern, $flags, $p . '/' );
 		   		foreach ( $retrieved_files as $file ) {
-			   		if ( ! in_array( $file, self::$found_files ) )
-			   			$found_files[] = $file;
+			   		if ( ! in_array( $file, self::$found_files ) ) {
+			   					   			$found_files[] = $file;
+			   		}
 		   		}
 
 		   		self::$found_files = array_merge( self::$found_files, $found_files );

includes/class-sensei-analysis.php

Doc Comments +8 added lines, -1 removed lines

@@ -21,6 +21,7 @@ 
 	/**
 	 * Constructor
 	 * @since  1.0.0
+	 * @param string $file
 	 * @return  void
 	 */
 	public function __construct ( $file ) {
@@ -215,6 +216,7 @@ 
 	/**
 	 * An individual users' profile view for analysis, showing their Courses
 	 * @since  1.2.0
+	 * @param integer $user_id
 	 * @return void
 	 */
 	public function analysis_user_profile_view( $user_id ) {
@@ -242,6 +244,7 @@ 
 	/**
 	 * An individual Course view for analysis, showing the Courses Lessons
 	 * @since  1.2.0
+	 * @param integer $course_id
 	 * @return void
 	 */
 	public function analysis_course_view( $course_id ) {
@@ -269,6 +272,8 @@ 
 	/**
 	 * An individual Course view for analysis, showing a specific Learners Lessons
 	 * @since  1.2.0
+	 * @param integer $course_id
+	 * @param integer $user_id
 	 * @return void
 	 */
 	public function analysis_user_course_view( $course_id, $user_id ) {
@@ -296,6 +301,7 @@ 
 	/**
 	 * An individual Course view for analysis, showing all the Learners
 	 * @since  1.2.0
+	 * @param integer $course_id
 	 * @return void
 	 */
 	public function analysis_course_users_view( $course_id ) {
@@ -323,6 +329,7 @@ 
 	/**
 	 * An individual Lesson view for analysis, showing all the Learners
 	 * @since  1.2.0
+	 * @param integer $lesson_id
 	 * @return void
 	 */
 	public function analysis_lesson_users_view( $lesson_id ) {
@@ -633,7 +640,7 @@ 
 	/**
 	 * Write array data to CSV
 	 * @since  1.2.0
-	 * @param  array  $report_array data array
+	 * @param  array  $report_data data array
 	 * @return void
 	 */
 	public function report_write_download( $report_data = array() ) {

Braces +13 added lines, -19 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * Sensei Analysis Class
@@ -153,24 +156,19 @@ 
 		if ( 0 < $lesson_id ) {
 			// Viewing a specific Lesson and all its Learners
 			$this->analysis_lesson_users_view( $lesson_id );
-		}
-		elseif ( 0 < $course_id && !$user_id && 'user' == $type ) {
+		} elseif ( 0 < $course_id && !$user_id && 'user' == $type ) {
 			// Viewing a specific Course and all its Learners
 			$this->analysis_course_users_view( $course_id );
-		}
-		elseif ( 0 < $course_id && 0 < $user_id ) {
+		} elseif ( 0 < $course_id && 0 < $user_id ) {
 			// Viewing a specific Learner on a specific Course, showing their Lessons
 			$this->analysis_user_course_view( $course_id, $user_id );
-		}
-		elseif( 0 < $course_id ) {
+		} elseif( 0 < $course_id ) {
 			// Viewing a specific Course and all it's Lessons
 			$this->analysis_course_view( $course_id );
-		}
-		elseif ( 0 < $user_id ) {
+		} elseif ( 0 < $user_id ) {
 			// Viewing a specific Learner, and their Courses
 			$this->analysis_user_profile_view( $user_id );
-		}
-		else {
+		} else {
 			// Overview of all Learners, all Courses, or all Lessons
 			$this->analysis_default_view( $type );
 		} // End If Statement
@@ -569,20 +567,16 @@ 
 			if ( 0 < $lesson_id ) {
 				// Viewing a specific Lesson and all its Learners
 				$sensei_analysis_report_object = $this->load_report_object( 'Lesson', $lesson_id );
-			}
-			elseif ( 0 < $course_id && 0 < $user_id ) {
+			} elseif ( 0 < $course_id && 0 < $user_id ) {
 				// Viewing a specific User on a specific Course
 				$sensei_analysis_report_object = $this->load_report_object( 'Course', $course_id, $user_id );
-			}
-			elseif( 0 < $course_id ) {
+			} elseif( 0 < $course_id ) {
 				// Viewing a specific Course and all it's Lessons, or it's Learners
 				$sensei_analysis_report_object = $this->load_report_object( 'Course', $course_id );
-			}
-			elseif ( 0 < $user_id ) {
+			} elseif ( 0 < $user_id ) {
 				// Viewing a specific Learner, and their Courses
 				$sensei_analysis_report_object = $this->load_report_object( 'User_Profile', $user_id );
-			}
-			else {
+			} else {
 				// Overview of all Learners, all Courses, or all Lessons
 				$sensei_analysis_report_object = $this->load_report_object( 'Overview', $type );
 			} // End If Statement

includes/class-sensei-course.php

Doc Comments +1 added lines, -3 removed lines

@@ -372,7 +372,7 @@ 
 	 *
 	 * @access public
 	 * @param int $post_id
-	 * @return void
+	 * @return integer|null
 	 */
 	public function meta_box_save ( $post_id ) {
 		global $post, $messages;
@@ -848,7 +848,6 @@ 
 	 * course_count function.
 	 *
 	 * @access public
-	 * @param array $exclude (default: array())
 	 * @param string $post_status (default: 'publish')
 	 * @return int
 	 */
@@ -1159,7 +1158,6 @@ 
      *
 	 * @since  1.4.0
 	 * @param  object  $user   Queried user object
-	 * @param  boolean $manage Whether the user has permission to manage the courses
 	 * @return string          HTML displayng course data
 	 */
 	public function load_user_courses_content( $user = false ) {

Braces +12 added lines, -10 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 Class
@@ -614,7 +617,7 @@ 
 
 			return $results_array;
 
-		}else{
+		} else{
 
 			//reset the pagination as this widgets do not need it
 			$post_args['paged'] = 1;
@@ -835,7 +838,7 @@ 
 
             return $html;
 
-        }else{
+        } else{
 
             echo $html;
 
@@ -982,8 +985,7 @@ 
 				$has_questions = get_post_meta( $lesson_id, '_quiz_has_questions', true );
 				if ( $has_questions && $boolean_check ) {
 					return true;
-				}
-				elseif ( $has_questions ) {
+				} elseif ( $has_questions ) {
 					$quiz_id = Sensei()->lesson->lesson_quizzes( $lesson_id );
 //					$questions = Sensei()->lesson->lesson_quiz_questions( $quiz_id );
 //					if( count( $questions ) > 0 ) {
@@ -1805,11 +1807,11 @@ 
 
             $course_id = $email['course_id'];
 
-        }elseif( isset( $email['lesson_id'] ) ){
+        } elseif( isset( $email['lesson_id'] ) ){
 
             $course_id = Sensei()->lesson->get_course_id( $email['lesson_id'] );
 
-        }elseif( isset( $email['quiz_id'] ) ){
+        } elseif( isset( $email['quiz_id'] ) ){
 
             $lesson_id = Sensei()->quiz->get_lesson_id( $email['quiz_id'] );
             $course_id = Sensei()->lesson->get_course_id( $lesson_id );
@@ -1870,7 +1872,7 @@ 
 
         if( isset( $_POST['disable_sensei_course_notification'] ) && 'on'== $_POST['disable_sensei_course_notification']  ) {
             $new_val = true;
-        }else{
+        } else{
             $new_val = false;
         }
 
@@ -2009,7 +2011,7 @@ 
 
                 $classes[] = 'user-status-completed';
 
-            }else{
+            } else{
 
                 $classes[] = 'user-status-active';
 
@@ -2826,7 +2828,7 @@ 
 
             $title = $term->name;
 
-        }else{
+        } else{
 
             $title = 'Course Category';
 

includes/class-sensei-emails.php

Doc Comments +5 added lines, -1 removed lines

@@ -40,6 +40,7 @@ 
 	 * Constructor for the email class hooks in all emails that can be sent.
 	 *
 	 * @access public
+	 * @param string $file
 	 * @return void
 	 */
 	function __construct( $file ) {
@@ -140,7 +141,7 @@ 
 	 * Wraps a message in the sensei mail template.
 	 *
 	 * @access public
-	 * @param mixed $content
+	 * @param null|string $content
 	 * @return string
 	 */
 	function wrap_message( $content ) {
@@ -198,6 +199,9 @@ 
 		remove_filter( 'wp_mail_content_type', array( $this, 'get_content_type' ) );
 	}
 
+	/**
+	 * @param string $email_template
+	 */
 	function get_content( $email_template ) {
 
 		$message = $this->load_template( $email_template );

Braces +3 added lines, -1 removed lines

@@ -210,7 +210,9 @@
 	function load_template( $template = '' ) {
 		global  $email_template;
 
-		if( ! $template ) return;
+		if( ! $template ) {
+			return;
+		}
 
 		$email_template = $template . '.php';
 		$template = Sensei_Templates::template_loader( '' );

includes/class-sensei-frontend.php

Doc Comments +2 added lines, -3 removed lines

@@ -473,7 +473,6 @@ 
 	 * Remove Sensei custom menu items depending on settings and logged in status.
 	 *
 	 * @access public
-	 * @param object $item
 	 * @return object $item
 	 */
 	public function sensei_wp_nav_menu_objects( $sorted_menu_items ) {
@@ -530,7 +529,7 @@ 
      *
      * @deprecated since 1.9.0
 	 * @since  1.2.0
-	 * @return void
+	 * @return string|null
 	 */
 	function sensei_course_image( $course_id, $width = '100', $height = '100', $return = false ) {
 
@@ -551,7 +550,7 @@ 
 	/**
 	 * sensei_lesson_image output for lesson image
 	 * @since  1.2.0
-	 * @return void
+	 * @return string|null
      * @deprecated since 1.9.0
 	 */
 	function sensei_lesson_image( $lesson_id, $width = '100', $height = '100', $return = false, $widget = false ) {

Braces +31 added lines, -13 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * Sensei Frontend Class
@@ -560,7 +563,7 @@ 
 
             return Sensei()->lesson->lesson_image( $lesson_id, $width, $height, $widget );
 
-        }else{
+        } else{
 
             echo Sensei()->lesson->lesson_image( $lesson_id, $width, $height, $widget );
 
@@ -661,7 +664,9 @@ 
 	public function sensei_breadcrumb( $id = 0 ) {
 
 		// Only output on lesson, quiz and taxonomy (module) pages
-		if( ! ( is_tax( 'module' ) || is_singular( 'lesson' ) || is_singular( 'quiz' ) ) ) return;
+		if( ! ( is_tax( 'module' ) || is_singular( 'lesson' ) || is_singular( 'quiz' ) ) ) {
+			return;
+		}
 
 		if( empty( $id )  ){
 
@@ -1065,17 +1070,26 @@ 
 
 						<p class="form-row form-row-wide">
 							<label for="sensei_reg_username"><?php _e( 'Username', 'woothemes-sensei' ); ?> <span class="required">*</span></label>
-							<input type="text" class="input-text" name="sensei_reg_username" id="sensei_reg_username" value="<?php if ( ! empty( $_POST['sensei_reg_username'] ) ) esc_attr_e( $_POST['sensei_reg_username'] ); ?>" />
+							<input type="text" class="input-text" name="sensei_reg_username" id="sensei_reg_username" value="<?php if ( ! empty( $_POST['sensei_reg_username'] ) ) {
+	esc_attr_e( $_POST['sensei_reg_username'] );
+}
+?>" />
 						</p>
 
 						<p class="form-row form-row-wide">
 							<label for="sensei_reg_email"><?php _e( 'Email address', 'woothemes-sensei' ); ?> <span class="required">*</span></label>
-							<input type="email" class="input-text" name="sensei_reg_email" id="sensei_reg_email" value="<?php if ( ! empty( $_POST['sensei_reg_email'] ) ) esc_attr_e( $_POST['sensei_reg_email'] ); ?>" />
+							<input type="email" class="input-text" name="sensei_reg_email" id="sensei_reg_email" value="<?php if ( ! empty( $_POST['sensei_reg_email'] ) ) {
+	esc_attr_e( $_POST['sensei_reg_email'] );
+}
+?>" />
 						</p>
 
 						<p class="form-row form-row-wide">
 							<label for="sensei_reg_password"><?php _e( 'Password', 'woothemes-sensei' ); ?> <span class="required">*</span></label>
-							<input type="password" class="input-text" name="sensei_reg_password" id="sensei_reg_password" value="<?php if ( ! empty( $_POST['sensei_reg_password'] ) ) esc_attr_e( $_POST['sensei_reg_password'] ); ?>" />
+							<input type="password" class="input-text" name="sensei_reg_password" id="sensei_reg_password" value="<?php if ( ! empty( $_POST['sensei_reg_password'] ) ) {
+	esc_attr_e( $_POST['sensei_reg_password'] );
+}
+?>" />
 						</p>
 
 						<!-- Spam Trap -->
@@ -1408,8 +1422,12 @@ 
 
 		if( WooThemes_Sensei_Utils::sensei_is_woocommerce_activated() ) {
 
-			if( ! is_user_logged_in() ) return;
-			if( ! isset( $post->ID ) ) return;
+			if( ! is_user_logged_in() ) {
+				return;
+			}
+			if( ! isset( $post->ID ) ) {
+				return;
+			}
 
 			$user_id = $current_user->ID;
 			$course_id = $post->ID;
@@ -1572,7 +1590,7 @@ 
 		    		//assigne the username to the creds array for further processing
 		    		$creds['user_login'] =  $user->user_login ;
 
-		    	}else{
+		    	} else{
 
 		    		// process this as a default username login
 		    		$creds['user_login'] = sanitize_text_field( $_REQUEST['log'] ) ;
@@ -1589,7 +1607,7 @@ 
 				if ( is_wp_error($user) ){ // on login failure
                     wp_redirect( esc_url_raw( add_query_arg('login', 'failed', $referrer) ) );
                     exit;
-				}else{ // on login success
+				} else{ // on login success
 
 					/**
 					* change the redirect url programatically
@@ -1606,7 +1624,7 @@ 
 
 				}	// end is_wp_error($user)
 
-		    }else{ // if username or password is empty
+		    } else{ // if username or password is empty
 
                 wp_redirect( esc_url_raw( add_query_arg('login', 'emptyfields', $referrer) ) );
 		        exit;
@@ -1615,7 +1633,7 @@ 
 
 	    	return ;
 
-	    }elseif( ( isset( $_GET['login'] ) ) ) {
+	    } elseif( ( isset( $_GET['login'] ) ) ) {
 	    	// else if this request is a redircect from a previously faile login request
 	    	$this->login_message_process();
 
@@ -1736,7 +1754,7 @@ 
 
 				$message = __('Incorrect login details', 'woothemes-sensei' );
 
-			}elseif( $_GET['login'] == 'emptyfields'  ){
+			} elseif( $_GET['login'] == 'emptyfields'  ){
 
 				$message= __('Please enter your username and password', 'woothemes-sensei' );
 			}

includes/class-sensei-grading.php

Doc Comments +5 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
 	 * Constructor
 	 * @since  1.3.0
      *
-     * @param $file
+     * @param string $file
 	 */
 	public function __construct ( $file ) {
 		$this->name = __( 'Grading', 'woothemes-sensei' );
@@ -330,6 +330,7 @@ 
 	/**
 	 * Return array of valid statuses for either Course or Lesson
 	 * @since  1.7.0
+	 * @param string $type
 	 * @return array
 	 */
 	public function get_stati( $type ) {
@@ -529,7 +530,7 @@ 
      * the grades as the Grader selected theme and saves the total grade and
      * individual question grades.
      *
-     * @return bool
+     * @return false|null
      */
     public function admin_process_grading_submission() {
 
@@ -906,8 +907,8 @@ 
      * Grading logic specifically for the gap fill questions
      *
      * @since 1.9.0
-     * @param $question_id
-     * @param $user_answer
+     * @param integer $question_id
+     * @param string $user_answer
      *
      * @return bool | int false or the grade given to the user answer
      */

Braces +20 added lines, -28 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * Sensei Grading Class
@@ -130,8 +133,7 @@ 
 		$object_name = 'WooThemes_Sensei_Grading_' . $name;
 		if ( is_null($optional_data) ) {
 			$sensei_grading_object = new $object_name( $data );
-		}
-		else {
+		} else {
 			$sensei_grading_object = new $object_name( $data, $optional_data );
 		} // End If Statement
 		if ( 'Main' == $name ) {
@@ -150,8 +152,7 @@ 
 
 		if ( isset( $_GET['quiz_id'] ) && 0 < intval( $_GET['quiz_id'] ) && isset( $_GET['user'] ) && 0 < intval( $_GET['user'] ) ) {
 			$this->grading_user_quiz_view();
-		}
-		else {
+		} else {
 			$this->grading_default_view();
 		} // End If Statement
 	} // End grading_page()
@@ -273,8 +274,7 @@ 
 			if ( version_compare($wp_version, '4.1', '>=') ) {
 				$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
 				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
-			}
-			else {
+			} else {
 				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', get_the_title( $course_id ) ); 
 			}
 		}
@@ -309,8 +309,7 @@ 
 			if ( version_compare($wp_version, '4.1', '>=') ) {
 				$url = add_query_arg( array( 'page' => $this->page_slug, 'course_id' => $course_id ), admin_url( 'admin.php' ) );
 				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;<a href="%s">%s</a></span>', esc_url( $url ), get_the_title( $course_id ) );
-			}
-			else {
+			} else {
 				$title .= sprintf( '&nbsp;&nbsp;<span class="course-title">&gt;&nbsp;&nbsp;%s</span>', get_the_title( $course_id ) ); 
 			}
 			$url = add_query_arg( array( 'page' => $this->page_slug, 'lesson_id' => $lesson_id ), admin_url( 'admin.php' ) );
@@ -379,8 +378,7 @@ 
 
 		if ( 'course' == $args['type'] ) {
 			$type = 'sensei_course_status';
-		}
-		else {
+		} else {
 			$type = 'sensei_lesson_status';
 		}
 		$cache_key = 'sensei-' . $args['type'] . '-statuses';
@@ -390,15 +388,13 @@ 
         // Restrict to specific posts
 		if ( isset( $args['post__in'] ) && !empty( $args['post__in'] ) && is_array( $args['post__in'] ) ) {
 			$query .= ' AND comment_post_ID IN (' . implode( ',', array_map( 'absint', $args['post__in'] ) ) . ')';
-		}
-		elseif ( !empty( $args['post_id'] ) ) {
+		} elseif ( !empty( $args['post_id'] ) ) {
 			$query .= $wpdb->prepare( ' AND comment_post_ID = %d', $args['post_id'] );
 		}
 		// Restrict to specific users
 		if ( isset( $args['user_id'] ) && is_array( $args['user_id'] ) ) {
 			$query .= ' AND user_id IN (' . implode( ',', array_map( 'absint', $args['user_id'] ) ) . ')';
-		}
-		elseif ( !empty( $args['user_id'] ) ) {
+		} elseif ( !empty( $args['user_id'] ) ) {
 			$query .= $wpdb->prepare( ' AND user_id = %d', $args['user_id'] );
 		}
 		$query .= ' GROUP BY comment_approved';
@@ -601,8 +597,7 @@ 
             // set the users total quiz grade
 			if ( 0 < intval( $quiz_grade_total ) ) {
             $grade = abs( round( ( doubleval( $quiz_grade ) * 100 ) / ( $quiz_grade_total ), 2 ) );
-			}
-			else {
+			} else {
 				$grade = 0;
 			}
             WooThemes_Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id );
@@ -615,8 +610,7 @@ 
                 // Student has reached the pass mark and lesson is complete
                 if ( $quiz_passmark <= $grade ) {
                     $lesson_status = 'passed';
-                }
-                else {
+                } else {
                     $lesson_status = 'failed';
                 } // End If Statement
             }
@@ -761,8 +755,7 @@ 
 			// Question has a zero grade, so skip grading
 			if ( 0 == $achievable_grade ) {
 				$all_question_grades[ $question_id ] = $achievable_grade;
-			}
-            elseif ( in_array( $question_type, $autogradable_question_types ) ) {
+			} elseif ( in_array( $question_type, $autogradable_question_types ) ) {
                 // Get user question grade
                 $question_grade = WooThemes_Sensei_Utils::sensei_grade_question_auto( $question_id, $question_type, $answer, $user_id );
                 $all_question_grades[ $question_id ] = $question_grade;
@@ -784,8 +777,7 @@ 
 			// Check for zero total from grades
 			if ( 0 < $quiz_total ) {
             $grade = abs( round( ( doubleval( $grade_total ) * 100 ) / ( $quiz_total ), 2 ) );
-			}
-			else {
+			} else {
 				$grade = 0;
 			}
             WooThemes_Sensei_Utils::sensei_grade_quiz( $quiz_id, $grade, $user_id, $quiz_grade_type );
@@ -945,19 +937,19 @@ 
 
                     return Sensei()->question->get_question_grade($question_id);
 
-                }else{
+                } else{
 
                     return false;
 
                 }
 
-            }else{
+            } else{
 
                 return false;
 
             }
 
-        }else{
+        } else{
 
             // Case Sensitive Check that the 'gap' is "exactly" equal to the given answer
             if ( trim(strtolower($gapfill_array[1])) == trim(strtolower( $user_answer )) ) {
@@ -970,13 +962,13 @@ 
 
                     return  Sensei()->question->get_question_grade( $question_id );
 
-                }else{
+                } else{
 
                     return false;
 
                 }
 
-            }else{
+            } else{
 
                 return false;
 

includes/class-sensei-learners.php

Doc Comments +1 added lines

@@ -21,6 +21,7 @@
 	/**
 	 * Constructor
 	 * @since  1.6.0
+	 * @param string $file
 	 * @return  void
 	 */
 	public function __construct ( $file ) {

Braces +15 added lines, -6 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * Sensei Learners Class
@@ -342,7 +345,7 @@ 
 
                     $name = $full_name;
 
-                }else{
+                } else{
 
                     $name = $full_name . ' ['. $user->display_name .']';
 
@@ -359,11 +362,17 @@ 
 
 		$result = false;
 
-		if( ! isset( $_POST['add_learner_submit'] ) ) return $result;
+		if( ! isset( $_POST['add_learner_submit'] ) ) {
+			return $result;
+		}
 
-		if ( ! isset( $_POST['add_learner_nonce'] ) || ! wp_verify_nonce( $_POST['add_learner_nonce'], 'add_learner_to_sensei' ) ) return $result;
+		if ( ! isset( $_POST['add_learner_nonce'] ) || ! wp_verify_nonce( $_POST['add_learner_nonce'], 'add_learner_to_sensei' ) ) {
+			return $result;
+		}
 
-		if( ( ! isset( $_POST['add_user_id'] ) || '' ==  $_POST['add_user_id'] ) || ! isset( $_POST['add_post_type'] ) || ! isset( $_POST['add_course_id'] ) || ! isset( $_POST['add_lesson_id'] ) ) return $result;
+		if( ( ! isset( $_POST['add_user_id'] ) || '' ==  $_POST['add_user_id'] ) || ! isset( $_POST['add_post_type'] ) || ! isset( $_POST['add_course_id'] ) || ! isset( $_POST['add_lesson_id'] ) ) {
+			return $result;
+		}
 
 		$post_type = $_POST['add_post_type'];
 		$user_id = absint( $_POST['add_user_id'] );
@@ -479,7 +488,7 @@ 
 
             $full_name = trim( $user->first_name   ) . ' ' . trim( $user->last_name  );
 
-        }else{
+        } else{
 
             $full_name =  $user->display_name;
 

includes/class-sensei-lesson.php

Doc Comments +2 added lines, -2 removed lines

@@ -239,7 +239,7 @@ 
 	 *
 	 * @access public
 	 * @param int $post_id
-	 * @return void
+	 * @return integer|null
 	 */
 	public function meta_box_save ( $post_id ) {
 
@@ -2418,7 +2418,7 @@ 
 	 *
 	 * @access private
 	 * @param array $data (default: array())
-	 * @return void
+	 * @return boolean
 	 */
 	private function lesson_delete_question( $data = array() ) {
 

Braces +28 added lines, -16 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * Sensei Lessons Class
@@ -378,8 +381,7 @@ 
 		$quiz_questions = Sensei()->lesson->lesson_quiz_questions( $quiz_id );
 		if( 0 < count( $quiz_questions ) ) {
 			update_post_meta( $post_id, '_quiz_has_questions', '1' );
-		}
-		else {
+		} else {
 			delete_post_meta( $post_id, '_quiz_has_questions' );
 		}
 
@@ -390,7 +392,9 @@ 
 
 	public function get_submitted_setting_value( $field = false ) {
 
-		if( ! $field ) return;
+		if( ! $field ) {
+			return;
+		}
 
 		$value = false;
 
@@ -1507,7 +1511,7 @@ 
 
             $field_name = 'answer_feedback_boolean';
 
-        }elseif( 'multiple-choice' == $question_type ){
+        } elseif( 'multiple-choice' == $question_type ){
 
             $field_name = 'answer_feedback_multiple_choice';
 
@@ -1603,7 +1607,9 @@ 
 
 		$html = '';
 
-		if( ! $lesson_id && ! $quiz_id ) return $html;
+		if( ! $lesson_id && ! $quiz_id ) {
+			return $html;
+		}
 
 		$settings = $this->get_quiz_settings( $quiz_id );
 
@@ -2258,11 +2264,11 @@ 
 
             $answer_feedback = $data[ 'answer_feedback_boolean' ];
 
-		}elseif( isset( $data[ 'answer_feedback_multiple_choice' ] ) && !empty( $data[ 'answer_feedback_multiple_choice' ] ) ){
+		} elseif( isset( $data[ 'answer_feedback_multiple_choice' ] ) && !empty( $data[ 'answer_feedback_multiple_choice' ] ) ){
 
             $answer_feedback = $data[ 'answer_feedback_multiple_choice' ];
 
-        }elseif( isset( $data[ 'answer_feedback' ] )  ){
+        } elseif( isset( $data[ 'answer_feedback' ] )  ){
 
             $answer_feedback = $data[ 'answer_feedback' ];
 
@@ -2275,8 +2281,7 @@ 
 		// Handle the extended question text
 		if ( isset( $data[ 'question_description' ] ) && ( '' != $data[ 'question_description' ] ) ) {
 			$post_content = $data[ 'question_description' ];
-		}
-		else {
+		} else {
 			$post_content = '';
 		}
 		// Question Query Arguments
@@ -2484,8 +2489,7 @@ 
 				'key' => '_lesson_course',
 				'value' => $course_id,
 			);
-		}
-		else {
+		} else {
 			// Simple check for connection to a Course
 			$post_args['meta_query'][] = array(
 				'key' => '_lesson_course',
@@ -2613,9 +2617,13 @@ 
 				// Fetch each question in the order in which they were asked
 				$questions = array();
 				foreach( $selected_questions as $question_id ) {
-					if( ! $question_id ) continue;
+					if( ! $question_id ) {
+						continue;
+					}
 					$question = get_post( $question_id );
-					if( ! isset( $question ) || ! isset( $question->ID ) ) continue;
+					if( ! isset( $question ) || ! isset( $question->ID ) ) {
+						continue;
+					}
 					$questions[] = $question;
 				}
 
@@ -2628,7 +2636,9 @@ 
 
 				// Set array of questions that already exist so we can prevent duplicates from appearing
 				foreach( $questions_array as $question ) {
-					if( 'question' != $question->post_type ) continue;
+					if( 'question' != $question->post_type ) {
+						continue;
+					}
 					$existing_questions[] = $question->ID;
 				}
 
@@ -2668,7 +2678,9 @@ 
 
 						// Add selected questions to existing questions array to prevent duplicates from being added
 						foreach( $questions_array as $cat_question ) {
-							if( in_array( $cat_question->ID, $existing_questions ) ) continue;
+							if( in_array( $cat_question->ID, $existing_questions ) ) {
+								continue;
+							}
 							$existing_questions[] = $cat_question->ID;
 						}
 					}

includes/class-sensei-list-table.php

Doc Comments +1 added lines

@@ -44,6 +44,7 @@
 	/**
 	 * Constructor
 	 * @since  1.2.0
+	 * @param string $token
 	 * @return  void
 	 */
 	public function __construct ( $token ) {

Braces +16 added lines, -9 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // Exit if accessed directly
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// Exit if accessed directly
 
 /**
  * Sensei Generic List Table Class
@@ -131,8 +134,9 @@ 
 	 * @return array
 	 */
 	function get_column_info() {
-		if ( isset( $this->_column_headers ) )
-			return $this->_column_headers;
+		if ( isset( $this->_column_headers ) ) {
+					return $this->_column_headers;
+		}
 
 		$columns = $this->get_columns();
 		$hidden = get_hidden_columns( $this->screen );
@@ -152,12 +156,14 @@ 
 
 		$sortable = array();
 		foreach ( $_sortable as $id => $data ) {
-			if ( empty( $data ) )
-				continue;
+			if ( empty( $data ) ) {
+							continue;
+			}
 
 			$data = (array) $data;
-			if ( !isset( $data[1] ) )
-				$data[1] = false;
+			if ( !isset( $data[1] ) ) {
+							$data[1] = false;
+			}
 
 			$sortable[$id] = $data;
 		}
@@ -188,8 +194,9 @@ 
 			$class = "class='$column_name column-$column_name'";
 
 			$style = '';
-			if ( in_array( $column_name, $hidden ) )
-				$style = ' style="display:none;"';
+			if ( in_array( $column_name, $hidden ) ) {
+							$style = ' style="display:none;"';
+			}
 
 			$attributes = "$class$style";