eventespresso / event-espresso-core

core/libraries/plugin_api/EE_Register_Messages_Shortcode_Library.lib.php

Spacing +29 added lines, -29 removed lines

@@ -62,44 +62,44 @@ 
      * }
      * @return void
      */
-    public static function register( $name = NULL, $setup_args = array() ) {
+    public static function register($name = NULL, $setup_args = array()) {
 
         //required fields MUST be present, so let's make sure they are.
-        if ( empty( $name ) || ! is_array( $setup_args ) || empty( $setup_args['autoloadpaths'] ) ) {
-            throw new EE_Error( __( 'In order to register a messages shortcode library with EE_Register_Messages_Shortcode_Library::register, you must include a "name" (a unique identifier for this set of message shortcodes), and an array containing the following keys: : "autoload_paths"', 'event_espresso' ) );
+        if (empty($name) || ! is_array($setup_args) || empty($setup_args['autoloadpaths'])) {
+            throw new EE_Error(__('In order to register a messages shortcode library with EE_Register_Messages_Shortcode_Library::register, you must include a "name" (a unique identifier for this set of message shortcodes), and an array containing the following keys: : "autoload_paths"', 'event_espresso'));
         }
 
 		//make sure we don't register twice
-		if( isset( self::$_ee_messages_shortcode_registry[ $name ] ) ){
+		if (isset(self::$_ee_messages_shortcode_registry[$name])) {
 			return;
 		}
 
 		//make sure this was called in the right place!
-		if ( ! did_action( 'EE_Brewing_Regular___messages_caf' ) || did_action( 'AHEE__EE_System__perform_activations_upgrades_and_migrations' )) {
-			EE_Error::doing_it_wrong(__METHOD__, sprintf( __('Should be only called on the "EE_Brewing_Regular___messages_caf" hook (Trying to register a library named %s).','event_espresso'), $name ), '4.3.0' );
+		if ( ! did_action('EE_Brewing_Regular___messages_caf') || did_action('AHEE__EE_System__perform_activations_upgrades_and_migrations')) {
+			EE_Error::doing_it_wrong(__METHOD__, sprintf(__('Should be only called on the "EE_Brewing_Regular___messages_caf" hook (Trying to register a library named %s).', 'event_espresso'), $name), '4.3.0');
 		}
 
         $name = (string) $name;
         self::$_ee_messages_shortcode_registry[$name] = array(
             'autoloadpaths' => (array) $setup_args['autoloadpaths'],
-            'list_type_shortcodes' => !empty( $setup_args['list_type_shortcodes'] ) ? (array) $setup_args['list_type_shortcodes'] : array()
+            'list_type_shortcodes' => ! empty($setup_args['list_type_shortcodes']) ? (array) $setup_args['list_type_shortcodes'] : array()
             );
 
          //add filters
-         add_filter( 'FHEE__EED_Messages___set_messages_paths___MSG_PATHS', array( 'EE_Register_Messages_Shortcode_Library', 'register_msgs_autoload_paths'), 10 );
+         add_filter('FHEE__EED_Messages___set_messages_paths___MSG_PATHS', array('EE_Register_Messages_Shortcode_Library', 'register_msgs_autoload_paths'), 10);
 
         //add below filters if the required callback is provided.
-        if ( !empty( $setup_args['msgr_validator_callback'] ) )
-            add_filter( 'FHEE__EE_messenger__get_validator_config', $setup_args['msgr_validator_callback'], 10, 2 );
+        if ( ! empty($setup_args['msgr_validator_callback']))
+            add_filter('FHEE__EE_messenger__get_validator_config', $setup_args['msgr_validator_callback'], 10, 2);
 
-        if ( !empty( $setup_args['msgr_template_fields_callback'] ) )
-            add_filter( 'FHEE__EE_messenger__get_template_fields', $setup_args['msgr_template_fields_callback'], 10, 2 );
+        if ( ! empty($setup_args['msgr_template_fields_callback']))
+            add_filter('FHEE__EE_messenger__get_template_fields', $setup_args['msgr_template_fields_callback'], 10, 2);
 
-        if ( !empty( $setup_args['valid_shortcodes_callback'] ) )
-            add_filter( 'FHEE__EE_Messages_Base__get_valid_shortcodes', $setup_args['valid_shortcodes_callback'], 10, 2 );
+        if ( ! empty($setup_args['valid_shortcodes_callback']))
+            add_filter('FHEE__EE_Messages_Base__get_valid_shortcodes', $setup_args['valid_shortcodes_callback'], 10, 2);
 
-        if ( !empty( $setup_args['list_type_shortcodes'] ) )
-            add_filter( 'FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes', array( 'EE_Register_Messages_Shortcode_Library', 'register_list_type_shortcodes'), 10 );
+        if ( ! empty($setup_args['list_type_shortcodes']))
+            add_filter('FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes', array('EE_Register_Messages_Shortcode_Library', 'register_list_type_shortcodes'), 10);
     }
 
 
@@ -112,9 +112,9 @@ 
      * @param  string $name name used to register the shortcode library.
      * @return  void
      */
-    public static function deregister( $name = NULL ) {
-    	if ( !empty( self::$_ee_messages_shortcode_registry[$name] ) )
-    		unset( self::$_ee_messages_shortcode_registry[$name] );
+    public static function deregister($name = NULL) {
+    	if ( ! empty(self::$_ee_messages_shortcode_registry[$name]))
+    		unset(self::$_ee_messages_shortcode_registry[$name]);
     }
 
 
@@ -127,13 +127,13 @@ 
      * @param array $paths array of paths to be checked by EE_messages autoloader.
      * @return array
      */
-    public static function register_msgs_autoload_paths( $paths  ) {
+    public static function register_msgs_autoload_paths($paths) {
 
-        if ( !empty( self::$_ee_messages_shortcode_registry ) ) {
-             foreach ( self::$_ee_messages_shortcode_registry as $st_reg ) {
-                if ( empty( $st_reg['autoloadpaths'] ) )
+        if ( ! empty(self::$_ee_messages_shortcode_registry)) {
+             foreach (self::$_ee_messages_shortcode_registry as $st_reg) {
+                if (empty($st_reg['autoloadpaths']))
                     continue;
-                $paths = array_merge( $paths, $st_reg['autoloadpaths'] );
+                $paths = array_merge($paths, $st_reg['autoloadpaths']);
             }
         }
 
@@ -151,13 +151,13 @@ 
      * @param  array $original_shortcodes
      * @return  array                                   Modifications to original shortcodes.
      */
-    public static function register_list_type_shortcodes( $original_shortcodes ) {
-        if ( empty( self::$_ee_messages_shortcode_registry ) )
+    public static function register_list_type_shortcodes($original_shortcodes) {
+        if (empty(self::$_ee_messages_shortcode_registry))
             return $original_shortcodes;
 
-        foreach ( self::$_ee_messages_shortcode_registry as $sc_reg ) {
-            if ( !empty( $sc_reg['list_type_shortcodes'] ) )
-                $original_shortcodes = array_merge( $original_shortcodes, $sc_reg['list_type_shortcodes'] );
+        foreach (self::$_ee_messages_shortcode_registry as $sc_reg) {
+            if ( ! empty($sc_reg['list_type_shortcodes']))
+                $original_shortcodes = array_merge($original_shortcodes, $sc_reg['list_type_shortcodes']);
         }
 
         return $original_shortcodes;

Braces +27 added lines, -17 removed lines

@@ -5,7 +5,9 @@ 
  * @subpackage plugin api, messages
  * @since           4.3.0
  */
-if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
+}
 
 /**
  * Use this to register or deregister a new shortcode library for the EE messages system.
@@ -89,17 +91,21 @@ 
          add_filter( 'FHEE__EED_Messages___set_messages_paths___MSG_PATHS', array( 'EE_Register_Messages_Shortcode_Library', 'register_msgs_autoload_paths'), 10 );
 
         //add below filters if the required callback is provided.
-        if ( !empty( $setup_args['msgr_validator_callback'] ) )
-            add_filter( 'FHEE__EE_messenger__get_validator_config', $setup_args['msgr_validator_callback'], 10, 2 );
+        if ( !empty( $setup_args['msgr_validator_callback'] ) ) {
+                    add_filter( 'FHEE__EE_messenger__get_validator_config', $setup_args['msgr_validator_callback'], 10, 2 );
+        }
 
-        if ( !empty( $setup_args['msgr_template_fields_callback'] ) )
-            add_filter( 'FHEE__EE_messenger__get_template_fields', $setup_args['msgr_template_fields_callback'], 10, 2 );
+        if ( !empty( $setup_args['msgr_template_fields_callback'] ) ) {
+                    add_filter( 'FHEE__EE_messenger__get_template_fields', $setup_args['msgr_template_fields_callback'], 10, 2 );
+        }
 
-        if ( !empty( $setup_args['valid_shortcodes_callback'] ) )
-            add_filter( 'FHEE__EE_Messages_Base__get_valid_shortcodes', $setup_args['valid_shortcodes_callback'], 10, 2 );
+        if ( !empty( $setup_args['valid_shortcodes_callback'] ) ) {
+                    add_filter( 'FHEE__EE_Messages_Base__get_valid_shortcodes', $setup_args['valid_shortcodes_callback'], 10, 2 );
+        }
 
-        if ( !empty( $setup_args['list_type_shortcodes'] ) )
-            add_filter( 'FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes', array( 'EE_Register_Messages_Shortcode_Library', 'register_list_type_shortcodes'), 10 );
+        if ( !empty( $setup_args['list_type_shortcodes'] ) ) {
+                    add_filter( 'FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes', array( 'EE_Register_Messages_Shortcode_Library', 'register_list_type_shortcodes'), 10 );
+        }
     }
 
 
@@ -113,8 +119,9 @@ 
      * @return  void
      */
     public static function deregister( $name = NULL ) {
-    	if ( !empty( self::$_ee_messages_shortcode_registry[$name] ) )
-    		unset( self::$_ee_messages_shortcode_registry[$name] );
+    	if ( !empty( self::$_ee_messages_shortcode_registry[$name] ) ) {
+    	    		unset( self::$_ee_messages_shortcode_registry[$name] );
+    	}
     }
 
 
@@ -131,8 +138,9 @@ 
 
         if ( !empty( self::$_ee_messages_shortcode_registry ) ) {
              foreach ( self::$_ee_messages_shortcode_registry as $st_reg ) {
-                if ( empty( $st_reg['autoloadpaths'] ) )
-                    continue;
+                if ( empty( $st_reg['autoloadpaths'] ) ) {
+                                    continue;
+                }
                 $paths = array_merge( $paths, $st_reg['autoloadpaths'] );
             }
         }
@@ -152,12 +160,14 @@ 
      * @return  array                                   Modifications to original shortcodes.
      */
     public static function register_list_type_shortcodes( $original_shortcodes ) {
-        if ( empty( self::$_ee_messages_shortcode_registry ) )
-            return $original_shortcodes;
+        if ( empty( self::$_ee_messages_shortcode_registry ) ) {
+                    return $original_shortcodes;
+        }
 
         foreach ( self::$_ee_messages_shortcode_registry as $sc_reg ) {
-            if ( !empty( $sc_reg['list_type_shortcodes'] ) )
-                $original_shortcodes = array_merge( $original_shortcodes, $sc_reg['list_type_shortcodes'] );
+            if ( !empty( $sc_reg['list_type_shortcodes'] ) ) {
+                            $original_shortcodes = array_merge( $original_shortcodes, $sc_reg['list_type_shortcodes'] );
+            }
         }
 
         return $original_shortcodes;

Indentation +133 added lines, -133 removed lines

@@ -18,56 +18,56 @@ 
 class EE_Register_Messages_Shortcode_Library implements EEI_Plugin_API {
 
 
-    /**
-     * holds values for registered messages shortcode libraries
-     * @var array
-     */
-    protected static $_ee_messages_shortcode_registry = array();
-
-
-
-
-    /**
-     * Helper method for registring a new shortcodes library class for the messages system.
-     *
-     * Note this is not used for adding shortcodes to existing libraries.  It's for registering anything
-     * related to registering a new EE_{shortcode_library_name}_Shortcodes.lib.php class.
-     *
-     * @since    4.3.0
-     *
-     * @param  array  $setup_args {
-     *       An array of arguments provided for registering the new messages shortcode library.
-     *
-     *       @type string $name                                         What is the name of this shortcode library
-     *                                                                              (e.g. 'question_list');
-     *       @type array  $autoloadpaths                          An array of paths to add to the messages
-     *                                                                              autoloader for the new shortcode library
-     *                                                                              class file.
-     *       @type string $msgr_validator_callback            Callback for a method that will register the
-     *                                                                              library with the messenger
-     *                                                                              _validator_config. Optional.
-     *       @type string $msgr_template_fields_callback  Callback for changing adding the
-     *                                                                              _template_fields property for messenger.
-     *                                                                              For example, the shortcode library may add
-     *                                                                              a new field to the message templates.
-     *                                                                              Optional.
-     *       @type string $valid_shortcodes_callback         Callback for message types
-     *                                                                              _valid_shortcodes array setup. Optional.
-     *       @type array  $list_type_shortcodes                 If there are any specific shortcodes with this
-     *                                                                             message shortcode library that should be
-     *                                                                             considered "list type" then include them in an
-     *                                                                             array.  List Type shortcodes are shortcodes that
-     *                                                                             have a corresponding field that indicates how
-     *                                                                             they are parsed. Optional.
-     * }
-     * @return void
-     */
-    public static function register( $name = NULL, $setup_args = array() ) {
-
-        //required fields MUST be present, so let's make sure they are.
-        if ( empty( $name ) || ! is_array( $setup_args ) || empty( $setup_args['autoloadpaths'] ) ) {
-            throw new EE_Error( __( 'In order to register a messages shortcode library with EE_Register_Messages_Shortcode_Library::register, you must include a "name" (a unique identifier for this set of message shortcodes), and an array containing the following keys: : "autoload_paths"', 'event_espresso' ) );
-        }
+	/**
+	 * holds values for registered messages shortcode libraries
+	 * @var array
+	 */
+	protected static $_ee_messages_shortcode_registry = array();
+
+
+
+
+	/**
+	 * Helper method for registring a new shortcodes library class for the messages system.
+	 *
+	 * Note this is not used for adding shortcodes to existing libraries.  It's for registering anything
+	 * related to registering a new EE_{shortcode_library_name}_Shortcodes.lib.php class.
+	 *
+	 * @since    4.3.0
+	 *
+	 * @param  array  $setup_args {
+	 *       An array of arguments provided for registering the new messages shortcode library.
+	 *
+	 *       @type string $name                                         What is the name of this shortcode library
+	 *                                                                              (e.g. 'question_list');
+	 *       @type array  $autoloadpaths                          An array of paths to add to the messages
+	 *                                                                              autoloader for the new shortcode library
+	 *                                                                              class file.
+	 *       @type string $msgr_validator_callback            Callback for a method that will register the
+	 *                                                                              library with the messenger
+	 *                                                                              _validator_config. Optional.
+	 *       @type string $msgr_template_fields_callback  Callback for changing adding the
+	 *                                                                              _template_fields property for messenger.
+	 *                                                                              For example, the shortcode library may add
+	 *                                                                              a new field to the message templates.
+	 *                                                                              Optional.
+	 *       @type string $valid_shortcodes_callback         Callback for message types
+	 *                                                                              _valid_shortcodes array setup. Optional.
+	 *       @type array  $list_type_shortcodes                 If there are any specific shortcodes with this
+	 *                                                                             message shortcode library that should be
+	 *                                                                             considered "list type" then include them in an
+	 *                                                                             array.  List Type shortcodes are shortcodes that
+	 *                                                                             have a corresponding field that indicates how
+	 *                                                                             they are parsed. Optional.
+	 * }
+	 * @return void
+	 */
+	public static function register( $name = NULL, $setup_args = array() ) {
+
+		//required fields MUST be present, so let's make sure they are.
+		if ( empty( $name ) || ! is_array( $setup_args ) || empty( $setup_args['autoloadpaths'] ) ) {
+			throw new EE_Error( __( 'In order to register a messages shortcode library with EE_Register_Messages_Shortcode_Library::register, you must include a "name" (a unique identifier for this set of message shortcodes), and an array containing the following keys: : "autoload_paths"', 'event_espresso' ) );
+		}
 
 		//make sure we don't register twice
 		if( isset( self::$_ee_messages_shortcode_registry[ $name ] ) ){
@@ -79,88 +79,88 @@ 
 			EE_Error::doing_it_wrong(__METHOD__, sprintf( __('Should be only called on the "EE_Brewing_Regular___messages_caf" hook (Trying to register a library named %s).','event_espresso'), $name ), '4.3.0' );
 		}
 
-        $name = (string) $name;
-        self::$_ee_messages_shortcode_registry[$name] = array(
-            'autoloadpaths' => (array) $setup_args['autoloadpaths'],
-            'list_type_shortcodes' => !empty( $setup_args['list_type_shortcodes'] ) ? (array) $setup_args['list_type_shortcodes'] : array()
-            );
-
-         //add filters
-         add_filter( 'FHEE__EED_Messages___set_messages_paths___MSG_PATHS', array( 'EE_Register_Messages_Shortcode_Library', 'register_msgs_autoload_paths'), 10 );
-
-        //add below filters if the required callback is provided.
-        if ( !empty( $setup_args['msgr_validator_callback'] ) )
-            add_filter( 'FHEE__EE_messenger__get_validator_config', $setup_args['msgr_validator_callback'], 10, 2 );
-
-        if ( !empty( $setup_args['msgr_template_fields_callback'] ) )
-            add_filter( 'FHEE__EE_messenger__get_template_fields', $setup_args['msgr_template_fields_callback'], 10, 2 );
-
-        if ( !empty( $setup_args['valid_shortcodes_callback'] ) )
-            add_filter( 'FHEE__EE_Messages_Base__get_valid_shortcodes', $setup_args['valid_shortcodes_callback'], 10, 2 );
-
-        if ( !empty( $setup_args['list_type_shortcodes'] ) )
-            add_filter( 'FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes', array( 'EE_Register_Messages_Shortcode_Library', 'register_list_type_shortcodes'), 10 );
-    }
-
-
-
-
-    /**
-     * This deregisters any messages shortcode library previously registered with the given name.
-     *
-     * @since    4.3.0
-     * @param  string $name name used to register the shortcode library.
-     * @return  void
-     */
-    public static function deregister( $name = NULL ) {
-    	if ( !empty( self::$_ee_messages_shortcode_registry[$name] ) )
-    		unset( self::$_ee_messages_shortcode_registry[$name] );
-    }
-
-
-
-     /**
-     * callback for FHEE__EED_Messages___set_messages_paths___MSG_PATHS filter.
-     *
-     * @since    4.3.0
-     *
-     * @param array $paths array of paths to be checked by EE_messages autoloader.
-     * @return array
-     */
-    public static function register_msgs_autoload_paths( $paths  ) {
-
-        if ( !empty( self::$_ee_messages_shortcode_registry ) ) {
-             foreach ( self::$_ee_messages_shortcode_registry as $st_reg ) {
-                if ( empty( $st_reg['autoloadpaths'] ) )
-                    continue;
-                $paths = array_merge( $paths, $st_reg['autoloadpaths'] );
-            }
-        }
-
-        return $paths;
-    }
-
-
-
-    /**
-     * This is the callback for the FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes
-     * filter which is used to add additional list type shortcodes.
-     *
-     * @since 4.3.0
-     *
-     * @param  array $original_shortcodes
-     * @return  array                                   Modifications to original shortcodes.
-     */
-    public static function register_list_type_shortcodes( $original_shortcodes ) {
-        if ( empty( self::$_ee_messages_shortcode_registry ) )
-            return $original_shortcodes;
-
-        foreach ( self::$_ee_messages_shortcode_registry as $sc_reg ) {
-            if ( !empty( $sc_reg['list_type_shortcodes'] ) )
-                $original_shortcodes = array_merge( $original_shortcodes, $sc_reg['list_type_shortcodes'] );
-        }
-
-        return $original_shortcodes;
-    }
+		$name = (string) $name;
+		self::$_ee_messages_shortcode_registry[$name] = array(
+			'autoloadpaths' => (array) $setup_args['autoloadpaths'],
+			'list_type_shortcodes' => !empty( $setup_args['list_type_shortcodes'] ) ? (array) $setup_args['list_type_shortcodes'] : array()
+			);
+
+		 //add filters
+		 add_filter( 'FHEE__EED_Messages___set_messages_paths___MSG_PATHS', array( 'EE_Register_Messages_Shortcode_Library', 'register_msgs_autoload_paths'), 10 );
+
+		//add below filters if the required callback is provided.
+		if ( !empty( $setup_args['msgr_validator_callback'] ) )
+			add_filter( 'FHEE__EE_messenger__get_validator_config', $setup_args['msgr_validator_callback'], 10, 2 );
+
+		if ( !empty( $setup_args['msgr_template_fields_callback'] ) )
+			add_filter( 'FHEE__EE_messenger__get_template_fields', $setup_args['msgr_template_fields_callback'], 10, 2 );
+
+		if ( !empty( $setup_args['valid_shortcodes_callback'] ) )
+			add_filter( 'FHEE__EE_Messages_Base__get_valid_shortcodes', $setup_args['valid_shortcodes_callback'], 10, 2 );
+
+		if ( !empty( $setup_args['list_type_shortcodes'] ) )
+			add_filter( 'FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes', array( 'EE_Register_Messages_Shortcode_Library', 'register_list_type_shortcodes'), 10 );
+	}
+
+
+
+
+	/**
+	 * This deregisters any messages shortcode library previously registered with the given name.
+	 *
+	 * @since    4.3.0
+	 * @param  string $name name used to register the shortcode library.
+	 * @return  void
+	 */
+	public static function deregister( $name = NULL ) {
+		if ( !empty( self::$_ee_messages_shortcode_registry[$name] ) )
+			unset( self::$_ee_messages_shortcode_registry[$name] );
+	}
+
+
+
+	 /**
+	  * callback for FHEE__EED_Messages___set_messages_paths___MSG_PATHS filter.
+	  *
+	  * @since    4.3.0
+	  *
+	  * @param array $paths array of paths to be checked by EE_messages autoloader.
+	  * @return array
+	  */
+	public static function register_msgs_autoload_paths( $paths  ) {
+
+		if ( !empty( self::$_ee_messages_shortcode_registry ) ) {
+			 foreach ( self::$_ee_messages_shortcode_registry as $st_reg ) {
+				if ( empty( $st_reg['autoloadpaths'] ) )
+					continue;
+				$paths = array_merge( $paths, $st_reg['autoloadpaths'] );
+			}
+		}
+
+		return $paths;
+	}
+
+
+
+	/**
+	 * This is the callback for the FHEE__EEH_Parse_Shortcodes___parse_message_template__list_type_shortcodes
+	 * filter which is used to add additional list type shortcodes.
+	 *
+	 * @since 4.3.0
+	 *
+	 * @param  array $original_shortcodes
+	 * @return  array                                   Modifications to original shortcodes.
+	 */
+	public static function register_list_type_shortcodes( $original_shortcodes ) {
+		if ( empty( self::$_ee_messages_shortcode_registry ) )
+			return $original_shortcodes;
+
+		foreach ( self::$_ee_messages_shortcode_registry as $sc_reg ) {
+			if ( !empty( $sc_reg['list_type_shortcodes'] ) )
+				$original_shortcodes = array_merge( $original_shortcodes, $sc_reg['list_type_shortcodes'] );
+		}
+
+		return $original_shortcodes;
+	}
 
 }

core/libraries/plugin_api/EE_Register_Messages_Template_Pack.lib.php

Indentation +2 added lines, -2 removed lines

@@ -185,7 +185,7 @@
 	 */
 	public static function deregister( $variation_ref = NULL ) {
 		if ( !empty( self::$_registry[$variation_ref] ) ) {
-    			unset( self::$_registry[$variation_ref] );
-    		}
+				unset( self::$_registry[$variation_ref] );
+			}
 	}
 }

Spacing +24 added lines, -24 removed lines

@@ -54,39 +54,39 @@ 
 	 *
 	 * @return void
 	 */
-	public static function register( $ref = NULL, $setup_args = array() ) {
+	public static function register($ref = NULL, $setup_args = array()) {
 
 		//check for required params
-		if ( empty( $ref ) || empty( $setup_args['path'] ) || empty( $setup_args['classname'] ) ) {
+		if (empty($ref) || empty($setup_args['path']) || empty($setup_args['classname'])) {
 			throw new EE_Error(
 				__('In order to register a new template pack for the EE Messages system, you must include a value to reference the template pack being registered and the setup_args must have the path for the new template pack class as well as the classname for the new Template Pack Class. ', 'event_espresso')
 				);
 		}
 
 		//make sure we don't register twice
-		if( isset( self::$_registry[ $ref ] ) ){
+		if (isset(self::$_registry[$ref])) {
 			return;
 		}
 
 		//check that incoming $ref doesn't already exist. If it does then we'll create a unique reference for this template pack.
-		if ( isset( self::$_registry[$ref] ) ) {
-			$ref = uniqid() . '_' . $ref;
+		if (isset(self::$_registry[$ref])) {
+			$ref = uniqid().'_'.$ref;
 		}
 
 
 		//make sure this was called in the right place!
-		 if ( ! did_action( 'EE_Brewing_Regular___messages_caf' ) || did_action( 'AHEE__EE_System__perform_activations_upgrades_and_migrations' )) {
+		 if ( ! did_action('EE_Brewing_Regular___messages_caf') || did_action('AHEE__EE_System__perform_activations_upgrades_and_migrations')) {
 			EE_Error::doing_it_wrong(
 				__METHOD__,
 				sprintf(
-					__('A EE Messages Template Pack given the reference "%s" has been attempted to be registered with the EE Messages System.  It may or may not work because it should be only called on the "EE_Brewing_Regular__messages_caf" hook.','event_espresso'),
+					__('A EE Messages Template Pack given the reference "%s" has been attempted to be registered with the EE Messages System.  It may or may not work because it should be only called on the "EE_Brewing_Regular__messages_caf" hook.', 'event_espresso'),
 					$ref
 				),
 				'4.5.0'
 			);
 		}
 
-		if ( self::_verify_class_not_exist( $setup_args['classname'] ) ) {
+		if (self::_verify_class_not_exist($setup_args['classname'])) {
 			self::$_registry[$ref] = array(
 				'path' => (string) $setup_args['path'],
 				'classname' => (string) $setup_args['classname']
@@ -94,8 +94,8 @@ 
 		}
 
 		//hook into the system
-		add_filter( 'FHEE__EED_Messages___set_messages_paths___MSG_PATHS', array( 'EE_Register_Messages_Template_Pack', 'set_template_pack_path'), 10 );
-		add_filter( 'FHEE__EED_Messages__get_template_packs__template_packs', array( 'EE_Register_Messages_Template_Pack', 'set_template_pack' ), 10 );
+		add_filter('FHEE__EED_Messages___set_messages_paths___MSG_PATHS', array('EE_Register_Messages_Template_Pack', 'set_template_pack_path'), 10);
+		add_filter('FHEE__EED_Messages__get_template_packs__template_packs', array('EE_Register_Messages_Template_Pack', 'set_template_pack'), 10);
 
 	}
 
@@ -112,8 +112,8 @@ 
 	 *
 	 * @return array
 	 */
-	public static function set_template_pack_path( $paths ) {
-		foreach( self::$_registry as $ref => $args ) {
+	public static function set_template_pack_path($paths) {
+		foreach (self::$_registry as $ref => $args) {
 			$paths[] = $args['path'];
 		}
 		return $paths;
@@ -130,15 +130,15 @@ 
 	 * @param EE_Messages_Template_Pack[] $template_packs
 	 * @return EE_Messages_Template_Pack[]
 	 */
-	public static function set_template_pack( $template_packs ) {
-		foreach( self::$_registry as $ref => $args ) {
+	public static function set_template_pack($template_packs) {
+		foreach (self::$_registry as $ref => $args) {
 			//verify class_exists
-			if ( ! class_exists( $args['classname'] ) ) {
-				require_once( $args['path'] . '/' . $args['classname'] . '.class.php' );
+			if ( ! class_exists($args['classname'])) {
+				require_once($args['path'].'/'.$args['classname'].'.class.php');
 			}
 
 			//check again!
-			if ( class_exists( $args['classname'] ) ) {
+			if (class_exists($args['classname'])) {
 				$template_pack = new $args['classname'];
 				$template_packs[$template_pack->dbref] = $template_pack;
 			}
@@ -159,12 +159,12 @@ 
 	 *
 	 * @return bool
 	 */
-	private static function _verify_class_not_exist( $classname ) {
+	private static function _verify_class_not_exist($classname) {
 
 		//loop through the existing registry and see if the classname is already present.
-		foreach ( self::$_registry as $ref => $args ) {
-			if ( $args['classname'] == $classname ) {
-				EE_Error::add_error( sprintf( __('The %s template pack that you just activated cannot be registered with the messages system because there is already a template pack active using the same classname.  Contact the author of this template pack to let them know of the conflict.  To stop seeing this message you will need to deactivate this template pack.', 'event_espresso' ), (string) $setup_args['classname'] ), __FILE__, __LINE__, __FUNCTION__ );
+		foreach (self::$_registry as $ref => $args) {
+			if ($args['classname'] == $classname) {
+				EE_Error::add_error(sprintf(__('The %s template pack that you just activated cannot be registered with the messages system because there is already a template pack active using the same classname.  Contact the author of this template pack to let them know of the conflict.  To stop seeing this message you will need to deactivate this template pack.', 'event_espresso'), (string) $setup_args['classname']), __FILE__, __LINE__, __FUNCTION__);
 				return false;
 			}
 		}
@@ -183,9 +183,9 @@ 
 	 *
 	 * @return void
 	 */
-	public static function deregister( $variation_ref = NULL ) {
-		if ( !empty( self::$_registry[$variation_ref] ) ) {
-    			unset( self::$_registry[$variation_ref] );
+	public static function deregister($variation_ref = NULL) {
+		if ( ! empty(self::$_registry[$variation_ref])) {
+    			unset(self::$_registry[$variation_ref]);
     		}
 	}
 }

Braces +3 added lines, -1 removed lines

@@ -5,7 +5,9 @@
  * @subpackage plugin api, messages
  * @since           4.5.0
  */
-if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
+}
 
 /**
  * Use this to register or deregister a new message template pack for the EE messages system.

core/libraries/plugin_api/EE_Register_Messages_Template_Variations.lib.php

Spacing +41 added lines, -41 removed lines

@@ -68,38 +68,38 @@ 
 	 * @throws EE_Error
 	 * @return void
 	 */
-	public static function register( $variation_ref = NULL,  $setup_args = array() ) {
+	public static function register($variation_ref = NULL, $setup_args = array()) {
 
 		//check for required params
-		if ( empty( $variation_ref ) ) {
+		if (empty($variation_ref)) {
 			throw new EE_Error(
 				__('In order to register variations for a EE_Message_Template_Pack, you must include a value to reference the variations being registered', 'event_espresso')
 				);
 		}
 
-		if ( ! is_array( $setup_args ) || empty( $setup_args['variations'] ) || empty( $setup_args['base_path'] ) || empty( $setup_args['base_url'] ) ) {
+		if ( ! is_array($setup_args) || empty($setup_args['variations']) || empty($setup_args['base_path']) || empty($setup_args['base_url'])) {
 			throw new EE_Error(
 				__('In order to register variations for a EE_Message_Template_Pack, you must include an array containing the following keys: "variations", "base_path", "base_url", "extension"', 'event_espresso')
 				);
 		}
 
 		//make sure we don't register twice
-		if( isset( self::$_registry[ $variation_ref ] ) ){
+		if (isset(self::$_registry[$variation_ref])) {
 			return;
 		}
 
 		//make sure variation ref is unique.
-		if ( isset( self::$_registry[$variation_ref] ) ) {
-			$variation_ref = uniqid() . '_' . $variation_ref;
+		if (isset(self::$_registry[$variation_ref])) {
+			$variation_ref = uniqid().'_'.$variation_ref;
 		}
 
 
 		//make sure this was called in the right place!
-		 if ( ! did_action( 'EE_Brewing_Regular___messages_caf' ) || did_action( 'AHEE__EE_System__perform_activations_upgrades_and_migrations' )) {
+		 if ( ! did_action('EE_Brewing_Regular___messages_caf') || did_action('AHEE__EE_System__perform_activations_upgrades_and_migrations')) {
 			EE_Error::doing_it_wrong(
 				__METHOD__,
 				sprintf(
-					__('Messages Templates Variations given the reference "%s" has been attempted to be registered with the EE Messages Template Pack System.  It may or may not work because it should be only called on the "EE_Brewing_Regular__messages_caf" hook.','event_espresso'),
+					__('Messages Templates Variations given the reference "%s" has been attempted to be registered with the EE Messages Template Pack System.  It may or may not work because it should be only called on the "EE_Brewing_Regular__messages_caf" hook.', 'event_espresso'),
 					$variation_ref
 				),
 				'4.5.0'
@@ -115,11 +115,11 @@ 
 
 
 		//check that no reserved variation names are in use and also checks if there are already existing variation names for a given template pack.  The former will throw an error.  The latter will remove the conflicting variation name but still register the others and will add EE_Error notice.
-		$validated = self::_verify_variations( $variation_ref, $validated );
+		$validated = self::_verify_variations($variation_ref, $validated);
 		self::$_registry[$variation_ref] = $validated;
 
-		add_filter( 'FHEE__EE_Messages_Template_Pack__get_variations', array( 'EE_Register_Messages_Template_Variations', 'get_variations' ), 10, 4 );
-		add_filter( 'FHEE__EE_Messages_Template_Pack__get_variation', array( 'EE_Register_Messages_Template_Variations', 'get_variation' ), 10, 8 );
+		add_filter('FHEE__EE_Messages_Template_Pack__get_variations', array('EE_Register_Messages_Template_Variations', 'get_variations'), 10, 4);
+		add_filter('FHEE__EE_Messages_Template_Pack__get_variation', array('EE_Register_Messages_Template_Variations', 'get_variation'), 10, 8);
 	}
 
 
@@ -136,13 +136,13 @@ 
 	 * @throws EE_Error
 	 * @return bool
 	 */
-	private static function _verify_variations( $variation_ref, $validated_variations ) {
-		foreach ( self::$_registry as $variation_ref => $settings ) {
-			foreach ( $settings['variations'] as $template_pack => $messenger ) {
-				foreach ( $messenger as $all_variations ) {
-					if ( isset( $all_variations['default'] ) ) {
+	private static function _verify_variations($variation_ref, $validated_variations) {
+		foreach (self::$_registry as $variation_ref => $settings) {
+			foreach ($settings['variations'] as $template_pack => $messenger) {
+				foreach ($messenger as $all_variations) {
+					if (isset($all_variations['default'])) {
 						throw new EE_Error(
-							sprintf( __('Variations registered through the EE_Register_Messages_Template_Variations api cannot override the default variation for the default template.  Please check the code registering variations with this reference, "%s" and modify.', 'event_espresso' ), $variation_ref )
+							sprintf(__('Variations registered through the EE_Register_Messages_Template_Variations api cannot override the default variation for the default template.  Please check the code registering variations with this reference, "%s" and modify.', 'event_espresso'), $variation_ref)
 							);
 					}
 				}
@@ -150,13 +150,13 @@ 
 		}
 
 		//is there already a variation registered with a given variation slug?
-		foreach ( $validated_variations['variations'] as $template_pack => $messenger  ) {
-			foreach ( $messenger as $message_type => $variations ) {
-				foreach( $variations as $slug => $label ) {
-					foreach ( self::$_registry as $registered_var => $reg_settings ) {
-						if ( isset( $reg_settings['variations'][$template_pack][$messenger][$message_type][$slug] ) ) {
-							unset( $validated_variations['variations'][$template_pack][$messenger][$message_type][$slug] );
-							EE_Error::add_error( sprintf( __('Unable to register the %s variation for the %s template pack with the %s messenger and %s message_type because a variation with this slug was already registered for this template pack and messenger and message type by an addon using this key %s.', 'event_espresso' ), $label, $template_pack, $messenger, $message_type, $registered_var  ) );
+		foreach ($validated_variations['variations'] as $template_pack => $messenger) {
+			foreach ($messenger as $message_type => $variations) {
+				foreach ($variations as $slug => $label) {
+					foreach (self::$_registry as $registered_var => $reg_settings) {
+						if (isset($reg_settings['variations'][$template_pack][$messenger][$message_type][$slug])) {
+							unset($validated_variations['variations'][$template_pack][$messenger][$message_type][$slug]);
+							EE_Error::add_error(sprintf(__('Unable to register the %s variation for the %s template pack with the %s messenger and %s message_type because a variation with this slug was already registered for this template pack and messenger and message type by an addon using this key %s.', 'event_espresso'), $label, $template_pack, $messenger, $message_type, $registered_var));
 						}
 					}
 				}
@@ -185,15 +185,15 @@ 
 	 *
 	 * @return string                    The path to the requested variation.
 	 */
-	public static function get_variation( $variation_path, $messenger, $message_type, $type, $variation, $file_extension, $url, EE_Messages_Template_Pack $template_pack ) {
+	public static function get_variation($variation_path, $messenger, $message_type, $type, $variation, $file_extension, $url, EE_Messages_Template_Pack $template_pack) {
 
 		//so let's loop through our registered variations and then pull any details matching the request.
-		foreach ( self::$_registry as $registry_slug => $registry_settings ) {
+		foreach (self::$_registry as $registry_slug => $registry_settings) {
 			$base = $url ? $registry_settings['base_url'] : $registry_settings['base_path'];
-			$file_string = $messenger . '_' . $type. '_' . $variation . $file_extension;
+			$file_string = $messenger.'_'.$type.'_'.$variation.$file_extension;
 			//see if this file exists
-			if ( is_readable( $registry_settings['base_path'] . $file_string ) ) {
-				return $base . $file_string;
+			if (is_readable($registry_settings['base_path'].$file_string)) {
+				return $base.$file_string;
 			}
 		}
 
@@ -217,21 +217,21 @@ 
 	 *
 	 * @return array                   new variations array (or existing one if nothing registered)
 	 */
-	public static function get_variations( $variations, $messenger,  $message_type, EE_Messages_Template_Pack $template_pack ) {
+	public static function get_variations($variations, $messenger, $message_type, EE_Messages_Template_Pack $template_pack) {
 		//first let's check if we even have registered variations and get out early.
-		if ( empty( self::$_registry ) ) {
+		if (empty(self::$_registry)) {
 			return $variations;
 		}
 
 		//do we have any new variations for the given messenger, $message_type, and template packs
-		foreach ( self::$_registry as $registry_slug => $registry_settings ) {
+		foreach (self::$_registry as $registry_slug => $registry_settings) {
 			//allow for different conditions.
-			if ( empty( $messenger) ) {
-				$variations =  array_merge( $registry_settings['variations'], $variations );
-			} else if ( ! empty( $messenger ) && empty( $message_type ) && ! empty( $registry_settings['variations'][$template_pack->dbref][$messenger] ) ) {
-				$variations =  array_merge( $registry_settings['variations'][$template_pack->dbref][$messenger], $variations );
-			} else if ( ! empty( $messenger ) && ! empty( $message_type ) && ! empty( $registry_settings['variations'][$template_pack->dbref][$messenger][$message_type] ) ) {
-				$variations = array_merge( $registry_settings['variations'][$template_pack->dbref][$messenger][$message_type], $variations );
+			if (empty($messenger)) {
+				$variations = array_merge($registry_settings['variations'], $variations);
+			} else if ( ! empty($messenger) && empty($message_type) && ! empty($registry_settings['variations'][$template_pack->dbref][$messenger])) {
+				$variations = array_merge($registry_settings['variations'][$template_pack->dbref][$messenger], $variations);
+			} else if ( ! empty($messenger) && ! empty($message_type) && ! empty($registry_settings['variations'][$template_pack->dbref][$messenger][$message_type])) {
+				$variations = array_merge($registry_settings['variations'][$template_pack->dbref][$messenger][$message_type], $variations);
 			}
 		}
 		return $variations;
@@ -250,9 +250,9 @@ 
 	 *
 	 * @return void
 	 */
-	public static function deregister( $variation_ref = NULL ) {
-		if ( !empty( self::$_registry[$variation_ref] ) ) {
-    			unset( self::$_registry[$variation_ref] );
+	public static function deregister($variation_ref = NULL) {
+		if ( ! empty(self::$_registry[$variation_ref])) {
+    			unset(self::$_registry[$variation_ref]);
     		}
 	}
 

Braces +3 added lines, -1 removed lines

@@ -5,7 +5,9 @@
  * @subpackage plugin api, messages
  * @since           4.5.0
  */
-if ( ! defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed');
+if ( ! defined('EVENT_ESPRESSO_VERSION')) {
+	exit('No direct script access allowed');
+}
 
 /**
  * Use this to register or deregister a new message template pack variation for the EE messages system.

Indentation +2 added lines, -2 removed lines

@@ -185,7 +185,7 @@
 	 */
 	public static function deregister( $variation_ref = NULL ) {
 		if ( !empty( self::$_registry[$variation_ref] ) ) {
-    			unset( self::$_registry[$variation_ref] );
-    		}
+				unset( self::$_registry[$variation_ref] );
+			}
 	}
 }

core/libraries/plugin_api/EE_Register_Model.lib.php

Braces +2 added lines, -1 removed lines

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

Indentation +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
 		}
 
 		if ( ! did_action( 'AHEE__EE_System__load_espresso_addons' ) || did_action( 'FHEE__EE_System__parse_model_names' ) || did_action( 'FHEE__EE_System__parse_implemented_model_names' )) {
-            EE_Error::doing_it_wrong(
+			EE_Error::doing_it_wrong(
 				__METHOD__,
 				sprintf(
 					__('An attempt was made to register "%s" as a group models has failed because it was not registered at the correct time.  Please use the "AHEE__EE_System__load_espresso_addons" hook to register models.','event_espresso'),
@@ -63,7 +63,7 @@ 
 				),
 				'4.5'
 			);
-        }
+		}
 		self::$_model_registry[$model_id] = $config;
 
 		if( ( isset( $config['model_paths'] ) && ! isset( $config['class_paths'] ) ) ||

Spacing +40 added lines, -40 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION'))
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('No direct script access allowed');
 
 /**
@@ -43,22 +43,22 @@ 
 	 * }
 	 * @throws EE_Error
 	 */
-	public static function register( $model_id = NULL, $config = array() ){
+	public static function register($model_id = NULL, $config = array()) {
 		//required fields MUST be present, so let's make sure they are.
-		if ( empty( $model_id ) || ! is_array( $config ) || empty( $config['model_paths'] )) {
-			throw new EE_Error( __( 'In order to register Models with EE_Register_Model::register(), you must include a "model_id" (a unique identifier for this set of models), and an array containing the following keys: "model_paths" (an array of full server paths to folders that contain models)', 'event_espresso' ));
+		if (empty($model_id) || ! is_array($config) || empty($config['model_paths'])) {
+			throw new EE_Error(__('In order to register Models with EE_Register_Model::register(), you must include a "model_id" (a unique identifier for this set of models), and an array containing the following keys: "model_paths" (an array of full server paths to folders that contain models)', 'event_espresso'));
 		}
 
 		//make sure we don't register twice
-		if( isset( self::$_model_registry[ $model_id ] ) ){
+		if (isset(self::$_model_registry[$model_id])) {
 			return;
 		}
 
-		if ( ! did_action( 'AHEE__EE_System__load_espresso_addons' ) || did_action( 'FHEE__EE_System__parse_model_names' ) || did_action( 'FHEE__EE_System__parse_implemented_model_names' )) {
+		if ( ! did_action('AHEE__EE_System__load_espresso_addons') || did_action('FHEE__EE_System__parse_model_names') || did_action('FHEE__EE_System__parse_implemented_model_names')) {
             EE_Error::doing_it_wrong(
 				__METHOD__,
 				sprintf(
-					__('An attempt was made to register "%s" as a group models has failed because it was not registered at the correct time.  Please use the "AHEE__EE_System__load_espresso_addons" hook to register models.','event_espresso'),
+					__('An attempt was made to register "%s" as a group models has failed because it was not registered at the correct time.  Please use the "AHEE__EE_System__load_espresso_addons" hook to register models.', 'event_espresso'),
 					$model_id
 				),
 				'4.5'
@@ -66,42 +66,42 @@ 
         }
 		self::$_model_registry[$model_id] = $config;
 
-		if( ( isset( $config['model_paths'] ) && ! isset( $config['class_paths'] ) ) ||
-				( ! isset( $config['model_paths'] ) && isset( $config['class_paths'] ) ) ){
-			throw new EE_Error( sprintf( __( 'You must register both "model_paths" AND "class_paths", not just one or the other You provided %s', 'event_espresso' ), implode(", ", array_keys( $config ) ) ) );
+		if ((isset($config['model_paths']) && ! isset($config['class_paths'])) ||
+				( ! isset($config['model_paths']) && isset($config['class_paths']))) {
+			throw new EE_Error(sprintf(__('You must register both "model_paths" AND "class_paths", not just one or the other You provided %s', 'event_espresso'), implode(", ", array_keys($config))));
 		}
-		if(isset($config['model_paths'])){
+		if (isset($config['model_paths'])) {
 			//make sure they passed in an array
-			if( ! is_array( $config['model_paths'] ) ){
-				$config['model_paths'] = array( $config['model_paths'] );
+			if ( ! is_array($config['model_paths'])) {
+				$config['model_paths'] = array($config['model_paths']);
 			}
 			//we want to add this as a model folder
 			//and autoload them all
 			$class_to_filepath_map = EEH_File::get_contents_of_folders($config['model_paths']);
 			EEH_Autoloader::register_autoloader($class_to_filepath_map);
 			$model_name_to_classname_map = array();
-			foreach(array_keys($class_to_filepath_map) as $classname){
-				$model_name_to_classname_map[str_replace("EEM_","",$classname)] = $classname;
+			foreach (array_keys($class_to_filepath_map) as $classname) {
+				$model_name_to_classname_map[str_replace("EEM_", "", $classname)] = $classname;
 			}
 			self::$_model_name_to_classname_map[$model_id] = $model_name_to_classname_map;
-			add_filter('FHEE__EE_System__parse_model_names', array('EE_Register_Model','add_addon_models'));
-			add_filter('FHEE__EE_System__parse_implemented_model_names', array('EE_Register_Model','add_addon_models'));
-			add_filter('FHEE__EE_Registry__load_model__paths',array('EE_Register_Model','add_model_folders'));
+			add_filter('FHEE__EE_System__parse_model_names', array('EE_Register_Model', 'add_addon_models'));
+			add_filter('FHEE__EE_System__parse_implemented_model_names', array('EE_Register_Model', 'add_addon_models'));
+			add_filter('FHEE__EE_Registry__load_model__paths', array('EE_Register_Model', 'add_model_folders'));
 			unset($config['model_paths']);
 		}
-		if(isset($config['class_paths'])){
+		if (isset($config['class_paths'])) {
 			//make sure they passed in an array
-			if( ! is_array( $config['class_paths'] ) ){
-				$config['class_paths'] = array( $config['class_paths'] );
+			if ( ! is_array($config['class_paths'])) {
+				$config['class_paths'] = array($config['class_paths']);
 			}
 			$class_to_filepath_map = EEH_File::get_contents_of_folders($config['class_paths']);
 			EEH_Autoloader::register_autoloader($class_to_filepath_map);
-			add_filter('FHEE__EE_Registry__load_class__paths',array('EE_Register_Model','add_class_folders'));
+			add_filter('FHEE__EE_Registry__load_class__paths', array('EE_Register_Model', 'add_class_folders'));
 			unset($config['class_paths']);
 		}
-		foreach($config as $unknown_key => $unknown_config){
-			self::deregister( $model_id );
-			throw new EE_Error(sprintf(__("The key '%s' is not a known key for registering a model", "event_espresso"),$unknown_key));
+		foreach ($config as $unknown_key => $unknown_config) {
+			self::deregister($model_id);
+			throw new EE_Error(sprintf(__("The key '%s' is not a known key for registering a model", "event_espresso"), $unknown_key));
 		}
 	}
 
@@ -110,9 +110,9 @@ 
 	 * @param array $core_models
 	 * @return array keys are model names (eg 'Event') and values are their classes (eg 'EE_Event')
 	 */
-	public static function add_addon_models( $core_models = array() ){
-		foreach(self::$_model_name_to_classname_map as $model_name_to_class_map){
-			$core_models = array_merge($core_models,$model_name_to_class_map);
+	public static function add_addon_models($core_models = array()) {
+		foreach (self::$_model_name_to_classname_map as $model_name_to_class_map) {
+			$core_models = array_merge($core_models, $model_name_to_class_map);
 		}
 		return $core_models;
 	}
@@ -121,10 +121,10 @@ 
 	 * @param array $folders
 	 * @return array of folder paths
 	 */
-	public static function add_model_folders( $folders = array() ){
-		foreach(self::$_model_registry as $config){
-			if(isset($config['model_paths'])){
-				$folders = array_merge($folders,$config['model_paths']);
+	public static function add_model_folders($folders = array()) {
+		foreach (self::$_model_registry as $config) {
+			if (isset($config['model_paths'])) {
+				$folders = array_merge($folders, $config['model_paths']);
 			}
 		}
 		return $folders;
@@ -134,10 +134,10 @@ 
 	 * @param array $folders
 	 * @return array of folder paths
 	 */
-	public static function add_class_folders( $folders = array() ){
-		foreach(self::$_model_registry as $config){
-			if(isset($config['class_paths'])){
-				$folders = array_merge( $folders, $config['class_paths'] );
+	public static function add_class_folders($folders = array()) {
+		foreach (self::$_model_registry as $config) {
+			if (isset($config['class_paths'])) {
+				$folders = array_merge($folders, $config['class_paths']);
 			}
 		}
 		return $folders;
@@ -149,10 +149,10 @@ 
 	 * deregister
 	 * @param string $model_id
 	 */
-	public static function deregister( $model_id = NULL ){
-		if( isset( self::$_model_registry[ $model_id ] ) ){
-			unset( self::$_model_registry[ $model_id ] );
-			unset( self::$_model_name_to_classname_map[ $model_id ] );
+	public static function deregister($model_id = NULL) {
+		if (isset(self::$_model_registry[$model_id])) {
+			unset(self::$_model_registry[$model_id]);
+			unset(self::$_model_name_to_classname_map[$model_id]);
 		}
 	}
 }

core/libraries/plugin_api/EE_Register_Widget.lib.php

Spacing +15 added lines, -15 removed lines

@@ -44,34 +44,34 @@ 
 	 * @throws EE_Error
 	 * @return void
 	 */
-	public static function register( $widget_id = NULL, $setup_args = array()  ) {
+	public static function register($widget_id = NULL, $setup_args = array()) {
 
 		//required fields MUST be present, so let's make sure they are.
-		if ( empty( $widget_id ) || ! is_array( $setup_args ) || empty( $setup_args['widget_paths'] )) {
-			throw new EE_Error( __( 'In order to register Widgets with EE_Register_Widget::register(), you must include a "widget_id" (a unique identifier for this set of widgets), and an array containing the following keys: "widget_paths" (an array of full server paths to folders that contain widgets, or to the widget files themselves)', 'event_espresso' ));
+		if (empty($widget_id) || ! is_array($setup_args) || empty($setup_args['widget_paths'])) {
+			throw new EE_Error(__('In order to register Widgets with EE_Register_Widget::register(), you must include a "widget_id" (a unique identifier for this set of widgets), and an array containing the following keys: "widget_paths" (an array of full server paths to folders that contain widgets, or to the widget files themselves)', 'event_espresso'));
 		}
 
 		//make sure we don't register twice
-		if( isset( self::$_settings[ $widget_id ] ) ){
+		if (isset(self::$_settings[$widget_id])) {
 			return;
 		}
 
 
 		//make sure this was called in the right place!
-		if ( ! did_action( 'AHEE__EE_System__load_espresso_addons' ) || did_action( 'AHEE__EE_System__register_shortcodes_modules_and_widgets' )) {
+		if ( ! did_action('AHEE__EE_System__load_espresso_addons') || did_action('AHEE__EE_System__register_shortcodes_modules_and_widgets')) {
 			EE_Error::doing_it_wrong(
 				__METHOD__,
-				__( 'An attempt to register widgets has failed because it was not registered at the correct time.  Please use the "AHEE__EE_System__register_shortcodes_modules_and_widgets" hook to register widgets.','event_espresso'),
+				__('An attempt to register widgets has failed because it was not registered at the correct time.  Please use the "AHEE__EE_System__register_shortcodes_modules_and_widgets" hook to register widgets.', 'event_espresso'),
 				'4.3.0'
 			);
 		}
 		//setup $_settings array from incoming values.
-		self::$_settings[ $widget_id ] = array(
+		self::$_settings[$widget_id] = array(
 			// array of full server paths to any EED_Widgets used by the widget
-			'widget_paths'  => isset( $setup_args['widget_paths'] ) ? (array)$setup_args['widget_paths'] : array(),
+			'widget_paths'  => isset($setup_args['widget_paths']) ? (array) $setup_args['widget_paths'] : array(),
 		);
 		// add to list of widgets to be registered
-		add_filter( 'FHEE__EE_Config__register_widgets__widgets_to_register', array( 'EE_Register_Widget', 'add_widgets' ));
+		add_filter('FHEE__EE_Config__register_widgets__widgets_to_register', array('EE_Register_Widget', 'add_widgets'));
 	}
 
 
@@ -83,9 +83,9 @@ 
 	 * @param array $widgets_to_register  array of paths to all widgets that require registering
 	 * @return array
 	 */
-	public static function add_widgets( $widgets_to_register = array() ) {
-		foreach( self::$_settings as $settings ) {
-			$widgets_to_register = array_merge( $widgets_to_register, $settings['widget_paths'] );
+	public static function add_widgets($widgets_to_register = array()) {
+		foreach (self::$_settings as $settings) {
+			$widgets_to_register = array_merge($widgets_to_register, $settings['widget_paths']);
 		}
 		return $widgets_to_register;
 	}
@@ -101,9 +101,9 @@ 
 	 * @param string  $widget_id the name for the widget that was previously registered
 	 * @return void
 	 */
-	public static function deregister( $widget_id = NULL ) {
-		if ( isset( self::$_settings[ $widget_id ] )) {
-			unset( self::$_settings[ $widget_id ] );
+	public static function deregister($widget_id = NULL) {
+		if (isset(self::$_settings[$widget_id])) {
+			unset(self::$_settings[$widget_id]);
 		}
 	}
 

core/libraries/plugin_api/db/EEE_Base_Class.lib.php

Spacing +21 added lines, -21 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION'))
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('No direct script access allowed');
 
 /**
@@ -47,12 +47,12 @@ 
 	 */
 	protected $_ = NULL;
 
-	public function __construct(){
-		if( ! $this->_model_name_extended){
+	public function __construct() {
+		if ( ! $this->_model_name_extended) {
 			throw new EE_Error(sprintf(__("When declaring a class extension, you must define its _model_name_extended property. It should be a model name like 'Attendee' or 'Event'", "event_espresso")));
 		}
-		if(did_action('AHEE__EE_'.$this->_model_name_extended.'__construct__end')){
-			throw new EE_Error(sprintf(__("Hooked in model object extension '%s' too late! The model object %s has already been used!", "event_espresso"),get_class($this),$this->_model_name_extended));
+		if (did_action('AHEE__EE_'.$this->_model_name_extended.'__construct__end')) {
+			throw new EE_Error(sprintf(__("Hooked in model object extension '%s' too late! The model object %s has already been used!", "event_espresso"), get_class($this), $this->_model_name_extended));
 		}
 		$this->_register_extending_methods();
 	}
@@ -61,13 +61,13 @@ 
 	 * scans the child of EEME_Base for functions starting with ext_, and magically makes them functions on the
 	 * model extended. (Internally uses filters, and the __call magic method)
 	 */
-	protected function _register_extending_methods(){
+	protected function _register_extending_methods() {
 		$all_methods = get_class_methods(get_class($this));
-		foreach($all_methods as $method_name){
-			if(strpos($method_name, self::extending_method_prefix) === 0){
+		foreach ($all_methods as $method_name) {
+			if (strpos($method_name, self::extending_method_prefix) === 0) {
 				$method_name_on_model = str_replace(self::extending_method_prefix, '', $method_name);
 				$callback_name = "FHEE__EE_{$this->_model_name_extended}__$method_name_on_model";
-				add_filter($callback_name,array($this,self::dynamic_callback_method_prefix.$method_name_on_model),10,10);
+				add_filter($callback_name, array($this, self::dynamic_callback_method_prefix.$method_name_on_model), 10, 10);
 			}
 		}
 	}
@@ -75,20 +75,20 @@ 
 	 * scans the child of EEME_Base for functions starting with ext_, and magically REMOVES them as functions on the
 	 * model extended. (Internally uses filters, and the __call magic method)
 	 */
-	public function deregister(){
+	public function deregister() {
 		$all_methods = get_class_methods(get_class($this));
-		foreach($all_methods as $method_name){
-			if(strpos($method_name, self::extending_method_prefix) === 0){
+		foreach ($all_methods as $method_name) {
+			if (strpos($method_name, self::extending_method_prefix) === 0) {
 				$method_name_on_model = str_replace(self::extending_method_prefix, '', $method_name);
 				$callback_name = "FHEE__EE_{$this->_model_name_extended}__$method_name_on_model";
-				remove_filter($callback_name,array($this,self::dynamic_callback_method_prefix.$method_name_on_model),10);
+				remove_filter($callback_name, array($this, self::dynamic_callback_method_prefix.$method_name_on_model), 10);
 			}
 		}
 	}
 
 
-	public function __call($callback_method_name,$args){
-		if(strpos($callback_method_name, self::dynamic_callback_method_prefix) === 0){
+	public function __call($callback_method_name, $args) {
+		if (strpos($callback_method_name, self::dynamic_callback_method_prefix) === 0) {
 			//it's a dynamic callback for a method name
 			$method_called_on_model = str_replace(self::dynamic_callback_method_prefix, '', $callback_method_name);
 			$original_return_val = $args[0];
@@ -96,14 +96,14 @@ 
 			$this->_ = $model_called;
 			$args_provided_to_method_on_model = $args[2];
 			$extending_method = self::extending_method_prefix.$method_called_on_model;
-			if(method_exists($this, $extending_method)){
-				return call_user_func_array(array($this,$extending_method), $args_provided_to_method_on_model);
-			}else{
-				throw new EE_Error(sprintf(__("An odd error occurred. Model '%s' had a method called on it that it didn't recognize. So it passed it onto the model extension '%s' (because it had a function named '%s' which should be able to handle it), but the function '%s' doesnt exist!)", "event_espresso"),$this->_model_name_extended,get_class($this),$extending_method,$extending_method));
+			if (method_exists($this, $extending_method)) {
+				return call_user_func_array(array($this, $extending_method), $args_provided_to_method_on_model);
+			} else {
+				throw new EE_Error(sprintf(__("An odd error occurred. Model '%s' had a method called on it that it didn't recognize. So it passed it onto the model extension '%s' (because it had a function named '%s' which should be able to handle it), but the function '%s' doesnt exist!)", "event_espresso"), $this->_model_name_extended, get_class($this), $extending_method, $extending_method));
 			}
 
-		}else{
-			throw new EE_Error(sprintf(__("There is no method named '%s' on '%s'", "event_espresso"),$callback_method_name,get_class($this)));
+		} else {
+			throw new EE_Error(sprintf(__("There is no method named '%s' on '%s'", "event_espresso"), $callback_method_name, get_class($this)));
 		}
 	}
 

Braces +4 added lines, -3 removed lines

@@ -1,7 +1,8 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION'))
+if (!defined('EVENT_ESPRESSO_VERSION')) {
 	exit('No direct script access allowed');
+}
 
 /**
  *
@@ -98,11 +99,11 @@ 
 			$extending_method = self::extending_method_prefix.$method_called_on_model;
 			if(method_exists($this, $extending_method)){
 				return call_user_func_array(array($this,$extending_method), $args_provided_to_method_on_model);
-			}else{
+			} else{
 				throw new EE_Error(sprintf(__("An odd error occurred. Model '%s' had a method called on it that it didn't recognize. So it passed it onto the model extension '%s' (because it had a function named '%s' which should be able to handle it), but the function '%s' doesnt exist!)", "event_espresso"),$this->_model_name_extended,get_class($this),$extending_method,$extending_method));
 			}
 
-		}else{
+		} else{
 			throw new EE_Error(sprintf(__("There is no method named '%s' on '%s'", "event_espresso"),$callback_method_name,get_class($this)));
 		}
 	}

core/libraries/shortcodes/EE_Email_Shortcodes.lib.php

Braces +8 added lines, -4 removed lines

@@ -1,7 +1,8 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  * Event Espresso
@@ -101,8 +102,9 @@ 
 	private function _get_event_admin_emails( $fancy_headers = TRUE ) {
 
 		if ( !empty( $this->_data->admin_email ) ) {
-			if ( ! $fancy_headers )
-				return $this->_data->admin_email;
+			if ( ! $fancy_headers ) {
+							return $this->_data->admin_email;
+			}
 			return !empty( $this->_data->fname ) ? $this->_data->fname . ' ' . $this->_data->lname . ' <' . $this->_data->admin_email . '>' : EE_Registry::instance()->CFG->organization->get_pretty( 'name' ) . ' <' . $this->_data->admin_email . '>';
 		}
 
@@ -142,7 +144,9 @@ 
 
 		foreach ( $admin_details as $admin ) {
 			//only add an admin email if it is present.
-			if ( empty( $admin->email ) || $admin->email == '' ) continue;
+			if ( empty( $admin->email ) || $admin->email == '' ) {
+				continue;
+			}
 
 			if ( ! $fancy_headers ) {
 				$admin_email[] = $admin->email;

Spacing +29 added lines, -29 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**
@@ -43,19 +43,19 @@ 
 		$this->description = esc_html__('All shortcodes related to emails', 'event_espresso');
 		$this->_shortcodes = array(
 			'[SITE_ADMIN_EMAIL]' => esc_html__('Will be replaced with the admin email for the site that Event Espresso is installed on', 'event_espresso'),
-			'[EVENT_AUTHOR_FORMATTED_EMAIL]' => sprintf( esc_html__('This will be replaced with a properly formatted list of Event Creator emails for the events in a registration. %1$sNOTE:%2$s If the event author has not filled out their WordPress user profile then the organization name will be used as the "From" name.', 'event_espresso'),'<strong>','</strong>' ),
-			'[EVENT_AUTHOR_EMAIL]' => sprintf( esc_html__('This is the same as %1$s shortcode except it is just a list of emails (not fancy headers).', 'event_espresso'), '[EVENT_AUTHOR_FORMATTED_EMAIL]' ),
-			'[CO_FORMATTED_EMAIL]' => esc_html__('This parses to the formatted email address of the organization name set in Your Organization Settings. "My Organization &lt;myorg@email.com&gt;"', 'event_espresso' ),
+			'[EVENT_AUTHOR_FORMATTED_EMAIL]' => sprintf(esc_html__('This will be replaced with a properly formatted list of Event Creator emails for the events in a registration. %1$sNOTE:%2$s If the event author has not filled out their WordPress user profile then the organization name will be used as the "From" name.', 'event_espresso'), '<strong>', '</strong>'),
+			'[EVENT_AUTHOR_EMAIL]' => sprintf(esc_html__('This is the same as %1$s shortcode except it is just a list of emails (not fancy headers).', 'event_espresso'), '[EVENT_AUTHOR_FORMATTED_EMAIL]'),
+			'[CO_FORMATTED_EMAIL]' => esc_html__('This parses to the formatted email address of the organization name set in Your Organization Settings. "My Organization &lt;myorg@email.com&gt;"', 'event_espresso'),
 			'[CO_EMAIL]' => esc_html__('This will parse to the email address only for the organization set in Your Organization Settings.', 'event_espresso'),
-			'[ESPRESSO_ADMIN_FORMATTED_EMAIL]' => esc_html__('This parses to the formatted email address of the organization name set in Your Organization Settings. "My Organization &lt;myorg@email.com&gt;"', 'event_espresso' ),
+			'[ESPRESSO_ADMIN_FORMATTED_EMAIL]' => esc_html__('This parses to the formatted email address of the organization name set in Your Organization Settings. "My Organization &lt;myorg@email.com&gt;"', 'event_espresso'),
 			'[ESPRESSO_ADMIN_EMAIL]' => esc_html__('This parses to the email address only for the organization set in Your Organization Settings page.', 'event_espresso')
 			);
 	}
 
 
-	protected function _parser( $shortcode ) {
+	protected function _parser($shortcode) {
 
-		switch ( $shortcode ) {
+		switch ($shortcode) {
 
 			case '[SITE_ADMIN_EMAIL]' :
 				return $this->_get_site_admin_email();
@@ -66,17 +66,17 @@ 
 				break;
 
 			case '[EVENT_AUTHOR_EMAIL]' :
-				return $this->_get_event_admin_emails( FALSE );
+				return $this->_get_event_admin_emails(FALSE);
 				break;
 
 			case '[CO_FORMATTED_EMAIL]' :
 			case '[ESPRESSO_ADMIN_FORMATTED_EMAIL]' :
-				return EE_Registry::instance()->CFG->organization->get_pretty( 'name' ) . ' <' . EE_Registry::instance()->CFG->organization->get_pretty( 'email' ) . '>';
+				return EE_Registry::instance()->CFG->organization->get_pretty('name').' <'.EE_Registry::instance()->CFG->organization->get_pretty('email').'>';
 				break;
 
 			case '[CO_EMAIL]' :
 			case '[ESPRESSO_ADMIN_EMAIL]' :
-				return EE_Registry::instance()->CFG->organization->get_pretty( 'email' );
+				return EE_Registry::instance()->CFG->organization->get_pretty('email');
 				break;
 
 			default :
@@ -98,12 +98,12 @@ 
 	}
 
 
-	private function _get_event_admin_emails( $fancy_headers = TRUE ) {
+	private function _get_event_admin_emails($fancy_headers = TRUE) {
 
-		if ( !empty( $this->_data->admin_email ) ) {
-			if ( ! $fancy_headers )
+		if ( ! empty($this->_data->admin_email)) {
+			if ( ! $fancy_headers)
 				return $this->_data->admin_email;
-			return !empty( $this->_data->fname ) ? $this->_data->fname . ' ' . $this->_data->lname . ' <' . $this->_data->admin_email . '>' : EE_Registry::instance()->CFG->organization->get_pretty( 'name' ) . ' <' . $this->_data->admin_email . '>';
+			return ! empty($this->_data->fname) ? $this->_data->fname.' '.$this->_data->lname.' <'.$this->_data->admin_email.'>' : EE_Registry::instance()->CFG->organization->get_pretty('name').' <'.$this->_data->admin_email.'>';
 		}
 
 		//k this shortcode has been used else where.  Since we don't know what particular event this is for, let's loop through the events and get an array of event admins for the events.  We'll return the formatted list of admin emails and let the messenger make sure we only pick one if this is for a field that can only have ONE!.
@@ -111,18 +111,18 @@ 
 		$admin_email = array();
 
 		//loop through events and set the list of event_ids to retrieve so we can do ONE query.
-		foreach ( $this->_data->events as $event ) {
+		foreach ($this->_data->events as $event) {
 			$ids[] = $event['ID'];
 		}
 
 		//get all the events
-		$events = EE_Registry::instance()->load_model('Event')->get_all( array(array('EVT_ID' => array('IN', $ids ) ) ) );
+		$events = EE_Registry::instance()->load_model('Event')->get_all(array(array('EVT_ID' => array('IN', $ids))));
 
 		//now loop through each event and setup the details
 		$admin_details = array();
 		$cnt = 0;
-		foreach ( $events as $event ) {
-			$user = get_userdata($event->get('EVT_wp_user') );
+		foreach ($events as $event) {
+			$user = get_userdata($event->get('EVT_wp_user'));
 			$admin_details[$cnt] = new stdClass();
 			$admin_details[$cnt]->email = $user->user_email;
 			$admin_details[$cnt]->first_name = $user->user_firstname;
@@ -131,28 +131,28 @@ 
 		}
 
 		//results?
-		if ( empty($admin_details) || !is_array($admin_details) ) {
+		if (empty($admin_details) || ! is_array($admin_details)) {
 			$msg[] = esc_html__('The admin details could not be retrieved from the database.', 'event_espresso');
-			$msg[] = sprintf( esc_html__('Query: %s', 'event_espresso'), $sql );
-			$msg[] = sprintf( esc_html__('Events Data: %s', 'event_espresso'), var_export($this->_data->events, TRUE) );
-			$msg[] = sprintf( esc_html__('Event IDS: %s', 'event_espresso'), var_export($ids, TRUE) );
-			$msg[] = sprintf( esc_html__('Query Results: %s', 'event_espresso'), var_export($admin_details) );
-			do_action( 'AHEE_log', __FILE__, __FUNCTION__, implode( PHP_EOL, $msg ), 'shortcode_parser' );
+			$msg[] = sprintf(esc_html__('Query: %s', 'event_espresso'), $sql);
+			$msg[] = sprintf(esc_html__('Events Data: %s', 'event_espresso'), var_export($this->_data->events, TRUE));
+			$msg[] = sprintf(esc_html__('Event IDS: %s', 'event_espresso'), var_export($ids, TRUE));
+			$msg[] = sprintf(esc_html__('Query Results: %s', 'event_espresso'), var_export($admin_details));
+			do_action('AHEE_log', __FILE__, __FUNCTION__, implode(PHP_EOL, $msg), 'shortcode_parser');
 		}
 
-		foreach ( $admin_details as $admin ) {
+		foreach ($admin_details as $admin) {
 			//only add an admin email if it is present.
-			if ( empty( $admin->email ) || $admin->email == '' ) continue;
+			if (empty($admin->email) || $admin->email == '') continue;
 
-			if ( ! $fancy_headers ) {
+			if ( ! $fancy_headers) {
 				$admin_email[] = $admin->email;
 				continue;
 			}
 
-			$admin_email[] = !empty( $admin->first_name ) ? $admin->first_name . ' ' . $admin->last_name . ' <' . $admin->email . '>' : EE_Registry::instance()->CFG->organization->get_pretty( 'name' ) . ' <' . $admin->email . '>';
+			$admin_email[] = ! empty($admin->first_name) ? $admin->first_name.' '.$admin->last_name.' <'.$admin->email.'>' : EE_Registry::instance()->CFG->organization->get_pretty('name').' <'.$admin->email.'>';
 		}
 
-		$admin_email = implode( ',', $admin_email );
+		$admin_email = implode(',', $admin_email);
 		return $admin_email;
 	}
 

core/libraries/shortcodes/EE_Event_Author_Shortcodes.lib.php

Braces +5 added lines, -3 removed lines

@@ -1,7 +1,8 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  * Event Espresso
@@ -58,8 +59,9 @@ 
 		//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 )
-			return '';
+		if ( ! $recipient instanceof EE_Messages_Addressee && ! $event instanceof EE_Event ) {
+					return '';
+		}
 
 		switch ( $shortcode ) {
 			case '[EVENT_AUTHOR_FNAME]' :

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/shortcodes/EE_Event_Meta_Shortcodes.lib.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if ( ! defined('EVENT_ESPRESSO_VERSION'))
 	exit('NO direct script access allowed');
 
 /**
@@ -51,19 +51,19 @@ 
 	 * @param  mixed (array|object) $data      incoming data object/array
 	 * @return string            parsed code.
 	 */
-	public function parser( $shortcode, $data, $extra_data = array() ) {
+	public function parser($shortcode, $data, $extra_data = array()) {
 		
 		//all shortcodes will be checked in the post_meta table (assuming the shortcode matches the post_meta key);
-		if ( empty( $this->_data['ID'] ) )
+		if (empty($this->_data['ID']))
 			return ''; //need the event id to do anything!
 
 		$meta = get_post_meta($this->_data['ID'], $shortcode, true);
 
-		return !empty( $meta ) ? $meta : '';
+		return ! empty($meta) ? $meta : '';
 	}
 
 
-	protected function _parser( $shortcode ) {}
+	protected function _parser($shortcode) {}
 
 
 } //end EE_Event_Meta_Shortcodes class
\ No newline at end of file

Braces +6 added lines, -3 removed lines

@@ -1,7 +1,8 @@ 
 <?php
 
-if (!defined('EVENT_ESPRESSO_VERSION') )
+if (!defined('EVENT_ESPRESSO_VERSION') ) {
 	exit('NO direct script access allowed');
+}
 
 /**
  * Event Espresso
@@ -54,8 +55,10 @@ 
 	public function parser( $shortcode, $data, $extra_data = array() ) {
 		
 		//all shortcodes will be checked in the post_meta table (assuming the shortcode matches the post_meta key);
-		if ( empty( $this->_data['ID'] ) )
-			return ''; //need the event id to do anything!
+		if ( empty( $this->_data['ID'] ) ) {
+					return '';
+		}
+		//need the event id to do anything!
 
 		$meta = get_post_meta($this->_data['ID'], $shortcode, true);