pryley / site-reviews

plugin/Modules/Schema/BaseType.php

Spacing +45 added lines, -45 removed lines

@@ -36,18 +36,18 @@ 
      * @param string $method
      * @return static
      */
-    public function __call($method, array $arguments)
+    public function __call( $method, array $arguments )
     {
-        return $this->setProperty($method, glsr_get($arguments, 0));
+        return $this->setProperty( $method, glsr_get( $arguments, 0 ) );
     }
 
     /**
      * @param string $type
      */
-    public function __construct($type = null)
+    public function __construct( $type = null )
     {
-        $this->type = !is_string($type)
-            ? (new ReflectionClass($this))->getShortName()
+        $this->type = !is_string( $type )
+            ? (new ReflectionClass( $this ))->getShortName()
             : $type;
         $this->setAllowedProperties();
     }
@@ -63,10 +63,10 @@ 
     /**
      * @return static
      */
-    public function addProperties(array $properties)
+    public function addProperties( array $properties )
     {
-        foreach ($properties as $property => $value) {
-            $this->setProperty($property, $value);
+        foreach( $properties as $property => $value ) {
+            $this->setProperty( $property, $value );
         }
         return $this;
     }
@@ -92,9 +92,9 @@ 
      * @param mixed $default
      * @return mixed
      */
-    public function getProperty($property, $default = null)
+    public function getProperty( $property, $default = null )
     {
-        return glsr_get($this->properties, $property, $default);
+        return glsr_get( $this->properties, $property, $default );
     }
 
     /**
@@ -110,10 +110,10 @@ 
      * @param mixed $callback
      * @return static
      */
-    public function doIf($condition, $callback)
+    public function doIf( $condition, $callback )
     {
-        if ($condition) {
-            $callback($this);
+        if( $condition ) {
+            $callback( $this );
         }
         return $this;
     }
@@ -130,18 +130,18 @@ 
      * @param mixed $offset
      * @return bool
      */
-    public function offsetExists($offset)
+    public function offsetExists( $offset )
     {
-        return array_key_exists($offset, $this->properties);
+        return array_key_exists( $offset, $this->properties );
     }
 
     /**
      * @param string $offset
      * @return mixed
      */
-    public function offsetGet($offset)
+    public function offsetGet( $offset )
     {
-        return $this->getProperty($offset);
+        return $this->getProperty( $offset );
     }
 
     /**
@@ -149,16 +149,16 @@ 
      * @param mixed $value
      * @return void
      */
-    public function offsetSet($offset, $value)
+    public function offsetSet( $offset, $value )
     {
-        $this->setProperty($offset, $value);
+        $this->setProperty( $offset, $value );
     }
 
     /**
      * @param string $offset
      * @return void
      */
-    public function offsetUnset($offset)
+    public function offsetUnset( $offset )
     {
         unset($this->properties[$offset]);
     }
@@ -168,11 +168,11 @@ 
      * @param mixed $value
      * @return static
      */
-    public function setProperty($property, $value)
+    public function setProperty( $property, $value )
     {
-        if (!in_array($property, $this->allowed)
-            && 'UnknownType' != (new ReflectionClass($this))->getShortName()) {
-            glsr_log()->warning($this->getType().' does not allow the "'.$property.'" property');
+        if( !in_array( $property, $this->allowed )
+            && 'UnknownType' != (new ReflectionClass( $this ))->getShortName() ) {
+            glsr_log()->warning( $this->getType().' does not allow the "'.$property.'" property' );
             return $this;
         }
         $this->properties[$property] = $value;
@@ -187,7 +187,7 @@ 
         return [
             '@context' => $this->getContext(),
             '@type' => $this->getType(),
-        ] + $this->serializeProperty($this->getProperties());
+        ] + $this->serializeProperty( $this->getProperties() );
     }
 
     /**
@@ -195,8 +195,8 @@ 
      */
     public function toScript()
     {
-        return sprintf('<script type="application/ld+json">%s</script>',
-            json_encode($this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES)
+        return sprintf( '<script type="application/ld+json">%s</script>',
+            json_encode( $this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES )
         );
     }
 
@@ -204,20 +204,20 @@ 
      * @param array|null $parents
      * @return array
      */
-    protected function getParents($parents = null)
+    protected function getParents( $parents = null )
     {
-        if (!isset($parents)) {
+        if( !isset($parents) ) {
             $parents = $this->parents;
         }
         $newParents = $parents;
-        foreach ($parents as $parent) {
-            $parentClass = glsr(Helper::class)->buildClassName($parent, __NAMESPACE__);
-            if (!class_exists($parentClass)) {
+        foreach( $parents as $parent ) {
+            $parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
+            if( !class_exists( $parentClass ) ) {
                 continue;
             }
-            $newParents = array_merge($newParents, $this->getParents((new $parentClass())->parents));
+            $newParents = array_merge( $newParents, $this->getParents( (new $parentClass())->parents ) );
         }
-        return array_values(array_unique($newParents));
+        return array_values( array_unique( $newParents ) );
     }
 
     /**
@@ -226,12 +226,12 @@ 
     protected function setAllowedProperties()
     {
         $parents = $this->getParents();
-        foreach ($parents as $parent) {
-            $parentClass = glsr(Helper::class)->buildClassName($parent, __NAMESPACE__);
-            if (!class_exists($parentClass)) {
+        foreach( $parents as $parent ) {
+            $parentClass = glsr( Helper::class )->buildClassName( $parent, __NAMESPACE__ );
+            if( !class_exists( $parentClass ) ) {
                 continue;
             }
-            $this->allowed = array_values(array_unique(array_merge((new $parentClass())->allowed, $this->allowed)));
+            $this->allowed = array_values( array_unique( array_merge( (new $parentClass())->allowed, $this->allowed ) ) );
         }
     }
 
@@ -239,19 +239,19 @@ 
      * @param mixed $property
      * @return array|string
      */
-    protected function serializeProperty($property)
+    protected function serializeProperty( $property )
     {
-        if (is_array($property)) {
-            return array_map([$this, 'serializeProperty'], $property);
+        if( is_array( $property ) ) {
+            return array_map( [$this, 'serializeProperty'], $property );
         }
-        if ($property instanceof Type) {
+        if( $property instanceof Type ) {
             $property = $property->toArray();
             unset($property['@context']);
         }
-        if ($property instanceof DateTimeInterface) {
-            $property = $property->format(DateTime::ATOM);
+        if( $property instanceof DateTimeInterface ) {
+            $property = $property->format( DateTime::ATOM );
         }
-        if (is_object($property)) {
+        if( is_object( $property ) ) {
             throw new InvalidProperty();
         }
         return $property;

plugin/Modules/Validator.php

Spacing +70 added lines, -70 removed lines

@@ -66,14 +66,14 @@ 
      * @param mixed $data
      * @return array
      */
-    public function validate($data, array $rules = [])
+    public function validate( $data, array $rules = [] )
     {
-        $this->normalizeData($data);
-        $this->setRules($rules);
-        foreach ($this->rules as $attribute => $rules) {
-            foreach ($rules as $rule) {
-                $this->validateAttribute($attribute, $rule);
-                if ($this->shouldStopValidating($attribute)) {
+        $this->normalizeData( $data );
+        $this->setRules( $rules );
+        foreach( $this->rules as $attribute => $rules ) {
+            foreach( $rules as $rule ) {
+                $this->validateAttribute( $attribute, $rule );
+                if( $this->shouldStopValidating( $attribute ) ) {
                     break;
                 }
             }
@@ -88,18 +88,18 @@ 
      * @return void
      * @throws BadMethodCallException
      */
-    public function validateAttribute($attribute, $rule)
+    public function validateAttribute( $attribute, $rule )
     {
-        list($rule, $parameters) = $this->parseRule($rule);
-        if ('' == $rule) {
+        list($rule, $parameters) = $this->parseRule( $rule );
+        if( '' == $rule ) {
             return;
         }
-        $value = $this->getValue($attribute);
-        if (!method_exists($this, $method = 'validate'.$rule)) {
-            throw new BadMethodCallException("Method [$method] does not exist.");
+        $value = $this->getValue( $attribute );
+        if( !method_exists( $this, $method = 'validate'.$rule ) ) {
+            throw new BadMethodCallException( "Method [$method] does not exist." );
         }
-        if (!$this->$method($value, $attribute, $parameters)) {
-            $this->addFailure($attribute, $rule, $parameters);
+        if( !$this->$method( $value, $attribute, $parameters ) ) {
+            $this->addFailure( $attribute, $rule, $parameters );
         }
     }
 
@@ -109,9 +109,9 @@ 
      * @param string $rule
      * @return void
      */
-    protected function addError($attribute, $rule, array $parameters)
+    protected function addError( $attribute, $rule, array $parameters )
     {
-        $message = $this->getMessage($attribute, $rule, $parameters);
+        $message = $this->getMessage( $attribute, $rule, $parameters );
         $this->errors[$attribute][] = $message;
     }
 
@@ -121,9 +121,9 @@ 
      * @param string $rule
      * @return void
      */
-    protected function addFailure($attribute, $rule, array $parameters)
+    protected function addFailure( $attribute, $rule, array $parameters )
     {
-        $this->addError($attribute, $rule, $parameters);
+        $this->addError( $attribute, $rule, $parameters );
         $this->failedRules[$attribute][$rule] = $parameters;
     }
 
@@ -132,9 +132,9 @@ 
      * @param string $attribute
      * @return string
      */
-    protected function getAttributeType($attribute)
+    protected function getAttributeType( $attribute )
     {
-        return !$this->hasRule($attribute, $this->numericRules)
+        return !$this->hasRule( $attribute, $this->numericRules )
             ? 'length'
             : '';
     }
@@ -145,13 +145,13 @@ 
      * @param string $rule
      * @return string|null
      */
-    protected function getMessage($attribute, $rule, array $parameters)
+    protected function getMessage( $attribute, $rule, array $parameters )
     {
-        if (in_array($rule, $this->sizeRules)) {
-            return $this->getSizeMessage($attribute, $rule, $parameters);
+        if( in_array( $rule, $this->sizeRules ) ) {
+            return $this->getSizeMessage( $attribute, $rule, $parameters );
         }
-        $lowerRule = glsr(Helper::class)->snakeCase($rule);
-        return $this->translator($lowerRule, $parameters);
+        $lowerRule = glsr( Helper::class )->snakeCase( $rule );
+        return $this->translator( $lowerRule, $parameters );
     }
 
     /**
@@ -160,15 +160,15 @@ 
      * @param string|array $rules
      * @return array|null
      */
-    protected function getRule($attribute, $rules)
+    protected function getRule( $attribute, $rules )
     {
-        if (!array_key_exists($attribute, $this->rules)) {
+        if( !array_key_exists( $attribute, $this->rules ) ) {
             return;
         }
-        $rules = (array) $rules;
-        foreach ($this->rules[$attribute] as $rule) {
-            list($rule, $parameters) = $this->parseRule($rule);
-            if (in_array($rule, $rules)) {
+        $rules = (array)$rules;
+        foreach( $this->rules[$attribute] as $rule ) {
+            list($rule, $parameters) = $this->parseRule( $rule );
+            if( in_array( $rule, $rules ) ) {
                 return [$rule, $parameters];
             }
         }
@@ -180,17 +180,17 @@ 
      * @param mixed $value
      * @return mixed
      */
-    protected function getSize($attribute, $value)
+    protected function getSize( $attribute, $value )
     {
-        $hasNumeric = $this->hasRule($attribute, $this->numericRules);
-        if (is_numeric($value) && $hasNumeric) {
+        $hasNumeric = $this->hasRule( $attribute, $this->numericRules );
+        if( is_numeric( $value ) && $hasNumeric ) {
             return $value;
-        } elseif (is_array($value)) {
-            return count($value);
+        } elseif( is_array( $value ) ) {
+            return count( $value );
         }
-        return function_exists('mb_strlen')
-            ? mb_strlen($value)
-            : strlen($value);
+        return function_exists( 'mb_strlen' )
+            ? mb_strlen( $value )
+            : strlen( $value );
     }
 
     /**
@@ -199,11 +199,11 @@ 
      * @param string $rule
      * @return string|null
      */
-    protected function getSizeMessage($attribute, $rule, array $parameters)
+    protected function getSizeMessage( $attribute, $rule, array $parameters )
     {
-        $type = $this->getAttributeType($attribute);
-        $lowerRule = glsr(Helper::class)->snakeCase($rule.$type);
-        return $this->translator($lowerRule, $parameters);
+        $type = $this->getAttributeType( $attribute );
+        $lowerRule = glsr( Helper::class )->snakeCase( $rule.$type );
+        return $this->translator( $lowerRule, $parameters );
     }
 
     /**
@@ -211,9 +211,9 @@ 
      * @param string $attribute
      * @return mixed
      */
-    protected function getValue($attribute)
+    protected function getValue( $attribute )
     {
-        if (isset($this->data[$attribute])) {
+        if( isset($this->data[$attribute]) ) {
             return $this->data[$attribute];
         }
     }
@@ -224,9 +224,9 @@ 
      * @param string|array $rules
      * @return bool
      */
-    protected function hasRule($attribute, $rules)
+    protected function hasRule( $attribute, $rules )
     {
-        return !is_null($this->getRule($attribute, $rules));
+        return !is_null( $this->getRule( $attribute, $rules ) );
     }
 
     /**
@@ -234,10 +234,10 @@ 
      * @param mixed $data
      * @return void
      */
-    protected function normalizeData($data)
+    protected function normalizeData( $data )
     {
-        $this->data = is_object($data)
-            ? get_object_vars($data)
+        $this->data = is_object( $data )
+            ? get_object_vars( $data )
             : $data;
     }
 
@@ -247,11 +247,11 @@ 
      * @param string $parameter
      * @return array
      */
-    protected function parseParameters($rule, $parameter)
+    protected function parseParameters( $rule, $parameter )
     {
-        return 'regex' == strtolower($rule)
+        return 'regex' == strtolower( $rule )
             ? [$parameter]
-            : str_getcsv($parameter);
+            : str_getcsv( $parameter );
     }
 
     /**
@@ -259,14 +259,14 @@ 
      * @param string $rule
      * @return array
      */
-    protected function parseRule($rule)
+    protected function parseRule( $rule )
     {
         $parameters = [];
-        if (false !== strpos($rule, ':')) {
-            list($rule, $parameter) = explode(':', $rule, 2);
-            $parameters = $this->parseParameters($rule, $parameter);
+        if( false !== strpos( $rule, ':' ) ) {
+            list($rule, $parameter) = explode( ':', $rule, 2 );
+            $parameters = $this->parseParameters( $rule, $parameter );
         }
-        $rule = glsr(Helper::class)->camelCase($rule);
+        $rule = glsr( Helper::class )->camelCase( $rule );
         return [$rule, $parameters];
     }
 
@@ -274,11 +274,11 @@ 
      * Set the validation rules.
      * @return void
      */
-    protected function setRules(array $rules)
+    protected function setRules( array $rules )
     {
-        foreach ($rules as $key => $rule) {
-            $rules[$key] = is_string($rule)
-                ? explode('|', $rule)
+        foreach( $rules as $key => $rule ) {
+            $rules[$key] = is_string( $rule )
+                ? explode( '|', $rule )
                 : $rule;
         }
         $this->rules = $rules;
@@ -289,11 +289,11 @@ 
      * @param string $attribute
      * @return bool
      */
-    protected function shouldStopValidating($attribute)
+    protected function shouldStopValidating( $attribute )
     {
-        return $this->hasRule($attribute, $this->implicitRules)
+        return $this->hasRule( $attribute, $this->implicitRules )
             && isset($this->failedRules[$attribute])
-            && array_intersect(array_keys($this->failedRules[$attribute]), $this->implicitRules);
+            && array_intersect( array_keys( $this->failedRules[$attribute] ), $this->implicitRules );
     }
 
     /**
@@ -302,11 +302,11 @@ 
      * @param string $attribute
      * @return void|string
      */
-    protected function translator($key, array $parameters)
+    protected function translator( $key, array $parameters )
     {
-        $strings = glsr(ValidationStringsDefaults::class)->defaults();
-        if (isset($strings[$key])) {
-            return $this->replace($strings[$key], $parameters);
+        $strings = glsr( ValidationStringsDefaults::class )->defaults();
+        if( isset($strings[$key]) ) {
+            return $this->replace( $strings[$key], $parameters );
         }
         return 'error';
     }

plugin/Modules/Validator/ValidationRules.php

Spacing +29 added lines, -29 removed lines

@@ -15,23 +15,23 @@ 
      * @param mixed $value
      * @return mixed
      */
-    abstract protected function getSize($attribute, $value);
+    abstract protected function getSize( $attribute, $value );
 
     /**
      * Replace all placeholders.
      * @param string $message
      * @return string
      */
-    protected function replace($message, array $parameters)
+    protected function replace( $message, array $parameters )
     {
-        if (false === strpos($message, '%s')) {
+        if( false === strpos( $message, '%s' ) ) {
             return $message;
         }
-        return preg_replace_callback('/(%s)/', function () use (&$parameters) {
-            foreach ($parameters as $key => $value) {
-                return array_shift($parameters);
+        return preg_replace_callback( '/(%s)/', function() use (&$parameters) {
+            foreach( $parameters as $key => $value ) {
+                return array_shift( $parameters );
             }
-        }, $message);
+        }, $message );
     }
 
     /**
@@ -41,10 +41,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateAccepted($value)
+    public function validateAccepted( $value )
     {
         $acceptable = ['yes', 'on', '1', 1, true, 'true'];
-        return $this->validateRequired($value) && in_array($value, $acceptable, true);
+        return $this->validateRequired( $value ) && in_array( $value, $acceptable, true );
     }
 
     /**
@@ -53,10 +53,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateBetween($value, $attribute, array $parameters)
+    public function validateBetween( $value, $attribute, array $parameters )
     {
-        $this->requireParameterCount(2, $parameters, 'between');
-        $size = $this->getSize($attribute, $value);
+        $this->requireParameterCount( 2, $parameters, 'between' );
+        $size = $this->getSize( $attribute, $value );
         return $size >= $parameters[0] && $size <= $parameters[1];
     }
 
@@ -65,9 +65,9 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateEmail($value)
+    public function validateEmail( $value )
     {
-        return false !== filter_var($value, FILTER_VALIDATE_EMAIL);
+        return false !== filter_var( $value, FILTER_VALIDATE_EMAIL );
     }
 
     /**
@@ -76,10 +76,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateMax($value, $attribute, array $parameters)
+    public function validateMax( $value, $attribute, array $parameters )
     {
-        $this->requireParameterCount(1, $parameters, 'max');
-        return $this->getSize($attribute, $value) <= $parameters[0];
+        $this->requireParameterCount( 1, $parameters, 'max' );
+        return $this->getSize( $attribute, $value ) <= $parameters[0];
     }
 
     /**
@@ -88,10 +88,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateMin($value, $attribute, array $parameters)
+    public function validateMin( $value, $attribute, array $parameters )
     {
-        $this->requireParameterCount(1, $parameters, 'min');
-        return $this->getSize($attribute, $value) >= $parameters[0];
+        $this->requireParameterCount( 1, $parameters, 'min' );
+        return $this->getSize( $attribute, $value ) >= $parameters[0];
     }
 
     /**
@@ -99,9 +99,9 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateNumber($value)
+    public function validateNumber( $value )
     {
-        return is_numeric($value);
+        return is_numeric( $value );
     }
 
     /**
@@ -109,11 +109,11 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateRequired($value)
+    public function validateRequired( $value )
     {
-        return is_null($value)
-            || (is_string($value) && '' === trim($value))
-            || (is_array($value) && count($value) < 1)
+        return is_null( $value )
+            || (is_string( $value ) && '' === trim( $value ))
+            || (is_array( $value ) && count( $value ) < 1)
             ? false
             : true;
     }
@@ -125,10 +125,10 @@ 
      * @return void
      * @throws InvalidArgumentException
      */
-    protected function requireParameterCount($count, array $parameters, $rule)
+    protected function requireParameterCount( $count, array $parameters, $rule )
     {
-        if (count($parameters) < $count) {
-            throw new InvalidArgumentException("Validation rule $rule requires at least $count parameters.");
+        if( count( $parameters ) < $count ) {
+            throw new InvalidArgumentException( "Validation rule $rule requires at least $count parameters." );
         }
     }
 }

plugin/Contracts/ShortcodeContract.php

Spacing +1 added lines, -1 removed lines

@@ -8,5 +8,5 @@
      * @params string|array $atts
      * @return string
      */
-    public function buildShortcode($atts = []);
+    public function buildShortcode( $atts = [] );
 }

plugin/Contracts/PartialContract.php

Spacing +1 added lines, -1 removed lines

@@ -7,5 +7,5 @@
     /**
      * @return void|string
      */
-    public function build(array $args = []);
+    public function build( array $args = [] );
 }

plugin/Contracts/ProviderContract.php

Spacing +1 added lines, -1 removed lines

@@ -9,5 +9,5 @@
     /**
      * @return void
      */
-    public function register(Application $app);
+    public function register( Application $app );
 }

plugin/Database/Cache.php

Spacing +23 added lines, -23 removed lines

@@ -13,25 +13,25 @@ 
      */
     public function getCloudflareIps()
     {
-        $ipAddresses = get_transient(Application::ID.'_cloudflare_ips');
-        if (false === $ipAddresses) {
-            $ipAddresses = array_fill_keys(['v4', 'v6'], []);
-            foreach (array_keys($ipAddresses) as $version) {
+        $ipAddresses = get_transient( Application::ID.'_cloudflare_ips' );
+        if( false === $ipAddresses ) {
+            $ipAddresses = array_fill_keys( ['v4', 'v6'], [] );
+            foreach( array_keys( $ipAddresses ) as $version ) {
                 $url = 'https://www.cloudflare.com/ips-'.$version;
-                $response = wp_remote_get($url);
-                if (is_wp_error($response)) {
-                    glsr_log()->error($response->get_error_message());
+                $response = wp_remote_get( $url );
+                if( is_wp_error( $response ) ) {
+                    glsr_log()->error( $response->get_error_message() );
                     continue;
                 }
-                if ('200' != ($statusCode = wp_remote_retrieve_response_code($response))) {
-                    glsr_log()->error('Unable to connect to '.$url.' ['.$statusCode.']');
+                if( '200' != ($statusCode = wp_remote_retrieve_response_code( $response )) ) {
+                    glsr_log()->error( 'Unable to connect to '.$url.' ['.$statusCode.']' );
                     continue;
                 }
                 $ipAddresses[$version] = array_filter(
-                    (array) preg_split('/\R/', wp_remote_retrieve_body($response))
+                    (array)preg_split( '/\R/', wp_remote_retrieve_body( $response ) )
                 );
             }
-            set_transient(Application::ID.'_cloudflare_ips', $ipAddresses, static::EXPIRY_TIME);
+            set_transient( Application::ID.'_cloudflare_ips', $ipAddresses, static::EXPIRY_TIME );
         }
         return $ipAddresses;
     }
@@ -40,16 +40,16 @@ 
      * @param string $metaKey
      * @return array
      */
-    public function getReviewCountsFor($metaKey)
+    public function getReviewCountsFor( $metaKey )
     {
-        $counts = wp_cache_get(Application::ID, $metaKey.'_count');
-        if (false === $counts) {
+        $counts = wp_cache_get( Application::ID, $metaKey.'_count' );
+        if( false === $counts ) {
             $counts = [];
-            $results = glsr(SqlQueries::class)->getReviewCountsFor($metaKey);
-            foreach ($results as $result) {
+            $results = glsr( SqlQueries::class )->getReviewCountsFor( $metaKey );
+            foreach( $results as $result ) {
                 $counts[$result->name] = $result->num_posts;
             }
-            wp_cache_set(Application::ID, $counts, $metaKey.'_count');
+            wp_cache_set( Application::ID, $counts, $metaKey.'_count' );
         }
         return $counts;
     }
@@ -59,14 +59,14 @@ 
      */
     public function getRemotePostTest()
     {
-        $test = get_transient(Application::ID.'_remote_post_test');
-        if (false === $test) {
-            $response = wp_remote_post('https://api.wordpress.org/stats/php/1.0/');
-            $test = !is_wp_error($response)
-                && in_array($response['response']['code'], range(200, 299))
+        $test = get_transient( Application::ID.'_remote_post_test' );
+        if( false === $test ) {
+            $response = wp_remote_post( 'https://api.wordpress.org/stats/php/1.0/' );
+            $test = !is_wp_error( $response )
+                && in_array( $response['response']['code'], range( 200, 299 ) )
                 ? 'Works'
                 : 'Does not work';
-            set_transient(Application::ID.'_remote_post_test', $test, static::EXPIRY_TIME);
+            set_transient( Application::ID.'_remote_post_test', $test, static::EXPIRY_TIME );
         }
         return $test;
     }

plugin/Blocks/SiteReviewsSummaryBlock.php

Spacing +9 added lines, -9 removed lines

@@ -50,18 +50,18 @@ 
     /**
      * @return string
      */
-    public function render(array $attributes)
+    public function render( array $attributes )
     {
         $attributes['class'] = $attributes['className'];
-        $shortcode = glsr(Shortcode::class);
-        if ('edit' == filter_input(INPUT_GET, 'context')) {
-            $attributes = $this->normalize($attributes);
+        $shortcode = glsr( Shortcode::class );
+        if( 'edit' == filter_input( INPUT_GET, 'context' ) ) {
+            $attributes = $this->normalize( $attributes );
             $this->filterShortcodeClass();
-            if (!$this->hasVisibleFields($shortcode, $attributes)) {
+            if( !$this->hasVisibleFields( $shortcode, $attributes ) ) {
                 $this->filterInterpolation();
             }
         }
-        return $shortcode->buildShortcode($attributes);
+        return $shortcode->buildShortcode( $attributes );
     }
 
     /**
@@ -69,9 +69,9 @@ 
      */
     protected function filterInterpolation()
     {
-        add_filter('site-reviews/interpolate/reviews-summary', function ($context) {
+        add_filter( 'site-reviews/interpolate/reviews-summary', function( $context ) {
             $context['class'] = 'glsr-default glsr-block-disabled';
-            $context['text'] = __('You have hidden all of the fields for this block.', 'site-reviews');
+            $context['text'] = __( 'You have hidden all of the fields for this block.', 'site-reviews' );
             return $context;
         });
     }
@@ -81,7 +81,7 @@ 
      */
     protected function filterShortcodeClass()
     {
-        add_filter('site-reviews/style', function () {
+        add_filter( 'site-reviews/style', function() {
             return 'default';
         });
     }

plugin/Blocks/SiteReviewsFormBlock.php

Spacing +18 added lines, -18 removed lines

@@ -38,20 +38,20 @@ 
     /**
      * @return string
      */
-    public function render(array $attributes)
+    public function render( array $attributes )
     {
         $attributes['class'] = $attributes['className'];
-        $shortcode = glsr(Shortcode::class);
-        if ('edit' == filter_input(INPUT_GET, 'context')) {
+        $shortcode = glsr( Shortcode::class );
+        if( 'edit' == filter_input( INPUT_GET, 'context' ) ) {
             $this->filterFormFields();
             $this->filterRatingField();
             $this->filterShortcodeClass();
             $this->filterSubmitButton();
-            if (!$this->hasVisibleFields($shortcode, $attributes)) {
+            if( !$this->hasVisibleFields( $shortcode, $attributes ) ) {
                 $this->filterInterpolation();
             }
         }
-        return $shortcode->buildShortcode($attributes);
+        return $shortcode->buildShortcode( $attributes );
     }
 
     /**
@@ -59,8 +59,8 @@ 
      */
     protected function filterFormFields()
     {
-        add_filter('site-reviews/config/forms/submission-form', function (array $config) {
-            array_walk($config, function (&$field) {
+        add_filter( 'site-reviews/config/forms/submission-form', function( array $config ) {
+            array_walk( $config, function( &$field ) {
                 $field['disabled'] = true;
                 $field['tabindex'] = '-1';
             });
@@ -73,9 +73,9 @@ 
      */
     protected function filterInterpolation()
     {
-        add_filter('site-reviews/interpolate/reviews-form', function ($context) {
+        add_filter( 'site-reviews/interpolate/reviews-form', function( $context ) {
             $context['class'] = 'glsr-default glsr-block-disabled';
-            $context['fields'] = __('You have hidden all of the fields for this block.', 'site-reviews');
+            $context['fields'] = __( 'You have hidden all of the fields for this block.', 'site-reviews' );
             $context['response'] = '';
             $context['submit_button'] = '';
             return $context;
@@ -87,15 +87,15 @@ 
      */
     protected function filterRatingField()
     {
-        add_filter('site-reviews/rendered/field', function ($html, $type, $args) {
-            if ('rating' == $args['path']) {
+        add_filter( 'site-reviews/rendered/field', function( $html, $type, $args ) {
+            if( 'rating' == $args['path'] ) {
                 $stars = '<span class="glsr-stars">';
-                $stars.= str_repeat('<span class="glsr-star glsr-star-empty" aria-hidden="true"></span>', 5);
-                $stars.= '</span>';
-                $html = preg_replace('/(.*)(<select.*)(<\/select>)(.*)/', '$1'.$stars.'$4', $html);
+                $stars .= str_repeat( '<span class="glsr-star glsr-star-empty" aria-hidden="true"></span>', 5 );
+                $stars .= '</span>';
+                $html = preg_replace( '/(.*)(<select.*)(<\/select>)(.*)/', '$1'.$stars.'$4', $html );
             }
             return $html;
-        }, 10, 3);
+        }, 10, 3 );
     }
 
     /**
@@ -103,7 +103,7 @@ 
      */
     protected function filterShortcodeClass()
     {
-        add_filter('site-reviews/style', function () {
+        add_filter( 'site-reviews/style', function() {
             return 'default';
         });
     }
@@ -113,8 +113,8 @@ 
      */
     protected function filterSubmitButton()
     {
-        add_filter('site-reviews/rendered/template/form/submit-button', function ($template) {
-            return str_replace('type="submit"', 'tabindex="-1"', $template);
+        add_filter( 'site-reviews/rendered/template/form/submit-button', function( $template ) {
+            return str_replace( 'type="submit"', 'tabindex="-1"', $template );
         });
     }
 }