pryley / site-reviews

plugin/Modules/Html/Partials/SiteReviews.php

Indentation +353 added lines, -353 removed lines

@@ -21,382 +21,382 @@
 
 class SiteReviews
 {
-    /**
-     * @var array
-     */
-    public $args;
+	/**
+	 * @var array
+	 */
+	public $args;
 
-    /**
-     * @var Review
-     */
-    public $current;
+	/**
+	 * @var Review
+	 */
+	public $current;
 
-    /**
-     * @var array
-     */
-    public $options;
+	/**
+	 * @var array
+	 */
+	public $options;
 
-    /**
-     * @var Reviews
-     */
-    protected $reviews;
+	/**
+	 * @var Reviews
+	 */
+	protected $reviews;
 
-    /**
-     * @param Reviews|null $reviews
-     * @return ReviewsHtml
-     */
-    public function build(array $args = [], $reviews = null)
-    {
-        $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);
-        $this->generateSchema();
-        return $this->buildReviews();
-    }
+	/**
+	 * @param Reviews|null $reviews
+	 * @return ReviewsHtml
+	 */
+	public function build(array $args = [], $reviews = null)
+	{
+		$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);
+		$this->generateSchema();
+		return $this->buildReviews();
+	}
 
-    /**
-     * @return ReviewHtml
-     */
-    public function buildReview(Review $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)
-                : false;
-            $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->current = null;
-        return new ReviewHtml($review, (array) $renderedFields);
-    }
+	/**
+	 * @return ReviewHtml
+	 */
+	public function buildReview(Review $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)
+				: false;
+			$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->current = null;
+		return new ReviewHtml($review, (array) $renderedFields);
+	}
 
-    /**
-     * @return ReviewsHtml
-     */
-    public function buildReviews()
-    {
-        $renderedReviews = [];
-        foreach ($this->reviews as $index => $review) {
-            $renderedReviews[] = $this->buildReview($review);
-        }
-        return new ReviewsHtml($renderedReviews, $this->reviews->max_num_pages, $this->args);
-    }
+	/**
+	 * @return ReviewsHtml
+	 */
+	public function buildReviews()
+	{
+		$renderedReviews = [];
+		foreach ($this->reviews as $index => $review) {
+			$renderedReviews[] = $this->buildReview($review);
+		}
+		return new ReviewsHtml($renderedReviews, $this->reviews->max_num_pages, $this->args);
+	}
 
-    /**
-     * @return void
-     */
-    public function generateSchema()
-    {
-        if (!wp_validate_boolean($this->args['schema'])) {
-            return;
-        }
-        glsr(Schema::class)->store(
-            glsr(Schema::class)->build($this->args)
-        );
-    }
+	/**
+	 * @return void
+	 */
+	public function generateSchema()
+	{
+		if (!wp_validate_boolean($this->args['schema'])) {
+			return;
+		}
+		glsr(Schema::class)->store(
+			glsr(Schema::class)->build($this->args)
+		);
+	}
 
-    /**
-     * @param string $text
-     * @return string
-     */
-    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, [
-                '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;
-        }
-        return $text;
-    }
+	/**
+	 * @param string $text
+	 * @return string
+	 */
+	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, [
+				'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;
+		}
+		return $text;
+	}
 
-    /**
-     * @param string $key
-     * @param string $path
-     * @return bool
-     */
-    public function isHidden($key, $path = '')
-    {
-        $isOptionEnabled = !empty($path)
-            ? $this->isOptionEnabled($path)
-            : true;
-        return in_array($key, $this->args['hide']) || !$isOptionEnabled;
-    }
+	/**
+	 * @param string $key
+	 * @param string $path
+	 * @return bool
+	 */
+	public function isHidden($key, $path = '')
+	{
+		$isOptionEnabled = !empty($path)
+			? $this->isOptionEnabled($path)
+			: true;
+		return in_array($key, $this->args['hide']) || !$isOptionEnabled;
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return bool
-     */
-    public function isHiddenOrEmpty($key, $value)
-    {
-        return $this->isHidden($key) || empty($value);
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return bool
+	 */
+	public function isHiddenOrEmpty($key, $value)
+	{
+		return $this->isHidden($key) || empty($value);
+	}
 
-    /**
-     * @param string $text
-     * @return string
-     */
-    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);
-        }
-        return wptexturize(nl2br($text));
-    }
+	/**
+	 * @param string $text
+	 * @return string
+	 */
+	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);
+		}
+		return wptexturize(nl2br($text));
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionAssignedTo($key, $value)
-    {
-        if ($this->isHidden($key, 'settings.reviews.assigned_links')) {
-            return;
-        }
-        $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);
-        return '<span>'.$assignedTo.'</span>';
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionAssignedTo($key, $value)
+	{
+		if ($this->isHidden($key, 'settings.reviews.assigned_links')) {
+			return;
+		}
+		$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);
+		return '<span>'.$assignedTo.'</span>';
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionAuthor($key, $value)
-    {
-        if (!$this->isHidden($key)) {
-            $name = Str::convertName(
-                $value,
-                glsr_get_option('reviews.name.format'),
-                glsr_get_option('reviews.name.initial')
-            );
-            return '<span>'.$name.'</span>';
-        }
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionAuthor($key, $value)
+	{
+		if (!$this->isHidden($key)) {
+			$name = Str::convertName(
+				$value,
+				glsr_get_option('reviews.name.format'),
+				glsr_get_option('reviews.name.initial')
+			);
+			return '<span>'.$name.'</span>';
+		}
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionAvatar($key, $value)
-    {
-        if ($this->isHidden($key, 'settings.reviews.avatars')) {
-            return;
-        }
-        $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),
-            'width' => $size,
-        ]);
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionAvatar($key, $value)
+	{
+		if ($this->isHidden($key, 'settings.reviews.avatars')) {
+			return;
+		}
+		$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),
+			'width' => $size,
+		]);
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionContent($key, $value)
-    {
-        $text = $this->normalizeText($value);
-        if (!$this->isHiddenOrEmpty($key, $text)) {
-            return '<p>'.$text.'</p>';
-        }
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionContent($key, $value)
+	{
+		$text = $this->normalizeText($value);
+		if (!$this->isHiddenOrEmpty($key, $text)) {
+			return '<p>'.$text.'</p>';
+		}
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionDate($key, $value)
-    {
-        if ($this->isHidden($key)) {
-            return;
-        }
-        $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));
-        }
-        return '<span>'.$date.'</span>';
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionDate($key, $value)
+	{
+		if ($this->isHidden($key)) {
+			return;
+		}
+		$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));
+		}
+		return '<span>'.$date.'</span>';
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionRating($key, $value)
-    {
-        if (!$this->isHiddenOrEmpty($key, $value)) {
-            return glsr_star_rating($value);
-        }
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionRating($key, $value)
+	{
+		if (!$this->isHiddenOrEmpty($key, $value)) {
+			return glsr_star_rating($value);
+		}
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionResponse($key, $value)
-    {
-        if ($this->isHiddenOrEmpty($key, $value)) {
-            return;
-        }
-        $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']);
-        return $response.$background;
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionResponse($key, $value)
+	{
+		if ($this->isHiddenOrEmpty($key, $value)) {
+			return;
+		}
+		$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']);
+		return $response.$background;
+	}
 
-    /**
-     * @param string $key
-     * @param string $value
-     * @return void|string
-     */
-    protected function buildOptionTitle($key, $value)
-    {
-        if ($this->isHidden($key)) {
-            return;
-        }
-        if (empty($value)) {
-            $value = __('No Title', 'site-reviews');
-        }
-        return '<h3>'.$value.'</h3>';
-    }
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return void|string
+	 */
+	protected function buildOptionTitle($key, $value)
+	{
+		if ($this->isHidden($key)) {
+			return;
+		}
+		if (empty($value)) {
+			$value = __('No Title', 'site-reviews');
+		}
+		return '<h3>'.$value.'</h3>';
+	}
 
-    /**
-     * @param string $avatarUrl
-     * @return string
-     */
-    protected function generateAvatar($avatarUrl)
-    {
-        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 = $this->current->email;
-        }
-        if ($newAvatar = get_avatar_url($authorIdOrEmail)) {
-            return $newAvatar;
-        }
-        return $avatarUrl;
-    }
+	/**
+	 * @param string $avatarUrl
+	 * @return string
+	 */
+	protected function generateAvatar($avatarUrl)
+	{
+		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 = $this->current->email;
+		}
+		if ($newAvatar = get_avatar_url($authorIdOrEmail)) {
+			return $newAvatar;
+		}
+		return $avatarUrl;
+	}
 
-    /**
-     * @param string $text
-     * @param int $limit
-     * @return int
-     */
-    protected function getExcerptIntlSplit($text, $limit)
-    {
-        $words = IntlRuleBasedBreakIterator::createWordInstance('');
-        $words->setText($text);
-        $count = 0;
-        foreach ($words as $offset) {
-            if (IntlRuleBasedBreakIterator::WORD_NONE === $words->getRuleStatus()) {
-                continue;
-            }
-            ++$count;
-            if ($count != $limit) {
-                continue;
-            }
-            return $offset;
-        }
-        return strlen($text);
-    }
+	/**
+	 * @param string $text
+	 * @param int $limit
+	 * @return int
+	 */
+	protected function getExcerptIntlSplit($text, $limit)
+	{
+		$words = IntlRuleBasedBreakIterator::createWordInstance('');
+		$words->setText($text);
+		$count = 0;
+		foreach ($words as $offset) {
+			if (IntlRuleBasedBreakIterator::WORD_NONE === $words->getRuleStatus()) {
+				continue;
+			}
+			++$count;
+			if ($count != $limit) {
+				continue;
+			}
+			return $offset;
+		}
+		return strlen($text);
+	}
 
-    /**
-     * @param string $text
-     * @param int $limit
-     * @return int
-     */
-    protected function getExcerptSplit($text, $limit)
-    {
-        if (str_word_count($text, 0) > $limit) {
-            $words = array_keys(str_word_count($text, 2));
-            return $words[$limit];
-        }
-        return strlen($text);
-    }
+	/**
+	 * @param string $text
+	 * @param int $limit
+	 * @return int
+	 */
+	protected function getExcerptSplit($text, $limit)
+	{
+		if (str_word_count($text, 0) > $limit) {
+			$words = array_keys(str_word_count($text, 2));
+			return $words[$limit];
+		}
+		return strlen($text);
+	}
 
-    /**
-     * @param string $path
-     * @param mixed $fallback
-     * @return mixed
-     */
-    protected function getOption($path, $fallback = '')
-    {
-        if (array_key_exists($path, $this->options)) {
-            return $this->options[$path];
-        }
-        return $fallback;
-    }
+	/**
+	 * @param string $path
+	 * @param mixed $fallback
+	 * @return mixed
+	 */
+	protected function getOption($path, $fallback = '')
+	{
+		if (array_key_exists($path, $this->options)) {
+			return $this->options[$path];
+		}
+		return $fallback;
+	}
 
-    /**
-     * @param string $path
-     * @return bool
-     */
-    protected function isOptionEnabled($path)
-    {
-        return 'yes' == $this->getOption($path);
-    }
+	/**
+	 * @param string $path
+	 * @return bool
+	 */
+	protected function isOptionEnabled($path)
+	{
+		return 'yes' == $this->getOption($path);
+	}
 
-    /**
-     * @return void
-     */
-    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)) {
-                return;
-            }
-            $value = glsr(Builder::class)->div($value, [
-                'class' => 'glsr-review-'.$key,
-            ]);
-        });
-    }
+	/**
+	 * @return void
+	 */
+	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)) {
+				return;
+			}
+			$value = glsr(Builder::class)->div($value, [
+				'class' => 'glsr-review-'.$key,
+			]);
+		});
+	}
 }

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

Indentation +176 added lines, -176 removed lines

@@ -12,190 +12,190 @@
 
 class QueryBuilder
 {
-    /**
-     * Build a WP_Query meta_query/tax_query.
-     * @return array
-     */
-    public function buildQuery(array $keys = [], array $values = [])
-    {
-        $queries = [];
-        foreach ($keys as $key) {
-            if (!array_key_exists($key, $values)) {
-                continue;
-            }
-            $methodName = Helper::buildMethodName($key, __FUNCTION__);
-            if (!method_exists($this, $methodName)) {
-                continue;
-            }
-            $query = call_user_func([$this, $methodName], $values[$key]);
-            if (is_array($query)) {
-                $queries[] = $query;
-            }
-        }
-        return $queries;
-    }
+	/**
+	 * Build a WP_Query meta_query/tax_query.
+	 * @return array
+	 */
+	public function buildQuery(array $keys = [], array $values = [])
+	{
+		$queries = [];
+		foreach ($keys as $key) {
+			if (!array_key_exists($key, $values)) {
+				continue;
+			}
+			$methodName = Helper::buildMethodName($key, __FUNCTION__);
+			if (!method_exists($this, $methodName)) {
+				continue;
+			}
+			$query = call_user_func([$this, $methodName], $values[$key]);
+			if (is_array($query)) {
+				$queries[] = $query;
+			}
+		}
+		return $queries;
+	}
 
-    /**
-     * @return string
-     */
-    public function buildSqlLines(array $values, array $conditions)
-    {
-        $string = '';
-        $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]))
-                : $value;
-        }
-        return $string;
-    }
+	/**
+	 * @return string
+	 */
+	public function buildSqlLines(array $values, array $conditions)
+	{
+		$string = '';
+		$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]))
+				: $value;
+		}
+		return $string;
+	}
 
-    /**
-     * Build a SQL 'OR' string from an array.
-     * @param string|array $values
-     * @param string $sprintfFormat
-     * @return string
-     */
-    public function buildSqlOr($values, $sprintfFormat)
-    {
-        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);
-    }
+	/**
+	 * Build a SQL 'OR' string from an array.
+	 * @param string|array $values
+	 * @param string $sprintfFormat
+	 * @return string
+	 */
+	public function buildSqlOr($values, $sprintfFormat)
+	{
+		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);
+	}
 
-    /**
-     * Search SQL filter for matching against post title only.
-     * @see http://wordpress.stackexchange.com/a/11826/1685
-     * @param string $search
-     * @return string
-     * @filter posts_search
-     */
-    public function filterSearchByTitle($search, WP_Query $query)
-    {
-        if (empty($search) || empty($query->get('search_terms'))) {
-            return $search;
-        }
-        global $wpdb;
-        $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);
-        }
-        if (!is_user_logged_in()) {
-            $search[] = "{$wpdb->posts}.post_password = ''";
-        }
-        return ' AND '.implode(' AND ', $search);
-    }
+	/**
+	 * Search SQL filter for matching against post title only.
+	 * @see http://wordpress.stackexchange.com/a/11826/1685
+	 * @param string $search
+	 * @return string
+	 * @filter posts_search
+	 */
+	public function filterSearchByTitle($search, WP_Query $query)
+	{
+		if (empty($search) || empty($query->get('search_terms'))) {
+			return $search;
+		}
+		global $wpdb;
+		$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);
+		}
+		if (!is_user_logged_in()) {
+			$search[] = "{$wpdb->posts}.post_password = ''";
+		}
+		return ' AND '.implode(' AND ', $search);
+	}
 
-    /**
-     * Get the current page number from the global query.
-     * @param bool $isEnabled
-     * @return int
-     */
-    public function getPaged($isEnabled = true)
-    {
-        return $isEnabled
-            ? max(1, intval(filter_input(INPUT_GET, glsr()->constant('PAGED_QUERY_VAR'))))
-            : 1;
-    }
+	/**
+	 * Get the current page number from the global query.
+	 * @param bool $isEnabled
+	 * @return int
+	 */
+	public function getPaged($isEnabled = true)
+	{
+		return $isEnabled
+			? max(1, intval(filter_input(INPUT_GET, glsr()->constant('PAGED_QUERY_VAR'))))
+			: 1;
+	}
 
-    /**
-     * @param string $value
-     * @return void|array
-     */
-    protected function buildQueryAssignedTo($value)
-    {
-        if (!empty($value)) {
-            $postIds = Arr::convertStringToArray($value, 'is_numeric');
-            return [
-                'compare' => 'IN',
-                'key' => '_assigned_to',
-                'value' => glsr(Multilingual::class)->getPostIds($postIds),
-            ];
-        }
-    }
+	/**
+	 * @param string $value
+	 * @return void|array
+	 */
+	protected function buildQueryAssignedTo($value)
+	{
+		if (!empty($value)) {
+			$postIds = Arr::convertStringToArray($value, 'is_numeric');
+			return [
+				'compare' => 'IN',
+				'key' => '_assigned_to',
+				'value' => glsr(Multilingual::class)->getPostIds($postIds),
+			];
+		}
+	}
 
-    /**
-     * @param array $value
-     * @return void|array
-     */
-    protected function buildQueryCategory($value)
-    {
-        if (!empty($value)) {
-            return [
-                'field' => 'term_id',
-                'taxonomy' => Application::TAXONOMY,
-                'terms' => $value,
-            ];
-        }
-    }
+	/**
+	 * @param array $value
+	 * @return void|array
+	 */
+	protected function buildQueryCategory($value)
+	{
+		if (!empty($value)) {
+			return [
+				'field' => 'term_id',
+				'taxonomy' => Application::TAXONOMY,
+				'terms' => $value,
+			];
+		}
+	}
 
-    /**
-     * @param string $value
-     * @return void|array
-     */
-    protected function buildQueryEmail($value)
-    {
-        if (!empty($value)) {
-            return [
-                'key' => '_email',
-                'value' => $value,
-            ];
-        }
-    }
+	/**
+	 * @param string $value
+	 * @return void|array
+	 */
+	protected function buildQueryEmail($value)
+	{
+		if (!empty($value)) {
+			return [
+				'key' => '_email',
+				'value' => $value,
+			];
+		}
+	}
 
-    /**
-     * @param string $value
-     * @return void|array
-     */
-    protected function buildQueryIpAddress($value)
-    {
-        if (!empty($value)) {
-            return [
-                'key' => '_ip_address',
-                'value' => $value,
-            ];
-        }
-    }
+	/**
+	 * @param string $value
+	 * @return void|array
+	 */
+	protected function buildQueryIpAddress($value)
+	{
+		if (!empty($value)) {
+			return [
+				'key' => '_ip_address',
+				'value' => $value,
+			];
+		}
+	}
 
-    /**
-     * @param string $value
-     * @return void|array
-     */
-    protected function buildQueryRating($value)
-    {
-        if (is_numeric($value)
-            && in_array(intval($value), range(1, glsr()->constant('MAX_RATING', Rating::class)))) {
-            return [
-                'compare' => '>=',
-                'key' => '_rating',
-                'value' => $value,
-            ];
-        }
-    }
+	/**
+	 * @param string $value
+	 * @return void|array
+	 */
+	protected function buildQueryRating($value)
+	{
+		if (is_numeric($value)
+			&& in_array(intval($value), range(1, glsr()->constant('MAX_RATING', Rating::class)))) {
+			return [
+				'compare' => '>=',
+				'key' => '_rating',
+				'value' => $value,
+			];
+		}
+	}
 
-    /**
-     * @param string $value
-     * @return void|array
-     */
-    protected function buildQueryType($value)
-    {
-        if (!in_array($value, ['', 'all'])) {
-            return [
-                'key' => '_review_type',
-                'value' => $value,
-            ];
-        }
-    }
+	/**
+	 * @param string $value
+	 * @return void|array
+	 */
+	protected function buildQueryType($value)
+	{
+		if (!in_array($value, ['', 'all'])) {
+			return [
+				'key' => '_review_type',
+				'value' => $value,
+			];
+		}
+	}
 }

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

Indentation +354 added lines, -354 removed lines

@@ -12,385 +12,385 @@
 
 class CountsManager
 {
-    const LIMIT = 500;
-    const META_AVERAGE = '_glsr_average';
-    const META_COUNT = '_glsr_count';
-    const META_RANKING = '_glsr_ranking';
+	const LIMIT = 500;
+	const META_AVERAGE = '_glsr_average';
+	const META_COUNT = '_glsr_count';
+	const META_RANKING = '_glsr_ranking';
 
-    /**
-     * @return array
-     * @todo verify the additional type checks are needed
-     */
-    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])) {
-                    continue;
-            }
-                $counts[$type] = array_fill_keys(range(0, glsr()->constant('MAX_RATING', Rating::class)), 0);
-            }
-            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)
-                : false;
-        }
-        return $counts;
-    }
+	/**
+	 * @return array
+	 * @todo verify the additional type checks are needed
+	 */
+	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])) {
+					continue;
+			}
+				$counts[$type] = array_fill_keys(range(0, glsr()->constant('MAX_RATING', Rating::class)), 0);
+			}
+			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)
+				: false;
+		}
+		return $counts;
+	}
 
-    /**
-     * @param int $postId
-     * @return array
-     */
-    public function buildPostCounts($postId)
-    {
-        return $this->buildCounts(['post_ids' => [$postId]]);
-    }
+	/**
+	 * @param int $postId
+	 * @return array
+	 */
+	public function buildPostCounts($postId)
+	{
+		return $this->buildCounts(['post_ids' => [$postId]]);
+	}
 
-    /**
-     * @param int $termTaxonomyId
-     * @return array
-     */
-    public function buildTermCounts($termTaxonomyId)
-    {
-        return $this->buildCounts(['term_ids' => [$termTaxonomyId]]);
-    }
+	/**
+	 * @param int $termTaxonomyId
+	 * @return array
+	 */
+	public function buildTermCounts($termTaxonomyId)
+	{
+		return $this->buildCounts(['term_ids' => [$termTaxonomyId]]);
+	}
 
-    /**
-     * @return void
-     */
-    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));
-        }
-        $postIds = glsr(SqlQueries::class)->getReviewsMeta('assigned_to');
-        foreach ($postIds as $postId) {
-            $this->setPostCounts($postId, $this->buildPostCounts($postId));
-        }
-        $this->setCounts($this->buildCounts());
-    }
+	/**
+	 * @return void
+	 */
+	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));
+		}
+		$postIds = glsr(SqlQueries::class)->getReviewsMeta('assigned_to');
+		foreach ($postIds as $postId) {
+			$this->setPostCounts($postId, $this->buildPostCounts($postId));
+		}
+		$this->setCounts($this->buildCounts());
+	}
 
-    /**
-     * @return void
-     */
-    public function decrease(Review $review)
-    {
-        $this->decreaseCounts($review);
-        $this->decreasePostCounts($review);
-        $this->decreaseTermCounts($review);
-    }
+	/**
+	 * @return void
+	 */
+	public function decrease(Review $review)
+	{
+		$this->decreaseCounts($review);
+		$this->decreasePostCounts($review);
+		$this->decreaseTermCounts($review);
+	}
 
-    /**
-     * @return void
-     */
-    public function decreaseCounts(Review $review)
-    {
-        $this->setCounts($this->decreaseRating(
-            $this->getCounts(),
-            $review->review_type,
-            $review->rating
-        ));
-    }
+	/**
+	 * @return void
+	 */
+	public function decreaseCounts(Review $review)
+	{
+		$this->setCounts($this->decreaseRating(
+			$this->getCounts(),
+			$review->review_type,
+			$review->rating
+		));
+	}
 
-    /**
-     * @return void
-     */
-    public function decreasePostCounts(Review $review)
-    {
-        if (empty($counts = $this->getPostCounts($review->assigned_to))) {
-            return;
-        }
-        $counts = $this->decreaseRating($counts, $review->review_type, $review->rating);
-        $this->setPostCounts($review->assigned_to, $counts);
-    }
+	/**
+	 * @return void
+	 */
+	public function decreasePostCounts(Review $review)
+	{
+		if (empty($counts = $this->getPostCounts($review->assigned_to))) {
+			return;
+		}
+		$counts = $this->decreaseRating($counts, $review->review_type, $review->rating);
+		$this->setPostCounts($review->assigned_to, $counts);
+	}
 
-    /**
-     * @return void
-     */
-    public function decreaseTermCounts(Review $review)
-    {
-        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);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function decreaseTermCounts(Review $review)
+	{
+		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);
+		}
+	}
 
-    /**
-     * @return array
-     */
-    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));
-        });
-        $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;
-        }
-        return $counts;
-    }
+	/**
+	 * @return array
+	 */
+	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));
+		});
+		$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;
+		}
+		return $counts;
+	}
 
-    /**
-     * @return array
-     */
-    public function get(array $args = [])
-    {
-        $args = $this->normalizeArgs($args);
-        $counts = [];
-        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['term_ids'] as $termId) {
-                $counts[] = $this->getTermCounts($termId);
-            }
-            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 array
+	 */
+	public function get(array $args = [])
+	{
+		$args = $this->normalizeArgs($args);
+		$counts = [];
+		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['term_ids'] as $termId) {
+				$counts[] = $this->getTermCounts($termId);
+			}
+			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 array
-     */
-    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);
-            return [];
-        }
-        return $counts;
-    }
+	/**
+	 * @return array
+	 */
+	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);
+			return [];
+		}
+		return $counts;
+	}
 
-    /**
-     * @param int $postId
-     * @return array
-     */
-    public function getPostCounts($postId)
-    {
-        return array_filter((array) get_post_meta($postId, static::META_COUNT, true));
-    }
+	/**
+	 * @param int $postId
+	 * @return array
+	 */
+	public function getPostCounts($postId)
+	{
+		return array_filter((array) get_post_meta($postId, static::META_COUNT, true));
+	}
 
-    /**
-     * @param int $termId
-     * @return array
-     */
-    public function getTermCounts($termId)
-    {
-        return array_filter((array) get_term_meta($termId, static::META_COUNT, true));
-    }
+	/**
+	 * @param int $termId
+	 * @return array
+	 */
+	public function getTermCounts($termId)
+	{
+		return array_filter((array) get_term_meta($termId, static::META_COUNT, true));
+	}
 
-    /**
-     * @return void
-     */
-    public function increase(Review $review)
-    {
-        $this->increaseCounts($review);
-        $this->increasePostCounts($review);
-        $this->increaseTermCounts($review);
-    }
+	/**
+	 * @return void
+	 */
+	public function increase(Review $review)
+	{
+		$this->increaseCounts($review);
+		$this->increasePostCounts($review);
+		$this->increaseTermCounts($review);
+	}
 
-    /**
-     * @return void
-     */
-    public function increaseCounts(Review $review)
-    {
-        if (empty($counts = $this->getCounts())) {
-            $counts = $this->buildCounts();
-        }
-        $this->setCounts($this->increaseRating($counts, $review->review_type, $review->rating));
-    }
+	/**
+	 * @return void
+	 */
+	public function increaseCounts(Review $review)
+	{
+		if (empty($counts = $this->getCounts())) {
+			$counts = $this->buildCounts();
+		}
+		$this->setCounts($this->increaseRating($counts, $review->review_type, $review->rating));
+	}
 
-    /**
-     * @return void
-     */
-    public function increasePostCounts(Review $review)
-    {
-        if (!(get_post($review->assigned_to) instanceof WP_Post)) {
-            return;
-        }
-        $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);
-    }
+	/**
+	 * @return void
+	 */
+	public function increasePostCounts(Review $review)
+	{
+		if (!(get_post($review->assigned_to) instanceof WP_Post)) {
+			return;
+		}
+		$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);
+	}
 
-    /**
-     * @return void
-     */
-    public function increaseTermCounts(Review $review)
-    {
-        $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);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function increaseTermCounts(Review $review)
+	{
+		$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);
+		}
+	}
 
-    /**
-     * @return void
-     */
-    public function setCounts(array $reviewCounts)
-    {
-        glsr(OptionManager::class)->set('counts', $reviewCounts);
-    }
+	/**
+	 * @return void
+	 */
+	public function setCounts(array $reviewCounts)
+	{
+		glsr(OptionManager::class)->set('counts', $reviewCounts);
+	}
 
-    /**
-     * @param int $postId
-     * @return void
-     */
-    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));
-    }
+	/**
+	 * @param int $postId
+	 * @return void
+	 */
+	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));
+	}
 
-    /**
-     * @param int $termId
-     * @return void
-     */
-    public function setTermCounts($termId, array $reviewCounts)
-    {
-        $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));
-    }
+	/**
+	 * @param int $termId
+	 * @return void
+	 */
+	public function setTermCounts($termId, array $reviewCounts)
+	{
+		$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));
+	}
 
-    /**
-     * @param string $type
-     * @param int $rating
-     * @return array
-     */
-    protected function decreaseRating(array $reviewCounts, $type, $rating)
-    {
-        if (isset($reviewCounts[$type][$rating])) {
-            $reviewCounts[$type][$rating] = max(0, $reviewCounts[$type][$rating] - 1);
-        }
-        return $reviewCounts;
-    }
+	/**
+	 * @param string $type
+	 * @param int $rating
+	 * @return array
+	 */
+	protected function decreaseRating(array $reviewCounts, $type, $rating)
+	{
+		if (isset($reviewCounts[$type][$rating])) {
+			$reviewCounts[$type][$rating] = max(0, $reviewCounts[$type][$rating] - 1);
+		}
+		return $reviewCounts;
+	}
 
-    /**
-     * @param string $type
-     * @param int $rating
-     * @return array
-     */
-    protected function increaseRating(array $reviewCounts, $type, $rating)
-    {
-        if (!array_key_exists($type, glsr()->reviewTypes)) {
-            return $reviewCounts;
-        }
-        if (!array_key_exists($type, $reviewCounts)) {
-            $reviewCounts[$type] = [];
-        }
-        $reviewCounts = $this->normalize($reviewCounts);
-        $reviewCounts[$type][$rating] = intval($reviewCounts[$type][$rating]) + 1;
-        return $reviewCounts;
-    }
+	/**
+	 * @param string $type
+	 * @param int $rating
+	 * @return array
+	 */
+	protected function increaseRating(array $reviewCounts, $type, $rating)
+	{
+		if (!array_key_exists($type, glsr()->reviewTypes)) {
+			return $reviewCounts;
+		}
+		if (!array_key_exists($type, $reviewCounts)) {
+			$reviewCounts[$type] = [];
+		}
+		$reviewCounts = $this->normalize($reviewCounts);
+		$reviewCounts[$type][$rating] = intval($reviewCounts[$type][$rating]) + 1;
+		return $reviewCounts;
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isMixedCount(array $args)
-    {
-        return !empty($args['post_ids']) && !empty($args['term_ids']);
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isMixedCount(array $args)
+	{
+		return !empty($args['post_ids']) && !empty($args['term_ids']);
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalize(array $reviewCounts)
-    {
-        if (empty($reviewCounts)) {
-            $reviewCounts = [[]];
-        }
-        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);
-        }
-        return $reviewCounts;
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalize(array $reviewCounts)
+	{
+		if (empty($reviewCounts)) {
+			$reviewCounts = [[]];
+		}
+		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);
+		}
+		return $reviewCounts;
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalizeArgs(array $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']);
-        return $args;
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalizeArgs(array $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']);
+		return $args;
+	}
 
-    /**
-     * @param string $type
-     * @return string
-     */
-    protected function normalizeType($type)
-    {
-        return empty($type) || !is_string($type)
-            ? 'local'
-            : $type;
-    }
+	/**
+	 * @param string $type
+	 * @return string
+	 */
+	protected function normalizeType($type)
+	{
+		return empty($type) || !is_string($type)
+			? 'local'
+			: $type;
+	}
 
-    /**
-     * @param int $lastPostId
-     * @return object
-     */
-    protected function queryReviews(array $args = [], $lastPostId = 0)
-    {
-        $reviews = glsr(SqlQueries::class)->getReviewCounts($args, $lastPostId, static::LIMIT);
-        $hasMore = is_array($reviews)
-            ? count($reviews) == static::LIMIT
-            : false;
-        return (object) [
-            'has_more' => $hasMore,
-            'reviews' => $reviews,
-        ];
-    }
+	/**
+	 * @param int $lastPostId
+	 * @return object
+	 */
+	protected function queryReviews(array $args = [], $lastPostId = 0)
+	{
+		$reviews = glsr(SqlQueries::class)->getReviewCounts($args, $lastPostId, static::LIMIT);
+		$hasMore = is_array($reviews)
+			? count($reviews) == static::LIMIT
+			: false;
+		return (object) [
+			'has_more' => $hasMore,
+			'reviews' => $reviews,
+		];
+	}
 }

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

Indentation +112 added lines, -112 removed lines

@@ -9,121 +9,121 @@
 
 class SettingsController extends Controller
 {
-    /**
-     * @param mixed $input
-     * @return array
-     * @callback register_setting
-     */
-    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'));
-            }
-            return $options;
-        }
-        return $input;
-    }
+	/**
+	 * @param mixed $input
+	 * @return array
+	 * @callback register_setting
+	 */
+	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'));
+			}
+			return $options;
+		}
+		return $input;
+	}
 
-    /**
-     * @return void
-     * @action admin_init
-     */
-    public function registerSettings()
-    {
-        register_setting(Application::ID.'-settings', OptionManager::databaseKey(), [
-            'sanitize_callback' => [$this, 'callbackRegisterSettings'],
-        ]);
-    }
+	/**
+	 * @return void
+	 * @action admin_init
+	 */
+	public function registerSettings()
+	{
+		register_setting(Application::ID.'-settings', OptionManager::databaseKey(), [
+			'sanitize_callback' => [$this, 'callbackRegisterSettings'],
+		]);
+	}
 
-    /**
-     * @return array
-     */
-    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', '');
-        }
-        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);
-        return $options;
-    }
+	/**
+	 * @return array
+	 */
+	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', '');
+		}
+		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);
+		return $options;
+	}
 
-    /**
-     * @return array
-     */
-    protected function sanitizeSubmissions(array $input, array $options)
-    {
-        $key = 'settings.submissions';
-        $inputForm = Arr::get($input, $key);
-        $defaultValue = isset($inputForm['required'])
-            ? $inputForm['required']
-            : [];
-        $options = Arr::set($options, $key.'.required', $defaultValue);
-        return $options;
-    }
+	/**
+	 * @return array
+	 */
+	protected function sanitizeSubmissions(array $input, array $options)
+	{
+		$key = 'settings.submissions';
+		$inputForm = Arr::get($input, $key);
+		$defaultValue = isset($inputForm['required'])
+			? $inputForm['required']
+			: [];
+		$options = Arr::set($options, $key.'.required', $defaultValue);
+		return $options;
+	}
 
-    /**
-     * @return array
-     */
-    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)));
-            $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);
-                }
-                if (isset($string['p2'])) {
-                    $string['p2'] = wp_kses($string['p2'], $allowedTags);
-                }
-            });
-        }
-        return $options;
-    }
+	/**
+	 * @return array
+	 */
+	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)));
+			$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);
+				}
+				if (isset($string['p2'])) {
+					$string['p2'] = wp_kses($string['p2'], $allowedTags);
+				}
+			});
+		}
+		return $options;
+	}
 
-    /**
-     * @param string $integrationSlug
-     * @return bool
-     */
-    protected function hasMultilingualIntegration($integrationSlug)
-    {
-        $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'),
-                $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'),
-                $integration->pluginName,
-                $integration->supportedVersion
-            ));
-            return false;
-        }
-        return true;
-    }
+	/**
+	 * @param string $integrationSlug
+	 * @return bool
+	 */
+	protected function hasMultilingualIntegration($integrationSlug)
+	{
+		$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'),
+				$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'),
+				$integration->pluginName,
+				$integration->supportedVersion
+			));
+			return false;
+		}
+		return true;
+	}
 }

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;

Braces +2 added lines, -1 removed lines

@@ -116,7 +116,8 @@
                 $integration->pluginName
             ));
             return false;
-        } elseif (!$integration->isSupported()) {
+        }
+        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,

plugin/Helpers/Arr.php

Indentation +180 added lines, -180 removed lines

@@ -4,197 +4,197 @@
 
 class Arr
 {
-    /**
-     * @return bool
-     */
-    public static function compareArrays(array $arr1, array $arr2)
-    {
-        sort($arr1);
-        sort($arr2);
-        return $arr1 == $arr2;
-    }
+	/**
+	 * @return bool
+	 */
+	public static function compareArrays(array $arr1, array $arr2)
+	{
+		sort($arr1);
+		sort($arr2);
+		return $arr1 == $arr2;
+	}
 
-    /**
-     * @param mixed $array
-     * @return array
-     */
-    public static function consolidateArray($array)
-    {
-        return is_array($array) || is_object($array)
-            ? (array) $array
-            : [];
-    }
+	/**
+	 * @param mixed $array
+	 * @return array
+	 */
+	public static function consolidateArray($array)
+	{
+		return is_array($array) || is_object($array)
+			? (array) $array
+			: [];
+	}
 
-    /**
-     * @return array
-     */
-    public static function convertDotNotationArray(array $array)
-    {
-        $results = [];
-        foreach ($array as $path => $value) {
-            $results = static::set($results, $path, $value);
-        }
-        return $results;
-    }
+	/**
+	 * @return array
+	 */
+	public static function convertDotNotationArray(array $array)
+	{
+		$results = [];
+		foreach ($array as $path => $value) {
+			$results = static::set($results, $path, $value);
+		}
+		return $results;
+	}
 
-    /**
-     * @param string $string
-     * @param mixed $callback
-     * @return array
-     */
-    public static function convertStringToArray($string, $callback = null)
-    {
-        $array = array_map('trim', explode(',', $string));
-        return $callback
-            ? array_filter($array, $callback)
-            : array_filter($array);
-    }
+	/**
+	 * @param string $string
+	 * @param mixed $callback
+	 * @return array
+	 */
+	public static function convertStringToArray($string, $callback = null)
+	{
+		$array = array_map('trim', explode(',', $string));
+		return $callback
+			? array_filter($array, $callback)
+			: array_filter($array);
+	}
 
-    /**
-     * @param bool $flattenValue
-     * @param string $prefix
-     * @return array
-     */
-    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).']';
-                }
-            } elseif (is_array($value)) {
-                $result = array_merge($result, static::flattenArray($value, $flattenValue, $newKey));
-                continue;
-            }
-            $result[$newKey] = $value;
-        }
-        return $result;
-    }
+	/**
+	 * @param bool $flattenValue
+	 * @param string $prefix
+	 * @return array
+	 */
+	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).']';
+				}
+			} elseif (is_array($value)) {
+				$result = array_merge($result, static::flattenArray($value, $flattenValue, $newKey));
+				continue;
+			}
+			$result[$newKey] = $value;
+		}
+		return $result;
+	}
 
-    /**
-     * Get a value from an array of values using a dot-notation path as reference.
-     * @param mixed $data
-     * @param string $path
-     * @param mixed $fallback
-     * @return mixed
-     */
-    public static function get($data, $path = '', $fallback = '')
-    {
-        $data = static::consolidateArray($data);
-        $keys = explode('.', $path);
-        foreach ($keys as $key) {
-            if (!isset($data[$key])) {
-                return $fallback;
-            }
-            $data = $data[$key];
-        }
-        return $data;
-    }
+	/**
+	 * Get a value from an array of values using a dot-notation path as reference.
+	 * @param mixed $data
+	 * @param string $path
+	 * @param mixed $fallback
+	 * @return mixed
+	 */
+	public static function get($data, $path = '', $fallback = '')
+	{
+		$data = static::consolidateArray($data);
+		$keys = explode('.', $path);
+		foreach ($keys as $key) {
+			if (!isset($data[$key])) {
+				return $fallback;
+			}
+			$data = $data[$key];
+		}
+		return $data;
+	}
 
-    /**
-     * @param string $key
-     * @param string $position
-     * @return array
-     */
-    public static function insertInArray(array $array, array $insert, $key, $position = 'before')
-    {
-        $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));
-        }
-        return array_merge($array, $insert);
-    }
+	/**
+	 * @param string $key
+	 * @param string $position
+	 * @return array
+	 */
+	public static function insertInArray(array $array, array $insert, $key, $position = 'before')
+	{
+		$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));
+		}
+		return array_merge($array, $insert);
+	}
 
-    /**
-     * @param mixed $array
-     * @return bool
-     */
-    public static function isIndexedFlatArray($array)
-    {
-        if (!is_array($array) || array_filter($array, 'is_array')) {
-            return false;
-        }
-        return wp_is_numeric_array($array);
-    }
+	/**
+	 * @param mixed $array
+	 * @return bool
+	 */
+	public static function isIndexedFlatArray($array)
+	{
+		if (!is_array($array) || array_filter($array, 'is_array')) {
+			return false;
+		}
+		return wp_is_numeric_array($array);
+	}
 
-    /**
-     * @param bool $prefixed
-     * @return array
-     */
-    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));
-            }
-            $prefixed[$prefix.$key] = $value;
-        }
-        return $prefixed;
-    }
+	/**
+	 * @param bool $prefixed
+	 * @return array
+	 */
+	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));
+			}
+			$prefixed[$prefix.$key] = $value;
+		}
+		return $prefixed;
+	}
 
-    /**
-     * @return array
-     */
-    public static function removeEmptyArrayValues(array $array)
-    {
-        $result = [];
-        foreach ($array as $key => $value) {
-            if (!$value) {
-                continue;
-            }
-            $result[$key] = is_array($value)
-                ? static::removeEmptyArrayValues($value)
-                : $value;
-        }
-        return $result;
-    }
+	/**
+	 * @return array
+	 */
+	public static function removeEmptyArrayValues(array $array)
+	{
+		$result = [];
+		foreach ($array as $key => $value) {
+			if (!$value) {
+				continue;
+			}
+			$result[$key] = is_array($value)
+				? static::removeEmptyArrayValues($value)
+				: $value;
+		}
+		return $result;
+	}
 
 
-    /**
-     * Set a value to an array of values using a dot-notation path as reference.
-     * @param string $path
-     * @param mixed $value
-     * @return array
-     */
-    public static function set(array $data, $path, $value)
-    {
-        $token = strtok($path, '.');
-        $ref = &$data;
-        while (false !== $token) {
-            $ref = static::consolidateArray($ref);
-            $ref = &$ref[$token];
-            $token = strtok('.');
-        }
-        $ref = $value;
-        return $data;
-    }
+	/**
+	 * Set a value to an array of values using a dot-notation path as reference.
+	 * @param string $path
+	 * @param mixed $value
+	 * @return array
+	 */
+	public static function set(array $data, $path, $value)
+	{
+		$token = strtok($path, '.');
+		$ref = &$data;
+		while (false !== $token) {
+			$ref = static::consolidateArray($ref);
+			$ref = &$ref[$token];
+			$token = strtok('.');
+		}
+		$ref = $value;
+		return $data;
+	}
 
-    /**
-     * @return array
-     */
-    public static function unique(array $values)
-    {
-        return array_filter(array_unique($values));
-    }
+	/**
+	 * @return array
+	 */
+	public static function unique(array $values)
+	{
+		return array_filter(array_unique($values));
+	}
 
-    /**
-     * @return array
-     */
-    public static function unprefixArrayKeys(array $values)
-    {
-        return static::prefixArrayKeys($values, false);
-    }
+	/**
+	 * @return array
+	 */
+	public static function unprefixArrayKeys(array $values)
+	{
+		return static::prefixArrayKeys($values, false);
+	}
 }

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

Indentation +43 added lines, -43 removed lines

@@ -14,50 +14,50 @@
  */
 class Multilingual
 {
-    protected $integration;
+	protected $integration;
 
-    /**
-     * @param string $method
-     * @param array $args
-     * @return 
-     */
-    public function __call($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);
-    }
+	/**
+	 * @param string $method
+	 * @param array $args
+	 * @return 
+	 */
+	public function __call($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);
+	}
 
-    /**
-     * @param string $integration
-     * @return false|\GeminiLabs\SiteReviews\Modules\Multilingual\Polylang|\GeminiLabs\SiteReviews\Modules\Multilingual\Wpml
-     */
-    public function getIntegration($integration = '')
-    {
-        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);
-        }
-        glsr_log()->error($integrationClass.' does not exist');
-        return false;
-    }
+	/**
+	 * @param string $integration
+	 * @return false|\GeminiLabs\SiteReviews\Modules\Multilingual\Polylang|\GeminiLabs\SiteReviews\Modules\Multilingual\Wpml
+	 */
+	public function getIntegration($integration = '')
+	{
+		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);
+		}
+		glsr_log()->error($integrationClass.' does not exist');
+		return false;
+	}
 
-    /**
-     * return bool
-     */
-    public function isIntegrated()
-    {
-        if (!empty($this->integration)) {
-            return true;
-        }
-        if ($integration = $this->getIntegration()) {
-            $this->integration = $integration;
-            return true;
-        }
-        return false;
-    }
+	/**
+	 * return bool
+	 */
+	public function isIntegrated()
+	{
+		if (!empty($this->integration)) {
+			return true;
+		}
+		if ($integration = $this->getIntegration()) {
+			$this->integration = $integration;
+			return true;
+		}
+		return false;
+	}
 }

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

Indentation +66 added lines, -66 removed lines

@@ -8,75 +8,75 @@
 
 class Wpml implements Contract
 {
-    public $pluginName = 'WPML';
-    public $supportedVersion = '3.3.5';
+	public $pluginName = 'WPML';
+	public $supportedVersion = '3.3.5';
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPost($postId)
-    {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
-            return;
-        }
-        if ($this->isEnabled()) {
-            $postId = apply_filters('wpml_object_id', $postId, 'any', true);
-        }
-        return get_post(intval($postId));
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPost($postId)
+	{
+		$postId = trim($postId);
+		if (!is_numeric($postId)) {
+			return;
+		}
+		if ($this->isEnabled()) {
+			$postId = apply_filters('wpml_object_id', $postId, 'any', true);
+		}
+		return get_post(intval($postId));
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPostIds(array $postIds)
-    {
-        if (!$this->isEnabled()) {
-            return $postIds;
-        }
-        $newPostIds = [];
-        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)) {
-                $translations = [];
-            }
-            $newPostIds = array_merge(
-                $newPostIds,
-                array_column($translations, 'element_id')
-            );
-        }
-        return Arr::unique($newPostIds);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPostIds(array $postIds)
+	{
+		if (!$this->isEnabled()) {
+			return $postIds;
+		}
+		$newPostIds = [];
+		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)) {
+				$translations = [];
+			}
+			$newPostIds = array_merge(
+				$newPostIds,
+				array_column($translations, 'element_id')
+			);
+		}
+		return Arr::unique($newPostIds);
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isActive()
-    {
-        return defined('ICL_SITEPRESS_VERSION');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isActive()
+	{
+		return defined('ICL_SITEPRESS_VERSION');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isEnabled()
-    {
-        return $this->isActive()
-            && 'wpml' == glsr(OptionManager::class)->get('settings.general.multilingual');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isEnabled()
+	{
+		return $this->isActive()
+			&& 'wpml' == glsr(OptionManager::class)->get('settings.general.multilingual');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isSupported()
-    {
-        return $this->isActive()
-            && version_compare(ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isSupported()
+	{
+		return $this->isActive()
+			&& version_compare(ICL_SITEPRESS_VERSION, $this->supportedVersion, '>=');
+	}
 }

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

Indentation +62 added lines, -62 removed lines

@@ -8,71 +8,71 @@
 
 class Polylang implements Contract
 {
-    public $pluginName = 'Polylang';
-    public $supportedVersion = '2.3';
+	public $pluginName = 'Polylang';
+	public $supportedVersion = '2.3';
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPost($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 (!empty($polylangPostId)) {
-            $postId = $polylangPostId;
-        }
-        return get_post(intval($postId));
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPost($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 (!empty($polylangPostId)) {
+			$postId = $polylangPostId;
+		}
+		return get_post(intval($postId));
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function getPostIds(array $postIds)
-    {
-        if (!$this->isEnabled()) {
-            return $postIds;
-        }
-        $newPostIds = [];
-        foreach (Arr::unique($postIds) as $postId) {
-            $newPostIds = array_merge(
-                $newPostIds,
-                array_values(pll_get_post_translations($postId))
-            );
-        }
-        return Arr::unique($newPostIds);
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function getPostIds(array $postIds)
+	{
+		if (!$this->isEnabled()) {
+			return $postIds;
+		}
+		$newPostIds = [];
+		foreach (Arr::unique($postIds) as $postId) {
+			$newPostIds = array_merge(
+				$newPostIds,
+				array_values(pll_get_post_translations($postId))
+			);
+		}
+		return Arr::unique($newPostIds);
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isActive()
-    {
-        return function_exists('PLL')
-            && function_exists('pll_get_post')
-            && function_exists('pll_get_post_language')
-            && function_exists('pll_get_post_translations');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isActive()
+	{
+		return function_exists('PLL')
+			&& function_exists('pll_get_post')
+			&& function_exists('pll_get_post_language')
+			&& function_exists('pll_get_post_translations');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isEnabled()
-    {
-        return $this->isActive()
-            && 'polylang' == glsr(OptionManager::class)->get('settings.general.multilingual');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isEnabled()
+	{
+		return $this->isActive()
+			&& 'polylang' == glsr(OptionManager::class)->get('settings.general.multilingual');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function isSupported()
-    {
-        return defined('POLYLANG_VERSION')
-            && version_compare(POLYLANG_VERSION, $this->supportedVersion, '>=');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function isSupported()
+	{
+		return defined('POLYLANG_VERSION')
+			&& version_compare(POLYLANG_VERSION, $this->supportedVersion, '>=');
+	}
 }

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, '>=' );
     }
 }