pryley / site-reviews

plugin/Handlers/EnqueueAdminAssets.php

Indentation +146 added lines, -146 removed lines

@@ -10,158 +10,158 @@
 
 class EnqueueAdminAssets
 {
-    /**
-     * @var array
-     */
-    protected $pointers;
+	/**
+	 * @var array
+	 */
+	protected $pointers;
 
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        $this->generatePointers($command->pointers);
-        $this->enqueueAssets();
-        $this->localizeAssets();
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		$this->generatePointers($command->pointers);
+		$this->enqueueAssets();
+		$this->localizeAssets();
+	}
 
-    /**
-     * @return void
-     */
-    public function enqueueAssets()
-    {
-        if (!$this->isCurrentScreen()) {
-            return;
-        }
-        wp_enqueue_style(
-            Application::ID,
-            glsr()->url('assets/styles/'.Application::ID.'-admin.css'),
-            [],
-            glsr()->version
-        );
-        wp_enqueue_script(
-            Application::ID,
-            glsr()->url('assets/scripts/'.Application::ID.'-admin.js'),
-            $this->getDependencies(),
-            glsr()->version,
-            true
-        );
-        if (!empty($this->pointers)) {
-            wp_enqueue_style('wp-pointer');
-            wp_enqueue_script('wp-pointer');
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function enqueueAssets()
+	{
+		if (!$this->isCurrentScreen()) {
+			return;
+		}
+		wp_enqueue_style(
+			Application::ID,
+			glsr()->url('assets/styles/'.Application::ID.'-admin.css'),
+			[],
+			glsr()->version
+		);
+		wp_enqueue_script(
+			Application::ID,
+			glsr()->url('assets/scripts/'.Application::ID.'-admin.js'),
+			$this->getDependencies(),
+			glsr()->version,
+			true
+		);
+		if (!empty($this->pointers)) {
+			wp_enqueue_style('wp-pointer');
+			wp_enqueue_script('wp-pointer');
+		}
+	}
 
-    /**
-     * @return void
-     */
-    public function localizeAssets()
-    {
-        $variables = [
-            'action' => Application::PREFIX.'action',
-            'addons' => [],
-            'ajaxurl' => admin_url('admin-ajax.php'),
-            'hideoptions' => [
-                'site_reviews' => glsr(SiteReviewsShortcode::class)->getHideOptions(),
-                'site_reviews_form' => glsr(SiteReviewsFormShortcode::class)->getHideOptions(),
-                'site_reviews_summary' => glsr(SiteReviewsSummaryShortcode::class)->getHideOptions(),
-            ],
-            'nameprefix' => Application::ID,
-            'nonce' => [
-                'change-status' => wp_create_nonce('change-status'),
-                'clear-console' => wp_create_nonce('clear-console'),
-                'count-reviews' => wp_create_nonce('count-reviews'),
-                'fetch-console' => wp_create_nonce('fetch-console'),
-                'mce-shortcode' => wp_create_nonce('mce-shortcode'),
-                'sync-reviews' => wp_create_nonce('sync-reviews'),
-                'toggle-pinned' => wp_create_nonce('toggle-pinned'),
-            ],
-            'pointers' => $this->pointers,
-            'shortcodes' => [],
-            'tinymce' => [
-                'glsr_shortcode' => glsr()->url('assets/scripts/mce-plugin.js'),
-            ],
-        ];
-        if (user_can_richedit()) {
-            $variables['shortcodes'] = $this->localizeShortcodes();
-        }
-        $variables = apply_filters('site-reviews/enqueue/admin/localize', $variables);
-        wp_localize_script(Application::ID, 'GLSR', $variables);
-    }
+	/**
+	 * @return void
+	 */
+	public function localizeAssets()
+	{
+		$variables = [
+			'action' => Application::PREFIX.'action',
+			'addons' => [],
+			'ajaxurl' => admin_url('admin-ajax.php'),
+			'hideoptions' => [
+				'site_reviews' => glsr(SiteReviewsShortcode::class)->getHideOptions(),
+				'site_reviews_form' => glsr(SiteReviewsFormShortcode::class)->getHideOptions(),
+				'site_reviews_summary' => glsr(SiteReviewsSummaryShortcode::class)->getHideOptions(),
+			],
+			'nameprefix' => Application::ID,
+			'nonce' => [
+				'change-status' => wp_create_nonce('change-status'),
+				'clear-console' => wp_create_nonce('clear-console'),
+				'count-reviews' => wp_create_nonce('count-reviews'),
+				'fetch-console' => wp_create_nonce('fetch-console'),
+				'mce-shortcode' => wp_create_nonce('mce-shortcode'),
+				'sync-reviews' => wp_create_nonce('sync-reviews'),
+				'toggle-pinned' => wp_create_nonce('toggle-pinned'),
+			],
+			'pointers' => $this->pointers,
+			'shortcodes' => [],
+			'tinymce' => [
+				'glsr_shortcode' => glsr()->url('assets/scripts/mce-plugin.js'),
+			],
+		];
+		if (user_can_richedit()) {
+			$variables['shortcodes'] = $this->localizeShortcodes();
+		}
+		$variables = apply_filters('site-reviews/enqueue/admin/localize', $variables);
+		wp_localize_script(Application::ID, 'GLSR', $variables);
+	}
 
-    /**
-     * @return array
-     */
-    protected function getDependencies()
-    {
-        $dependencies = apply_filters('site-reviews/enqueue/admin/dependencies', []);
-        $dependencies = array_merge($dependencies, [
-            'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
-        ]);
-        return $dependencies;
-    }
+	/**
+	 * @return array
+	 */
+	protected function getDependencies()
+	{
+		$dependencies = apply_filters('site-reviews/enqueue/admin/dependencies', []);
+		$dependencies = array_merge($dependencies, [
+			'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
+		]);
+		return $dependencies;
+	}
 
-    /**
-     * @return array
-     */
-    protected function generatePointer(array $pointer)
-    {
-        return [
-            'id' => $pointer['id'],
-            'options' => [
-                'content' => '<h3>'.$pointer['title'].'</h3><p>'.$pointer['content'].'</p>',
-                'position' => $pointer['position'],
-            ],
-            'screen' => $pointer['screen'],
-            'target' => $pointer['target'],
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function generatePointer(array $pointer)
+	{
+		return [
+			'id' => $pointer['id'],
+			'options' => [
+				'content' => '<h3>'.$pointer['title'].'</h3><p>'.$pointer['content'].'</p>',
+				'position' => $pointer['position'],
+			],
+			'screen' => $pointer['screen'],
+			'target' => $pointer['target'],
+		];
+	}
 
-    /**
-     * @return void
-     */
-    protected function generatePointers(array $pointers)
-    {
-        $dismissedPointers = get_user_meta(get_current_user_id(), 'dismissed_wp_pointers', true);
-        $dismissedPointers = explode(',', (string) $dismissedPointers);
-        $generatedPointers = [];
-        foreach ($pointers as $pointer) {
-            if ($pointer['screen'] != glsr_current_screen()->id) {
-                continue;
-            }
-            if (in_array($pointer['id'], $dismissedPointers)) {
-                continue;
-            }
-            $generatedPointers[] = $this->generatePointer($pointer);
-        }
-        $this->pointers = $generatedPointers;
-    }
+	/**
+	 * @return void
+	 */
+	protected function generatePointers(array $pointers)
+	{
+		$dismissedPointers = get_user_meta(get_current_user_id(), 'dismissed_wp_pointers', true);
+		$dismissedPointers = explode(',', (string) $dismissedPointers);
+		$generatedPointers = [];
+		foreach ($pointers as $pointer) {
+			if ($pointer['screen'] != glsr_current_screen()->id) {
+				continue;
+			}
+			if (in_array($pointer['id'], $dismissedPointers)) {
+				continue;
+			}
+			$generatedPointers[] = $this->generatePointer($pointer);
+		}
+		$this->pointers = $generatedPointers;
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isCurrentScreen()
-    {
-        $screen = glsr_current_screen();
-        return Application::POST_TYPE == $screen->post_type
-            || 'dashboard' == $screen->id
-            || 'plugins_page_'.Application::ID == $screen->id
-            || 'post' == $screen->base
-            || 'widgets' == $screen->id;
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isCurrentScreen()
+	{
+		$screen = glsr_current_screen();
+		return Application::POST_TYPE == $screen->post_type
+			|| 'dashboard' == $screen->id
+			|| 'plugins_page_'.Application::ID == $screen->id
+			|| 'post' == $screen->base
+			|| 'widgets' == $screen->id;
+	}
 
-    /**
-     * @return array
-     */
-    protected function localizeShortcodes()
-    {
-        $variables = [];
-        foreach (glsr()->mceShortcodes as $tag => $args) {
-            if (empty($args['required'])) {
-                continue;
-            }
-            $variables[$tag] = $args['required'];
-        }
-        return $variables;
-    }
+	/**
+	 * @return array
+	 */
+	protected function localizeShortcodes()
+	{
+		$variables = [];
+		foreach (glsr()->mceShortcodes as $tag => $args) {
+			if (empty($args['required'])) {
+				continue;
+			}
+			$variables[$tag] = $args['required'];
+		}
+		return $variables;
+	}
 }

Spacing +36 added lines, -36 removed lines

@@ -18,9 +18,9 @@ 
     /**
      * @return void
      */
-    public function handle(Command $command)
+    public function handle( Command $command )
     {
-        $this->generatePointers($command->pointers);
+        $this->generatePointers( $command->pointers );
         $this->enqueueAssets();
         $this->localizeAssets();
     }
@@ -30,25 +30,25 @@ 
      */
     public function enqueueAssets()
     {
-        if (!$this->isCurrentScreen()) {
+        if( !$this->isCurrentScreen() ) {
             return;
         }
         wp_enqueue_style(
             Application::ID,
-            glsr()->url('assets/styles/'.Application::ID.'-admin.css'),
+            glsr()->url( 'assets/styles/'.Application::ID.'-admin.css' ),
             [],
             glsr()->version
         );
         wp_enqueue_script(
             Application::ID,
-            glsr()->url('assets/scripts/'.Application::ID.'-admin.js'),
+            glsr()->url( 'assets/scripts/'.Application::ID.'-admin.js' ),
             $this->getDependencies(),
             glsr()->version,
             true
         );
-        if (!empty($this->pointers)) {
-            wp_enqueue_style('wp-pointer');
-            wp_enqueue_script('wp-pointer');
+        if( !empty($this->pointers) ) {
+            wp_enqueue_style( 'wp-pointer' );
+            wp_enqueue_script( 'wp-pointer' );
         }
     }
 
@@ -60,33 +60,33 @@ 
         $variables = [
             'action' => Application::PREFIX.'action',
             'addons' => [],
-            'ajaxurl' => admin_url('admin-ajax.php'),
+            'ajaxurl' => admin_url( 'admin-ajax.php' ),
             'hideoptions' => [
-                'site_reviews' => glsr(SiteReviewsShortcode::class)->getHideOptions(),
-                'site_reviews_form' => glsr(SiteReviewsFormShortcode::class)->getHideOptions(),
-                'site_reviews_summary' => glsr(SiteReviewsSummaryShortcode::class)->getHideOptions(),
+                'site_reviews' => glsr( SiteReviewsShortcode::class )->getHideOptions(),
+                'site_reviews_form' => glsr( SiteReviewsFormShortcode::class )->getHideOptions(),
+                'site_reviews_summary' => glsr( SiteReviewsSummaryShortcode::class )->getHideOptions(),
             ],
             'nameprefix' => Application::ID,
             'nonce' => [
-                'change-status' => wp_create_nonce('change-status'),
-                'clear-console' => wp_create_nonce('clear-console'),
-                'count-reviews' => wp_create_nonce('count-reviews'),
-                'fetch-console' => wp_create_nonce('fetch-console'),
-                'mce-shortcode' => wp_create_nonce('mce-shortcode'),
-                'sync-reviews' => wp_create_nonce('sync-reviews'),
-                'toggle-pinned' => wp_create_nonce('toggle-pinned'),
+                'change-status' => wp_create_nonce( 'change-status' ),
+                'clear-console' => wp_create_nonce( 'clear-console' ),
+                'count-reviews' => wp_create_nonce( 'count-reviews' ),
+                'fetch-console' => wp_create_nonce( 'fetch-console' ),
+                'mce-shortcode' => wp_create_nonce( 'mce-shortcode' ),
+                'sync-reviews' => wp_create_nonce( 'sync-reviews' ),
+                'toggle-pinned' => wp_create_nonce( 'toggle-pinned' ),
             ],
             'pointers' => $this->pointers,
             'shortcodes' => [],
             'tinymce' => [
-                'glsr_shortcode' => glsr()->url('assets/scripts/mce-plugin.js'),
+                'glsr_shortcode' => glsr()->url( 'assets/scripts/mce-plugin.js' ),
             ],
         ];
-        if (user_can_richedit()) {
+        if( user_can_richedit() ) {
             $variables['shortcodes'] = $this->localizeShortcodes();
         }
-        $variables = apply_filters('site-reviews/enqueue/admin/localize', $variables);
-        wp_localize_script(Application::ID, 'GLSR', $variables);
+        $variables = apply_filters( 'site-reviews/enqueue/admin/localize', $variables );
+        wp_localize_script( Application::ID, 'GLSR', $variables );
     }
 
     /**
@@ -94,17 +94,17 @@ 
      */
     protected function getDependencies()
     {
-        $dependencies = apply_filters('site-reviews/enqueue/admin/dependencies', []);
-        $dependencies = array_merge($dependencies, [
+        $dependencies = apply_filters( 'site-reviews/enqueue/admin/dependencies', [] );
+        $dependencies = array_merge( $dependencies, [
             'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
-        ]);
+        ] );
         return $dependencies;
     }
 
     /**
      * @return array
      */
-    protected function generatePointer(array $pointer)
+    protected function generatePointer( array $pointer )
     {
         return [
             'id' => $pointer['id'],
@@ -120,19 +120,19 @@ 
     /**
      * @return void
      */
-    protected function generatePointers(array $pointers)
+    protected function generatePointers( array $pointers )
     {
-        $dismissedPointers = get_user_meta(get_current_user_id(), 'dismissed_wp_pointers', true);
-        $dismissedPointers = explode(',', (string) $dismissedPointers);
+        $dismissedPointers = get_user_meta( get_current_user_id(), 'dismissed_wp_pointers', true );
+        $dismissedPointers = explode( ',', (string)$dismissedPointers );
         $generatedPointers = [];
-        foreach ($pointers as $pointer) {
-            if ($pointer['screen'] != glsr_current_screen()->id) {
+        foreach( $pointers as $pointer ) {
+            if( $pointer['screen'] != glsr_current_screen()->id ) {
                 continue;
             }
-            if (in_array($pointer['id'], $dismissedPointers)) {
+            if( in_array( $pointer['id'], $dismissedPointers ) ) {
                 continue;
             }
-            $generatedPointers[] = $this->generatePointer($pointer);
+            $generatedPointers[] = $this->generatePointer( $pointer );
         }
         $this->pointers = $generatedPointers;
     }
@@ -156,8 +156,8 @@ 
     protected function localizeShortcodes()
     {
         $variables = [];
-        foreach (glsr()->mceShortcodes as $tag => $args) {
-            if (empty($args['required'])) {
+        foreach( glsr()->mceShortcodes as $tag => $args ) {
+            if( empty($args['required']) ) {
                 continue;
             }
             $variables[$tag] = $args['required'];

plugin/Modules/Rating.php

Indentation +191 added lines, -191 removed lines

@@ -4,207 +4,207 @@
 
 class Rating
 {
-    /**
-     * The more sure we are of the confidence interval (the higher the confidence level), the less
-     * precise the estimation will be as the margin for error will be higher.
-     * @see http://homepages.math.uic.edu/~bpower6/stat101/Confidence%20Intervals.pdf
-     * @see https://www.thecalculator.co/math/Confidence-Interval-Calculator-210.html
-     * @see https://www.youtube.com/watch?v=grodoLzThy4
-     * @see https://en.wikipedia.org/wiki/Standard_score
-     * @var array
-     */
-    const CONFIDENCE_LEVEL_Z_SCORES = [
-        50 => 0.67449,
-        70 => 1.04,
-        75 => 1.15035,
-        80 => 1.282,
-        85 => 1.44,
-        90 => 1.64485,
-        92 => 1.75,
-        95 => 1.95996,
-        96 => 2.05,
-        97 => 2.17009,
-        98 => 2.326,
-        99 => 2.57583,
-        '99.5' => 2.81,
-        '99.8' => 3.08,
-        '99.9' => 3.29053,
-    ];
+	/**
+	 * The more sure we are of the confidence interval (the higher the confidence level), the less
+	 * precise the estimation will be as the margin for error will be higher.
+	 * @see http://homepages.math.uic.edu/~bpower6/stat101/Confidence%20Intervals.pdf
+	 * @see https://www.thecalculator.co/math/Confidence-Interval-Calculator-210.html
+	 * @see https://www.youtube.com/watch?v=grodoLzThy4
+	 * @see https://en.wikipedia.org/wiki/Standard_score
+	 * @var array
+	 */
+	const CONFIDENCE_LEVEL_Z_SCORES = [
+		50 => 0.67449,
+		70 => 1.04,
+		75 => 1.15035,
+		80 => 1.282,
+		85 => 1.44,
+		90 => 1.64485,
+		92 => 1.75,
+		95 => 1.95996,
+		96 => 2.05,
+		97 => 2.17009,
+		98 => 2.326,
+		99 => 2.57583,
+		'99.5' => 2.81,
+		'99.8' => 3.08,
+		'99.9' => 3.29053,
+	];
 
-    /**
-     * @var int
-     */
-    const MAX_RATING = 5;
+	/**
+	 * @var int
+	 */
+	const MAX_RATING = 5;
 
-    /**
-     * @var int
-     */
-    const MIN_RATING = 1;
+	/**
+	 * @var int
+	 */
+	const MIN_RATING = 1;
 
-    /**
-     * @param int $roundBy
-     * @return float
-     */
-    public function getAverage(array $ratingCounts, $roundBy = 1)
-    {
-        $average = array_sum($ratingCounts);
-        if ($average > 0) {
-            $average = $this->getTotalSum($ratingCounts) / $average;
-        }
-        $roundedAverage = round($average, intval($roundBy));
-        return floatval(apply_filters('site-reviews/rating/average', $roundedAverage, $ratingCounts, $average));
-    }
+	/**
+	 * @param int $roundBy
+	 * @return float
+	 */
+	public function getAverage(array $ratingCounts, $roundBy = 1)
+	{
+		$average = array_sum($ratingCounts);
+		if ($average > 0) {
+			$average = $this->getTotalSum($ratingCounts) / $average;
+		}
+		$roundedAverage = round($average, intval($roundBy));
+		return floatval(apply_filters('site-reviews/rating/average', $roundedAverage, $ratingCounts, $average));
+	}
 
-    /**
-     * Get the lower bound for up/down ratings
-     * Method receives an up/down ratings array: [1, -1, -1, 1, 1, -1].
-     * @see http://www.evanmiller.org/how-not-to-sort-by-average-rating.html
-     * @see https://news.ycombinator.com/item?id=10481507
-     * @see https://dataorigami.net/blogs/napkin-folding/79030467-an-algorithm-to-sort-top-comments
-     * @see http://julesjacobs.github.io/2015/08/17/bayesian-scoring-of-ratings.html
-     * @param int $confidencePercentage
-     * @return int|float
-     */
-    public function getLowerBound(array $upDownCounts = [0, 0], $confidencePercentage = 95)
-    {
-        $numRatings = array_sum($upDownCounts);
-        if ($numRatings < 1) {
-            return 0;
-        }
-        $z = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
-        $phat = 1 * $upDownCounts[1] / $numRatings;
-        return ($phat + $z * $z / (2 * $numRatings) - $z * sqrt(($phat * (1 - $phat) + $z * $z / (4 * $numRatings)) / $numRatings)) / (1 + $z * $z / $numRatings);
-    }
+	/**
+	 * Get the lower bound for up/down ratings
+	 * Method receives an up/down ratings array: [1, -1, -1, 1, 1, -1].
+	 * @see http://www.evanmiller.org/how-not-to-sort-by-average-rating.html
+	 * @see https://news.ycombinator.com/item?id=10481507
+	 * @see https://dataorigami.net/blogs/napkin-folding/79030467-an-algorithm-to-sort-top-comments
+	 * @see http://julesjacobs.github.io/2015/08/17/bayesian-scoring-of-ratings.html
+	 * @param int $confidencePercentage
+	 * @return int|float
+	 */
+	public function getLowerBound(array $upDownCounts = [0, 0], $confidencePercentage = 95)
+	{
+		$numRatings = array_sum($upDownCounts);
+		if ($numRatings < 1) {
+			return 0;
+		}
+		$z = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
+		$phat = 1 * $upDownCounts[1] / $numRatings;
+		return ($phat + $z * $z / (2 * $numRatings) - $z * sqrt(($phat * (1 - $phat) + $z * $z / (4 * $numRatings)) / $numRatings)) / (1 + $z * $z / $numRatings);
+	}
 
-    /**
-     * @return int|float
-     */
-    public function getOverallPercentage(array $ratingCounts)
-    {
-        return round($this->getAverage($ratingCounts) * 100 / glsr()->constant('MAX_RATING', __CLASS__), 2);
-    }
+	/**
+	 * @return int|float
+	 */
+	public function getOverallPercentage(array $ratingCounts)
+	{
+		return round($this->getAverage($ratingCounts) * 100 / glsr()->constant('MAX_RATING', __CLASS__), 2);
+	}
 
-    /**
-     * @return array
-     */
-    public function getPercentages(array $ratingCounts)
-    {
-        $total = array_sum($ratingCounts);
-        foreach ($ratingCounts as $index => $count) {
-            if (empty($count)) {
-                continue;
-            }
-            $ratingCounts[$index] = $count / $total * 100;
-        }
-        return $this->getRoundedPercentages($ratingCounts);
-    }
+	/**
+	 * @return array
+	 */
+	public function getPercentages(array $ratingCounts)
+	{
+		$total = array_sum($ratingCounts);
+		foreach ($ratingCounts as $index => $count) {
+			if (empty($count)) {
+				continue;
+			}
+			$ratingCounts[$index] = $count / $total * 100;
+		}
+		return $this->getRoundedPercentages($ratingCounts);
+	}
 
-    /**
-     * @return float
-     */
-    public function getRanking(array $ratingCounts)
-    {
-        return floatval(apply_filters('site-reviews/rating/ranking',
-            $this->getRankingUsingImdb($ratingCounts),
-            $ratingCounts,
-            $this
-        ));
-    }
+	/**
+	 * @return float
+	 */
+	public function getRanking(array $ratingCounts)
+	{
+		return floatval(apply_filters('site-reviews/rating/ranking',
+			$this->getRankingUsingImdb($ratingCounts),
+			$ratingCounts,
+			$this
+		));
+	}
 
-    /**
-     * Get the bayesian ranking for an array of reviews
-     * This formula is the same one used by IMDB to rank their top 250 films.
-     * @see https://www.xkcd.com/937/
-     * @see https://districtdatalabs.silvrback.com/computing-a-bayesian-estimate-of-star-rating-means
-     * @see http://fulmicoton.com/posts/bayesian_rating/
-     * @see https://stats.stackexchange.com/questions/93974/is-there-an-equivalent-to-lower-bound-of-wilson-score-confidence-interval-for-va
-     * @param int $confidencePercentage
-     * @return int|float
-     */
-    public function getRankingUsingImdb(array $ratingCounts, $confidencePercentage = 70)
-    {
-        $avgRating = $this->getAverage($ratingCounts);
-        // Represents a prior (your prior opinion without data) for the average star rating. A higher prior also means a higher margin for error.
-        // This could also be the average score of all items instead of a fixed value.
-        $bayesMean = ($confidencePercentage / 100) * glsr()->constant('MAX_RATING', __CLASS__); // prior, 70% = 3.5
-        // Represents the number of ratings expected to begin observing a pattern that would put confidence in the prior.
-        $bayesMinimal = 10; // confidence
-        $numOfReviews = array_sum($ratingCounts);
-        return $avgRating > 0
-            ? (($bayesMinimal * $bayesMean) + ($avgRating * $numOfReviews)) / ($bayesMinimal + $numOfReviews)
-            : 0;
-    }
+	/**
+	 * Get the bayesian ranking for an array of reviews
+	 * This formula is the same one used by IMDB to rank their top 250 films.
+	 * @see https://www.xkcd.com/937/
+	 * @see https://districtdatalabs.silvrback.com/computing-a-bayesian-estimate-of-star-rating-means
+	 * @see http://fulmicoton.com/posts/bayesian_rating/
+	 * @see https://stats.stackexchange.com/questions/93974/is-there-an-equivalent-to-lower-bound-of-wilson-score-confidence-interval-for-va
+	 * @param int $confidencePercentage
+	 * @return int|float
+	 */
+	public function getRankingUsingImdb(array $ratingCounts, $confidencePercentage = 70)
+	{
+		$avgRating = $this->getAverage($ratingCounts);
+		// Represents a prior (your prior opinion without data) for the average star rating. A higher prior also means a higher margin for error.
+		// This could also be the average score of all items instead of a fixed value.
+		$bayesMean = ($confidencePercentage / 100) * glsr()->constant('MAX_RATING', __CLASS__); // prior, 70% = 3.5
+		// Represents the number of ratings expected to begin observing a pattern that would put confidence in the prior.
+		$bayesMinimal = 10; // confidence
+		$numOfReviews = array_sum($ratingCounts);
+		return $avgRating > 0
+			? (($bayesMinimal * $bayesMean) + ($avgRating * $numOfReviews)) / ($bayesMinimal + $numOfReviews)
+			: 0;
+	}
 
-    /**
-     * The quality of a 5 star rating depends not only on the average number of stars but also on
-     * the number of reviews. This method calculates the bayesian ranking of a page by its number
-     * of reviews and their rating.
-     * @see http://www.evanmiller.org/ranking-items-with-star-ratings.html
-     * @see https://stackoverflow.com/questions/1411199/what-is-a-better-way-to-sort-by-a-5-star-rating/1411268
-     * @see http://julesjacobs.github.io/2015/08/17/bayesian-scoring-of-ratings.html
-     * @param int $confidencePercentage
-     * @return float
-     */
-    public function getRankingUsingZScores(array $ratingCounts, $confidencePercentage = 90)
-    {
-        $ratingCountsSum = array_sum($ratingCounts) + glsr()->constant('MAX_RATING', __CLASS__);
-        $weight = $this->getWeight($ratingCounts, $ratingCountsSum);
-        $weightPow2 = $this->getWeight($ratingCounts, $ratingCountsSum, true);
-        $zScore = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
-        return $weight - $zScore * sqrt(($weightPow2 - pow($weight, 2)) / ($ratingCountsSum + 1));
-    }
+	/**
+	 * The quality of a 5 star rating depends not only on the average number of stars but also on
+	 * the number of reviews. This method calculates the bayesian ranking of a page by its number
+	 * of reviews and their rating.
+	 * @see http://www.evanmiller.org/ranking-items-with-star-ratings.html
+	 * @see https://stackoverflow.com/questions/1411199/what-is-a-better-way-to-sort-by-a-5-star-rating/1411268
+	 * @see http://julesjacobs.github.io/2015/08/17/bayesian-scoring-of-ratings.html
+	 * @param int $confidencePercentage
+	 * @return float
+	 */
+	public function getRankingUsingZScores(array $ratingCounts, $confidencePercentage = 90)
+	{
+		$ratingCountsSum = array_sum($ratingCounts) + glsr()->constant('MAX_RATING', __CLASS__);
+		$weight = $this->getWeight($ratingCounts, $ratingCountsSum);
+		$weightPow2 = $this->getWeight($ratingCounts, $ratingCountsSum, true);
+		$zScore = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
+		return $weight - $zScore * sqrt(($weightPow2 - pow($weight, 2)) / ($ratingCountsSum + 1));
+	}
 
-    /**
-     * @param int $target
-     * @return array
-     */
-    protected function getRoundedPercentages(array $percentages, $totalPercent = 100)
-    {
-        array_walk($percentages, function (&$percent, $index) {
-            $percent = [
-                'index' => $index,
-                'percent' => floor($percent),
-                'remainder' => fmod($percent, 1),
-            ];
-        });
-        $indexes = glsr_array_column($percentages, 'index');
-        $remainders = glsr_array_column($percentages, 'remainder');
-        array_multisort($remainders, SORT_DESC, SORT_STRING, $indexes, SORT_DESC, $percentages);
-        $i = 0;
-        if (array_sum(glsr_array_column($percentages, 'percent')) > 0) {
-            while (array_sum(glsr_array_column($percentages, 'percent')) < $totalPercent) {
-                ++$percentages[$i]['percent'];
-                ++$i;
-            }
-        }
-        array_multisort($indexes, SORT_DESC, $percentages);
-        return array_combine($indexes, glsr_array_column($percentages, 'percent'));
-    }
+	/**
+	 * @param int $target
+	 * @return array
+	 */
+	protected function getRoundedPercentages(array $percentages, $totalPercent = 100)
+	{
+		array_walk($percentages, function (&$percent, $index) {
+			$percent = [
+				'index' => $index,
+				'percent' => floor($percent),
+				'remainder' => fmod($percent, 1),
+			];
+		});
+		$indexes = glsr_array_column($percentages, 'index');
+		$remainders = glsr_array_column($percentages, 'remainder');
+		array_multisort($remainders, SORT_DESC, SORT_STRING, $indexes, SORT_DESC, $percentages);
+		$i = 0;
+		if (array_sum(glsr_array_column($percentages, 'percent')) > 0) {
+			while (array_sum(glsr_array_column($percentages, 'percent')) < $totalPercent) {
+				++$percentages[$i]['percent'];
+				++$i;
+			}
+		}
+		array_multisort($indexes, SORT_DESC, $percentages);
+		return array_combine($indexes, glsr_array_column($percentages, 'percent'));
+	}
 
-    /**
-     * @return int
-     */
-    protected function getTotalSum(array $ratingCounts)
-    {
-        return array_reduce(array_keys($ratingCounts), function ($carry, $index) use ($ratingCounts) {
-            return $carry + ($index * $ratingCounts[$index]);
-        });
-    }
+	/**
+	 * @return int
+	 */
+	protected function getTotalSum(array $ratingCounts)
+	{
+		return array_reduce(array_keys($ratingCounts), function ($carry, $index) use ($ratingCounts) {
+			return $carry + ($index * $ratingCounts[$index]);
+		});
+	}
 
-    /**
-     * @param int|float $ratingCountsSum
-     * @param bool $powerOf2
-     * @return float
-     */
-    protected function getWeight(array $ratingCounts, $ratingCountsSum, $powerOf2 = false)
-    {
-        return array_reduce(array_keys($ratingCounts),
-            function ($count, $rating) use ($ratingCounts, $ratingCountsSum, $powerOf2) {
-                $ratingLevel = $powerOf2
-                    ? pow($rating, 2)
-                    : $rating;
-                return $count + ($ratingLevel * ($ratingCounts[$rating] + 1)) / $ratingCountsSum;
-            }
-        );
-    }
+	/**
+	 * @param int|float $ratingCountsSum
+	 * @param bool $powerOf2
+	 * @return float
+	 */
+	protected function getWeight(array $ratingCounts, $ratingCountsSum, $powerOf2 = false)
+	{
+		return array_reduce(array_keys($ratingCounts),
+			function ($count, $rating) use ($ratingCounts, $ratingCountsSum, $powerOf2) {
+				$ratingLevel = $powerOf2
+					? pow($rating, 2)
+					: $rating;
+				return $count + ($ratingLevel * ($ratingCounts[$rating] + 1)) / $ratingCountsSum;
+			}
+		);
+	}
 }

Spacing +47 added lines, -47 removed lines

@@ -45,14 +45,14 @@ 
      * @param int $roundBy
      * @return float
      */
-    public function getAverage(array $ratingCounts, $roundBy = 1)
+    public function getAverage( array $ratingCounts, $roundBy = 1 )
     {
-        $average = array_sum($ratingCounts);
-        if ($average > 0) {
-            $average = $this->getTotalSum($ratingCounts) / $average;
+        $average = array_sum( $ratingCounts );
+        if( $average > 0 ) {
+            $average = $this->getTotalSum( $ratingCounts ) / $average;
         }
-        $roundedAverage = round($average, intval($roundBy));
-        return floatval(apply_filters('site-reviews/rating/average', $roundedAverage, $ratingCounts, $average));
+        $roundedAverage = round( $average, intval( $roundBy ) );
+        return floatval( apply_filters( 'site-reviews/rating/average', $roundedAverage, $ratingCounts, $average ) );
     }
 
     /**
@@ -65,50 +65,50 @@ 
      * @param int $confidencePercentage
      * @return int|float
      */
-    public function getLowerBound(array $upDownCounts = [0, 0], $confidencePercentage = 95)
+    public function getLowerBound( array $upDownCounts = [0, 0], $confidencePercentage = 95 )
     {
-        $numRatings = array_sum($upDownCounts);
-        if ($numRatings < 1) {
+        $numRatings = array_sum( $upDownCounts );
+        if( $numRatings < 1 ) {
             return 0;
         }
         $z = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
         $phat = 1 * $upDownCounts[1] / $numRatings;
-        return ($phat + $z * $z / (2 * $numRatings) - $z * sqrt(($phat * (1 - $phat) + $z * $z / (4 * $numRatings)) / $numRatings)) / (1 + $z * $z / $numRatings);
+        return ($phat + $z * $z / (2 * $numRatings) - $z * sqrt( ($phat * (1 - $phat) + $z * $z / (4 * $numRatings)) / $numRatings )) / (1 + $z * $z / $numRatings);
     }
 
     /**
      * @return int|float
      */
-    public function getOverallPercentage(array $ratingCounts)
+    public function getOverallPercentage( array $ratingCounts )
     {
-        return round($this->getAverage($ratingCounts) * 100 / glsr()->constant('MAX_RATING', __CLASS__), 2);
+        return round( $this->getAverage( $ratingCounts ) * 100 / glsr()->constant( 'MAX_RATING', __CLASS__ ), 2 );
     }
 
     /**
      * @return array
      */
-    public function getPercentages(array $ratingCounts)
+    public function getPercentages( array $ratingCounts )
     {
-        $total = array_sum($ratingCounts);
-        foreach ($ratingCounts as $index => $count) {
-            if (empty($count)) {
+        $total = array_sum( $ratingCounts );
+        foreach( $ratingCounts as $index => $count ) {
+            if( empty($count) ) {
                 continue;
             }
             $ratingCounts[$index] = $count / $total * 100;
         }
-        return $this->getRoundedPercentages($ratingCounts);
+        return $this->getRoundedPercentages( $ratingCounts );
     }
 
     /**
      * @return float
      */
-    public function getRanking(array $ratingCounts)
+    public function getRanking( array $ratingCounts )
     {
-        return floatval(apply_filters('site-reviews/rating/ranking',
-            $this->getRankingUsingImdb($ratingCounts),
+        return floatval( apply_filters( 'site-reviews/rating/ranking',
+            $this->getRankingUsingImdb( $ratingCounts ),
             $ratingCounts,
             $this
-        ));
+        ) );
     }
 
     /**
@@ -121,15 +121,15 @@ 
      * @param int $confidencePercentage
      * @return int|float
      */
-    public function getRankingUsingImdb(array $ratingCounts, $confidencePercentage = 70)
+    public function getRankingUsingImdb( array $ratingCounts, $confidencePercentage = 70 )
     {
-        $avgRating = $this->getAverage($ratingCounts);
+        $avgRating = $this->getAverage( $ratingCounts );
         // Represents a prior (your prior opinion without data) for the average star rating. A higher prior also means a higher margin for error.
         // This could also be the average score of all items instead of a fixed value.
-        $bayesMean = ($confidencePercentage / 100) * glsr()->constant('MAX_RATING', __CLASS__); // prior, 70% = 3.5
+        $bayesMean = ($confidencePercentage / 100) * glsr()->constant( 'MAX_RATING', __CLASS__ ); // prior, 70% = 3.5
         // Represents the number of ratings expected to begin observing a pattern that would put confidence in the prior.
         $bayesMinimal = 10; // confidence
-        $numOfReviews = array_sum($ratingCounts);
+        $numOfReviews = array_sum( $ratingCounts );
         return $avgRating > 0
             ? (($bayesMinimal * $bayesMean) + ($avgRating * $numOfReviews)) / ($bayesMinimal + $numOfReviews)
             : 0;
@@ -145,48 +145,48 @@ 
      * @param int $confidencePercentage
      * @return float
      */
-    public function getRankingUsingZScores(array $ratingCounts, $confidencePercentage = 90)
+    public function getRankingUsingZScores( array $ratingCounts, $confidencePercentage = 90 )
     {
-        $ratingCountsSum = array_sum($ratingCounts) + glsr()->constant('MAX_RATING', __CLASS__);
-        $weight = $this->getWeight($ratingCounts, $ratingCountsSum);
-        $weightPow2 = $this->getWeight($ratingCounts, $ratingCountsSum, true);
+        $ratingCountsSum = array_sum( $ratingCounts ) + glsr()->constant( 'MAX_RATING', __CLASS__ );
+        $weight = $this->getWeight( $ratingCounts, $ratingCountsSum );
+        $weightPow2 = $this->getWeight( $ratingCounts, $ratingCountsSum, true );
         $zScore = static::CONFIDENCE_LEVEL_Z_SCORES[$confidencePercentage];
-        return $weight - $zScore * sqrt(($weightPow2 - pow($weight, 2)) / ($ratingCountsSum + 1));
+        return $weight - $zScore * sqrt( ($weightPow2 - pow( $weight, 2 )) / ($ratingCountsSum + 1) );
     }
 
     /**
      * @param int $target
      * @return array
      */
-    protected function getRoundedPercentages(array $percentages, $totalPercent = 100)
+    protected function getRoundedPercentages( array $percentages, $totalPercent = 100 )
     {
-        array_walk($percentages, function (&$percent, $index) {
+        array_walk( $percentages, function( &$percent, $index ) {
             $percent = [
                 'index' => $index,
-                'percent' => floor($percent),
-                'remainder' => fmod($percent, 1),
+                'percent' => floor( $percent ),
+                'remainder' => fmod( $percent, 1 ),
             ];
         });
-        $indexes = glsr_array_column($percentages, 'index');
-        $remainders = glsr_array_column($percentages, 'remainder');
-        array_multisort($remainders, SORT_DESC, SORT_STRING, $indexes, SORT_DESC, $percentages);
+        $indexes = glsr_array_column( $percentages, 'index' );
+        $remainders = glsr_array_column( $percentages, 'remainder' );
+        array_multisort( $remainders, SORT_DESC, SORT_STRING, $indexes, SORT_DESC, $percentages );
         $i = 0;
-        if (array_sum(glsr_array_column($percentages, 'percent')) > 0) {
-            while (array_sum(glsr_array_column($percentages, 'percent')) < $totalPercent) {
+        if( array_sum( glsr_array_column( $percentages, 'percent' ) ) > 0 ) {
+            while( array_sum( glsr_array_column( $percentages, 'percent' ) ) < $totalPercent ) {
                 ++$percentages[$i]['percent'];
                 ++$i;
             }
         }
-        array_multisort($indexes, SORT_DESC, $percentages);
-        return array_combine($indexes, glsr_array_column($percentages, 'percent'));
+        array_multisort( $indexes, SORT_DESC, $percentages );
+        return array_combine( $indexes, glsr_array_column( $percentages, 'percent' ) );
     }
 
     /**
      * @return int
      */
-    protected function getTotalSum(array $ratingCounts)
+    protected function getTotalSum( array $ratingCounts )
     {
-        return array_reduce(array_keys($ratingCounts), function ($carry, $index) use ($ratingCounts) {
+        return array_reduce( array_keys( $ratingCounts ), function( $carry, $index ) use ($ratingCounts) {
             return $carry + ($index * $ratingCounts[$index]);
         });
     }
@@ -196,12 +196,12 @@ 
      * @param bool $powerOf2
      * @return float
      */
-    protected function getWeight(array $ratingCounts, $ratingCountsSum, $powerOf2 = false)
+    protected function getWeight( array $ratingCounts, $ratingCountsSum, $powerOf2 = false )
     {
-        return array_reduce(array_keys($ratingCounts),
-            function ($count, $rating) use ($ratingCounts, $ratingCountsSum, $powerOf2) {
+        return array_reduce( array_keys( $ratingCounts ),
+            function( $count, $rating ) use ($ratingCounts, $ratingCountsSum, $powerOf2) {
                 $ratingLevel = $powerOf2
-                    ? pow($rating, 2)
+                    ? pow( $rating, 2 )
                     : $rating;
                 return $count + ($ratingLevel * ($ratingCounts[$rating] + 1)) / $ratingCountsSum;
             }

plugin/Database/SqlQueries.php

Indentation +106 added lines, -106 removed lines

@@ -8,23 +8,23 @@ 
 
 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;
+	}
 
-    /**
-     * @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
@@ -32,17 +32,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
@@ -57,17 +57,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
@@ -75,16 +75,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
@@ -94,20 +94,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
@@ -120,20 +120,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
@@ -144,42 +144,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 +44 added lines, -44 removed lines

@@ -22,17 +22,17 @@ 
      * @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 );
     }
 
     /**
@@ -40,23 +40,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}
-        ");
+        " );
     }
 
     /**
@@ -64,17 +64,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
-        ");
+        " );
     }
 
     /**
@@ -82,9 +82,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
@@ -93,8 +93,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 ) );
     }
 
     /**
@@ -102,12 +102,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
@@ -119,7 +119,7 @@ 
             GROUP BY p.ID
             ORDER By p.ID ASC
             LIMIT {$limit}
-        ");
+        " );
     }
 
     /**
@@ -127,13 +127,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
@@ -143,8 +143,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;
     }
 
@@ -152,34 +152,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/Shortcodes/SiteReviewsFormShortcode.php

Indentation +24 added lines, -24 removed lines

@@ -6,29 +6,29 @@
 
 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'),
+		];
+	}
 
-    /**
-     * @param array|string $atts
-     * @param string $type
-     * @return array
-     */
-    public function normalizeAtts($atts, $type = 'shortcode')
-    {
-        $atts = parent::normalizeAtts($atts, $type);
-        if (empty($atts['id'])) {
-            $atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
-        }
-        return $atts;
-    }
+	/**
+	 * @param array|string $atts
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeAtts($atts, $type = 'shortcode')
+	{
+		$atts = parent::normalizeAtts($atts, $type);
+		if (empty($atts['id'])) {
+			$atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
+		}
+		return $atts;
+	}
 }

Spacing +10 added lines, -10 removed lines

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

plugin/Database/QueryBuilder.php

Indentation +176 added lines, -176 removed lines

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

Spacing +46 added lines, -46 removed lines

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

plugin/Modules/Multilingual.php

Indentation +45 added lines, -45 removed lines

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

Spacing +14 added lines, -14 removed lines

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

views/pages/settings/translations.php

Spacing +9 added lines, -9 removed lines

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

plugin/Modules/ReviewLimits.php

Indentation +93 added lines, -93 removed lines

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

Spacing +29 added lines, -29 removed lines

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

plugin/Modules/Validator/ValidationRules.php

Indentation +113 added lines, -113 removed lines

@@ -10,126 +10,126 @@
  */
 trait ValidationRules
 {
-    /**
-     * Get the size of an attribute.
-     * @param string $attribute
-     * @param mixed $value
-     * @return mixed
-     */
-    abstract protected function getSize($attribute, $value);
+	/**
+	 * Get the size of an attribute.
+	 * @param string $attribute
+	 * @param mixed $value
+	 * @return mixed
+	 */
+	abstract protected function getSize($attribute, $value);
 
-    /**
-     * Replace all placeholders.
-     * @param string $message
-     * @return string
-     */
-    protected function replace($message, array $parameters)
-    {
-        if (!Str::contains($message, '%s')) {
-            return $message;
-        }
-        return preg_replace_callback('/(%s)/', function () use (&$parameters) {
-            foreach ($parameters as $key => $value) {
-                return array_shift($parameters);
-            }
-        }, $message);
-    }
+	/**
+	 * Replace all placeholders.
+	 * @param string $message
+	 * @return string
+	 */
+	protected function replace($message, array $parameters)
+	{
+		if (!Str::contains($message, '%s')) {
+			return $message;
+		}
+		return preg_replace_callback('/(%s)/', function () use (&$parameters) {
+			foreach ($parameters as $key => $value) {
+				return array_shift($parameters);
+			}
+		}, $message);
+	}
 
-    /**
-     * Validate that an attribute was "accepted".
-     * This validation rule implies the attribute is "required".
-     * @param string $attribute
-     * @param mixed $value
-     * @return bool
-     */
-    public function validateAccepted($value)
-    {
-        $acceptable = ['yes', 'on', '1', 1, true, 'true'];
-        return $this->validateRequired($value) && in_array($value, $acceptable, true);
-    }
+	/**
+	 * Validate that an attribute was "accepted".
+	 * This validation rule implies the attribute is "required".
+	 * @param string $attribute
+	 * @param mixed $value
+	 * @return bool
+	 */
+	public function validateAccepted($value)
+	{
+		$acceptable = ['yes', 'on', '1', 1, true, 'true'];
+		return $this->validateRequired($value) && in_array($value, $acceptable, true);
+	}
 
-    /**
-     * Validate the size of an attribute is between a set of values.
-     * @param string $attribute
-     * @param mixed $value
-     * @return bool
-     */
-    public function validateBetween($value, $attribute, array $parameters)
-    {
-        $this->requireParameterCount(2, $parameters, 'between');
-        $size = $this->getSize($attribute, $value);
-        return $size >= $parameters[0] && $size <= $parameters[1];
-    }
+	/**
+	 * Validate the size of an attribute is between a set of values.
+	 * @param string $attribute
+	 * @param mixed $value
+	 * @return bool
+	 */
+	public function validateBetween($value, $attribute, array $parameters)
+	{
+		$this->requireParameterCount(2, $parameters, 'between');
+		$size = $this->getSize($attribute, $value);
+		return $size >= $parameters[0] && $size <= $parameters[1];
+	}
 
-    /**
-     * Validate that an attribute is a valid e-mail address.
-     * @param mixed $value
-     * @return bool
-     */
-    public function validateEmail($value)
-    {
-        return false !== filter_var($value, FILTER_VALIDATE_EMAIL);
-    }
+	/**
+	 * Validate that an attribute is a valid e-mail address.
+	 * @param mixed $value
+	 * @return bool
+	 */
+	public function validateEmail($value)
+	{
+		return false !== filter_var($value, FILTER_VALIDATE_EMAIL);
+	}
 
-    /**
-     * Validate the size of an attribute is less than a maximum value.
-     * @param string $attribute
-     * @param mixed $value
-     * @return bool
-     */
-    public function validateMax($value, $attribute, array $parameters)
-    {
-        $this->requireParameterCount(1, $parameters, 'max');
-        return $this->getSize($attribute, $value) <= $parameters[0];
-    }
+	/**
+	 * Validate the size of an attribute is less than a maximum value.
+	 * @param string $attribute
+	 * @param mixed $value
+	 * @return bool
+	 */
+	public function validateMax($value, $attribute, array $parameters)
+	{
+		$this->requireParameterCount(1, $parameters, 'max');
+		return $this->getSize($attribute, $value) <= $parameters[0];
+	}
 
-    /**
-     * Validate the size of an attribute is greater than a minimum value.
-     * @param string $attribute
-     * @param mixed $value
-     * @return bool
-     */
-    public function validateMin($value, $attribute, array $parameters)
-    {
-        $this->requireParameterCount(1, $parameters, 'min');
-        return $this->getSize($attribute, $value) >= $parameters[0];
-    }
+	/**
+	 * Validate the size of an attribute is greater than a minimum value.
+	 * @param string $attribute
+	 * @param mixed $value
+	 * @return bool
+	 */
+	public function validateMin($value, $attribute, array $parameters)
+	{
+		$this->requireParameterCount(1, $parameters, 'min');
+		return $this->getSize($attribute, $value) >= $parameters[0];
+	}
 
-    /**
-     * Validate that an attribute is numeric.
-     * @param mixed $value
-     * @return bool
-     */
-    public function validateNumber($value)
-    {
-        return is_numeric($value);
-    }
+	/**
+	 * Validate that an attribute is numeric.
+	 * @param mixed $value
+	 * @return bool
+	 */
+	public function validateNumber($value)
+	{
+		return is_numeric($value);
+	}
 
-    /**
-     * Validate that a required attribute exists.
-     * @param mixed $value
-     * @return bool
-     */
-    public function validateRequired($value)
-    {
-        return is_null($value)
-            || (is_string($value) && in_array(trim($value), ['', '[]']))
-            || (is_array($value) && empty($value))
-            ? false
-            : true;
-    }
+	/**
+	 * Validate that a required attribute exists.
+	 * @param mixed $value
+	 * @return bool
+	 */
+	public function validateRequired($value)
+	{
+		return is_null($value)
+			|| (is_string($value) && in_array(trim($value), ['', '[]']))
+			|| (is_array($value) && empty($value))
+			? false
+			: true;
+	}
 
-    /**
-     * Require a certain number of parameters to be present.
-     * @param int $count
-     * @param string $rule
-     * @return void
-     * @throws InvalidArgumentException
-     */
-    protected function requireParameterCount($count, array $parameters, $rule)
-    {
-        if (count($parameters) < $count) {
-            throw new InvalidArgumentException("Validation rule $rule requires at least $count parameters.");
-        }
-    }
+	/**
+	 * Require a certain number of parameters to be present.
+	 * @param int $count
+	 * @param string $rule
+	 * @return void
+	 * @throws InvalidArgumentException
+	 */
+	protected function requireParameterCount($count, array $parameters, $rule)
+	{
+		if (count($parameters) < $count) {
+			throw new InvalidArgumentException("Validation rule $rule requires at least $count parameters.");
+		}
+	}
 }

Spacing +29 added lines, -29 removed lines

@@ -16,23 +16,23 @@ 
      * @param mixed $value
      * @return mixed
      */
-    abstract protected function getSize($attribute, $value);
+    abstract protected function getSize( $attribute, $value );
 
     /**
      * Replace all placeholders.
      * @param string $message
      * @return string
      */
-    protected function replace($message, array $parameters)
+    protected function replace( $message, array $parameters )
     {
-        if (!Str::contains($message, '%s')) {
+        if( !Str::contains( $message, '%s' ) ) {
             return $message;
         }
-        return preg_replace_callback('/(%s)/', function () use (&$parameters) {
-            foreach ($parameters as $key => $value) {
-                return array_shift($parameters);
+        return preg_replace_callback( '/(%s)/', function() use (&$parameters) {
+            foreach( $parameters as $key => $value ) {
+                return array_shift( $parameters );
             }
-        }, $message);
+        }, $message );
     }
 
     /**
@@ -42,10 +42,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateAccepted($value)
+    public function validateAccepted( $value )
     {
         $acceptable = ['yes', 'on', '1', 1, true, 'true'];
-        return $this->validateRequired($value) && in_array($value, $acceptable, true);
+        return $this->validateRequired( $value ) && in_array( $value, $acceptable, true );
     }
 
     /**
@@ -54,10 +54,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateBetween($value, $attribute, array $parameters)
+    public function validateBetween( $value, $attribute, array $parameters )
     {
-        $this->requireParameterCount(2, $parameters, 'between');
-        $size = $this->getSize($attribute, $value);
+        $this->requireParameterCount( 2, $parameters, 'between' );
+        $size = $this->getSize( $attribute, $value );
         return $size >= $parameters[0] && $size <= $parameters[1];
     }
 
@@ -66,9 +66,9 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateEmail($value)
+    public function validateEmail( $value )
     {
-        return false !== filter_var($value, FILTER_VALIDATE_EMAIL);
+        return false !== filter_var( $value, FILTER_VALIDATE_EMAIL );
     }
 
     /**
@@ -77,10 +77,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateMax($value, $attribute, array $parameters)
+    public function validateMax( $value, $attribute, array $parameters )
     {
-        $this->requireParameterCount(1, $parameters, 'max');
-        return $this->getSize($attribute, $value) <= $parameters[0];
+        $this->requireParameterCount( 1, $parameters, 'max' );
+        return $this->getSize( $attribute, $value ) <= $parameters[0];
     }
 
     /**
@@ -89,10 +89,10 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateMin($value, $attribute, array $parameters)
+    public function validateMin( $value, $attribute, array $parameters )
     {
-        $this->requireParameterCount(1, $parameters, 'min');
-        return $this->getSize($attribute, $value) >= $parameters[0];
+        $this->requireParameterCount( 1, $parameters, 'min' );
+        return $this->getSize( $attribute, $value ) >= $parameters[0];
     }
 
     /**
@@ -100,9 +100,9 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateNumber($value)
+    public function validateNumber( $value )
     {
-        return is_numeric($value);
+        return is_numeric( $value );
     }
 
     /**
@@ -110,11 +110,11 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function validateRequired($value)
+    public function validateRequired( $value )
     {
-        return is_null($value)
-            || (is_string($value) && in_array(trim($value), ['', '[]']))
-            || (is_array($value) && empty($value))
+        return is_null( $value )
+            || (is_string( $value ) && in_array( trim( $value ), ['', '[]'] ))
+            || (is_array( $value ) && empty($value))
             ? false
             : true;
     }
@@ -126,10 +126,10 @@ 
      * @return void
      * @throws InvalidArgumentException
      */
-    protected function requireParameterCount($count, array $parameters, $rule)
+    protected function requireParameterCount( $count, array $parameters, $rule )
     {
-        if (count($parameters) < $count) {
-            throw new InvalidArgumentException("Validation rule $rule requires at least $count parameters.");
+        if( count( $parameters ) < $count ) {
+            throw new InvalidArgumentException( "Validation rule $rule requires at least $count parameters." );
         }
     }
 }