pryley / site-reviews

plugin/Modules/Translation.php

Spacing +71 added lines, -71 removed lines

@@ -30,10 +30,10 @@ 
     public function all()
     {
         $translations = $this->translations();
-        $entries = $this->filter($translations, $this->entries())->results();
-        array_walk($translations, function (&$entry) use ($entries) {
-            $entry['desc'] = array_key_exists($entry['id'], $entries)
-                ? $this->getEntryString($entries[$entry['id']], 'msgctxt')
+        $entries = $this->filter( $translations, $this->entries() )->results();
+        array_walk( $translations, function( &$entry ) use ($entries) {
+            $entry['desc'] = array_key_exists( $entry['id'], $entries )
+                ? $this->getEntryString( $entries[$entry['id']], 'msgctxt' )
                 : '';
         });
         return $translations;
@@ -44,10 +44,10 @@ 
      */
     public function entries()
     {
-        if (!isset($this->entries)) {
-            $potFile = glsr()->path(glsr()->languages.'/'.Application::ID.'.pot');
-            $entries = $this->extractEntriesFromPotFile($potFile);
-            $entries = apply_filters('site-reviews/translation/entries', $entries);
+        if( !isset($this->entries) ) {
+            $potFile = glsr()->path( glsr()->languages.'/'.Application::ID.'.pot' );
+            $entries = $this->extractEntriesFromPotFile( $potFile );
+            $entries = apply_filters( 'site-reviews/translation/entries', $entries );
             $this->entries = $entries;
         }
         return $this->entries;
@@ -58,24 +58,24 @@ 
      * @param array|null $entries
      * @return static
      */
-    public function exclude($entriesToExclude = null, $entries = null)
+    public function exclude( $entriesToExclude = null, $entries = null )
     {
-        return $this->filter($entriesToExclude, $entries, false);
+        return $this->filter( $entriesToExclude, $entries, false );
     }
 
     /**
      * @param string $potFile
      * @return array
      */
-    public function extractEntriesFromPotFile($potFile, array $entries = [])
+    public function extractEntriesFromPotFile( $potFile, array $entries = [] )
     {
         try {
-            $potEntries = $this->normalize(Parser::parseFile($potFile)->getEntries());
-            foreach ($potEntries as $key => $entry) {
-                $entries[html_entity_decode($key, ENT_COMPAT, 'UTF-8')] = $entry;
+            $potEntries = $this->normalize( Parser::parseFile( $potFile )->getEntries() );
+            foreach( $potEntries as $key => $entry ) {
+                $entries[html_entity_decode( $key, ENT_COMPAT, 'UTF-8' )] = $entry;
             }
-        } catch (Exception $e) {
-            glsr_log()->error($e->getMessage());
+        } catch( Exception $e ) {
+            glsr_log()->error( $e->getMessage() );
         }
         return $entries;
     }
@@ -86,18 +86,18 @@ 
      * @param bool $intersect
      * @return static
      */
-    public function filter($filterWith = null, $entries = null, $intersect = true)
+    public function filter( $filterWith = null, $entries = null, $intersect = true )
     {
-        if (!is_array($entries)) {
+        if( !is_array( $entries ) ) {
             $entries = $this->results;
         }
-        if (!is_array($filterWith)) {
+        if( !is_array( $filterWith ) ) {
             $filterWith = $this->translations();
         }
-        $keys = array_flip(glsr_array_column($filterWith, 'id'));
+        $keys = array_flip( glsr_array_column( $filterWith, 'id' ) );
         $this->results = $intersect
-            ? array_intersect_key($entries, $keys)
-            : array_diff_key($entries, $keys);
+            ? array_intersect_key( $entries, $keys )
+            : array_diff_key( $entries, $keys );
         return $this;
     }
 
@@ -105,20 +105,20 @@ 
      * @param string $template
      * @return string
      */
-    public function render($template, array $entry)
+    public function render( $template, array $entry )
     {
         $data = array_combine(
-            array_map(function ($key) { return 'data.'.$key; }, array_keys($entry)),
+            array_map( function( $key ) { return 'data.'.$key; }, array_keys( $entry ) ),
             $entry
         );
         $data['data.class'] = $data['data.error'] = '';
-        if (false === array_search($entry['s1'], glsr_array_column($this->entries(), 'msgid'))) {
+        if( false === array_search( $entry['s1'], glsr_array_column( $this->entries(), 'msgid' ) ) ) {
             $data['data.class'] = 'is-invalid';
-            $data['data.error'] = __('This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews');
+            $data['data.error'] = __( 'This custom translation is no longer valid as the original text has been changed or removed.', 'site-reviews' );
         }
-        return glsr(Template::class)->build('partials/translations/'.$template, [
-            'context' => array_map('esc_html', $data),
-        ]);
+        return glsr( Template::class )->build( 'partials/translations/'.$template, [
+            'context' => array_map( 'esc_html', $data ),
+        ] );
     }
 
     /**
@@ -128,10 +128,10 @@ 
     public function renderAll()
     {
         $rendered = '';
-        foreach ($this->all() as $index => $entry) {
+        foreach( $this->all() as $index => $entry ) {
             $entry['index'] = $index;
             $entry['prefix'] = OptionManager::databaseKey();
-            $rendered.= $this->render($entry['type'], $entry);
+            $rendered .= $this->render( $entry['type'], $entry );
         }
         return $rendered;
     }
@@ -140,25 +140,25 @@ 
      * @param bool $resetAfterRender
      * @return string
      */
-    public function renderResults($resetAfterRender = true)
+    public function renderResults( $resetAfterRender = true )
     {
         $rendered = '';
-        foreach ($this->results as $id => $entry) {
+        foreach( $this->results as $id => $entry ) {
             $data = [
-                'desc' => $this->getEntryString($entry, 'msgctxt'),
+                'desc' => $this->getEntryString( $entry, 'msgctxt' ),
                 'id' => $id,
-                'p1' => $this->getEntryString($entry, 'msgid_plural'),
-                's1' => $this->getEntryString($entry, 'msgid'),
+                'p1' => $this->getEntryString( $entry, 'msgid_plural' ),
+                's1' => $this->getEntryString( $entry, 'msgid' ),
             ];
             $text = !empty($data['p1'])
-                ? sprintf('%s | %s', $data['s1'], $data['p1'])
+                ? sprintf( '%s | %s', $data['s1'], $data['p1'] )
                 : $data['s1'];
-            $rendered.= $this->render('result', [
-                'entry' => json_encode($data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE),
-                'text' => wp_strip_all_tags($text),
-            ]);
+            $rendered .= $this->render( 'result', [
+                'entry' => json_encode( $data, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE ),
+                'text' => wp_strip_all_tags( $text ),
+            ] );
         }
-        if ($resetAfterRender) {
+        if( $resetAfterRender ) {
             $this->reset();
         }
         return $rendered;
@@ -186,18 +186,18 @@ 
      * @param string $needle
      * @return static
      */
-    public function search($needle = '')
+    public function search( $needle = '' )
     {
         $this->reset();
-        $needle = trim(strtolower($needle));
-        foreach ($this->entries() as $key => $entry) {
-            $single = strtolower($this->getEntryString($entry, 'msgid'));
-            $plural = strtolower($this->getEntryString($entry, 'msgid_plural'));
-            if (strlen($needle) < static::SEARCH_THRESHOLD) {
-                if (in_array($needle, [$single, $plural])) {
+        $needle = trim( strtolower( $needle ) );
+        foreach( $this->entries() as $key => $entry ) {
+            $single = strtolower( $this->getEntryString( $entry, 'msgid' ) );
+            $plural = strtolower( $this->getEntryString( $entry, 'msgid_plural' ) );
+            if( strlen( $needle ) < static::SEARCH_THRESHOLD ) {
+                if( in_array( $needle, [$single, $plural] ) ) {
                     $this->results[$key] = $entry;
                 }
-            } elseif (Str::contains(sprintf('%s %s', $single, $plural), $needle)) {
+            } elseif( Str::contains( sprintf( '%s %s', $single, $plural ), $needle ) ) {
                 $this->results[$key] = $entry;
             }
         }
@@ -211,10 +211,10 @@ 
     public function translations()
     {
         static $translations;
-        if (empty($translations)) {
-            $settings = glsr(OptionManager::class)->get('settings');
+        if( empty($translations) ) {
+            $settings = glsr( OptionManager::class )->get( 'settings' );
             $translations = isset($settings['strings'])
-                ? $this->normalizeSettings((array) $settings['strings'])
+                ? $this->normalizeSettings( (array)$settings['strings'] )
                 : [];
         }
         return $translations;
@@ -224,28 +224,28 @@ 
      * @param string $key
      * @return string
      */
-    protected function getEntryString(array $entry, $key)
+    protected function getEntryString( array $entry, $key )
     {
         return isset($entry[$key])
-            ? implode('', (array) $entry[$key])
+            ? implode( '', (array)$entry[$key] )
             : '';
     }
 
     /**
      * @return array
      */
-    protected function normalize(array $entries)
+    protected function normalize( array $entries )
     {
         $keys = [
             'msgctxt', 'msgid', 'msgid_plural', 'msgstr', 'msgstr[0]', 'msgstr[1]',
         ];
-        array_walk($entries, function (&$entry) use ($keys) {
-            foreach ($keys as $key) {
+        array_walk( $entries, function( &$entry ) use ($keys) {
+            foreach( $keys as $key ) {
                 try {
-                    $entry = $this->normalizeEntryString($entry, $key);
-                } catch (\TypeError $error) {
-                    glsr_log()->once('error', 'Translation/normalize', $error);
-                    glsr_log()->once('debug', 'Translation/normalize', $entry);
+                    $entry = $this->normalizeEntryString( $entry, $key );
+                } catch( \TypeError $error ) {
+                    glsr_log()->once( 'error', 'Translation/normalize', $error );
+                    glsr_log()->once( 'debug', 'Translation/normalize', $entry );
                 }
             }
         });
@@ -256,10 +256,10 @@ 
      * @param string $key
      * @return array
      */
-    protected function normalizeEntryString(array $entry, $key)
+    protected function normalizeEntryString( array $entry, $key )
     {
-        if (isset($entry[$key])) {
-            $entry[$key] = $this->getEntryString($entry, $key);
+        if( isset($entry[$key]) ) {
+            $entry[$key] = $this->getEntryString( $entry, $key );
         }
         return $entry;
     }
@@ -267,15 +267,15 @@ 
     /**
      * @return array
      */
-    protected function normalizeSettings(array $strings)
+    protected function normalizeSettings( array $strings )
     {
-        $defaultString = array_fill_keys(['id', 's1', 's2', 'p1', 'p2'], '');
-        $strings = array_filter($strings, 'is_array');
-        foreach ($strings as &$string) {
+        $defaultString = array_fill_keys( ['id', 's1', 's2', 'p1', 'p2'], '' );
+        $strings = array_filter( $strings, 'is_array' );
+        foreach( $strings as &$string ) {
             $string['type'] = isset($string['p1']) ? 'plural' : 'single';
-            $string = wp_parse_args($string, $defaultString);
+            $string = wp_parse_args( $string, $defaultString );
         }
-        return array_filter($strings, function ($string) {
+        return array_filter( $strings, function( $string ) {
             return !empty($string['id']);
         });
     }

plugin/Modules/Date.php

Spacing +14 added lines, -14 removed lines

@@ -28,26 +28,26 @@
     /**
      * @return string
      */
-    public function relative($date)
+    public function relative( $date )
     {
-        $diff = time() - strtotime($date);
-        foreach (static::$TIME_PERIODS as $i => $timePeriod) {
-            if ($diff > $timePeriod[0]) {
+        $diff = time() - strtotime( $date );
+        foreach( static::$TIME_PERIODS as $i => $timePeriod ) {
+            if( $diff > $timePeriod[0] ) {
                 continue;
             }
-            $unit = intval(floor($diff / $timePeriod[1]));
+            $unit = intval( floor( $diff / $timePeriod[1] ) );
             $relativeDates = [
-                _n('%s second ago', '%s seconds ago', $unit, 'site-reviews'),
-                _n('%s minute ago', '%s minutes ago', $unit, 'site-reviews'),
-                _n('an hour ago', '%s hours ago', $unit, 'site-reviews'),
-                _n('yesterday', '%s days ago', $unit, 'site-reviews'),
-                _n('a week ago', '%s weeks ago', $unit, 'site-reviews'),
-                _n('%s month ago', '%s months ago', $unit, 'site-reviews'),
-                _n('%s year ago', '%s years ago', $unit, 'site-reviews'),
+                _n( '%s second ago', '%s seconds ago', $unit, 'site-reviews' ),
+                _n( '%s minute ago', '%s minutes ago', $unit, 'site-reviews' ),
+                _n( 'an hour ago', '%s hours ago', $unit, 'site-reviews' ),
+                _n( 'yesterday', '%s days ago', $unit, 'site-reviews' ),
+                _n( 'a week ago', '%s weeks ago', $unit, 'site-reviews' ),
+                _n( '%s month ago', '%s months ago', $unit, 'site-reviews' ),
+                _n( '%s year ago', '%s years ago', $unit, 'site-reviews' ),
             ];
             $relativeDate = $relativeDates[$i];
-            return Str::contains($relativeDate, '%s')
-                ? sprintf($relativeDate, $unit)
+            return Str::contains( $relativeDate, '%s' )
+                ? sprintf( $relativeDate, $unit )
                 : $relativeDate;
         }
     }

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 = Str::snakeCase($rule);
-        return $this->translator($lowerRule, $parameters);
+        $lowerRule = Str::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 = Str::snakeCase($rule.$type);
-        return $this->translator($lowerRule, $parameters);
+        $type = $this->getAttributeType( $attribute );
+        $lowerRule = Str::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 (Str::contains($rule, ':')) {
-            list($rule, $parameter) = explode(':', $rule, 2);
-            $parameters = $this->parseParameters($rule, $parameter);
+        if( Str::contains( $rule, ':' ) ) {
+            list($rule, $parameter) = explode( ':', $rule, 2 );
+            $parameters = $this->parseParameters( $rule, $parameter );
         }
-        $rule = Str::camelCase($rule);
+        $rule = Str::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

@@ -16,23 +16,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 (!Str::contains($message, '%s')) {
+        if( !Str::contains( $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 );
     }
 
     /**
@@ -42,10 +42,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 );
     }
 
     /**
@@ -54,10 +54,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];
     }
 
@@ -66,9 +66,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 );
     }
 
     /**
@@ -77,10 +77,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];
     }
 
     /**
@@ -89,10 +89,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];
     }
 
     /**
@@ -100,9 +100,9 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateNumber($value)
+    public function validateNumber( $value )
     {
-        return is_numeric($value);
+        return is_numeric( $value );
     }
 
     /**
@@ -110,11 +110,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;
     }
@@ -126,10 +126,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/Modules/Translator.php

Spacing +25 added lines, -25 removed lines

@@ -11,31 +11,31 @@ 
      * @param string $domain
      * @return string
      */
-    public function translate($original, $domain, array $args)
+    public function translate( $original, $domain, array $args )
     {
-        $domains = apply_filters('site-reviews/translator/domains', [Application::ID]);
-        if (!in_array($domain, $domains)) {
+        $domains = apply_filters( 'site-reviews/translator/domains', [Application::ID] );
+        if( !in_array( $domain, $domains ) ) {
             return $original;
         }
-        $args = $this->normalizeTranslationArgs($args);
-        $strings = $this->getTranslationStrings($args['single'], $args['plural']);
-        if (empty($strings)) {
+        $args = $this->normalizeTranslationArgs( $args );
+        $strings = $this->getTranslationStrings( $args['single'], $args['plural'] );
+        if( empty($strings) ) {
             return $original;
         }
-        $string = current($strings);
+        $string = current( $strings );
         return 'plural' == $string['type']
-            ? $this->translatePlural($domain, $string, $args)
-            : $this->translateSingle($domain, $string, $args);
+            ? $this->translatePlural( $domain, $string, $args )
+            : $this->translateSingle( $domain, $string, $args );
     }
 
     /**
      * Used when search/replacing a default text-domain translation
      * @return string
      */
-    public function getTranslation(array $args)
+    public function getTranslation( array $args )
     {
-        $args = $this->normalizeTranslationArgs($args);
-        return get_translations_for_domain(Application::ID)->translate_plural($args['single'], $args['plural'], $args['number']);
+        $args = $this->normalizeTranslationArgs( $args );
+        return get_translations_for_domain( Application::ID )->translate_plural( $args['single'], $args['plural'], $args['number'] );
     }
 
     /**
@@ -43,18 +43,18 @@ 
      * @param string $plural
      * @return array
      */
-    protected function getTranslationStrings($single, $plural)
+    protected function getTranslationStrings( $single, $plural )
     {
-        return array_filter(glsr(Translation::class)->translations(), function ($string) use ($single, $plural) {
-            return $string['s1'] == html_entity_decode($single, ENT_COMPAT, 'UTF-8')
-                && $string['p1'] == html_entity_decode($plural, ENT_COMPAT, 'UTF-8');
+        return array_filter( glsr( Translation::class )->translations(), function( $string ) use ($single, $plural) {
+            return $string['s1'] == html_entity_decode( $single, ENT_COMPAT, 'UTF-8' )
+                && $string['p1'] == html_entity_decode( $plural, ENT_COMPAT, 'UTF-8' );
         });
     }
 
     /**
      * @return array
      */
-    protected function normalizeTranslationArgs(array $args)
+    protected function normalizeTranslationArgs( array $args )
     {
         $defaults = [
             'context' => '',
@@ -62,22 +62,22 @@ 
             'plural' => '',
             'single' => '',
         ];
-        return shortcode_atts($defaults, $args);
+        return shortcode_atts( $defaults, $args );
     }
 
     /**
      * @param string $domain
      * @return string
      */
-    protected function translatePlural($domain, array $string, array $args)
+    protected function translatePlural( $domain, array $string, array $args )
     {
-        if (!empty($string['s2'])) {
+        if( !empty($string['s2']) ) {
             $args['single'] = $string['s2'];
         }
-        if (!empty($string['p2'])) {
+        if( !empty($string['p2']) ) {
             $args['plural'] = $string['p2'];
         }
-        return get_translations_for_domain($domain)->translate_plural(
+        return get_translations_for_domain( $domain )->translate_plural(
             $args['single'],
             $args['plural'],
             $args['number'],
@@ -89,12 +89,12 @@ 
      * @param string $domain
      * @return string
      */
-    protected function translateSingle($domain, array $string, array $args)
+    protected function translateSingle( $domain, array $string, array $args )
     {
-        if (!empty($string['s2'])) {
+        if( !empty($string['s2']) ) {
             $args['single'] = $string['s2'];
         }
-        return get_translations_for_domain($domain)->translate(
+        return get_translations_for_domain( $domain )->translate(
             $args['single'],
             $args['context']
         );

plugin/Helpers/Str.php

Spacing +49 added lines, -49 removed lines

@@ -10,10 +10,10 @@ 
      * @param string $string
      * @return string
      */
-    public static function camelCase($string)
+    public static function camelCase( $string )
     {
-        $string = ucwords(str_replace(['-', '_'], ' ', trim($string)));
-        return str_replace(' ', '', $string);
+        $string = ucwords( str_replace( ['-', '_'], ' ', trim( $string ) ) );
+        return str_replace( ' ', '', $string );
     }
 
     /**
@@ -21,9 +21,9 @@ 
      * @param string $needle
      * @return bool
      */
-    public static function contains($haystack, $needle)
+    public static function contains( $haystack, $needle )
     {
-        return false !== strpos($haystack, $needle);
+        return false !== strpos( $haystack, $needle );
     }
 
     /**
@@ -32,27 +32,27 @@ 
      * @param string $initialType period|period_space|space
      * @return string
      */
-    public static function convertName($name, $nameType = '', $initialType = '')
+    public static function convertName( $name, $nameType = '', $initialType = '' )
     {
-        $names = preg_split('/\W/', $name, 0, PREG_SPLIT_NO_EMPTY);
-        $firstName = array_shift($names);
-        $lastName = array_pop($names);
+        $names = preg_split( '/\W/', $name, 0, PREG_SPLIT_NO_EMPTY );
+        $firstName = array_shift( $names );
+        $lastName = array_pop( $names );
         $initialTypes = [
             'period' => '.',
             'period_space' => '. ',
             'space' => ' ',
         ];
-        $initialPunctuation = (string) Arr::get($initialTypes, $initialType, ' ');
-        if ('initials' == $nameType) {
-            return static::convertToInitials($name, $initialPunctuation);
+        $initialPunctuation = (string)Arr::get( $initialTypes, $initialType, ' ' );
+        if( 'initials' == $nameType ) {
+            return static::convertToInitials( $name, $initialPunctuation );
         }
         $nameTypes = [
             'first' => $firstName,
-            'first_initial' => substr($firstName, 0, 1).$initialPunctuation.$lastName,
+            'first_initial' => substr( $firstName, 0, 1 ).$initialPunctuation.$lastName,
             'last' => $lastName,
-            'last_initial' => $firstName.' '.substr($lastName, 0, 1).$initialPunctuation,
+            'last_initial' => $firstName.' '.substr( $lastName, 0, 1 ).$initialPunctuation,
         ];
-        return trim((string) Arr::get($nameTypes, $nameType, $name));
+        return trim( (string)Arr::get( $nameTypes, $nameType, $name ) );
     }
 
     /**
@@ -60,9 +60,9 @@ 
      * @param string $prefix
      * @return string
      */
-    public static function convertPathToId($path, $prefix = '')
+    public static function convertPathToId( $path, $prefix = '' )
     {
-        return str_replace(['[', ']'], ['-', ''], static::convertPathToName($path, $prefix));
+        return str_replace( ['[', ']'], ['-', ''], static::convertPathToName( $path, $prefix ) );
     }
 
     /**
@@ -70,12 +70,12 @@ 
      * @param string $prefix
      * @return string
      */
-    public static function convertPathToName($path, $prefix = '')
+    public static function convertPathToName( $path, $prefix = '' )
     {
-        $levels = explode('.', $path);
-        return array_reduce($levels, function ($result, $value) {
+        $levels = explode( '.', $path );
+        return array_reduce( $levels, function( $result, $value ) {
             return $result .= '['.$value.']';
-        }, $prefix);
+        }, $prefix );
     }
 
     /**
@@ -83,11 +83,11 @@ 
      * @param string $initialPunctuation
      * @return string
      */
-    public static function convertToInitials($name, $initialPunctuation = '')
+    public static function convertToInitials( $name, $initialPunctuation = '' )
     {
-        preg_match_all('/(?<=\s|\b)\pL/u', $name, $matches);
-        return array_reduce($matches[0], function ($carry, $word) use ($initialPunctuation) {
-            return $carry.strtoupper(substr($word, 0, 1)).$initialPunctuation;
+        preg_match_all( '/(?<=\s|\b)\pL/u', $name, $matches );
+        return array_reduce( $matches[0], function( $carry, $word ) use ($initialPunctuation) {
+            return $carry.strtoupper( substr( $word, 0, 1 ) ).$initialPunctuation;
         });
     }
 
@@ -95,9 +95,9 @@ 
      * @param string $string
      * @return string
      */
-    public static function dashCase($string)
+    public static function dashCase( $string )
     {
-        return str_replace('_', '-', static::snakeCase($string));
+        return str_replace( '_', '-', static::snakeCase( $string ) );
     }
 
     /**
@@ -105,11 +105,11 @@ 
      * @param string $haystack
      * @return bool
      */
-    public static function endsWith($needle, $haystack)
+    public static function endsWith( $needle, $haystack )
     {
-        $length = strlen($needle);
+        $length = strlen( $needle );
         return 0 != $length
-            ? substr($haystack, -$length) === $needle
+            ? substr( $haystack, -$length ) === $needle
             : true;
     }
 
@@ -119,12 +119,12 @@ 
      * @param string|null $trim
      * @return string
      */
-    public static function prefix($prefix, $string, $trim = null)
+    public static function prefix( $prefix, $string, $trim = null )
     {
-        if (null === $trim) {
+        if( null === $trim ) {
             $trim = $prefix;
         }
-        return $prefix.trim(static::removePrefix($trim, $string));
+        return $prefix.trim( static::removePrefix( $trim, $string ) );
     }
 
     /**
@@ -132,10 +132,10 @@ 
      * @param string $string
      * @return string
      */
-    public static function removePrefix($prefix, $string)
+    public static function removePrefix( $prefix, $string )
     {
-        return static::startsWith($prefix, $string)
-            ? substr($string, strlen($prefix))
+        return static::startsWith( $prefix, $string )
+            ? substr( $string, strlen( $prefix ) )
             : $string;
     }
 
@@ -143,16 +143,16 @@ 
      * @param string $string
      * @return string
      */
-    public static function snakeCase($string)
+    public static function snakeCase( $string )
     {
-        if (!ctype_lower($string)) {
-            $string = preg_replace('/\s+/u', '', $string);
-            $string = preg_replace('/(.)(?=[A-Z])/u', '$1_', $string);
-            $string = function_exists('mb_strtolower')
-                ? mb_strtolower($string, 'UTF-8')
-                : strtolower($string);
+        if( !ctype_lower( $string ) ) {
+            $string = preg_replace( '/\s+/u', '', $string );
+            $string = preg_replace( '/(.)(?=[A-Z])/u', '$1_', $string );
+            $string = function_exists( 'mb_strtolower' )
+                ? mb_strtolower( $string, 'UTF-8' )
+                : strtolower( $string );
         }
-        return str_replace('-', '_', $string);
+        return str_replace( '-', '_', $string );
     }
 
     /**
@@ -160,9 +160,9 @@ 
      * @param string $haystack
      * @return bool
      */
-    public static function startsWith($needle, $haystack)
+    public static function startsWith( $needle, $haystack )
     {
-        return substr($haystack, 0, strlen($needle)) === $needle;
+        return substr( $haystack, 0, strlen( $needle ) ) === $needle;
     }
 
     /**
@@ -170,10 +170,10 @@ 
      * @param int $length
      * @return string
      */
-    public static function truncate($string, $length)
+    public static function truncate( $string, $length )
     {
-        return strlen($string) > $length
-            ? substr($string, 0, $length)
+        return strlen( $string ) > $length
+            ? substr( $string, 0, $length )
             : $string;
     }
 }

plugin/Controllers/TranslationController.php

Spacing +47 added lines, -47 removed lines

@@ -15,7 +15,7 @@ 
      */
     public $translator;
 
-    public function __construct(Translator $translator)
+    public function __construct( Translator $translator )
     {
         $this->translator = $translator;
     }
@@ -25,15 +25,15 @@ 
      * @return array
      * @filter bulk_post_updated_messages
      */
-    public function filterBulkUpdateMessages($messages, array $counts)
+    public function filterBulkUpdateMessages( $messages, array $counts )
     {
-        $messages = Arr::consolidateArray($messages);
+        $messages = Arr::consolidateArray( $messages );
         $messages[Application::POST_TYPE] = [
-            'updated' => _n('%s review updated.', '%s reviews updated.', $counts['updated'], 'site-reviews'),
-            'locked' => _n('%s review not updated, somebody is editing it.', '%s reviews not updated, somebody is editing them.', $counts['locked'], 'site-reviews'),
-            'deleted' => _n('%s review permanently deleted.', '%s reviews permanently deleted.', $counts['deleted'], 'site-reviews'),
-            'trashed' => _n('%s review moved to the Trash.', '%s reviews moved to the Trash.', $counts['trashed'], 'site-reviews'),
-            'untrashed' => _n('%s review restored from the Trash.', '%s reviews restored from the Trash.', $counts['untrashed'], 'site-reviews'),
+            'updated' => _n( '%s review updated.', '%s reviews updated.', $counts['updated'], 'site-reviews' ),
+            'locked' => _n( '%s review not updated, somebody is editing it.', '%s reviews not updated, somebody is editing them.', $counts['locked'], 'site-reviews' ),
+            'deleted' => _n( '%s review permanently deleted.', '%s reviews permanently deleted.', $counts['deleted'], 'site-reviews' ),
+            'trashed' => _n( '%s review moved to the Trash.', '%s reviews moved to the Trash.', $counts['trashed'], 'site-reviews' ),
+            'untrashed' => _n( '%s review restored from the Trash.', '%s reviews restored from the Trash.', $counts['untrashed'], 'site-reviews' ),
         ];
         return $messages;
     }
@@ -45,9 +45,9 @@ 
      * @return string
      * @filter gettext
      */
-    public function filterGettext($translation, $text, $domain)
+    public function filterGettext( $translation, $text, $domain )
     {
-        return apply_filters('site-reviews/gettext/'.$domain, $translation, $text);
+        return apply_filters( 'site-reviews/gettext/'.$domain, $translation, $text );
     }
 
     /**
@@ -56,11 +56,11 @@ 
      * @return string
      * @filter site-reviews/gettext/site-reviews
      */
-    public function filterGettextSiteReviews($translation, $text)
+    public function filterGettextSiteReviews( $translation, $text )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'single' => $text,
-        ]);
+        ] );
     }
 
     /**
@@ -71,9 +71,9 @@ 
      * @return string
      * @filter gettext_with_context
      */
-    public function filterGettextWithContext($translation, $text, $context, $domain)
+    public function filterGettextWithContext( $translation, $text, $context, $domain )
     {
-        return apply_filters('site-reviews/gettext_with_context/'.$domain, $translation, $text, $context);
+        return apply_filters( 'site-reviews/gettext_with_context/'.$domain, $translation, $text, $context );
     }
 
     /**
@@ -83,12 +83,12 @@ 
      * @return string
      * @filter site-reviews/gettext_with_context/site-reviews
      */
-    public function filterGettextWithContextSiteReviews($translation, $text, $context)
+    public function filterGettextWithContextSiteReviews( $translation, $text, $context )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'context' => $context,
             'single' => $text,
-        ]);
+        ] );
     }
 
     /**
@@ -100,9 +100,9 @@ 
      * @return string
      * @filter ngettext
      */
-    public function filterNgettext($translation, $single, $plural, $number, $domain)
+    public function filterNgettext( $translation, $single, $plural, $number, $domain )
     {
-        return apply_filters('site-reviews/ngettext/'.$domain, $translation, $single, $plural, $number);
+        return apply_filters( 'site-reviews/ngettext/'.$domain, $translation, $single, $plural, $number );
     }
 
     /**
@@ -113,13 +113,13 @@ 
      * @return string
      * @filter site-reviews/ngettext/site-reviews
      */
-    public function filterNgettextSiteReviews($translation, $single, $plural, $number)
+    public function filterNgettextSiteReviews( $translation, $single, $plural, $number )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'number' => $number,
             'plural' => $plural,
             'single' => $single,
-        ]);
+        ] );
     }
 
     /**
@@ -132,9 +132,9 @@ 
      * @return string
      * @filter ngettext_with_context
      */
-    public function filterNgettextWithContext($translation, $single, $plural, $number, $context, $domain)
+    public function filterNgettextWithContext( $translation, $single, $plural, $number, $context, $domain )
     {
-        return apply_filters('site-reviews/ngettext_with_context/'.$domain, $translation, $single, $plural, $number, $context);
+        return apply_filters( 'site-reviews/ngettext_with_context/'.$domain, $translation, $single, $plural, $number, $context );
     }
 
     /**
@@ -146,14 +146,14 @@ 
      * @return string
      * @filter site-reviews/ngettext_with_context/site-reviews
      */
-    public function filterNgettextWithContextSiteReviews($translation, $single, $plural, $number, $context)
+    public function filterNgettextWithContextSiteReviews( $translation, $single, $plural, $number, $context )
     {
-        return $this->translator->translate($translation, Application::ID, [
+        return $this->translator->translate( $translation, Application::ID, [
             'context' => $context,
             'number' => $number,
             'plural' => $plural,
             'single' => $single,
-        ]);
+        ] );
     }
 
     /**
@@ -162,11 +162,11 @@ 
      * @return array
      * @filter display_post_states
      */
-    public function filterPostStates($postStates, $post)
+    public function filterPostStates( $postStates, $post )
     {
-        $postStates = Arr::consolidateArray($postStates);
-        if (Application::POST_TYPE == Arr::get($post, 'post_type') && array_key_exists('pending', $postStates)) {
-            $postStates['pending'] = __('Unapproved', 'site-reviews');
+        $postStates = Arr::consolidateArray( $postStates );
+        if( Application::POST_TYPE == Arr::get( $post, 'post_type' ) && array_key_exists( 'pending', $postStates ) ) {
+            $postStates['pending'] = __( 'Unapproved', 'site-reviews' );
         }
         return $postStates;
     }
@@ -178,10 +178,10 @@ 
      * @filter site-reviews/gettext/default
      * @filter site-reviews/gettext_with_context/default
      */
-    public function filterPostStatusLabels($translation, $text)
+    public function filterPostStatusLabels( $translation, $text )
     {
         return $this->canModifyTranslation()
-            ? glsr(Labels::class)->filterPostStatusLabels($translation, $text)
+            ? glsr( Labels::class )->filterPostStatusLabels( $translation, $text )
             : $translation;
     }
 
@@ -193,22 +193,22 @@ 
      * @return string
      * @filter site-reviews/ngettext/default
      */
-    public function filterPostStatusText($translation, $single, $plural, $number)
+    public function filterPostStatusText( $translation, $single, $plural, $number )
     {
-        if ($this->canModifyTranslation()) {
+        if( $this->canModifyTranslation() ) {
             $strings = [
-                'Published' => __('Approved', 'site-reviews'),
-                'Pending' => __('Unapproved', 'site-reviews'),
+                'Published' => __( 'Approved', 'site-reviews' ),
+                'Pending' => __( 'Unapproved', 'site-reviews' ),
             ];
-            foreach ($strings as $search => $replace) {
-                if (!Str::contains($single, $search)) {
+            foreach( $strings as $search => $replace ) {
+                if( !Str::contains( $single, $search ) ) {
                     continue;
                 }
-                return $this->translator->getTranslation([
+                return $this->translator->getTranslation( [
                     'number' => $number,
-                    'plural' => str_replace($search, $replace, $plural),
-                    'single' => str_replace($search, $replace, $single),
-                ]);
+                    'plural' => str_replace( $search, $replace, $plural ),
+                    'single' => str_replace( $search, $replace, $single ),
+                ] );
             }
         }
         return $translation;
@@ -220,8 +220,8 @@ 
      */
     public function translatePostStatusLabels()
     {
-        if ($this->canModifyTranslation()) {
-            glsr(Labels::class)->translatePostStatusLabels();
+        if( $this->canModifyTranslation() ) {
+            glsr( Labels::class )->translatePostStatusLabels();
         }
     }
 
@@ -232,6 +232,6 @@ 
     {
         $screen = glsr_current_screen();
         return Application::POST_TYPE == $screen->post_type 
-            && in_array($screen->base, ['edit', 'post']);
+            && in_array( $screen->base, ['edit', 'post'] );
     }
 }

plugin/Modules/Html/Partials/Pagination.php

Spacing +15 added lines, -15 removed lines

@@ -18,19 +18,19 @@ 
     /**
      * @return void|string
      */
-    public function build(array $args = [])
+    public function build( array $args = [] )
     {
-        $this->args = $this->normalize($args);
-        if ($this->args['total'] < 2) {
+        $this->args = $this->normalize( $args );
+        if( $this->args['total'] < 2 ) {
             return;
         }
-        return glsr(Template::class)->build('templates/pagination', [
+        return glsr( Template::class )->build( 'templates/pagination', [
             'context' => [
-                'links' => apply_filters('site-reviews/paginate_links', $this->buildLinks(), $this->args),
+                'links' => apply_filters( 'site-reviews/paginate_links', $this->buildLinks(), $this->args ),
                 'loader' => '<div class="glsr-loader"></div>',
-                'screen_reader_text' => __('Site Reviews navigation', 'site-reviews'),
+                'screen_reader_text' => __( 'Site Reviews navigation', 'site-reviews' ),
             ],
-        ]);
+        ] );
     }
 
     /**
@@ -38,24 +38,24 @@ 
      */
     protected function buildLinks()
     {
-        $args = glsr(Style::class)->paginationArgs($this->args);
-        if ('array' == $args['type']) {
+        $args = glsr( Style::class )->paginationArgs( $this->args );
+        if( 'array' == $args['type'] ) {
             $args['type'] = 'plain';
         }
-        return paginate_links($args);
+        return paginate_links( $args );
     }
 
     /**
      * @return array
      */
-    protected function normalize(array $args)
+    protected function normalize( array $args )
     {
-        if ($baseUrl = Arr::get($args, 'baseUrl')) {
+        if( $baseUrl = Arr::get( $args, 'baseUrl' ) ) {
             $args['base'] = $baseUrl.'%_%';
         }
-        return wp_parse_args(array_filter($args), [
-            'current' => glsr(QueryBuilder::class)->getPaged(),
+        return wp_parse_args( array_filter( $args ), [
+            'current' => glsr( QueryBuilder::class )->getPaged(),
             'total' => 1,
-        ]);
+        ] );
     }
 }

plugin/Modules/Html/ReviewsHtml.php

Spacing +26 added lines, -26 removed lines

@@ -28,13 +28,13 @@ 
      */
     public $reviews;
 
-    public function __construct(array $renderedReviews, $maxPageCount, array $args)
+    public function __construct( array $renderedReviews, $maxPageCount, array $args )
     {
         $this->args = $args;
         $this->max_num_pages = $maxPageCount;
         $this->reviews = $renderedReviews;
         $this->pagination = $this->buildPagination();
-        parent::__construct($renderedReviews, ArrayObject::STD_PROP_LIST | ArrayObject::ARRAY_AS_PROPS);
+        parent::__construct( $renderedReviews, ArrayObject::STD_PROP_LIST | ArrayObject::ARRAY_AS_PROPS );
     }
 
     /**
@@ -42,7 +42,7 @@ 
      */
     public function __toString()
     {
-        return glsr(Template::class)->build('templates/reviews', [
+        return glsr( Template::class )->build( 'templates/reviews', [
             'args' => $this->args,
             'context' => [
                 'assigned_to' => $this->args['assigned_to'],
@@ -52,7 +52,7 @@ 
                 'pagination' => $this->getPagination(),
                 'reviews' => $this->getReviews(),
             ],
-        ]);
+        ] );
     }
 
     /**
@@ -60,7 +60,7 @@ 
      */
     public function getPagination()
     {
-        return wp_validate_boolean($this->args['pagination'])
+        return wp_validate_boolean( $this->args['pagination'] )
             ? $this->pagination
             : '';
     }
@@ -72,26 +72,26 @@ 
     {
         $html = empty($this->reviews)
             ? $this->getReviewsFallback()
-            : implode(PHP_EOL, $this->reviews);
+            : implode( PHP_EOL, $this->reviews );
         $wrapper = '<div class="glsr-reviews">%s</div>';
-        $wrapper = apply_filters('site-reviews/reviews/reviews-wrapper', $wrapper);
-        return sprintf($wrapper, $html);
+        $wrapper = apply_filters( 'site-reviews/reviews/reviews-wrapper', $wrapper );
+        return sprintf( $wrapper, $html );
     }
 
     /**
      * @param mixed $key
      * @return mixed
      */
-    public function offsetGet($key)
+    public function offsetGet( $key )
     {
-        if ('navigation' == $key) {
+        if( 'navigation' == $key ) {
             glsr()->deprecated[] = 'The $reviewsHtml->navigation property has been been deprecated. Please use the $reviewsHtml->pagination property instead.';
             return $this->pagination;
         }
-        if (property_exists($this, $key)) {
+        if( property_exists( $this, $key ) ) {
             return $this->$key;
         }
-        return array_key_exists($key, $this->reviews)
+        return array_key_exists( $key, $this->reviews )
             ? $this->reviews[$key]
             : null;
     }
@@ -101,15 +101,15 @@ 
      */
     protected function buildPagination()
     {
-        $html = glsr(Partial::class)->build('pagination', [
-            'baseUrl' => Arr::get($this->args, 'pagedUrl'),
-            'current' => Arr::get($this->args, 'paged'),
+        $html = glsr( Partial::class )->build( 'pagination', [
+            'baseUrl' => Arr::get( $this->args, 'pagedUrl' ),
+            'current' => Arr::get( $this->args, 'paged' ),
             'total' => $this->max_num_pages,
-        ]);
-        $html.= sprintf('<glsr-pagination hidden data-atts=\'%s\'></glsr-pagination>', $this->args['json']);
+        ] );
+        $html .= sprintf( '<glsr-pagination hidden data-atts=\'%s\'></glsr-pagination>', $this->args['json'] );
         $wrapper = '<div class="glsr-pagination">%s</div>';
-        $wrapper = apply_filters('site-reviews/reviews/pagination-wrapper', $wrapper);
-        return sprintf($wrapper, $html);
+        $wrapper = apply_filters( 'site-reviews/reviews/pagination-wrapper', $wrapper );
+        return sprintf( $wrapper, $html );
     }
 
     /**
@@ -120,12 +120,12 @@ 
         $defaults = [
             'glsr-default',
         ];
-        if ('ajax' == $this->args['pagination']) {
+        if( 'ajax' == $this->args['pagination'] ) {
             $defaults[] = 'glsr-ajax-pagination';
         }
-        $classes = explode(' ', $this->args['class']);
-        $classes = array_unique(array_merge($defaults, array_filter($classes)));
-        return implode(' ', $classes);
+        $classes = explode( ' ', $this->args['class'] );
+        $classes = array_unique( array_merge( $defaults, array_filter( $classes ) ) );
+        return implode( ' ', $classes );
     }
 
     /**
@@ -133,10 +133,10 @@ 
      */
     protected function getReviewsFallback()
     {
-        if (empty($this->args['fallback']) && glsr(OptionManager::class)->getBool('settings.reviews.fallback')) {
-            $this->args['fallback'] = __('There are no reviews yet. Be the first one to write one.', 'site-reviews');
+        if( empty($this->args['fallback']) && glsr( OptionManager::class )->getBool( 'settings.reviews.fallback' ) ) {
+            $this->args['fallback'] = __( 'There are no reviews yet. Be the first one to write one.', 'site-reviews' );
         }
         $fallback = '<p class="glsr-no-margins">'.$this->args['fallback'].'</p>';
-        return apply_filters('site-reviews/reviews/fallback', $fallback, $this->args);
+        return apply_filters( 'site-reviews/reviews/fallback', $fallback, $this->args );
     }
 }