pryley / site-reviews

plugin/Shortcodes/SiteReviewsFormShortcode.php

Spacing +10 added lines, -10 removed lines

@@ -9,12 +9,12 @@ 
     protected function hideOptions()
     {
         return [
-            'rating' => __('Hide the rating field', 'site-reviews'),
-            'title' => __('Hide the title field', 'site-reviews'),
-            'content' => __('Hide the review field', 'site-reviews'),
-            'name' => __('Hide the name field', 'site-reviews'),
-            'email' => __('Hide the email field', 'site-reviews'),
-            'terms' => __('Hide the terms field', 'site-reviews'),
+            'rating' => __( 'Hide the rating field', 'site-reviews' ),
+            'title' => __( 'Hide the title field', 'site-reviews' ),
+            'content' => __( 'Hide the review field', 'site-reviews' ),
+            'name' => __( 'Hide the name field', 'site-reviews' ),
+            'email' => __( 'Hide the email field', 'site-reviews' ),
+            'terms' => __( 'Hide the terms field', 'site-reviews' ),
         ];
     }
 
@@ -23,11 +23,11 @@ 
      * @param string $type
      * @return array
      */
-    public function normalizeAtts($atts, $type = 'shortcode')
+    public function normalizeAtts( $atts, $type = 'shortcode' )
     {
-        $atts = parent::normalizeAtts($atts, $type);
-        if (empty($atts['id'])) {
-            $atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
+        $atts = parent::normalizeAtts( $atts, $type );
+        if( empty($atts['id']) ) {
+            $atts['id'] = Application::PREFIX.substr( md5( serialize( $atts ) ), 0, 8 );
         }
         return $atts;
     }

plugin/Database/QueryBuilder.php

Spacing +46 added lines, -46 removed lines

@@ -16,19 +16,19 @@ 
      * Build a WP_Query meta_query/tax_query.
      * @return array
      */
-    public function buildQuery(array $keys = [], array $values = [])
+    public function buildQuery( array $keys = [], array $values = [] )
     {
         $queries = [];
-        foreach ($keys as $key) {
-            if (!array_key_exists($key, $values)) {
+        foreach( $keys as $key ) {
+            if( !array_key_exists( $key, $values ) ) {
                 continue;
             }
-            $methodName = Helper::buildMethodName($key, __FUNCTION__);
-            if (!method_exists($this, $methodName)) {
+            $methodName = Helper::buildMethodName( $key, __FUNCTION__ );
+            if( !method_exists( $this, $methodName ) ) {
                 continue;
             }
-            $query = call_user_func([$this, $methodName], $values[$key]);
-            if (is_array($query)) {
+            $query = call_user_func( [$this, $methodName], $values[$key] );
+            if( is_array( $query ) ) {
                 $queries[] = $query;
             }
         }
@@ -38,17 +38,17 @@ 
     /**
      * @return string
      */
-    public function buildSqlLines(array $values, array $conditions)
+    public function buildSqlLines( array $values, array $conditions )
     {
         $string = '';
-        $values = array_filter($values);
-        foreach ($conditions as $key => $value) {
-            if (!isset($values[$key])) {
+        $values = array_filter( $values );
+        foreach( $conditions as $key => $value ) {
+            if( !isset($values[$key]) ) {
                 continue;
             }
-            $values[$key] = implode(',', (array) $values[$key]);
-            $string.= Str::contains($value, '%s')
-                ? sprintf($value, strval($values[$key]))
+            $values[$key] = implode( ',', (array)$values[$key] );
+            $string .= Str::contains( $value, '%s' )
+                ? sprintf( $value, strval( $values[$key] ) )
                 : $value;
         }
         return $string;
@@ -60,16 +60,16 @@ 
      * @param string $sprintfFormat
      * @return string
      */
-    public function buildSqlOr($values, $sprintfFormat)
+    public function buildSqlOr( $values, $sprintfFormat )
     {
-        if (!is_array($values)) {
-            $values = explode(',', $values);
+        if( !is_array( $values ) ) {
+            $values = explode( ',', $values );
         }
-        $values = array_filter(array_map('trim', (array) $values));
-        $values = array_map(function ($value) use ($sprintfFormat) {
-            return sprintf($sprintfFormat, $value);
-        }, $values);
-        return implode(' OR ', $values);
+        $values = array_filter( array_map( 'trim', (array)$values ) );
+        $values = array_map( function( $value ) use ($sprintfFormat) {
+            return sprintf( $sprintfFormat, $value );
+        }, $values );
+        return implode( ' OR ', $values );
     }
 
     /**
@@ -79,23 +79,23 @@ 
      * @return string
      * @filter posts_search
      */
-    public function filterSearchByTitle($search, WP_Query $query)
+    public function filterSearchByTitle( $search, WP_Query $query )
     {
-        if (empty($search) || empty($query->get('search_terms'))) {
+        if( empty($search) || empty($query->get( 'search_terms' )) ) {
             return $search;
         }
         global $wpdb;
-        $n = empty($query->get('exact'))
+        $n = empty($query->get( 'exact' ))
             ? '%'
             : '';
         $search = [];
-        foreach ((array) $query->get('search_terms') as $term) {
-            $search[] = $wpdb->prepare("{$wpdb->posts}.post_title LIKE %s", $n.$wpdb->esc_like($term).$n);
+        foreach( (array)$query->get( 'search_terms' ) as $term ) {
+            $search[] = $wpdb->prepare( "{$wpdb->posts}.post_title LIKE %s", $n.$wpdb->esc_like( $term ).$n );
         }
-        if (!is_user_logged_in()) {
+        if( !is_user_logged_in() ) {
             $search[] = "{$wpdb->posts}.post_password = ''";
         }
-        return ' AND '.implode(' AND ', $search);
+        return ' AND '.implode( ' AND ', $search );
     }
 
     /**
@@ -103,10 +103,10 @@ 
      * @param bool $isEnabled
      * @return int
      */
-    public function getPaged($isEnabled = true)
+    public function getPaged( $isEnabled = true )
     {
         return $isEnabled
-            ? max(1, intval(filter_input(INPUT_GET, glsr()->constant('PAGED_QUERY_VAR'))))
+            ? max( 1, intval( filter_input( INPUT_GET, glsr()->constant( 'PAGED_QUERY_VAR' ) ) ) )
             : 1;
     }
 
@@ -114,14 +114,14 @@ 
      * @param string $value
      * @return void|array
      */
-    protected function buildQueryAssignedTo($value)
+    protected function buildQueryAssignedTo( $value )
     {
-        if (!empty($value)) {
-            $postIds = Arr::convertStringToArray($value, 'is_numeric');
+        if( !empty($value) ) {
+            $postIds = Arr::convertStringToArray( $value, 'is_numeric' );
             return [
                 'compare' => 'IN',
                 'key' => '_assigned_to',
-                'value' => glsr(Multilingual::class)->getPostIds($postIds),
+                'value' => glsr( Multilingual::class )->getPostIds( $postIds ),
             ];
         }
     }
@@ -130,9 +130,9 @@ 
      * @param array $value
      * @return void|array
      */
-    protected function buildQueryCategory($value)
+    protected function buildQueryCategory( $value )
     {
-        if (!empty($value)) {
+        if( !empty($value) ) {
             return [
                 'field' => 'term_id',
                 'taxonomy' => Application::TAXONOMY,
@@ -145,9 +145,9 @@ 
      * @param string $value
      * @return void|array
      */
-    protected function buildQueryEmail($value)
+    protected function buildQueryEmail( $value )
     {
-        if (!empty($value)) {
+        if( !empty($value) ) {
             return [
                 'key' => '_email',
                 'value' => $value,
@@ -159,9 +159,9 @@ 
      * @param string $value
      * @return void|array
      */
-    protected function buildQueryIpAddress($value)
+    protected function buildQueryIpAddress( $value )
     {
-        if (!empty($value)) {
+        if( !empty($value) ) {
             return [
                 'key' => '_ip_address',
                 'value' => $value,
@@ -173,10 +173,10 @@ 
      * @param string $value
      * @return void|array
      */
-    protected function buildQueryRating($value)
+    protected function buildQueryRating( $value )
     {
-        if (is_numeric($value)
-            && in_array(intval($value), range(1, glsr()->constant('MAX_RATING', Rating::class)))) {
+        if( is_numeric( $value )
+            && in_array( intval( $value ), range( 1, glsr()->constant( 'MAX_RATING', Rating::class ) ) ) ) {
             return [
                 'compare' => '>=',
                 'key' => '_rating',
@@ -189,9 +189,9 @@ 
      * @param string $value
      * @return void|array
      */
-    protected function buildQueryType($value)
+    protected function buildQueryType( $value )
     {
-        if (!in_array($value, ['', 'all'])) {
+        if( !in_array( $value, ['', 'all'] ) ) {
             return [
                 'key' => '_review_type',
                 'value' => $value,

plugin/Helpers/Arr.php

Spacing +56 added lines, -56 removed lines

@@ -7,10 +7,10 @@ 
     /**
      * @return bool
      */
-    public static function compareArrays(array $arr1, array $arr2)
+    public static function compareArrays( array $arr1, array $arr2 )
     {
-        sort($arr1);
-        sort($arr2);
+        sort( $arr1 );
+        sort( $arr2 );
         return $arr1 == $arr2;
     }
 
@@ -18,21 +18,21 @@ 
      * @param mixed $array
      * @return array
      */
-    public static function consolidateArray($array)
+    public static function consolidateArray( $array )
     {
-        return is_array($array) || is_object($array)
-            ? (array) $array
+        return is_array( $array ) || is_object( $array )
+            ? (array)$array
             : [];
     }
 
     /**
      * @return array
      */
-    public static function convertDotNotationArray(array $array)
+    public static function convertDotNotationArray( array $array )
     {
         $results = [];
-        foreach ($array as $path => $value) {
-            $results = static::set($results, $path, $value);
+        foreach( $array as $path => $value ) {
+            $results = static::set( $results, $path, $value );
         }
         return $results;
     }
@@ -42,12 +42,12 @@ 
      * @param mixed $callback
      * @return array
      */
-    public static function convertStringToArray($string, $callback = null)
+    public static function convertStringToArray( $string, $callback = null )
     {
-        $array = array_map('trim', explode(',', $string));
+        $array = array_map( 'trim', explode( ',', $string ) );
         return $callback
-            ? array_filter($array, $callback)
-            : array_filter($array);
+            ? array_filter( $array, $callback )
+            : array_filter( $array );
     }
 
     /**
@@ -55,17 +55,17 @@ 
      * @param string $prefix
      * @return array
      */
-    public static function flattenArray(array $array, $flattenValue = false, $prefix = '')
+    public static function flattenArray( array $array, $flattenValue = false, $prefix = '' )
     {
         $result = [];
-        foreach ($array as $key => $value) {
-            $newKey = ltrim($prefix.'.'.$key, '.');
-            if (static::isIndexedFlatArray($value)) {
-                if ($flattenValue) {
-                    $value = '['.implode(', ', $value).']';
+        foreach( $array as $key => $value ) {
+            $newKey = ltrim( $prefix.'.'.$key, '.' );
+            if( static::isIndexedFlatArray( $value ) ) {
+                if( $flattenValue ) {
+                    $value = '['.implode( ', ', $value ).']';
                 }
-            } elseif (is_array($value)) {
-                $result = array_merge($result, static::flattenArray($value, $flattenValue, $newKey));
+            } elseif( is_array( $value ) ) {
+                $result = array_merge( $result, static::flattenArray( $value, $flattenValue, $newKey ) );
                 continue;
             }
             $result[$newKey] = $value;
@@ -80,12 +80,12 @@ 
      * @param mixed $fallback
      * @return mixed
      */
-    public static function get($data, $path = '', $fallback = '')
+    public static function get( $data, $path = '', $fallback = '' )
     {
-        $data = static::consolidateArray($data);
-        $keys = explode('.', $path);
-        foreach ($keys as $key) {
-            if (!isset($data[$key])) {
+        $data = static::consolidateArray( $data );
+        $keys = explode( '.', $path );
+        foreach( $keys as $key ) {
+            if( !isset($data[$key]) ) {
                 return $fallback;
             }
             $data = $data[$key];
@@ -98,47 +98,47 @@ 
      * @param string $position
      * @return array
      */
-    public static function insertInArray(array $array, array $insert, $key, $position = 'before')
+    public static function insertInArray( array $array, array $insert, $key, $position = 'before' )
     {
-        $keyPosition = intval(array_search($key, array_keys($array)));
-        if ('after' == $position) {
+        $keyPosition = intval( array_search( $key, array_keys( $array ) ) );
+        if( 'after' == $position ) {
             ++$keyPosition;
         }
-        if (false !== $keyPosition) {
-            $result = array_slice($array, 0, $keyPosition);
-            $result = array_merge($result, $insert);
-            return array_merge($result, array_slice($array, $keyPosition));
+        if( false !== $keyPosition ) {
+            $result = array_slice( $array, 0, $keyPosition );
+            $result = array_merge( $result, $insert );
+            return array_merge( $result, array_slice( $array, $keyPosition ) );
         }
-        return array_merge($array, $insert);
+        return array_merge( $array, $insert );
     }
 
     /**
      * @param mixed $array
      * @return bool
      */
-    public static function isIndexedFlatArray($array)
+    public static function isIndexedFlatArray( $array )
     {
-        if (!is_array($array) || array_filter($array, 'is_array')) {
+        if( !is_array( $array ) || array_filter( $array, 'is_array' ) ) {
             return false;
         }
-        return wp_is_numeric_array($array);
+        return wp_is_numeric_array( $array );
     }
 
     /**
      * @param bool $prefixed
      * @return array
      */
-    public static function prefixArrayKeys(array $values, $prefixed = true)
+    public static function prefixArrayKeys( array $values, $prefixed = true )
     {
         $trim = '_';
         $prefix = $prefixed
             ? $trim
             : '';
         $prefixed = [];
-        foreach ($values as $key => $value) {
-            $key = trim($key);
-            if (0 === strpos($key, $trim)) {
-                $key = substr($key, strlen($trim));
+        foreach( $values as $key => $value ) {
+            $key = trim( $key );
+            if( 0 === strpos( $key, $trim ) ) {
+                $key = substr( $key, strlen( $trim ) );
             }
             $prefixed[$prefix.$key] = $value;
         }
@@ -148,15 +148,15 @@ 
     /**
      * @return array
      */
-    public static function removeEmptyArrayValues(array $array)
+    public static function removeEmptyArrayValues( array $array )
     {
         $result = [];
-        foreach ($array as $key => $value) {
-            if (!$value) {
+        foreach( $array as $key => $value ) {
+            if( !$value ) {
                 continue;
             }
-            $result[$key] = is_array($value)
-                ? static::removeEmptyArrayValues($value)
+            $result[$key] = is_array( $value )
+                ? static::removeEmptyArrayValues( $value )
                 : $value;
         }
         return $result;
@@ -169,14 +169,14 @@ 
      * @param mixed $value
      * @return array
      */
-    public static function set(array $data, $path, $value)
+    public static function set( array $data, $path, $value )
     {
-        $token = strtok($path, '.');
+        $token = strtok( $path, '.' );
         $ref = &$data;
-        while (false !== $token) {
-            $ref = static::consolidateArray($ref);
+        while( false !== $token ) {
+            $ref = static::consolidateArray( $ref );
             $ref = &$ref[$token];
-            $token = strtok('.');
+            $token = strtok( '.' );
         }
         $ref = $value;
         return $data;
@@ -185,16 +185,16 @@ 
     /**
      * @return array
      */
-    public static function unique(array $values)
+    public static function unique( array $values )
     {
-        return array_filter(array_unique($values));
+        return array_filter( array_unique( $values ) );
     }
 
     /**
      * @return array
      */
-    public static function unprefixArrayKeys(array $values)
+    public static function unprefixArrayKeys( array $values )
     {
-        return static::prefixArrayKeys($values, false);
+        return static::prefixArrayKeys( $values, false );
     }
 }

plugin/Modules/Multilingual.php

Spacing +14 added lines, -14 removed lines

@@ -21,29 +21,29 @@ 
      * @param array $args
      * @return 
      */
-    public function __call($method, $args = [])
+    public function __call( $method, $args = [] )
     {
-        if ($this->isIntegrated() && method_exists($this->integration, $method)) {
-            return call_user_func_array([$this->integration, $method], $args);
+        if( $this->isIntegrated() && method_exists( $this->integration, $method ) ) {
+            return call_user_func_array( [$this->integration, $method], $args );
         }
-        return Arr::get($args, 0, false);
+        return Arr::get( $args, 0, false );
     }
 
     /**
      * @param string $integration
      * @return false|\GeminiLabs\SiteReviews\Modules\Multilingual\Polylang|\GeminiLabs\SiteReviews\Modules\Multilingual\Wpml
      */
-    public function getIntegration($integration = '')
+    public function getIntegration( $integration = '' )
     {
-        if (empty($integration)) {
-            $integration = glsr(OptionManager::class)->get('settings.general.multilingual');
+        if( empty($integration) ) {
+            $integration = glsr( OptionManager::class )->get( 'settings.general.multilingual' );
         }
-        if (!empty($integration)) {
-            $integrationClass = 'GeminiLabs\SiteReviews\Modules\Multilingual\\'.ucfirst($integration);
-            if (class_exists($integrationClass)) {
-                return glsr($integrationClass);
+        if( !empty($integration) ) {
+            $integrationClass = 'GeminiLabs\SiteReviews\Modules\Multilingual\\'.ucfirst( $integration );
+            if( class_exists( $integrationClass ) ) {
+                return glsr( $integrationClass );
             }
-            glsr_log()->error($integrationClass.' does not exist');
+            glsr_log()->error( $integrationClass.' does not exist' );
         }
         return false;
     }
@@ -53,10 +53,10 @@ 
      */
     public function isIntegrated()
     {
-        if (!empty($this->integration)) {
+        if( !empty($this->integration) ) {
             return true;
         }
-        if ($integration = $this->getIntegration()) {
+        if( $integration = $this->getIntegration() ) {
             $this->integration = $integration;
             return true;
         }

views/pages/settings/translations.php

Spacing +9 added lines, -9 removed lines

@@ -1,22 +1,22 @@ 
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
-<p><?= sprintf(__('Here you can customise any text of the plugin, including the submission form labels and placeholders. However, if you have a multilingual website you should use the %s plugin instead.', 'site-reviews'), '<a href="https://wordpress.org/plugins/loco-translate/">Loco Translate</a>'); ?></p>
+<p><?= sprintf( __( 'Here you can customise any text of the plugin, including the submission form labels and placeholders. However, if you have a multilingual website you should use the %s plugin instead.', 'site-reviews' ), '<a href="https://wordpress.org/plugins/loco-translate/">Loco Translate</a>' ); ?></p>
 
 <div class="glsr-strings-form">
     <div class="glsr-search-box" id="glsr-search-translations">
-        <span class="screen-reader-text"><?= __('Search for translatable text', 'site-reviews'); ?></span>
+        <span class="screen-reader-text"><?= __( 'Search for translatable text', 'site-reviews' ); ?></span>
         <div class="glsr-spinner">
             <span class="spinner"></span>
         </div>
-        <input type="search" class="glsr-search-input" autocomplete="off" placeholder="<?= __('Search here for text to translate...', 'site-reviews'); ?>">
-        <?php wp_nonce_field('search-translations', '_search_nonce', false); ?>
+        <input type="search" class="glsr-search-input" autocomplete="off" placeholder="<?= __( 'Search here for text to translate...', 'site-reviews' ); ?>">
+        <?php wp_nonce_field( 'search-translations', '_search_nonce', false ); ?>
         <div class="glsr-search-results" data-prefix="{{ database_key }}"></div>
     </div>
     <table class="glsr-strings-table wp-list-table widefat striped {{ class }}">
         <thead>
             <tr>
-                <th scope="col" class="manage-column column-primary"><?= __('Original Text', 'site-reviews'); ?></th>
-                <th scope="col" class="manage-column"><?= __('Custom Translation', 'site-reviews'); ?></th>
+                <th scope="col" class="manage-column column-primary"><?= __( 'Original Text', 'site-reviews' ); ?></th>
+                <th scope="col" class="manage-column"><?= __( 'Custom Translation', 'site-reviews' ); ?></th>
             </tr>
         </thead>
         <tbody>{{ translations }}</tbody>
@@ -25,8 +25,8 @@ 
 </div>
 
 <script type="text/html" id="tmpl-glsr-string-plural">
-<?php include glsr()->path('views/partials/translations/plural.php'); ?>
+<?php include glsr()->path( 'views/partials/translations/plural.php' ); ?>
 </script>
 <script type="text/html" id="tmpl-glsr-string-single">
-<?php include glsr()->path('views/partials/translations/single.php'); ?>
+<?php include glsr()->path( 'views/partials/translations/single.php' ); ?>
 </script>

plugin/Modules/ReviewLimits.php

Spacing +29 added lines, -29 removed lines

@@ -14,26 +14,26 @@ 
      * @return array
      * @filter site-reviews/get/reviews/query
      */
-    public function filterReviewsQuery(array $parameters, array $args)
+    public function filterReviewsQuery( array $parameters, array $args )
     {
-        if ($authorId = get_current_user_id()) {
+        if( $authorId = get_current_user_id() ) {
             $parameters['author'] = $authorId;
         }
         $parameters['post_status'] = ['pending', 'publish'];
-        return apply_filters('site-reviews/review-limits/query', $parameters, $args);
+        return apply_filters( 'site-reviews/review-limits/query', $parameters, $args );
     }
 
     /**
      * @return bool
      */
-    public function hasReachedLimit(array $request = [])
+    public function hasReachedLimit( array $request = [] )
     {
         $this->request = $request;
         $method = Helper::buildMethodName(
-            glsr(OptionManager::class)->get('settings.submissions.limit'), 'validateBy'
+            glsr( OptionManager::class )->get( 'settings.submissions.limit' ), 'validateBy'
         );
-        return method_exists($this, $method)
-            ? !call_user_func([$this, $method])
+        return method_exists( $this, $method )
+            ? !call_user_func( [$this, $method] )
             : false;
     }
 
@@ -42,41 +42,41 @@ 
      * @param string $whitelist
      * @return bool
      */
-    public function isWhitelisted($value, $whitelist)
+    public function isWhitelisted( $value, $whitelist )
     {
-        if (empty($whitelist)) {
+        if( empty($whitelist) ) {
             return false;
         }
-        return in_array($value, array_filter(explode("\n", $whitelist), 'trim'));
+        return in_array( $value, array_filter( explode( "\n", $whitelist ), 'trim' ) );
     }
 
     /**
      * @param string $whitelistName
      * @return string
      */
-    protected function getWhitelist($whitelistName)
+    protected function getWhitelist( $whitelistName )
     {
-        return glsr(OptionManager::class)->get('settings.submissions.limit_whitelist.'.$whitelistName);
+        return glsr( OptionManager::class )->get( 'settings.submissions.limit_whitelist.'.$whitelistName );
     }
 
     /**
      * @return bool
      */
-    protected function validate($key, $value, $addMetaQuery = true)
+    protected function validate( $key, $value, $addMetaQuery = true )
     {
-        if ($this->isWhitelisted($value, $this->getWhitelist($key))) {
+        if( $this->isWhitelisted( $value, $this->getWhitelist( $key ) ) ) {
             return true;
         }
-        add_filter('site-reviews/get/reviews/query', [$this, 'filterReviewsQuery'], 5, 2);
-        $args = ['assigned_to' => Arr::get($this->request, 'assign_to')];
-        if ($addMetaQuery) {
+        add_filter( 'site-reviews/get/reviews/query', [$this, 'filterReviewsQuery'], 5, 2 );
+        $args = ['assigned_to' => Arr::get( $this->request, 'assign_to' )];
+        if( $addMetaQuery ) {
             $args[$key] = $value;
         }
-        $reviews = glsr_get_reviews($args);
-        remove_filter('site-reviews/get/reviews/query', [$this, 'filterReviewsQuery'], 5);
-        $result = 0 === count($reviews);
-        $result = apply_filters('site-reviews/review-limits/validate', $result, $reviews, $this->request, $key);
-        return wp_validate_boolean($result);
+        $reviews = glsr_get_reviews( $args );
+        remove_filter( 'site-reviews/get/reviews/query', [$this, 'filterReviewsQuery'], 5 );
+        $result = 0 === count( $reviews );
+        $result = apply_filters( 'site-reviews/review-limits/validate', $result, $reviews, $this->request, $key );
+        return wp_validate_boolean( $result );
     }
 
     /**
@@ -84,8 +84,8 @@ 
      */
     protected function validateByEmail()
     {
-        glsr_log()->debug('Email is: '.Arr::get($this->request, 'email'));
-        return $this->validate('email', Arr::get($this->request, 'email'));
+        glsr_log()->debug( 'Email is: '.Arr::get( $this->request, 'email' ) );
+        return $this->validate( 'email', Arr::get( $this->request, 'email' ) );
     }
 
     /**
@@ -93,8 +93,8 @@ 
      */
     protected function validateByIpAddress()
     {
-        glsr_log()->debug('IP Address is: '.Arr::get($this->request, 'ip_address'));
-        return $this->validate('ip_address', Arr::get($this->request, 'ip_address'));
+        glsr_log()->debug( 'IP Address is: '.Arr::get( $this->request, 'ip_address' ) );
+        return $this->validate( 'ip_address', Arr::get( $this->request, 'ip_address' ) );
     }
 
     /**
@@ -103,10 +103,10 @@ 
     protected function validateByUsername()
     {
         $user = wp_get_current_user();
-        if (!$user->exists()) {
+        if( !$user->exists() ) {
             return true;
         }
-        glsr_log()->debug('Username is: '.$user->user_login);
-        return $this->validate('username', $user->user_login, false);
+        glsr_log()->debug( 'Username is: '.$user->user_login );
+        return $this->validate( 'username', $user->user_login, false );
     }
 }

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) && in_array(trim($value), ['', '[]']))
-            || (is_array($value) && empty($value))
+        return is_null( $value )
+            || (is_string( $value ) && in_array( trim( $value ), ['', '[]'] ))
+            || (is_array( $value ) && empty($value))
             ? 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/Contracts/MultilingualContract.php

Spacing +2 added lines, -2 removed lines

@@ -8,12 +8,12 @@
      * @param int|string $postId
      * @return int
      */
-    public function getPostId($postId);
+    public function getPostId( $postId );
 
     /**
      * @return array
      */
-    public function getPostIds(array $postIds);
+    public function getPostIds( array $postIds );
 
     /**
      * @return bool

views/pages/documentation/index.php

Spacing +7 added lines, -7 removed lines

@@ -1,20 +1,20 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <div class="wrap">
     <h1 class="wp-heading-inline">
-        <?= esc_html(get_admin_page_title()); ?>
-        <a href="<?= admin_url('edit.php?post_type='.glsr()->post_type.'&page=welcome'); ?>" class="page-title-action"><?= __('About', 'site-reviews'); ?></a>
+        <?= esc_html( get_admin_page_title() ); ?>
+        <a href="<?= admin_url( 'edit.php?post_type='.glsr()->post_type.'&page=welcome' ); ?>" class="page-title-action"><?= __( 'About', 'site-reviews' ); ?></a>
     </h1>
     <?= $notices; ?>
-    <p><?= __('Click an active tab to expand/collapse all sections.', 'site-reviews'); ?></p>
+    <p><?= __( 'Click an active tab to expand/collapse all sections.', 'site-reviews' ); ?></p>
     <h2 class="glsr-nav-tab-wrapper nav-tab-wrapper">
-        <?php foreach ($tabs as $id => $title) : ?>
+        <?php foreach( $tabs as $id => $title ) : ?>
         <a class="glsr-nav-tab nav-tab" href="#<?= $id; ?>"><?= $title; ?></a>
         <?php endforeach; ?>
     </h2>
-    <?php foreach ($tabs as $id => $title) : ?>
+    <?php foreach( $tabs as $id => $title ) : ?>
     <div class="glsr-nav-view ui-tabs-hide" id="<?= $id; ?>">
-        <?php glsr()->render('pages/documentation/'.$id, $data); ?>
+        <?php glsr()->render( 'pages/documentation/'.$id, $data ); ?>
     </div>
     <?php endforeach; ?>
     <input type="hidden" name="_active_tab">