woothemes / sensei

includes/class-sensei-wc.php

Doc Comments +8 added lines, -7 removed lines

@@ -973,7 +973,7 @@ 
      * @since   1.2.0
      * @since  1.9.0 move to class Sensei_WC
      *
-     * @param   WC_Order $order
+     * @param   integer $order
      *
      * @return  void
      */
@@ -1191,6 +1191,7 @@ 
      * @since   1.2.0
      * @since   1.9.0 Move function to the Sensei_WC class
      * @param   integer| WC_Order $order_id order ID
+     * @param integer $order_id
      * @return  void
      */
     public static function cancel_order ( $order_id ) {
@@ -1309,7 +1310,7 @@ 
      * @param  int 			$course_id  (default: 0)
      * @param  array/Object $order_user (default: array()) Specific user's data.
      *
-     * @return bool|int
+     * @return boolean|string
      */
     public static function course_update ( $course_id = 0, $order_user = array()  ) {
 
@@ -1381,7 +1382,7 @@ 
      *
      * @param  boolean $guest_checkout Current guest checkout setting
      *
-     * @return boolean                 Modified guest checkout setting
+     * @return string|boolean                 Modified guest checkout setting
      */
     public static function disable_guest_checkout( $guest_checkout ) {
 
@@ -1555,7 +1556,7 @@ 
      * Get all the valid subscription types.
      *
      * @since 1.9.0
-     * @return array
+     * @return string[]
      */
     public static function get_subscription_types(){
 
@@ -1573,8 +1574,8 @@ 
      *
      * @since 1.9.0
      *
-     * @param $user_id
-     * @param $product_id
+     * @param integer $user_id
+     * @param string $product_id
      * @param $course_id
      *
      * @return bool
@@ -1638,7 +1639,7 @@ 
 	 * Get all the orders for a specific user and product combination
 	 *
 	 * @param int $user_id
-	 * @param $product_id
+	 * @param integer $product_id
 	 *
 	 * @return array $orders
 	 */

Braces +12 added lines, -5 removed lines

@@ -1,5 +1,8 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) exit; // security check, don't load file outside WP
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
+// security check, don't load file outside WP
 
 /**
  * Sensei WooCommerce class
@@ -1200,7 +1203,7 @@ 
 
 			$order = $order_id;
 
-		}else{
+		} else{
 
 			$order = new WC_Order( $order_id );
 		}
@@ -1315,7 +1318,9 @@ 
 
         global $current_user;
 
-        if ( ! isset( $current_user ) || !$current_user->ID > 0 ) return false;
+        if ( ! isset( $current_user ) || !$current_user->ID > 0 ) {
+        	return false;
+        }
 
         $data_update = false;
 
@@ -1440,7 +1445,9 @@ 
 
         $order = new WC_Order( $order_id );
 
-        if ( ! isset ( $order ) ) return '';
+        if ( ! isset ( $order ) ) {
+        	return '';
+        }
 
         if ( $order_status == 'wc-processing' && ( $order->post_status == 'wc-on-hold' || $order->post_status == 'wc-pending' || $order->post_status == 'wc-failed' ) ) {
 
@@ -1538,7 +1545,7 @@ 
 
             $user_access_permission = true;
 
-        }else{
+        } else{
 
             $user_access_permission = false;
             // do not show the WC permissions message

includes/class-sensei.php

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
 
 /**
  * Responsible for loading Sensei and setting up the Main WordPress hooks.
@@ -155,7 +158,9 @@ 
         $this->init();
 
         // Installation
-        if ( is_admin() && ! defined( 'DOING_AJAX' ) ) $this->install();
+        if ( is_admin() && ! defined( 'DOING_AJAX' ) ) {
+        	$this->install();
+        }
 
         // Run this on activation.
         register_activation_hook( $this->file, array( $this, 'activation' ) );
@@ -506,7 +511,7 @@ 
 
             $sensei_plugin_path =  $this->plugin_path;
 
-        }else{
+        } else{
 
             $sensei_plugin_path = plugin_dir_path( __FILE__ );
 
@@ -719,7 +724,9 @@ 
      */
     public function access_settings () {
 
-        if( sensei_all_access() ) return true;
+        if( sensei_all_access() ) {
+        	return true;
+        }
 
         if ( isset( $this->settings->settings['access_permission'] ) && ( true == $this->settings->settings['access_permission'] ) ) {
             if ( is_user_logged_in() ) {
@@ -772,8 +779,9 @@ 
         }
         $where = "WHERE comment_type IN ('" . join("', '", array_unique( $statuses ) ) . "')";
 
-        if ( $post_id > 0 )
-            $where .= $wpdb->prepare( " AND comment_post_ID = %d", $post_id );
+        if ( $post_id > 0 ) {
+                    $where .= $wpdb->prepare( " AND comment_post_ID = %d", $post_id );
+        }
 
         $count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
 
@@ -781,16 +789,19 @@ 
         $approved = array('0' => 'moderated', '1' => 'approved', 'spam' => 'spam', 'trash' => 'trash', 'post-trashed' => 'post-trashed');
         foreach ( (array) $count as $row ) {
             // Don't count post-trashed toward totals
-            if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] )
-                $total += $row['num_comments'];
-            if ( isset( $approved[$row['comment_approved']] ) )
-                $stats[$approved[$row['comment_approved']]] = $row['num_comments'];
+            if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+                            $total += $row['num_comments'];
+            }
+            if ( isset( $approved[$row['comment_approved']] ) ) {
+                            $stats[$approved[$row['comment_approved']]] = $row['num_comments'];
+            }
         }
 
         $stats['total_comments'] = $total;
         foreach ( $approved as $key ) {
-            if ( empty($stats[$key]) )
-                $stats[$key] = 0;
+            if ( empty($stats[$key]) ) {
+                            $stats[$key] = 0;
+            }
         }
 
         $stats = (object) $stats;
@@ -831,8 +842,9 @@ 
     public function get_image_size( $image_size ) {
 
         // Only return sizes we define in settings
-        if ( ! in_array( $image_size, array( 'course_archive_image', 'course_single_image', 'lesson_archive_image', 'lesson_single_image' ) ) )
-            return apply_filters( 'sensei_get_image_size_' . $image_size, '' );
+        if ( ! in_array( $image_size, array( 'course_archive_image', 'course_single_image', 'lesson_archive_image', 'lesson_single_image' ) ) ) {
+                    return apply_filters( 'sensei_get_image_size_' . $image_size, '' );
+        }
 
         if( ! isset( $this->settings->settings[ $image_size . '_width' ] ) ) {
             $this->settings->settings[ $image_size . '_width' ] = false;
@@ -895,7 +907,7 @@ 
             require_once( 'class-sensei-modules.php');
             Sensei()->modules = new Sensei_Core_Modules( $this->file );
 
-        }else{
+        } else{
             // fallback for people still using the modules extension.
             global $sensei_modules;
             Sensei()->modules = $sensei_modules;
@@ -942,7 +954,7 @@ 
 
             update_option('sensei_flush_rewrite_rules', '2');
 
-        }elseif( '2' == $option ) {
+        } elseif( '2' == $option ) {
 
             flush_rewrite_rules();
             update_option('sensei_flush_rewrite_rules', '0');