pryley / site-reviews

plugin/Database/SqlQueries.php

Indentation +120 added lines, -120 removed lines

@@ -8,47 +8,47 @@ 
 
 class SqlQueries
 {
-    protected $db;
-    protected $postType;
+	protected $db;
+	protected $postType;
 
-    public function __construct()
-    {
-        global $wpdb;
-        $this->db = $wpdb;
-        $this->postType = Application::POST_TYPE;
-    }
+	public function __construct()
+	{
+		global $wpdb;
+		$this->db = $wpdb;
+		$this->postType = Application::POST_TYPE;
+	}
 
-    /**
-     * @return bool
-     */
-    public function deletePostCountMetaKeys()
-    {
-        $this->db->query("
+	/**
+	 * @return bool
+	 */
+	public function deletePostCountMetaKeys()
+	{
+		$this->db->query("
             DELETE
             FROM {$this->db->postmeta}
             WHERE meta_key LIKE '_glsr_%'
         ");
-    }
+	}
 
-    /**
-     * @return bool
-     */
-    public function deleteTermCountMetaKeys()
-    {
-        $this->db->query("
+	/**
+	 * @return bool
+	 */
+	public function deleteTermCountMetaKeys()
+	{
+		$this->db->query("
             DELETE
             FROM {$this->db->termmeta}
             WHERE meta_key LIKE '_glsr_%'
         ");
-    }
+	}
 
-    /**
-     * @param string $metaReviewId
-     * @return int
-     */
-    public function getPostIdFromReviewId($metaReviewId)
-    {
-        $postId = $this->db->get_var("
+	/**
+	 * @param string $metaReviewId
+	 * @return int
+	 */
+	public function getPostIdFromReviewId($metaReviewId)
+	{
+		$postId = $this->db->get_var("
             SELECT p.ID
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -56,17 +56,17 @@ 
             AND m.meta_key = '_review_id'
             AND m.meta_value = '{$metaReviewId}'
         ");
-        return intval($postId);
-    }
+		return intval($postId);
+	}
 
-    /**
-     * @param int $lastPostId
-     * @param int $limit
-     * @return array
-     */
-    public function getReviewCounts(array $args, $lastPostId = 0, $limit = 500)
-    {
-        return (array) $this->db->get_results("
+	/**
+	 * @param int $lastPostId
+	 * @param int $limit
+	 * @return array
+	 */
+	public function getReviewCounts(array $args, $lastPostId = 0, $limit = 500)
+	{
+		return (array) $this->db->get_results("
             SELECT DISTINCT p.ID, m1.meta_value AS rating, m2.meta_value AS type
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -81,17 +81,17 @@ 
             ORDER By p.ID ASC
             LIMIT {$limit}
         ");
-    }
+	}
 
-    /**
-     * @todo remove this?
-     * @param string $metaKey
-     * @return array
-     */
-    public function getReviewCountsFor($metaKey)
-    {
-        $metaKey = Str::prefix('_', $metaKey);
-        return (array) $this->db->get_results("
+	/**
+	 * @todo remove this?
+	 * @param string $metaKey
+	 * @return array
+	 */
+	public function getReviewCountsFor($metaKey)
+	{
+		$metaKey = Str::prefix('_', $metaKey);
+		return (array) $this->db->get_results("
             SELECT DISTINCT m.meta_value AS name, COUNT(*) num_posts
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -99,16 +99,16 @@ 
             AND m.meta_key = '{$metaKey}'
             GROUP BY name
         ");
-    }
+	}
 
-    /**
-     * @todo remove this?
-     * @param string $reviewType
-     * @return array
-     */
-    public function getReviewIdsByType($reviewType)
-    {
-        $results = $this->db->get_col("
+	/**
+	 * @todo remove this?
+	 * @param string $reviewType
+	 * @return array
+	 */
+	public function getReviewIdsByType($reviewType)
+	{
+		$results = $this->db->get_col("
             SELECT DISTINCT m1.meta_value AS review_id
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -118,20 +118,20 @@ 
             AND m2.meta_key = '_review_type'
             AND m2.meta_value = '{$reviewType}'
         ");
-        return array_keys(array_flip($results));
-    }
+		return array_keys(array_flip($results));
+	}
 
-    /**
-     * @param int $greaterThanId
-     * @param int $limit
-     * @return array
-     */
-    public function getReviewRatingsFromIds(array $postIds, $greaterThanId = 0, $limit = 100)
-    {
-        sort($postIds);
-        $postIds = array_slice($postIds, intval(array_search($greaterThanId, $postIds)), $limit);
-        $postIds = implode(',', $postIds);
-        return (array) $this->db->get_results("
+	/**
+	 * @param int $greaterThanId
+	 * @param int $limit
+	 * @return array
+	 */
+	public function getReviewRatingsFromIds(array $postIds, $greaterThanId = 0, $limit = 100)
+	{
+		sort($postIds);
+		$postIds = array_slice($postIds, intval(array_search($greaterThanId, $postIds)), $limit);
+		$postIds = implode(',', $postIds);
+		return (array) $this->db->get_results("
             SELECT p.ID, m.meta_value AS rating
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -144,20 +144,20 @@ 
             ORDER By p.ID ASC
             LIMIT {$limit}
         ");
-    }
+	}
 
-    /**
-     * @param string $key
-     * @param string $status
-     * @return array
-     */
-    public function getReviewsMeta($key, $status = 'publish')
-    {
-        $postStatusQuery = 'all' != $status && !empty($status)
-            ? "AND p.post_status = '{$status}'"
-            : '';
-        $key = Str::prefix('_', $key);
-        $values = $this->db->get_col("
+	/**
+	 * @param string $key
+	 * @param string $status
+	 * @return array
+	 */
+	public function getReviewsMeta($key, $status = 'publish')
+	{
+		$postStatusQuery = 'all' != $status && !empty($status)
+			? "AND p.post_status = '{$status}'"
+			: '';
+		$key = Str::prefix('_', $key);
+		$values = $this->db->get_col("
             SELECT DISTINCT m.meta_value
             FROM {$this->db->postmeta} m
             LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id
@@ -168,42 +168,42 @@ 
             GROUP BY p.ID -- remove duplicate meta_value entries
             ORDER BY m.meta_id ASC -- sort by oldest meta_value
         ");
-        sort($values);
-        return $values;
-    }
+		sort($values);
+		return $values;
+	}
 
-    /**
-     * @param string $and
-     * @return string
-     */
-    protected function getAndForCounts(array $args, $and = '')
-    {
-        $postIds = implode(',', array_filter(Arr::get($args, 'post_ids', [])));
-        $termIds = implode(',', array_filter(Arr::get($args, 'term_ids', [])));
-        if (!empty($args['type'])) {
-            $and.= "AND m2.meta_value = '{$args['type']}' ";
-        }
-        if ($postIds) {
-            $and.= "AND m3.meta_key = '_assigned_to' AND m3.meta_value IN ({$postIds}) ";
-        }
-        if ($termIds) {
-            $and.= "AND tr.term_taxonomy_id IN ({$termIds}) ";
-        }
-        return apply_filters('site-reviews/query/and-for-counts', $and);
-    }
+	/**
+	 * @param string $and
+	 * @return string
+	 */
+	protected function getAndForCounts(array $args, $and = '')
+	{
+		$postIds = implode(',', array_filter(Arr::get($args, 'post_ids', [])));
+		$termIds = implode(',', array_filter(Arr::get($args, 'term_ids', [])));
+		if (!empty($args['type'])) {
+			$and.= "AND m2.meta_value = '{$args['type']}' ";
+		}
+		if ($postIds) {
+			$and.= "AND m3.meta_key = '_assigned_to' AND m3.meta_value IN ({$postIds}) ";
+		}
+		if ($termIds) {
+			$and.= "AND tr.term_taxonomy_id IN ({$termIds}) ";
+		}
+		return apply_filters('site-reviews/query/and-for-counts', $and);
+	}
 
-    /**
-     * @param string $innerJoin
-     * @return string
-     */
-    protected function getInnerJoinForCounts(array $args, $innerJoin = '')
-    {
-        if (!empty(Arr::get($args, 'post_ids'))) {
-            $innerJoin.= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id ";
-        }
-        if (!empty(Arr::get($args, 'term_ids'))) {
-            $innerJoin.= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id ";
-        }
-        return apply_filters('site-reviews/query/inner-join-for-counts', $innerJoin);
-    }
+	/**
+	 * @param string $innerJoin
+	 * @return string
+	 */
+	protected function getInnerJoinForCounts(array $args, $innerJoin = '')
+	{
+		if (!empty(Arr::get($args, 'post_ids'))) {
+			$innerJoin.= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id ";
+		}
+		if (!empty(Arr::get($args, 'term_ids'))) {
+			$innerJoin.= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id ";
+		}
+		return apply_filters('site-reviews/query/inner-join-for-counts', $innerJoin);
+	}
 }

Spacing +48 added lines, -48 removed lines

@@ -23,11 +23,11 @@ 
      */
     public function deletePostCountMetaKeys()
     {
-        $this->db->query("
+        $this->db->query( "
             DELETE
             FROM {$this->db->postmeta}
             WHERE meta_key LIKE '_glsr_%'
-        ");
+        " );
     }
 
     /**
@@ -35,28 +35,28 @@ 
      */
     public function deleteTermCountMetaKeys()
     {
-        $this->db->query("
+        $this->db->query( "
             DELETE
             FROM {$this->db->termmeta}
             WHERE meta_key LIKE '_glsr_%'
-        ");
+        " );
     }
 
     /**
      * @param string $metaReviewId
      * @return int
      */
-    public function getPostIdFromReviewId($metaReviewId)
+    public function getPostIdFromReviewId( $metaReviewId )
     {
-        $postId = $this->db->get_var("
+        $postId = $this->db->get_var( "
             SELECT p.ID
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
             WHERE p.post_type = '{$this->postType}'
             AND m.meta_key = '_review_id'
             AND m.meta_value = '{$metaReviewId}'
-        ");
-        return intval($postId);
+        " );
+        return intval( $postId );
     }
 
     /**
@@ -64,23 +64,23 @@ 
      * @param int $limit
      * @return array
      */
-    public function getReviewCounts(array $args, $lastPostId = 0, $limit = 500)
+    public function getReviewCounts( array $args, $lastPostId = 0, $limit = 500 )
     {
-        return (array) $this->db->get_results("
+        return (array)$this->db->get_results( "
             SELECT DISTINCT p.ID, m1.meta_value AS rating, m2.meta_value AS type
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
             INNER JOIN {$this->db->postmeta} AS m2 ON p.ID = m2.post_id
-            {$this->getInnerJoinForCounts($args)}
+            {$this->getInnerJoinForCounts( $args )}
             WHERE p.ID > {$lastPostId}
             AND p.post_status = 'publish'
             AND p.post_type = '{$this->postType}'
             AND m1.meta_key = '_rating'
             AND m2.meta_key = '_review_type'
-            {$this->getAndForCounts($args)}
+            {$this->getAndForCounts( $args )}
             ORDER By p.ID ASC
             LIMIT {$limit}
-        ");
+        " );
     }
 
     /**
@@ -88,17 +88,17 @@ 
      * @param string $metaKey
      * @return array
      */
-    public function getReviewCountsFor($metaKey)
+    public function getReviewCountsFor( $metaKey )
     {
-        $metaKey = Str::prefix('_', $metaKey);
-        return (array) $this->db->get_results("
+        $metaKey = Str::prefix( '_', $metaKey );
+        return (array)$this->db->get_results( "
             SELECT DISTINCT m.meta_value AS name, COUNT(*) num_posts
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
             WHERE p.post_type = '{$this->postType}'
             AND m.meta_key = '{$metaKey}'
             GROUP BY name
-        ");
+        " );
     }
 
     /**
@@ -106,9 +106,9 @@ 
      * @param string $reviewType
      * @return array
      */
-    public function getReviewIdsByType($reviewType)
+    public function getReviewIdsByType( $reviewType )
     {
-        $results = $this->db->get_col("
+        $results = $this->db->get_col( "
             SELECT DISTINCT m1.meta_value AS review_id
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m1 ON p.ID = m1.post_id
@@ -117,8 +117,8 @@ 
             AND m1.meta_key = '_review_id'
             AND m2.meta_key = '_review_type'
             AND m2.meta_value = '{$reviewType}'
-        ");
-        return array_keys(array_flip($results));
+        " );
+        return array_keys( array_flip( $results ) );
     }
 
     /**
@@ -126,12 +126,12 @@ 
      * @param int $limit
      * @return array
      */
-    public function getReviewRatingsFromIds(array $postIds, $greaterThanId = 0, $limit = 100)
+    public function getReviewRatingsFromIds( array $postIds, $greaterThanId = 0, $limit = 100 )
     {
-        sort($postIds);
-        $postIds = array_slice($postIds, intval(array_search($greaterThanId, $postIds)), $limit);
-        $postIds = implode(',', $postIds);
-        return (array) $this->db->get_results("
+        sort( $postIds );
+        $postIds = array_slice( $postIds, intval( array_search( $greaterThanId, $postIds ) ), $limit );
+        $postIds = implode( ',', $postIds );
+        return (array)$this->db->get_results( "
             SELECT p.ID, m.meta_value AS rating
             FROM {$this->db->posts} AS p
             INNER JOIN {$this->db->postmeta} AS m ON p.ID = m.post_id
@@ -143,7 +143,7 @@ 
             GROUP BY p.ID
             ORDER By p.ID ASC
             LIMIT {$limit}
-        ");
+        " );
     }
 
     /**
@@ -151,13 +151,13 @@ 
      * @param string $status
      * @return array
      */
-    public function getReviewsMeta($key, $status = 'publish')
+    public function getReviewsMeta( $key, $status = 'publish' )
     {
         $postStatusQuery = 'all' != $status && !empty($status)
             ? "AND p.post_status = '{$status}'"
             : '';
-        $key = Str::prefix('_', $key);
-        $values = $this->db->get_col("
+        $key = Str::prefix( '_', $key );
+        $values = $this->db->get_col( "
             SELECT DISTINCT m.meta_value
             FROM {$this->db->postmeta} m
             LEFT JOIN {$this->db->posts} p ON p.ID = m.post_id
@@ -167,8 +167,8 @@ 
             $postStatusQuery
             GROUP BY p.ID -- remove duplicate meta_value entries
             ORDER BY m.meta_id ASC -- sort by oldest meta_value
-        ");
-        sort($values);
+        " );
+        sort( $values );
         return $values;
     }
 
@@ -176,34 +176,34 @@ 
      * @param string $and
      * @return string
      */
-    protected function getAndForCounts(array $args, $and = '')
+    protected function getAndForCounts( array $args, $and = '' )
     {
-        $postIds = implode(',', array_filter(Arr::get($args, 'post_ids', [])));
-        $termIds = implode(',', array_filter(Arr::get($args, 'term_ids', [])));
-        if (!empty($args['type'])) {
-            $and.= "AND m2.meta_value = '{$args['type']}' ";
+        $postIds = implode( ',', array_filter( Arr::get( $args, 'post_ids', [] ) ) );
+        $termIds = implode( ',', array_filter( Arr::get( $args, 'term_ids', [] ) ) );
+        if( !empty($args['type']) ) {
+            $and .= "AND m2.meta_value = '{$args['type']}' ";
         }
-        if ($postIds) {
-            $and.= "AND m3.meta_key = '_assigned_to' AND m3.meta_value IN ({$postIds}) ";
+        if( $postIds ) {
+            $and .= "AND m3.meta_key = '_assigned_to' AND m3.meta_value IN ({$postIds}) ";
         }
-        if ($termIds) {
-            $and.= "AND tr.term_taxonomy_id IN ({$termIds}) ";
+        if( $termIds ) {
+            $and .= "AND tr.term_taxonomy_id IN ({$termIds}) ";
         }
-        return apply_filters('site-reviews/query/and-for-counts', $and);
+        return apply_filters( 'site-reviews/query/and-for-counts', $and );
     }
 
     /**
      * @param string $innerJoin
      * @return string
      */
-    protected function getInnerJoinForCounts(array $args, $innerJoin = '')
+    protected function getInnerJoinForCounts( array $args, $innerJoin = '' )
     {
-        if (!empty(Arr::get($args, 'post_ids'))) {
-            $innerJoin.= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id ";
+        if( !empty(Arr::get( $args, 'post_ids' )) ) {
+            $innerJoin .= "INNER JOIN {$this->db->postmeta} AS m3 ON p.ID = m3.post_id ";
         }
-        if (!empty(Arr::get($args, 'term_ids'))) {
-            $innerJoin.= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id ";
+        if( !empty(Arr::get( $args, 'term_ids' )) ) {
+            $innerJoin .= "INNER JOIN {$this->db->term_relationships} AS tr ON p.ID = tr.object_id ";
         }
-        return apply_filters('site-reviews/query/inner-join-for-counts', $innerJoin);
+        return apply_filters( 'site-reviews/query/inner-join-for-counts', $innerJoin );
     }
 }

plugin/Database/TermCountsManager.php

Indentation +82 added lines, -82 removed lines

@@ -9,93 +9,93 @@
 
 class TermCountsManager
 {
-    /**
-     * @var CountsManager
-     */
-    protected $manager;
+	/**
+	 * @var CountsManager
+	 */
+	protected $manager;
 
-    public function __construct()
-    {
-        $this->manager = glsr(CountsManager::class);
-    }
+	public function __construct()
+	{
+		$this->manager = glsr(CountsManager::class);
+	}
 
-    /**
-     * @param int $termTaxonomyId
-     * @return array
-     */
-    public function build($termTaxonomyId)
-    {
-        return $this->manager->buildCounts([
-            'term_ids' => [$termTaxonomyId],
-        ]);
-    }
+	/**
+	 * @param int $termTaxonomyId
+	 * @return array
+	 */
+	public function build($termTaxonomyId)
+	{
+		return $this->manager->buildCounts([
+			'term_ids' => [$termTaxonomyId],
+		]);
+	}
 
-    /**
-     * @return void
-     */
-    public function decrease(Review $review)
-    {
-        foreach ($review->term_ids as $termId) {
-            if (empty($counts = $this->get($termId))) {
-                continue;
-            }
-            $this->update($termId,
-                $this->manager->decreaseRating($counts, $review->review_type, $review->rating)
-            );
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function decrease(Review $review)
+	{
+		foreach ($review->term_ids as $termId) {
+			if (empty($counts = $this->get($termId))) {
+				continue;
+			}
+			$this->update($termId,
+				$this->manager->decreaseRating($counts, $review->review_type, $review->rating)
+			);
+		}
+	}
 
-    /**
-     * @param int $termId
-     * @return array
-     */
-    public function get($termId)
-    {
-        return array_filter((array) get_term_meta($termId, CountsManager::META_COUNT, true));
-    }
+	/**
+	 * @param int $termId
+	 * @return array
+	 */
+	public function get($termId)
+	{
+		return array_filter((array) get_term_meta($termId, CountsManager::META_COUNT, true));
+	}
 
-    /**
-     * @return void
-     */
-    public function increase(Review $review)
-    {
-        $terms = glsr(ReviewManager::class)->normalizeTerms(implode(',', $review->term_ids));
-        foreach ($terms as $term) {
-            $counts = $this->get($term['term_id']);
-            $counts = empty($counts)
-                ? $this->build($term['term_taxonomy_id'])
-                : $this->manager->increaseRating($counts, $review->review_type, $review->rating);
-            $this->update($term['term_id'], $counts);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function increase(Review $review)
+	{
+		$terms = glsr(ReviewManager::class)->normalizeTerms(implode(',', $review->term_ids));
+		foreach ($terms as $term) {
+			$counts = $this->get($term['term_id']);
+			$counts = empty($counts)
+				? $this->build($term['term_taxonomy_id'])
+				: $this->manager->increaseRating($counts, $review->review_type, $review->rating);
+			$this->update($term['term_id'], $counts);
+		}
+	}
 
-    /**
-     * @param int $termId
-     * @return void
-     */
-    public function update($termId, array $reviewCounts)
-    {
-        $term = get_term($termId, Application::TAXONOMY);
-        if (!isset($term->term_id)) {
-            return;
-        }
-        $ratingCounts = $this->manager->flatten($reviewCounts);
-        update_term_meta($termId, CountsManager::META_COUNT, $reviewCounts);
-        update_term_meta($termId, CountsManager::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
-        update_term_meta($termId, CountsManager::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
-    }
+	/**
+	 * @param int $termId
+	 * @return void
+	 */
+	public function update($termId, array $reviewCounts)
+	{
+		$term = get_term($termId, Application::TAXONOMY);
+		if (!isset($term->term_id)) {
+			return;
+		}
+		$ratingCounts = $this->manager->flatten($reviewCounts);
+		update_term_meta($termId, CountsManager::META_COUNT, $reviewCounts);
+		update_term_meta($termId, CountsManager::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
+		update_term_meta($termId, CountsManager::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
+	}
 
-    /**
-     * @return void
-     */
-    public function updateAll()
-    {
-        glsr(SqlQueries::class)->deleteTermCountMetaKeys();
-        $terms = glsr(Database::class)->getTerms([
-            'fields' => 'all',
-        ]);
-        foreach ($terms as $term) {
-            $this->update($term->term_id, $this->build($term->term_taxonomy_id));
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function updateAll()
+	{
+		glsr(SqlQueries::class)->deleteTermCountMetaKeys();
+		$terms = glsr(Database::class)->getTerms([
+			'fields' => 'all',
+		]);
+		foreach ($terms as $term) {
+			$this->update($term->term_id, $this->build($term->term_taxonomy_id));
+		}
+	}
 }

Spacing +30 added lines, -30 removed lines

@@ -16,31 +16,31 @@ 
 
     public function __construct()
     {
-        $this->manager = glsr(CountsManager::class);
+        $this->manager = glsr( CountsManager::class );
     }
 
     /**
      * @param int $termTaxonomyId
      * @return array
      */
-    public function build($termTaxonomyId)
+    public function build( $termTaxonomyId )
     {
-        return $this->manager->buildCounts([
+        return $this->manager->buildCounts( [
             'term_ids' => [$termTaxonomyId],
-        ]);
+        ] );
     }
 
     /**
      * @return void
      */
-    public function decrease(Review $review)
+    public function decrease( Review $review )
     {
-        foreach ($review->term_ids as $termId) {
-            if (empty($counts = $this->get($termId))) {
+        foreach( $review->term_ids as $termId ) {
+            if( empty($counts = $this->get( $termId )) ) {
                 continue;
             }
-            $this->update($termId,
-                $this->manager->decreaseRating($counts, $review->review_type, $review->rating)
+            $this->update( $termId,
+                $this->manager->decreaseRating( $counts, $review->review_type, $review->rating )
             );
         }
     }
@@ -49,23 +49,23 @@ 
      * @param int $termId
      * @return array
      */
-    public function get($termId)
+    public function get( $termId )
     {
-        return array_filter((array) get_term_meta($termId, CountsManager::META_COUNT, true));
+        return array_filter( (array)get_term_meta( $termId, CountsManager::META_COUNT, true ) );
     }
 
     /**
      * @return void
      */
-    public function increase(Review $review)
+    public function increase( Review $review )
     {
-        $terms = glsr(ReviewManager::class)->normalizeTerms(implode(',', $review->term_ids));
-        foreach ($terms as $term) {
-            $counts = $this->get($term['term_id']);
+        $terms = glsr( ReviewManager::class )->normalizeTerms( implode( ',', $review->term_ids ) );
+        foreach( $terms as $term ) {
+            $counts = $this->get( $term['term_id'] );
             $counts = empty($counts)
-                ? $this->build($term['term_taxonomy_id'])
-                : $this->manager->increaseRating($counts, $review->review_type, $review->rating);
-            $this->update($term['term_id'], $counts);
+                ? $this->build( $term['term_taxonomy_id'] )
+                : $this->manager->increaseRating( $counts, $review->review_type, $review->rating );
+            $this->update( $term['term_id'], $counts );
         }
     }
 
@@ -73,16 +73,16 @@ 
      * @param int $termId
      * @return void
      */
-    public function update($termId, array $reviewCounts)
+    public function update( $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->manager->flatten($reviewCounts);
-        update_term_meta($termId, CountsManager::META_COUNT, $reviewCounts);
-        update_term_meta($termId, CountsManager::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
-        update_term_meta($termId, CountsManager::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
+        $ratingCounts = $this->manager->flatten( $reviewCounts );
+        update_term_meta( $termId, CountsManager::META_COUNT, $reviewCounts );
+        update_term_meta( $termId, CountsManager::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) );
+        update_term_meta( $termId, CountsManager::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) );
     }
 
     /**
@@ -90,12 +90,12 @@ 
      */
     public function updateAll()
     {
-        glsr(SqlQueries::class)->deleteTermCountMetaKeys();
-        $terms = glsr(Database::class)->getTerms([
+        glsr( SqlQueries::class )->deleteTermCountMetaKeys();
+        $terms = glsr( Database::class )->getTerms( [
             'fields' => 'all',
-        ]);
-        foreach ($terms as $term) {
-            $this->update($term->term_id, $this->build($term->term_taxonomy_id));
+        ] );
+        foreach( $terms as $term ) {
+            $this->update( $term->term_id, $this->build( $term->term_taxonomy_id ) );
         }
     }
 }

plugin/Database/PostCountsManager.php

Indentation +74 added lines, -74 removed lines

@@ -7,85 +7,85 @@
 
 class PostCountsManager
 {
-    /**
-     * @var CountsManager
-     */
-    protected $manager;
+	/**
+	 * @var CountsManager
+	 */
+	protected $manager;
 
-    public function __construct()
-    {
-        $this->manager = glsr(CountsManager::class);
-    }
+	public function __construct()
+	{
+		$this->manager = glsr(CountsManager::class);
+	}
 
-    /**
-     * @param int $postId
-     * @return array
-     */
-    public function build($postId)
-    {
-        return $this->manager->buildCounts([
-            'post_ids' => [$postId],
-        ]);
-    }
+	/**
+	 * @param int $postId
+	 * @return array
+	 */
+	public function build($postId)
+	{
+		return $this->manager->buildCounts([
+			'post_ids' => [$postId],
+		]);
+	}
 
-    /**
-     * @return void
-     */
-    public function decrease(Review $review)
-    {
-        if (empty($counts = $this->get($review->assigned_to))) {
-            return;
-        }
-        $this->update($review->assigned_to,
-            $this->manager->decreaseRating($counts, $review->review_type, $review->rating)
-        );
-    }
+	/**
+	 * @return void
+	 */
+	public function decrease(Review $review)
+	{
+		if (empty($counts = $this->get($review->assigned_to))) {
+			return;
+		}
+		$this->update($review->assigned_to,
+			$this->manager->decreaseRating($counts, $review->review_type, $review->rating)
+		);
+	}
 
-    /**
-     * @param int $postId
-     * @return array
-     */
-    public function get($postId)
-    {
-        return array_filter((array) get_post_meta($postId, CountsManager::META_COUNT, true));
-    }
+	/**
+	 * @param int $postId
+	 * @return array
+	 */
+	public function get($postId)
+	{
+		return array_filter((array) get_post_meta($postId, CountsManager::META_COUNT, true));
+	}
 
-    /**
-     * @return void
-     */
-    public function increase(Review $review)
-    {
-        if (!(get_post($review->assigned_to) instanceof \WP_Post)) {
-            return;
-        }
-        $counts = $this->get($review->assigned_to);
-        $counts = empty($counts)
-            ? $this->build($review->assigned_to)
-            : $this->manager->increaseRating($counts, $review->review_type, $review->rating);
-        $this->update($review->assigned_to, $counts);
-    }
+	/**
+	 * @return void
+	 */
+	public function increase(Review $review)
+	{
+		if (!(get_post($review->assigned_to) instanceof \WP_Post)) {
+			return;
+		}
+		$counts = $this->get($review->assigned_to);
+		$counts = empty($counts)
+			? $this->build($review->assigned_to)
+			: $this->manager->increaseRating($counts, $review->review_type, $review->rating);
+		$this->update($review->assigned_to, $counts);
+	}
 
-    /**
-     * @param int $postId
-     * @return void
-     */
-    public function update($postId, array $reviewCounts)
-    {
-        $ratingCounts = $this->manager->flatten($reviewCounts);
-        update_post_meta($postId, CountsManager::META_COUNT, $reviewCounts);
-        update_post_meta($postId, CountsManager::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
-        update_post_meta($postId, CountsManager::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
-    }
+	/**
+	 * @param int $postId
+	 * @return void
+	 */
+	public function update($postId, array $reviewCounts)
+	{
+		$ratingCounts = $this->manager->flatten($reviewCounts);
+		update_post_meta($postId, CountsManager::META_COUNT, $reviewCounts);
+		update_post_meta($postId, CountsManager::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
+		update_post_meta($postId, CountsManager::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
+	}
 
-    /**
-     * @return void
-     */
-    public function updateAll()
-    {
-        glsr(SqlQueries::class)->deletePostCountMetaKeys();
-        $postIds = glsr(SqlQueries::class)->getReviewsMeta('assigned_to');
-        foreach ($postIds as $postId) {
-            $this->update($postId, $this->build($postId));
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function updateAll()
+	{
+		glsr(SqlQueries::class)->deletePostCountMetaKeys();
+		$postIds = glsr(SqlQueries::class)->getReviewsMeta('assigned_to');
+		foreach ($postIds as $postId) {
+			$this->update($postId, $this->build($postId));
+		}
+	}
 }

Spacing +25 added lines, -25 removed lines

@@ -14,30 +14,30 @@ 
 
     public function __construct()
     {
-        $this->manager = glsr(CountsManager::class);
+        $this->manager = glsr( CountsManager::class );
     }
 
     /**
      * @param int $postId
      * @return array
      */
-    public function build($postId)
+    public function build( $postId )
     {
-        return $this->manager->buildCounts([
+        return $this->manager->buildCounts( [
             'post_ids' => [$postId],
-        ]);
+        ] );
     }
 
     /**
      * @return void
      */
-    public function decrease(Review $review)
+    public function decrease( Review $review )
     {
-        if (empty($counts = $this->get($review->assigned_to))) {
+        if( empty($counts = $this->get( $review->assigned_to )) ) {
             return;
         }
-        $this->update($review->assigned_to,
-            $this->manager->decreaseRating($counts, $review->review_type, $review->rating)
+        $this->update( $review->assigned_to,
+            $this->manager->decreaseRating( $counts, $review->review_type, $review->rating )
         );
     }
 
@@ -45,36 +45,36 @@ 
      * @param int $postId
      * @return array
      */
-    public function get($postId)
+    public function get( $postId )
     {
-        return array_filter((array) get_post_meta($postId, CountsManager::META_COUNT, true));
+        return array_filter( (array)get_post_meta( $postId, CountsManager::META_COUNT, true ) );
     }
 
     /**
      * @return void
      */
-    public function increase(Review $review)
+    public function increase( Review $review )
     {
-        if (!(get_post($review->assigned_to) instanceof \WP_Post)) {
+        if( !(get_post( $review->assigned_to ) instanceof \WP_Post) ) {
             return;
         }
-        $counts = $this->get($review->assigned_to);
+        $counts = $this->get( $review->assigned_to );
         $counts = empty($counts)
-            ? $this->build($review->assigned_to)
-            : $this->manager->increaseRating($counts, $review->review_type, $review->rating);
-        $this->update($review->assigned_to, $counts);
+            ? $this->build( $review->assigned_to )
+            : $this->manager->increaseRating( $counts, $review->review_type, $review->rating );
+        $this->update( $review->assigned_to, $counts );
     }
 
     /**
      * @param int $postId
      * @return void
      */
-    public function update($postId, array $reviewCounts)
+    public function update( $postId, array $reviewCounts )
     {
-        $ratingCounts = $this->manager->flatten($reviewCounts);
-        update_post_meta($postId, CountsManager::META_COUNT, $reviewCounts);
-        update_post_meta($postId, CountsManager::META_AVERAGE, glsr(Rating::class)->getAverage($ratingCounts));
-        update_post_meta($postId, CountsManager::META_RANKING, glsr(Rating::class)->getRanking($ratingCounts));
+        $ratingCounts = $this->manager->flatten( $reviewCounts );
+        update_post_meta( $postId, CountsManager::META_COUNT, $reviewCounts );
+        update_post_meta( $postId, CountsManager::META_AVERAGE, glsr( Rating::class )->getAverage( $ratingCounts ) );
+        update_post_meta( $postId, CountsManager::META_RANKING, glsr( Rating::class )->getRanking( $ratingCounts ) );
     }
 
     /**
@@ -82,10 +82,10 @@ 
      */
     public function updateAll()
     {
-        glsr(SqlQueries::class)->deletePostCountMetaKeys();
-        $postIds = glsr(SqlQueries::class)->getReviewsMeta('assigned_to');
-        foreach ($postIds as $postId) {
-            $this->update($postId, $this->build($postId));
+        glsr( SqlQueries::class )->deletePostCountMetaKeys();
+        $postIds = glsr( SqlQueries::class )->getReviewsMeta( 'assigned_to' );
+        foreach( $postIds as $postId ) {
+            $this->update( $postId, $this->build( $postId ) );
         }
     }
 }

plugin/Shortcodes/SiteReviewsFormShortcode.php

Indentation +11 added lines, -11 removed lines

@@ -4,15 +4,15 @@
 
 class SiteReviewsFormShortcode extends Shortcode
 {
-    protected function hideOptions()
-    {
-        return [
-            'rating' => __('Hide the rating field', 'site-reviews'),
-            'title' => __('Hide the title field', 'site-reviews'),
-            'content' => __('Hide the review field', 'site-reviews'),
-            'name' => __('Hide the name field', 'site-reviews'),
-            'email' => __('Hide the email field', 'site-reviews'),
-            'terms' => __('Hide the terms field', 'site-reviews'),
-        ];
-    }
+	protected function hideOptions()
+	{
+		return [
+			'rating' => __('Hide the rating field', 'site-reviews'),
+			'title' => __('Hide the title field', 'site-reviews'),
+			'content' => __('Hide the review field', 'site-reviews'),
+			'name' => __('Hide the name field', 'site-reviews'),
+			'email' => __('Hide the email field', 'site-reviews'),
+			'terms' => __('Hide the terms field', 'site-reviews'),
+		];
+	}
 }

Spacing +6 added lines, -6 removed lines

@@ -7,12 +7,12 @@
     protected function hideOptions()
     {
         return [
-            'rating' => __('Hide the rating field', 'site-reviews'),
-            'title' => __('Hide the title field', 'site-reviews'),
-            'content' => __('Hide the review field', 'site-reviews'),
-            'name' => __('Hide the name field', 'site-reviews'),
-            'email' => __('Hide the email field', 'site-reviews'),
-            'terms' => __('Hide the terms field', 'site-reviews'),
+            'rating' => __( 'Hide the rating field', 'site-reviews' ),
+            'title' => __( 'Hide the title field', 'site-reviews' ),
+            'content' => __( 'Hide the review field', 'site-reviews' ),
+            'name' => __( 'Hide the name field', 'site-reviews' ),
+            'email' => __( 'Hide the email field', 'site-reviews' ),
+            'terms' => __( 'Hide the terms field', 'site-reviews' ),
         ];
     }
 }

plugin/Shortcodes/Shortcode.php

Indentation +234 added lines, -234 removed lines

@@ -12,238 +12,238 @@
 
 abstract class Shortcode implements ShortcodeContract
 {
-    /**
-     * @var string
-     */
-    protected $partialName;
-
-    /**
-     * @var string
-     */
-    protected $shortcodeName;
-
-    public function __construct()
-    {
-        $this->partialName = $this->getShortcodePartialName();
-        $this->shortcodeName = $this->getShortcodeName();
-    }
-
-    /**
-     * @param string|array $atts
-     * @param string $type
-     * @return string
-     */
-    public function build($atts, array $args = [], $type = 'shortcode')
-    {
-        $args = $this->normalizeArgs($args, $type);
-        $atts = $this->normalizeAtts($atts, $type);
-        $partial = glsr(Partial::class)->build($this->partialName, $atts);
-        $title = !empty($atts['title'])
-            ? $args['before_title'].$atts['title'].$args['after_title']
-            : '';
-        $debug = sprintf('<glsr-%1$s hidden data-atts=\'%2$s\'></glsr-%1$s>', $type, $atts['json']);
-        return $args['before_widget'].$title.$partial.$debug.$args['after_widget'];
-    }
-
-    /**
-     * @param string|array $atts
-     * @return string
-     */
-    public function buildShortcode($atts = [])
-    {
-        return $this->build($atts);
-    }
-
-    /**
-     * @return array
-     */
-    public function getDefaults($atts)
-    {
-        return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
-    }
-
-    /**
-     * @return array
-     */
-    public function getHideOptions()
-    {
-        $options = $this->hideOptions();
-        return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodeClassName($replace = '', $search = 'Shortcode')
-    {
-        return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodeDefaultsClassName()
-    {
-        return Helper::buildClassName(
-            $this->getShortcodeClassName('Defaults'),
-            'Defaults'
-        );
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodeName()
-    {
-        return Str::snakeCase($this->getShortcodeClassName());
-    }
-
-    /**
-     * @return string
-     */
-    public function getShortcodePartialName()
-    {
-        return Str::dashCase($this->getShortcodeClassName());
-    }
-
-    /**
-     * @param array|string $args
-     * @param string $type
-     * @return array
-     */
-    public function normalizeArgs($args, $type = 'shortcode')
-    {
-        $args = wp_parse_args($args, [
-            'before_widget' => '<div class="glsr-'.$type.' '.$type.'-'.$this->partialName.'">',
-            'after_widget' => '</div>',
-            'before_title' => '<h3 class="glsr-'.$type.'-title">',
-            'after_title' => '</h3>',
-        ]);
-        return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
-    }
-
-    /**
-     * @param array|string $atts
-     * @param string $type
-     * @return array
-     */
-    public function normalizeAtts($atts, $type = 'shortcode')
-    {
-        $atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
-        $atts = $this->getDefaults($atts);
-        array_walk($atts, function (&$value, $key) {
-            $methodName = Helper::buildMethodName($key, 'normalize');
-            if (!method_exists($this, $methodName)) {
-                return;
-            }
-            $value = $this->$methodName($value);
-        });
-        $this->setId($atts);
-        return $atts;
-    }
-
-    /**
-     * @return array
-     */
-    abstract protected function hideOptions();
-
-    /**
-     * @param string $postId
-     * @return int|string
-     */
-    protected function normalizeAssignedTo($postId)
-    {
-        if ('parent_id' == $postId) {
-            $postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
-        } elseif ('post_id' == $postId) {
-            $postId = intval(get_the_ID());
-        }
-        return $postId;
-    }
-
-    /**
-     * @param string $postId
-     * @return int|string
-     */
-    protected function normalizeAssignTo($postId)
-    {
-        return $this->normalizeAssignedTo($postId);
-    }
-
-    /**
-     * @param string|array $hide
-     * @return array
-     */
-    protected function normalizeHide($hide)
-    {
-        if (is_string($hide)) {
-            $hide = explode(',', $hide);
-        }
-        $hideKeys = array_keys($this->getHideOptions());
-        return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
-            return in_array($value, $hideKeys);
-        });
-    }
-
-    /**
-     * @param string $id
-     * @return string
-     */
-    protected function normalizeId($id)
-    {
-        return sanitize_title($id);
-    }
-
-    /**
-     * @param string $labels
-     * @return array
-     */
-    protected function normalizeLabels($labels)
-    {
-        $defaults = [
-            __('Excellent', 'site-reviews'),
-            __('Very good', 'site-reviews'),
-            __('Average', 'site-reviews'),
-            __('Poor', 'site-reviews'),
-            __('Terrible', 'site-reviews'),
-        ];
-        $maxRating = (int) glsr()->constant('MAX_RATING', Rating::class);
-        $defaults = array_pad(array_slice($defaults, 0, $maxRating), $maxRating, '');
-        $labels = array_map('trim', explode(',', $labels));
-        foreach ($defaults as $i => $label) {
-            if (empty($labels[$i])) {
-                continue;
-            }
-            $defaults[$i] = $labels[$i];
-        }
-        return array_combine(range($maxRating, 1), $defaults);
-    }
-
-    /**
-     * @param string $schema
-     * @return bool
-     */
-    protected function normalizeSchema($schema)
-    {
-        return wp_validate_boolean($schema);
-    }
-
-    /**
-     * @param string $text
-     * @return string
-     */
-    protected function normalizeText($text)
-    {
-        return trim($text);
-    }
-
-    /**
-     * @return void
-     */
-    protected function setId(array &$atts)
-    {
-        if (empty($atts['id'])) {
-            $atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
-        }
-    }
+	/**
+	 * @var string
+	 */
+	protected $partialName;
+
+	/**
+	 * @var string
+	 */
+	protected $shortcodeName;
+
+	public function __construct()
+	{
+		$this->partialName = $this->getShortcodePartialName();
+		$this->shortcodeName = $this->getShortcodeName();
+	}
+
+	/**
+	 * @param string|array $atts
+	 * @param string $type
+	 * @return string
+	 */
+	public function build($atts, array $args = [], $type = 'shortcode')
+	{
+		$args = $this->normalizeArgs($args, $type);
+		$atts = $this->normalizeAtts($atts, $type);
+		$partial = glsr(Partial::class)->build($this->partialName, $atts);
+		$title = !empty($atts['title'])
+			? $args['before_title'].$atts['title'].$args['after_title']
+			: '';
+		$debug = sprintf('<glsr-%1$s hidden data-atts=\'%2$s\'></glsr-%1$s>', $type, $atts['json']);
+		return $args['before_widget'].$title.$partial.$debug.$args['after_widget'];
+	}
+
+	/**
+	 * @param string|array $atts
+	 * @return string
+	 */
+	public function buildShortcode($atts = [])
+	{
+		return $this->build($atts);
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getDefaults($atts)
+	{
+		return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
+	}
+
+	/**
+	 * @return array
+	 */
+	public function getHideOptions()
+	{
+		$options = $this->hideOptions();
+		return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodeClassName($replace = '', $search = 'Shortcode')
+	{
+		return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodeDefaultsClassName()
+	{
+		return Helper::buildClassName(
+			$this->getShortcodeClassName('Defaults'),
+			'Defaults'
+		);
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodeName()
+	{
+		return Str::snakeCase($this->getShortcodeClassName());
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getShortcodePartialName()
+	{
+		return Str::dashCase($this->getShortcodeClassName());
+	}
+
+	/**
+	 * @param array|string $args
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeArgs($args, $type = 'shortcode')
+	{
+		$args = wp_parse_args($args, [
+			'before_widget' => '<div class="glsr-'.$type.' '.$type.'-'.$this->partialName.'">',
+			'after_widget' => '</div>',
+			'before_title' => '<h3 class="glsr-'.$type.'-title">',
+			'after_title' => '</h3>',
+		]);
+		return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
+	}
+
+	/**
+	 * @param array|string $atts
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeAtts($atts, $type = 'shortcode')
+	{
+		$atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
+		$atts = $this->getDefaults($atts);
+		array_walk($atts, function (&$value, $key) {
+			$methodName = Helper::buildMethodName($key, 'normalize');
+			if (!method_exists($this, $methodName)) {
+				return;
+			}
+			$value = $this->$methodName($value);
+		});
+		$this->setId($atts);
+		return $atts;
+	}
+
+	/**
+	 * @return array
+	 */
+	abstract protected function hideOptions();
+
+	/**
+	 * @param string $postId
+	 * @return int|string
+	 */
+	protected function normalizeAssignedTo($postId)
+	{
+		if ('parent_id' == $postId) {
+			$postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
+		} elseif ('post_id' == $postId) {
+			$postId = intval(get_the_ID());
+		}
+		return $postId;
+	}
+
+	/**
+	 * @param string $postId
+	 * @return int|string
+	 */
+	protected function normalizeAssignTo($postId)
+	{
+		return $this->normalizeAssignedTo($postId);
+	}
+
+	/**
+	 * @param string|array $hide
+	 * @return array
+	 */
+	protected function normalizeHide($hide)
+	{
+		if (is_string($hide)) {
+			$hide = explode(',', $hide);
+		}
+		$hideKeys = array_keys($this->getHideOptions());
+		return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
+			return in_array($value, $hideKeys);
+		});
+	}
+
+	/**
+	 * @param string $id
+	 * @return string
+	 */
+	protected function normalizeId($id)
+	{
+		return sanitize_title($id);
+	}
+
+	/**
+	 * @param string $labels
+	 * @return array
+	 */
+	protected function normalizeLabels($labels)
+	{
+		$defaults = [
+			__('Excellent', 'site-reviews'),
+			__('Very good', 'site-reviews'),
+			__('Average', 'site-reviews'),
+			__('Poor', 'site-reviews'),
+			__('Terrible', 'site-reviews'),
+		];
+		$maxRating = (int) glsr()->constant('MAX_RATING', Rating::class);
+		$defaults = array_pad(array_slice($defaults, 0, $maxRating), $maxRating, '');
+		$labels = array_map('trim', explode(',', $labels));
+		foreach ($defaults as $i => $label) {
+			if (empty($labels[$i])) {
+				continue;
+			}
+			$defaults[$i] = $labels[$i];
+		}
+		return array_combine(range($maxRating, 1), $defaults);
+	}
+
+	/**
+	 * @param string $schema
+	 * @return bool
+	 */
+	protected function normalizeSchema($schema)
+	{
+		return wp_validate_boolean($schema);
+	}
+
+	/**
+	 * @param string $text
+	 * @return string
+	 */
+	protected function normalizeText($text)
+	{
+		return trim($text);
+	}
+
+	/**
+	 * @return void
+	 */
+	protected function setId(array &$atts)
+	{
+		if (empty($atts['id'])) {
+			$atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
+		}
+	}
 }

Spacing +61 added lines, -61 removed lines

@@ -33,15 +33,15 @@ 
      * @param string $type
      * @return string
      */
-    public function build($atts, array $args = [], $type = 'shortcode')
+    public function build( $atts, array $args = [], $type = 'shortcode' )
     {
-        $args = $this->normalizeArgs($args, $type);
-        $atts = $this->normalizeAtts($atts, $type);
-        $partial = glsr(Partial::class)->build($this->partialName, $atts);
+        $args = $this->normalizeArgs( $args, $type );
+        $atts = $this->normalizeAtts( $atts, $type );
+        $partial = glsr( Partial::class )->build( $this->partialName, $atts );
         $title = !empty($atts['title'])
             ? $args['before_title'].$atts['title'].$args['after_title']
             : '';
-        $debug = sprintf('<glsr-%1$s hidden data-atts=\'%2$s\'></glsr-%1$s>', $type, $atts['json']);
+        $debug = sprintf( '<glsr-%1$s hidden data-atts=\'%2$s\'></glsr-%1$s>', $type, $atts['json'] );
         return $args['before_widget'].$title.$partial.$debug.$args['after_widget'];
     }
 
@@ -49,17 +49,17 @@ 
      * @param string|array $atts
      * @return string
      */
-    public function buildShortcode($atts = [])
+    public function buildShortcode( $atts = [] )
     {
-        return $this->build($atts);
+        return $this->build( $atts );
     }
 
     /**
      * @return array
      */
-    public function getDefaults($atts)
+    public function getDefaults( $atts )
     {
-        return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
+        return glsr( $this->getShortcodeDefaultsClassName() )->restrict( wp_parse_args( $atts ) );
     }
 
     /**
@@ -68,15 +68,15 @@ 
     public function getHideOptions()
     {
         $options = $this->hideOptions();
-        return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
+        return apply_filters( 'site-reviews/shortcode/hide-options', $options, $this->shortcodeName );
     }
 
     /**
      * @return string
      */
-    public function getShortcodeClassName($replace = '', $search = 'Shortcode')
+    public function getShortcodeClassName( $replace = '', $search = 'Shortcode' )
     {
-        return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
+        return str_replace( $search, $replace, (new ReflectionClass( $this ))->getShortName() );
     }
 
     /**
@@ -85,7 +85,7 @@ 
     public function getShortcodeDefaultsClassName()
     {
         return Helper::buildClassName(
-            $this->getShortcodeClassName('Defaults'),
+            $this->getShortcodeClassName( 'Defaults' ),
             'Defaults'
         );
     }
@@ -95,7 +95,7 @@ 
      */
     public function getShortcodeName()
     {
-        return Str::snakeCase($this->getShortcodeClassName());
+        return Str::snakeCase( $this->getShortcodeClassName() );
     }
 
     /**
@@ -103,7 +103,7 @@ 
      */
     public function getShortcodePartialName()
     {
-        return Str::dashCase($this->getShortcodeClassName());
+        return Str::dashCase( $this->getShortcodeClassName() );
     }
 
     /**
@@ -111,15 +111,15 @@ 
      * @param string $type
      * @return array
      */
-    public function normalizeArgs($args, $type = 'shortcode')
+    public function normalizeArgs( $args, $type = 'shortcode' )
     {
-        $args = wp_parse_args($args, [
+        $args = wp_parse_args( $args, [
             'before_widget' => '<div class="glsr-'.$type.' '.$type.'-'.$this->partialName.'">',
             'after_widget' => '</div>',
             'before_title' => '<h3 class="glsr-'.$type.'-title">',
             'after_title' => '</h3>',
-        ]);
-        return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
+        ] );
+        return apply_filters( 'site-reviews/shortcode/args', $args, $type, $this->partialName );
     }
 
     /**
@@ -127,18 +127,18 @@ 
      * @param string $type
      * @return array
      */
-    public function normalizeAtts($atts, $type = 'shortcode')
+    public function normalizeAtts( $atts, $type = 'shortcode' )
     {
-        $atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
-        $atts = $this->getDefaults($atts);
-        array_walk($atts, function (&$value, $key) {
-            $methodName = Helper::buildMethodName($key, 'normalize');
-            if (!method_exists($this, $methodName)) {
+        $atts = apply_filters( 'site-reviews/shortcode/atts', $atts, $type, $this->partialName );
+        $atts = $this->getDefaults( $atts );
+        array_walk( $atts, function( &$value, $key ) {
+            $methodName = Helper::buildMethodName( $key, 'normalize' );
+            if( !method_exists( $this, $methodName ) ) {
                 return;
             }
-            $value = $this->$methodName($value);
+            $value = $this->$methodName( $value );
         });
-        $this->setId($atts);
+        $this->setId( $atts );
         return $atts;
     }
 
@@ -151,12 +151,12 @@ 
      * @param string $postId
      * @return int|string
      */
-    protected function normalizeAssignedTo($postId)
+    protected function normalizeAssignedTo( $postId )
     {
-        if ('parent_id' == $postId) {
-            $postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
-        } elseif ('post_id' == $postId) {
-            $postId = intval(get_the_ID());
+        if( 'parent_id' == $postId ) {
+            $postId = intval( wp_get_post_parent_id( intval( get_the_ID() ) ) );
+        } elseif( 'post_id' == $postId ) {
+            $postId = intval( get_the_ID() );
         }
         return $postId;
     }
@@ -165,23 +165,23 @@ 
      * @param string $postId
      * @return int|string
      */
-    protected function normalizeAssignTo($postId)
+    protected function normalizeAssignTo( $postId )
     {
-        return $this->normalizeAssignedTo($postId);
+        return $this->normalizeAssignedTo( $postId );
     }
 
     /**
      * @param string|array $hide
      * @return array
      */
-    protected function normalizeHide($hide)
+    protected function normalizeHide( $hide )
     {
-        if (is_string($hide)) {
-            $hide = explode(',', $hide);
+        if( is_string( $hide ) ) {
+            $hide = explode( ',', $hide );
         }
-        $hideKeys = array_keys($this->getHideOptions());
-        return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
-            return in_array($value, $hideKeys);
+        $hideKeys = array_keys( $this->getHideOptions() );
+        return array_filter( array_map( 'trim', $hide ), function( $value ) use ($hideKeys) {
+            return in_array( $value, $hideKeys );
         });
     }
 
@@ -189,61 +189,61 @@ 
      * @param string $id
      * @return string
      */
-    protected function normalizeId($id)
+    protected function normalizeId( $id )
     {
-        return sanitize_title($id);
+        return sanitize_title( $id );
     }
 
     /**
      * @param string $labels
      * @return array
      */
-    protected function normalizeLabels($labels)
+    protected function normalizeLabels( $labels )
     {
         $defaults = [
-            __('Excellent', 'site-reviews'),
-            __('Very good', 'site-reviews'),
-            __('Average', 'site-reviews'),
-            __('Poor', 'site-reviews'),
-            __('Terrible', 'site-reviews'),
+            __( 'Excellent', 'site-reviews' ),
+            __( 'Very good', 'site-reviews' ),
+            __( 'Average', 'site-reviews' ),
+            __( 'Poor', 'site-reviews' ),
+            __( 'Terrible', 'site-reviews' ),
         ];
-        $maxRating = (int) glsr()->constant('MAX_RATING', Rating::class);
-        $defaults = array_pad(array_slice($defaults, 0, $maxRating), $maxRating, '');
-        $labels = array_map('trim', explode(',', $labels));
-        foreach ($defaults as $i => $label) {
-            if (empty($labels[$i])) {
+        $maxRating = (int)glsr()->constant( 'MAX_RATING', Rating::class );
+        $defaults = array_pad( array_slice( $defaults, 0, $maxRating ), $maxRating, '' );
+        $labels = array_map( 'trim', explode( ',', $labels ) );
+        foreach( $defaults as $i => $label ) {
+            if( empty($labels[$i]) ) {
                 continue;
             }
             $defaults[$i] = $labels[$i];
         }
-        return array_combine(range($maxRating, 1), $defaults);
+        return array_combine( range( $maxRating, 1 ), $defaults );
     }
 
     /**
      * @param string $schema
      * @return bool
      */
-    protected function normalizeSchema($schema)
+    protected function normalizeSchema( $schema )
     {
-        return wp_validate_boolean($schema);
+        return wp_validate_boolean( $schema );
     }
 
     /**
      * @param string $text
      * @return string
      */
-    protected function normalizeText($text)
+    protected function normalizeText( $text )
     {
-        return trim($text);
+        return trim( $text );
     }
 
     /**
      * @return void
      */
-    protected function setId(array &$atts)
+    protected function setId( array &$atts )
     {
-        if (empty($atts['id'])) {
-            $atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
+        if( empty($atts['id']) ) {
+            $atts['id'] = Application::PREFIX.substr( md5( serialize( $atts ) ), 0, 8 );
         }
     }
 }

plugin/Modules/Html/Partials/SiteReviewsSummary.php

Indentation +193 added lines, -193 removed lines

@@ -10,197 +10,197 @@
 
 class SiteReviewsSummary
 {
-    /**
-     * @var array
-     */
-    protected $args;
-
-    /**
-     * @var float
-     */
-    protected $averageRating;
-
-    /**
-     * @var array
-     */
-    protected $ratingCounts;
-
-    /**
-     * @return void|string
-     */
-    public function build(array $args = [])
-    {
-        $this->args = $args;
-        $this->ratingCounts = glsr(ReviewManager::class)->getRatingCounts($args);
-        if (!array_sum($this->ratingCounts) && $this->isHidden('if_empty')) {
-            return;
-        }
-        $this->averageRating = glsr(Rating::class)->getAverage($this->ratingCounts);
-        $this->generateSchema();
-        return glsr(Template::class)->build('templates/reviews-summary', [
-            'context' => [
-                'assigned_to' => $this->args['assigned_to'],
-                'category' => $this->args['category'],
-                'class' => $this->getClass(),
-                'id' => $this->args['id'],
-                'percentages' => $this->buildPercentage(),
-                'rating' => $this->buildRating(),
-                'stars' => $this->buildStars(),
-                'text' => $this->buildText(),
-            ],
-        ]);
-    }
-
-    /**
-     * @return void|string
-     */
-    protected function buildPercentage()
-    {
-        if ($this->isHidden('bars')) {
-            return;
-        }
-        $percentages = preg_filter('/$/', '%', glsr(Rating::class)->getPercentages($this->ratingCounts));
-        $bars = array_reduce(range(glsr()->constant('MAX_RATING', Rating::class), 1), function ($carry, $level) use ($percentages) {
-            $label = $this->buildPercentageLabel($this->args['labels'][$level]);
-            $background = $this->buildPercentageBackground($percentages[$level]);
-            $count = apply_filters('site-reviews/summary/counts',
-                $percentages[$level],
-                $this->ratingCounts[$level]
-            );
-            $percent = $this->buildPercentageCount($count);
-            $value = $label.$background.$percent;
-            $value = apply_filters('site-reviews/summary/wrap/bar', $value, $this->args, [
-                'percent' => wp_strip_all_tags($count, true),
-                'rating' => $level,
-            ]);
-            return $carry.glsr(Builder::class)->div($value, [
-                'class' => 'glsr-bar',
-            ]);
-        });
-        return $this->wrap('percentage', $bars);
-    }
-
-    /**
-     * @param string $percent
-     * @return string
-     */
-    protected function buildPercentageBackground($percent)
-    {
-        $backgroundPercent = glsr(Builder::class)->span([
-            'class' => 'glsr-bar-background-percent',
-            'style' => 'width:'.$percent,
-        ]);
-        return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
-    }
-
-    /**
-     * @param string $count
-     * @return string
-     */
-    protected function buildPercentageCount($count)
-    {
-        return '<span class="glsr-bar-percent">'.$count.'</span>';
-    }
-
-    /**
-     * @param string $label
-     * @return string
-     */
-    protected function buildPercentageLabel($label)
-    {
-        return '<span class="glsr-bar-label">'.$label.'</span>';
-    }
-
-    /**
-     * @return void|string
-     */
-    protected function buildRating()
-    {
-        if ($this->isHidden('rating')) {
-            return;
-        }
-        return $this->wrap('rating', '<span>'.$this->averageRating.'</span>');
-    }
-
-    /**
-     * @return void|string
-     */
-    protected function buildStars()
-    {
-        if ($this->isHidden('stars')) {
-            return;
-        }
-        $stars = glsr_star_rating($this->averageRating);
-        return $this->wrap('stars', $stars);
-    }
-
-    /**
-     * @return void|string
-     */
-    protected function buildText()
-    {
-        if ($this->isHidden('summary')) {
-            return;
-        }
-        $count = intval(array_sum($this->ratingCounts));
-        if (empty($this->args['text'])) {
-            // @todo document this change
-            $this->args['text'] = _nx(
-                '{rating} out of {max} stars (based on {num} review)',
-                '{rating} out of {max} stars (based on {num} reviews)',
-                $count,
-                'Do not translate {rating}, {max}, and {num}, they are template tags.',
-                'site-reviews'
-            );
-        }
-        $summary = str_replace(
-            ['{rating}', '{max}', '{num}'],
-            [$this->averageRating, glsr()->constant('MAX_RATING', Rating::class), $count],
-            $this->args['text']
-        );
-        return $this->wrap('text', '<span>'.$summary.'</span>');
-    }
-
-    /**
-     * @return void
-     */
-    protected function generateSchema()
-    {
-        if (!wp_validate_boolean($this->args['schema'])) {
-            return;
-        }
-        glsr(Schema::class)->store(
-            glsr(Schema::class)->buildSummary($this->args)
-        );
-    }
-
-    /**
-     * @return string
-     */
-    protected function getClass()
-    {
-        return trim('glsr-summary glsr-default '.$this->args['class']);
-    }
-
-    /**
-     * @param string $key
-     * @return bool
-     */
-    protected function isHidden($key)
-    {
-        return in_array($key, $this->args['hide']);
-    }
-
-    /**
-     * @param string $key
-     * @param string $value
-     * @return string
-     */
-    protected function wrap($key, $value)
-    {
-        $value = apply_filters('site-reviews/summary/wrap/'.$key, $value, $this->args);
-        return glsr(Builder::class)->div($value, [
-            'class' => 'glsr-summary-'.$key,
-        ]);
-    }
+	/**
+	 * @var array
+	 */
+	protected $args;
+
+	/**
+	 * @var float
+	 */
+	protected $averageRating;
+
+	/**
+	 * @var array
+	 */
+	protected $ratingCounts;
+
+	/**
+	 * @return void|string
+	 */
+	public function build(array $args = [])
+	{
+		$this->args = $args;
+		$this->ratingCounts = glsr(ReviewManager::class)->getRatingCounts($args);
+		if (!array_sum($this->ratingCounts) && $this->isHidden('if_empty')) {
+			return;
+		}
+		$this->averageRating = glsr(Rating::class)->getAverage($this->ratingCounts);
+		$this->generateSchema();
+		return glsr(Template::class)->build('templates/reviews-summary', [
+			'context' => [
+				'assigned_to' => $this->args['assigned_to'],
+				'category' => $this->args['category'],
+				'class' => $this->getClass(),
+				'id' => $this->args['id'],
+				'percentages' => $this->buildPercentage(),
+				'rating' => $this->buildRating(),
+				'stars' => $this->buildStars(),
+				'text' => $this->buildText(),
+			],
+		]);
+	}
+
+	/**
+	 * @return void|string
+	 */
+	protected function buildPercentage()
+	{
+		if ($this->isHidden('bars')) {
+			return;
+		}
+		$percentages = preg_filter('/$/', '%', glsr(Rating::class)->getPercentages($this->ratingCounts));
+		$bars = array_reduce(range(glsr()->constant('MAX_RATING', Rating::class), 1), function ($carry, $level) use ($percentages) {
+			$label = $this->buildPercentageLabel($this->args['labels'][$level]);
+			$background = $this->buildPercentageBackground($percentages[$level]);
+			$count = apply_filters('site-reviews/summary/counts',
+				$percentages[$level],
+				$this->ratingCounts[$level]
+			);
+			$percent = $this->buildPercentageCount($count);
+			$value = $label.$background.$percent;
+			$value = apply_filters('site-reviews/summary/wrap/bar', $value, $this->args, [
+				'percent' => wp_strip_all_tags($count, true),
+				'rating' => $level,
+			]);
+			return $carry.glsr(Builder::class)->div($value, [
+				'class' => 'glsr-bar',
+			]);
+		});
+		return $this->wrap('percentage', $bars);
+	}
+
+	/**
+	 * @param string $percent
+	 * @return string
+	 */
+	protected function buildPercentageBackground($percent)
+	{
+		$backgroundPercent = glsr(Builder::class)->span([
+			'class' => 'glsr-bar-background-percent',
+			'style' => 'width:'.$percent,
+		]);
+		return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
+	}
+
+	/**
+	 * @param string $count
+	 * @return string
+	 */
+	protected function buildPercentageCount($count)
+	{
+		return '<span class="glsr-bar-percent">'.$count.'</span>';
+	}
+
+	/**
+	 * @param string $label
+	 * @return string
+	 */
+	protected function buildPercentageLabel($label)
+	{
+		return '<span class="glsr-bar-label">'.$label.'</span>';
+	}
+
+	/**
+	 * @return void|string
+	 */
+	protected function buildRating()
+	{
+		if ($this->isHidden('rating')) {
+			return;
+		}
+		return $this->wrap('rating', '<span>'.$this->averageRating.'</span>');
+	}
+
+	/**
+	 * @return void|string
+	 */
+	protected function buildStars()
+	{
+		if ($this->isHidden('stars')) {
+			return;
+		}
+		$stars = glsr_star_rating($this->averageRating);
+		return $this->wrap('stars', $stars);
+	}
+
+	/**
+	 * @return void|string
+	 */
+	protected function buildText()
+	{
+		if ($this->isHidden('summary')) {
+			return;
+		}
+		$count = intval(array_sum($this->ratingCounts));
+		if (empty($this->args['text'])) {
+			// @todo document this change
+			$this->args['text'] = _nx(
+				'{rating} out of {max} stars (based on {num} review)',
+				'{rating} out of {max} stars (based on {num} reviews)',
+				$count,
+				'Do not translate {rating}, {max}, and {num}, they are template tags.',
+				'site-reviews'
+			);
+		}
+		$summary = str_replace(
+			['{rating}', '{max}', '{num}'],
+			[$this->averageRating, glsr()->constant('MAX_RATING', Rating::class), $count],
+			$this->args['text']
+		);
+		return $this->wrap('text', '<span>'.$summary.'</span>');
+	}
+
+	/**
+	 * @return void
+	 */
+	protected function generateSchema()
+	{
+		if (!wp_validate_boolean($this->args['schema'])) {
+			return;
+		}
+		glsr(Schema::class)->store(
+			glsr(Schema::class)->buildSummary($this->args)
+		);
+	}
+
+	/**
+	 * @return string
+	 */
+	protected function getClass()
+	{
+		return trim('glsr-summary glsr-default '.$this->args['class']);
+	}
+
+	/**
+	 * @param string $key
+	 * @return bool
+	 */
+	protected function isHidden($key)
+	{
+		return in_array($key, $this->args['hide']);
+	}
+
+	/**
+	 * @param string $key
+	 * @param string $value
+	 * @return string
+	 */
+	protected function wrap($key, $value)
+	{
+		$value = apply_filters('site-reviews/summary/wrap/'.$key, $value, $this->args);
+		return glsr(Builder::class)->div($value, [
+			'class' => 'glsr-summary-'.$key,
+		]);
+	}
 }

Spacing +44 added lines, -44 removed lines

@@ -28,16 +28,16 @@ 
     /**
      * @return void|string
      */
-    public function build(array $args = [])
+    public function build( array $args = [] )
     {
         $this->args = $args;
-        $this->ratingCounts = glsr(ReviewManager::class)->getRatingCounts($args);
-        if (!array_sum($this->ratingCounts) && $this->isHidden('if_empty')) {
+        $this->ratingCounts = glsr( ReviewManager::class )->getRatingCounts( $args );
+        if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ) ) {
             return;
         }
-        $this->averageRating = glsr(Rating::class)->getAverage($this->ratingCounts);
+        $this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
         $this->generateSchema();
-        return glsr(Template::class)->build('templates/reviews-summary', [
+        return glsr( Template::class )->build( 'templates/reviews-summary', [
             'context' => [
                 'assigned_to' => $this->args['assigned_to'],
                 'category' => $this->args['category'],
@@ -48,7 +48,7 @@ 
                 'stars' => $this->buildStars(),
                 'text' => $this->buildText(),
             ],
-        ]);
+        ] );
     }
 
     /**
@@ -56,40 +56,40 @@ 
      */
     protected function buildPercentage()
     {
-        if ($this->isHidden('bars')) {
+        if( $this->isHidden( 'bars' ) ) {
             return;
         }
-        $percentages = preg_filter('/$/', '%', glsr(Rating::class)->getPercentages($this->ratingCounts));
-        $bars = array_reduce(range(glsr()->constant('MAX_RATING', Rating::class), 1), function ($carry, $level) use ($percentages) {
-            $label = $this->buildPercentageLabel($this->args['labels'][$level]);
-            $background = $this->buildPercentageBackground($percentages[$level]);
-            $count = apply_filters('site-reviews/summary/counts',
+        $percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ) );
+        $bars = array_reduce( range( glsr()->constant( 'MAX_RATING', Rating::class ), 1 ), function( $carry, $level ) use ($percentages) {
+            $label = $this->buildPercentageLabel( $this->args['labels'][$level] );
+            $background = $this->buildPercentageBackground( $percentages[$level] );
+            $count = apply_filters( 'site-reviews/summary/counts',
                 $percentages[$level],
                 $this->ratingCounts[$level]
             );
-            $percent = $this->buildPercentageCount($count);
+            $percent = $this->buildPercentageCount( $count );
             $value = $label.$background.$percent;
-            $value = apply_filters('site-reviews/summary/wrap/bar', $value, $this->args, [
-                'percent' => wp_strip_all_tags($count, true),
+            $value = apply_filters( 'site-reviews/summary/wrap/bar', $value, $this->args, [
+                'percent' => wp_strip_all_tags( $count, true ),
                 'rating' => $level,
-            ]);
-            return $carry.glsr(Builder::class)->div($value, [
+            ] );
+            return $carry.glsr( Builder::class )->div( $value, [
                 'class' => 'glsr-bar',
-            ]);
+            ] );
         });
-        return $this->wrap('percentage', $bars);
+        return $this->wrap( 'percentage', $bars );
     }
 
     /**
      * @param string $percent
      * @return string
      */
-    protected function buildPercentageBackground($percent)
+    protected function buildPercentageBackground( $percent )
     {
-        $backgroundPercent = glsr(Builder::class)->span([
+        $backgroundPercent = glsr( Builder::class )->span( [
             'class' => 'glsr-bar-background-percent',
             'style' => 'width:'.$percent,
-        ]);
+        ] );
         return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
     }
 
@@ -97,7 +97,7 @@ 
      * @param string $count
      * @return string
      */
-    protected function buildPercentageCount($count)
+    protected function buildPercentageCount( $count )
     {
         return '<span class="glsr-bar-percent">'.$count.'</span>';
     }
@@ -106,7 +106,7 @@ 
      * @param string $label
      * @return string
      */
-    protected function buildPercentageLabel($label)
+    protected function buildPercentageLabel( $label )
     {
         return '<span class="glsr-bar-label">'.$label.'</span>';
     }
@@ -116,10 +116,10 @@ 
      */
     protected function buildRating()
     {
-        if ($this->isHidden('rating')) {
+        if( $this->isHidden( 'rating' ) ) {
             return;
         }
-        return $this->wrap('rating', '<span>'.$this->averageRating.'</span>');
+        return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
     }
 
     /**
@@ -127,11 +127,11 @@ 
      */
     protected function buildStars()
     {
-        if ($this->isHidden('stars')) {
+        if( $this->isHidden( 'stars' ) ) {
             return;
         }
-        $stars = glsr_star_rating($this->averageRating);
-        return $this->wrap('stars', $stars);
+        $stars = glsr_star_rating( $this->averageRating );
+        return $this->wrap( 'stars', $stars );
     }
 
     /**
@@ -139,11 +139,11 @@ 
      */
     protected function buildText()
     {
-        if ($this->isHidden('summary')) {
+        if( $this->isHidden( 'summary' ) ) {
             return;
         }
-        $count = intval(array_sum($this->ratingCounts));
-        if (empty($this->args['text'])) {
+        $count = intval( array_sum( $this->ratingCounts ) );
+        if( empty($this->args['text']) ) {
             // @todo document this change
             $this->args['text'] = _nx(
                 '{rating} out of {max} stars (based on {num} review)',
@@ -155,10 +155,10 @@ 
         }
         $summary = str_replace(
             ['{rating}', '{max}', '{num}'],
-            [$this->averageRating, glsr()->constant('MAX_RATING', Rating::class), $count],
+            [$this->averageRating, glsr()->constant( 'MAX_RATING', Rating::class ), $count],
             $this->args['text']
         );
-        return $this->wrap('text', '<span>'.$summary.'</span>');
+        return $this->wrap( 'text', '<span>'.$summary.'</span>' );
     }
 
     /**
@@ -166,11 +166,11 @@ 
      */
     protected function generateSchema()
     {
-        if (!wp_validate_boolean($this->args['schema'])) {
+        if( !wp_validate_boolean( $this->args['schema'] ) ) {
             return;
         }
-        glsr(Schema::class)->store(
-            glsr(Schema::class)->buildSummary($this->args)
+        glsr( Schema::class )->store(
+            glsr( Schema::class )->buildSummary( $this->args )
         );
     }
 
@@ -179,16 +179,16 @@ 
      */
     protected function getClass()
     {
-        return trim('glsr-summary glsr-default '.$this->args['class']);
+        return trim( 'glsr-summary glsr-default '.$this->args['class'] );
     }
 
     /**
      * @param string $key
      * @return bool
      */
-    protected function isHidden($key)
+    protected function isHidden( $key )
     {
-        return in_array($key, $this->args['hide']);
+        return in_array( $key, $this->args['hide'] );
     }
 
     /**
@@ -196,11 +196,11 @@ 
      * @param string $value
      * @return string
      */
-    protected function wrap($key, $value)
+    protected function wrap( $key, $value )
     {
-        $value = apply_filters('site-reviews/summary/wrap/'.$key, $value, $this->args);
-        return glsr(Builder::class)->div($value, [
+        $value = apply_filters( 'site-reviews/summary/wrap/'.$key, $value, $this->args );
+        return glsr( Builder::class )->div( $value, [
             'class' => 'glsr-summary-'.$key,
-        ]);
+        ] );
     }
 }

plugin/Modules/Html/Template.php

Indentation +77 added lines, -77 removed lines

@@ -7,86 +7,86 @@
 
 class Template
 {
-    /**
-     * @param string $templatePath
-     * @return void|string
-     */
-    public function build($templatePath, array $data = [])
-    {
-        $data = $this->normalize($data);
-        $path = Str::removePrefix('templates/', $templatePath);
-        $template = glsr()->build($templatePath, $data);
-        $template = apply_filters('site-reviews/build/template/'.$path, $template, $data);
-        $template = $this->interpolate($template, $data, $path);
-        $template = apply_filters('site-reviews/rendered/template', $template, $templatePath, $data);
-        $template = apply_filters('site-reviews/rendered/template/'.$path, $template, $data);
-        return $template;
-    }
+	/**
+	 * @param string $templatePath
+	 * @return void|string
+	 */
+	public function build($templatePath, array $data = [])
+	{
+		$data = $this->normalize($data);
+		$path = Str::removePrefix('templates/', $templatePath);
+		$template = glsr()->build($templatePath, $data);
+		$template = apply_filters('site-reviews/build/template/'.$path, $template, $data);
+		$template = $this->interpolate($template, $data, $path);
+		$template = apply_filters('site-reviews/rendered/template', $template, $templatePath, $data);
+		$template = apply_filters('site-reviews/rendered/template/'.$path, $template, $data);
+		return $template;
+	}
 
-    /**
-     * Interpolate context values into template placeholders.
-     * @param string $template
-     * @param string $templatePath
-     * @return string
-     */
-    public function interpolate($template, array $data = [], $templatePath)
-    {
-        $context = $this->normalizeContext(Arr::get($data, 'context', []));
-        $context = apply_filters('site-reviews/interpolate/'.$templatePath, $context, $template, $data);
-        return $this->interpolateContext($template, $context);
-    }
+	/**
+	 * Interpolate context values into template placeholders.
+	 * @param string $template
+	 * @param string $templatePath
+	 * @return string
+	 */
+	public function interpolate($template, array $data = [], $templatePath)
+	{
+		$context = $this->normalizeContext(Arr::get($data, 'context', []));
+		$context = apply_filters('site-reviews/interpolate/'.$templatePath, $context, $template, $data);
+		return $this->interpolateContext($template, $context);
+	}
 
-    /**
-     * Interpolate context values into template placeholders.
-     * @param string $text
-     * @return string
-     */
-    public function interpolateContext($text, array $context = [])
-    {
-        foreach ($context as $key => $value) {
-            $text = strtr(
-                $text,
-                array_fill_keys(['{'.$key.'}', '{{ '.$key.' }}'], $value)
-            );
-        }
-        return trim($text);
-    }
+	/**
+	 * Interpolate context values into template placeholders.
+	 * @param string $text
+	 * @return string
+	 */
+	public function interpolateContext($text, array $context = [])
+	{
+		foreach ($context as $key => $value) {
+			$text = strtr(
+				$text,
+				array_fill_keys(['{'.$key.'}', '{{ '.$key.' }}'], $value)
+			);
+		}
+		return trim($text);
+	}
 
-    /**
-     * @param string $templatePath
-     * @return void|string
-     */
-    public function render($templatePath, array $data = [])
-    {
-        echo $this->build($templatePath, $data);
-    }
+	/**
+	 * @param string $templatePath
+	 * @return void|string
+	 */
+	public function render($templatePath, array $data = [])
+	{
+		echo $this->build($templatePath, $data);
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalize(array $data)
-    {
-        $arrayKeys = ['context', 'globals'];
-        $data = wp_parse_args($data, array_fill_keys($arrayKeys, []));
-        foreach ($arrayKeys as $key) {
-            if (is_array($data[$key])) {
-                continue;
-            }
-            $data[$key] = [];
-        }
-        return $data;
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalize(array $data)
+	{
+		$arrayKeys = ['context', 'globals'];
+		$data = wp_parse_args($data, array_fill_keys($arrayKeys, []));
+		foreach ($arrayKeys as $key) {
+			if (is_array($data[$key])) {
+				continue;
+			}
+			$data[$key] = [];
+		}
+		return $data;
+	}
 
-    /**
-     * @return array
-     */
-    protected function normalizeContext(array $context)
-    {
-        $context = array_filter($context, function ($value) {
-            return !is_array($value) && !is_object($value);
-        });
-        return array_map(function ($value) {
-            return (string) $value;
-        }, $context);
-    }
+	/**
+	 * @return array
+	 */
+	protected function normalizeContext(array $context)
+	{
+		$context = array_filter($context, function ($value) {
+			return !is_array($value) && !is_object($value);
+		});
+		return array_map(function ($value) {
+			return (string) $value;
+		}, $context);
+	}
 }

Spacing +28 added lines, -28 removed lines

@@ -11,15 +11,15 @@ 
      * @param string $templatePath
      * @return void|string
      */
-    public function build($templatePath, array $data = [])
+    public function build( $templatePath, array $data = [] )
     {
-        $data = $this->normalize($data);
-        $path = Str::removePrefix('templates/', $templatePath);
-        $template = glsr()->build($templatePath, $data);
-        $template = apply_filters('site-reviews/build/template/'.$path, $template, $data);
-        $template = $this->interpolate($template, $data, $path);
-        $template = apply_filters('site-reviews/rendered/template', $template, $templatePath, $data);
-        $template = apply_filters('site-reviews/rendered/template/'.$path, $template, $data);
+        $data = $this->normalize( $data );
+        $path = Str::removePrefix( 'templates/', $templatePath );
+        $template = glsr()->build( $templatePath, $data );
+        $template = apply_filters( 'site-reviews/build/template/'.$path, $template, $data );
+        $template = $this->interpolate( $template, $data, $path );
+        $template = apply_filters( 'site-reviews/rendered/template', $template, $templatePath, $data );
+        $template = apply_filters( 'site-reviews/rendered/template/'.$path, $template, $data );
         return $template;
     }
 
@@ -29,11 +29,11 @@ 
      * @param string $templatePath
      * @return string
      */
-    public function interpolate($template, array $data = [], $templatePath)
+    public function interpolate( $template, array $data = [], $templatePath )
     {
-        $context = $this->normalizeContext(Arr::get($data, 'context', []));
-        $context = apply_filters('site-reviews/interpolate/'.$templatePath, $context, $template, $data);
-        return $this->interpolateContext($template, $context);
+        $context = $this->normalizeContext( Arr::get( $data, 'context', [] ) );
+        $context = apply_filters( 'site-reviews/interpolate/'.$templatePath, $context, $template, $data );
+        return $this->interpolateContext( $template, $context );
     }
 
     /**
@@ -41,35 +41,35 @@ 
      * @param string $text
      * @return string
      */
-    public function interpolateContext($text, array $context = [])
+    public function interpolateContext( $text, array $context = [] )
     {
-        foreach ($context as $key => $value) {
+        foreach( $context as $key => $value ) {
             $text = strtr(
                 $text,
-                array_fill_keys(['{'.$key.'}', '{{ '.$key.' }}'], $value)
+                array_fill_keys( ['{'.$key.'}', '{{ '.$key.' }}'], $value )
             );
         }
-        return trim($text);
+        return trim( $text );
     }
 
     /**
      * @param string $templatePath
      * @return void|string
      */
-    public function render($templatePath, array $data = [])
+    public function render( $templatePath, array $data = [] )
     {
-        echo $this->build($templatePath, $data);
+        echo $this->build( $templatePath, $data );
     }
 
     /**
      * @return array
      */
-    protected function normalize(array $data)
+    protected function normalize( array $data )
     {
         $arrayKeys = ['context', 'globals'];
-        $data = wp_parse_args($data, array_fill_keys($arrayKeys, []));
-        foreach ($arrayKeys as $key) {
-            if (is_array($data[$key])) {
+        $data = wp_parse_args( $data, array_fill_keys( $arrayKeys, [] ) );
+        foreach( $arrayKeys as $key ) {
+            if( is_array( $data[$key] ) ) {
                 continue;
             }
             $data[$key] = [];
@@ -80,13 +80,13 @@ 
     /**
      * @return array
      */
-    protected function normalizeContext(array $context)
+    protected function normalizeContext( array $context )
     {
-        $context = array_filter($context, function ($value) {
-            return !is_array($value) && !is_object($value);
+        $context = array_filter( $context, function( $value ) {
+            return !is_array( $value ) && !is_object( $value );
         });
-        return array_map(function ($value) {
-            return (string) $value;
-        }, $context);
+        return array_map( function( $value ) {
+            return (string)$value;
+        }, $context );
     }
 }

plugin/Helpers/Arr.php

Indentation +211 added lines, -211 removed lines

@@ -4,215 +4,215 @@
 
 class Arr
 {
-    /**
-     * @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
-            : [];
-    }
-
-    /**
-     * @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 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;
-    }
-
-    /**
-     * @param string $key
-     * @return array
-     */
-    public static function insertAfter($key, array $array, array $insert)
-    {
-        return static::insertInArray($array, $insert, $key, 'after');
-    }
-
-    /**
-     * @param string $key
-     * @return array
-     */
-    public static function insertBefore($key, array $array, array $insert)
-    {
-        return static::insertInArray($array, $insert, $key, 'before');
-    }
-
-    /**
-     * @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 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;
-    }
-
-
-    /**
-     * 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 unprefixArrayKeys(array $values)
-    {
-        return static::prefixArrayKeys($values, false);
-    }
+	/**
+	 * @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
+			: [];
+	}
+
+	/**
+	 * @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 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;
+	}
+
+	/**
+	 * @param string $key
+	 * @return array
+	 */
+	public static function insertAfter($key, array $array, array $insert)
+	{
+		return static::insertInArray($array, $insert, $key, 'after');
+	}
+
+	/**
+	 * @param string $key
+	 * @return array
+	 */
+	public static function insertBefore($key, array $array, array $insert)
+	{
+		return static::insertInArray($array, $insert, $key, 'before');
+	}
+
+	/**
+	 * @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 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;
+	}
+
+
+	/**
+	 * 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 unprefixArrayKeys(array $values)
+	{
+		return static::prefixArrayKeys($values, false);
+	}
 }

Spacing +60 added lines, -60 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];
@@ -97,18 +97,18 @@ 
      * @param string $key
      * @return array
      */
-    public static function insertAfter($key, array $array, array $insert)
+    public static function insertAfter( $key, array $array, array $insert )
     {
-        return static::insertInArray($array, $insert, $key, 'after');
+        return static::insertInArray( $array, $insert, $key, 'after' );
     }
 
     /**
      * @param string $key
      * @return array
      */
-    public static function insertBefore($key, array $array, array $insert)
+    public static function insertBefore( $key, array $array, array $insert )
     {
-        return static::insertInArray($array, $insert, $key, 'before');
+        return static::insertInArray( $array, $insert, $key, 'before' );
     }
 
     /**
@@ -116,47 +116,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;
         }
@@ -166,15 +166,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;
@@ -187,14 +187,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;
@@ -203,16 +203,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 );
     }
 }

views/pages/tools/general.php

Spacing +17 added lines, -17 removed lines

@@ -1,42 +1,42 @@ 
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
-<?php if (glsr()->hasPermission('settings')) : ?>
+<?php if( glsr()->hasPermission( 'settings' ) ) : ?>
 <div class="glsr-card card">
     <h3>Export Settings</h3>
     <p>Export the Site Reviews settings for this site to a <code>*.json</code> file. This allows you to easily import the plugin settings into another site.</p>
-    <p>To export your Site Reviews' reviews and categories, please use the WordPress <a href="<?= admin_url('export.php'); ?>">Export</a> tool.</p>
+    <p>To export your Site Reviews' reviews and categories, please use the WordPress <a href="<?= admin_url( 'export.php' ); ?>">Export</a> tool.</p>
     <form method="post">
         <input type="hidden" name="{{ id }}[_action]" value="export-settings">
-        <?php wp_nonce_field('export-settings'); ?>
-        <?php submit_button(__('Export Settings', 'site-reviews'), 'secondary'); ?>
+        <?php wp_nonce_field( 'export-settings' ); ?>
+        <?php submit_button( __( 'Export Settings', 'site-reviews' ), 'secondary' ); ?>
     </form>
 </div>
 <?php endif; ?>
 
-<?php if (glsr()->hasPermission('settings')) : ?>
+<?php if( glsr()->hasPermission( 'settings' ) ) : ?>
 <div class="glsr-card card">
     <h3>Import Settings</h3>
     <p>Import the Site Reviews settings from a <code>*.json</code> file. This file can be obtained by exporting the settings on another site using the export tool below.</p>
-    <p>To import your Site Reviews' reviews and categories from another website, please use the WordPress <a href="<?= admin_url('import.php'); ?>">Import</a> tool.</p>
+    <p>To import your Site Reviews' reviews and categories from another website, please use the WordPress <a href="<?= admin_url( 'import.php' ); ?>">Import</a> tool.</p>
     <form method="post" enctype="multipart/form-data">
         <input type="file" name="import-file">
         <input type="hidden" name="{{ id }}[_action]" value="import-settings">
-        <?php wp_nonce_field('import-settings'); ?>
-        <?php submit_button(__('Import Settings', 'site-reviews'), 'secondary'); ?>
+        <?php wp_nonce_field( 'import-settings' ); ?>
+        <?php submit_button( __( 'Import Settings', 'site-reviews' ), 'secondary' ); ?>
     </form>
 </div>
 <?php endif; ?>
 
-<?php if (glsr()->hasPermission('settings')) : ?>
+<?php if( glsr()->hasPermission( 'settings' ) ) : ?>
 <div class="glsr-card card">
     <h3>Migrate Plugin</h3>
     <p>Run this tool if your reviews stopped working correctly after upgrading the plugin to the latest version (i.e. read-only reviews, zero-star ratings, missing role capabilities, etc.).</p>
     <form method="post">
         <input type="hidden" name="{{ id }}[_action]" value="migrate-reviews">
-        <?php wp_nonce_field('migrate-reviews'); ?>
+        <?php wp_nonce_field( 'migrate-reviews' ); ?>
         <p class="submit">
             <button type="submit" class="glsr-button button" name="migrate-reviews" id="migrate-reviews">
-                <span data-loading="<?= __('Migrating Reviews, please wait...', 'site-reviews'); ?>"><?= __('Run Migration', 'site-reviews'); ?></span>
+                <span data-loading="<?= __( 'Migrating Reviews, please wait...', 'site-reviews' ); ?>"><?= __( 'Run Migration', 'site-reviews' ); ?></span>
             </button>
         </p>
     </form>
@@ -49,26 +49,26 @@ 
     <p>If you suspect that the rating counts are incorrect (perhaps you have cloned a page that had reviews assigned to it, or edited/deleted reviews directly from your database), you can recalculate them here.</p>
     <form method="post">
         <input type="hidden" name="{{ id }}[_action]" value="count-reviews">
-        <?php wp_nonce_field('count-reviews'); ?>
+        <?php wp_nonce_field( 'count-reviews' ); ?>
         <p class="submit">
             <button type="submit" class="glsr-button button" name="count-reviews" id="count-reviews">
-                <span data-loading="<?= __('Recalculating Counts, please wait...', 'site-reviews'); ?>"><?= __('Recalculate Counts', 'site-reviews'); ?></span>
+                <span data-loading="<?= __( 'Recalculating Counts, please wait...', 'site-reviews' ); ?>"><?= __( 'Recalculate Counts', 'site-reviews' ); ?></span>
             </button>
         </p>
     </form>
 </div>
 
-<?php if (glsr()->hasPermission('settings')) : ?>
+<?php if( glsr()->hasPermission( 'settings' ) ) : ?>
 <div class="glsr-card card">
     <h3>Reset Permissions</h3>
     <p>Site Reviews provides custom post_type capabilities that mirror the capabilities of your posts by default. For example, if a user role has permission to edit others posts, then that role will also have permission to edit other users reviews.</p>
     <p>If you have changed the capabilities of your user roles and you suspect that Site Reviews is not working correctly due to your changes, you may use this tool to reset the Site Reviews capabilities for your user roles.</p>
     <form method="post">
         <input type="hidden" name="{{ id }}[_action]" value="reset-permissions">
-        <?php wp_nonce_field('reset-permissions'); ?>
+        <?php wp_nonce_field( 'reset-permissions' ); ?>
         <p class="submit">
             <button type="submit" class="glsr-button button" name="reset-permissions" id="reset-permissions">
-                <span data-loading="<?= __('Resetting Permissions, please wait...', 'site-reviews'); ?>"><?= __('Reset Permissions', 'site-reviews'); ?></span>
+                <span data-loading="<?= __( 'Resetting Permissions, please wait...', 'site-reviews' ); ?>"><?= __( 'Reset Permissions', 'site-reviews' ); ?></span>
             </button>
         </p>
     </form>