eventespresso / event-espresso-core

core/domain/values/assets/Asset.php

Spacing +2 added lines, -2 removed lines

@@ -82,7 +82,7 @@ 
      */
     private function setType($type)
     {
-        if (! in_array($type, $this->validAssetTypes(), true)) {
+        if ( ! in_array($type, $this->validAssetTypes(), true)) {
             throw new InvalidDataTypeException(
                 'Asset::$type',
                 $type,
@@ -99,7 +99,7 @@ 
      */
     private function setHandle($handle)
     {
-        if (! is_string($handle)) {
+        if ( ! is_string($handle)) {
             throw new InvalidDataTypeException(
                 '$handle',
                 $handle,

Indentation +202 added lines, -202 removed lines

@@ -15,206 +15,206 @@
  */
 abstract class Asset
 {
-    /**
-     * indicates the file extension for a CSS file
-     */
-    const EXT_CSS = '.css';
-
-    /**
-     * indicates the file extension for a JS file
-     */
-    const EXT_JS = '.js';
-
-    /**
-     * indicates the file extension for a JS file
-     */
-    const EXT_PHP = '.php';
-
-    /**
-     * indicates the file extension for a build distribution CSS file
-     */
-    const FILE_EXTENSION_DISTRIBUTION_CSS = '.dist.css';
-
-    /**
-     * indicates the file extension for a build distribution JS file
-     */
-    const FILE_EXTENSION_DISTRIBUTION_JS = '.dist.js';
-
-    /**
-     * Indicates the file extension for a build distribution dependencies json file.
-     */
-    const FILE_EXTENSION_DISTRIBUTION_DEPS = '.dist.deps.php';
-
-    /**
-     * indicates a Cascading Style Sheet asset
-     */
-    const TYPE_CSS = 'css';
-
-    /**
-     * indicates a Javascript asset
-     */
-    const TYPE_JS = 'js';
-
-    /**
-     * indicates a JSON asset
-     */
-    CONST TYPE_JSON = 'json';
-    /**
-     * indicates a PHP asset
-     */
-    CONST TYPE_PHP = 'php';
-
-    /**
-     * indicates a Javascript manifest file
-     */
-    const TYPE_MANIFEST = 'manifest';
-
-    /**
-     * @var DomainInterface $domain
-     */
-    protected $domain;
-
-    /**
-     * @var string $type
-     */
-    private $type;
-
-    /**
-     * @var string $handle
-     */
-    private $handle;
-
-    /**
-     * @var bool $registered
-     */
-    private $registered = false;
-
-    /**
-     * @var bool $enqueue_immediately
-     */
-    private $enqueue_immediately = false;
-
-
-    /**
-     * Asset constructor.
-     *
-     * @param                 $type
-     * @param string          $handle
-     * @param DomainInterface $domain
-     * @throws InvalidDataTypeException
-     */
-    public function __construct($type, $handle, DomainInterface $domain)
-    {
-        $this->domain = $domain;
-        $this->setType($type);
-        $this->setHandle($handle);
-    }
-
-
-    /**
-     * @return array
-     */
-    public function validAssetTypes()
-    {
-        return array(
-            Asset::TYPE_CSS,
-            Asset::TYPE_JS,
-            Asset::TYPE_MANIFEST,
-        );
-    }
-
-
-    /**
-     * @param string $type
-     * @throws InvalidDataTypeException
-     */
-    private function setType($type)
-    {
-        if (! in_array($type, $this->validAssetTypes(), true)) {
-            throw new InvalidDataTypeException(
-                'Asset::$type',
-                $type,
-                'one of the TYPE_* class constants on \EventEspresso\core\domain\values\Asset is required'
-            );
-        }
-        $this->type = $type;
-    }
-
-
-    /**
-     * @param string $handle
-     * @throws InvalidDataTypeException
-     */
-    private function setHandle($handle)
-    {
-        if (! is_string($handle)) {
-            throw new InvalidDataTypeException(
-                '$handle',
-                $handle,
-                'string'
-            );
-        }
-        $this->handle = $handle;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function assetNamespace()
-    {
-        return $this->domain->assetNamespace();
-    }
-
-
-    /**
-     * @return string
-     */
-    public function type()
-    {
-        return $this->type;
-    }
-
-
-    /**
-     * @return string
-     */
-    public function handle()
-    {
-        return $this->handle;
-    }
-
-    /**
-     * @return bool
-     */
-    public function isRegistered()
-    {
-        return $this->registered;
-    }
-
-    /**
-     * @param bool $registered
-     */
-    public function setRegistered($registered = true)
-    {
-        $this->registered = filter_var($registered, FILTER_VALIDATE_BOOLEAN);
-    }
-
-
-    /**
-     * @return bool
-     */
-    public function enqueueImmediately()
-    {
-        return $this->enqueue_immediately;
-    }
-
-
-    /**
-     * @param bool $enqueue_immediately
-     */
-    public function setEnqueueImmediately($enqueue_immediately = true)
-    {
-        $this->enqueue_immediately = filter_var($enqueue_immediately, FILTER_VALIDATE_BOOLEAN);
-    }
+	/**
+	 * indicates the file extension for a CSS file
+	 */
+	const EXT_CSS = '.css';
+
+	/**
+	 * indicates the file extension for a JS file
+	 */
+	const EXT_JS = '.js';
+
+	/**
+	 * indicates the file extension for a JS file
+	 */
+	const EXT_PHP = '.php';
+
+	/**
+	 * indicates the file extension for a build distribution CSS file
+	 */
+	const FILE_EXTENSION_DISTRIBUTION_CSS = '.dist.css';
+
+	/**
+	 * indicates the file extension for a build distribution JS file
+	 */
+	const FILE_EXTENSION_DISTRIBUTION_JS = '.dist.js';
+
+	/**
+	 * Indicates the file extension for a build distribution dependencies json file.
+	 */
+	const FILE_EXTENSION_DISTRIBUTION_DEPS = '.dist.deps.php';
+
+	/**
+	 * indicates a Cascading Style Sheet asset
+	 */
+	const TYPE_CSS = 'css';
+
+	/**
+	 * indicates a Javascript asset
+	 */
+	const TYPE_JS = 'js';
+
+	/**
+	 * indicates a JSON asset
+	 */
+	CONST TYPE_JSON = 'json';
+	/**
+	 * indicates a PHP asset
+	 */
+	CONST TYPE_PHP = 'php';
+
+	/**
+	 * indicates a Javascript manifest file
+	 */
+	const TYPE_MANIFEST = 'manifest';
+
+	/**
+	 * @var DomainInterface $domain
+	 */
+	protected $domain;
+
+	/**
+	 * @var string $type
+	 */
+	private $type;
+
+	/**
+	 * @var string $handle
+	 */
+	private $handle;
+
+	/**
+	 * @var bool $registered
+	 */
+	private $registered = false;
+
+	/**
+	 * @var bool $enqueue_immediately
+	 */
+	private $enqueue_immediately = false;
+
+
+	/**
+	 * Asset constructor.
+	 *
+	 * @param                 $type
+	 * @param string          $handle
+	 * @param DomainInterface $domain
+	 * @throws InvalidDataTypeException
+	 */
+	public function __construct($type, $handle, DomainInterface $domain)
+	{
+		$this->domain = $domain;
+		$this->setType($type);
+		$this->setHandle($handle);
+	}
+
+
+	/**
+	 * @return array
+	 */
+	public function validAssetTypes()
+	{
+		return array(
+			Asset::TYPE_CSS,
+			Asset::TYPE_JS,
+			Asset::TYPE_MANIFEST,
+		);
+	}
+
+
+	/**
+	 * @param string $type
+	 * @throws InvalidDataTypeException
+	 */
+	private function setType($type)
+	{
+		if (! in_array($type, $this->validAssetTypes(), true)) {
+			throw new InvalidDataTypeException(
+				'Asset::$type',
+				$type,
+				'one of the TYPE_* class constants on \EventEspresso\core\domain\values\Asset is required'
+			);
+		}
+		$this->type = $type;
+	}
+
+
+	/**
+	 * @param string $handle
+	 * @throws InvalidDataTypeException
+	 */
+	private function setHandle($handle)
+	{
+		if (! is_string($handle)) {
+			throw new InvalidDataTypeException(
+				'$handle',
+				$handle,
+				'string'
+			);
+		}
+		$this->handle = $handle;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function assetNamespace()
+	{
+		return $this->domain->assetNamespace();
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function type()
+	{
+		return $this->type;
+	}
+
+
+	/**
+	 * @return string
+	 */
+	public function handle()
+	{
+		return $this->handle;
+	}
+
+	/**
+	 * @return bool
+	 */
+	public function isRegistered()
+	{
+		return $this->registered;
+	}
+
+	/**
+	 * @param bool $registered
+	 */
+	public function setRegistered($registered = true)
+	{
+		$this->registered = filter_var($registered, FILTER_VALIDATE_BOOLEAN);
+	}
+
+
+	/**
+	 * @return bool
+	 */
+	public function enqueueImmediately()
+	{
+		return $this->enqueue_immediately;
+	}
+
+
+	/**
+	 * @param bool $enqueue_immediately
+	 */
+	public function setEnqueueImmediately($enqueue_immediately = true)
+	{
+		$this->enqueue_immediately = filter_var($enqueue_immediately, FILTER_VALIDATE_BOOLEAN);
+	}
 }

core/domain/values/assets/StylesheetAsset.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
      */
     private function setMedia($media)
     {
-        if (! is_string($media)) {
+        if ( ! is_string($media)) {
             throw new InvalidDataTypeException(
                 '$media',
                 $media,

Indentation +59 added lines, -59 removed lines

@@ -17,71 +17,71 @@
 class StylesheetAsset extends BrowserAsset
 {
 
-    /**
-     * @var string $media
-     */
-    private $media;
+	/**
+	 * @var string $media
+	 */
+	private $media;
 
 
-    /**
-     * CssFile constructor.
-     *
-     * @param                 $handle
-     * @param string          $source
-     * @param array           $dependencies
-     * @param DomainInterface $domain
-     * @param string          $media
-     * @param string          $version
-     * @throws InvalidDataTypeException
-     * @throws DomainException
-     */
-    public function __construct(
-        $handle,
-        $source,
-        array $dependencies,
-        DomainInterface $domain,
-        $media = 'all',
-        $version = ''
-    ) {
-        parent::__construct(Asset::TYPE_CSS, $handle, $source, $dependencies, $domain, $version);
-        $this->setMedia($media);
-    }
+	/**
+	 * CssFile constructor.
+	 *
+	 * @param                 $handle
+	 * @param string          $source
+	 * @param array           $dependencies
+	 * @param DomainInterface $domain
+	 * @param string          $media
+	 * @param string          $version
+	 * @throws InvalidDataTypeException
+	 * @throws DomainException
+	 */
+	public function __construct(
+		$handle,
+		$source,
+		array $dependencies,
+		DomainInterface $domain,
+		$media = 'all',
+		$version = ''
+	) {
+		parent::__construct(Asset::TYPE_CSS, $handle, $source, $dependencies, $domain, $version);
+		$this->setMedia($media);
+	}
 
 
-    /**
-     * @return string
-     */
-    public function media()
-    {
-        return $this->media;
-    }
+	/**
+	 * @return string
+	 */
+	public function media()
+	{
+		return $this->media;
+	}
 
 
-    /**
-     * @param string $media
-     * @throws InvalidDataTypeException
-     */
-    private function setMedia($media)
-    {
-        if (! is_string($media)) {
-            throw new InvalidDataTypeException(
-                '$media',
-                $media,
-                'string'
-            );
-        }
-        $this->media = $media;
-    }
+	/**
+	 * @param string $media
+	 * @throws InvalidDataTypeException
+	 */
+	private function setMedia($media)
+	{
+		if (! is_string($media)) {
+			throw new InvalidDataTypeException(
+				'$media',
+				$media,
+				'string'
+			);
+		}
+		$this->media = $media;
+	}
 
 
-    /**
-     * @since 4.9.62.p
-     */
-    public function enqueueAsset()
-    {
-        if ($this->source() === '') {
-            return;
-        }
-        wp_enqueue_style($this->handle());
-    }
+	/**
+	 * @since 4.9.62.p
+	 */
+	public function enqueueAsset()
+	{
+		if ($this->source() === '') {
+			return;
+		}
+		wp_enqueue_style($this->handle());
+	}
 }

widgets/EspressoWidget.php

Spacing +1 added lines, -1 removed lines

@@ -35,6 +35,6 @@
      */
     public static function getIdBase($widget_class)
     {
-        return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)) . '-widget';
+        return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)).'-widget';
     }
 }

Indentation +22 added lines, -22 removed lines

@@ -11,28 +11,28 @@
  */
 class EspressoWidget extends \WP_Widget
 {
-    /**
-     * @param string $name
-     * @param array  $widget_options
-     * @param array  $control_options
-     */
-    public function __construct($name = '', array $widget_options = array(), array $control_options = array())
-    {
-        $id_base = EspressoWidget::getIdBase(get_class($this));
-        $control_options['id_base'] = $id_base;
-        $control_options['height'] = isset($control_options['height']) ? $control_options['height'] : 300;
-        $control_options['width'] = isset($control_options['width']) ? $control_options['width'] : 350;
-        // Register widget with WordPress
-        parent::__construct($id_base, $name, $widget_options, $control_options);
-    }
+	/**
+	 * @param string $name
+	 * @param array  $widget_options
+	 * @param array  $control_options
+	 */
+	public function __construct($name = '', array $widget_options = array(), array $control_options = array())
+	{
+		$id_base = EspressoWidget::getIdBase(get_class($this));
+		$control_options['id_base'] = $id_base;
+		$control_options['height'] = isset($control_options['height']) ? $control_options['height'] : 300;
+		$control_options['width'] = isset($control_options['width']) ? $control_options['width'] : 350;
+		// Register widget with WordPress
+		parent::__construct($id_base, $name, $widget_options, $control_options);
+	}
 
 
-    /**
-     * @param string $widget_class
-     * @return string
-     */
-    public static function getIdBase($widget_class)
-    {
-        return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)) . '-widget';
-    }
+	/**
+	 * @param string $widget_class
+	 * @return string
+	 */
+	public static function getIdBase($widget_class)
+	{
+		return sanitize_title(str_replace(array('EEW_', '_'), array('EE_', '-'), $widget_class)) . '-widget';
+	}
 }

modules/ticket_selector/TicketSelectorIframe.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
                 $this
             )
         );
-        if (! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
+        if ( ! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
             $this->addStylesheets(array('site_theme' => ''));
         }
         $this->addScripts(
@@ -79,7 +79,7 @@ 
             array(),
             $this
         );
-        if (! empty($js_attributes)) {
+        if ( ! empty($js_attributes)) {
             $this->addScriptAttributes($js_attributes);
         }
         $this->addLocalizedVars(

Indentation +73 added lines, -73 removed lines

@@ -19,77 +19,77 @@
  */
 class TicketSelectorIframe extends Iframe
 {
-    /**
-     * TicketSelectorIframe constructor.
-     *
-     * @param EEM_Event        $event_model
-     * @param CurrentPage      $current_page
-     * @param RequestInterface $request
-     * @throws EE_Error
-     */
-    public function __construct(EEM_Event $event_model, CurrentPage $current_page, RequestInterface $request)
-    {
-        $current_page->setEspressoPage(true);
-        $ticket_selector = LoaderFactory::getLoader()->getNew(DisplayTicketSelector::class);
-        $ticket_selector->setIframe();
-        $event = $event_model->get_one_by_ID($request->getRequestParam('event', 0, 'int'));
-        parent::__construct(
-            esc_html__('Ticket Selector', 'event_espresso'),
-            $ticket_selector->display($event)
-        );
-        $this->addStylesheets(
-            apply_filters(
-                'FHEE__EED_Ticket_Selector__ticket_selector_iframe__css',
-                array(
-                    'ticket_selector_embed' => TICKET_SELECTOR_ASSETS_URL
-                                               . 'ticket_selector_embed.css?ver='
-                                               . EVENT_ESPRESSO_VERSION,
-                    'ticket_selector'       => TICKET_SELECTOR_ASSETS_URL
-                                               . 'ticket_selector.css?ver='
-                                               . EVENT_ESPRESSO_VERSION,
-                ),
-                $this
-            )
-        );
-        if (! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
-            $this->addStylesheets(array('site_theme' => ''));
-        }
-        $this->addScripts(
-            apply_filters(
-                'FHEE__EventEspresso_modules_events_archive_EventsArchiveIframe__display__js',
-                array(
-                    'ticket_selector_iframe_embed' => TICKET_SELECTOR_ASSETS_URL
-                                                      . 'ticket_selector_iframe_embed.js?ver='
-                                                      . EVENT_ESPRESSO_VERSION,
-                ),
-                $this
-            )
-        );
-        $js_attributes = apply_filters(
-            'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__js_attributes',
-            array(),
-            $this
-        );
-        if (! empty($js_attributes)) {
-            $this->addScriptAttributes($js_attributes);
-        }
-        $this->addLocalizedVars(
-            apply_filters(
-                'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__localized_vars',
-                array(
-                    'ticket_selector_iframe' => true,
-                    'EEDTicketSelectorMsg'   => wp_strip_all_tags(
-                        __(
-                            'Please choose at least one ticket before continuing.',
-                            'event_espresso'
-                        )
-                    ),
-                )
-            )
-        );
-        do_action(
-            'AHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__complete',
-            $this
-        );
-    }
+	/**
+	 * TicketSelectorIframe constructor.
+	 *
+	 * @param EEM_Event        $event_model
+	 * @param CurrentPage      $current_page
+	 * @param RequestInterface $request
+	 * @throws EE_Error
+	 */
+	public function __construct(EEM_Event $event_model, CurrentPage $current_page, RequestInterface $request)
+	{
+		$current_page->setEspressoPage(true);
+		$ticket_selector = LoaderFactory::getLoader()->getNew(DisplayTicketSelector::class);
+		$ticket_selector->setIframe();
+		$event = $event_model->get_one_by_ID($request->getRequestParam('event', 0, 'int'));
+		parent::__construct(
+			esc_html__('Ticket Selector', 'event_espresso'),
+			$ticket_selector->display($event)
+		);
+		$this->addStylesheets(
+			apply_filters(
+				'FHEE__EED_Ticket_Selector__ticket_selector_iframe__css',
+				array(
+					'ticket_selector_embed' => TICKET_SELECTOR_ASSETS_URL
+											   . 'ticket_selector_embed.css?ver='
+											   . EVENT_ESPRESSO_VERSION,
+					'ticket_selector'       => TICKET_SELECTOR_ASSETS_URL
+											   . 'ticket_selector.css?ver='
+											   . EVENT_ESPRESSO_VERSION,
+				),
+				$this
+			)
+		);
+		if (! apply_filters('FHEE__EED_Ticket_Selector__ticket_selector_iframe__load_theme_css', false, $this)) {
+			$this->addStylesheets(array('site_theme' => ''));
+		}
+		$this->addScripts(
+			apply_filters(
+				'FHEE__EventEspresso_modules_events_archive_EventsArchiveIframe__display__js',
+				array(
+					'ticket_selector_iframe_embed' => TICKET_SELECTOR_ASSETS_URL
+													  . 'ticket_selector_iframe_embed.js?ver='
+													  . EVENT_ESPRESSO_VERSION,
+				),
+				$this
+			)
+		);
+		$js_attributes = apply_filters(
+			'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__js_attributes',
+			array(),
+			$this
+		);
+		if (! empty($js_attributes)) {
+			$this->addScriptAttributes($js_attributes);
+		}
+		$this->addLocalizedVars(
+			apply_filters(
+				'FHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__localized_vars',
+				array(
+					'ticket_selector_iframe' => true,
+					'EEDTicketSelectorMsg'   => wp_strip_all_tags(
+						__(
+							'Please choose at least one ticket before continuing.',
+							'event_espresso'
+						)
+					),
+				)
+			)
+		);
+		do_action(
+			'AHEE__EventEspresso_modules_ticket_selector_TicketSelectorIframe__construct__complete',
+			$this
+		);
+	}
 }

caffeinated/modules/recaptcha/ReCaptcha/RequestMethod/SocketPost.php

Indentation +83 added lines, -83 removed lines

@@ -33,87 +33,87 @@
  */
 class SocketPost implements RequestMethod
 {
-    /**
-     * reCAPTCHA service host.
-     *
-     * @const string
-     */
-    const RECAPTCHA_HOST = 'www.google.com';
-
-    /**
-     * @const string reCAPTCHA service path
-     */
-    const SITE_VERIFY_PATH = '/recaptcha/api/siteverify';
-
-    /**
-     * @const string Bad request error
-     */
-    const BAD_REQUEST = '{"success": false, "error-codes": ["invalid-request"]}';
-
-    /**
-     * @const string Bad response error
-     */
-    const BAD_RESPONSE = '{"success": false, "error-codes": ["invalid-response"]}';
-
-    /**
-     * Socket to the reCAPTCHA service
-     *
-     * @var Socket
-     */
-    private $socket;
-
-    /**
-     * Constructor
-     *
-     * @param \ReCaptcha\RequestMethod\Socket $socket optional socket, injectable for testing
-     */
-    public function __construct(Socket $socket = null)
-    {
-        if (! is_null($socket)) {
-            $this->socket = $socket;
-        } else {
-            $this->socket = new Socket();
-        }
-    }
-
-    /**
-     * Submit the POST request with the specified parameters.
-     *
-     * @param RequestParameters $params Request parameters
-     * @return string Body of the reCAPTCHA response
-     */
-    public function submit(RequestParameters $params)
-    {
-        $errno = 0;
-        $errstr = '';
-
-        if ($this->socket->fsockopen('ssl://' . self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
-            $content = $params->toQueryString();
-
-            $request = "POST " . self::SITE_VERIFY_PATH . " HTTP/1.1\r\n";
-            $request .= "Host: " . self::RECAPTCHA_HOST . "\r\n";
-            $request .= "Content-Type: application/x-www-form-urlencoded\r\n";
-            $request .= "Content-length: " . strlen($content) . "\r\n";
-            $request .= "Connection: close\r\n\r\n";
-            $request .= $content . "\r\n\r\n";
-
-            $this->socket->fwrite($request);
-            $response = '';
-
-            while (! $this->socket->feof()) {
-                $response .= $this->socket->fgets(4096);
-            }
-
-            $this->socket->fclose();
-
-            if (0 === strpos($response, 'HTTP/1.1 200 OK')) {
-                $parts = preg_split("#\n\s*\n#Uis", $response);
-                return $parts[1];
-            }
-
-            return self::BAD_RESPONSE;
-        }
-
-        return self::BAD_REQUEST;
-    }
+	/**
+	 * reCAPTCHA service host.
+	 *
+	 * @const string
+	 */
+	const RECAPTCHA_HOST = 'www.google.com';
+
+	/**
+	 * @const string reCAPTCHA service path
+	 */
+	const SITE_VERIFY_PATH = '/recaptcha/api/siteverify';
+
+	/**
+	 * @const string Bad request error
+	 */
+	const BAD_REQUEST = '{"success": false, "error-codes": ["invalid-request"]}';
+
+	/**
+	 * @const string Bad response error
+	 */
+	const BAD_RESPONSE = '{"success": false, "error-codes": ["invalid-response"]}';
+
+	/**
+	 * Socket to the reCAPTCHA service
+	 *
+	 * @var Socket
+	 */
+	private $socket;
+
+	/**
+	 * Constructor
+	 *
+	 * @param \ReCaptcha\RequestMethod\Socket $socket optional socket, injectable for testing
+	 */
+	public function __construct(Socket $socket = null)
+	{
+		if (! is_null($socket)) {
+			$this->socket = $socket;
+		} else {
+			$this->socket = new Socket();
+		}
+	}
+
+	/**
+	 * Submit the POST request with the specified parameters.
+	 *
+	 * @param RequestParameters $params Request parameters
+	 * @return string Body of the reCAPTCHA response
+	 */
+	public function submit(RequestParameters $params)
+	{
+		$errno = 0;
+		$errstr = '';
+
+		if ($this->socket->fsockopen('ssl://' . self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
+			$content = $params->toQueryString();
+
+			$request = "POST " . self::SITE_VERIFY_PATH . " HTTP/1.1\r\n";
+			$request .= "Host: " . self::RECAPTCHA_HOST . "\r\n";
+			$request .= "Content-Type: application/x-www-form-urlencoded\r\n";
+			$request .= "Content-length: " . strlen($content) . "\r\n";
+			$request .= "Connection: close\r\n\r\n";
+			$request .= $content . "\r\n\r\n";
+
+			$this->socket->fwrite($request);
+			$response = '';
+
+			while (! $this->socket->feof()) {
+				$response .= $this->socket->fgets(4096);
+			}
+
+			$this->socket->fclose();
+
+			if (0 === strpos($response, 'HTTP/1.1 200 OK')) {
+				$parts = preg_split("#\n\s*\n#Uis", $response);
+				return $parts[1];
+			}
+
+			return self::BAD_RESPONSE;
+		}
+
+		return self::BAD_REQUEST;
+	}
 }

Spacing +7 added lines, -7 removed lines

@@ -69,7 +69,7 @@ 
      */
     public function __construct(Socket $socket = null)
     {
-        if (! is_null($socket)) {
+        if ( ! is_null($socket)) {
             $this->socket = $socket;
         } else {
             $this->socket = new Socket();
@@ -87,20 +87,20 @@ 
         $errno = 0;
         $errstr = '';
 
-        if ($this->socket->fsockopen('ssl://' . self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
+        if ($this->socket->fsockopen('ssl://'.self::RECAPTCHA_HOST, 443, $errno, $errstr, 30) !== false) {
             $content = $params->toQueryString();
 
-            $request = "POST " . self::SITE_VERIFY_PATH . " HTTP/1.1\r\n";
-            $request .= "Host: " . self::RECAPTCHA_HOST . "\r\n";
+            $request = "POST ".self::SITE_VERIFY_PATH." HTTP/1.1\r\n";
+            $request .= "Host: ".self::RECAPTCHA_HOST."\r\n";
             $request .= "Content-Type: application/x-www-form-urlencoded\r\n";
-            $request .= "Content-length: " . strlen($content) . "\r\n";
+            $request .= "Content-length: ".strlen($content)."\r\n";
             $request .= "Connection: close\r\n\r\n";
-            $request .= $content . "\r\n\r\n";
+            $request .= $content."\r\n\r\n";
 
             $this->socket->fwrite($request);
             $response = '';
 
-            while (! $this->socket->feof()) {
+            while ( ! $this->socket->feof()) {
                 $response .= $this->socket->fgets(4096);
             }
 

caffeinated/modules/recaptcha/ReCaptcha/RequestMethod/Socket.php

Indentation +69 added lines, -69 removed lines

@@ -29,79 +29,79 @@
  */
 class Socket
 {
-    private $handle = null;
+	private $handle = null;
 
-    /**
-     * fsockopen
-     *
-     * @see http://php.net/fsockopen
-     * @param string $hostname
-     * @param int    $port
-     * @param int    $errno
-     * @param string $errstr
-     * @param float  $timeout
-     * @return resource
-     */
-    public function fsockopen($hostname, $port = -1, &$errno = 0, &$errstr = '', $timeout = null)
-    {
-        $this->handle = fsockopen(
-            $hostname,
-            $port,
-            $errno,
-            $errstr,
-            (is_null($timeout) ? ini_get("default_socket_timeout") : $timeout)
-        );
+	/**
+	 * fsockopen
+	 *
+	 * @see http://php.net/fsockopen
+	 * @param string $hostname
+	 * @param int    $port
+	 * @param int    $errno
+	 * @param string $errstr
+	 * @param float  $timeout
+	 * @return resource
+	 */
+	public function fsockopen($hostname, $port = -1, &$errno = 0, &$errstr = '', $timeout = null)
+	{
+		$this->handle = fsockopen(
+			$hostname,
+			$port,
+			$errno,
+			$errstr,
+			(is_null($timeout) ? ini_get("default_socket_timeout") : $timeout)
+		);
 
-        if ($this->handle != false && $errno === 0 && $errstr === '') {
-            return $this->handle;
-        } else {
-            return false;
-        }
-    }
+		if ($this->handle != false && $errno === 0 && $errstr === '') {
+			return $this->handle;
+		} else {
+			return false;
+		}
+	}
 
-    /**
-     * fwrite
-     *
-     * @see http://php.net/fwrite
-     * @param string $string
-     * @param int    $length
-     * @return int | bool
-     */
-    public function fwrite($string, $length = null)
-    {
-        return fwrite($this->handle, $string, (is_null($length) ? strlen($string) : $length));
-    }
+	/**
+	 * fwrite
+	 *
+	 * @see http://php.net/fwrite
+	 * @param string $string
+	 * @param int    $length
+	 * @return int | bool
+	 */
+	public function fwrite($string, $length = null)
+	{
+		return fwrite($this->handle, $string, (is_null($length) ? strlen($string) : $length));
+	}
 
-    /**
-     * fgets
-     *
-     * @see http://php.net/fgets
-     * @param int $length
-     */
-    public function fgets($length = null)
-    {
-        return fgets($this->handle, $length);
-    }
+	/**
+	 * fgets
+	 *
+	 * @see http://php.net/fgets
+	 * @param int $length
+	 */
+	public function fgets($length = null)
+	{
+		return fgets($this->handle, $length);
+	}
 
-    /**
-     * feof
-     *
-     * @see http://php.net/feof
-     * @return bool
-     */
-    public function feof()
-    {
-        return feof($this->handle);
-    }
+	/**
+	 * feof
+	 *
+	 * @see http://php.net/feof
+	 * @return bool
+	 */
+	public function feof()
+	{
+		return feof($this->handle);
+	}
 
-    /**
-     * fclose
-     *
-     * @see http://php.net/fclose
-     * @return bool
-     */
-    public function fclose()
-    {
-        return fclose($this->handle);
-    }
+	/**
+	 * fclose
+	 *
+	 * @see http://php.net/fclose
+	 * @return bool
+	 */
+	public function fclose()
+	{
+		return fclose($this->handle);
+	}
 }

core/exceptions/ExceptionLogger.php

Spacing +8 added lines, -8 removed lines

@@ -33,18 +33,18 @@
      */
     public function log(\Exception $exception, $time = 0)
     {
-        if (! $time) {
+        if ( ! $time) {
             $time = time();
         }
         $exception_log = '----------------------------------------------------------------------------------------';
         $exception_log .= PHP_EOL;
-        $exception_log .= '[' . date('Y-m-d H:i:s', $time) . ']  Exception Details' . PHP_EOL;
-        $exception_log .= 'Message: ' . $exception->getMessage() . PHP_EOL;
-        $exception_log .= 'Code: ' . $exception->getCode() . PHP_EOL;
-        $exception_log .= 'File: ' . $exception->getFile() . PHP_EOL;
-        $exception_log .= 'Line No: ' . $exception->getLine() . PHP_EOL;
-        $exception_log .= 'Stack trace: ' . PHP_EOL;
-        $exception_log .= $exception->getTraceAsString() . PHP_EOL;
+        $exception_log .= '['.date('Y-m-d H:i:s', $time).']  Exception Details'.PHP_EOL;
+        $exception_log .= 'Message: '.$exception->getMessage().PHP_EOL;
+        $exception_log .= 'Code: '.$exception->getCode().PHP_EOL;
+        $exception_log .= 'File: '.$exception->getFile().PHP_EOL;
+        $exception_log .= 'Line No: '.$exception->getLine().PHP_EOL;
+        $exception_log .= 'Stack trace: '.PHP_EOL;
+        $exception_log .= $exception->getTraceAsString().PHP_EOL;
         $exception_log .= '----------------------------------------------------------------------------------------';
         $exception_log .= PHP_EOL;
         error_log($exception_log);

Indentation +33 added lines, -33 removed lines

@@ -12,39 +12,39 @@
  */
 class ExceptionLogger
 {
-    /**
-     * ExceptionLogger constructor.
-     *
-     * @param \Exception $exception
-     */
-    public function __construct(\Exception $exception)
-    {
-        $this->log($exception);
-    }
+	/**
+	 * ExceptionLogger constructor.
+	 *
+	 * @param \Exception $exception
+	 */
+	public function __construct(\Exception $exception)
+	{
+		$this->log($exception);
+	}
 
 
-    /**
-     * write exception details to log file
-     *
-     * @param \Exception $exception
-     * @param int        $time
-     */
-    public function log(\Exception $exception, $time = 0)
-    {
-        if (! $time) {
-            $time = time();
-        }
-        $exception_log = '----------------------------------------------------------------------------------------';
-        $exception_log .= PHP_EOL;
-        $exception_log .= '[' . date('Y-m-d H:i:s', $time) . ']  Exception Details' . PHP_EOL;
-        $exception_log .= 'Message: ' . $exception->getMessage() . PHP_EOL;
-        $exception_log .= 'Code: ' . $exception->getCode() . PHP_EOL;
-        $exception_log .= 'File: ' . $exception->getFile() . PHP_EOL;
-        $exception_log .= 'Line No: ' . $exception->getLine() . PHP_EOL;
-        $exception_log .= 'Stack trace: ' . PHP_EOL;
-        $exception_log .= $exception->getTraceAsString() . PHP_EOL;
-        $exception_log .= '----------------------------------------------------------------------------------------';
-        $exception_log .= PHP_EOL;
-        error_log($exception_log);
-    }
+	/**
+	 * write exception details to log file
+	 *
+	 * @param \Exception $exception
+	 * @param int        $time
+	 */
+	public function log(\Exception $exception, $time = 0)
+	{
+		if (! $time) {
+			$time = time();
+		}
+		$exception_log = '----------------------------------------------------------------------------------------';
+		$exception_log .= PHP_EOL;
+		$exception_log .= '[' . date('Y-m-d H:i:s', $time) . ']  Exception Details' . PHP_EOL;
+		$exception_log .= 'Message: ' . $exception->getMessage() . PHP_EOL;
+		$exception_log .= 'Code: ' . $exception->getCode() . PHP_EOL;
+		$exception_log .= 'File: ' . $exception->getFile() . PHP_EOL;
+		$exception_log .= 'Line No: ' . $exception->getLine() . PHP_EOL;
+		$exception_log .= 'Stack trace: ' . PHP_EOL;
+		$exception_log .= $exception->getTraceAsString() . PHP_EOL;
+		$exception_log .= '----------------------------------------------------------------------------------------';
+		$exception_log .= PHP_EOL;
+		error_log($exception_log);
+	}
 }

core/services/formatters/FormatterBase.php

Spacing +2 added lines, -2 removed lines

@@ -24,14 +24,14 @@
      */
     public function formatArray($input)
     {
-        if (! is_array($input)) {
+        if ( ! is_array($input)) {
             throw new InvalidDataTypeException('input', $input, 'array');
         }
         // we can use $this inside the closure in PHP 5.3, so pass in a variable pointing to this instead
         $formatter = $this;
         array_walk_recursive(
             $input,
-            function (&$value, $key) use ($formatter) {
+            function(&$value, $key) use ($formatter) {
                 $value = $formatter->format($value);
             }
         );

Indentation +22 added lines, -22 removed lines

@@ -14,26 +14,26 @@
  */
 abstract class FormatterBase implements FormatterInterface
 {
-    /**
-     * Recursively applies the formatting to all VALUES in this multi-dimensional array
-     *
-     * @param array $input
-     * @return array
-     * @throws InvalidDataTypeException if $input is not an array
-     */
-    public function formatArray($input)
-    {
-        if (! is_array($input)) {
-            throw new InvalidDataTypeException('input', $input, 'array');
-        }
-        // we can use $this inside the closure in PHP 5.3, so pass in a variable pointing to this instead
-        $formatter = $this;
-        array_walk_recursive(
-            $input,
-            function (&$value, $key) use ($formatter) {
-                $value = $formatter->format($value);
-            }
-        );
-        return $input;
-    }
+	/**
+	 * Recursively applies the formatting to all VALUES in this multi-dimensional array
+	 *
+	 * @param array $input
+	 * @return array
+	 * @throws InvalidDataTypeException if $input is not an array
+	 */
+	public function formatArray($input)
+	{
+		if (! is_array($input)) {
+			throw new InvalidDataTypeException('input', $input, 'array');
+		}
+		// we can use $this inside the closure in PHP 5.3, so pass in a variable pointing to this instead
+		$formatter = $this;
+		array_walk_recursive(
+			$input,
+			function (&$value, $key) use ($formatter) {
+				$value = $formatter->format($value);
+			}
+		);
+		return $input;
+	}
 }

core/services/request/RequestStackBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
             throw new InvalidRequestStackMiddlewareException($middleware_app_class);
         }
         // is $middleware_app_class a valid FQCN (or class is already loaded) ?
-        if (! class_exists($middleware_app_class)) {
+        if ( ! class_exists($middleware_app_class)) {
             throw new InvalidRequestStackMiddlewareException($middleware_app_class);
         }
         return $middleware_app;

Indentation +85 added lines, -85 removed lines

@@ -20,95 +20,95 @@
  */
 class RequestStackBuilder extends SplDoublyLinkedList
 {
-    private LoaderInterface $loader;
+	private LoaderInterface $loader;
 
 
-    /**
-     * RequestStackBuilder constructor.
-     *
-     * @param LoaderInterface $loader
-     */
-    public function __construct(LoaderInterface $loader)
-    {
-        $this->loader = $loader;
-        $this->setIteratorMode(SplDoublyLinkedList::IT_MODE_LIFO | SplDoublyLinkedList::IT_MODE_KEEP);
-    }
+	/**
+	 * RequestStackBuilder constructor.
+	 *
+	 * @param LoaderInterface $loader
+	 */
+	public function __construct(LoaderInterface $loader)
+	{
+		$this->loader = $loader;
+		$this->setIteratorMode(SplDoublyLinkedList::IT_MODE_LIFO | SplDoublyLinkedList::IT_MODE_KEEP);
+	}
 
 
-    /**
-     * builds decorated middleware stack
-     * by continuously injecting previous middleware app into the next
-     *
-     * @param RequestStackCoreAppInterface $application
-     * @return RequestStack
-     * @throws Exception
-     * @throws Throwable
-     */
-    public function resolve(RequestStackCoreAppInterface $application): RequestStack
-    {
-        $core_app = $application;
-        // NOW... because the RequestStack is following the decorator pattern,
-        // the first stack app we add will end up at the center of the stack,
-        // and will end up being the last item to actually run, but we don't want that!
-        // Basically we're dealing with TWO stacks, and transferring items from one to the other,
-        // BUT... we want the final stack to be in the same order as the first.
-        // So we need to reverse the iterator mode when transferring items,
-        // because if we don't, the second stack will end  up in the incorrect order.
-        $this->setIteratorMode(SplDoublyLinkedList::IT_MODE_FIFO | SplDoublyLinkedList::IT_MODE_KEEP);
-        for ($this->rewind(); $this->valid(); $this->next()) {
-            try {
-                $middleware_app = $this->validateMiddlewareAppDetails($this->current(), true);
-                $middleware_app_class = array_shift($middleware_app);
-                $middleware_app_args = is_array($middleware_app) ? $middleware_app : array();
-                $middleware_app_args = array($application, $this->loader) + $middleware_app_args;
-                $application = $this->loader->getShared($middleware_app_class, $middleware_app_args);
-            } catch (InvalidRequestStackMiddlewareException $exception) {
-                if (WP_DEBUG) {
-                    new ExceptionStackTraceDisplay($exception);
-                    continue;
-                }
-                error_log($exception->getMessage());
-            }
-        }
-        return new RequestStack($application, $core_app);
-    }
+	/**
+	 * builds decorated middleware stack
+	 * by continuously injecting previous middleware app into the next
+	 *
+	 * @param RequestStackCoreAppInterface $application
+	 * @return RequestStack
+	 * @throws Exception
+	 * @throws Throwable
+	 */
+	public function resolve(RequestStackCoreAppInterface $application): RequestStack
+	{
+		$core_app = $application;
+		// NOW... because the RequestStack is following the decorator pattern,
+		// the first stack app we add will end up at the center of the stack,
+		// and will end up being the last item to actually run, but we don't want that!
+		// Basically we're dealing with TWO stacks, and transferring items from one to the other,
+		// BUT... we want the final stack to be in the same order as the first.
+		// So we need to reverse the iterator mode when transferring items,
+		// because if we don't, the second stack will end  up in the incorrect order.
+		$this->setIteratorMode(SplDoublyLinkedList::IT_MODE_FIFO | SplDoublyLinkedList::IT_MODE_KEEP);
+		for ($this->rewind(); $this->valid(); $this->next()) {
+			try {
+				$middleware_app = $this->validateMiddlewareAppDetails($this->current(), true);
+				$middleware_app_class = array_shift($middleware_app);
+				$middleware_app_args = is_array($middleware_app) ? $middleware_app : array();
+				$middleware_app_args = array($application, $this->loader) + $middleware_app_args;
+				$application = $this->loader->getShared($middleware_app_class, $middleware_app_args);
+			} catch (InvalidRequestStackMiddlewareException $exception) {
+				if (WP_DEBUG) {
+					new ExceptionStackTraceDisplay($exception);
+					continue;
+				}
+				error_log($exception->getMessage());
+			}
+		}
+		return new RequestStack($application, $core_app);
+	}
 
 
-    /**
-     * Ensures that the app details that have been pushed onto RequestStackBuilder
-     * are all ordered correctly so that the middleware can be properly constructed
-     *
-     * @param array $middleware_app
-     * @param bool  $recurse
-     * @return array
-     * @throws InvalidRequestStackMiddlewareException
-     */
-    protected function validateMiddlewareAppDetails(array $middleware_app, bool $recurse = false): array
-    {
-        $middleware_app_class = reset($middleware_app);
-        // is array empty ?
-        if ($middleware_app_class === false) {
-            throw new InvalidRequestStackMiddlewareException('invalid middleware');
-        }
-        // are the class and arguments in the wrong order ?
-        if (is_array($middleware_app_class)) {
-            if ($recurse === true) {
-                return $this->validateMiddlewareAppDetails(array_reverse($middleware_app));
-            }
-            throw new InvalidRequestStackMiddlewareException($middleware_app_class);
-        }
-        // is filter callback working like legacy middleware and sending a numerically indexed array ?
-        if (is_int($middleware_app_class)) {
-            if ($recurse === true) {
-                $middleware_app = array_reverse($middleware_app);
-                return $this->validateMiddlewareAppDetails(array(reset($middleware_app), array()));
-            }
-            throw new InvalidRequestStackMiddlewareException($middleware_app_class);
-        }
-        // is $middleware_app_class a valid FQCN (or class is already loaded) ?
-        if (! class_exists($middleware_app_class)) {
-            throw new InvalidRequestStackMiddlewareException($middleware_app_class);
-        }
-        return $middleware_app;
-    }
+	/**
+	 * Ensures that the app details that have been pushed onto RequestStackBuilder
+	 * are all ordered correctly so that the middleware can be properly constructed
+	 *
+	 * @param array $middleware_app
+	 * @param bool  $recurse
+	 * @return array
+	 * @throws InvalidRequestStackMiddlewareException
+	 */
+	protected function validateMiddlewareAppDetails(array $middleware_app, bool $recurse = false): array
+	{
+		$middleware_app_class = reset($middleware_app);
+		// is array empty ?
+		if ($middleware_app_class === false) {
+			throw new InvalidRequestStackMiddlewareException('invalid middleware');
+		}
+		// are the class and arguments in the wrong order ?
+		if (is_array($middleware_app_class)) {
+			if ($recurse === true) {
+				return $this->validateMiddlewareAppDetails(array_reverse($middleware_app));
+			}
+			throw new InvalidRequestStackMiddlewareException($middleware_app_class);
+		}
+		// is filter callback working like legacy middleware and sending a numerically indexed array ?
+		if (is_int($middleware_app_class)) {
+			if ($recurse === true) {
+				$middleware_app = array_reverse($middleware_app);
+				return $this->validateMiddlewareAppDetails(array(reset($middleware_app), array()));
+			}
+			throw new InvalidRequestStackMiddlewareException($middleware_app_class);
+		}
+		// is $middleware_app_class a valid FQCN (or class is already loaded) ?
+		if (! class_exists($middleware_app_class)) {
+			throw new InvalidRequestStackMiddlewareException($middleware_app_class);
+		}
+		return $middleware_app;
+	}
 }