eventespresso / event-espresso-core

core/services/blocks/BlockRendererInterface.php

Indentation +7 added lines, -7 removed lines

@@ -14,11 +14,11 @@
 interface BlockRendererInterface
 {
 
-    /**
-     * This receives an array of attributes and returns rendered content for the block using those attributes.
-     *
-     * @param array $attributes
-     * @return string Rendered Content
-     */
-    public function render(array $attributes);
+	/**
+	 * This receives an array of attributes and returns rendered content for the block using those attributes.
+	 *
+	 * @param array $attributes
+	 * @return string Rendered Content
+	 */
+	public function render(array $attributes);
 }

core/services/blocks/BlockRenderer.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      */
     private function setTemplateRootPath()
     {
-        $this->template_root_path = $this->domain->pluginPath() . 'ui/blocks/';
+        $this->template_root_path = $this->domain->pluginPath().'ui/blocks/';
     }
 
 

Indentation +44 added lines, -44 removed lines

@@ -15,57 +15,57 @@
 abstract class BlockRenderer implements BlockRendererInterface
 {
 
-    /**
-     * @var DomainInterface
-     */
-    protected $domain;
+	/**
+	 * @var DomainInterface
+	 */
+	protected $domain;
 
-    /**
-     * @var string
-     */
-    private $template_root_path;
+	/**
+	 * @var string
+	 */
+	private $template_root_path;
 
 
-    /**
-     * BlockRenderer constructor.
-     *
-     * @param DomainInterface $domain
-     */
-    public function __construct(DomainInterface $domain)
-    {
-        $this->domain = $domain;
-        $this->setTemplateRootPath();
-    }
+	/**
+	 * BlockRenderer constructor.
+	 *
+	 * @param DomainInterface $domain
+	 */
+	public function __construct(DomainInterface $domain)
+	{
+		$this->domain = $domain;
+		$this->setTemplateRootPath();
+	}
 
 
-    /**
-     * Sets the root path to the main block template.
-     */
-    private function setTemplateRootPath()
-    {
-        $this->template_root_path = $this->domain->pluginPath() . 'ui/blocks/';
-    }
+	/**
+	 * Sets the root path to the main block template.
+	 */
+	private function setTemplateRootPath()
+	{
+		$this->template_root_path = $this->domain->pluginPath() . 'ui/blocks/';
+	}
 
 
-    /**
-     * Exposes the root path for the main block template.
-     * @return string
-     */
-    public function templateRootPath()
-    {
-        return $this->template_root_path;
-    }
+	/**
+	 * Exposes the root path for the main block template.
+	 * @return string
+	 */
+	public function templateRootPath()
+	{
+		return $this->template_root_path;
+	}
 
 
-    /**
-     * converts GraphQL GUID into EE DB ID
-     *
-     * @param string $GUID
-     * @return int
-     */
-    protected function parseGUID($GUID)
-    {
-        $parts = Relay::fromGlobalId($GUID);
-        return ! empty($parts['id']) ? $parts['id'] : 0;
-    }
+	/**
+	 * converts GraphQL GUID into EE DB ID
+	 *
+	 * @param string $GUID
+	 * @return int
+	 */
+	protected function parseGUID($GUID)
+	{
+		$parts = Relay::fromGlobalId($GUID);
+		return ! empty($parts['id']) ? $parts['id'] : 0;
+	}
 }

core/EE_Network_Config.core.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
     public static function instance()
     {
         // check if class object is instantiated, and instantiated properly
-        if (! self::$_instance instanceof EE_Network_Config) {
+        if ( ! self::$_instance instanceof EE_Network_Config) {
             self::$_instance = new self();
         }
         return self::$_instance;
@@ -136,7 +136,7 @@ 
         // need to bust cache for comparing original if this is a multisite install
         if (is_multisite()) {
             global $current_site;
-            $cache_key = $current_site->id . ':ee_network_config';
+            $cache_key = $current_site->id.':ee_network_config';
             wp_cache_delete($cache_key, 'site-options');
         }
 

Indentation +197 added lines, -197 removed lines

@@ -14,180 +14,180 @@ 
 final class EE_Network_Config
 {
 
-    /**
-     * @var EE_Network_Config $_instance
-     */
-    private static $_instance;
-
-    /**
-     * addons can add their specific network_config objects to this property
-     *
-     * @var EE_Config_Base[] $addons
-     */
-    public $addons;
-
-    /**
-     * @var EE_Network_Core_Config $core
-     */
-    public $core;
-
-
-    /**
-     * @singleton method used to instantiate class object
-     * @return EE_Network_Config instance
-     */
-    public static function instance()
-    {
-        // check if class object is instantiated, and instantiated properly
-        if (! self::$_instance instanceof EE_Network_Config) {
-            self::$_instance = new self();
-        }
-        return self::$_instance;
-    }
-
-
-    /**
-     * class constructor
-     */
-    private function __construct()
-    {
-        do_action('AHEE__EE_Network_Config__construct__begin', $this);
-        // set defaults
-        $this->core = apply_filters('FHEE__EE_Network_Config___construct__core', new EE_Network_Core_Config());
-        $this->addons = array();
-
-        $this->_load_config();
-
-        // construct__end hook
-        do_action('AHEE__EE_Network_Config__construct__end', $this);
-    }
-
-
-    /**
-     * load EE Network Config options
-     *
-     * @return void
-     */
-    private function _load_config()
-    {
-        // load network config start hook
-        do_action('AHEE__EE_Network_Config___load_config__start', $this);
-        $config = $this->get_config();
-        foreach ($config as $config_prop => $settings) {
-            if ($config_prop === 'core' && ! $settings instanceof EE_Network_Core_Config) {
-                $core = new EE_Network_Core_Config();
-                foreach ($settings as $prop => $setting) {
-                    if (property_exists($core, $prop)) {
-                        $core->{$prop} = $setting;
-                    }
-                }
-                $settings = $core;
-                add_filter('FHEE__EE_Network_Config___load_config__update_network_config', '__return_true');
-            }
-            if (is_object($settings) && property_exists($this, $config_prop)) {
-                $this->{$config_prop} = apply_filters(
-                    'FHEE__EE_Network_Config___load_config__config_settings',
-                    $settings,
-                    $config_prop,
-                    $this
-                );
-                if (method_exists($settings, 'populate')) {
-                    $this->{$config_prop}->populate();
-                }
-                if (method_exists($settings, 'do_hooks')) {
-                    $this->{$config_prop}->do_hooks();
-                }
-            }
-        }
-        if (apply_filters('FHEE__EE_Network_Config___load_config__update_network_config', false)) {
-            $this->update_config();
-        }
-
-        // load network config end hook
-        do_action('AHEE__EE_Network_Config___load_config__end', $this);
-    }
-
-
-    /**
-     * get_config
-     *
-     * @return array of network config stuff
-     */
-    public function get_config()
-    {
-        // grab network configuration
-        $CFG = get_site_option('ee_network_config', array());
-        $CFG = apply_filters('FHEE__EE_Network_Config__get_config__CFG', $CFG);
-        return $CFG;
-    }
-
-
-    /**
-     * update_config
-     *
-     * @param bool $add_success
-     * @param bool $add_error
-     * @return bool success
-     */
-    public function update_config($add_success = false, $add_error = true)
-    {
-        do_action('AHEE__EE_Network_Config__update_config__begin', $this);
-
-        // need to bust cache for comparing original if this is a multisite install
-        if (is_multisite()) {
-            global $current_site;
-            $cache_key = $current_site->id . ':ee_network_config';
-            wp_cache_delete($cache_key, 'site-options');
-        }
-
-        // we have to compare existing saved config with config in memory because if there is no difference that means
-        // that the method executed fine but there just was no update.  WordPress doesn't distinguish between false because
-        // there were 0 records updated because of no change vs false because some error produced problems with the update.
-        $original = get_site_option('ee_network_config');
-
-        if ($original == $this) {
-            return true;
-        }
-        // update
-        $saved = update_site_option('ee_network_config', $this);
-
-        do_action('AHEE__EE_Network_Config__update_config__end', $this, $saved);
-        // if config remains the same or was updated successfully
-        if ($saved) {
-            if ($add_success) {
-                $msg = is_multisite() ? esc_html__(
-                    'The Event Espresso Network Configuration Settings have been successfully updated.',
-                    'event_espresso'
-                ) : esc_html__('Extra Event Espresso Configuration settings were successfully updated.', 'event_espresso');
-                EE_Error::add_success($msg);
-            }
-            return true;
-        }
-        if ($add_error) {
-            $msg = is_multisite() ? esc_html__(
-                'The Event Espresso Network Configuration Settings were not updated.',
-                'event_espresso'
-            ) : esc_html__('Extra Event Espresso Network Configuration settings were not updated.', 'event_espresso');
-            EE_Error::add_error($msg, __FILE__, __FUNCTION__, __LINE__);
-        }
-        return false;
-    }
-
-
-    /**
-     * __sleep
-     *
-     * @return array
-     */
-    public function __sleep()
-    {
-        return apply_filters(
-            'FHEE__EE_Network_Config__sleep',
-            array(
-                'core',
-            )
-        );
-    }
+	/**
+	 * @var EE_Network_Config $_instance
+	 */
+	private static $_instance;
+
+	/**
+	 * addons can add their specific network_config objects to this property
+	 *
+	 * @var EE_Config_Base[] $addons
+	 */
+	public $addons;
+
+	/**
+	 * @var EE_Network_Core_Config $core
+	 */
+	public $core;
+
+
+	/**
+	 * @singleton method used to instantiate class object
+	 * @return EE_Network_Config instance
+	 */
+	public static function instance()
+	{
+		// check if class object is instantiated, and instantiated properly
+		if (! self::$_instance instanceof EE_Network_Config) {
+			self::$_instance = new self();
+		}
+		return self::$_instance;
+	}
+
+
+	/**
+	 * class constructor
+	 */
+	private function __construct()
+	{
+		do_action('AHEE__EE_Network_Config__construct__begin', $this);
+		// set defaults
+		$this->core = apply_filters('FHEE__EE_Network_Config___construct__core', new EE_Network_Core_Config());
+		$this->addons = array();
+
+		$this->_load_config();
+
+		// construct__end hook
+		do_action('AHEE__EE_Network_Config__construct__end', $this);
+	}
+
+
+	/**
+	 * load EE Network Config options
+	 *
+	 * @return void
+	 */
+	private function _load_config()
+	{
+		// load network config start hook
+		do_action('AHEE__EE_Network_Config___load_config__start', $this);
+		$config = $this->get_config();
+		foreach ($config as $config_prop => $settings) {
+			if ($config_prop === 'core' && ! $settings instanceof EE_Network_Core_Config) {
+				$core = new EE_Network_Core_Config();
+				foreach ($settings as $prop => $setting) {
+					if (property_exists($core, $prop)) {
+						$core->{$prop} = $setting;
+					}
+				}
+				$settings = $core;
+				add_filter('FHEE__EE_Network_Config___load_config__update_network_config', '__return_true');
+			}
+			if (is_object($settings) && property_exists($this, $config_prop)) {
+				$this->{$config_prop} = apply_filters(
+					'FHEE__EE_Network_Config___load_config__config_settings',
+					$settings,
+					$config_prop,
+					$this
+				);
+				if (method_exists($settings, 'populate')) {
+					$this->{$config_prop}->populate();
+				}
+				if (method_exists($settings, 'do_hooks')) {
+					$this->{$config_prop}->do_hooks();
+				}
+			}
+		}
+		if (apply_filters('FHEE__EE_Network_Config___load_config__update_network_config', false)) {
+			$this->update_config();
+		}
+
+		// load network config end hook
+		do_action('AHEE__EE_Network_Config___load_config__end', $this);
+	}
+
+
+	/**
+	 * get_config
+	 *
+	 * @return array of network config stuff
+	 */
+	public function get_config()
+	{
+		// grab network configuration
+		$CFG = get_site_option('ee_network_config', array());
+		$CFG = apply_filters('FHEE__EE_Network_Config__get_config__CFG', $CFG);
+		return $CFG;
+	}
+
+
+	/**
+	 * update_config
+	 *
+	 * @param bool $add_success
+	 * @param bool $add_error
+	 * @return bool success
+	 */
+	public function update_config($add_success = false, $add_error = true)
+	{
+		do_action('AHEE__EE_Network_Config__update_config__begin', $this);
+
+		// need to bust cache for comparing original if this is a multisite install
+		if (is_multisite()) {
+			global $current_site;
+			$cache_key = $current_site->id . ':ee_network_config';
+			wp_cache_delete($cache_key, 'site-options');
+		}
+
+		// we have to compare existing saved config with config in memory because if there is no difference that means
+		// that the method executed fine but there just was no update.  WordPress doesn't distinguish between false because
+		// there were 0 records updated because of no change vs false because some error produced problems with the update.
+		$original = get_site_option('ee_network_config');
+
+		if ($original == $this) {
+			return true;
+		}
+		// update
+		$saved = update_site_option('ee_network_config', $this);
+
+		do_action('AHEE__EE_Network_Config__update_config__end', $this, $saved);
+		// if config remains the same or was updated successfully
+		if ($saved) {
+			if ($add_success) {
+				$msg = is_multisite() ? esc_html__(
+					'The Event Espresso Network Configuration Settings have been successfully updated.',
+					'event_espresso'
+				) : esc_html__('Extra Event Espresso Configuration settings were successfully updated.', 'event_espresso');
+				EE_Error::add_success($msg);
+			}
+			return true;
+		}
+		if ($add_error) {
+			$msg = is_multisite() ? esc_html__(
+				'The Event Espresso Network Configuration Settings were not updated.',
+				'event_espresso'
+			) : esc_html__('Extra Event Espresso Network Configuration settings were not updated.', 'event_espresso');
+			EE_Error::add_error($msg, __FILE__, __FUNCTION__, __LINE__);
+		}
+		return false;
+	}
+
+
+	/**
+	 * __sleep
+	 *
+	 * @return array
+	 */
+	public function __sleep()
+	{
+		return apply_filters(
+			'FHEE__EE_Network_Config__sleep',
+			array(
+				'core',
+			)
+		);
+	}
 }
 
 
@@ -197,27 +197,27 @@ 
 class EE_Network_Core_Config extends EE_Config_Base
 {
 
-    /**
-     * PUE site license key
-     *
-     * @var string $site_license_key
-     */
-    public $site_license_key;
-
-    /**
-     * This indicates whether messages system processing should be done on the same request or not.
-     *
-     * @var boolean $do_messages_on_same_request
-     */
-    public $do_messages_on_same_request;
-
-
-    /**
-     * EE_Network_Core_Config constructor.
-     */
-    public function __construct()
-    {
-        $this->site_license_key = '';
-        $this->do_messages_on_same_request = false;
-    }
+	/**
+	 * PUE site license key
+	 *
+	 * @var string $site_license_key
+	 */
+	public $site_license_key;
+
+	/**
+	 * This indicates whether messages system processing should be done on the same request or not.
+	 *
+	 * @var boolean $do_messages_on_same_request
+	 */
+	public $do_messages_on_same_request;
+
+
+	/**
+	 * EE_Network_Core_Config constructor.
+	 */
+	public function __construct()
+	{
+		$this->site_license_key = '';
+		$this->do_messages_on_same_request = false;
+	}
 }

core/services/assets/BlockAssetManagerInterface.php

Indentation +38 added lines, -38 removed lines

@@ -12,42 +12,42 @@
  */
 interface BlockAssetManagerInterface
 {
-    /**
-     * @since 4.9.71.p
-     * @return string
-     */
-    public function assetNamespace();
-
-    /**
-     * @since 4.9.71.p
-     * @return void
-     */
-    public function setAssetHandles();
-
-    /**
-     * @since 4.9.71.p
-     * @return string
-     */
-    public function getEditorScriptHandle();
-
-
-    /**
-     * @since 4.9.71.p
-     * @return string
-     */
-    public function getEditorStyleHandle();
-
-
-    /**
-     * @since 4.9.71.p
-     * @return string
-     */
-    public function getScriptHandle();
-
-
-    /**
-     * @since 4.9.71.p
-     * @return string
-     */
-    public function getStyleHandle();
+	/**
+	 * @since 4.9.71.p
+	 * @return string
+	 */
+	public function assetNamespace();
+
+	/**
+	 * @since 4.9.71.p
+	 * @return void
+	 */
+	public function setAssetHandles();
+
+	/**
+	 * @since 4.9.71.p
+	 * @return string
+	 */
+	public function getEditorScriptHandle();
+
+
+	/**
+	 * @since 4.9.71.p
+	 * @return string
+	 */
+	public function getEditorStyleHandle();
+
+
+	/**
+	 * @since 4.9.71.p
+	 * @return string
+	 */
+	public function getScriptHandle();
+
+
+	/**
+	 * @since 4.9.71.p
+	 * @return string
+	 */
+	public function getStyleHandle();
 }
\ No newline at end of file

core/services/dependencies/DependencyResolverInterface.php

Indentation +11 added lines, -11 removed lines

@@ -15,16 +15,16 @@
  */
 interface DependencyResolverInterface
 {
-    /**
-     * Used to configure and/or setup any aliases or recursions required by the DependencyResolver
-     *
-     * @since 4.9.71.p
-     */
-    public function initialize();
+	/**
+	 * Used to configure and/or setup any aliases or recursions required by the DependencyResolver
+	 *
+	 * @since 4.9.71.p
+	 */
+	public function initialize();
 
-    /**
-     * @param string $fqcn Fully Qualified Class Name
-     * @since 4.9.71.p
-     */
-    public function resolveDependenciesForClass($fqcn);
+	/**
+	 * @param string $fqcn Fully Qualified Class Name
+	 * @since 4.9.71.p
+	 */
+	public function resolveDependenciesForClass($fqcn);
 }

core/libraries/rest_api/RestIncomingQueryParamContext.php

Indentation +51 added lines, -51 removed lines

@@ -18,61 +18,61 @@
  */
 class RestIncomingQueryParamContext
 {
-    /**
-     * @var EEM_Base
-     */
-    private $model;
-    /**
-     * @var string
-     */
-    private $requested_version;
-    /**
-     * @var boolean
-     */
-    private $writing;
+	/**
+	 * @var EEM_Base
+	 */
+	private $model;
+	/**
+	 * @var string
+	 */
+	private $requested_version;
+	/**
+	 * @var boolean
+	 */
+	private $writing;
 
-    /**
-     * RestIncomingQueryParamContext constructor.
-     * @param EEM_Base $model
-     * @param string $requested_version
-     * @param boolean $writing
-     */
-    public function __construct(EEM_Base $model, $requested_version, $writing)
-    {
-        $this->model = $model;
-        $this->requested_version = (string) $requested_version;
-        $this->writing = filter_var($writing, FILTER_VALIDATE_BOOLEAN);
-    }
+	/**
+	 * RestIncomingQueryParamContext constructor.
+	 * @param EEM_Base $model
+	 * @param string $requested_version
+	 * @param boolean $writing
+	 */
+	public function __construct(EEM_Base $model, $requested_version, $writing)
+	{
+		$this->model = $model;
+		$this->requested_version = (string) $requested_version;
+		$this->writing = filter_var($writing, FILTER_VALIDATE_BOOLEAN);
+	}
 
-    /**
-     * Gets the model currently being requested, eg class EEM_Event
-     * @since 4.9.72.p
-     * @return EEM_Base
-     */
-    public function getModel()
-    {
-        return $this->model;
-    }
+	/**
+	 * Gets the model currently being requested, eg class EEM_Event
+	 * @since 4.9.72.p
+	 * @return EEM_Base
+	 */
+	public function getModel()
+	{
+		return $this->model;
+	}
 
-    /**
-     * Gets the version being requested, eg 4.8.36
-     * @since 4.9.72.p
-     * @return string
-     */
-    public function getRequestedVersion()
-    {
-        return $this->requested_version;
-    }
+	/**
+	 * Gets the version being requested, eg 4.8.36
+	 * @since 4.9.72.p
+	 * @return string
+	 */
+	public function getRequestedVersion()
+	{
+		return $this->requested_version;
+	}
 
-    /**
-     * Gets if the current request is for a writing request or just simple read
-     * @since 4.9.72.p
-     * @return bool
-     */
-    public function isWriting()
-    {
-        return $this->writing;
-    }
+	/**
+	 * Gets if the current request is for a writing request or just simple read
+	 * @since 4.9.72.p
+	 * @return bool
+	 */
+	public function isWriting()
+	{
+		return $this->writing;
+	}
 }
 // End of file RestIncomingQueryParamContext.php
 // Location: EventEspresso\core\libraries\rest_api/RestIncomingQueryParamContext.php

core/wordpress-shims.php

Spacing +9 added lines, -9 removed lines

@@ -4,7 +4,7 @@ 
  * Note: this file should only be required right before calling the function the shim is for.  This is to ensure that
  * it does not override any existing definition of the function in WP.
  */
-if (! function_exists('get_preview_post_link')) {
+if ( ! function_exists('get_preview_post_link')) {
     /**
      * Function was added in WordPress 4.4.0
      *
@@ -16,13 +16,13 @@ 
     function get_preview_post_link($post = null, $query_args = array(), $preview_link = '')
     {
         $post = get_post($post);
-        if (! $post) {
+        if ( ! $post) {
             return '';
         }
 
         $post_type_object = get_post_type_object($post->post_type);
         if (is_post_type_viewable($post_type_object)) {
-            if (! $preview_link) {
+            if ( ! $preview_link) {
                 $preview_link = set_url_scheme(get_permalink($post));
             }
 
@@ -42,12 +42,12 @@ 
     }
 }
 
-if (! function_exists('is_post_type_viewable')) {
+if ( ! function_exists('is_post_type_viewable')) {
     function is_post_type_viewable($post_type)
     {
         if (is_scalar($post_type)) {
             $post_type = get_post_type_object($post_type);
-            if (! $post_type) {
+            if ( ! $post_type) {
                 return false;
             }
         }
@@ -56,7 +56,7 @@ 
     }
 }
 
-if (! function_exists('wp_scripts_get_suffix')) {
+if ( ! function_exists('wp_scripts_get_suffix')) {
     /**
      * Returns the suffix that can be used for the scripts.
      *
@@ -73,11 +73,11 @@ 
         static $suffixes;
 
         if ($suffixes === null) {
-            include(ABSPATH . WPINC . '/version.php'); // include an unmodified $wp_version
+            include(ABSPATH.WPINC.'/version.php'); // include an unmodified $wp_version
 
             $develop_src = false !== strpos($wp_version, '-src');
 
-            if (! defined('SCRIPT_DEBUG')) {
+            if ( ! defined('SCRIPT_DEBUG')) {
                 define('SCRIPT_DEBUG', $develop_src);
             }
             $suffix = SCRIPT_DEBUG ? '' : '.min';
@@ -94,7 +94,7 @@ 
     }
 }
 
-if (! function_exists('get_user_locale')) {
+if ( ! function_exists('get_user_locale')) {
     /**
      * Shim for get_user_locale that was added in WP 4.7.0
      *

Indentation +82 added lines, -82 removed lines

@@ -5,105 +5,105 @@
  * it does not override any existing definition of the function in WP.
  */
 if (! function_exists('get_preview_post_link')) {
-    /**
-     * Function was added in WordPress 4.4.0
-     *
-     * @param null   $post
-     * @param array  $query_args
-     * @param string $preview_link
-     * @return mixed
-     */
-    function get_preview_post_link($post = null, $query_args = array(), $preview_link = '')
-    {
-        $post = get_post($post);
-        if (! $post) {
-            return '';
-        }
+	/**
+	 * Function was added in WordPress 4.4.0
+	 *
+	 * @param null   $post
+	 * @param array  $query_args
+	 * @param string $preview_link
+	 * @return mixed
+	 */
+	function get_preview_post_link($post = null, $query_args = array(), $preview_link = '')
+	{
+		$post = get_post($post);
+		if (! $post) {
+			return '';
+		}
 
-        $post_type_object = get_post_type_object($post->post_type);
-        if (is_post_type_viewable($post_type_object)) {
-            if (! $preview_link) {
-                $preview_link = set_url_scheme(get_permalink($post));
-            }
+		$post_type_object = get_post_type_object($post->post_type);
+		if (is_post_type_viewable($post_type_object)) {
+			if (! $preview_link) {
+				$preview_link = set_url_scheme(get_permalink($post));
+			}
 
-            $query_args['preview'] = 'true';
-            $preview_link = add_query_arg($query_args, $preview_link);
-        }
+			$query_args['preview'] = 'true';
+			$preview_link = add_query_arg($query_args, $preview_link);
+		}
 
-        /**
-         * Filters the URL used for a post preview.
-         *
-         * @since 2.0.5
-         * @since 4.0.0 Added the `$post` parameter.
-         * @param string  $preview_link URL used for the post preview.
-         * @param WP_Post $post         Post object.
-         */
-        return apply_filters('preview_post_link', $preview_link, $post);
-    }
+		/**
+		 * Filters the URL used for a post preview.
+		 *
+		 * @since 2.0.5
+		 * @since 4.0.0 Added the `$post` parameter.
+		 * @param string  $preview_link URL used for the post preview.
+		 * @param WP_Post $post         Post object.
+		 */
+		return apply_filters('preview_post_link', $preview_link, $post);
+	}
 }
 
 if (! function_exists('is_post_type_viewable')) {
-    function is_post_type_viewable($post_type)
-    {
-        if (is_scalar($post_type)) {
-            $post_type = get_post_type_object($post_type);
-            if (! $post_type) {
-                return false;
-            }
-        }
+	function is_post_type_viewable($post_type)
+	{
+		if (is_scalar($post_type)) {
+			$post_type = get_post_type_object($post_type);
+			if (! $post_type) {
+				return false;
+			}
+		}
 
-        return $post_type->publicly_queryable || ($post_type->_builtin && $post_type->public);
-    }
+		return $post_type->publicly_queryable || ($post_type->_builtin && $post_type->public);
+	}
 }
 
 if (! function_exists('wp_scripts_get_suffix')) {
-    /**
-     * Returns the suffix that can be used for the scripts.
-     *
-     * There are two suffix types, the normal one and the dev suffix.
-     *
-     * @since 5.0.0
-     *
-     * @param string $type The type of suffix to retrieve.
-     * @return string The script suffix.
-     */
-    function wp_scripts_get_suffix($type = '')
-    {
-        global $wp_version;
-        static $suffixes;
+	/**
+	 * Returns the suffix that can be used for the scripts.
+	 *
+	 * There are two suffix types, the normal one and the dev suffix.
+	 *
+	 * @since 5.0.0
+	 *
+	 * @param string $type The type of suffix to retrieve.
+	 * @return string The script suffix.
+	 */
+	function wp_scripts_get_suffix($type = '')
+	{
+		global $wp_version;
+		static $suffixes;
 
-        if ($suffixes === null) {
-            include(ABSPATH . WPINC . '/version.php'); // include an unmodified $wp_version
+		if ($suffixes === null) {
+			include(ABSPATH . WPINC . '/version.php'); // include an unmodified $wp_version
 
-            $develop_src = false !== strpos($wp_version, '-src');
+			$develop_src = false !== strpos($wp_version, '-src');
 
-            if (! defined('SCRIPT_DEBUG')) {
-                define('SCRIPT_DEBUG', $develop_src);
-            }
-            $suffix = SCRIPT_DEBUG ? '' : '.min';
-            $dev_suffix = $develop_src ? '' : '.min';
+			if (! defined('SCRIPT_DEBUG')) {
+				define('SCRIPT_DEBUG', $develop_src);
+			}
+			$suffix = SCRIPT_DEBUG ? '' : '.min';
+			$dev_suffix = $develop_src ? '' : '.min';
 
-            $suffixes = array('suffix' => $suffix, 'dev_suffix' => $dev_suffix);
-        }
+			$suffixes = array('suffix' => $suffix, 'dev_suffix' => $dev_suffix);
+		}
 
-        if ($type === 'dev') {
-            return $suffixes['dev_suffix'];
-        }
+		if ($type === 'dev') {
+			return $suffixes['dev_suffix'];
+		}
 
-        return $suffixes['suffix'];
-    }
+		return $suffixes['suffix'];
+	}
 }
 
 if (! function_exists('get_user_locale')) {
-    /**
-     * Shim for get_user_locale that was added in WP 4.7.0
-     *
-     * @param int $user_id
-     * @return string
-     * @since 4.9.73.p
-     */
-    function get_user_locale($user_id = 0)
-    {
-        return get_locale();
-    }
+	/**
+	 * Shim for get_user_locale that was added in WP 4.7.0
+	 *
+	 * @param int $user_id
+	 * @return string
+	 * @since 4.9.73.p
+	 */
+	function get_user_locale($user_id = 0)
+	{
+		return get_locale();
+	}
 }
\ No newline at end of file

core/db_models/relations/EE_HABTM_Relation.php

Spacing +18 added lines, -18 removed lines

@@ -65,18 +65,18 @@ 
         // remember the model relation chain to the JOIN model, because we'll
         // need it for get_join_statement()
         $this->_model_relation_chain_to_join_model = $model_relation_chain;
-        $this_table_pk_field                       = $this->get_this_model()->get_primary_key_field();// get_foreign_key_to($this->get_other_model()->get_this_model_name());
+        $this_table_pk_field                       = $this->get_this_model()->get_primary_key_field(); // get_foreign_key_to($this->get_other_model()->get_this_model_name());
         $join_table_fk_field_to_this_table         = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
         $this_table_alias                          = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
             $model_relation_chain,
             $this->get_this_model()->get_this_model_name()
-        ) . $this_table_pk_field->get_table_alias();
+        ).$this_table_pk_field->get_table_alias();
 
         $join_table_alias = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
             $model_relation_chain,
             $this->get_join_model()->get_this_model_name()
-        ) . $join_table_fk_field_to_this_table->get_table_alias();
-        $join_table       = $this->get_join_model()->get_table_for_alias($join_table_alias);
+        ).$join_table_fk_field_to_this_table->get_table_alias();
+        $join_table = $this->get_join_model()->get_table_for_alias($join_table_alias);
         // phew! ok, we have all the info we need, now we can create the SQL join string
         $SQL = $this->_left_join(
             $join_table,
@@ -84,7 +84,7 @@ 
             $join_table_fk_field_to_this_table->get_table_column(),
             $this_table_alias,
             $this_table_pk_field->get_table_column()
-        ) . $this->get_join_model()->_construct_internal_join_to_table_with_alias($join_table_alias);
+        ).$this->get_join_model()->_construct_internal_join_to_table_with_alias($join_table_alias);
 
         return $SQL;
     }
@@ -113,14 +113,14 @@ 
         $join_table_alias                   = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
             $this->_model_relation_chain_to_join_model,
             $this->get_join_model()->get_this_model_name()
-        ) . $join_table_fk_field_to_this_table->get_table_alias();
+        ).$join_table_fk_field_to_this_table->get_table_alias();
         $other_table_pk_field               = $this->get_other_model()->get_primary_key_field();
         $join_table_fk_field_to_other_table = $this->get_join_model()->get_foreign_key_to($this->get_other_model()->get_this_model_name());
         $other_table_alias                  = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
             $model_relation_chain,
             $this->get_other_model()->get_this_model_name()
-        ) . $other_table_pk_field->get_table_alias();
-        $other_table                        = $this->get_other_model()->get_table_for_alias($other_table_alias);
+        ).$other_table_pk_field->get_table_alias();
+        $other_table = $this->get_other_model()->get_table_for_alias($other_table_alias);
 
         $SQL = $this->_left_join(
             $other_table,
@@ -128,7 +128,7 @@ 
             $other_table_pk_field->get_table_column(),
             $join_table_alias,
             $join_table_fk_field_to_other_table->get_table_column()
-        ) . $this->get_other_model()->_construct_internal_join_to_table_with_alias($other_table_alias);
+        ).$this->get_other_model()->_construct_internal_join_to_table_with_alias($other_table_alias);
         return $SQL;
     }
 
@@ -161,17 +161,17 @@ 
         );
 
         // if $where_query exists lets add them to the query_params.
-        if (! empty($extra_join_model_fields_n_values)) {
+        if ( ! empty($extra_join_model_fields_n_values)) {
             // make sure we strip any of the join model names from the $where_query cause we don't need that in here (why? because client code may have used the same conditionals for get_all_related which DOES need the join model name)
             // make sure we strip THIS models name from the query param
             $parsed_query = array();
             foreach ($extra_join_model_fields_n_values as $query_param => $val) {
-                $query_param                = str_replace(
-                    $this->get_join_model()->get_this_model_name() . ".",
+                $query_param = str_replace(
+                    $this->get_join_model()->get_this_model_name().".",
                     "",
                     $query_param
                 );
-                $parsed_query[ $query_param ] = $val;
+                $parsed_query[$query_param] = $val;
             }
             $all_fields = array_merge($foreign_keys, $parsed_query);
         }
@@ -181,7 +181,7 @@ 
         // new row.
         // Again, if you want more sophisticated logic or insertions (handling more columns than just 2 foreign keys to
         // the other tables) use the joining model directly!
-        if (! $existing_entry_in_join_table) {
+        if ( ! $existing_entry_in_join_table) {
             $this->get_join_model()->insert($all_fields);
         } else {
             $this->get_join_model()->update(
@@ -217,17 +217,17 @@ 
         );
 
         // if $where_query exists lets add them to the query_params.
-        if (! empty($where_query)) {
+        if ( ! empty($where_query)) {
             // make sure we strip any of the join model names from the $where_query cause we don't need that in here (why? because client code may have used the same conditionals for get_all_related which DOES need the join model name)
             // make sure we strip THIS models name from the query param
             $parsed_query = array();
             foreach ($where_query as $query_param => $val) {
-                $query_param                = str_replace(
-                    $this->get_join_model()->get_this_model_name() . ".",
+                $query_param = str_replace(
+                    $this->get_join_model()->get_this_model_name().".",
                     "",
                     $query_param
                 );
-                $parsed_query[ $query_param ] = $val;
+                $parsed_query[$query_param] = $val;
             }
             $cols_n_values = array_merge($cols_n_values, $parsed_query);
         }

Indentation +239 added lines, -239 removed lines

@@ -9,266 +9,266 @@
  */
 class EE_HABTM_Relation extends EE_Model_Relation_Base
 {
-    /**
-     * Model which defines the relation between two other models. Eg, the EE_Event_Question_Group model,
-     * which joins EE_Event and EE_Question_Group
-     *
-     * @var EEM_Base
-     */
-    protected $_joining_model_name;
+	/**
+	 * Model which defines the relation between two other models. Eg, the EE_Event_Question_Group model,
+	 * which joins EE_Event and EE_Question_Group
+	 *
+	 * @var EEM_Base
+	 */
+	protected $_joining_model_name;
 
-    protected $_model_relation_chain_to_join_model;
+	protected $_model_relation_chain_to_join_model;
 
 
-    /**
-     * Object representing the relationship between two models. HasAndBelongsToMany relations always use a join-table
-     * (and an ee joining-model.) This knows how to join the models,
-     * get related models across the relation, and add-and-remove the relationships.
-     *
-     * @param bool    $joining_model_name
-     * @param boolean $block_deletes                 for this type of relation, we block by default for now. if there
-     *                                               are related models across this relation, block (prevent and add an
-     *                                               error) the deletion of this model
-     * @param string  $blocking_delete_error_message a customized error message on blocking deletes instead of the
-     *                                               default
-     */
-    public function __construct($joining_model_name, $block_deletes = true, $blocking_delete_error_message = '')
-    {
-        $this->_joining_model_name = $joining_model_name;
-        parent::__construct($block_deletes, $blocking_delete_error_message);
-    }
+	/**
+	 * Object representing the relationship between two models. HasAndBelongsToMany relations always use a join-table
+	 * (and an ee joining-model.) This knows how to join the models,
+	 * get related models across the relation, and add-and-remove the relationships.
+	 *
+	 * @param bool    $joining_model_name
+	 * @param boolean $block_deletes                 for this type of relation, we block by default for now. if there
+	 *                                               are related models across this relation, block (prevent and add an
+	 *                                               error) the deletion of this model
+	 * @param string  $blocking_delete_error_message a customized error message on blocking deletes instead of the
+	 *                                               default
+	 */
+	public function __construct($joining_model_name, $block_deletes = true, $blocking_delete_error_message = '')
+	{
+		$this->_joining_model_name = $joining_model_name;
+		parent::__construct($block_deletes, $blocking_delete_error_message);
+	}
 
-    /**
-     * Gets the joining model's object
-     *
-     * @return EEM_Base
-     */
-    public function get_join_model()
-    {
-        return $this->_get_model($this->_joining_model_name);
-    }
+	/**
+	 * Gets the joining model's object
+	 *
+	 * @return EEM_Base
+	 */
+	public function get_join_model()
+	{
+		return $this->_get_model($this->_joining_model_name);
+	}
 
 
-    /**
-     * Gets the SQL string for joining the main model's table containing the pk to the join table. Eg "LEFT JOIN
-     * real_join_table AS join_table_alias ON this_table_alias.pk = join_table_alias.fk_to_this_table"
-     *
-     * @param string $model_relation_chain like 'Event.Event_Venue.Venue'
-     * @return string of SQL
-     * @throws \EE_Error
-     */
-    public function get_join_to_intermediate_model_statement($model_relation_chain)
-    {
-        // create sql like
-        // LEFT JOIN join_table AS join_table_alias ON this_table_alias.this_table_pk = join_table_alias.join_table_fk_to_this
-        // LEFT JOIN other_table AS other_table_alias ON join_table_alias.join_table_fk_to_other = other_table_alias.other_table_pk
-        // remember the model relation chain to the JOIN model, because we'll
-        // need it for get_join_statement()
-        $this->_model_relation_chain_to_join_model = $model_relation_chain;
-        $this_table_pk_field                       = $this->get_this_model()->get_primary_key_field();// get_foreign_key_to($this->get_other_model()->get_this_model_name());
-        $join_table_fk_field_to_this_table         = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
-        $this_table_alias                          = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
-            $model_relation_chain,
-            $this->get_this_model()->get_this_model_name()
-        ) . $this_table_pk_field->get_table_alias();
+	/**
+	 * Gets the SQL string for joining the main model's table containing the pk to the join table. Eg "LEFT JOIN
+	 * real_join_table AS join_table_alias ON this_table_alias.pk = join_table_alias.fk_to_this_table"
+	 *
+	 * @param string $model_relation_chain like 'Event.Event_Venue.Venue'
+	 * @return string of SQL
+	 * @throws \EE_Error
+	 */
+	public function get_join_to_intermediate_model_statement($model_relation_chain)
+	{
+		// create sql like
+		// LEFT JOIN join_table AS join_table_alias ON this_table_alias.this_table_pk = join_table_alias.join_table_fk_to_this
+		// LEFT JOIN other_table AS other_table_alias ON join_table_alias.join_table_fk_to_other = other_table_alias.other_table_pk
+		// remember the model relation chain to the JOIN model, because we'll
+		// need it for get_join_statement()
+		$this->_model_relation_chain_to_join_model = $model_relation_chain;
+		$this_table_pk_field                       = $this->get_this_model()->get_primary_key_field();// get_foreign_key_to($this->get_other_model()->get_this_model_name());
+		$join_table_fk_field_to_this_table         = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
+		$this_table_alias                          = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
+			$model_relation_chain,
+			$this->get_this_model()->get_this_model_name()
+		) . $this_table_pk_field->get_table_alias();
 
-        $join_table_alias = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
-            $model_relation_chain,
-            $this->get_join_model()->get_this_model_name()
-        ) . $join_table_fk_field_to_this_table->get_table_alias();
-        $join_table       = $this->get_join_model()->get_table_for_alias($join_table_alias);
-        // phew! ok, we have all the info we need, now we can create the SQL join string
-        $SQL = $this->_left_join(
-            $join_table,
-            $join_table_alias,
-            $join_table_fk_field_to_this_table->get_table_column(),
-            $this_table_alias,
-            $this_table_pk_field->get_table_column()
-        ) . $this->get_join_model()->_construct_internal_join_to_table_with_alias($join_table_alias);
+		$join_table_alias = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
+			$model_relation_chain,
+			$this->get_join_model()->get_this_model_name()
+		) . $join_table_fk_field_to_this_table->get_table_alias();
+		$join_table       = $this->get_join_model()->get_table_for_alias($join_table_alias);
+		// phew! ok, we have all the info we need, now we can create the SQL join string
+		$SQL = $this->_left_join(
+			$join_table,
+			$join_table_alias,
+			$join_table_fk_field_to_this_table->get_table_column(),
+			$this_table_alias,
+			$this_table_pk_field->get_table_column()
+		) . $this->get_join_model()->_construct_internal_join_to_table_with_alias($join_table_alias);
 
-        return $SQL;
-    }
+		return $SQL;
+	}
 
 
-    /**
-     * Gets the SQL string for joining the join table to the other model's pk's table. Eg "LEFT JOIN real_other_table
-     * AS other_table_alias ON join_table_alias.fk_to_other_table = other_table_alias.pk" If you want to join between
-     * modelA -> joinModelAB -> modelB (eg, Event -> Event_Question_Group -> Question_Group), you should prepend the
-     * result of this function with results from get_join_to_intermediate_model_statement(), so that you join first to
-     * the intermediate join table, and then to the other model's pk's table
-     *
-     * @param string $model_relation_chain like 'Event.Event_Venue.Venue'
-     * @return string of SQL
-     * @throws \EE_Error
-     */
-    public function get_join_statement($model_relation_chain)
-    {
-        if ($this->_model_relation_chain_to_join_model === null) {
-            throw new EE_Error(sprintf(esc_html__(
-                'When using EE_HABTM_Relation to create a join, you must call get_join_to_intermediate_model_statement BEFORE get_join_statement',
-                'event_espresso'
-            )));
-        }
-        $join_table_fk_field_to_this_table  = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
-        $join_table_alias                   = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
-            $this->_model_relation_chain_to_join_model,
-            $this->get_join_model()->get_this_model_name()
-        ) . $join_table_fk_field_to_this_table->get_table_alias();
-        $other_table_pk_field               = $this->get_other_model()->get_primary_key_field();
-        $join_table_fk_field_to_other_table = $this->get_join_model()->get_foreign_key_to($this->get_other_model()->get_this_model_name());
-        $other_table_alias                  = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
-            $model_relation_chain,
-            $this->get_other_model()->get_this_model_name()
-        ) . $other_table_pk_field->get_table_alias();
-        $other_table                        = $this->get_other_model()->get_table_for_alias($other_table_alias);
+	/**
+	 * Gets the SQL string for joining the join table to the other model's pk's table. Eg "LEFT JOIN real_other_table
+	 * AS other_table_alias ON join_table_alias.fk_to_other_table = other_table_alias.pk" If you want to join between
+	 * modelA -> joinModelAB -> modelB (eg, Event -> Event_Question_Group -> Question_Group), you should prepend the
+	 * result of this function with results from get_join_to_intermediate_model_statement(), so that you join first to
+	 * the intermediate join table, and then to the other model's pk's table
+	 *
+	 * @param string $model_relation_chain like 'Event.Event_Venue.Venue'
+	 * @return string of SQL
+	 * @throws \EE_Error
+	 */
+	public function get_join_statement($model_relation_chain)
+	{
+		if ($this->_model_relation_chain_to_join_model === null) {
+			throw new EE_Error(sprintf(esc_html__(
+				'When using EE_HABTM_Relation to create a join, you must call get_join_to_intermediate_model_statement BEFORE get_join_statement',
+				'event_espresso'
+			)));
+		}
+		$join_table_fk_field_to_this_table  = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
+		$join_table_alias                   = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
+			$this->_model_relation_chain_to_join_model,
+			$this->get_join_model()->get_this_model_name()
+		) . $join_table_fk_field_to_this_table->get_table_alias();
+		$other_table_pk_field               = $this->get_other_model()->get_primary_key_field();
+		$join_table_fk_field_to_other_table = $this->get_join_model()->get_foreign_key_to($this->get_other_model()->get_this_model_name());
+		$other_table_alias                  = EE_Model_Parser::extract_table_alias_model_relation_chain_prefix(
+			$model_relation_chain,
+			$this->get_other_model()->get_this_model_name()
+		) . $other_table_pk_field->get_table_alias();
+		$other_table                        = $this->get_other_model()->get_table_for_alias($other_table_alias);
 
-        $SQL = $this->_left_join(
-            $other_table,
-            $other_table_alias,
-            $other_table_pk_field->get_table_column(),
-            $join_table_alias,
-            $join_table_fk_field_to_other_table->get_table_column()
-        ) . $this->get_other_model()->_construct_internal_join_to_table_with_alias($other_table_alias);
-        return $SQL;
-    }
+		$SQL = $this->_left_join(
+			$other_table,
+			$other_table_alias,
+			$other_table_pk_field->get_table_column(),
+			$join_table_alias,
+			$join_table_fk_field_to_other_table->get_table_column()
+		) . $this->get_other_model()->_construct_internal_join_to_table_with_alias($other_table_alias);
+		return $SQL;
+	}
 
 
-    /**
-     * Ensures there is an entry in the join table between these two models. Feel free to do this manually if you like.
-     * If the join table has additional columns (eg, the Event_Question_Group table has a is_primary column), then
-     * you'll want to directly use the EEM_Event_Question_Group model to add the entry to the table and set those extra
-     * columns' values
-     *
-     * @param EE_Base_Class|int $this_obj_or_id
-     * @param EE_Base_Class|int $other_obj_or_id
-     * @param array             $extra_join_model_fields_n_values col=>val pairs that are used as extra conditions for
-     *                                                            checking existing values and for setting new rows if
-     *                                                            no exact matches.
-     * @return EE_Base_Class
-     * @throws \EE_Error
-     */
-    public function add_relation_to($this_obj_or_id, $other_obj_or_id, $extra_join_model_fields_n_values = array())
-    {
-        $this_model_obj  = $this->get_this_model()->ensure_is_obj($this_obj_or_id, true);
-        $other_model_obj = $this->get_other_model()->ensure_is_obj($other_obj_or_id, true);
-        // check if such a relationship already exists
-        $join_model_fk_to_this_model  = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
-        $join_model_fk_to_other_model = $this->get_join_model()->get_foreign_key_to($this->get_other_model()->get_this_model_name());
+	/**
+	 * Ensures there is an entry in the join table between these two models. Feel free to do this manually if you like.
+	 * If the join table has additional columns (eg, the Event_Question_Group table has a is_primary column), then
+	 * you'll want to directly use the EEM_Event_Question_Group model to add the entry to the table and set those extra
+	 * columns' values
+	 *
+	 * @param EE_Base_Class|int $this_obj_or_id
+	 * @param EE_Base_Class|int $other_obj_or_id
+	 * @param array             $extra_join_model_fields_n_values col=>val pairs that are used as extra conditions for
+	 *                                                            checking existing values and for setting new rows if
+	 *                                                            no exact matches.
+	 * @return EE_Base_Class
+	 * @throws \EE_Error
+	 */
+	public function add_relation_to($this_obj_or_id, $other_obj_or_id, $extra_join_model_fields_n_values = array())
+	{
+		$this_model_obj  = $this->get_this_model()->ensure_is_obj($this_obj_or_id, true);
+		$other_model_obj = $this->get_other_model()->ensure_is_obj($other_obj_or_id, true);
+		// check if such a relationship already exists
+		$join_model_fk_to_this_model  = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
+		$join_model_fk_to_other_model = $this->get_join_model()->get_foreign_key_to($this->get_other_model()->get_this_model_name());
 
-        $foreign_keys = $all_fields = array(
-            $join_model_fk_to_this_model->get_name()  => $this_model_obj->ID(),
-            $join_model_fk_to_other_model->get_name() => $other_model_obj->ID(),
-        );
+		$foreign_keys = $all_fields = array(
+			$join_model_fk_to_this_model->get_name()  => $this_model_obj->ID(),
+			$join_model_fk_to_other_model->get_name() => $other_model_obj->ID(),
+		);
 
-        // if $where_query exists lets add them to the query_params.
-        if (! empty($extra_join_model_fields_n_values)) {
-            // make sure we strip any of the join model names from the $where_query cause we don't need that in here (why? because client code may have used the same conditionals for get_all_related which DOES need the join model name)
-            // make sure we strip THIS models name from the query param
-            $parsed_query = array();
-            foreach ($extra_join_model_fields_n_values as $query_param => $val) {
-                $query_param                = str_replace(
-                    $this->get_join_model()->get_this_model_name() . ".",
-                    "",
-                    $query_param
-                );
-                $parsed_query[ $query_param ] = $val;
-            }
-            $all_fields = array_merge($foreign_keys, $parsed_query);
-        }
+		// if $where_query exists lets add them to the query_params.
+		if (! empty($extra_join_model_fields_n_values)) {
+			// make sure we strip any of the join model names from the $where_query cause we don't need that in here (why? because client code may have used the same conditionals for get_all_related which DOES need the join model name)
+			// make sure we strip THIS models name from the query param
+			$parsed_query = array();
+			foreach ($extra_join_model_fields_n_values as $query_param => $val) {
+				$query_param                = str_replace(
+					$this->get_join_model()->get_this_model_name() . ".",
+					"",
+					$query_param
+				);
+				$parsed_query[ $query_param ] = $val;
+			}
+			$all_fields = array_merge($foreign_keys, $parsed_query);
+		}
 
-        $existing_entry_in_join_table = $this->get_join_model()->get_one(array($foreign_keys));
-        // If there is already an entry in the join table, indicating a relationship, update it instead of adding a
-        // new row.
-        // Again, if you want more sophisticated logic or insertions (handling more columns than just 2 foreign keys to
-        // the other tables) use the joining model directly!
-        if (! $existing_entry_in_join_table) {
-            $this->get_join_model()->insert($all_fields);
-        } else {
-            $this->get_join_model()->update(
-                $all_fields,
-                [$foreign_keys]
-            );
-        }
-        return $other_model_obj;
-    }
+		$existing_entry_in_join_table = $this->get_join_model()->get_one(array($foreign_keys));
+		// If there is already an entry in the join table, indicating a relationship, update it instead of adding a
+		// new row.
+		// Again, if you want more sophisticated logic or insertions (handling more columns than just 2 foreign keys to
+		// the other tables) use the joining model directly!
+		if (! $existing_entry_in_join_table) {
+			$this->get_join_model()->insert($all_fields);
+		} else {
+			$this->get_join_model()->update(
+				$all_fields,
+				[$foreign_keys]
+			);
+		}
+		return $other_model_obj;
+	}
 
 
-    /**
-     * Deletes any rows in the join table that have foreign keys matching the other model objects specified
-     *
-     * @param EE_Base_Class|int $this_obj_or_id
-     * @param EE_Base_Class|int $other_obj_or_id
-     * @param array             $where_query col=>val pairs that are used as extra conditions for checking existing
-     *                                       values and for removing existing rows if exact matches exist.
-     * @return EE_Base_Class
-     * @throws \EE_Error
-     */
-    public function remove_relation_to($this_obj_or_id, $other_obj_or_id, $where_query = array())
-    {
-        $this_model_obj  = $this->get_this_model()->ensure_is_obj($this_obj_or_id, true);
-        $other_model_obj = $this->get_other_model()->ensure_is_obj($other_obj_or_id, true);
-        // check if such a relationship already exists
-        $join_model_fk_to_this_model  = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
-        $join_model_fk_to_other_model = $this->get_join_model()->get_foreign_key_to($this->get_other_model()->get_this_model_name());
+	/**
+	 * Deletes any rows in the join table that have foreign keys matching the other model objects specified
+	 *
+	 * @param EE_Base_Class|int $this_obj_or_id
+	 * @param EE_Base_Class|int $other_obj_or_id
+	 * @param array             $where_query col=>val pairs that are used as extra conditions for checking existing
+	 *                                       values and for removing existing rows if exact matches exist.
+	 * @return EE_Base_Class
+	 * @throws \EE_Error
+	 */
+	public function remove_relation_to($this_obj_or_id, $other_obj_or_id, $where_query = array())
+	{
+		$this_model_obj  = $this->get_this_model()->ensure_is_obj($this_obj_or_id, true);
+		$other_model_obj = $this->get_other_model()->ensure_is_obj($other_obj_or_id, true);
+		// check if such a relationship already exists
+		$join_model_fk_to_this_model  = $this->get_join_model()->get_foreign_key_to($this->get_this_model()->get_this_model_name());
+		$join_model_fk_to_other_model = $this->get_join_model()->get_foreign_key_to($this->get_other_model()->get_this_model_name());
 
-        $cols_n_values = array(
-            $join_model_fk_to_this_model->get_name()  => $this_model_obj->ID(),
-            $join_model_fk_to_other_model->get_name() => $other_model_obj->ID(),
-        );
+		$cols_n_values = array(
+			$join_model_fk_to_this_model->get_name()  => $this_model_obj->ID(),
+			$join_model_fk_to_other_model->get_name() => $other_model_obj->ID(),
+		);
 
-        // if $where_query exists lets add them to the query_params.
-        if (! empty($where_query)) {
-            // make sure we strip any of the join model names from the $where_query cause we don't need that in here (why? because client code may have used the same conditionals for get_all_related which DOES need the join model name)
-            // make sure we strip THIS models name from the query param
-            $parsed_query = array();
-            foreach ($where_query as $query_param => $val) {
-                $query_param                = str_replace(
-                    $this->get_join_model()->get_this_model_name() . ".",
-                    "",
-                    $query_param
-                );
-                $parsed_query[ $query_param ] = $val;
-            }
-            $cols_n_values = array_merge($cols_n_values, $parsed_query);
-        }
+		// if $where_query exists lets add them to the query_params.
+		if (! empty($where_query)) {
+			// make sure we strip any of the join model names from the $where_query cause we don't need that in here (why? because client code may have used the same conditionals for get_all_related which DOES need the join model name)
+			// make sure we strip THIS models name from the query param
+			$parsed_query = array();
+			foreach ($where_query as $query_param => $val) {
+				$query_param                = str_replace(
+					$this->get_join_model()->get_this_model_name() . ".",
+					"",
+					$query_param
+				);
+				$parsed_query[ $query_param ] = $val;
+			}
+			$cols_n_values = array_merge($cols_n_values, $parsed_query);
+		}
 
-        $this->get_join_model()->delete(array($cols_n_values));
-        return $other_model_obj;
-    }
+		$this->get_join_model()->delete(array($cols_n_values));
+		return $other_model_obj;
+	}
 
-    /**
-     * Gets all the non-key fields (ie, not the primary key and not foreign keys) on the join model.
-     * @since 4.9.76.p
-     * @return EE_Model_Field_Base[]
-     * @throws EE_Error
-     */
-    public function getNonKeyFields()
-    {
-        // all fields besides the primary key and two foreign keys should be parameters
-        $join_model = $this->get_join_model();
-        $standard_fields = array();
-        if ($join_model->has_primary_key_field()) {
-            $standard_fields[] = $join_model->primary_key_name();
-        }
-        if ($this->get_this_model()->has_primary_key_field()) {
-            $standard_fields[] = $this->get_this_model()->primary_key_name();
-        }
-        if ($this->get_other_model()->has_primary_key_field()) {
-            $standard_fields[] = $this->get_other_model()->primary_key_name();
-        }
-        return array_diff_key(
-            $join_model->field_settings(),
-            array_flip($standard_fields)
-        );
-    }
+	/**
+	 * Gets all the non-key fields (ie, not the primary key and not foreign keys) on the join model.
+	 * @since 4.9.76.p
+	 * @return EE_Model_Field_Base[]
+	 * @throws EE_Error
+	 */
+	public function getNonKeyFields()
+	{
+		// all fields besides the primary key and two foreign keys should be parameters
+		$join_model = $this->get_join_model();
+		$standard_fields = array();
+		if ($join_model->has_primary_key_field()) {
+			$standard_fields[] = $join_model->primary_key_name();
+		}
+		if ($this->get_this_model()->has_primary_key_field()) {
+			$standard_fields[] = $this->get_this_model()->primary_key_name();
+		}
+		if ($this->get_other_model()->has_primary_key_field()) {
+			$standard_fields[] = $this->get_other_model()->primary_key_name();
+		}
+		return array_diff_key(
+			$join_model->field_settings(),
+			array_flip($standard_fields)
+		);
+	}
 
-    /**
-     * Returns true if the join model has non-key fields (ie, fields that aren't the primary key or foreign keys.)
-     * @since 4.9.76.p
-     * @return boolean
-     */
-    public function hasNonKeyFields()
-    {
-        return count($this->get_join_model()->field_settings()) > 3;
-    }
+	/**
+	 * Returns true if the join model has non-key fields (ie, fields that aren't the primary key or foreign keys.)
+	 * @since 4.9.76.p
+	 * @return boolean
+	 */
+	public function hasNonKeyFields()
+	{
+		return count($this->get_join_model()->field_settings()) > 3;
+	}
 }

core/libraries/rest_api/calculations/Attendee.php

Indentation +37 added lines, -37 removed lines

@@ -17,43 +17,43 @@
 class Attendee extends AttendeeCalculationsBase
 {
 
-    /**
-     * @param array                  $wpdb_row
-     * @param WP_REST_Request        $request
-     * @param AttendeeControllerBase $controller
-     * @since 4.9.66.p
-     * @return string
-     */
-    public function userAvatar(array $wpdb_row, WP_REST_Request $request, AttendeeControllerBase $controller)
-    {
-        if (is_array($wpdb_row) && isset($wpdb_row['Attendee_Meta.ATT_email'])) {
-            $email_address = $wpdb_row['Attendee_Meta.ATT_email'];
-        }
-        if (empty($email_address)) {
-            return get_avatar_url('', array('force_default' => true));
-        }
-        $avatar = get_avatar_url($email_address);
-        return $avatar ? $avatar : '';
-    }
+	/**
+	 * @param array                  $wpdb_row
+	 * @param WP_REST_Request        $request
+	 * @param AttendeeControllerBase $controller
+	 * @since 4.9.66.p
+	 * @return string
+	 */
+	public function userAvatar(array $wpdb_row, WP_REST_Request $request, AttendeeControllerBase $controller)
+	{
+		if (is_array($wpdb_row) && isset($wpdb_row['Attendee_Meta.ATT_email'])) {
+			$email_address = $wpdb_row['Attendee_Meta.ATT_email'];
+		}
+		if (empty($email_address)) {
+			return get_avatar_url('', array('force_default' => true));
+		}
+		$avatar = get_avatar_url($email_address);
+		return $avatar ? $avatar : '';
+	}
 
 
-    /**
-     * Provides an array for all the calculations possible that outlines a json schema for those calculations.
-     * Array is indexed by calculation (snake case) and value is the schema for that calculation.
-     *
-     * @since 4.9.68.p
-     * @return array
-     */
-    public function schemaForCalculations()
-    {
-        return array(
-            'user_avatar' => array(
-                'description' => esc_html__(
-                    'The avatar url for the attendee (if available).',
-                    'event_espresso'
-                ),
-                'type'        => 'string',
-            ),
-        );
-    }
+	/**
+	 * Provides an array for all the calculations possible that outlines a json schema for those calculations.
+	 * Array is indexed by calculation (snake case) and value is the schema for that calculation.
+	 *
+	 * @since 4.9.68.p
+	 * @return array
+	 */
+	public function schemaForCalculations()
+	{
+		return array(
+			'user_avatar' => array(
+				'description' => esc_html__(
+					'The avatar url for the attendee (if available).',
+					'event_espresso'
+				),
+				'type'        => 'string',
+			),
+		);
+	}
 }