pryley / site-reviews

plugin/Defaults/DefaultsAbstract.php

Spacing +32 added lines, -32 removed lines

@@ -29,17 +29,17 @@ 
      * @param string $name
      * @return void|array
      */
-    public function __call($name, array $args = [])
+    public function __call( $name, array $args = [] )
     {
-        if (!method_exists($this, $name) || !in_array($name, $this->callable)) {
+        if( !method_exists( $this, $name ) || !in_array( $name, $this->callable ) ) {
             return;
         }
-        $args[0] = $this->mapKeys(Arr::get($args, 0, []));
-        $defaults = call_user_func_array([$this, $name], $args);
-        $hookName = (new ReflectionClass($this))->getShortName();
-        $hookName = str_replace('Defaults', '', $hookName);
-        $hookName = Str::dashCase($hookName);
-        return apply_filters('site-reviews/defaults/'.$hookName, $defaults, $name);
+        $args[0] = $this->mapKeys( Arr::get( $args, 0, [] ) );
+        $defaults = call_user_func_array( [$this, $name], $args );
+        $hookName = (new ReflectionClass( $this ))->getShortName();
+        $hookName = str_replace( 'Defaults', '', $hookName );
+        $hookName = Str::dashCase( $hookName );
+        return apply_filters( 'site-reviews/defaults/'.$hookName, $defaults, $name );
     }
 
     /**
@@ -50,38 +50,38 @@ 
     /**
      * @return array
      */
-    protected function filter(array $values = [])
+    protected function filter( array $values = [] )
     {
-        return $this->normalize($this->merge(array_filter($values)), $values);
+        return $this->normalize( $this->merge( array_filter( $values ) ), $values );
     }
 
     /**
      * @return string
      */
-    protected function filteredJson(array $values = [])
+    protected function filteredJson( array $values = [] )
     {
         $defaults = $this->flattenArray(
-            array_diff_key($this->defaults(), array_flip($this->guarded))
+            array_diff_key( $this->defaults(), array_flip( $this->guarded ) )
         );
         $values = $this->flattenArray(
-            shortcode_atts($defaults, $values)
+            shortcode_atts( $defaults, $values )
         );
-        $filtered = array_filter(array_diff_assoc($values, $defaults), function ($value) {
-            return !$this->isEmpty($value);
+        $filtered = array_filter( array_diff_assoc( $values, $defaults ), function( $value ) {
+            return !$this->isEmpty( $value );
         });
-        return json_encode($filtered, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
+        return json_encode( $filtered, JSON_HEX_APOS | JSON_HEX_QUOT | JSON_HEX_TAG | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE );
     }
 
     /**
      * @return array
      */
-    protected function flattenArray(array $values)
+    protected function flattenArray( array $values )
     {
-        array_walk($values, function (&$value) {
-            if (!is_array($value)) {
+        array_walk( $values, function( &$value ) {
+            if( !is_array( $value ) ) {
                 return;
             }
-            $value = implode(',', $value);
+            $value = implode( ',', $value );
         });
         return $values;
     }
@@ -90,18 +90,18 @@ 
      * @param mixed $var
      * @return bool
      */
-    protected function isEmpty($var)
+    protected function isEmpty( $var )
     {
-        return !is_numeric($var) && !is_bool($var) && empty($var);
+        return !is_numeric( $var ) && !is_bool( $var ) && empty($var);
     }
 
     /**
      * @return array
      */
-    protected function mapKeys(array $args)
+    protected function mapKeys( array $args )
     {
-        foreach ($this->mapped as $old => $new) {
-            if (array_key_exists($old, $args)) {
+        foreach( $this->mapped as $old => $new ) {
+            if( array_key_exists( $old, $args ) ) {
                 $args[$new] = $args[$old];
                 unset($args[$old]);
             }
@@ -112,26 +112,26 @@ 
     /**
      * @return array
      */
-    protected function merge(array $values = [])
+    protected function merge( array $values = [] )
     {
-        return $this->normalize(wp_parse_args($values, $this->defaults()), $values);
+        return $this->normalize( wp_parse_args( $values, $this->defaults() ), $values );
     }
 
     /**
      * @return array
      */
-    protected function normalize(array $values, array $originalValues)
+    protected function normalize( array $values, array $originalValues )
     {
-        $values['json'] = $this->filteredJson($originalValues);
+        $values['json'] = $this->filteredJson( $originalValues );
         return $values;
     }
 
     /**
      * @return array
      */
-    protected function restrict(array $values = [])
+    protected function restrict( array $values = [] )
     {
-        return $this->normalize(shortcode_atts($this->defaults(), $values), $values);
+        return $this->normalize( shortcode_atts( $this->defaults(), $values ), $values );
     }
 
     /**
@@ -139,6 +139,6 @@ 
      */
     protected function unguarded()
     {
-        return array_diff_key($this->defaults(), array_flip($this->guarded));
+        return array_diff_key( $this->defaults(), array_flip( $this->guarded ) );
     }
 }

plugin/Controllers/EditorController/Labels.php

Spacing +33 added lines, -33 removed lines

@@ -12,10 +12,10 @@ 
      * @param string $test
      * @return string
      */
-    public function filterPostStatusLabels($translation, $text)
+    public function filterPostStatusLabels( $translation, $text )
     {
         $replacements = $this->getStatusLabels();
-        return array_key_exists($text, $replacements)
+        return array_key_exists( $text, $replacements )
             ? $replacements[$text]
             : $translation;
     }
@@ -23,18 +23,18 @@ 
     /**
      * @return array
      */
-    public function filterUpdateMessages(array $messages)
+    public function filterUpdateMessages( array $messages )
     {
         $post = get_post();
-        if (!($post instanceof WP_Post)) {
+        if( !($post instanceof WP_Post) ) {
             return;
         }
         $strings = $this->getReviewLabels();
-        $restored = filter_input(INPUT_GET, 'revision');
-        if ($revisionTitle = wp_post_revision_title(intval($restored), false)) {
-            $restored = sprintf($strings['restored'], $revisionTitle);
+        $restored = filter_input( INPUT_GET, 'revision' );
+        if( $revisionTitle = wp_post_revision_title( intval( $restored ), false ) ) {
+            $restored = sprintf( $strings['restored'], $revisionTitle );
         }
-        $scheduled_date = date_i18n('M j, Y @ H:i', strtotime($post->post_date));
+        $scheduled_date = date_i18n( 'M j, Y @ H:i', strtotime( $post->post_date ) );
         $messages[Application::POST_TYPE] = [
              1 => $strings['updated'],
              4 => $strings['updated'],
@@ -42,7 +42,7 @@ 
              6 => $strings['published'],
              7 => $strings['saved'],
              8 => $strings['submitted'],
-             9 => sprintf($strings['scheduled'], '<strong>'.$scheduled_date.'</strong>'),
+             9 => sprintf( $strings['scheduled'], '<strong>'.$scheduled_date.'</strong>' ),
             10 => $strings['draft_updated'],
             50 => $strings['approved'],
             51 => $strings['unapproved'],
@@ -58,13 +58,13 @@ 
     {
         global $wp_scripts;
         $strings = [
-            'savePending' => __('Save as Unapproved', 'site-reviews'),
-            'published' => __('Approved', 'site-reviews'),
+            'savePending' => __( 'Save as Unapproved', 'site-reviews' ),
+            'published' => __( 'Approved', 'site-reviews' ),
         ];
-        if (isset($wp_scripts->registered['post']->extra['data'])) {
+        if( isset($wp_scripts->registered['post']->extra['data']) ) {
             $l10n = &$wp_scripts->registered['post']->extra['data'];
-            foreach ($strings as $search => $replace) {
-                $l10n = preg_replace('/("'.$search.'":")([^"]+)/', '$1'.$replace, $l10n);
+            foreach( $strings as $search => $replace ) {
+                $l10n = preg_replace( '/("'.$search.'":")([^"]+)/', '$1'.$replace, $l10n );
             }
         }
     }
@@ -75,18 +75,18 @@ 
     protected function getReviewLabels()
     {
         return [
-            'approved' => __('Review has been approved and published.', 'site-reviews'),
-            'draft_updated' => __('Review draft updated.', 'site-reviews'),
-            'preview' => __('Preview review', 'site-reviews'),
-            'published' => __('Review approved and published.', 'site-reviews'),
-            'restored' => __('Review restored to revision from %s.', 'site-reviews'),
-            'reverted' => __('Review has been reverted to its original submission state (title, content, and submission date).', 'site-reviews'),
-            'saved' => __('Review saved.', 'site-reviews'),
-            'scheduled' => __('Review scheduled for: %s.', 'site-reviews'),
-            'submitted' => __('Review submitted.', 'site-reviews'),
-            'unapproved' => __('Review has been unapproved and is now pending.', 'site-reviews'),
-            'updated' => __('Review updated.', 'site-reviews'),
-            'view' => __('View review', 'site-reviews'),
+            'approved' => __( 'Review has been approved and published.', 'site-reviews' ),
+            'draft_updated' => __( 'Review draft updated.', 'site-reviews' ),
+            'preview' => __( 'Preview review', 'site-reviews' ),
+            'published' => __( 'Review approved and published.', 'site-reviews' ),
+            'restored' => __( 'Review restored to revision from %s.', 'site-reviews' ),
+            'reverted' => __( 'Review has been reverted to its original submission state (title, content, and submission date).', 'site-reviews' ),
+            'saved' => __( 'Review saved.', 'site-reviews' ),
+            'scheduled' => __( 'Review scheduled for: %s.', 'site-reviews' ),
+            'submitted' => __( 'Review submitted.', 'site-reviews' ),
+            'unapproved' => __( 'Review has been unapproved and is now pending.', 'site-reviews' ),
+            'updated' => __( 'Review updated.', 'site-reviews' ),
+            'view' => __( 'View review', 'site-reviews' ),
         ];
     }
 
@@ -97,14 +97,14 @@ 
     protected function getStatusLabels()
     {
         static $labels;
-        if (empty($labels)) {
+        if( empty($labels) ) {
             $labels = [
-                'Pending' => __('Unapproved', 'site-reviews'),
-                'Pending Review' => __('Unapproved', 'site-reviews'),
-                'Privately Published' => __('Privately Approved', 'site-reviews'),
-                'Publish' => __('Approve', 'site-reviews'),
-                'Published' => __('Approved', 'site-reviews'),
-                'Save as Pending' => __('Save as Unapproved', 'site-reviews'),
+                'Pending' => __( 'Unapproved', 'site-reviews' ),
+                'Pending Review' => __( 'Unapproved', 'site-reviews' ),
+                'Privately Published' => __( 'Privately Approved', 'site-reviews' ),
+                'Publish' => __( 'Approve', 'site-reviews' ),
+                'Published' => __( 'Approved', 'site-reviews' ),
+                'Save as Pending' => __( 'Save as Unapproved', 'site-reviews' ),
             ];
         }
         return $labels;

plugin/Container.php

Spacing +46 added lines, -46 removed lines

@@ -45,7 +45,7 @@ 
      */
     public static function load()
     {
-        if (empty(static::$instance)) {
+        if( empty(static::$instance) ) {
             static::$instance = new static();
         }
         return static::$instance;
@@ -55,16 +55,16 @@ 
      * @param string $property
      * @return mixed
      */
-    public function __get($property)
+    public function __get( $property )
     {
-        if (property_exists($this, $property) && !in_array($property, static::PROTECTED_PROPERTIES)) {
+        if( property_exists( $this, $property ) && !in_array( $property, static::PROTECTED_PROPERTIES ) ) {
             return $this->$property;
         }
-        $constant = 'static::'.strtoupper(Str::snakeCase($property));
-        if (defined($constant)) {
-            return constant($constant);
+        $constant = 'static::'.strtoupper( Str::snakeCase( $property ) );
+        if( defined( $constant ) ) {
+            return constant( $constant );
         }
-        return Arr::get($this->storage, $property, null);
+        return Arr::get( $this->storage, $property, null );
     }
 
     /**
@@ -72,14 +72,14 @@ 
      * @param string $value
      * @return void
      */
-    public function __set($property, $value)
+    public function __set( $property, $value )
     {
-        if (!property_exists($this, $property) || in_array($property, static::PROTECTED_PROPERTIES)) {
+        if( !property_exists( $this, $property ) || in_array( $property, static::PROTECTED_PROPERTIES ) ) {
             $this->storage[$property] = $value;
-        } elseif (!isset($this->$property)) {
+        } elseif( !isset($this->$property) ) {
             $this->$property = $value;
         } else {
-            throw new Exception(sprintf('The "%s" property cannot be changed once set.', $property));
+            throw new Exception( sprintf( 'The "%s" property cannot be changed once set.', $property ) );
         }
     }
 
@@ -89,7 +89,7 @@ 
      * @param mixed $concrete
      * @return mixed
      */
-    public function bind($alias, $concrete)
+    public function bind( $alias, $concrete )
     {
         $this->services[$alias] = $concrete;
     }
@@ -99,21 +99,21 @@ 
      * @param mixed $abstract
      * @return mixed
      */
-    public function make($abstract)
+    public function make( $abstract )
     {
-        if (!isset($this->services[$abstract])) {
-            $abstract = $this->addNamespace($abstract);
+        if( !isset($this->services[$abstract]) ) {
+            $abstract = $this->addNamespace( $abstract );
         }
-        if (isset($this->services[$abstract])) {
+        if( isset($this->services[$abstract]) ) {
             $abstract = $this->services[$abstract];
         }
-        if (is_callable($abstract)) {
-            return call_user_func_array($abstract, [$this]);
+        if( is_callable( $abstract ) ) {
+            return call_user_func_array( $abstract, [$this] );
         }
-        if (is_object($abstract)) {
+        if( is_object( $abstract ) ) {
             return $abstract;
         }
-        return $this->resolve($abstract);
+        return $this->resolve( $abstract );
     }
 
     /**
@@ -127,9 +127,9 @@ 
     /**
      * @return mixed
      */
-    public function sessionGet($key, $fallback = '')
+    public function sessionGet( $key, $fallback = '' )
     {
-        $value = Arr::get($this->session, $key, $fallback);
+        $value = Arr::get( $this->session, $key, $fallback );
         unset($this->session[$key]);
         return $value;
     }
@@ -137,7 +137,7 @@ 
     /**
      * @return void
      */
-    public function sessionSet($key, $value)
+    public function sessionSet( $key, $value )
     {
         $this->session[$key] = $value;
     }
@@ -148,9 +148,9 @@ 
      * @param callable|string|null $binding
      * @return void
      */
-    public function singleton($alias, $binding)
+    public function singleton( $alias, $binding )
     {
-        $this->bind($alias, $this->make($binding));
+        $this->bind( $alias, $this->make( $binding ) );
     }
 
     /**
@@ -158,9 +158,9 @@ 
      * @param string $abstract
      * @return string
      */
-    protected function addNamespace($abstract)
+    protected function addNamespace( $abstract )
     {
-        if (!Str::contains($abstract, __NAMESPACE__) && !class_exists($abstract)) {
+        if( !Str::contains( $abstract, __NAMESPACE__ ) && !class_exists( $abstract ) ) {
             $abstract = __NAMESPACE__.'\\'.$abstract;
         }
         return $abstract;
@@ -172,21 +172,21 @@ 
      * @return mixed
      * @throws Exception
      */
-    protected function resolve($concrete)
+    protected function resolve( $concrete )
     {
-        if ($concrete instanceof Closure) {
-            return $concrete($this);
+        if( $concrete instanceof Closure ) {
+            return $concrete( $this );
         }
-        $reflector = new ReflectionClass($concrete);
-        if (!$reflector->isInstantiable()) {
-            throw new Exception('Target ['.$concrete.'] is not instantiable.');
+        $reflector = new ReflectionClass( $concrete );
+        if( !$reflector->isInstantiable() ) {
+            throw new Exception( 'Target ['.$concrete.'] is not instantiable.' );
         }
         $constructor = $reflector->getConstructor();
-        if (empty($constructor)) {
+        if( empty($constructor) ) {
             return new $concrete();
         }
         return $reflector->newInstanceArgs(
-            $this->resolveDependencies($constructor->getParameters())
+            $this->resolveDependencies( $constructor->getParameters() )
         );
     }
 
@@ -195,12 +195,12 @@ 
      * @return mixed
      * @throws Exception
      */
-    protected function resolveClass(ReflectionParameter $parameter)
+    protected function resolveClass( ReflectionParameter $parameter )
     {
         try {
-            return $this->make($parameter->getClass()->name);
-        } catch (Exception $error) {
-            if ($parameter->isOptional()) {
+            return $this->make( $parameter->getClass()->name );
+        } catch( Exception $error ) {
+            if( $parameter->isOptional() ) {
                 return $parameter->getDefaultValue();
             }
             throw $error;
@@ -211,13 +211,13 @@ 
      * Resolve all of the dependencies from the ReflectionParameters.
      * @return array
      */
-    protected function resolveDependencies(array $dependencies)
+    protected function resolveDependencies( array $dependencies )
     {
         $results = [];
-        foreach ($dependencies as $dependency) {
-            $results[] = !is_null($class = $dependency->getClass())
-                ? $this->resolveClass($dependency)
-                : $this->resolveDependency($dependency);
+        foreach( $dependencies as $dependency ) {
+            $results[] = !is_null( $class = $dependency->getClass() )
+                ? $this->resolveClass( $dependency )
+                : $this->resolveDependency( $dependency );
         }
         return $results;
     }
@@ -226,9 +226,9 @@ 
      * Resolve a single ReflectionParameter dependency.
      * @return array|null
      */
-    protected function resolveDependency(ReflectionParameter $parameter)
+    protected function resolveDependency( ReflectionParameter $parameter )
     {
-        if ($parameter->isArray() && $parameter->isDefaultValueAvailable()) {
+        if( $parameter->isArray() && $parameter->isDefaultValueAvailable() ) {
             return $parameter->getDefaultValue();
         }
         return null;

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/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/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/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/Controllers/PublicController.php

Spacing +18 added lines, -18 removed lines

@@ -28,14 +28,14 @@ 
      * @return string
      * @filter script_loader_tag
      */
-    public function filterEnqueuedScripts($tag, $handle)
+    public function filterEnqueuedScripts( $tag, $handle )
     {
         $scripts = [Application::ID.'/google-recaptcha'];
-        if (in_array($handle, apply_filters('site-reviews/async-scripts', $scripts))) {
-            $tag = str_replace(' src=', ' async src=', $tag);
+        if( in_array( $handle, apply_filters( 'site-reviews/async-scripts', $scripts ) ) ) {
+            $tag = str_replace( ' src=', ' async src=', $tag );
         }
-        if (in_array($handle, apply_filters('site-reviews/defer-scripts', $scripts))) {
-            $tag = str_replace(' src=', ' defer src=', $tag);
+        if( in_array( $handle, apply_filters( 'site-reviews/defer-scripts', $scripts ) ) ) {
+            $tag = str_replace( ' src=', ' defer src=', $tag );
         }
         return $tag;
     }
@@ -44,10 +44,10 @@ 
      * @return array
      * @filter site-reviews/config/forms/submission-form
      */
-    public function filterFieldOrder(array $config)
+    public function filterFieldOrder( array $config )
     {
-        $order = (array) apply_filters('site-reviews/submission-form/order', array_keys($config));
-        return array_intersect_key(array_merge(array_flip($order), $config), $config);
+        $order = (array)apply_filters( 'site-reviews/submission-form/order', array_keys( $config ) );
+        return array_intersect_key( array_merge( array_flip( $order ), $config ), $config );
     }
 
     /**
@@ -55,18 +55,18 @@ 
      * @return string
      * @filter site-reviews/render/view
      */
-    public function filterRenderView($view)
+    public function filterRenderView( $view )
     {
-        return glsr(Style::class)->filterView($view);
+        return glsr( Style::class )->filterView( $view );
     }
 
     /**
      * @return void
      * @action site-reviews/builder
      */
-    public function modifyBuilder(Builder $instance)
+    public function modifyBuilder( Builder $instance )
     {
-        call_user_func_array([glsr(Style::class), 'modifyField'], [$instance]);
+        call_user_func_array( [glsr( Style::class ), 'modifyField'], [$instance] );
     }
 
     /**
@@ -75,18 +75,18 @@ 
      */
     public function renderSchema()
     {
-        glsr(Schema::class)->render();
+        glsr( Schema::class )->render();
     }
 
     /**
      * @return CreateReview
      */
-    public function routerSubmitReview(array $request)
+    public function routerSubmitReview( array $request )
     {
-        $validated = glsr(ValidateReview::class)->validate($request);
-        $command = new CreateReview($validated->request);
-        if (empty($validated->error) && !$validated->recaptchaIsUnset) {
-            $this->execute($command);
+        $validated = glsr( ValidateReview::class )->validate( $request );
+        $command = new CreateReview( $validated->request );
+        if( empty($validated->error) && !$validated->recaptchaIsUnset ) {
+            $this->execute( $command );
         }
         return $command;
     }

plugin/Database/Cache.php

Spacing +20 added lines, -20 removed lines

@@ -11,24 +11,24 @@ 
      */
     public function getCloudflareIps()
     {
-        if (false === ($ipAddresses = get_transient(Application::ID.'_cloudflare_ips'))) {
-            $ipAddresses = array_fill_keys(['v4', 'v6'], []);
-            foreach (array_keys($ipAddresses) as $version) {
+        if( false === ($ipAddresses = get_transient( Application::ID.'_cloudflare_ips' )) ) {
+            $ipAddresses = array_fill_keys( ['v4', 'v6'], [] );
+            foreach( array_keys( $ipAddresses ) as $version ) {
                 $url = 'https://www.cloudflare.com/ips-'.$version;
-                $response = wp_remote_get($url, ['sslverify' => false]);
-                if (is_wp_error($response)) {
-                    glsr_log()->error($response->get_error_message());
+                $response = wp_remote_get( $url, ['sslverify' => false] );
+                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, WEEK_IN_SECONDS);
+            set_transient( Application::ID.'_cloudflare_ips', $ipAddresses, WEEK_IN_SECONDS );
         }
         return $ipAddresses;
     }
@@ -37,16 +37,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;
     }
@@ -56,12 +56,12 @@ 
      */
     public function getRemotePostTest()
     {
-        if (false === ($test = get_transient(Application::ID.'_remote_post_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))
+        if( false === ($test = get_transient( Application::ID.'_remote_post_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, WEEK_IN_SECONDS);
+            set_transient( Application::ID.'_remote_post_test', $test, WEEK_IN_SECONDS );
         }
         return $test;
     }