pryley / site-reviews

plugin/Modules/Html/Partials/SiteReviews.php

Spacing +116 added lines, -116 removed lines

@@ -45,13 +45,13 @@ 
      * @param Reviews|null $reviews
      * @return ReviewsHtml
      */
-    public function build(array $args = [], $reviews = null)
+    public function build( array $args = [], $reviews = null )
     {
-        $this->args = glsr(SiteReviewsDefaults::class)->merge($args);
-        $this->options = Arr::flattenArray(glsr(OptionManager::class)->all());
+        $this->args = glsr( SiteReviewsDefaults::class )->merge( $args );
+        $this->options = Arr::flattenArray( glsr( OptionManager::class )->all() );
         $this->reviews = $reviews instanceof Reviews
             ? $reviews
-            : glsr(ReviewManager::class)->get($this->args);
+            : glsr( ReviewManager::class )->get( $this->args );
         $this->generateSchema();
         return $this->buildReviews();
     }
@@ -59,26 +59,26 @@ 
     /**
      * @return ReviewHtml
      */
-    public function buildReview(Review $review)
+    public function buildReview( Review $review )
     {
-        $review = apply_filters('site-reviews/review/build/before', $review);
+        $review = apply_filters( 'site-reviews/review/build/before', $review );
         $this->current = $review;
         $renderedFields = [];
-        foreach ($review as $key => $value) {
-            $method = Helper::buildMethodName($key, 'buildOption');
-            $field = method_exists($this, $method)
-                ? $this->$method($key, $value)
+        foreach( $review as $key => $value ) {
+            $method = Helper::buildMethodName( $key, 'buildOption' );
+            $field = method_exists( $this, $method )
+                ? $this->$method( $key, $value )
                 : false;
-            $field = apply_filters('site-reviews/review/build/'.$key, $field, $value, $review, $this);
-            if (false === $field) {
+            $field = apply_filters( 'site-reviews/review/build/'.$key, $field, $value, $review, $this );
+            if( false === $field ) {
                 continue;
             }
             $renderedFields[$key] = $field;
         }
-        $this->wrap($renderedFields, $review);
-        $renderedFields = apply_filters('site-reviews/review/build/after', $renderedFields, $review, $this);
+        $this->wrap( $renderedFields, $review );
+        $renderedFields = apply_filters( 'site-reviews/review/build/after', $renderedFields, $review, $this );
         $this->current = null;
-        return new ReviewHtml($review, (array) $renderedFields);
+        return new ReviewHtml( $review, (array)$renderedFields );
     }
 
     /**
@@ -87,10 +87,10 @@ 
     public function buildReviews()
     {
         $renderedReviews = [];
-        foreach ($this->reviews as $index => $review) {
-            $renderedReviews[] = $this->buildReview($review);
+        foreach( $this->reviews as $index => $review ) {
+            $renderedReviews[] = $this->buildReview( $review );
         }
-        return new ReviewsHtml($renderedReviews, $this->reviews->max_num_pages, $this->args);
+        return new ReviewsHtml( $renderedReviews, $this->reviews->max_num_pages, $this->args );
     }
 
     /**
@@ -98,11 +98,11 @@ 
      */
     public function generateSchema()
     {
-        if (!wp_validate_boolean($this->args['schema'])) {
+        if( !wp_validate_boolean( $this->args['schema'] ) ) {
             return;
         }
-        glsr(Schema::class)->store(
-            glsr(Schema::class)->build($this->args)
+        glsr( Schema::class )->store(
+            glsr( Schema::class )->build( $this->args )
         );
     }
 
@@ -110,20 +110,20 @@ 
      * @param string $text
      * @return string
      */
-    public function getExcerpt($text)
+    public function getExcerpt( $text )
     {
-        $limit = intval($this->getOption('settings.reviews.excerpts_length', 55));
-        $split = extension_loaded('intl')
-            ? $this->getExcerptIntlSplit($text, $limit)
-            : $this->getExcerptSplit($text, $limit);
-        $hiddenText = substr($text, $split);
-        if (!empty($hiddenText)) {
-            $showMore = glsr(Builder::class)->span($hiddenText, [
+        $limit = intval( $this->getOption( 'settings.reviews.excerpts_length', 55 ) );
+        $split = extension_loaded( 'intl' )
+            ? $this->getExcerptIntlSplit( $text, $limit )
+            : $this->getExcerptSplit( $text, $limit );
+        $hiddenText = substr( $text, $split );
+        if( !empty($hiddenText) ) {
+            $showMore = glsr( Builder::class )->span( $hiddenText, [
                 'class' => 'glsr-hidden glsr-hidden-text',
-                'data-show-less' => __('Show less', 'site-reviews'),
-                'data-show-more' => __('Show more', 'site-reviews'),
-            ]);
-            $text = ltrim(substr($text, 0, $split)).$showMore;
+                'data-show-less' => __( 'Show less', 'site-reviews' ),
+                'data-show-more' => __( 'Show more', 'site-reviews' ),
+            ] );
+            $text = ltrim( substr( $text, 0, $split ) ).$showMore;
         }
         return $text;
     }
@@ -133,12 +133,12 @@ 
      * @param string $path
      * @return bool
      */
-    public function isHidden($key, $path = '')
+    public function isHidden( $key, $path = '' )
     {
         $isOptionEnabled = !empty($path)
-            ? $this->isOptionEnabled($path)
+            ? $this->isOptionEnabled( $path )
             : true;
-        return in_array($key, $this->args['hide']) || !$isOptionEnabled;
+        return in_array( $key, $this->args['hide'] ) || !$isOptionEnabled;
     }
 
     /**
@@ -146,25 +146,25 @@ 
      * @param string $value
      * @return bool
      */
-    public function isHiddenOrEmpty($key, $value)
+    public function isHiddenOrEmpty( $key, $value )
     {
-        return $this->isHidden($key) || empty($value);
+        return $this->isHidden( $key ) || empty($value);
     }
 
     /**
      * @param string $text
      * @return string
      */
-    public function normalizeText($text)
+    public function normalizeText( $text )
     {
-        $text = wp_kses($text, wp_kses_allowed_html());
-        $text = convert_smilies(strip_shortcodes($text));
-        $text = str_replace(']]>', ']]>', $text);
-        $text = preg_replace('/(\R){2,}/', '$1', $text);
-        if ($this->isOptionEnabled('settings.reviews.excerpts')) {
-            $text = $this->getExcerpt($text);
+        $text = wp_kses( $text, wp_kses_allowed_html() );
+        $text = convert_smilies( strip_shortcodes( $text ) );
+        $text = str_replace( ']]>', ']]>', $text );
+        $text = preg_replace( '/(\R){2,}/', '$1', $text );
+        if( $this->isOptionEnabled( 'settings.reviews.excerpts' ) ) {
+            $text = $this->getExcerpt( $text );
         }
-        return wptexturize(nl2br($text));
+        return wptexturize( nl2br( $text ) );
     }
 
     /**
@@ -172,19 +172,19 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionAssignedTo($key, $value)
+    protected function buildOptionAssignedTo( $key, $value )
     {
-        if ($this->isHidden($key, 'settings.reviews.assigned_links')) {
+        if( $this->isHidden( $key, 'settings.reviews.assigned_links' ) ) {
             return;
         }
-        $post = glsr(Multilingual::class)->getPost($value);
-        if (!($post instanceof WP_Post)) {
+        $post = glsr( Multilingual::class )->getPost( $value );
+        if( !($post instanceof WP_Post) ) {
             return;
         }
-        $permalink = glsr(Builder::class)->a(get_the_title($post->ID), [
-            'href' => get_the_permalink($post->ID),
-        ]);
-        $assignedTo = sprintf(__('Review of %s', 'site-reviews'), $permalink);
+        $permalink = glsr( Builder::class )->a( get_the_title( $post->ID ), [
+            'href' => get_the_permalink( $post->ID ),
+        ] );
+        $assignedTo = sprintf( __( 'Review of %s', 'site-reviews' ), $permalink );
         return '<span>'.$assignedTo.'</span>';
     }
 
@@ -193,13 +193,13 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionAuthor($key, $value)
+    protected function buildOptionAuthor( $key, $value )
     {
-        if (!$this->isHidden($key)) {
+        if( !$this->isHidden( $key ) ) {
             $name = Str::convertName(
                 $value,
-                glsr_get_option('reviews.name.format'),
-                glsr_get_option('reviews.name.initial')
+                glsr_get_option( 'reviews.name.format' ),
+                glsr_get_option( 'reviews.name.initial' )
             );
             return '<span>'.$name.'</span>';
         }
@@ -210,18 +210,18 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionAvatar($key, $value)
+    protected function buildOptionAvatar( $key, $value )
     {
-        if ($this->isHidden($key, 'settings.reviews.avatars')) {
+        if( $this->isHidden( $key, 'settings.reviews.avatars' ) ) {
             return;
         }
-        $size = $this->getOption('settings.reviews.avatars_size', 40);
-        return glsr(Builder::class)->img([
+        $size = $this->getOption( 'settings.reviews.avatars_size', 40 );
+        return glsr( Builder::class )->img( [
             'height' => $size,
-            'src' => $this->generateAvatar($value),
-            'style' => sprintf('width:%1$spx; height:%1$spx;', $size),
+            'src' => $this->generateAvatar( $value ),
+            'style' => sprintf( 'width:%1$spx; height:%1$spx;', $size ),
             'width' => $size,
-        ]);
+        ] );
     }
 
     /**
@@ -229,10 +229,10 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionContent($key, $value)
+    protected function buildOptionContent( $key, $value )
     {
-        $text = $this->normalizeText($value);
-        if (!$this->isHiddenOrEmpty($key, $text)) {
+        $text = $this->normalizeText( $value );
+        if( !$this->isHiddenOrEmpty( $key, $text ) ) {
             return '<p>'.$text.'</p>';
         }
     }
@@ -242,19 +242,19 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionDate($key, $value)
+    protected function buildOptionDate( $key, $value )
     {
-        if ($this->isHidden($key)) {
+        if( $this->isHidden( $key ) ) {
             return;
         }
-        $dateFormat = $this->getOption('settings.reviews.date.format', 'default');
-        if ('relative' == $dateFormat) {
-            $date = glsr(Date::class)->relative($value);
+        $dateFormat = $this->getOption( 'settings.reviews.date.format', 'default' );
+        if( 'relative' == $dateFormat ) {
+            $date = glsr( Date::class )->relative( $value );
         } else {
             $format = 'custom' == $dateFormat
-                ? $this->getOption('settings.reviews.date.custom', 'M j, Y')
-                : glsr(OptionManager::class)->getWP('date_format', 'F j, Y');
-            $date = date_i18n($format, strtotime($value));
+                ? $this->getOption( 'settings.reviews.date.custom', 'M j, Y' )
+                : glsr( OptionManager::class )->getWP( 'date_format', 'F j, Y' );
+            $date = date_i18n( $format, strtotime( $value ) );
         }
         return '<span>'.$date.'</span>';
     }
@@ -264,10 +264,10 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionRating($key, $value)
+    protected function buildOptionRating( $key, $value )
     {
-        if (!$this->isHiddenOrEmpty($key, $value)) {
-            return glsr_star_rating($value);
+        if( !$this->isHiddenOrEmpty( $key, $value ) ) {
+            return glsr_star_rating( $value );
         }
     }
 
@@ -276,16 +276,16 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionResponse($key, $value)
+    protected function buildOptionResponse( $key, $value )
     {
-        if ($this->isHiddenOrEmpty($key, $value)) {
+        if( $this->isHiddenOrEmpty( $key, $value ) ) {
             return;
         }
-        $title = sprintf(__('Response from %s', 'site-reviews'), get_bloginfo('name'));
-        $text = $this->normalizeText($value);
+        $title = sprintf( __( 'Response from %s', 'site-reviews' ), get_bloginfo( 'name' ) );
+        $text = $this->normalizeText( $value );
         $text = '<p><strong>'.$title.'</strong></p><p>'.$text.'</p>';
-        $response = glsr(Builder::class)->div($text, ['class' => 'glsr-review-response-inner']);
-        $background = glsr(Builder::class)->div(['class' => 'glsr-review-response-background']);
+        $response = glsr( Builder::class )->div( $text, ['class' => 'glsr-review-response-inner'] );
+        $background = glsr( Builder::class )->div( ['class' => 'glsr-review-response-background'] );
         return $response.$background;
     }
 
@@ -294,13 +294,13 @@ 
      * @param string $value
      * @return void|string
      */
-    protected function buildOptionTitle($key, $value)
+    protected function buildOptionTitle( $key, $value )
     {
-        if ($this->isHidden($key)) {
+        if( $this->isHidden( $key ) ) {
             return;
         }
-        if (empty($value)) {
-            $value = __('No Title', 'site-reviews');
+        if( empty($value) ) {
+            $value = __( 'No Title', 'site-reviews' );
         }
         return '<h3>'.$value.'</h3>';
     }
@@ -309,16 +309,16 @@ 
      * @param string $avatarUrl
      * @return string
      */
-    protected function generateAvatar($avatarUrl)
+    protected function generateAvatar( $avatarUrl )
     {
-        if (!$this->isOptionEnabled('settings.reviews.avatars_regenerate') || 'local' != $this->current->review_type) {
+        if( !$this->isOptionEnabled( 'settings.reviews.avatars_regenerate' ) || 'local' != $this->current->review_type ) {
             return $avatarUrl;
         }
-        $authorIdOrEmail = get_the_author_meta('ID', $this->current->user_id);
-        if (empty($authorIdOrEmail)) {
+        $authorIdOrEmail = get_the_author_meta( 'ID', $this->current->user_id );
+        if( empty($authorIdOrEmail) ) {
             $authorIdOrEmail = $this->current->email;
         }
-        if ($newAvatar = get_avatar_url($authorIdOrEmail)) {
+        if( $newAvatar = get_avatar_url( $authorIdOrEmail ) ) {
             return $newAvatar;
         }
         return $avatarUrl;
@@ -329,22 +329,22 @@ 
      * @param int $limit
      * @return int
      */
-    protected function getExcerptIntlSplit($text, $limit)
+    protected function getExcerptIntlSplit( $text, $limit )
     {
-        $words = IntlRuleBasedBreakIterator::createWordInstance('');
-        $words->setText($text);
+        $words = IntlRuleBasedBreakIterator::createWordInstance( '' );
+        $words->setText( $text );
         $count = 0;
-        foreach ($words as $offset) {
-            if (IntlRuleBasedBreakIterator::WORD_NONE === $words->getRuleStatus()) {
+        foreach( $words as $offset ) {
+            if( IntlRuleBasedBreakIterator::WORD_NONE === $words->getRuleStatus() ) {
                 continue;
             }
             ++$count;
-            if ($count != $limit) {
+            if( $count != $limit ) {
                 continue;
             }
             return $offset;
         }
-        return strlen($text);
+        return strlen( $text );
     }
 
     /**
@@ -352,13 +352,13 @@ 
      * @param int $limit
      * @return int
      */
-    protected function getExcerptSplit($text, $limit)
+    protected function getExcerptSplit( $text, $limit )
     {
-        if (str_word_count($text, 0) > $limit) {
-            $words = array_keys(str_word_count($text, 2));
+        if( str_word_count( $text, 0 ) > $limit ) {
+            $words = array_keys( str_word_count( $text, 2 ) );
             return $words[$limit];
         }
-        return strlen($text);
+        return strlen( $text );
     }
 
     /**
@@ -366,9 +366,9 @@ 
      * @param mixed $fallback
      * @return mixed
      */
-    protected function getOption($path, $fallback = '')
+    protected function getOption( $path, $fallback = '' )
     {
-        if (array_key_exists($path, $this->options)) {
+        if( array_key_exists( $path, $this->options ) ) {
             return $this->options[$path];
         }
         return $fallback;
@@ -378,25 +378,25 @@ 
      * @param string $path
      * @return bool
      */
-    protected function isOptionEnabled($path)
+    protected function isOptionEnabled( $path )
     {
-        return 'yes' == $this->getOption($path);
+        return 'yes' == $this->getOption( $path );
     }
 
     /**
      * @return void
      */
-    protected function wrap(array &$renderedFields, Review $review)
+    protected function wrap( array &$renderedFields, Review $review )
     {
-        $renderedFields = apply_filters('site-reviews/review/wrap', $renderedFields, $review, $this);
-        array_walk($renderedFields, function (&$value, $key) use ($review) {
-            $value = apply_filters('site-reviews/review/wrap/'.$key, $value, $review);
-            if (empty($value)) {
+        $renderedFields = apply_filters( 'site-reviews/review/wrap', $renderedFields, $review, $this );
+        array_walk( $renderedFields, function( &$value, $key ) use ($review) {
+            $value = apply_filters( 'site-reviews/review/wrap/'.$key, $value, $review );
+            if( empty($value) ) {
                 return;
             }
-            $value = glsr(Builder::class)->div($value, [
+            $value = glsr( Builder::class )->div( $value, [
                 'class' => 'glsr-review-'.$key,
-            ]);
+            ] );
         });
     }
 }

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/Database/CountsManager.php

Spacing +128 added lines, -128 removed lines

@@ -21,26 +21,26 @@ 
      * @return array
      * @todo verify the additional type checks are needed
      */
-    public function buildCounts(array $args = [])
+    public function buildCounts( array $args = [] )
     {
         $counts = [];
-        $query = $this->queryReviews($args);
-        while ($query) {
-            $types = array_keys(array_flip(glsr_array_column($query->reviews, 'type')));
-            $types = array_unique(array_merge(['local'], $types));
-            foreach ($types as $type) {
-                $type = $this->normalizeType($type);
-                if (isset($counts[$type])) {
+        $query = $this->queryReviews( $args );
+        while( $query ) {
+            $types = array_keys( array_flip( glsr_array_column( $query->reviews, 'type' ) ) );
+            $types = array_unique( array_merge( ['local'], $types ) );
+            foreach( $types as $type ) {
+                $type = $this->normalizeType( $type );
+                if( isset($counts[$type]) ) {
                     continue;
             }
-                $counts[$type] = array_fill_keys(range(0, glsr()->constant('MAX_RATING', Rating::class)), 0);
+                $counts[$type] = array_fill_keys( range( 0, glsr()->constant( 'MAX_RATING', Rating::class ) ), 0 );
             }
-            foreach ($query->reviews as $review) {
-                $type = $this->normalizeType($review->type);
+            foreach( $query->reviews as $review ) {
+                $type = $this->normalizeType( $review->type );
                 ++$counts[$type][$review->rating];
             }
             $query = $query->has_more
-                ? $this->queryReviews($args, end($query->reviews)->ID)
+                ? $this->queryReviews( $args, end( $query->reviews )->ID )
                 : false;
         }
         return $counts;
@@ -50,18 +50,18 @@ 
      * @param int $postId
      * @return array
      */
-    public function buildPostCounts($postId)
+    public function buildPostCounts( $postId )
     {
-        return $this->buildCounts(['post_ids' => [$postId]]);
+        return $this->buildCounts( ['post_ids' => [$postId]] );
     }
 
     /**
      * @param int $termTaxonomyId
      * @return array
      */
-    public function buildTermCounts($termTaxonomyId)
+    public function buildTermCounts( $termTaxonomyId )
     {
-        return $this->buildCounts(['term_ids' => [$termTaxonomyId]]);
+        return $this->buildCounts( ['term_ids' => [$termTaxonomyId]] );
     }
 
     /**
@@ -69,80 +69,80 @@ 
      */
     public function countAll()
     {
-        $terms = glsr(Database::class)->getTerms(['fields' => 'all']);
-        foreach ($terms as $term) {
-            $this->setTermCounts($term->term_id, $this->buildTermCounts($term->term_taxonomy_id));
+        $terms = glsr( Database::class )->getTerms( ['fields' => 'all'] );
+        foreach( $terms as $term ) {
+            $this->setTermCounts( $term->term_id, $this->buildTermCounts( $term->term_taxonomy_id ) );
         }
-        $postIds = glsr(SqlQueries::class)->getReviewsMeta('assigned_to');
-        foreach ($postIds as $postId) {
-            $this->setPostCounts($postId, $this->buildPostCounts($postId));
+        $postIds = glsr( SqlQueries::class )->getReviewsMeta( 'assigned_to' );
+        foreach( $postIds as $postId ) {
+            $this->setPostCounts( $postId, $this->buildPostCounts( $postId ) );
         }
-        $this->setCounts($this->buildCounts());
+        $this->setCounts( $this->buildCounts() );
     }
 
     /**
      * @return void
      */
-    public function decrease(Review $review)
+    public function decrease( Review $review )
     {
-        $this->decreaseCounts($review);
-        $this->decreasePostCounts($review);
-        $this->decreaseTermCounts($review);
+        $this->decreaseCounts( $review );
+        $this->decreasePostCounts( $review );
+        $this->decreaseTermCounts( $review );
     }
 
     /**
      * @return void
      */
-    public function decreaseCounts(Review $review)
+    public function decreaseCounts( Review $review )
     {
-        $this->setCounts($this->decreaseRating(
+        $this->setCounts( $this->decreaseRating(
             $this->getCounts(),
             $review->review_type,
             $review->rating
-        ));
+        ) );
     }
 
     /**
      * @return void
      */
-    public function decreasePostCounts(Review $review)
+    public function decreasePostCounts( Review $review )
     {
-        if (empty($counts = $this->getPostCounts($review->assigned_to))) {
+        if( empty($counts = $this->getPostCounts( $review->assigned_to )) ) {
             return;
         }
-        $counts = $this->decreaseRating($counts, $review->review_type, $review->rating);
-        $this->setPostCounts($review->assigned_to, $counts);
+        $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
+        $this->setPostCounts( $review->assigned_to, $counts );
     }
 
     /**
      * @return void
      */
-    public function decreaseTermCounts(Review $review)
+    public function decreaseTermCounts( Review $review )
     {
-        foreach ($review->term_ids as $termId) {
-            if (empty($counts = $this->getTermCounts($termId))) {
+        foreach( $review->term_ids as $termId ) {
+            if( empty($counts = $this->getTermCounts( $termId )) ) {
                 continue;
             }
-            $counts = $this->decreaseRating($counts, $review->review_type, $review->rating);
-            $this->setTermCounts($termId, $counts);
+            $counts = $this->decreaseRating( $counts, $review->review_type, $review->rating );
+            $this->setTermCounts( $termId, $counts );
         }
     }
 
     /**
      * @return array
      */
-    public function flatten(array $reviewCounts, array $args = [])
+    public function flatten( array $reviewCounts, array $args = [] )
     {
         $counts = [];
-        array_walk_recursive($reviewCounts, function ($num, $index) use (&$counts) {
-            $counts[$index] = $num + intval(Arr::get($counts, $index, 0));
+        array_walk_recursive( $reviewCounts, function( $num, $index ) use (&$counts) {
+            $counts[$index] = $num + intval( Arr::get( $counts, $index, 0 ) );
         });
-        $args = wp_parse_args($args, [
-            'max' => glsr()->constant('MAX_RATING', Rating::class),
-            'min' => glsr()->constant('MIN_RATING', Rating::class),
-        ]);
-        foreach ($counts as $index => &$num) {
-            if ($index >= intval($args['min']) && $index <= intval($args['max'])) {
+        $args = wp_parse_args( $args, [
+            'max' => glsr()->constant( 'MAX_RATING', Rating::class ),
+            'min' => glsr()->constant( 'MIN_RATING', Rating::class ),
+        ] );
+        foreach( $counts as $index => &$num ) {
+            if( $index >= intval( $args['min'] ) && $index <= intval( $args['max'] ) ) {
                 continue;
             }
             $num = 0;
@@ -153,26 +153,26 @@ 
     /**
      * @return array
      */
-    public function get(array $args = [])
+    public function get( array $args = [] )
     {
-        $args = $this->normalizeArgs($args);
+        $args = $this->normalizeArgs( $args );
         $counts = [];
-        if ($this->isMixedCount($args)) {
-            $counts = [$this->buildCounts($args)]; // force query the database
+        if( $this->isMixedCount( $args ) ) {
+            $counts = [$this->buildCounts( $args )]; // force query the database
         } else {
-            foreach ($args['post_ids'] as $postId) {
-                $counts[] = $this->getPostCounts($postId);
+            foreach( $args['post_ids'] as $postId ) {
+                $counts[] = $this->getPostCounts( $postId );
         }
-            foreach ($args['term_ids'] as $termId) {
-                $counts[] = $this->getTermCounts($termId);
+            foreach( $args['term_ids'] as $termId ) {
+                $counts[] = $this->getTermCounts( $termId );
             }
-            if (empty($counts)) {
+            if( empty($counts) ) {
                 $counts[] = $this->getCounts();
             }
         }
-        return in_array($args['type'], ['', 'all'])
-            ? $this->normalize([$this->flatten($counts)])
-            : $this->normalize(glsr_array_column($counts, $args['type']));
+        return in_array( $args['type'], ['', 'all'] )
+            ? $this->normalize( [$this->flatten( $counts )] )
+            : $this->normalize( glsr_array_column( $counts, $args['type'] ) );
     }
 
     /**
@@ -180,9 +180,9 @@ 
      */
     public function getCounts()
     {
-        $counts = glsr(OptionManager::class)->get('counts', []);
-        if (!is_array($counts)) {
-            glsr_log()->error('Review counts is not an array; possibly due to incorrectly imported reviews.')->debug($counts);
+        $counts = glsr( OptionManager::class )->get( 'counts', [] );
+        if( !is_array( $counts ) ) {
+            glsr_log()->error( 'Review counts is not an array; possibly due to incorrectly imported reviews.' )->debug( $counts );
             return [];
         }
         return $counts;
@@ -192,105 +192,105 @@ 
      * @param int $postId
      * @return array
      */
-    public function getPostCounts($postId)
+    public function getPostCounts( $postId )
     {
-        return array_filter((array) get_post_meta($postId, static::META_COUNT, true));
+        return array_filter( (array)get_post_meta( $postId, static::META_COUNT, true ) );
     }
 
     /**
      * @param int $termId
      * @return array
      */
-    public function getTermCounts($termId)
+    public function getTermCounts( $termId )
     {
-        return array_filter((array) get_term_meta($termId, static::META_COUNT, true));
+        return array_filter( (array)get_term_meta( $termId, static::META_COUNT, true ) );
     }
 
     /**
      * @return void
      */
-    public function increase(Review $review)
+    public function increase( Review $review )
     {
-        $this->increaseCounts($review);
-        $this->increasePostCounts($review);
-        $this->increaseTermCounts($review);
+        $this->increaseCounts( $review );
+        $this->increasePostCounts( $review );
+        $this->increaseTermCounts( $review );
     }
 
     /**
      * @return void
      */
-    public function increaseCounts(Review $review)
+    public function increaseCounts( Review $review )
     {
-        if (empty($counts = $this->getCounts())) {
+        if( empty($counts = $this->getCounts()) ) {
             $counts = $this->buildCounts();
         }
-        $this->setCounts($this->increaseRating($counts, $review->review_type, $review->rating));
+        $this->setCounts( $this->increaseRating( $counts, $review->review_type, $review->rating ) );
     }
 
     /**
      * @return void
      */
-    public function increasePostCounts(Review $review)
+    public function increasePostCounts( Review $review )
     {
-        if (!(get_post($review->assigned_to) instanceof WP_Post)) {
+        if( !(get_post( $review->assigned_to ) instanceof WP_Post) ) {
             return;
         }
-        $counts = $this->getPostCounts($review->assigned_to);
+        $counts = $this->getPostCounts( $review->assigned_to );
         $counts = empty($counts)
-            ? $this->buildPostCounts($review->assigned_to)
-            : $this->increaseRating($counts, $review->review_type, $review->rating);
-        $this->setPostCounts($review->assigned_to, $counts);
+            ? $this->buildPostCounts( $review->assigned_to )
+            : $this->increaseRating( $counts, $review->review_type, $review->rating );
+        $this->setPostCounts( $review->assigned_to, $counts );
     }
 
     /**
      * @return void
      */
-    public function increaseTermCounts(Review $review)
+    public function increaseTermCounts( Review $review )
     {
-        $terms = glsr(ReviewManager::class)->normalizeTerms(implode(',', $review->term_ids));
-        foreach ($terms as $term) {
-            $counts = $this->getTermCounts($term['term_id']);
+        $terms = glsr( ReviewManager::class )->normalizeTerms( implode( ',', $review->term_ids ) );
+        foreach( $terms as $term ) {
+            $counts = $this->getTermCounts( $term['term_id'] );
             $counts = empty($counts)
-                ? $this->buildTermCounts($term['term_taxonomy_id'])
-                : $this->increaseRating($counts, $review->review_type, $review->rating);
-            $this->setTermCounts($term['term_id'], $counts);
+                ? $this->buildTermCounts( $term['term_taxonomy_id'] )
+                : $this->increaseRating( $counts, $review->review_type, $review->rating );
+            $this->setTermCounts( $term['term_id'], $counts );
         }
     }
 
     /**
      * @return void
      */
-    public function setCounts(array $reviewCounts)
+    public function setCounts( array $reviewCounts )
     {
-        glsr(OptionManager::class)->set('counts', $reviewCounts);
+        glsr( OptionManager::class )->set( 'counts', $reviewCounts );
     }
 
     /**
      * @param int $postId
      * @return void
      */
-    public function setPostCounts($postId, array $reviewCounts)
+    public function setPostCounts( $postId, array $reviewCounts )
     {
-        $ratingCounts = $this->flatten($reviewCounts);
-        update_post_meta($postId, static::META_COUNT, $reviewCounts);
-        update_post_meta($postId, static::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
-        update_post_meta($postId, static::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
+        $ratingCounts = $this->flatten( $reviewCounts );
+        update_post_meta( $postId, static::META_COUNT, $reviewCounts );
+        update_post_meta( $postId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) );
+        update_post_meta( $postId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) );
     }
 
     /**
      * @param int $termId
      * @return void
      */
-    public function setTermCounts($termId, array $reviewCounts)
+    public function setTermCounts( $termId, array $reviewCounts )
     {
-        $term = get_term($termId, Application::TAXONOMY);
-        if (!isset($term->term_id)) {
+        $term = get_term( $termId, Application::TAXONOMY );
+        if( !isset($term->term_id) ) {
             return;
         }
-        $ratingCounts = $this->flatten($reviewCounts);
-        update_term_meta($termId, static::META_COUNT, $reviewCounts);
-        update_term_meta($termId, static::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
-        update_term_meta($termId, static::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
+        $ratingCounts = $this->flatten( $reviewCounts );
+        update_term_meta( $termId, static::META_COUNT, $reviewCounts );
+        update_term_meta( $termId, static::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) );
+        update_term_meta( $termId, static::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) );
     }
 
     /**
@@ -298,10 +298,10 @@ 
      * @param int $rating
      * @return array
      */
-    protected function decreaseRating(array $reviewCounts, $type, $rating)
+    protected function decreaseRating( array $reviewCounts, $type, $rating )
     {
-        if (isset($reviewCounts[$type][$rating])) {
-            $reviewCounts[$type][$rating] = max(0, $reviewCounts[$type][$rating] - 1);
+        if( isset($reviewCounts[$type][$rating]) ) {
+            $reviewCounts[$type][$rating] = max( 0, $reviewCounts[$type][$rating] - 1 );
         }
         return $reviewCounts;
     }
@@ -311,23 +311,23 @@ 
      * @param int $rating
      * @return array
      */
-    protected function increaseRating(array $reviewCounts, $type, $rating)
+    protected function increaseRating( array $reviewCounts, $type, $rating )
     {
-        if (!array_key_exists($type, glsr()->reviewTypes)) {
+        if( !array_key_exists( $type, glsr()->reviewTypes ) ) {
             return $reviewCounts;
         }
-        if (!array_key_exists($type, $reviewCounts)) {
+        if( !array_key_exists( $type, $reviewCounts ) ) {
             $reviewCounts[$type] = [];
         }
-        $reviewCounts = $this->normalize($reviewCounts);
-        $reviewCounts[$type][$rating] = intval($reviewCounts[$type][$rating]) + 1;
+        $reviewCounts = $this->normalize( $reviewCounts );
+        $reviewCounts[$type][$rating] = intval( $reviewCounts[$type][$rating] ) + 1;
         return $reviewCounts;
     }
 
     /**
      * @return bool
      */
-    protected function isMixedCount(array $args)
+    protected function isMixedCount( array $args )
     {
         return !empty($args['post_ids']) && !empty($args['term_ids']);
     }
@@ -335,19 +335,19 @@ 
     /**
      * @return array
      */
-    protected function normalize(array $reviewCounts)
+    protected function normalize( array $reviewCounts )
     {
-        if (empty($reviewCounts)) {
+        if( empty($reviewCounts) ) {
             $reviewCounts = [[]];
         }
-        foreach ($reviewCounts as &$counts) {
-            foreach (range(0, glsr()->constant('MAX_RATING', Rating::class)) as $index) {
-                if (isset($counts[$index])) {
+        foreach( $reviewCounts as &$counts ) {
+            foreach( range( 0, glsr()->constant( 'MAX_RATING', Rating::class ) ) as $index ) {
+                if( isset($counts[$index]) ) {
                     continue;
                 }
                 $counts[$index] = 0;
             }
-            ksort($counts);
+            ksort( $counts );
         }
         return $reviewCounts;
     }
@@ -355,15 +355,15 @@ 
     /**
      * @return array
      */
-    protected function normalizeArgs(array $args)
+    protected function normalizeArgs( array $args )
     {
-        $args = wp_parse_args(array_filter($args), [
+        $args = wp_parse_args( array_filter( $args ), [
             'post_ids' => [],
             'term_ids' => [],
             'type' => 'local',
-        ]);
-        $args['post_ids'] = glsr(Multilingual::class)->getPostIds($args['post_ids']);
-        $args['type'] = $this->normalizeType($args['type']);
+        ] );
+        $args['post_ids'] = glsr( Multilingual::class )->getPostIds( $args['post_ids'] );
+        $args['type'] = $this->normalizeType( $args['type'] );
         return $args;
     }
 
@@ -371,9 +371,9 @@ 
      * @param string $type
      * @return string
      */
-    protected function normalizeType($type)
+    protected function normalizeType( $type )
     {
-        return empty($type) || !is_string($type)
+        return empty($type) || !is_string( $type )
             ? 'local'
             : $type;
     }
@@ -382,13 +382,13 @@ 
      * @param int $lastPostId
      * @return object
      */
-    protected function queryReviews(array $args = [], $lastPostId = 0)
+    protected function queryReviews( array $args = [], $lastPostId = 0 )
     {
-        $reviews = glsr(SqlQueries::class)->getReviewCounts($args, $lastPostId, static::LIMIT);
-        $hasMore = is_array($reviews)
-            ? count($reviews) == static::LIMIT
+        $reviews = glsr( SqlQueries::class )->getReviewCounts( $args, $lastPostId, static::LIMIT );
+        $hasMore = is_array( $reviews )
+            ? count( $reviews ) == static::LIMIT
             : false;
-        return (object) [
+        return (object)[
             'has_more' => $hasMore,
             'reviews' => $reviews,
         ];

plugin/Controllers/SettingsController.php

Spacing +44 added lines, -44 removed lines

@@ -14,17 +14,17 @@ 
      * @return array
      * @callback register_setting
      */
-    public function callbackRegisterSettings($input)
+    public function callbackRegisterSettings( $input )
     {
-        $settings = Arr::consolidateArray($input);
-        if (1 === count($settings) && array_key_exists('settings', $settings)) {
-            $options = array_replace_recursive(glsr(OptionManager::class)->all(), $input);
-            $options = $this->sanitizeGeneral($input, $options);
-            $options = $this->sanitizeSubmissions($input, $options);
-            $options = $this->sanitizeTranslations($input, $options);
-            $options = apply_filters('site-reviews/settings/callback', $options, $settings);
-            if (filter_input(INPUT_POST, 'option_page') == Application::ID.'-settings') {
-                glsr(Notice::class)->addSuccess(__('Settings updated.', 'site-reviews'));
+        $settings = Arr::consolidateArray( $input );
+        if( 1 === count( $settings ) && array_key_exists( 'settings', $settings ) ) {
+            $options = array_replace_recursive( glsr( OptionManager::class )->all(), $input );
+            $options = $this->sanitizeGeneral( $input, $options );
+            $options = $this->sanitizeSubmissions( $input, $options );
+            $options = $this->sanitizeTranslations( $input, $options );
+            $options = apply_filters( 'site-reviews/settings/callback', $options, $settings );
+            if( filter_input( INPUT_POST, 'option_page' ) == Application::ID.'-settings' ) {
+                glsr( Notice::class )->addSuccess( __( 'Settings updated.', 'site-reviews' ) );
             }
             return $options;
         }
@@ -37,63 +37,63 @@ 
      */
     public function registerSettings()
     {
-        register_setting(Application::ID.'-settings', OptionManager::databaseKey(), [
+        register_setting( Application::ID.'-settings', OptionManager::databaseKey(), [
             'sanitize_callback' => [$this, 'callbackRegisterSettings'],
-        ]);
+        ] );
     }
 
     /**
      * @return array
      */
-    protected function sanitizeGeneral(array $input, array $options)
+    protected function sanitizeGeneral( array $input, array $options )
     {
         $key = 'settings.general';
-        $inputForm = Arr::get($input, $key);
-        if (!$this->hasMultilingualIntegration(Arr::get($inputForm, 'multilingual'))) {
-            $options = Arr::set($options, $key.'.multilingual', '');
+        $inputForm = Arr::get( $input, $key );
+        if( !$this->hasMultilingualIntegration( Arr::get( $inputForm, 'multilingual' ) ) ) {
+            $options = Arr::set( $options, $key.'.multilingual', '' );
         }
-        if ('' == trim(Arr::get($inputForm, 'notification_message'))) {
-            $defaultValue = Arr::get(glsr()->defaults, $key.'.notification_message');
-            $options = Arr::set($options, $key.'.notification_message', $defaultValue);
+        if( '' == trim( Arr::get( $inputForm, 'notification_message' ) ) ) {
+            $defaultValue = Arr::get( glsr()->defaults, $key.'.notification_message' );
+            $options = Arr::set( $options, $key.'.notification_message', $defaultValue );
         }
-        $defaultValue = Arr::get($inputForm, 'notifications', []);
-        $options = Arr::set($options, $key.'.notifications', $defaultValue);
+        $defaultValue = Arr::get( $inputForm, 'notifications', [] );
+        $options = Arr::set( $options, $key.'.notifications', $defaultValue );
         return $options;
     }
 
     /**
      * @return array
      */
-    protected function sanitizeSubmissions(array $input, array $options)
+    protected function sanitizeSubmissions( array $input, array $options )
     {
         $key = 'settings.submissions';
-        $inputForm = Arr::get($input, $key);
+        $inputForm = Arr::get( $input, $key );
         $defaultValue = isset($inputForm['required'])
             ? $inputForm['required']
             : [];
-        $options = Arr::set($options, $key.'.required', $defaultValue);
+        $options = Arr::set( $options, $key.'.required', $defaultValue );
         return $options;
     }
 
     /**
      * @return array
      */
-    protected function sanitizeTranslations(array $input, array $options)
+    protected function sanitizeTranslations( array $input, array $options )
     {
         $key = 'settings.strings';
-        $inputForm = Arr::consolidateArray(Arr::get($input, $key));
-        if (!empty($inputForm)) {
-            $options = Arr::set($options, $key, array_values(array_filter($inputForm)));
+        $inputForm = Arr::consolidateArray( Arr::get( $input, $key ) );
+        if( !empty($inputForm) ) {
+            $options = Arr::set( $options, $key, array_values( array_filter( $inputForm ) ) );
             $allowedTags = [
                 'a' => ['class' => [], 'href' => [], 'target' => []],
                 'span' => ['class' => []],
             ];
-            array_walk($options['settings']['strings'], function (&$string) use ($allowedTags) {
-                if (isset($string['s2'])) {
-                    $string['s2'] = wp_kses($string['s2'], $allowedTags);
+            array_walk( $options['settings']['strings'], function( &$string ) use ($allowedTags) {
+                if( isset($string['s2']) ) {
+                    $string['s2'] = wp_kses( $string['s2'], $allowedTags );
                 }
-                if (isset($string['p2'])) {
-                    $string['p2'] = wp_kses($string['p2'], $allowedTags);
+                if( isset($string['p2']) ) {
+                    $string['p2'] = wp_kses( $string['p2'], $allowedTags );
                 }
             });
         }
@@ -104,24 +104,24 @@ 
      * @param string $integrationSlug
      * @return bool
      */
-    protected function hasMultilingualIntegration($integrationSlug)
+    protected function hasMultilingualIntegration( $integrationSlug )
     {
-        $integration = glsr(Multilingual::class)->getIntegration($integrationSlug);
-        if (!$integration) {
+        $integration = glsr( Multilingual::class )->getIntegration( $integrationSlug );
+        if( !$integration ) {
             return false;
         }
-        if (!$integration->isActive()) {
-            glsr(Notice::class)->addError(sprintf(
-                __('Please install/activate the %s plugin to enable integration.', 'site-reviews'),
+        if( !$integration->isActive() ) {
+            glsr( Notice::class )->addError( sprintf(
+                __( 'Please install/activate the %s plugin to enable integration.', 'site-reviews' ),
                 $integration->pluginName
-            ));
+            ) );
             return false;
-        } elseif (!$integration->isSupported()) {
-            glsr(Notice::class)->addError(sprintf(
-                __('Please update the %s plugin to v%s or greater to enable integration.', 'site-reviews'),
+        } elseif( !$integration->isSupported() ) {
+            glsr( Notice::class )->addError( sprintf(
+                __( 'Please update the %s plugin to v%s or greater to enable integration.', 'site-reviews' ),
                 $integration->pluginName,
                 $integration->supportedVersion
-            ));
+            ) );
             return false;
         }
         return true;

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 +12 added lines, -12 removed lines

@@ -21,28 +21,28 @@ 
      * @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 = ucfirst(glsr(OptionManager::class)->get('settings.general.multilingual'));
+        if( empty($integration) ) {
+            $integration = ucfirst( glsr( OptionManager::class )->get( 'settings.general.multilingual' ) );
         }
         $integrationClass = 'GeminiLabs\SiteReviews\Modules\Multilingual\\'.$integration;
-        if (class_exists($integrationClass)) {
-            return glsr($integrationClass);
+        if( class_exists( $integrationClass ) ) {
+            return glsr( $integrationClass );
         }
-        glsr_log()->error($integrationClass.' does not exist');
+        glsr_log()->error( $integrationClass.' does not exist' );
         return false;
     }
 
@@ -51,10 +51,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;
         }

plugin/Modules/Multilingual/Wpml.php

Spacing +19 added lines, -19 removed lines

@@ -14,44 +14,44 @@ 
     /**
      * {@inheritdoc}
      */
-    public function getPost($postId)
+    public function getPost( $postId )
     {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
+        $postId = trim( $postId );
+        if( !is_numeric( $postId ) ) {
             return;
         }
-        if ($this->isEnabled()) {
-            $postId = apply_filters('wpml_object_id', $postId, 'any', true);
+        if( $this->isEnabled() ) {
+            $postId = apply_filters( 'wpml_object_id', $postId, 'any', true );
         }
-        return get_post(intval($postId));
+        return get_post( intval( $postId ) );
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getPostIds(array $postIds)
+    public function getPostIds( array $postIds )
     {
-        if (!$this->isEnabled()) {
+        if( !$this->isEnabled() ) {
             return $postIds;
         }
         $newPostIds = [];
-        foreach (Arr::unique($postIds) as $postId) {
-            $postType = get_post_type($postId);
-            if (!$postType) {
+        foreach( Arr::unique( $postIds ) as $postId ) {
+            $postType = get_post_type( $postId );
+            if( !$postType ) {
                 continue;
             }
             $elementType = 'post_'.$postType;
-            $trid = apply_filters('wpml_element_trid', null, $postId, $elementType);
-            $translations = apply_filters('wpml_get_element_translations', null, $trid, $elementType);
-            if (!is_array($translations)) {
+            $trid = apply_filters( 'wpml_element_trid', null, $postId, $elementType );
+            $translations = apply_filters( 'wpml_get_element_translations', null, $trid, $elementType );
+            if( !is_array( $translations ) ) {
                 $translations = [];
             }
             $newPostIds = array_merge(
                 $newPostIds,
-                array_column($translations, 'element_id')
+                array_column( $translations, 'element_id' )
             );
         }
-        return Arr::unique($newPostIds);
+        return Arr::unique( $newPostIds );
     }
 
     /**
@@ -59,7 +59,7 @@ 
      */
     public function isActive()
     {
-        return defined('ICL_SITEPRESS_VERSION');
+        return defined( 'ICL_SITEPRESS_VERSION' );
     }
 
     /**
@@ -68,7 +68,7 @@ 
     public function isEnabled()
     {
         return $this->isActive()
-            && 'wpml' == glsr(OptionManager::class)->get('settings.general.multilingual');
+            && 'wpml' == glsr( OptionManager::class )->get( 'settings.general.multilingual' );
     }
 
     /**
@@ -77,6 +77,6 @@ 
     public function isSupported()
     {
         return $this->isActive()
-            && version_compare(ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=');
+            && version_compare( ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=' );
     }
 }

plugin/Modules/Multilingual/Polylang.php

Spacing +19 added lines, -19 removed lines

@@ -14,37 +14,37 @@ 
     /**
      * {@inheritdoc}
      */
-    public function getPost($postId)
+    public function getPost( $postId )
     {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
+        $postId = trim( $postId );
+        if( !is_numeric( $postId ) ) {
             return;
         }
-        if ($this->isEnabled()) {
-            $polylangPostId = pll_get_post($postId, pll_get_post_language(get_the_ID()));
+        if( $this->isEnabled() ) {
+            $polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ) );
         }
-        if (!empty($polylangPostId)) {
+        if( !empty($polylangPostId) ) {
             $postId = $polylangPostId;
         }
-        return get_post(intval($postId));
+        return get_post( intval( $postId ) );
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getPostIds(array $postIds)
+    public function getPostIds( array $postIds )
     {
-        if (!$this->isEnabled()) {
+        if( !$this->isEnabled() ) {
             return $postIds;
         }
         $newPostIds = [];
-        foreach (Arr::unique($postIds) as $postId) {
+        foreach( Arr::unique( $postIds ) as $postId ) {
             $newPostIds = array_merge(
                 $newPostIds,
-                array_values(pll_get_post_translations($postId))
+                array_values( pll_get_post_translations( $postId ) )
             );
         }
-        return Arr::unique($newPostIds);
+        return Arr::unique( $newPostIds );
     }
 
     /**
@@ -52,10 +52,10 @@ 
      */
     public function isActive()
     {
-        return function_exists('PLL')
-            && function_exists('pll_get_post')
-            && function_exists('pll_get_post_language')
-            && function_exists('pll_get_post_translations');
+        return function_exists( 'PLL' )
+            && function_exists( 'pll_get_post' )
+            && function_exists( 'pll_get_post_language' )
+            && function_exists( 'pll_get_post_translations' );
     }
 
     /**
@@ -64,7 +64,7 @@ 
     public function isEnabled()
     {
         return $this->isActive()
-            && 'polylang' == glsr(OptionManager::class)->get('settings.general.multilingual');
+            && 'polylang' == glsr( OptionManager::class )->get( 'settings.general.multilingual' );
     }
 
     /**
@@ -72,7 +72,7 @@ 
      */
     public function isSupported()
     {
-        return defined('POLYLANG_VERSION')
-            && version_compare(POLYLANG_VERSION, $this->supportedVersion, '>=');
+        return defined( 'POLYLANG_VERSION' )
+            && version_compare( POLYLANG_VERSION, $this->supportedVersion, '>=' );
     }
 }