eventespresso / event-espresso-core

core/data_migration_scripts/EE_Data_Migration_Script_Stage.core.php

Spacing +9 added lines, -9 removed lines

@@ -34,7 +34,7 @@ 
  * _migration_step() (ie, it its a count of rows in the old attendees table in _count_records_to_migrate(), it should also be OLD attendee rows migrated
  * on each call to _migration_step().
  */
-abstract class EE_Data_Migration_Script_Stage extends EE_Data_Migration_Class_Base{
+abstract class EE_Data_Migration_Script_Stage extends EE_Data_Migration_Class_Base {
 	/**
 	 * The migration script this is a stage of
 	 * @var EE_Data_Migration_Script_Base
@@ -47,7 +47,7 @@ 
 	 * called by EE_Data_Migration_Script_Base's __construct() method so children don't have to
 	 * @param EE_Data_Migration_Script_Base $migration_script
 	 */
-	public function _construct_finalize($migration_script){
+	public function _construct_finalize($migration_script) {
 		$this->_migration_script = $migration_script;
 	}
 
@@ -58,13 +58,13 @@ 
 	 * @param int $num_items_to_migrate
 	 * @return int
 	 */
-	public function migration_step($num_items_to_migrate=50){
+	public function migration_step($num_items_to_migrate = 50) {
 		//before we run the migration step, we want ot take note of warnings that get outputted
 		ob_start();
 		$items_migrated = $this->_migration_step($num_items_to_migrate);
 		$output = ob_get_contents();
 		ob_end_clean();
-		if( $output ){
+		if ($output) {
 			$this->add_error($output);
 		}
 		$this->_records_migrated += $items_migrated;
@@ -82,7 +82,7 @@ 
 	 * @param int $num_items_to_migrate
 	 * @return int number of items ACTUALLY migrated
 	 */
-	abstract protected function _migration_step($num_items_to_migrate=50);
+	abstract protected function _migration_step($num_items_to_migrate = 50);
 
 	/**
 	 * Counts the records that have been migrated so far
@@ -96,7 +96,7 @@ 
 	 * returns an array of strings describing errors
 	 * @return array
 	 */
-	public function get_errors(){
+	public function get_errors() {
 		return $this->_errors;
 	}
 
@@ -106,9 +106,9 @@ 
 	 * to have been made from the properties_as_array() function.
 	 * @param array $array_of_properties like what's produced from properties_as_array() method
 	 */
-	public function instantiate_from_array_of_properties($array_of_properties){
+	public function instantiate_from_array_of_properties($array_of_properties) {
 		unset($array_of_properties['class']);
-		foreach($array_of_properties as $property_name => $property_value){
+		foreach ($array_of_properties as $property_name => $property_value) {
 			$this->{$property_name} = $property_value;
 		}
 	}
@@ -117,7 +117,7 @@ 
 	 * Gets the script this is a stage of
 	 * @return EE_Data_Migration_Script_Base
 	 */
-	protected function get_migration_script(){
+	protected function get_migration_script() {
 		return $this->_migration_script;
 	}
 }

core/helpers/EEH_Sideloader.helper.php

Spacing +37 added lines, -37 removed lines

@@ -28,7 +28,7 @@ 
 
 
 
-require_once( EE_HELPERS . 'EEH_Base.helper.php' );
+require_once(EE_HELPERS.'EEH_Base.helper.php');
 class EEH_Sideloader extends EEH_Base {
 
 	private $_upload_to;
@@ -43,8 +43,8 @@ 
 	 * @access public
 	 * @param array $init array fo initializing the sideloader if keys match the properties.
 	 */
-	public function __construct( $init = array() ) {
-		$this->_init( $init );
+	public function __construct($init = array()) {
+		$this->_init($init);
 	}
 
 
@@ -55,24 +55,24 @@ 
 	 * @param  array  $init array on init (keys match properties others ignored)
 	 * @return void
 	 */
-	private function _init( $init ) {
+	private function _init($init) {
 		$defaults = array(
 			'_upload_to' => $this->_get_wp_uploads_dir(),
 			'_upload_from' => '',
 			'_permissions' => 0644,
-			'_new_file_name' => 'EE_Sideloader_' . uniqid() . '.default'
+			'_new_file_name' => 'EE_Sideloader_'.uniqid().'.default'
 			);
 
-		$props = array_merge( $defaults, $init );
+		$props = array_merge($defaults, $init);
 
-		foreach ( $props as $key => $val ) {
-			if ( EEH_Class_Tools::has_property( $this, $key ) ) {
+		foreach ($props as $key => $val) {
+			if (EEH_Class_Tools::has_property($this, $key)) {
 				$this->{$key} = $val;
 			}
 		}
 
 		//make sure we include the required wp file for needed functions
-		require_once( ABSPATH . 'wp-admin/includes/file.php' );
+		require_once(ABSPATH.'wp-admin/includes/file.php');
 	}
 
 
@@ -80,16 +80,16 @@ 
 	private function _get_wp_uploads_dir() {}
 
 	//setters
-	public function set_upload_to( $upload_to_folder ) {
+	public function set_upload_to($upload_to_folder) {
 		$this->_upload_to = $upload_to_folder;
 	}
-	public function set_upload_from( $upload_from_folder ) {
+	public function set_upload_from($upload_from_folder) {
 		$this->_upload_from_folder = $upload_from_folder;
 	}
-	public function set_permissions( $permissions ) {
+	public function set_permissions($permissions) {
 		$this->_permissions = $permissions;
 	}
-	public function set_new_file_name( $new_file_name ) {
+	public function set_new_file_name($new_file_name) {
 		$this->_new_file_name = $new_file_name;
 	}
 
@@ -111,34 +111,34 @@ 
 	//upload methods
 	public function sideload() {
 		//setup temp dir
-		$temp_file = wp_tempnam( $this->_upload_from );
+		$temp_file = wp_tempnam($this->_upload_from);
 
-		if ( !$temp_file ) {
-			EE_Error::add_error( __('Something went wrong with the upload.  Unable to create a tmp file for the uploaded file on the server', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__ );
+		if ( ! $temp_file) {
+			EE_Error::add_error(__('Something went wrong with the upload.  Unable to create a tmp file for the uploaded file on the server', 'event_espresso'), __FILE__, __FUNCTION__, __LINE__);
 			return false;
 		}
 
-		do_action( 'AHEE__EEH_Sideloader__sideload__before', $this, $temp_file );
+		do_action('AHEE__EEH_Sideloader__sideload__before', $this, $temp_file);
 
-		$wp_remote_args = apply_filters( 'FHEE__EEH_Sideloader__sideload__wp_remote_args', array( 'timeout' => 500, 'stream' => true, 'filename' => $temp_file ), $this, $temp_file );
+		$wp_remote_args = apply_filters('FHEE__EEH_Sideloader__sideload__wp_remote_args', array('timeout' => 500, 'stream' => true, 'filename' => $temp_file), $this, $temp_file);
 
-		$response = wp_safe_remote_get( $this->_upload_from, $wp_remote_args );
+		$response = wp_safe_remote_get($this->_upload_from, $wp_remote_args);
 
-		if ( is_wp_error( $response ) || 200 != wp_remote_retrieve_response_code( $response ) ) {
-			unlink( $temp_file );
-			if ( defined( 'WP_DEBUG' ) && WP_DEBUG ) {
-				EE_Error::add_error( sprintf( __('Unable to upload the file.  Either the path given to upload from is incorrect, or something else happened.  Here is the response returned:<br />%s<br />Here is the path given: %s', 'event_espresso'), var_export( $response, true ), $this->_upload_from ), __FILE__, __FUNCTION__, __LINE__ );
+		if (is_wp_error($response) || 200 != wp_remote_retrieve_response_code($response)) {
+			unlink($temp_file);
+			if (defined('WP_DEBUG') && WP_DEBUG) {
+				EE_Error::add_error(sprintf(__('Unable to upload the file.  Either the path given to upload from is incorrect, or something else happened.  Here is the response returned:<br />%s<br />Here is the path given: %s', 'event_espresso'), var_export($response, true), $this->_upload_from), __FILE__, __FUNCTION__, __LINE__);
 			}
 			return false;
 		}
 
 		//possible md5 check
-		$content_md5 = wp_remote_retrieve_header( $response, 'content-md5' );
-		if ( $content_md5 ) {
-			$md5_check = verify_file_md5( $temp_file, $content_md5 );
-			if ( is_wp_error( $md5_check ) ) {
-				unlink( $temp_file );
-				EE_Error::add_error( $md5_check->get_error_message(), __FILE__, __FUNCTION__, __LINE__ );
+		$content_md5 = wp_remote_retrieve_header($response, 'content-md5');
+		if ($content_md5) {
+			$md5_check = verify_file_md5($temp_file, $content_md5);
+			if (is_wp_error($md5_check)) {
+				unlink($temp_file);
+				EE_Error::add_error($md5_check->get_error_message(), __FILE__, __FUNCTION__, __LINE__);
 				return false;
 			}
 		}
@@ -146,24 +146,24 @@ 
 		$file = $temp_file;
 
 		//now we have the file, let's get it in the right directory with the right name.
-		$path = apply_filters( 'FHEE__EEH_Sideloader__sideload__new_path', $this->_upload_to . $this->_new_file_name, $this );
+		$path = apply_filters('FHEE__EEH_Sideloader__sideload__new_path', $this->_upload_to.$this->_new_file_name, $this);
 
 		//move file in
-		if ( false === @ rename( $file, $path ) ) {
-			unlink( $temp_file );
-			EE_Error::add_error(  sprintf( __('Unable to move the file to new location (possible permissions errors). This is the path the class attempted to move the file to: %s', 'event_espresso' ), $path ), __FILE__, __FUNCTION__, __LINE__ );
+		if (false === @ rename($file, $path)) {
+			unlink($temp_file);
+			EE_Error::add_error(sprintf(__('Unable to move the file to new location (possible permissions errors). This is the path the class attempted to move the file to: %s', 'event_espresso'), $path), __FILE__, __FUNCTION__, __LINE__);
 			return false;
 		}
 
 		//set permissions
-		$permissions = apply_filters( 'FHEE__EEH_Sideloader__sideload__permissions_applied', $this->_permissions, $this );
-		chmod( $path, $permissions );
+		$permissions = apply_filters('FHEE__EEH_Sideloader__sideload__permissions_applied', $this->_permissions, $this);
+		chmod($path, $permissions);
 
 		//that's it.  let's allow for actions after file uploaded.
-		do_action( 'AHEE__EE_Sideloader__sideload_after', $this, $path );
+		do_action('AHEE__EE_Sideloader__sideload_after', $this, $path);
 
 		//unlink tempfile
-		@unlink( $temp_file );
+		@unlink($temp_file);
 		return true;
 	}
 

core/libraries/shortcodes/EE_Event_Author_Shortcodes.lib.php

Spacing +22 added lines, -22 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**
@@ -49,54 +49,54 @@ 
 
 
 
-	protected function _parser( $shortcode ) {
+	protected function _parser($shortcode) {
 		//make sure we end up with a copy of the EE_Messages_Addressee object
 		$recipient = $this->_data instanceof EE_Messages_Addressee ? $this->_data : NULL;
-		$recipient = ! $recipient instanceof EE_Messages_Addressee && is_array($this->_data) && isset( $this->_data['data'] ) && $this->_data['data'] instanceof EE_Messages_Addressee ? $this->_data['data'] : $recipient;
-		$recipient = ! $recipient instanceof EE_Messages_Addressee && !empty( $this->_extra_data['data'] ) && $this->_extra_data['data'] instanceof EE_Messages_Addressee ? $this->_extra_data['data'] : $recipient;
+		$recipient = ! $recipient instanceof EE_Messages_Addressee && is_array($this->_data) && isset($this->_data['data']) && $this->_data['data'] instanceof EE_Messages_Addressee ? $this->_data['data'] : $recipient;
+		$recipient = ! $recipient instanceof EE_Messages_Addressee && ! empty($this->_extra_data['data']) && $this->_extra_data['data'] instanceof EE_Messages_Addressee ? $this->_extra_data['data'] : $recipient;
 
 		//now it's possible that $recipient is not an instance of EE_Messages_Addressee in which case we need to see if $this->_data is an instance of $event.
 		$event = $this->_data instanceof EE_Event ? $this->_data : NULL;
 
-		if ( ! $recipient instanceof EE_Messages_Addressee && ! $event instanceof EE_Event )
+		if ( ! $recipient instanceof EE_Messages_Addressee && ! $event instanceof EE_Event)
 			return '';
 
-		switch ( $shortcode ) {
+		switch ($shortcode) {
 			case '[EVENT_AUTHOR_FNAME]' :
-				$fname =  !empty( $recipient ) ? $recipient->fname : NULL;
-				if ( empty( $fname ) && !empty( $event ) ) {
-					$user = $this->_get_author_for_event( $event );
+				$fname = ! empty($recipient) ? $recipient->fname : NULL;
+				if (empty($fname) && ! empty($event)) {
+					$user = $this->_get_author_for_event($event);
 					$fname = $user->first_name;
 				}
 				return $fname;
 				break;
 
 			case '[EVENT_AUTHOR_LNAME]' :
-				$lname =  !empty( $recipient ) ? $recipient->lname : NULL;
-				if ( empty( $lname ) && !empty( $event ) ) {
-					$user = $this->_get_author_for_event( $event );
+				$lname = ! empty($recipient) ? $recipient->lname : NULL;
+				if (empty($lname) && ! empty($event)) {
+					$user = $this->_get_author_for_event($event);
 					$lname = $user->last_name;
 				}
 				return $lname;
 				break;
 
 			case '[EVENT_AUTHOR_FORMATTED_EMAIL]' :
-				if ( !empty( $recipient ) ) {
-					$email =  !empty( $recipient->fname ) ? $recipient->fname . ' ' . $recipient->lname . '<' . $recipient->admin_email . '>' : EE_Registry::instance()->CFG->organization->get_pretty( 'name' ) . '<' . $recipient->admin_email . '>';
+				if ( ! empty($recipient)) {
+					$email = ! empty($recipient->fname) ? $recipient->fname.' '.$recipient->lname.'<'.$recipient->admin_email.'>' : EE_Registry::instance()->CFG->organization->get_pretty('name').'<'.$recipient->admin_email.'>';
 				} else {
 					$email = NULL;
 				}
-				if ( empty( $email ) && ! empty( $event ) ) {
-					$user = $this->_get_author_for_event( $event );
-					$email = ! empty( $user->first_name ) ? $user->first_name . ' ' . $user->last_name . '<' . $user->user_email . '>' : EE_Registry::instance()->CFG->organization->get_pretty( 'name' ) . '<' . $user->user_email . '>';
+				if (empty($email) && ! empty($event)) {
+					$user = $this->_get_author_for_event($event);
+					$email = ! empty($user->first_name) ? $user->first_name.' '.$user->last_name.'<'.$user->user_email.'>' : EE_Registry::instance()->CFG->organization->get_pretty('name').'<'.$user->user_email.'>';
 				}
 				return $email;
 				break;
 
 			case '[EVENT_AUTHOR_EMAIL]' :
-				$email = ! empty( $recipient ) ? $recipient->admin_email : NULL;
-				if ( empty( $email ) && !empty( $event ) ) {
-					$user = $this->_get_author_for_event( $event );
+				$email = ! empty($recipient) ? $recipient->admin_email : NULL;
+				if (empty($email) && ! empty($event)) {
+					$user = $this->_get_author_for_event($event);
 					$email = $user->user_email;
 				}
 				return $email;
@@ -117,9 +117,9 @@ 
 	 *
 	 * @return WP_User
 	 */
-	private function _get_author_for_event( EE_Event $event ) {
+	private function _get_author_for_event(EE_Event $event) {
 		$author_id = $event->wp_user();
-		$user_data = get_userdata( (int) $author_id );
+		$user_data = get_userdata((int) $author_id);
 		return $user_data;
 	}
 

core/libraries/rest_api/Model_Version_Info.php

Braces +2 added lines, -2 removed lines

@@ -231,7 +231,7 @@ 
 		$model_names = $this->models_for_requested_version();
 		if( isset( $model_names[ $model_name ] ) ) {
 			return true;
-		}else{
+		} else{
 			return false;
 		}
 	}
@@ -249,7 +249,7 @@ 
 	public function load_model( $model_name ) {
 		if( $this->is_model_name_in_this_version( $model_name ) ) {
 			return \EE_Registry::instance()->load_model( $model_name );
-		}else{
+		} else{
 			throw new \EE_Error(
 				sprintf(
 					__(

Spacing +57 added lines, -57 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 namespace EventEspresso\core\libraries\rest_api;
 
-if ( !defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 /**
@@ -88,7 +88,7 @@ 
 	protected $_cached_fields_on_models = array();
 
 
-	public function __construct( $requested_version ) {
+	public function __construct($requested_version) {
 		$this->_requested_version = $requested_version;
 		$this->_model_changes = array(
 			'4.8.29' => array(
@@ -110,13 +110,13 @@ 
 			'table_column' => 'N/A',
 			'always_available' => true,
 		);
-		foreach( $this->_resource_changes as $version => $model_classnames ) {
-			foreach( $model_classnames as $model_classname => $extra_fields ) {
-				foreach( $extra_fields as $fieldname => $field_data ) {
-					$this->_resource_changes[ $model_classname ][ $fieldname ][ 'name' ] = $fieldname;
-					foreach( $defaults as $attribute => $default_value ) {
-						if( ! isset( $this->_resource_changes[ $model_classname ][ $fieldname ][ $attribute ] ) ) {
-							$this->_resource_changes[ $model_classname ][ $fieldname ][ $attribute ] = $default_value;
+		foreach ($this->_resource_changes as $version => $model_classnames) {
+			foreach ($model_classnames as $model_classname => $extra_fields) {
+				foreach ($extra_fields as $fieldname => $field_data) {
+					$this->_resource_changes[$model_classname][$fieldname]['name'] = $fieldname;
+					foreach ($defaults as $attribute => $default_value) {
+						if ( ! isset($this->_resource_changes[$model_classname][$fieldname][$attribute])) {
+							$this->_resource_changes[$model_classname][$fieldname][$attribute] = $default_value;
 						}
 					}
 				}
@@ -132,11 +132,11 @@ 
 	 * @return array
 	 */
 	public function model_changes_between_requested_version_and_current() {
-		if( $this->_cached_model_changes_between_requested_version_and_current === null ) {
+		if ($this->_cached_model_changes_between_requested_version_and_current === null) {
 			$model_changes = array();
-			foreach( $this->model_changes() as $version => $models_changed_in_version ) {
-				if( $version <= \EED_Core_Rest_Api::core_version() && $version > $this->requested_version() ) {
-					$model_changes[ $version ] = $models_changed_in_version;
+			foreach ($this->model_changes() as $version => $models_changed_in_version) {
+				if ($version <= \EED_Core_Rest_Api::core_version() && $version > $this->requested_version()) {
+					$model_changes[$version] = $models_changed_in_version;
 				}
 			}
 			$this->_cached_model_changes_between_requested_version_and_current = $model_changes;
@@ -152,11 +152,11 @@ 
 	 * @return array
 	 */
 	public function resource_changes_between_requested_version_and_current() {
-		if( $this->_cached_resource_changes_between_requested_version_and_current === null ) {
+		if ($this->_cached_resource_changes_between_requested_version_and_current === null) {
 			$resource_changes = array();
-			foreach( $this->resource_changes() as $version => $model_classnames ) {
-				if( $version <= \EED_Core_Rest_Api::core_version()  && $version > $this->requested_version()  ) {
-					$resource_changes[ $version ] = $model_classnames;
+			foreach ($this->resource_changes() as $version => $model_classnames) {
+				if ($version <= \EED_Core_Rest_Api::core_version() && $version > $this->requested_version()) {
+					$resource_changes[$version] = $model_classnames;
 				}
 			}
 			$this->_cached_resource_changes_between_requested_version_and_current = $resource_changes;
@@ -168,7 +168,7 @@ 
 	 * If a request was sent to 'wp-json/ee/v4.7/events' this would be '4.7'
 	 * @return string like '4.6'
 	 */
-	public function requested_version(){
+	public function requested_version() {
 		return $this->_requested_version;
 	}
 
@@ -182,7 +182,7 @@ 
 	 * If a version is missing then we don't know anything about what changes it introduced from the previous version
 	 * @return array
 	 */
-	public function model_changes(){
+	public function model_changes() {
 		return $this->_model_changes;
 	}
 
@@ -193,12 +193,12 @@ 
 	 * @return array keys are model names, values are their classname
 	 */
 	public function models_for_requested_version() {
-		if( $this->_cached_models_for_requested_version === null ) {
+		if ($this->_cached_models_for_requested_version === null) {
 			$all_models_in_current_version = \EE_Registry::instance()->non_abstract_db_models;
-			foreach( $this->model_changes_between_requested_version_and_current() as $version => $models_changed ) {
-				foreach( $models_changed as $model_name => $new_indicator_or_fields_added ) {
-					if( $new_indicator_or_fields_added === Model_Version_Info::model_added ) {
-						unset( $all_models_in_current_version[ $model_name ] );
+			foreach ($this->model_changes_between_requested_version_and_current() as $version => $models_changed) {
+				foreach ($models_changed as $model_name => $new_indicator_or_fields_added) {
+					if ($new_indicator_or_fields_added === Model_Version_Info::model_added) {
+						unset($all_models_in_current_version[$model_name]);
 					}
 				}
 			}
@@ -217,11 +217,11 @@ 
 	 * @param string $model_name eg 'Event'
 	 * @return boolean
 	 */
-	public function is_model_name_in_this_version( $model_name ) {
+	public function is_model_name_in_this_version($model_name) {
 		$model_names = $this->models_for_requested_version();
-		if( isset( $model_names[ $model_name ] ) ) {
+		if (isset($model_names[$model_name])) {
 			return true;
-		}else{
+		} else {
 			return false;
 		}
 	}
@@ -236,10 +236,10 @@ 
 	 * @return \EEM_Base
 	 * @throws \EE_Error
 	 */
-	public function load_model( $model_name ) {
-		if( $this->is_model_name_in_this_version( $model_name ) ) {
-			return \EE_Registry::instance()->load_model( $model_name );
-		}else{
+	public function load_model($model_name) {
+		if ($this->is_model_name_in_this_version($model_name)) {
+			return \EE_Registry::instance()->load_model($model_name);
+		} else {
 			throw new \EE_Error(
 				sprintf(
 					__(
@@ -261,21 +261,21 @@ 
 	 * @param \EEM_Base $model
 	 * @return array|\EE_Model_Field_Base[]
 	 */
-	public function fields_on_model_in_this_version( $model ) {
-		if( ! isset( $this->_cached_fields_on_models[ $model->get_this_model_name() ] ) ) {
+	public function fields_on_model_in_this_version($model) {
+		if ( ! isset($this->_cached_fields_on_models[$model->get_this_model_name()])) {
 			//get all model changes between the requested version and current core version
 			$changes = $this->model_changes_between_requested_version_and_current();
 			//fetch all fields currently on this model
 			$current_fields = $model->field_settings();
 			//remove all fields that have been added since
-			foreach( $changes as $version => $changes_in_version ) {
-				if(
-					isset( $changes_in_version[ $model->get_this_model_name() ] )
-					&& $changes_in_version[ $model->get_this_model_name() ] !== Model_Version_Info::model_added
+			foreach ($changes as $version => $changes_in_version) {
+				if (
+					isset($changes_in_version[$model->get_this_model_name()])
+					&& $changes_in_version[$model->get_this_model_name()] !== Model_Version_Info::model_added
 				) {
 					$current_fields = array_diff_key(
 						$current_fields,
-						array_flip( $changes_in_version[ $model->get_this_model_name() ] )
+						array_flip($changes_in_version[$model->get_this_model_name()])
 					);
 				}
 			}
@@ -293,9 +293,9 @@ 
 	 * @param array $classnames
 	 * @return boolean
 	 */
-	public function is_subclass_of_one( $object, $classnames ) {
-		foreach( $classnames as $classname ) {
-			if( is_a( $object, $classname ) ) {
+	public function is_subclass_of_one($object, $classnames) {
+		foreach ($classnames as $classname) {
+			if (is_a($object, $classname)) {
 				return true;
 			}
 		}
@@ -306,10 +306,10 @@ 
 	 * Returns the list of model field classes that that the API basically ignores
 	 * @return array
 	 */
-	public function fields_ignored(){
+	public function fields_ignored() {
 		return apply_filters(
 			'FHEE__Controller_Model_Read_fields_ignored',
-			array( 'EE_Foreign_Key_Field_Base', 'EE_Any_Foreign_Model_Name_Field' )
+			array('EE_Foreign_Key_Field_Base', 'EE_Any_Foreign_Model_Name_Field')
 		);
 	}
 
@@ -318,8 +318,8 @@ 
 	 * @param EE_Model_Field_Base
 	 * @return boolean
 	 */
-	public function field_is_ignored( $field_obj ){
-		 return $this->is_subclass_of_one( $field_obj, $this->fields_ignored() );
+	public function field_is_ignored($field_obj) {
+		 return $this->is_subclass_of_one($field_obj, $this->fields_ignored());
 	}
 
 	/**
@@ -330,7 +330,7 @@ 
 	public function fields_that_have_rendered_format() {
 		return apply_filters(
 			'FHEE__Controller_Model_Read__fields_raw',
-			array( 'EE_Post_Content_Field', 'EE_Full_HTML_Field' )
+			array('EE_Post_Content_Field', 'EE_Full_HTML_Field')
 		);
 	}
 
@@ -339,8 +339,8 @@ 
 	 * @param EE_Model_Field_Base
 	 * @return boolean
 	 */
-	public function field_has_rendered_format( $field_obj ){
-		 return $this->is_subclass_of_one( $field_obj, $this->fields_that_have_rendered_format() );
+	public function field_has_rendered_format($field_obj) {
+		 return $this->is_subclass_of_one($field_obj, $this->fields_that_have_rendered_format());
 	}
 
 	/**
@@ -352,7 +352,7 @@ 
 	public function fields_that_have_pretty_format() {
 		return apply_filters(
 			'FHEE__Controller_Model_Read__fields_pretty',
-			array( 'EE_Enum_Integer_Field', 'EE_Enum_Text_Field', 'EE_Money_Field' )
+			array('EE_Enum_Integer_Field', 'EE_Enum_Text_Field', 'EE_Money_Field')
 		);
 	}
 
@@ -361,8 +361,8 @@ 
 	 * @param EE_Model_Field_Base
 	 * @return boolean
 	 */
-	public function field_has_pretty_format( $field_obj ){
-		 return $this->is_subclass_of_one( $field_obj, $this->fields_that_have_pretty_format() );
+	public function field_has_pretty_format($field_obj) {
+		 return $this->is_subclass_of_one($field_obj, $this->fields_that_have_pretty_format());
 	}
 
 	/**
@@ -379,12 +379,12 @@ 
 	 * @param \EEM_Base $model
 	 * @return array
 	 */
-	public function extra_resource_properties_for_model( $model ) {
+	public function extra_resource_properties_for_model($model) {
 		$extra_properties = array();
-		foreach( $this->resource_changes_between_requested_version_and_current() as $version => $model_classnames ) {
-			foreach( $model_classnames as $model_classname => $properties_added_in_this_version ) {
-				if( is_subclass_of( $model, $model_classname ) ) {
-					$extra_properties = array_merge( $extra_properties, $properties_added_in_this_version );
+		foreach ($this->resource_changes_between_requested_version_and_current() as $version => $model_classnames) {
+			foreach ($model_classnames as $model_classname => $properties_added_in_this_version) {
+				if (is_subclass_of($model, $model_classname)) {
+					$extra_properties = array_merge($extra_properties, $properties_added_in_this_version);
 				}
 			}
 		}

core/libraries/batch/JobHandlerBaseClasses/JobHandler.php

Spacing +2 added lines, -2 removed lines

@@ -13,8 +13,8 @@
  */
 namespace EventEspressoBatchRequest\JobHandlerBaseClasses;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 

core/libraries/form_sections/strategies/layout/EE_Fieldset_Section_Layout.strategy.php

Spacing +11 added lines, -11 removed lines

@@ -11,7 +11,7 @@ 
  * @since		 	   $VID:$
  *
  */
-class EE_Fieldset_Section_Layout extends EE_Div_Per_Section_Layout{
+class EE_Fieldset_Section_Layout extends EE_Div_Per_Section_Layout {
 
 	/**
 	 * legend_class
@@ -32,10 +32,10 @@ 
 	 *
 	 * @param array $options
 	 */
-	function __construct( $options = array() ){
-		foreach( $options as $key => $value ) {
-			$key = '_' . $key;
-			if ( property_exists( $this, $key )) {
+	function __construct($options = array()) {
+		foreach ($options as $key => $value) {
+			$key = '_'.$key;
+			if (property_exists($this, $key)) {
 				$this->{$key} = $value;
 			}
 		}
@@ -47,8 +47,8 @@ 
 	 * @return string
 	 */
 	public function layout_form_begin() {
-		$html = EEH_HTML::nl(1) . '<fieldset id="' . $this->_form_section->html_id() . '" class="' . $this->_form_section->html_class() . '" style="' . $this->_form_section->html_style() . '">';
-		$html .= '<legend class="' . $this->legend_class() . '">' . $this->legend_text() . '</legend>';
+		$html = EEH_HTML::nl(1).'<fieldset id="'.$this->_form_section->html_id().'" class="'.$this->_form_section->html_class().'" style="'.$this->_form_section->html_style().'">';
+		$html .= '<legend class="'.$this->legend_class().'">'.$this->legend_text().'</legend>';
 		return $html;
 	}
 
@@ -58,8 +58,8 @@ 
 	 * closing div tag for a form
 	 * @return string
 	 */
-	public function layout_form_end(){
-		return EEH_HTML::nl(-1) . '</fieldset>';
+	public function layout_form_end() {
+		return EEH_HTML::nl(-1).'</fieldset>';
 	}
 
 
@@ -67,7 +67,7 @@ 
 	/**
 	 * @param string $legend_class
 	 */
-	public function set_legend_class( $legend_class ) {
+	public function set_legend_class($legend_class) {
 		$this->_legend_class = $legend_class;
 	}
 
@@ -85,7 +85,7 @@ 
 	/**
 	 * @param string $legend_text
 	 */
-	public function set_legend_text( $legend_text ) {
+	public function set_legend_text($legend_text) {
 		$this->_legend_text = $legend_text;
 	}
 

admin_pages/messages/help_tabs/messages_overview_messengers.help_tab.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
 <p><strong><?php _e('Messengers', 'event_espresso'); ?></strong></p>
 <p>
-<?php printf( __('Messengers are the vehicles that deliver messages to individuals (recipients). By default, every install of Event Espresso has the %sEmail%s messenger active. The Email messenger is a vehicle for delivering messages.  Other possible vehicles might be any social media applications, 3rd party API\'s etc.', 'event_espresso'), '<strong>', '</strong>' ); ?>
+<?php printf(__('Messengers are the vehicles that deliver messages to individuals (recipients). By default, every install of Event Espresso has the %sEmail%s messenger active. The Email messenger is a vehicle for delivering messages.  Other possible vehicles might be any social media applications, 3rd party API\'s etc.', 'event_espresso'), '<strong>', '</strong>'); ?>
 </p>
\ No newline at end of file

admin_pages/messages/help_tabs/messages_settings_messengers.help_tab.php

Spacing +5 added lines, -5 removed lines

@@ -1,11 +1,11 @@
 <p><strong><?php _e('Activating / Deactivating Messengers', 'event_espresso'); ?></strong></p>
 <p><?php _e('You can select Messengers via the tabs across the top of the settings page. The available messengers you see depends on what version of Event Espresso you have and what addons are installed. Every install include an "Email" messenger tab.  When you click one of those tabs it will display that messenger.', 'event_espresso'); ?></p>
 <p><?php _e('There are two ways to determine whether a messenger is active or not.  The first way is via the messenger tab itself.', 'event_espresso'); ?></p>
-<p><?php echo _e('The green colored gear <img class="inline-text" src="' . EE_MSG_ASSETS_URL . 'images/email-tab-active.png' . '" alt="' . esc_attr__('Active Email Tab', 'event_espresso') . '" /> indicates that this messenger is currently active.', 'event_espresso'); echo _e(' The white colored gear <img class="inline-text" src="' . EE_MSG_ASSETS_URL . 'images/email-tab-inactive.png' . '" alt="' . esc_attr__('Inactive Email Tab', 'event_espresso') . '" /> indicates the messenger is inactive. This is very helpful for seeing at a glance all the messengers that are active when you first view the page.', 'event_espresso'); ?></p>
+<p><?php echo _e('The green colored gear <img class="inline-text" src="'.EE_MSG_ASSETS_URL.'images/email-tab-active.png'.'" alt="'.esc_attr__('Active Email Tab', 'event_espresso').'" /> indicates that this messenger is currently active.', 'event_espresso'); echo _e(' The white colored gear <img class="inline-text" src="'.EE_MSG_ASSETS_URL.'images/email-tab-inactive.png'.'" alt="'.esc_attr__('Inactive Email Tab', 'event_espresso').'" /> indicates the messenger is inactive. This is very helpful for seeing at a glance all the messengers that are active when you first view the page.', 'event_espresso'); ?></p>
 <p><?php _e('The second way to determine whether a messenger is active or not is via the "on/off" button in the top right corner of the active messenger displayed content:', 'event_espresso'); ?></p>
-<p><?php printf( __('%1$s means of course that the messenger is active and %2$s means the messenger is inactive.', 'event_espresso'), '<div class="switch"><input id="ee-on-off-toggle-on" class="ee-on-off-toggle ee-toggle-round-flat" type="checkbox" checked="checked" disabled><label for="ee-on-off-toggle-on"></label></div>', '<div class="switch"><input id="ee-on-off-toggle-on" class="ee-on-off-toggle ee-toggle-round-flat" type="checkbox" disabled><label for="ee-on-off-toggle-on"></label></div>' ); ?></p>
-<p><?php _e( 'The on/off toggle is also what you use to activate or deactivate a messenger.', 'event_espresso' ); ?></p>
-<p><?php _e( 'What happens when you click the toggle to activate is the messenger is activated and the system determines what default message types are activated with the messenger.  Then, if there are any default settings for either the messenger or message types those settings are saved.  Next, the system will generate any default templates (if none have been generated before, if there are previously generated templates then they are reactivated).  Finally, you will see the display change to reflect that the messenger is active. If the messenger has settings you can modify them then. Any message types that have settings will also automatically expand so you can see the default settings and make any changes as necessary to fit your needs. Usually the defaults are sufficient however.', 'event_espresso'); ?></p>
-<p><?php _e( 'When you deactivate a messenger, the system will first check if there are any custom event templates for that messenger. If there are you will be unable to deactivate the messenger. This is a fail safe to make sure you know that no messages will go out for those specific events so you don\'t accidentally deactivate.  If this check passes, then the system will deactivate any global templates for that messenger (note the templates are not erased, they just become inactive, so if you decide to reactivate the messenger later all your customizations are preserved). Then the display will change to reflect the deactivation.', 'event_espresso'); ?></p>
+<p><?php printf(__('%1$s means of course that the messenger is active and %2$s means the messenger is inactive.', 'event_espresso'), '<div class="switch"><input id="ee-on-off-toggle-on" class="ee-on-off-toggle ee-toggle-round-flat" type="checkbox" checked="checked" disabled><label for="ee-on-off-toggle-on"></label></div>', '<div class="switch"><input id="ee-on-off-toggle-on" class="ee-on-off-toggle ee-toggle-round-flat" type="checkbox" disabled><label for="ee-on-off-toggle-on"></label></div>'); ?></p>
+<p><?php _e('The on/off toggle is also what you use to activate or deactivate a messenger.', 'event_espresso'); ?></p>
+<p><?php _e('What happens when you click the toggle to activate is the messenger is activated and the system determines what default message types are activated with the messenger.  Then, if there are any default settings for either the messenger or message types those settings are saved.  Next, the system will generate any default templates (if none have been generated before, if there are previously generated templates then they are reactivated).  Finally, you will see the display change to reflect that the messenger is active. If the messenger has settings you can modify them then. Any message types that have settings will also automatically expand so you can see the default settings and make any changes as necessary to fit your needs. Usually the defaults are sufficient however.', 'event_espresso'); ?></p>
+<p><?php _e('When you deactivate a messenger, the system will first check if there are any custom event templates for that messenger. If there are you will be unable to deactivate the messenger. This is a fail safe to make sure you know that no messages will go out for those specific events so you don\'t accidentally deactivate.  If this check passes, then the system will deactivate any global templates for that messenger (note the templates are not erased, they just become inactive, so if you decide to reactivate the messenger later all your customizations are preserved). Then the display will change to reflect the deactivation.', 'event_espresso'); ?></p>
 <p><strong><?php _e('Important', 'event_espresso'); ?></strong><br />
 <?php _e('Although customizations made to global templates are preserved when a messenger is deactivated, any settings for that messenger (or the message types that were attached to it) are lost on deactivation.  Also, once you deactivate a messenger, no more messages will be delivered using that messenger for any of your events.', 'event_espresso'); ?></p>

admin_pages/messages/templates/ee_msg_details_mt_settings_tab_item.template.php

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <li id="<?php echo $slug_id; ?>" class="ui-widget-content ui-corner-tr mt-tab <?php echo $class; ?>">
-	<div id="<?php echo $slug_id; ?>-handle" class="mt-handlediv no-drag" title="<?php esc_attr_e( 'Click to toggle', 'event_espresso' ); ?>"><br></div>
+	<div id="<?php echo $slug_id; ?>-handle" class="mt-handlediv no-drag" title="<?php esc_attr_e('Click to toggle', 'event_espresso'); ?>"><br></div>
 	<strong class="ui-widget-header"><?php echo $label; ?></strong>
 	<?php echo $content; ?>
 	<span class="mt_nonce hidden"><?php echo $mt_nonce; ?></span>