eventespresso / event-espresso-core

core/db_models/strategies/EE_Restriction_Generator_Meta.strategy.php

Spacing +13 added lines, -13 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 /**
@@ -14,7 +14,7 @@ 
  * @author				Mike Nelson
  *
  */
-class EE_Restriction_Generator_Meta extends EE_Restriction_Generator_Base{
+class EE_Restriction_Generator_Meta extends EE_Restriction_Generator_Base {
 
 	/**
 	 *
@@ -33,7 +33,7 @@ 
 	 * @param string $key_field_name
 	 * @param string $value_field_name
 	 */
-	public function __construct( $key_field_name, $value_field_name ) {
+	public function __construct($key_field_name, $value_field_name) {
 		$this->_key_field_name = $key_field_name;
 		$this->_value_field_name = $value_field_name;
 	}
@@ -45,21 +45,21 @@ 
 	 * @throws EE_Error
 	 */
 	protected function _generate_restrictions() {
-		$whitelisted_meta_keys = apply_filters( 'FHEE__EE_Restriction_Generator_Meta___generate_restrictions__whitelisted_meta_keys', array() );
-		$blacklisted_meta_keys = apply_filters( 'FHEE__EE_Restriction_Generator_Meta___generate_restrictions__blacklisted_meta_keys', array() );
+		$whitelisted_meta_keys = apply_filters('FHEE__EE_Restriction_Generator_Meta___generate_restrictions__whitelisted_meta_keys', array());
+		$blacklisted_meta_keys = apply_filters('FHEE__EE_Restriction_Generator_Meta___generate_restrictions__blacklisted_meta_keys', array());
 		$conditions = array(
-			$this->_key_field_name => array( 'NOT_LIKE', "\\\\_%" ),//each slash is escaped because we are using double quotes, and 
+			$this->_key_field_name => array('NOT_LIKE', "\\\\_%"), //each slash is escaped because we are using double quotes, and 
 			//stripslashes will be called on this because the models assume this is from user input
-			$this->_value_field_name => array( 'NOT_REGEXP', '^[aOs]:[\d]:.*$')					
+			$this->_value_field_name => array('NOT_REGEXP', '^[aOs]:[\d]:.*$')					
 		);
-		if( ! empty( $blacklisted_meta_keys ) ) {
-			$conditions[ $this->_key_field_name . '*blacklisted' ] = array( 'NOT_IN', $blacklisted_meta_keys );
+		if ( ! empty($blacklisted_meta_keys)) {
+			$conditions[$this->_key_field_name.'*blacklisted'] = array('NOT_IN', $blacklisted_meta_keys);
 		}
-		if( ! empty( $whitelisted_meta_keys ) ) {
+		if ( ! empty($whitelisted_meta_keys)) {
 			$conditions = array(
 				'OR*whitelisted-or-normal' => array(
 					'AND' => $conditions,
-					$this->_key_field_name . '*whitelisted' => array( 'IN', $whitelisted_meta_keys ) 
+					$this->_key_field_name.'*whitelisted' => array('IN', $whitelisted_meta_keys) 
 				)
 			);
 		}
@@ -67,7 +67,7 @@ 
 			//only allow access to non-protected metas if they're an admin
 			EE_Restriction_Generator_Base::get_default_restrictions_cap() => new EE_Return_None_Where_Conditions(),
 			//don't allow access to protected metas to anyone. If they want that, don't apply caps to the query
-			'apply-to-all-queries-using-caps' => new EE_Default_Where_Conditions( $conditions ),
+			'apply-to-all-queries-using-caps' => new EE_Default_Where_Conditions($conditions),
 		);
 	}
 }

core/libraries/rest_api/controllers/Base.php

Doc Comments +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
 	 *
 	 * @param string $route
 	 * @param string $regex
-	 * @param array  $match_keys EXCLUDING matching the entire regex
+	 * @param string[]  $match_keys EXCLUDING matching the entire regex
 	 * @return array where  $match_keys are the keys (the first value of $match_keys
 	 * becomes the first key of the return value, etc. Eg passing in $match_keys of
 	 * array( 'model', 'id' ), will, if the regex is successful, will return

Spacing +60 added lines, -60 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 namespace EventEspresso\core\libraries\rest_api\controllers;
 use EventEspresso\core\libraries\rest_api\Rest_Exception;
-if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 /**
@@ -49,7 +49,7 @@ 
 
 
 	public function __construct() {
-		$this->_debug_mode = defined( 'EE_REST_API_DEBUG_MODE' ) ? EE_REST_API_DEBUG_MODE : false;
+		$this->_debug_mode = defined('EE_REST_API_DEBUG_MODE') ? EE_REST_API_DEBUG_MODE : false;
 	}
 
 
@@ -57,7 +57,7 @@ 
 	 * Sets the version the user requested
 	 * @param string $version eg '4.8'
 	 */
-	public function set_requested_version( $version ) {
+	public function set_requested_version($version) {
 		$this->_requested_version = $version;
 	}
 
@@ -66,8 +66,8 @@ 
 	 * @param string $key
 	 * @param string|array $info
 	 */
-	protected function _set_debug_info( $key, $info ){
-		$this->_debug_info[ $key ] = $info;
+	protected function _set_debug_info($key, $info) {
+		$this->_debug_info[$key] = $info;
 	}
 
 	/**
@@ -79,14 +79,14 @@ 
 	 * @param boolean      $use_ee_prefix whether to use the EE prefix on the header, or fallback to
 	 *                                    the standard WP one
 	 */
-	protected function _set_response_header( $header_key, $value, $use_ee_prefix = true ) {
-		if( is_array( $value ) ) {
-			foreach( $value as $value_key => $value_value ) {
-				$this->_set_response_header(  $header_key . '[' . $value_key . ']', $value_value );
+	protected function _set_response_header($header_key, $value, $use_ee_prefix = true) {
+		if (is_array($value)) {
+			foreach ($value as $value_key => $value_value) {
+				$this->_set_response_header($header_key.'['.$value_key.']', $value_value);
 			}
 		} else {
 			$prefix = $use_ee_prefix ? Base::header_prefix_for_ee : Base::header_prefix_for_wp;
-			$this->_response_headers[ $prefix . $header_key  ] = $value;
+			$this->_response_headers[$prefix.$header_key] = $value;
 		}
 	}
 
@@ -95,7 +95,7 @@ 
 	 * @return array
 	 */
 	protected function _get_response_headers() {
-		return apply_filters( 'FHEE__EventEspresso\core\libraries\rest_api\controllers\Base___get_response_headers',
+		return apply_filters('FHEE__EventEspresso\core\libraries\rest_api\controllers\Base___get_response_headers',
 			$this->_response_headers,
 			$this,
 			$this->_requested_version
@@ -107,13 +107,13 @@ 
 	 * @param \WP_Error $wp_error_response
 	 * @return \WP_Error
 	 */
-	protected function _add_ee_errors_to_response( \WP_Error $wp_error_response ) {
+	protected function _add_ee_errors_to_response(\WP_Error $wp_error_response) {
 		$notices_during_checkin = \EE_Error::get_raw_notices();
-		if( ! empty( $notices_during_checkin[ 'errors' ] ) ) {
-			foreach( $notices_during_checkin[ 'errors' ] as $error_code => $error_message ) {
+		if ( ! empty($notices_during_checkin['errors'])) {
+			foreach ($notices_during_checkin['errors'] as $error_code => $error_message) {
 				$wp_error_response->add(
-					sanitize_key( $error_code ),
-					strip_tags( $error_message ) );
+					sanitize_key($error_code),
+					strip_tags($error_message) );
 			}
 		}
 		return $wp_error_response;
@@ -131,27 +131,27 @@ 
 	 * @param array|\WP_Error|\Exception $response
 	 * @return \WP_REST_Response
 	 */
-	public function send_response( $response ) {
-		if( $response instanceof Rest_Exception ) {
-			$response = new \WP_Error( $response->get_string_code(), $response->getMessage(), $response->get_data() );
+	public function send_response($response) {
+		if ($response instanceof Rest_Exception) {
+			$response = new \WP_Error($response->get_string_code(), $response->getMessage(), $response->get_data());
 		}
-		if( $response instanceof \Exception ) {
+		if ($response instanceof \Exception) {
 			$code = $response->getCode() ? $response->getCode() : 'error_occurred';
-			$response = new \WP_Error( $code, $response->getMessage() );
+			$response = new \WP_Error($code, $response->getMessage());
 		}
-		if( $response instanceof \WP_Error ) {
-			$response = $this->_add_ee_errors_to_response( $response );
-			$rest_response = $this->_create_rest_response_from_wp_error( $response );
-		}else{
-			$rest_response = new \WP_REST_Response( $response, 200 );
+		if ($response instanceof \WP_Error) {
+			$response = $this->_add_ee_errors_to_response($response);
+			$rest_response = $this->_create_rest_response_from_wp_error($response);
+		} else {
+			$rest_response = new \WP_REST_Response($response, 200);
 		}
 		$headers = array();
-		if( $this->_debug_mode && is_array( $this->_debug_info ) ) {
-			foreach( $this->_debug_info  as $debug_key => $debug_info ) {
-				if( is_array( $debug_info ) ) {
-					$debug_info = wp_json_encode( $debug_info );
+		if ($this->_debug_mode && is_array($this->_debug_info)) {
+			foreach ($this->_debug_info  as $debug_key => $debug_info) {
+				if (is_array($debug_info)) {
+					$debug_info = wp_json_encode($debug_info);
 				}
-				$headers[ 'X-EE4-Debug-' . ucwords( $debug_key ) ] = $debug_info;
+				$headers['X-EE4-Debug-'.ucwords($debug_key)] = $debug_info;
 			}
 		}
 		$headers = array_merge(
@@ -160,7 +160,7 @@ 
 			$this->_get_headers_from_ee_notices()
 		);
 
-		$rest_response->set_headers( $headers );
+		$rest_response->set_headers($headers);
 		return $rest_response;
 	}
 
@@ -171,31 +171,31 @@ 
 	 * @param \WP_Error $wp_error
 	 * @return \WP_REST_Response
 	 */
-	protected function _create_rest_response_from_wp_error( \WP_Error $wp_error ) {
+	protected function _create_rest_response_from_wp_error(\WP_Error $wp_error) {
 		$error_data = $wp_error->get_error_data();
-		if ( is_array( $error_data ) && isset( $error_data['status'] ) ) {
+		if (is_array($error_data) && isset($error_data['status'])) {
 			$status = $error_data['status'];
 		} else {
 			$status = 500;
 		}
 
 		$errors = array();
-		foreach ( (array) $wp_error->errors as $code => $messages ) {
-			foreach ( (array) $messages as $message ) {
+		foreach ((array) $wp_error->errors as $code => $messages) {
+			foreach ((array) $messages as $message) {
 				$errors[] = array(
 					'code'    => $code,
 					'message' => $message,
-					'data'    => $wp_error->get_error_data( $code )
+					'data'    => $wp_error->get_error_data($code)
 				);
 			}
 		}
-		$data = isset( $errors[0] ) ? $errors[0] : array();
-		if ( count( $errors ) > 1 ) {
+		$data = isset($errors[0]) ? $errors[0] : array();
+		if (count($errors) > 1) {
 			// Remove the primary error.
-			array_shift( $errors );
+			array_shift($errors);
 			$data['additional_errors'] = $errors;
 		}
-		return new \WP_REST_Response( $data, $status );
+		return new \WP_REST_Response($data, $status);
 	}
 
 	/**
@@ -205,12 +205,12 @@ 
 	protected function _get_headers_from_ee_notices() {
 		$headers = array();
 		$notices = \EE_Error::get_raw_notices();
-		foreach( $notices as $notice_type => $sub_notices ) {
-			if( ! is_array( $sub_notices ) ) {
+		foreach ($notices as $notice_type => $sub_notices) {
+			if ( ! is_array($sub_notices)) {
 				continue;
 			}
-			foreach( $sub_notices as $notice_code => $sub_notice ) {
-				$headers[ 'X-EE4-Notices-' . \EEH_Inflector::humanize( $notice_type ) . '[' . $notice_code . ']' ] = strip_tags( $sub_notice );
+			foreach ($sub_notices as $notice_code => $sub_notice) {
+				$headers['X-EE4-Notices-'.\EEH_Inflector::humanize($notice_type).'['.$notice_code.']'] = strip_tags($sub_notice);
 			}
 		}
 		return apply_filters(
@@ -228,14 +228,14 @@ 
 	 * @param string $route
 	 * @return string
 	 */
-	public function get_requested_version( $route = null ) {
+	public function get_requested_version($route = null) {
 		$matches = $this->parse_route(
 			$route,
-			'~' . \EED_Core_Rest_Api::ee_api_namespace_for_regex . '~',
-			array( 'version' )
+			'~'.\EED_Core_Rest_Api::ee_api_namespace_for_regex.'~',
+			array('version')
 			);
-		if( isset( $matches[ 'version' ] ) ) {
-			return $matches[ 'version' ];
+		if (isset($matches['version'])) {
+			return $matches['version'];
 		} else {
 			return \EED_Core_Rest_Api::latest_rest_api_version();
 		}
@@ -260,23 +260,23 @@ 
 	 * array( 'model' => 'foo', 'id' => 'bar' )
 	 * @throws \EE_Error if it couldn't be parsed
 	 */
-	public function parse_route( $route, $regex, $match_keys ) {
+	public function parse_route($route, $regex, $match_keys) {
 		$indexed_matches = array();
-		$success = preg_match( $regex, $route, $matches );
-		if(
-			is_array( $matches ) ) {
+		$success = preg_match($regex, $route, $matches);
+		if (
+			is_array($matches) ) {
 			//skip the overall regex match. Who cares
-			for( $i = 1; $i <= count( $match_keys ); $i++ ) {
-				if( ! isset( $matches[ $i ] ) ) {
+			for ($i = 1; $i <= count($match_keys); $i++) {
+				if ( ! isset($matches[$i])) {
 					$success = false;
 				} else {
-					$indexed_matches[ $match_keys[ $i - 1 ] ] = $matches[ $i ];
+					$indexed_matches[$match_keys[$i - 1]] = $matches[$i];
 				}
 			}
 		}
-		if( ! $success ) {
+		if ( ! $success) {
 			throw new \EE_Error(
-				__( 'We could not parse the URL. Please contact Event Espresso Support', 'event_espresso' ),
+				__('We could not parse the URL. Please contact Event Espresso Support', 'event_espresso'),
 				'endpoint_parsing_error'
 			);
 		}

Braces +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
 		if( $response instanceof \WP_Error ) {
 			$response = $this->_add_ee_errors_to_response( $response );
 			$rest_response = $this->_create_rest_response_from_wp_error( $response );
-		}else{
+		} else{
 			$rest_response = new \WP_REST_Response( $response, 200 );
 		}
 		$headers = array();

core/db_models/strategies/EE_CPT_Minimum_Where_Conditions.strategy.php

Doc Comments +4 added lines, -1 removed lines

@@ -7,6 +7,10 @@ 
 
 	protected $_post_type;
 	protected $_meta_field;
+
+	/**
+	 * @param string $post_type
+	 */
 	function __construct($post_type, $meta_field_to_chk = ''){
 		$this->_post_type = $post_type;
 		$this->_meta_field = $meta_field_to_chk;
@@ -27,7 +31,6 @@ 
 	}
 	/**
 	 * Gets the where default where conditions for a custom post type model
-	 * @param string $model_relation_path. Eg, from Event to Payment, this should be "Registration.Transaction.Payment"
 	 * @return array like EEM_Base::get_all's $query_params's index [0] (where conditions)
 	 */
 	protected function _get_default_where_conditions() {

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  *
@@ -15,11 +15,11 @@ 
  * @since		 	   4.8.29.rc.010
  *
  */
-class EE_CPT_Minimum_Where_Conditions extends EE_Default_Where_Conditions{
+class EE_CPT_Minimum_Where_Conditions extends EE_Default_Where_Conditions {
 
 	protected $_post_type;
 	protected $_meta_field;
-	function __construct($post_type, $meta_field_to_chk = ''){
+	function __construct($post_type, $meta_field_to_chk = '') {
 		$this->_post_type = $post_type;
 		$this->_meta_field = $meta_field_to_chk;
 	}
@@ -29,10 +29,10 @@ 
 	 * @param string $column column name
 	 * @return EE_Model_Field_Base
 	 */
-	protected function _get_field_on_column($column){
+	protected function _get_field_on_column($column) {
 		$all_fields = $this->_model->field_settings(true);
-		foreach($all_fields as $field_name => $field_obj){
-			if($column == $field_obj->get_table_column()){
+		foreach ($all_fields as $field_name => $field_obj) {
+			if ($column == $field_obj->get_table_column()) {
 				return $field_obj;
 			}
 		}

Indentation +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 /**
  *
  * Class EE_CPT_Minimum_Where_Conditions
-  * 
+ * 
  * Strategy specifically for adding where conditions specific to CPT models.
  * But only sets the minimum, so any row of the right type will get used
  *

core/db_models/strategies/EE_CPT_Where_Conditions.strategy.php

Doc Comments -1 removed lines

@@ -17,7 +17,6 @@
 class EE_CPT_Where_Conditions extends EE_CPT_Minimum_Where_Conditions{
 	/**
 	 * Gets the where default where conditions for a custom post type model
-	 * @param string $model_relation_path. Eg, from Event to Payment, this should be "Registration.Transaction.Payment"
 	 * @return array like EEM_Base::get_all's $query_params's index [0] (where conditions)
 	 */
 	protected function _get_default_where_conditions() {

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  *
@@ -14,7 +14,7 @@ 
  * * @since		 	   4.6.0
  *
  */
-class EE_CPT_Where_Conditions extends EE_CPT_Minimum_Where_Conditions{
+class EE_CPT_Where_Conditions extends EE_CPT_Minimum_Where_Conditions {
 	/**
 	 * Gets the where default where conditions for a custom post type model
 	 * @param string $model_relation_path. Eg, from Event to Payment, this should be "Registration.Transaction.Payment"
@@ -25,7 +25,7 @@ 
 		return array_merge( 
 			parent::_get_default_where_conditions(),
 			array( 
-				$status_field->get_name() => array('NOT IN',array('auto-draft','trash') )
+				$status_field->get_name() => array('NOT IN', array('auto-draft', 'trash'))
 			)
 		);
 	}

Indentation +1 added lines, -1 removed lines

@@ -5,7 +5,7 @@
 /**
  *
  * Class EE_Default_Where_Conditions
-  * 
+ * 
  * Strategy specifically for adding where conditions specific to CPT models.
  *
  * @package         Event Espresso

modules/ticket_sales_monitor/EED_Ticket_Sales_Monitor.module.php

Doc Comments +4 added lines, -4 removed lines

@@ -238,7 +238,7 @@ 
 	 * @access    protected
 	 * @param    \EE_Ticket $ticket
 	 * @param int           $quantity
-	 * @return bool
+	 * @return integer
 	 * @throws \EE_Error
 	 */
 	protected function _reserve_ticket( EE_Ticket $ticket, $quantity = 1 ) {
@@ -257,7 +257,7 @@ 
 	 * @access protected
 	 * @param  EE_Ticket $ticket
 	 * @param  int       $quantity
-	 * @return bool
+	 * @return integer
 	 * @throws \EE_Error
 	 */
 	protected function _release_reserved_ticket( EE_Ticket $ticket, $quantity = 1 ) {
@@ -280,7 +280,7 @@ 
 	 *
 	 * @access 	protected
 	 * @param 	\EE_Ticket   $ticket
-	 * @return 	bool
+	 * @return 	boolean|null
 	 * @throws \EE_Error
 	 */
 	protected function _ticket_sold_out( EE_Ticket $ticket ) {
@@ -299,7 +299,7 @@ 
 	 *
 	 * @access    protected
 	 * @param    \EE_Ticket $ticket
-	 * @return bool
+	 * @return boolean|null
 	 */
 	protected function _ticket_quantity_decremented( EE_Ticket $ticket ) {
 		if ( self::debug ) {

Indentation +16 added lines, -16 removed lines

@@ -399,13 +399,13 @@ 
 		if ( self::debug ) {
 			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
 		}
-        $refresh_msg = '';
-        $none_added_msg = '';
+		$refresh_msg = '';
+		$none_added_msg = '';
 		if (defined('DOING_AJAX') && DOING_AJAX) {
-            $refresh_msg = __('Please refresh the page to view updated ticket quantities.',
-                'event_espresso');
-            $none_added_msg = __('No tickets were added for the event.', 'event_espresso');
-        }
+			$refresh_msg = __('Please refresh the page to view updated ticket quantities.',
+				'event_espresso');
+			$none_added_msg = __('No tickets were added for the event.', 'event_espresso');
+		}
 		if ( ! empty( $this->sold_out_tickets ) ) {
 			EE_Error::add_attention(
 				sprintf(
@@ -415,16 +415,16 @@ 
 					),
 					'<br />',
 					implode( '<br />', $this->sold_out_tickets ),
-                    $none_added_msg,
-                    $refresh_msg
+					$none_added_msg,
+					$refresh_msg
 				)
 			);
 			// alter code flow in the Ticket Selector for better UX
 			add_filter( 'FHEE__EED_Ticket_Selector__process_ticket_selections__tckts_slctd', '__return_true' );
 			add_filter( 'FHEE__EED_Ticket_Selector__process_ticket_selections__success', '__return_false' );
 			$this->sold_out_tickets = array();
-            // and reset the cart
-            EED_Ticket_Sales_Monitor::session_cart_reset(EE_Registry::instance()->SSN);
+			// and reset the cart
+			EED_Ticket_Sales_Monitor::session_cart_reset(EE_Registry::instance()->SSN);
 		}
 		if ( ! empty( $this->decremented_tickets ) ) {
 			EE_Error::add_attention(
@@ -435,8 +435,8 @@ 
 					),
 					'<br />',
 					implode( '<br />', $this->decremented_tickets ),
-                    $none_added_msg,
-                    $refresh_msg
+					$none_added_msg,
+					$refresh_msg
 				)
 			);
 			$this->decremented_tickets = array();
@@ -607,11 +607,11 @@ 
 				}
 			}
 		}
-        if (self::debug) {
-            echo "<br /><br /> RESET COMPLETED ";
-        }
+		if (self::debug) {
+			echo "<br /><br /> RESET COMPLETED ";
+		}
 
-    }
+	}
 
 
 

Spacing +161 added lines, -161 removed lines

@@ -1,6 +1,6 @@ 
 <?php
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  * Class EED_Ticket_Sales_Monitor
@@ -17,7 +17,7 @@ 
  */
 class EED_Ticket_Sales_Monitor extends EED_Module {
 
-	const debug = false; 	//	true false
+	const debug = false; //	true false
 
 	/**
 	 * an array of raw ticket data from EED_Ticket_Selector
@@ -59,13 +59,13 @@ 
 	 */
 	public static function set_hooks() {
 		// check ticket reserves AFTER MER does it's check (hence priority 20)
-		add_filter( 'FHEE__EE_Ticket_Selector___add_ticket_to_cart__ticket_qty',
-			array( 'EED_Ticket_Sales_Monitor', 'validate_ticket_sale' ),
+		add_filter('FHEE__EE_Ticket_Selector___add_ticket_to_cart__ticket_qty',
+			array('EED_Ticket_Sales_Monitor', 'validate_ticket_sale'),
 			20, 3
 		);
 		// add notices for sold out tickets
-		add_action( 'AHEE__EE_Ticket_Selector__process_ticket_selections__after_tickets_added_to_cart',
-			array( 'EED_Ticket_Sales_Monitor', 'post_notices' ),
+		add_action('AHEE__EE_Ticket_Selector__process_ticket_selections__after_tickets_added_to_cart',
+			array('EED_Ticket_Sales_Monitor', 'post_notices'),
 			10
 		);
 		// handle ticket quantities adjusted in cart
@@ -77,40 +77,40 @@ 
 		// handle tickets deleted from cart
 		add_action(
 			'FHEE__EED_Multi_Event_Registration__delete_ticket__ticket_removed_from_cart',
-			array( 'EED_Ticket_Sales_Monitor', 'ticket_removed_from_cart' ),
+			array('EED_Ticket_Sales_Monitor', 'ticket_removed_from_cart'),
 			10, 2
 		);
 		// handle emptied carts
 		add_action(
 			'AHEE__EE_Session__reset_cart__before_reset',
-			array( 'EED_Ticket_Sales_Monitor', 'session_cart_reset' ),
+			array('EED_Ticket_Sales_Monitor', 'session_cart_reset'),
 			10, 1
 		);
 		add_action(
 			'AHEE__EED_Multi_Event_Registration__empty_event_cart__before_delete_cart',
-			array( 'EED_Ticket_Sales_Monitor', 'session_cart_reset' ),
+			array('EED_Ticket_Sales_Monitor', 'session_cart_reset'),
 			10, 1
 		);
 		// handle cancelled registrations
 		add_action(
 			'AHEE__EE_Session__reset_checkout__before_reset',
-			array( 'EED_Ticket_Sales_Monitor', 'session_checkout_reset' ),
+			array('EED_Ticket_Sales_Monitor', 'session_checkout_reset'),
 			10, 1
 		);
 		// cron tasks
 		add_action(
 			'AHEE__EE_Cron_Tasks__finalize_abandoned_transactions__abandoned_transaction',
-			array( 'EED_Ticket_Sales_Monitor', 'process_abandoned_transactions' ),
+			array('EED_Ticket_Sales_Monitor', 'process_abandoned_transactions'),
 			10, 1
 		);
 		add_action(
 			'AHEE__EE_Cron_Tasks__process_expired_transactions__incomplete_transaction',
-			array( 'EED_Ticket_Sales_Monitor', 'process_abandoned_transactions' ),
+			array('EED_Ticket_Sales_Monitor', 'process_abandoned_transactions'),
 			10, 1
 		);
 		add_action(
 			'AHEE__EE_Cron_Tasks__process_expired_transactions__failed_transaction',
-			array( 'EED_Ticket_Sales_Monitor', 'process_failed_transactions' ),
+			array('EED_Ticket_Sales_Monitor', 'process_failed_transactions'),
 			10, 1
 		);
 
@@ -134,7 +134,7 @@ 
 	 * @return EED_Ticket_Sales_Monitor
 	 */
 	public static function instance() {
-		return parent::get_instance( __CLASS__ );
+		return parent::get_instance(__CLASS__);
 	}
 
 
@@ -146,7 +146,7 @@ 
 	 * @param WP_Query $WP_Query
 	 * @return    void
 	 */
-	public function run( $WP_Query ) {
+	public function run($WP_Query) {
 	}
 
 
@@ -164,14 +164,14 @@ 
 	 * @param \EE_Ticket $ticket
 	 * @return bool
 	 */
-	public static function validate_ticket_sale( $qty = 1, EE_Ticket $ticket  ) {
-		$qty = absint( $qty );
-		if ( $qty > 0 ) {
-			$qty = EED_Ticket_Sales_Monitor::instance()->_validate_ticket_sale( $ticket, $qty );
+	public static function validate_ticket_sale($qty = 1, EE_Ticket $ticket) {
+		$qty = absint($qty);
+		if ($qty > 0) {
+			$qty = EED_Ticket_Sales_Monitor::instance()->_validate_ticket_sale($ticket, $qty);
 		}
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "()";
-			echo "<br /><br /><b> RETURNED QTY: " . $qty . '</b>';
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."()";
+			echo "<br /><br /><b> RETURNED QTY: ".$qty.'</b>';
 		}
 		return $qty;
 	}
@@ -187,42 +187,42 @@ 
 	 * @param int          $qty
 	 * @return int
 	 */
-	protected function _validate_ticket_sale( EE_Ticket $ticket, $qty = 1 ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
+	protected function _validate_ticket_sale(EE_Ticket $ticket, $qty = 1) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
 		}
-		if ( ! $ticket instanceof EE_Ticket ) {
+		if ( ! $ticket instanceof EE_Ticket) {
 			return 0;
 		}
-		if ( self::debug ) {
-			echo "<br /><b> . ticket->ID: " . $ticket->ID() . '</b>';
-			echo "<br /> . original ticket->reserved: " . $ticket->reserved();
+		if (self::debug) {
+			echo "<br /><b> . ticket->ID: ".$ticket->ID().'</b>';
+			echo "<br /> . original ticket->reserved: ".$ticket->reserved();
 		}
 		$ticket->refresh_from_db();
 		// first let's determine the ticket availability based on sales
-		$available = $ticket->qty( 'saleable' );
-		if ( self::debug ) {
-			echo "<br /> . . . ticket->qty: " . $ticket->qty();
-			echo "<br /> . . . ticket->sold: " . $ticket->sold();
-			echo "<br /> . . . ticket->reserved: " . $ticket->reserved();
-			echo "<br /> . . . ticket->qty(saleable): " . $ticket->qty( 'saleable' );
-			echo "<br /> . . . available: " . $available;
-		}
-		if ( $available < 1 ) {
-			$this->_ticket_sold_out( $ticket );
+		$available = $ticket->qty('saleable');
+		if (self::debug) {
+			echo "<br /> . . . ticket->qty: ".$ticket->qty();
+			echo "<br /> . . . ticket->sold: ".$ticket->sold();
+			echo "<br /> . . . ticket->reserved: ".$ticket->reserved();
+			echo "<br /> . . . ticket->qty(saleable): ".$ticket->qty('saleable');
+			echo "<br /> . . . available: ".$available;
+		}
+		if ($available < 1) {
+			$this->_ticket_sold_out($ticket);
 			return 0;
 		}
-		if ( self::debug ) {
-			echo "<br /> . . . qty: " . $qty;
+		if (self::debug) {
+			echo "<br /> . . . qty: ".$qty;
 		}
-		if ( $available < $qty ) {
+		if ($available < $qty) {
 			$qty = $available;
-			if ( self::debug ) {
-				echo "<br /> . . . QTY ADJUSTED: " . $qty;
+			if (self::debug) {
+				echo "<br /> . . . QTY ADJUSTED: ".$qty;
 			}
-			$this->_ticket_quantity_decremented( $ticket );
+			$this->_ticket_quantity_decremented($ticket);
 		}
-		$this->_reserve_ticket( $ticket, $qty );
+		$this->_reserve_ticket($ticket, $qty);
 		return $qty;
 	}
 
@@ -238,11 +238,11 @@ 
 	 * @return bool
 	 * @throws \EE_Error
 	 */
-	protected function _reserve_ticket( EE_Ticket $ticket, $quantity = 1 ) {
-		if ( self::debug ) {
-			echo "<br /><br /> . . . INCREASE RESERVED: " . $quantity;
+	protected function _reserve_ticket(EE_Ticket $ticket, $quantity = 1) {
+		if (self::debug) {
+			echo "<br /><br /> . . . INCREASE RESERVED: ".$quantity;
 		}
-		$ticket->increase_reserved( $quantity );
+		$ticket->increase_reserved($quantity);
 		return $ticket->save();
 	}
 
@@ -257,14 +257,14 @@ 
 	 * @return bool
 	 * @throws \EE_Error
 	 */
-	protected function _release_reserved_ticket( EE_Ticket $ticket, $quantity = 1 ) {
-		if ( self::debug ) {
-			echo "<br /> . . . ticket->ID: " . $ticket->ID();
-			echo "<br /> . . . ticket->reserved: " . $ticket->reserved();
+	protected function _release_reserved_ticket(EE_Ticket $ticket, $quantity = 1) {
+		if (self::debug) {
+			echo "<br /> . . . ticket->ID: ".$ticket->ID();
+			echo "<br /> . . . ticket->reserved: ".$ticket->reserved();
 		}
-		$ticket->decrease_reserved( $quantity );
-		if ( self::debug ) {
-			echo "<br /> . . . ticket->reserved: " . $ticket->reserved();
+		$ticket->decrease_reserved($quantity);
+		if (self::debug) {
+			echo "<br /> . . . ticket->reserved: ".$ticket->reserved();
 		}
 		return $ticket->save() ? 1 : 0;
 	}
@@ -280,12 +280,12 @@ 
 	 * @return 	bool
 	 * @throws \EE_Error
 	 */
-	protected function _ticket_sold_out( EE_Ticket $ticket ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
-			echo "<br /> . . ticket->name: " . $this->_get_ticket_and_event_name( $ticket );
+	protected function _ticket_sold_out(EE_Ticket $ticket) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
+			echo "<br /> . . ticket->name: ".$this->_get_ticket_and_event_name($ticket);
 		}
-		$this->sold_out_tickets[] = $this->_get_ticket_and_event_name( $ticket );
+		$this->sold_out_tickets[] = $this->_get_ticket_and_event_name($ticket);
 	}
 
 
@@ -298,12 +298,12 @@ 
 	 * @param    \EE_Ticket $ticket
 	 * @return bool
 	 */
-	protected function _ticket_quantity_decremented( EE_Ticket $ticket ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
-			echo "<br /> . . ticket->name: " . $this->_get_ticket_and_event_name( $ticket );
+	protected function _ticket_quantity_decremented(EE_Ticket $ticket) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
+			echo "<br /> . . ticket->name: ".$this->_get_ticket_and_event_name($ticket);
 		}
-		$this->decremented_tickets[] = $this->_get_ticket_and_event_name( $ticket );
+		$this->decremented_tickets[] = $this->_get_ticket_and_event_name($ticket);
 	}
 
 
@@ -316,11 +316,11 @@ 
 	 * @param    \EE_Ticket $ticket
 	 * @return string
 	 */
-	protected function _get_ticket_and_event_name( EE_Ticket $ticket ) {
+	protected function _get_ticket_and_event_name(EE_Ticket $ticket) {
 		$event = $ticket->get_related_event();
-		if ( $event instanceof EE_Event ) {
+		if ($event instanceof EE_Event) {
 			$ticket_name = sprintf(
-				_x( '%1$s for %2$s', 'ticket name for event name', 'event_espresso' ),
+				_x('%1$s for %2$s', 'ticket name for event name', 'event_espresso'),
 				$ticket->name(),
 				$event->name()
 			);
@@ -345,13 +345,13 @@ 
 	 * @param  int          $quantity
 	 * @return void
 	 */
-	public static function ticket_quantity_updated( EE_Line_Item $line_item, $quantity = 1 ) {
-		$ticket = EEM_Ticket::instance()->get_one_by_ID( absint( $line_item->OBJ_ID() ) );
-		if ( $ticket instanceof EE_Ticket ) {
-			if ( $quantity > 0 ) {
-				EED_Ticket_Sales_Monitor::instance()->_reserve_ticket( $ticket, $quantity );
+	public static function ticket_quantity_updated(EE_Line_Item $line_item, $quantity = 1) {
+		$ticket = EEM_Ticket::instance()->get_one_by_ID(absint($line_item->OBJ_ID()));
+		if ($ticket instanceof EE_Ticket) {
+			if ($quantity > 0) {
+				EED_Ticket_Sales_Monitor::instance()->_reserve_ticket($ticket, $quantity);
 			} else {
-				EED_Ticket_Sales_Monitor::instance()->_release_reserved_ticket( $ticket, $quantity );
+				EED_Ticket_Sales_Monitor::instance()->_release_reserved_ticket($ticket, $quantity);
 			}
 		}
 	}
@@ -367,8 +367,8 @@ 
 	 * @param  int       $quantity
 	 * @return void
 	 */
-	public static function ticket_removed_from_cart( EE_Ticket $ticket, $quantity = 1 ) {
-		EED_Ticket_Sales_Monitor::instance()->_release_reserved_ticket( $ticket, $quantity );
+	public static function ticket_removed_from_cart(EE_Ticket $ticket, $quantity = 1) {
+		EED_Ticket_Sales_Monitor::instance()->_release_reserved_ticket($ticket, $quantity);
 	}
 
 
@@ -396,8 +396,8 @@ 
 	 * @return    void
 	 */
 	protected function _post_notices() {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
 		}
         $refresh_msg = '';
         $none_added_msg = '';
@@ -406,35 +406,35 @@ 
                 'event_espresso');
             $none_added_msg = __('No tickets were added for the event.', 'event_espresso');
         }
-		if ( ! empty( $this->sold_out_tickets ) ) {
+		if ( ! empty($this->sold_out_tickets)) {
 			EE_Error::add_attention(
 				sprintf(
 					apply_filters(
 						'FHEE__EED_Ticket_Sales_Monitor___post_notices__sold_out_tickets_notice',
-						__( 'We\'re sorry...%1$sThe following items have sold out since you first viewed this page, and can no longer be registered for:%1$s%1$s%2$s%1$s%1$sPlease note that availability can change at any time due to cancellations, so please check back again later if registration for this event(s) is important to you.%1$s%1$s%3$s%1$s%4$s%1$s', 'event_espresso' )
+						__('We\'re sorry...%1$sThe following items have sold out since you first viewed this page, and can no longer be registered for:%1$s%1$s%2$s%1$s%1$sPlease note that availability can change at any time due to cancellations, so please check back again later if registration for this event(s) is important to you.%1$s%1$s%3$s%1$s%4$s%1$s', 'event_espresso')
 					),
 					'<br />',
-					implode( '<br />', $this->sold_out_tickets ),
+					implode('<br />', $this->sold_out_tickets),
                     $none_added_msg,
                     $refresh_msg
 				)
 			);
 			// alter code flow in the Ticket Selector for better UX
-			add_filter( 'FHEE__EED_Ticket_Selector__process_ticket_selections__tckts_slctd', '__return_true' );
-			add_filter( 'FHEE__EED_Ticket_Selector__process_ticket_selections__success', '__return_false' );
+			add_filter('FHEE__EED_Ticket_Selector__process_ticket_selections__tckts_slctd', '__return_true');
+			add_filter('FHEE__EED_Ticket_Selector__process_ticket_selections__success', '__return_false');
 			$this->sold_out_tickets = array();
             // and reset the cart
             EED_Ticket_Sales_Monitor::session_cart_reset(EE_Registry::instance()->SSN);
 		}
-		if ( ! empty( $this->decremented_tickets ) ) {
+		if ( ! empty($this->decremented_tickets)) {
 			EE_Error::add_attention(
 				sprintf(
 					apply_filters(
 						'FHEE__EED_Ticket_Sales_Monitor___ticket_quantity_decremented__notice',
-						__( 'We\'re sorry...%1$sDue to sales that have occurred since you first viewed the last page, the following items have had their quantities adjusted to match the current available amount:%1$s%1$s%2$s%1$s%1$sPlease note that availability can change at any time due to cancellations, so please check back again later if registration for this event(s) is important to you.%1$s%1$s%3$s%1$s%4$s%1$s', 'event_espresso' )
+						__('We\'re sorry...%1$sDue to sales that have occurred since you first viewed the last page, the following items have had their quantities adjusted to match the current available amount:%1$s%1$s%2$s%1$s%1$sPlease note that availability can change at any time due to cancellations, so please check back again later if registration for this event(s) is important to you.%1$s%1$s%3$s%1$s%4$s%1$s', 'event_espresso')
 					),
 					'<br />',
-					implode( '<br />', $this->decremented_tickets ),
+					implode('<br />', $this->decremented_tickets),
                     $none_added_msg,
                     $refresh_msg
 				)
@@ -458,27 +458,27 @@ 
 	 * @param 	EE_Transaction 	$transaction
 	 * @return int
 	 */
-	protected function _release_all_reserved_tickets_for_transaction( EE_Transaction $transaction ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
-			echo "<br /> . transaction->ID: " . $transaction->ID();
+	protected function _release_all_reserved_tickets_for_transaction(EE_Transaction $transaction) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
+			echo "<br /> . transaction->ID: ".$transaction->ID();
 		}
 		/** @type EE_Transaction_Processor $transaction_processor */
-		$transaction_processor = EE_Registry::instance()->load_class( 'Transaction_Processor' );
+		$transaction_processor = EE_Registry::instance()->load_class('Transaction_Processor');
 		// check if 'finalize_registration' step has been completed...
-		$finalized = $transaction_processor->reg_step_completed( $transaction, 'finalize_registration' );
-		if ( self::debug ) {
+		$finalized = $transaction_processor->reg_step_completed($transaction, 'finalize_registration');
+		if (self::debug) {
 			// DEBUG LOG
 			EEH_Debug_Tools::log(
 				__CLASS__, __FUNCTION__, __LINE__,
-				array( 'finalized' => $finalized ),
-				false, 'EE_Transaction: ' . $transaction->ID()
+				array('finalized' => $finalized),
+				false, 'EE_Transaction: '.$transaction->ID()
 			);
 		}
 		// how many tickets were released
 		$count = 0;
-		if ( self::debug ) {
-			echo "<br /> . . . finalized: " . $finalized;
+		if (self::debug) {
+			echo "<br /> . . . finalized: ".$finalized;
 		}
 		$release_tickets_with_TXN_status = array(
 			EEM_Transaction::failed_status_code,
@@ -486,13 +486,13 @@ 
 			EEM_Transaction::incomplete_status_code,
 		);
 		// if the session is getting cleared BEFORE the TXN has been finalized
-		if ( ! $finalized || in_array( $transaction->status_ID(), $release_tickets_with_TXN_status ) ) {
+		if ( ! $finalized || in_array($transaction->status_ID(), $release_tickets_with_TXN_status)) {
 			// let's cancel any reserved tickets
 			$registrations = $transaction->registrations();
-			if ( ! empty( $registrations ) ) {
-				foreach ( $registrations as $registration ) {
-					if ( $registration instanceof EE_Registration ) {
-						$count += $this->_release_reserved_ticket_for_registration( $registration, $transaction );
+			if ( ! empty($registrations)) {
+				foreach ($registrations as $registration) {
+					if ($registration instanceof EE_Registration) {
+						$count += $this->_release_reserved_ticket_for_registration($registration, $transaction);
 					}
 				}
 			}
@@ -513,12 +513,12 @@ 
 	 * @return 	int
 	 * @throws 	\EE_Error
 	 */
-	protected function _release_reserved_ticket_for_registration( EE_Registration $registration, EE_Transaction $transaction ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
-			echo "<br /> . . registration->ID: " . $registration->ID();
-			echo "<br /> . . registration->status_ID: " . $registration->status_ID();
-			echo "<br /> . . transaction->status_ID(): " . $transaction->status_ID();
+	protected function _release_reserved_ticket_for_registration(EE_Registration $registration, EE_Transaction $transaction) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
+			echo "<br /> . . registration->ID: ".$registration->ID();
+			echo "<br /> . . registration->status_ID: ".$registration->status_ID();
+			echo "<br /> . . transaction->status_ID(): ".$transaction->status_ID();
 		}
 		if (
 			// release Tickets for Failed Transactions and Abandoned Transactions
@@ -531,8 +531,8 @@ 
 			)
 		) {
 			$ticket = $registration->ticket();
-			if ( $ticket instanceof EE_Ticket ) {
-				return $this->_release_reserved_ticket( $ticket );
+			if ($ticket instanceof EE_Ticket) {
+				return $this->_release_reserved_ticket($ticket);
 			}
 		}
 		return 0;
@@ -552,20 +552,20 @@ 
 	 * @param    EE_Session $session
 	 * @return    void
 	 */
-	public static function session_cart_reset( EE_Session $session ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
+	public static function session_cart_reset(EE_Session $session) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
 		}
 		$cart = $session->cart();
-		if ( $cart instanceof EE_Cart ) {
-			if ( self::debug ) {
+		if ($cart instanceof EE_Cart) {
+			if (self::debug) {
 				echo "<br /><br /> cart instanceof EE_Cart: ";
 			}
-			EED_Ticket_Sales_Monitor::instance()->_session_cart_reset( $cart );
+			EED_Ticket_Sales_Monitor::instance()->_session_cart_reset($cart);
 		} else {
-			if ( self::debug ) {
+			if (self::debug) {
 				echo "<br /><br /> invalid EE_Cart: ";
-				var_dump( $cart );
+				var_dump($cart);
 			}
 		}
 	}
@@ -580,30 +580,30 @@ 
 	 * @param    EE_Cart $cart
 	 * @return    void
 	 */
-	protected function _session_cart_reset( EE_Cart $cart ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
+	protected function _session_cart_reset(EE_Cart $cart) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
 		}
-		EE_Registry::instance()->load_helper( 'Line_Item' );
+		EE_Registry::instance()->load_helper('Line_Item');
 		$ticket_line_items = $cart->get_tickets();
-		if ( empty( $ticket_line_items ) ) {
+		if (empty($ticket_line_items)) {
 			return;
 		}
-		foreach ( $ticket_line_items as $ticket_line_item ) {
-			if ( self::debug ) {
-				echo "<br /> . ticket_line_item->ID(): " . $ticket_line_item->ID();
+		foreach ($ticket_line_items as $ticket_line_item) {
+			if (self::debug) {
+				echo "<br /> . ticket_line_item->ID(): ".$ticket_line_item->ID();
 			}
-			if ( $ticket_line_item instanceof EE_Line_Item && $ticket_line_item->OBJ_type() == 'Ticket' ) {
-				if ( self::debug ) {
-					echo "<br /> . . ticket_line_item->OBJ_ID(): " . $ticket_line_item->OBJ_ID();
+			if ($ticket_line_item instanceof EE_Line_Item && $ticket_line_item->OBJ_type() == 'Ticket') {
+				if (self::debug) {
+					echo "<br /> . . ticket_line_item->OBJ_ID(): ".$ticket_line_item->OBJ_ID();
 				}
-				$ticket = EEM_Ticket::instance()->get_one_by_ID( $ticket_line_item->OBJ_ID() );
-				if ( $ticket instanceof EE_Ticket ) {
-					if ( self::debug ) {
-						echo "<br /> . . ticket->ID(): " . $ticket->ID();
-						echo "<br /> . . ticket_line_item->quantity(): " . $ticket_line_item->quantity();
+				$ticket = EEM_Ticket::instance()->get_one_by_ID($ticket_line_item->OBJ_ID());
+				if ($ticket instanceof EE_Ticket) {
+					if (self::debug) {
+						echo "<br /> . . ticket->ID(): ".$ticket->ID();
+						echo "<br /> . . ticket_line_item->quantity(): ".$ticket_line_item->quantity();
 					}
-					$this->_release_reserved_ticket( $ticket, $ticket_line_item->quantity() );
+					$this->_release_reserved_ticket($ticket, $ticket_line_item->quantity());
 				}
 			}
 		}
@@ -627,10 +627,10 @@ 
 	 * @param    EE_Session $session
 	 * @return    void
 	 */
-	public static function session_checkout_reset( EE_Session $session ) {
+	public static function session_checkout_reset(EE_Session $session) {
 		$checkout = $session->checkout();
-		if ( $checkout instanceof EE_Checkout ) {
-			EED_Ticket_Sales_Monitor::instance()->_session_checkout_reset( $checkout );
+		if ($checkout instanceof EE_Checkout) {
+			EED_Ticket_Sales_Monitor::instance()->_session_checkout_reset($checkout);
 		}
 	}
 
@@ -644,15 +644,15 @@ 
 	 * @param    EE_Checkout $checkout
 	 * @return    void
 	 */
-	protected function _session_checkout_reset( EE_Checkout $checkout ) {
-		if ( self::debug ) {
-			echo "<br /><br /> " . __LINE__ . ") " . __METHOD__ . "() ";
+	protected function _session_checkout_reset(EE_Checkout $checkout) {
+		if (self::debug) {
+			echo "<br /><br /> ".__LINE__.") ".__METHOD__."() ";
 		}
 		// we want to release the each registration's reserved tickets if the session was cleared, but not if this is a revisit
-		if ( $checkout->revisit || ! $checkout->transaction instanceof EE_Transaction ) {
+		if ($checkout->revisit || ! $checkout->transaction instanceof EE_Transaction) {
 			return;
 		}
-		$this->_release_all_reserved_tickets_for_transaction( $checkout->transaction );
+		$this->_release_all_reserved_tickets_for_transaction($checkout->transaction);
 	}
 
 
@@ -668,7 +668,7 @@ 
 	 * @param    EE_Session $session
 	 * @return    void
 	 */
-	public static function session_expired_reset( EE_Session $session ) {
+	public static function session_expired_reset(EE_Session $session) {
 
 	}
 
@@ -687,30 +687,30 @@ 
 	 * @param    EE_Transaction $transaction
 	 * @return    void
 	 */
-	public static function process_abandoned_transactions( EE_Transaction $transaction ) {
+	public static function process_abandoned_transactions(EE_Transaction $transaction) {
 		// is this TXN free or has any money been paid towards this TXN? If so, then leave it alone
-		if ( $transaction->is_free() || $transaction->paid() > 0 ) {
-			if ( self::debug ) {
+		if ($transaction->is_free() || $transaction->paid() > 0) {
+			if (self::debug) {
 				// DEBUG LOG
 				EEH_Debug_Tools::log(
 					__CLASS__, __FUNCTION__, __LINE__,
-					array( $transaction ),
-					false, 'EE_Transaction: ' . $transaction->ID()
+					array($transaction),
+					false, 'EE_Transaction: '.$transaction->ID()
 				);
 			}
 			return;
 		}
 		// have their been any successful payments made ?
 		$payments = $transaction->payments();
-		foreach ( $payments as $payment ) {
-			if ( $payment instanceof EE_Payment ) {
-				if ( $payment->status() === EEM_Payment::status_id_approved ) {
-					if ( self::debug ) {
+		foreach ($payments as $payment) {
+			if ($payment instanceof EE_Payment) {
+				if ($payment->status() === EEM_Payment::status_id_approved) {
+					if (self::debug) {
 						// DEBUG LOG
 						EEH_Debug_Tools::log(
 							__CLASS__, __FUNCTION__, __LINE__,
-							array( $payment ),
-							false, 'EE_Transaction: ' . $transaction->ID()
+							array($payment),
+							false, 'EE_Transaction: '.$transaction->ID()
 						);
 					}
 					return;
@@ -718,7 +718,7 @@ 
 			}
 		}
 		// since you haven't even attempted to pay for your ticket...
-		EED_Ticket_Sales_Monitor::instance()->_release_all_reserved_tickets_for_transaction( $transaction );
+		EED_Ticket_Sales_Monitor::instance()->_release_all_reserved_tickets_for_transaction($transaction);
 	}
 
 
@@ -735,9 +735,9 @@ 
 	 * @param    EE_Transaction $transaction
 	 * @return    void
 	 */
-	public static function process_failed_transactions( EE_Transaction $transaction ) {
+	public static function process_failed_transactions(EE_Transaction $transaction) {
 		// since you haven't even attempted to pay for your ticket...
-		EED_Ticket_Sales_Monitor::instance()->_release_all_reserved_tickets_for_transaction( $transaction );
+		EED_Ticket_Sales_Monitor::instance()->_release_all_reserved_tickets_for_transaction($transaction);
 	}
 
 

caffeinated/modules/recaptcha/ReCaptcha/RequestMethod.php

Indentation +7 added lines, -7 removed lines

@@ -32,11 +32,11 @@
 interface RequestMethod
 {
 
-    /**
-     * Submit the request with the specified parameters.
-     *
-     * @param RequestParameters $params Request parameters
-     * @return string Body of the reCAPTCHA response
-     */
-    public function submit(RequestParameters $params);
+	/**
+	 * Submit the request with the specified parameters.
+	 *
+	 * @param RequestParameters $params Request parameters
+	 * @return string Body of the reCAPTCHA response
+	 */
+	public function submit(RequestParameters $params);
 }

caffeinated/admin/extend/messages/espresso_events_Messages_Hooks_Extend.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 	 *
 	 * @param  EE_Event $event 	EE event object
 	 * @param  array    $data   The request data from the form
-	 * @return bool     		success or fail
+	 * @return integer     		success or fail
 	 */
 	public function attach_evt_message_templates( $event, $data ) {
 		//first we remove all existing relations on the Event for message types.

Spacing +41 added lines, -41 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**
@@ -22,21 +22,21 @@ 
 	 *
 	 * @param \EE_Admin_Page $admin_page
 	 */
-	public function __construct( EE_Admin_Page $admin_page ) {
+	public function __construct(EE_Admin_Page $admin_page) {
 		/**
 		 * Add cap restriction ... metaboxes should not show if user does not have the ability to edit_custom_messages
 		 */
 		if (
-			! EE_Registry::instance()->CAP->current_user_can( 'ee_edit_messages', 'messages_events_editor_metabox' )
+			! EE_Registry::instance()->CAP->current_user_can('ee_edit_messages', 'messages_events_editor_metabox')
 		) {
 			return;
 		}
 		add_filter(
 			'FHEE__Events_Admin_Page___insert_update_cpt_item__event_update_callbacks',
-			array( $this, 'caf_updates' ),
+			array($this, 'caf_updates'),
 			10
 		);
-		parent::__construct( $admin_page );
+		parent::__construct($admin_page);
 	}
 
 
@@ -49,13 +49,13 @@ 
 	 */
 	protected function _extend_properties() {
 
-		define( 'EE_MSGS_EXTEND_ASSETS_URL', EE_CORE_CAF_ADMIN_EXTEND_URL . 'messages/assets/' );
+		define('EE_MSGS_EXTEND_ASSETS_URL', EE_CORE_CAF_ADMIN_EXTEND_URL.'messages/assets/');
 		$this->_ajax_func = array(
 			'ee_msgs_create_new_custom' => 'create_new_custom'
 			);
 		$this->_metaboxes = array(
 			0 => array(
-				'page_route' => array('edit','create_new'),
+				'page_route' => array('edit', 'create_new'),
 				'func' => 'messages_metabox',
 				'label' => __('Notifications', 'event_espresso'),
 				'priority' => 'high'
@@ -66,11 +66,11 @@ 
 		$this->_scripts_styles = array(
 			'registers' => array(
 				'events_msg_admin' => array(
-					'url' => EE_MSGS_EXTEND_ASSETS_URL . 'events_messages_admin.js',
+					'url' => EE_MSGS_EXTEND_ASSETS_URL.'events_messages_admin.js',
 					'depends' => array('ee-dialog', 'ee-parse-uri', 'ee-serialize-full-array')
 				),
 				'events_msg_admin_css' => array(
-					'url' => EE_MSGS_EXTEND_ASSETS_URL . 'ee_msg_events_admin.css',
+					'url' => EE_MSGS_EXTEND_ASSETS_URL.'ee_msg_events_admin.css',
 					'type' => 'css'
 				)
 			),
@@ -82,8 +82,8 @@ 
 	}
 
 
-	public function caf_updates( $update_callbacks ) {
-		$update_callbacks[] = array( $this, 'attach_evt_message_templates' );
+	public function caf_updates($update_callbacks) {
+		$update_callbacks[] = array($this, 'attach_evt_message_templates');
 		return $update_callbacks;
 	}
 
@@ -97,13 +97,13 @@ 
 	 * @param  array    $data   The request data from the form
 	 * @return bool     		success or fail
 	 */
-	public function attach_evt_message_templates( $event, $data ) {
+	public function attach_evt_message_templates($event, $data) {
 		//first we remove all existing relations on the Event for message types.
 		$event->_remove_relations('Message_Template_Group');
 		//now let's just loop through the selected templates and add relations!
-		if ( isset( $data[ 'event_message_templates_relation' ] ) ) {
-			foreach ( $data[ 'event_message_templates_relation' ] as $grp_ID ) {
-				$event->_add_relation_to( $grp_ID, 'Message_Template_Group' );
+		if (isset($data['event_message_templates_relation'])) {
+			foreach ($data['event_message_templates_relation'] as $grp_ID) {
+				$event->_add_relation_to($grp_ID, 'Message_Template_Group');
 			}
 		}
 		//now save
@@ -118,25 +118,25 @@ 
 	 * @return string
 	 * @throws \EE_Error
 	 */
-	public function messages_metabox( $event, $callback_args ) {
+	public function messages_metabox($event, $callback_args) {
 		//let's get the active messengers (b/c messenger objects have the active message templates)
 		//convert 'evt_id' to 'EVT_ID'
-		$this->_req_data['EVT_ID'] = isset( $this->_req_data['EVT_ID'] ) ? $this->_req_data['EVT_ID'] : NULL;
-		$this->_req_data['EVT_ID'] = isset( $this->_req_data['post'] ) && empty( $this->_req_data['EVT_ID'] )
+		$this->_req_data['EVT_ID'] = isset($this->_req_data['EVT_ID']) ? $this->_req_data['EVT_ID'] : NULL;
+		$this->_req_data['EVT_ID'] = isset($this->_req_data['post']) && empty($this->_req_data['EVT_ID'])
 			? $this->_req_data['post']
 			: $this->_req_data['EVT_ID'];
 
-		$this->_req_data['EVT_ID'] = empty($this->_req_data['EVT_ID'] ) && isset($this->_req_data['evt_id'] )
+		$this->_req_data['EVT_ID'] = empty($this->_req_data['EVT_ID']) && isset($this->_req_data['evt_id'])
 			? $this->_req_data['evt_id']
 			: $this->_req_data['EVT_ID'];
 		/** @type EE_Message_Resource_Manager $message_resource_manager */
-		$message_resource_manager = EE_Registry::instance()->load_lib( 'Message_Resource_Manager' );
+		$message_resource_manager = EE_Registry::instance()->load_lib('Message_Resource_Manager');
 		$active_messengers = $message_resource_manager->active_messengers();
 		$tabs = array();
 
 		//empty messengers?
 		//Note message types will always have at least one available because every messenger has a default message type associated with it (payment) if no other message types are selected.
-		if ( empty( $active_messengers ) ) {
+		if (empty($active_messengers)) {
 			$msg_activate_url = EE_Admin_Page::add_query_args_and_nonce(
 				array('action' => 'settings'),
 				EE_MSG_ADMIN_URL
@@ -145,11 +145,11 @@ 
 				__('There are no active messengers. So no notifications will go out for %1$sany%2$s events.  You will want to %3$sActivate a Messenger%4$s.', 'event_espresso'),
 				'<strong>',
 				'</strong>',
-				'<a href="' . $msg_activate_url . '">',
+				'<a href="'.$msg_activate_url.'">',
 				'</a>'
 			);
-			$error_content = '<div class="error"><p>' . $error_msg . '</p></div>';
-			$internal_content = '<div id="messages-error"><p>' . $error_msg . '</p></div>';
+			$error_content = '<div class="error"><p>'.$error_msg.'</p></div>';
+			$internal_content = '<div id="messages-error"><p>'.$error_msg.'</p></div>';
 
 			echo $error_content;
 			echo $internal_content;
@@ -158,20 +158,20 @@ 
 
 		$event_id = isset($this->_req_data['EVT_ID']) ? $this->_req_data['EVT_ID'] : NULL;
 		//get content for active messengers
-		foreach ( $active_messengers as $name => $messenger ) {
+		foreach ($active_messengers as $name => $messenger) {
 			//first check if there are any active message types for this messenger.
-			$active_mts = $message_resource_manager->get_active_message_types_for_messenger( $name );
-			if ( empty( $active_mts ) ) {
+			$active_mts = $message_resource_manager->get_active_message_types_for_messenger($name);
+			if (empty($active_mts)) {
 				continue;
 			}
 
 			$tab_content = $messenger->get_messenger_admin_page_content(
 				'events',
 				'edit',
-				array( 'event' => $event_id )
+				array('event' => $event_id)
 			);
 
-			if ( ! empty( $tab_content ) ) {
+			if ( ! empty($tab_content)) {
 				$tabs[$name] = $tab_content;
 			}
 		}
@@ -179,23 +179,23 @@ 
 
 		//we want this to be tabbed content so let's use the EEH_Tabbed_Content::display helper.
 		$tabbed_content = EEH_Tabbed_Content::display($tabs);
-		if ( $tabbed_content instanceof WP_Error ) {
+		if ($tabbed_content instanceof WP_Error) {
 			$tabbed_content = $tabbed_content->get_error_message();
 		}
 
 		$notices = '
 	<div id="espresso-ajax-loading" class="ajax-loader-grey">
-		<span class="ee-spinner ee-spin"></span><span class="hidden">' . __('loading...', 'event_espresso') . '</span>
+		<span class="ee-spinner ee-spin"></span><span class="hidden">' . __('loading...', 'event_espresso').'</span>
 	</div>
 	<div class="ee-notices"></div>';
 
-		if ( defined('DOING_AJAX' ) ) {
+		if (defined('DOING_AJAX')) {
 			return $tabbed_content;
 		}
 
-		do_action( 'AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__before_content' );
-		echo $notices . '<div class="messages-tabs-content">' . $tabbed_content . '</div>';
-		do_action( 'AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__after_content' );
+		do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__before_content');
+		echo $notices.'<div class="messages-tabs-content">'.$tabbed_content.'</div>';
+		do_action('AHEE__espresso_events_Messages_Hooks_Extend__messages_metabox__after_content');
 
 	}
 
@@ -210,15 +210,15 @@ 
 	 */
 	public function create_new_custom() {
 
-		if ( ! EE_Registry::instance()->CAP->current_user_can( 'ee_edit_messages', 'create_new_custom_ajax' ) ) {
-			wp_die( __('You don\'t have privileges to do this action', 'event_espresso' ) );
+		if ( ! EE_Registry::instance()->CAP->current_user_can('ee_edit_messages', 'create_new_custom_ajax')) {
+			wp_die(__('You don\'t have privileges to do this action', 'event_espresso'));
 		}
 
 		//let's clean up the _POST global a bit for downstream usage of name and description.
-		$_POST['templateName'] = !empty( $this->_req_data['custom_template_args']['MTP_name'] )
+		$_POST['templateName'] = ! empty($this->_req_data['custom_template_args']['MTP_name'])
 			? $this->_req_data['custom_template_args']['MTP_name']
 			: '';
-		$_POST['templateDescription'] = !empty( $this->_req_data['custom_template_args']['MTP_description'] )
+		$_POST['templateDescription'] = ! empty($this->_req_data['custom_template_args']['MTP_description'])
 			? $this->_req_data['custom_template_args']['MTP_description']
 			: '';
 
@@ -227,7 +227,7 @@ 
 		$this->_set_page_object();
 
 		// is this a template switch if so EE_Admin_Page child needs this object
-		$this->_page_object->set_hook_object( $this );
+		$this->_page_object->set_hook_object($this);
 
 		$this->_page_object->add_message_template(
 			$this->_req_data['messageType'],
@@ -251,7 +251,7 @@ 
 	 */
 	public function edit_admin_footer() {
 		EEH_Template::display_template(
-			EE_CORE_CAF_ADMIN_EXTEND . 'messages/templates/create_custom_template_form.template.php'
+			EE_CORE_CAF_ADMIN_EXTEND.'messages/templates/create_custom_template_form.template.php'
 		);
 	}
 

Braces +2 added lines, -1 removed lines

@@ -1,6 +1,7 @@
 <?php
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  * espresso_events_Messages_Hooks_Extend

core/db_models/EEM_Message_Template_Group.model.php

Doc Comments +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
 	 * @param        $messenger
 	 * @param string $orderby
 	 * @param string $order
-	 * @return array all (including trashed or inactive) message template group objects for the given messenger
+	 * @return EE_Base_Class[] all (including trashed or inactive) message template group objects for the given messenger
 	 */
 	public function get_all_message_templates_by_messenger($messenger, $orderby = 'GRP_ID', $order = 'ASC' ) {
 		return $this->get_all_deleted_and_undeleted(
@@ -343,7 +343,7 @@ 
 	/**
 	 * This sends things to the validator for the given messenger and message type.
 	 *
-	 * @param  array $fields the incoming fields to check.
+	 * @param  string $fields the incoming fields to check.
 	 *                       Note this array is in the formatted fields from the form fields setup.
 	 *                       So we need to reformat this into an array of expected field refs by the validator.
 	 *                       Note also that this is not only the fields for the Message Template Group

Indentation +8 added lines, -8 removed lines

@@ -1,14 +1,14 @@
 <?php if (!defined('EVENT_ESPRESSO_VERSION') )
 	exit('NO direct script access allowed');
 /**
- * EEM_Message_Template_Group
- *
- * @package		Event Espresso
- * @subpackage	includes/models/EEM_Message_Template_Group.model.php
- * @author		Darren Ethier
- *
- *
- */
+	 * EEM_Message_Template_Group
+	 *
+	 * @package		Event Espresso
+	 * @subpackage	includes/models/EEM_Message_Template_Group.model.php
+	 * @author		Darren Ethier
+	 *
+	 *
+	 */
 require_once ( EE_MODELS . 'EEM_Soft_Delete_Base.model.php' );
 class EEM_Message_Template_Group extends EEM_Soft_Delete_Base {
 

Spacing +61 added lines, -61 removed lines

@@ -1,4 +1,4 @@ 
-<?php if (!defined('EVENT_ESPRESSO_VERSION') )
+<?php if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 /**
  * EEM_Message_Template_Group
@@ -9,7 +9,7 @@ 
  *
  *
  */
-require_once ( EE_MODELS . 'EEM_Soft_Delete_Base.model.php' );
+require_once (EE_MODELS.'EEM_Soft_Delete_Base.model.php');
 class EEM_Message_Template_Group extends EEM_Soft_Delete_Base {
 
 	// private instance of the EEM_Message_Template_Group object
@@ -17,7 +17,7 @@ 
 
 
 
-	protected function __construct( $timezone = NULL ) {
+	protected function __construct($timezone = NULL) {
 		$this->singular_item = __('Message Template Group', 'event_espresso');
 		$this->plural_item = __('Message Template Groups', 'event_espresso');
 		$this->_tables = array(
@@ -26,11 +26,11 @@ 
 		$this->_fields = array(
 			'Message_Template_Group' => array(
 				'GRP_ID' => new EE_Primary_Key_Int_Field('GRP_ID', __('Message Template Group ID', 'event_espresso')),
-				'MTP_name' => new EE_Plain_Text_Field( 'MTP_name', __('The name of the template group', 'event_espresso'), FALSE, '' ),
-				'MTP_description' => new EE_Simple_HTML_Field( 'MTP_description', __('A brief description about this template.', 'event_espresso' ), FALSE, '' ),
-				'MTP_user_id'=> new EE_WP_User_Field('MTP_user_id', __('Template Creator ID', 'event_espresso'), FALSE, get_current_user_id() ),
-				'MTP_messenger'=>new EE_Plain_Text_Field('MTP_messenger', __('Messenger Used for Template', 'event_espresso'), FALSE, 'email' ),
-				'MTP_message_type'=>new EE_Plain_Text_Field('MTP_message_type', __('Message Type', 'event_espresso'),false,'registration'),
+				'MTP_name' => new EE_Plain_Text_Field('MTP_name', __('The name of the template group', 'event_espresso'), FALSE, ''),
+				'MTP_description' => new EE_Simple_HTML_Field('MTP_description', __('A brief description about this template.', 'event_espresso'), FALSE, ''),
+				'MTP_user_id'=> new EE_WP_User_Field('MTP_user_id', __('Template Creator ID', 'event_espresso'), FALSE, get_current_user_id()),
+				'MTP_messenger'=>new EE_Plain_Text_Field('MTP_messenger', __('Messenger Used for Template', 'event_espresso'), FALSE, 'email'),
+				'MTP_message_type'=>new EE_Plain_Text_Field('MTP_message_type', __('Message Type', 'event_espresso'), false, 'registration'),
 				'MTP_is_global'=>new EE_Boolean_Field('MTP_is_global', __('Flag indicating if Template Group is Global', 'event_espresso'), false, true),
 				'MTP_is_override'=>new EE_Boolean_Field('MTP_is_override', __('Flag indicating if Template Group overrides any other Templates for the messenger/messagetype combination', 'event_espresso'), false, false),
 				'MTP_deleted'=>new EE_Trashed_Flag_Field('MTP_deleted', __('Flag indicating whether this has been trashed', 'event_espresso'), false, false),
@@ -43,12 +43,12 @@ 
 			'Event' => new EE_HABTM_Relation('Event_Message_Template'),
 			'WP_User' => new EE_Belongs_To_Relation()
 		);
-		foreach( $this->_cap_contexts_to_cap_action_map as $context => $action ){
-			$this->_cap_restriction_generators[ $context ] = new EE_Restriction_Generator_Global( 'MTP_is_global');
+		foreach ($this->_cap_contexts_to_cap_action_map as $context => $action) {
+			$this->_cap_restriction_generators[$context] = new EE_Restriction_Generator_Global('MTP_is_global');
 		}
 		$this->_caps_slug = 'messages';
 
-		parent::__construct( $timezone );
+		parent::__construct($timezone);
 	}
 
 
@@ -71,8 +71,8 @@ 
 		$limit = null,
 		$count = false
 	) {
-		$query_params = array( array('Event.EVT_ID' => $EVT_ID), 'order_by' => array($orderby => $order), 'limit' => $limit );
-		return $count ? $this->count_deleted( $query_params, 'GRP_ID', TRUE ) : $this->get_all_deleted( $query_params );
+		$query_params = array(array('Event.EVT_ID' => $EVT_ID), 'order_by' => array($orderby => $order), 'limit' => $limit);
+		return $count ? $this->count_deleted($query_params, 'GRP_ID', TRUE) : $this->get_all_deleted($query_params);
 	}
 
 
@@ -86,9 +86,9 @@ 
 	 * @param string $order
 	 * @return array all (including trashed or inactive) message template group objects for the given messenger
 	 */
-	public function get_all_message_templates_by_messenger($messenger, $orderby = 'GRP_ID', $order = 'ASC' ) {
+	public function get_all_message_templates_by_messenger($messenger, $orderby = 'GRP_ID', $order = 'ASC') {
 		return $this->get_all_deleted_and_undeleted(
-			array( array( 'MTP_messenger' => $messenger ), 'order_by' => array( $orderby => $order ) )
+			array(array('MTP_messenger' => $messenger), 'order_by' => array($orderby => $order))
 		);
 	}
 
@@ -99,18 +99,18 @@ 
 	 * @param  array  $_where any existing where conditions to append these to.
 	 * @return array          original where conditions or original with additional filters.
 	 */
-	protected function _maybe_mtp_filters( $_where = array() ) {
+	protected function _maybe_mtp_filters($_where = array()) {
 		//account for messenger or message type filters
 		if (
-			isset( $_REQUEST[ 'ee_messenger_filter_by' ] )
-			&& $_REQUEST[ 'ee_messenger_filter_by' ] != 'none_selected'
-			&& $_REQUEST[ 'ee_messenger_filter_by' ] != 'all'
+			isset($_REQUEST['ee_messenger_filter_by'])
+			&& $_REQUEST['ee_messenger_filter_by'] != 'none_selected'
+			&& $_REQUEST['ee_messenger_filter_by'] != 'all'
 		) {
-			$_where['MTP_messenger'] =  $_REQUEST['ee_messenger_filter_by'] ;
+			$_where['MTP_messenger'] = $_REQUEST['ee_messenger_filter_by'];
 		}
 
 		if (
-			isset( $_REQUEST['ee_message_type_filter_by'])
+			isset($_REQUEST['ee_message_type_filter_by'])
 			&& $_REQUEST['ee_message_type_filter_by'] != 'none_selected'
 		) {
 			$_where['MTP_message_type'] = $_REQUEST['ee_message_type_filter_by'];
@@ -141,7 +141,7 @@ 
 		$global = true,
 		$user_check = false
 	) {
-		$_where = $global ? array('MTP_is_global' => TRUE ) : array('MTP_is_global' => FALSE );
+		$_where = $global ? array('MTP_is_global' => TRUE) : array('MTP_is_global' => FALSE);
 		$_where['MTP_is_active'] = TRUE;
 		$_where = $this->_maybe_mtp_filters($_where);
 
@@ -156,9 +156,9 @@ 
 			$_where['MTP_user_id'] = get_current_user_id();
 		}
 
-		$query_params = array( $_where, 'order_by' => array($orderby => $order), 'limit' => $limit );
+		$query_params = array($_where, 'order_by' => array($orderby => $order), 'limit' => $limit);
 
-		return $count ? $this->count($query_params, 'GRP_ID', TRUE ) : $this->get_all($query_params);
+		return $count ? $this->count($query_params, 'GRP_ID', TRUE) : $this->get_all($query_params);
 	}
 
 
@@ -173,14 +173,14 @@ 
 	 * @param bool   $count
 	 * @return mixed array on success, FALSE on fail
 	 */
-	public function get_all_message_templates( $orderby = 'GRP_ID', $order = 'ASC', $limit = null, $count = false ) {
+	public function get_all_message_templates($orderby = 'GRP_ID', $order = 'ASC', $limit = null, $count = false) {
 		$_where = $this->_maybe_mtp_filters();
 
-		$query_params = array( $_where, 'order_by' => array($orderby => $order), 'limit' => $limit );
+		$query_params = array($_where, 'order_by' => array($orderby => $order), 'limit' => $limit);
 
 		$r_templates = $count
-			? $this->count_deleted_and_undeleted($query_params, 'GRP_ID', TRUE )
-			: $this->get_all_deleted_and_undeleted( $query_params );
+			? $this->count_deleted_and_undeleted($query_params, 'GRP_ID', TRUE)
+			: $this->get_all_deleted_and_undeleted($query_params);
 
 		return $r_templates;
 	}
@@ -194,10 +194,10 @@ 
 	 * @param  array  $query_params same as EEM_Base::get_all()
 	 * @return  EE_Message_Template_Group[]
 	 */
-	public function get_all_custom_templates_by_event( $EVT_ID, $query_params = array() ) {
-		$where = array_merge( $query_params, array( 'Event.EVT_ID' => $EVT_ID ) );
+	public function get_all_custom_templates_by_event($EVT_ID, $query_params = array()) {
+		$where = array_merge($query_params, array('Event.EVT_ID' => $EVT_ID));
 		return $this->get_all(
-			array( $where )
+			array($where)
 		);
 	}
 
@@ -222,13 +222,13 @@ 
 		$count = false,
 		$global = true
 	) {
-		$_where = $global ? array('MTP_is_global' => TRUE ) : array('MTP_is_global' => FALSE );
+		$_where = $global ? array('MTP_is_global' => TRUE) : array('MTP_is_global' => FALSE);
 		$_where['MTP_is_active'] = TRUE;
 		$_where = $this->_maybe_mtp_filters($_where);
 
-		$query_params = array( $_where, 'order_by' => array($orderby => $order), 'limit' => $limit );
+		$query_params = array($_where, 'order_by' => array($orderby => $order), 'limit' => $limit);
 
-		return $count ? $this->count_deleted($query_params, 'GRP_ID', TRUE ) : $this->get_all_deleted( $query_params );
+		return $count ? $this->count_deleted($query_params, 'GRP_ID', TRUE) : $this->get_all_deleted($query_params);
 	}
 
 
@@ -264,9 +264,9 @@ 
 			'MTP_is_active' => $active
 		);
 
-		$query_params = array( $_where, 'order_by' => array($orderby=>$order), 'limit' => $limit );
+		$query_params = array($_where, 'order_by' => array($orderby=>$order), 'limit' => $limit);
 
-		return $count ? $this->count($query_params, 'GRP_ID', TRUE ) : $this->get_all( $query_params );
+		return $count ? $this->count($query_params, 'GRP_ID', TRUE) : $this->get_all($query_params);
 	}
 
 
@@ -300,13 +300,13 @@ 
 			'MTP_is_global' => TRUE,
 		);
 
-		if ( $active != 'all' ) {
+		if ($active != 'all') {
 			$_where['MTP_is_active'] = $active;
 		}
 
-		$query_params = array( $_where, 'order_by' => array( $orderby => $order ), 'limit' => $limit );
+		$query_params = array($_where, 'order_by' => array($orderby => $order), 'limit' => $limit);
 
-		return $count ? $this->count( $query_params, 'GRP_ID', TRUE ) : $this->get_all( $query_params );
+		return $count ? $this->count($query_params, 'GRP_ID', TRUE) : $this->get_all($query_params);
 	}
 
 
@@ -319,7 +319,7 @@ 
 	 * @param  array  $query_params same as EEM_Base->get_all()
 	 * @return EE_Message_Template_Group[]
 	 */
-	public function get_custom_message_template_by_m_and_mt( $messenger, $message_type, $query_params = array() ) {
+	public function get_custom_message_template_by_m_and_mt($messenger, $message_type, $query_params = array()) {
 		return $this->get_all(
 			array_merge(
 				$query_params,
@@ -356,14 +356,14 @@ 
 		$assembled_fields = array();
 
 		//let's loop through all the fields and set them up in the right format
-		foreach ( $fields as $index => $value ) {
+		foreach ($fields as $index => $value) {
 			// first let's figure out if the value['content'] in the current index is an array.
 			//  If it is then this is special fields that are used in parsing special shortcodes (i.e. 'attendee_list').
-			if ( is_array($value['content']) ) {
+			if (is_array($value['content'])) {
 				$assembled_fields[$value['name']] = $value['content']['main'];
 				//loop through the content and get the other fields.
-				foreach ( $value['content'] as $name => $val ) {
-					if ( $name == 'main' ) continue;
+				foreach ($value['content'] as $name => $val) {
+					if ($name == 'main') continue;
 					$assembled_fields[$name] = $val;
 				}
 				continue;
@@ -375,15 +375,15 @@ 
 
 		// now we've got the assembled_fields.
 		// We need to setup the string for the appropriate validator class and call that.
-		$m_ref = ucwords( str_replace('_',' ', $messenger ) );
-		$m_ref = str_replace( ' ', '_', $m_ref );
-		$mt_ref = ucwords( str_replace('_', ' ', $message_type ) );
-		$mt_ref = str_replace( ' ', '_', $mt_ref );
+		$m_ref = ucwords(str_replace('_', ' ', $messenger));
+		$m_ref = str_replace(' ', '_', $m_ref);
+		$mt_ref = ucwords(str_replace('_', ' ', $message_type));
+		$mt_ref = str_replace(' ', '_', $mt_ref);
 
-		$classname = 'EE_Messages_' . $m_ref . '_' . $mt_ref . '_Validator';
+		$classname = 'EE_Messages_'.$m_ref.'_'.$mt_ref.'_Validator';
 
-		if ( !class_exists( $classname ) ) {
-			$msg[] = __( 'The Validator class was unable to load', 'event_espresso');
+		if ( ! class_exists($classname)) {
+			$msg[] = __('The Validator class was unable to load', 'event_espresso');
 			$msg[] = sprintf(
 				__(
 					'The class name compiled was %s. Please check and make sure the spelling and case is correct for the class name and that there is an autoloader in place for this class',
@@ -391,11 +391,11 @@ 
 				),
 				$classname
 			);
-			throw new EE_Error( implode( '||', $msg ) );
+			throw new EE_Error(implode('||', $msg));
 		}
 
-		$a = new ReflectionClass( $classname );
-		$_VLD = $a->newInstance( $assembled_fields, $context );
+		$a = new ReflectionClass($classname);
+		$_VLD = $a->newInstance($assembled_fields, $context);
 		$result = $_VLD->validate();
 		return $result;
 	}
@@ -414,18 +414,18 @@ 
 	 *
 	 * @return int  count of updated records.
 	 */
-	public function deactivate_message_template_groups_for( $messenger_names = array(), $message_type_names = array() ) {
+	public function deactivate_message_template_groups_for($messenger_names = array(), $message_type_names = array()) {
 		$query_args = array();
-		if ( empty( $messenger_names ) && empty( $message_type_names ) ) {
+		if (empty($messenger_names) && empty($message_type_names)) {
 			return 0;
 		}
-		if ( ! empty( $messenger_names ) ) {
-			$query_args[ 0 ][ 'MTP_messenger' ] = array( 'IN', (array) $messenger_names );
+		if ( ! empty($messenger_names)) {
+			$query_args[0]['MTP_messenger'] = array('IN', (array) $messenger_names);
 		}
-		if ( ! empty( $message_type_names ) ) {
-			$query_args[ 0 ][ 'MTP_message_type' ] = array( 'IN', (array) $message_type_names );
+		if ( ! empty($message_type_names)) {
+			$query_args[0]['MTP_message_type'] = array('IN', (array) $message_type_names);
 		}
-		return $this->update( array( 'MTP_is_active' => false ), $query_args );
+		return $this->update(array('MTP_is_active' => false), $query_args);
 	}
 
 

Braces +5 added lines, -2 removed lines

@@ -1,5 +1,6 @@ 
-<?php if (!defined('EVENT_ESPRESSO_VERSION') )
+<?php if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 /**
  * EEM_Message_Template_Group
  *
@@ -363,7 +364,9 @@ 
 				$assembled_fields[$value['name']] = $value['content']['main'];
 				//loop through the content and get the other fields.
 				foreach ( $value['content'] as $name => $val ) {
-					if ( $name == 'main' ) continue;
+					if ( $name == 'main' ) {
+						continue;
+					}
 					$assembled_fields[$name] = $val;
 				}
 				continue;

shortcodes/espresso_checkout/EES_Espresso_Checkout.shortcode.php

Spacing +6 added lines, -6 removed lines

@@ -51,17 +51,17 @@ 
 	 * @param WP $WP
 	 * @return    void
 	 */
-	public function run( WP $WP ) {
-		if ( did_action( 'pre_get_posts' ) && did_action( 'send_headers' ) ) {
+	public function run(WP $WP) {
+		if (did_action('pre_get_posts') && did_action('send_headers')) {
 			global $wp_query;
 			EED_Single_Page_Checkout::load_reg_steps();
-			EED_Single_Page_Checkout::init( $wp_query );
+			EED_Single_Page_Checkout::init($wp_query);
 		} else {
 			// hook into the top of pre_get_posts to set the reg step routing, which gives other modules or plugins a chance to modify the reg steps, but just before the routes get called
-			add_action( 'pre_get_posts', array( 'EED_Single_Page_Checkout', 'load_reg_steps' ), 1 );
+			add_action('pre_get_posts', array('EED_Single_Page_Checkout', 'load_reg_steps'), 1);
 			// this will trigger the EED_Single_Page_Checkout module's run() method during the pre_get_posts hook point,
 			// this allows us to initialize things, enqueue assets, etc,
-			add_action( 'pre_get_posts', array( 'EED_Single_Page_Checkout', 'init' ), 10, 1 );
+			add_action('pre_get_posts', array('EED_Single_Page_Checkout', 'init'), 10, 1);
 		}
 	}
 
@@ -74,7 +74,7 @@ 
 	 *  @param		array 	$attributes
 	 *  @return 	string
 	 */
-	public function process_shortcode( $attributes = array() ) {
+	public function process_shortcode($attributes = array()) {
 		return EE_Registry::instance()->REQ->get_output();
 	}
 

Braces +3 added lines, -1 removed lines

@@ -1,4 +1,6 @@
-<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
+<?php if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
+}
 /**
  * Event Espresso
  *