eventespresso / event-espresso-core

core/exceptions/InvalidIdentifierException.php

Spacing +5 added lines, -5 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 namespace EventEspresso\core\exceptions;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 /**
  * Class InvalidIdentifierException
@@ -23,8 +23,8 @@ 
 	  * @param int        $code
 	  * @param \Exception $previous
 	  */
-	 public function __construct( $actual, $expected, $message = '', $code = 0, \Exception $previous = null ) {
-		 if ( empty( $message ) ) {
+	 public function __construct($actual, $expected, $message = '', $code = 0, \Exception $previous = null) {
+		 if (empty($message)) {
 			 $message = sprintf(
 				 __(
 					 'The supplied identifier "%1$s" is invalid. A value like "%2$s" was expected.',
@@ -34,7 +34,7 @@ 
 				 $expected
 			 );
 		 }
-		 parent::__construct( $message, $code, $previous );
+		 parent::__construct($message, $code, $previous);
 	 }
 
  }

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/admin/templates/admin_details_legend.template.php

Spacing +7 added lines, -7 removed lines

@@ -2,25 +2,25 @@
 //this displays any legends with an array of $items that are indexed by id for each item and each item itself is an array of 'icon' and 'desc'.
 
 //figure out the columns based on the count of items (we want a max of 6 items per column).
-$per_col = isset( $per_column ) ? $per_column : 5;
+$per_col = isset($per_column) ? $per_column : 5;
 $count = 1;
 ?>
 <div class="ee-list-table-legend-container">
 	<h3><?php _e('Legend', 'event_espresso'); ?></h3>
 	<dl class="alignleft ee-list-table-legend">
-		<?php foreach ( $items as $item => $details ) : ?>
-			<?php if ( $per_col < $count ) : ?>
+		<?php foreach ($items as $item => $details) : ?>
+			<?php if ($per_col < $count) : ?>
 				</dl>
 				<dl class="alignleft ee-list-table-legend">
 			<?php $count = 1; endif; ?>
 			<dt id="ee-legend-item-<?php echo $item; ?>">
-				<?php $class = ! empty( $details['class'] ) ? $details['class'] : 'ee-legend-img-container'; ?>
-				<?php if ( strpos( $details[ 'class' ], '<span' ) !== false ) {
+				<?php $class = ! empty($details['class']) ? $details['class'] : 'ee-legend-img-container'; ?>
+				<?php if (strpos($details['class'], '<span') !== false) {
 					echo $class;
 				} else { ?>
 				<span class="<?php echo $class; ?>">
-					<?php if ( ! empty( $details[ 'icon' ] ) ) : ?>
-						<img src="<?php echo $details[ 'icon' ]; ?>" class="ee-legend-icon" alt="<?php echo esc_attr( $details[ 'desc' ] ); ?>"/>
+					<?php if ( ! empty($details['icon'])) : ?>
+						<img src="<?php echo $details['icon']; ?>" class="ee-legend-icon" alt="<?php echo esc_attr($details['desc']); ?>"/>
 					<?php endif; ?>
 				</span>
 				<?php } ?>

core/admin/templates/admin_details_wrapper.template.php

Spacing +5 added lines, -5 removed lines

@@ -4,7 +4,7 @@ 
 /** @var string|WP_Screen $current_page */
 global $post_type, $post;
 //action for registering metaboxes
-do_action( 'add_meta_boxes', $post_type, $post );
+do_action('add_meta_boxes', $post_type, $post);
 ?>
 <?php if ( ! empty($admin_page_header)) : ?>
 <div id="admin-page-header">
@@ -15,7 +15,7 @@ 
 		<!-- admin-page-header -->
 	<div id="post-body" class="metabox-holder columns-2">
 
-	<?php if ( ! empty( $post_body_content )) : ?>
+	<?php if ( ! empty($post_body_content)) : ?>
 		<div id="post-body-content">
 			<?php echo $post_body_content; ?>
 		</div>
@@ -23,13 +23,13 @@ 
 	<?php endif; ?>
 
 		<div id="postbox-container-1" class="postbox-container">
-			<?php do_meta_boxes( $current_page, 'side', NULL ); ?>
+			<?php do_meta_boxes($current_page, 'side', NULL); ?>
 		</div>
 		<!-- postbox-container-1 -->
 
 		<div id="postbox-container-2" class="postbox-container">
-			<?php do_meta_boxes( $current_page, 'normal', NULL ); ?>
-			<?php do_meta_boxes( $current_page, 'advanced', NULL ); ?>
+			<?php do_meta_boxes($current_page, 'normal', NULL); ?>
+			<?php do_meta_boxes($current_page, 'advanced', NULL); ?>
 		</div>
 		<!-- postbox-container-2 -->
         <div class="clear"></div>

core/services/collections/CollectionDetailsInterface.php

Spacing +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 namespace EventEspresso\core\services\collections;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 

core/services/collections/CollectionDetails.php

Spacing +37 added lines, -37 removed lines

@@ -8,8 +8,8 @@ 
 use EventEspresso\core\services\locators\FqcnLocator;
 use EventEspresso\core\services\locators\LocatorInterface;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 
@@ -170,13 +170,13 @@ 
 		$identifier_callback = '',
 		LocatorInterface $file_locator = null
 	) {
-		$this->setCollectionName( $collection_name );
-		$this->setCollectionInterface( $collection_interface );
-		$this->setCollectionFQCNs( $collection_FQCNs );
-		$this->setCollectionPaths( $collection_paths );
-		$this->setFileMasks( $file_mask );
-		$this->setIdentifierType( $identifier_type );
-		$this->setIdentifierCallback( $identifier_callback );
+		$this->setCollectionName($collection_name);
+		$this->setCollectionInterface($collection_interface);
+		$this->setCollectionFQCNs($collection_FQCNs);
+		$this->setCollectionPaths($collection_paths);
+		$this->setFileMasks($file_mask);
+		$this->setIdentifierType($identifier_type);
+		$this->setIdentifierCallback($identifier_callback);
 		$this->file_locator = $file_locator;
 	}
 
@@ -197,9 +197,9 @@ 
 	 * @param string $collection_interface
 	 * @throws \EventEspresso\core\exceptions\InvalidInterfaceException
 	 */
-	protected function setCollectionInterface( $collection_interface ) {
-		if ( ! ( interface_exists( $collection_interface ) || class_exists( $collection_interface ) ) ) {
-			throw new InvalidInterfaceException( $collection_interface );
+	protected function setCollectionInterface($collection_interface) {
+		if ( ! (interface_exists($collection_interface) || class_exists($collection_interface))) {
+			throw new InvalidInterfaceException($collection_interface);
 		}
 		$this->collection_interface = $collection_interface;
 	}
@@ -225,14 +225,14 @@ 
 	 * @param string $collection_name
 	 * @throws \EventEspresso\core\exceptions\InvalidDataTypeException
 	 */
-	protected function setCollectionName( $collection_name ) {
-		if ( ! is_string( $collection_name ) ) {
-			throw new InvalidDataTypeException( '$collection_name', $collection_name, 'string' );
+	protected function setCollectionName($collection_name) {
+		if ( ! is_string($collection_name)) {
+			throw new InvalidDataTypeException('$collection_name', $collection_name, 'string');
 		}
 		$this->collection_name = str_replace(
 			'-',
 			'_',
-			sanitize_title_with_dashes( $collection_name, '', 'save' )
+			sanitize_title_with_dashes($collection_name, '', 'save')
 		);
 	}
 
@@ -253,7 +253,7 @@ 
 	 * @param string $identifier_type
 	 * @throws \EventEspresso\core\exceptions\InvalidIdentifierException
 	 */
-	protected function setIdentifierType( $identifier_type ) {
+	protected function setIdentifierType($identifier_type) {
 		if (
 			! (
 				$identifier_type === CollectionDetails::ID_CLASS_NAME
@@ -286,9 +286,9 @@ 
 	 * @param string $identifier_callback
 	 * @throws \EventEspresso\core\exceptions\InvalidDataTypeException
 	 */
-	protected function setIdentifierCallback( $identifier_callback = 'identifier' ) {
-		if ( ! is_string( $identifier_callback ) ) {
-			throw new InvalidDataTypeException( '$identifier_callback', $identifier_callback, 'string' );
+	protected function setIdentifierCallback($identifier_callback = 'identifier') {
+		if ( ! is_string($identifier_callback)) {
+			throw new InvalidDataTypeException('$identifier_callback', $identifier_callback, 'string');
 		}
 		$this->identifier_callback = $identifier_callback;
 	}
@@ -313,12 +313,12 @@ 
 	 * @param string $file_mask
 	 * @throws \EventEspresso\core\exceptions\InvalidDataTypeException
 	 */
-	protected function setFileMasks( $file_mask ) {
-		$this->file_mask = ! empty( $file_mask ) ? $file_mask : '*.php';
+	protected function setFileMasks($file_mask) {
+		$this->file_mask = ! empty($file_mask) ? $file_mask : '*.php';
 		// we know our default is a string, so if it's not a string now,
 		// then that means the incoming parameter was something else
-		if ( ! is_string( $this->file_mask ) ) {
-			throw new InvalidDataTypeException( '$file_mask', $this->file_mask, 'string' );
+		if ( ! is_string($this->file_mask)) {
+			throw new InvalidDataTypeException('$file_mask', $this->file_mask, 'string');
 		}
 	}
 
@@ -340,13 +340,13 @@ 
 	 * @throws \EventEspresso\core\exceptions\InvalidClassException
 	 * @throws \EventEspresso\core\exceptions\InvalidDataTypeException
 	 */
-	public function setCollectionFQCNs( $collection_FQCNs ) {
-		foreach ( (array) $collection_FQCNs as $collection_FQCN ) {
-			if ( ! empty( $collection_FQCN ) ) {
-				if ( class_exists( $collection_FQCN ) ) {
+	public function setCollectionFQCNs($collection_FQCNs) {
+		foreach ((array) $collection_FQCNs as $collection_FQCN) {
+			if ( ! empty($collection_FQCN)) {
+				if (class_exists($collection_FQCN)) {
 					$this->collection_FQCNs[] = $collection_FQCN;
 				} else {
-					foreach ( $this->getFQCNsFromPartialNamespace( $collection_FQCN ) as $FQCN ) {
+					foreach ($this->getFQCNsFromPartialNamespace($collection_FQCN) as $FQCN) {
 						$this->collection_FQCNs[] = $FQCN;
 					}
 				}
@@ -363,11 +363,11 @@ 
 	 * @throws \EventEspresso\core\exceptions\InvalidDataTypeException
 	 * @throws \EventEspresso\core\exceptions\InvalidClassException
 	 */
-	protected function getFQCNsFromPartialNamespace( $partial_FQCN ) {
-		if ( ! $this->file_locator instanceof FqcnLocator ) {
+	protected function getFQCNsFromPartialNamespace($partial_FQCN) {
+		if ( ! $this->file_locator instanceof FqcnLocator) {
 			$this->file_locator = new FqcnLocator();
 		}
-		$this->file_locator->locate( $partial_FQCN );
+		$this->file_locator->locate($partial_FQCN);
 		return $this->file_locator->getFQCNs();
 	}
 
@@ -388,11 +388,11 @@ 
 	 * @param string|array $collection_paths
 	 * @throws \EventEspresso\core\exceptions\InvalidFilePathException
 	 */
-	public function setCollectionPaths( $collection_paths ) {
-		foreach ( (array) $collection_paths as $collection_path ) {
-			if ( ! empty( $collection_path ) ) {
-				if ( ! is_readable( $collection_path ) ) {
-					throw new InvalidFilePathException( $collection_path );
+	public function setCollectionPaths($collection_paths) {
+		foreach ((array) $collection_paths as $collection_path) {
+			if ( ! empty($collection_path)) {
+				if ( ! is_readable($collection_path)) {
+					throw new InvalidFilePathException($collection_path);
 				}
 				$this->collection_paths[] = $collection_path;
 			}

core/services/database/TableAnalysis.php

Spacing +18 added lines, -18 removed lines

@@ -24,10 +24,10 @@ 
 	 * @param string $table_name
 	 * @return string $tableName, having ensured it has the wpdb prefix on the front
 	 */
-	public function ensureTableNameHasPrefix( $table_name )
+	public function ensureTableNameHasPrefix($table_name)
 	{
 		global $wpdb;
-		return strpos( $table_name, $wpdb->base_prefix ) === 0 ? $table_name : $wpdb->prefix . $table_name;
+		return strpos($table_name, $wpdb->base_prefix) === 0 ? $table_name : $wpdb->prefix.$table_name;
 	}
 
 
@@ -39,13 +39,13 @@ 
 	 * @param string $table_name
 	 * @return bool
 	 */
-	public function tableIsEmpty( $table_name )
+	public function tableIsEmpty($table_name)
 	{
 		global $wpdb;
-		$table_name = $this->ensureTableNameHasPrefix( $table_name );
-		if ( $this->tableExists( $table_name ) ) {
-			$count = $wpdb->get_var( "SELECT COUNT(*) FROM $table_name" );
-			return absint( $count ) === 0 ? true : false;
+		$table_name = $this->ensureTableNameHasPrefix($table_name);
+		if ($this->tableExists($table_name)) {
+			$count = $wpdb->get_var("SELECT COUNT(*) FROM $table_name");
+			return absint($count) === 0 ? true : false;
 		}
 		return false;
 	}
@@ -60,24 +60,24 @@ 
 	 * @param $table_name
 	 * @return bool
 	 */
-	public function tableExists( $table_name )
+	public function tableExists($table_name)
 	{
 		global $wpdb, $EZSQL_ERROR;
-		$table_name = $this->ensureTableNameHasPrefix( $table_name );
+		$table_name = $this->ensureTableNameHasPrefix($table_name);
 		//ignore if this causes an sql error
 		$old_error = $wpdb->last_error;
 		$old_suppress_errors = $wpdb->suppress_errors();
-		$old_show_errors_value = $wpdb->show_errors( FALSE );
+		$old_show_errors_value = $wpdb->show_errors(FALSE);
 		$ezsql_error_cache = $EZSQL_ERROR;
-		$wpdb->get_results( "SELECT * from $table_name LIMIT 1");
-		$wpdb->show_errors( $old_show_errors_value );
-		$wpdb->suppress_errors( $old_suppress_errors );
+		$wpdb->get_results("SELECT * from $table_name LIMIT 1");
+		$wpdb->show_errors($old_show_errors_value);
+		$wpdb->suppress_errors($old_suppress_errors);
 		$new_error = $wpdb->last_error;
 		$wpdb->last_error = $old_error;
 		$EZSQL_ERROR = $ezsql_error_cache;
 		//if there was a table doesn't exist error
-		if( ! empty( $new_error ) ) {
-			if(
+		if ( ! empty($new_error)) {
+			if (
 				in_array(
 					\EEH_Activation::last_wpdb_error_code(),
 					array(
@@ -87,14 +87,14 @@ 
 					)
 				)
 				||
-				preg_match( '~^Table .* doesn\'t exist~', $new_error ) //in case not using mysql and error codes aren't reliable, just check for this error string
+				preg_match('~^Table .* doesn\'t exist~', $new_error) //in case not using mysql and error codes aren't reliable, just check for this error string
 			) {
 				return false;
 			} else {
 				//log this because that's weird. Just use the normal PHP error log
 				error_log(
 					sprintf(
-						__( 'Event Espresso error detected when checking if table existed: %1$s (it wasn\'t just that the table didn\'t exist either)', 'event_espresso' ),
+						__('Event Espresso error detected when checking if table existed: %1$s (it wasn\'t just that the table didn\'t exist either)', 'event_espresso'),
 					$new_error
 					)
 				);
@@ -124,7 +124,7 @@ 
      *  @type string Index_comment
      * }
      */
-	public function showIndexes($table_name, $index_name){
+	public function showIndexes($table_name, $index_name) {
 	    global $wpdb;
         $table_name = $this->ensureTableNameHasPrefix($table_name);
         $index_exists_query = "SHOW INDEX FROM {$table_name} WHERE Key_name = '{$index_name}'";

Indentation +29 added lines, -29 removed lines

@@ -16,11 +16,11 @@ 
  */
 class TableAnalysis extends \EE_Base {
 
-    /**
-     * The maximum number of characters that can be indexed on a column using utf8mb4 collation,
-     * see https://events.codebasehq.com/redirect?https://make.wordpress.org/core/2015/04/02/the-utf8mb4-upgrade/
-     */
-    const INDEX_COLUMN_SIZE = 191;
+	/**
+	 * The maximum number of characters that can be indexed on a column using utf8mb4 collation,
+	 * see https://events.codebasehq.com/redirect?https://make.wordpress.org/core/2015/04/02/the-utf8mb4-upgrade/
+	 */
+	const INDEX_COLUMN_SIZE = 191;
 	/**
 	 * Returns the table name which will definitely have the wpdb prefix on the front,
 	 * except if it currently has the wpdb->base_prefix on the front, in which case
@@ -111,29 +111,29 @@ 
 
 
 
-    /**
-     * @param $table_name
-     * @param $index_name
-     * @return array of columns used on that index, Each entry is an object with the following properties {
-     *  @type string Table
-     *  @type string Non_unique "0" or "1"
-     *  @type string Key_name
-     *  @type string Seq_in_index
-     *  @type string Column_name
-     *  @type string Collation
-     *  @type string Cardinality
-     *  @type string Sub_part on a column, usually this is just the number of characters from this column to use in indexing
-     *  @type string|null Packed
-     *  @type string Null
-     *  @type string Index_type
-     *  @type string Comment
-     *  @type string Index_comment
-     * }
-     */
+	/**
+	 * @param $table_name
+	 * @param $index_name
+	 * @return array of columns used on that index, Each entry is an object with the following properties {
+	 *  @type string Table
+	 *  @type string Non_unique "0" or "1"
+	 *  @type string Key_name
+	 *  @type string Seq_in_index
+	 *  @type string Column_name
+	 *  @type string Collation
+	 *  @type string Cardinality
+	 *  @type string Sub_part on a column, usually this is just the number of characters from this column to use in indexing
+	 *  @type string|null Packed
+	 *  @type string Null
+	 *  @type string Index_type
+	 *  @type string Comment
+	 *  @type string Index_comment
+	 * }
+	 */
 	public function showIndexes($table_name, $index_name){
-	    global $wpdb;
-        $table_name = $this->ensureTableNameHasPrefix($table_name);
-        $index_exists_query = "SHOW INDEX FROM {$table_name} WHERE Key_name = '{$index_name}'";
-        return $wpdb->get_results($index_exists_query);
-    }
+		global $wpdb;
+		$table_name = $this->ensureTableNameHasPrefix($table_name);
+		$index_exists_query = "SHOW INDEX FROM {$table_name} WHERE Key_name = '{$index_name}'";
+		return $wpdb->get_results($index_exists_query);
+	}
 }

core/services/commands/CommandInterface.php

Spacing +2 added lines, -2 removed lines

@@ -1,8 +1,8 @@
 <?php
 namespace EventEspresso\core\services\commands;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }
 
 

core/services/commands/registration/SingleRegistrationCommand.php

Spacing +2 added lines, -2 removed lines

@@ -3,8 +3,8 @@
 
 use EventEspresso\core\services\commands\Command;
 
-if ( ! defined( 'EVENT_ESPRESSO_VERSION' ) ) {
-	exit( 'No direct script access allowed' );
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
 }