pryley / site-reviews

plugin/Database/SqlQueries.php

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,10 +127,10 @@ 
      * @param string $status
      * @return array
      */
-    public function getReviewsMeta($key, $status = 'publish')
+    public function getReviewsMeta( $key, $status = 'publish' )
     {
-        $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
@@ -140,8 +140,8 @@ 
             AND p.post_status = '{$status}'
             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;
     }
 
@@ -149,34 +149,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/Controllers/TaxonomyController.php

Spacing +15 added lines, -15 removed lines

@@ -36,14 +36,14 @@ 
      */
     public function renderTaxonomyFilter()
     {
-        if (!is_object_in_taxonomy(glsr_current_screen()->post_type, Application::TAXONOMY)) {
+        if( !is_object_in_taxonomy( glsr_current_screen()->post_type, Application::TAXONOMY ) ) {
             return;
         }
-        echo glsr(Builder::class)->label(__('Filter by category', 'site-reviews'), [
+        echo glsr( Builder::class )->label( __( 'Filter by category', 'site-reviews' ), [
             'class' => 'screen-reader-text',
             'for' => Application::TAXONOMY,
-        ]);
-        wp_dropdown_categories([
+        ] );
+        wp_dropdown_categories( [
             'depth' => 3,
             'hide_empty' => true,
             'hide_if_empty' => true,
@@ -55,7 +55,7 @@ 
             'show_option_all' => $this->getShowOptionAll(),
             'taxonomy' => Application::TAXONOMY,
             'value_field' => 'slug',
-        ]);
+        ] );
     }
 
     /**
@@ -68,17 +68,17 @@ 
      * @return void
      * @action set_object_terms
      */
-    public function restrictTermSelection($postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds)
+    public function restrictTermSelection( $postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds )
     {
-        if (Application::TAXONOMY != $taxonomy || count($newTTIds) <= 1) {
+        if( Application::TAXONOMY != $taxonomy || count( $newTTIds ) <= 1 ) {
             return;
         }
-        $diff = array_diff($newTTIds, $oldTTIds);
-        if (empty($newTerm = array_shift($diff))) {
-            $newTerm = array_shift($newTTIds);
+        $diff = array_diff( $newTTIds, $oldTTIds );
+        if( empty($newTerm = array_shift( $diff )) ) {
+            $newTerm = array_shift( $newTTIds );
         }
-        if ($newTerm) {
-            wp_set_object_terms($postId, intval($newTerm), $taxonomy);
+        if( $newTerm ) {
+            wp_set_object_terms( $postId, intval( $newTerm ), $taxonomy );
         }
     }
 
@@ -88,7 +88,7 @@ 
     protected function getSelected()
     {
         global $wp_query;
-        return Arr::get($wp_query->query, Application::TAXONOMY);
+        return Arr::get( $wp_query->query, Application::TAXONOMY );
     }
 
     /**
@@ -96,9 +96,9 @@ 
      */
     protected function getShowOptionAll()
     {
-        $taxonomy = get_taxonomy(Application::TAXONOMY);
+        $taxonomy = get_taxonomy( Application::TAXONOMY );
         return $taxonomy
-            ? ucfirst(strtolower($taxonomy->labels->all_items))
+            ? ucfirst( strtolower( $taxonomy->labels->all_items ) )
             : '';
     }
 }

plugin/Controllers/ReviewController.php

Spacing +44 added lines, -44 removed lines

@@ -23,22 +23,22 @@ 
      * @return void
      * @action set_object_terms
      */
-    public function onAfterChangeCategory($postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds)
+    public function onAfterChangeCategory( $postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds )
     {
-        sort($newTTIds);
-        sort($oldTTIds);
-        if ($newTTIds === $oldTTIds || !$this->isReviewPostId($postId)) {
+        sort( $newTTIds );
+        sort( $oldTTIds );
+        if( $newTTIds === $oldTTIds || !$this->isReviewPostId( $postId ) ) {
             return;
         }
-        $review = glsr_get_review($postId);
-        $ignoredIds = array_intersect($oldTTIds, $newTTIds);
-        $decreasedIds = array_diff($oldTTIds, $ignoredIds);
-        $increasedIds = array_diff($newTTIds, $ignoredIds);
-        if ($review->term_ids = glsr(Database::class)->getTermIds($decreasedIds, 'term_taxonomy_id')) {
-            glsr(CountsManager::class)->decreaseTermCounts($review);
+        $review = glsr_get_review( $postId );
+        $ignoredIds = array_intersect( $oldTTIds, $newTTIds );
+        $decreasedIds = array_diff( $oldTTIds, $ignoredIds );
+        $increasedIds = array_diff( $newTTIds, $ignoredIds );
+        if( $review->term_ids = glsr( Database::class )->getTermIds( $decreasedIds, 'term_taxonomy_id' ) ) {
+            glsr( CountsManager::class )->decreaseTermCounts( $review );
         }
-        if ($review->term_ids = glsr(Database::class)->getTermIds($increasedIds, 'term_taxonomy_id')) {
-            glsr(CountsManager::class)->increaseTermCounts($review);
+        if( $review->term_ids = glsr( Database::class )->getTermIds( $increasedIds, 'term_taxonomy_id' ) ) {
+            glsr( CountsManager::class )->increaseTermCounts( $review );
         }
     }
 
@@ -49,16 +49,16 @@ 
      * @return void
      * @action transition_post_status
      */
-    public function onAfterChangeStatus($newStatus, $oldStatus, $post)
+    public function onAfterChangeStatus( $newStatus, $oldStatus, $post )
     {
-        if (Application::POST_TYPE != Arr::get($post, 'post_type') || in_array($oldStatus, ['new', $newStatus])) {
+        if( Application::POST_TYPE != Arr::get( $post, 'post_type' ) || in_array( $oldStatus, ['new', $newStatus] ) ) {
             return;
         }
-        $review = glsr_get_review($post);
-        if ('publish' == $post->post_status) {
-            glsr(CountsManager::class)->increase($review);
+        $review = glsr_get_review( $post );
+        if( 'publish' == $post->post_status ) {
+            glsr( CountsManager::class )->increase( $review );
         } else {
-            glsr(CountsManager::class)->decrease($review);
+            glsr( CountsManager::class )->decrease( $review );
         }
     }
 
@@ -66,13 +66,13 @@ 
      * @return void
      * @action site-reviews/review/created
      */
-    public function onAfterCreate(Review $review)
+    public function onAfterCreate( Review $review )
     {
-        if ('publish' !== $review->status) {
+        if( 'publish' !== $review->status ) {
             return;
         }
-        glsr(CountsManager::class)->increaseCounts($review);
-        glsr(CountsManager::class)->increasePostCounts($review);
+        glsr( CountsManager::class )->increaseCounts( $review );
+        glsr( CountsManager::class )->increasePostCounts( $review );
     }
 
     /**
@@ -80,14 +80,14 @@ 
      * @return void
      * @action before_delete_post
      */
-    public function onBeforeDelete($postId)
+    public function onBeforeDelete( $postId )
     {
-        if (!$this->isReviewPostId($postId)) {
+        if( !$this->isReviewPostId( $postId ) ) {
             return;
         }
-        $review = glsr_get_review($postId);
-        if ('trash' !== $review->status) { // do not run for trashed posts
-            glsr(CountsManager::class)->decrease($review);
+        $review = glsr_get_review( $postId );
+        if( 'trash' !== $review->status ) { // do not run for trashed posts
+            glsr( CountsManager::class )->decrease( $review );
         }
     }
 
@@ -99,53 +99,53 @@ 
      * @return void
      * @action update_postmeta
      */
-    public function onBeforeUpdate($metaId, $postId, $metaKey, $metaValue)
+    public function onBeforeUpdate( $metaId, $postId, $metaKey, $metaValue )
     {
-        if (!$this->isReviewPostId($postId)) {
+        if( !$this->isReviewPostId( $postId ) ) {
             return;
         }
-        $metaKey = Str::removePrefix('_', $metaKey);
-        if (!in_array($metaKey, ['assigned_to', 'rating', 'review_type'])) {
+        $metaKey = Str::removePrefix( '_', $metaKey );
+        if( !in_array( $metaKey, ['assigned_to', 'rating', 'review_type'] ) ) {
             return;
         }
-        $review = glsr_get_review($postId);
-        if ($review->$metaKey == $metaValue) {
+        $review = glsr_get_review( $postId );
+        if( $review->$metaKey == $metaValue ) {
             return;
         }
-        $method = Helper::buildMethodName($metaKey, 'onBeforeChange');
-        call_user_func([$this, $method], $review, $metaValue);
+        $method = Helper::buildMethodName( $metaKey, 'onBeforeChange' );
+        call_user_func( [$this, $method], $review, $metaValue );
     }
 
     /**
      * @param string|int $assignedTo
      * @return void
      */
-    public function onBeforeChangeAssignedTo(Review $review, $assignedTo)
+    public function onBeforeChangeAssignedTo( Review $review, $assignedTo )
     {
-        glsr(CountsManager::class)->decreasePostCounts($review);
+        glsr( CountsManager::class )->decreasePostCounts( $review );
         $review->assigned_to = $assignedTo;
-        glsr(CountsManager::class)->increasePostCounts($review);
+        glsr( CountsManager::class )->increasePostCounts( $review );
     }
 
     /**
      * @param string|int $rating
      * @return void
      */
-    public function onBeforeChangeRating(Review $review, $rating)
+    public function onBeforeChangeRating( Review $review, $rating )
     {
-        glsr(CountsManager::class)->decrease($review);
+        glsr( CountsManager::class )->decrease( $review );
         $review->rating = $rating;
-        glsr(CountsManager::class)->increase($review);
+        glsr( CountsManager::class )->increase( $review );
     }
 
     /**
      * @param string $reviewType
      * @return void
      */
-    public function onBeforeChangeReviewType(Review $review, $reviewType)
+    public function onBeforeChangeReviewType( Review $review, $reviewType )
     {
-        glsr(CountsManager::class)->decrease($review);
+        glsr( CountsManager::class )->decrease( $review );
         $review->review_type = $reviewType;
-        glsr(CountsManager::class)->increase($review);
+        glsr( CountsManager::class )->increase( $review );
     }
 }

plugin/Controllers/AdminController.php

Spacing +71 added lines, -71 removed lines

@@ -22,9 +22,9 @@ 
      */
     public function enqueueAssets()
     {
-        $command = new EnqueueAdminAssets([
+        $command = new EnqueueAdminAssets( [
             'pointers' => [[
-                'content' => __('You can pin exceptional reviews so that they are always shown first.', 'site-reviews'),
+                'content' => __( 'You can pin exceptional reviews so that they are always shown first.', 'site-reviews' ),
                 'id' => 'glsr-pointer-pinned',
                 'position' => [
                     'edge' => 'right',
@@ -32,24 +32,24 @@ 
                 ],
                 'screen' => Application::POST_TYPE,
                 'target' => '#misc-pub-pinned',
-                'title' => __('Pin Your Reviews', 'site-reviews'),
+                'title' => __( 'Pin Your Reviews', 'site-reviews' ),
             ]],
-        ]);
-        $this->execute($command);
+        ] );
+        $this->execute( $command );
     }
 
     /**
      * @return array
      * @filter plugin_action_links_site-reviews/site-reviews.php
      */
-    public function filterActionLinks(array $links)
+    public function filterActionLinks( array $links )
     {
-        $links['documentation'] = glsr(Builder::class)->a(__('Help', 'site-reviews'), [
-            'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=documentation'),
-        ]);
-        $links['settings'] = glsr(Builder::class)->a(__('Settings', 'site-reviews'), [
-            'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=settings'),
-        ]);
+        $links['documentation'] = glsr( Builder::class )->a( __( 'Help', 'site-reviews' ), [
+            'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=documentation' ),
+        ] );
+        $links['settings'] = glsr( Builder::class )->a( __( 'Settings', 'site-reviews' ), [
+            'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=settings' ),
+        ] );
         return $links;
     }
 
@@ -59,9 +59,9 @@ 
      * @return array
      * @filter map_meta_cap
      */
-    public function filterCreateCapability($capabilities, $capability)
+    public function filterCreateCapability( $capabilities, $capability )
     {
-        if ($capability == 'create_'.Application::POST_TYPE) {
+        if( $capability == 'create_'.Application::POST_TYPE ) {
             $capabilities[] = 'do_not_allow';
         }
         return $capabilities;
@@ -72,23 +72,23 @@ 
      * @return array
      * @filter dashboard_glance_items
      */
-    public function filterDashboardGlanceItems($items)
+    public function filterDashboardGlanceItems( $items )
     {
-        $postCount = wp_count_posts(Application::POST_TYPE);
-        if (empty($postCount->publish)) {
+        $postCount = wp_count_posts( Application::POST_TYPE );
+        if( empty($postCount->publish) ) {
             return $items;
         }
-        $text = _n('%s Review', '%s Reviews', $postCount->publish, 'site-reviews');
-        $text = sprintf($text, number_format_i18n($postCount->publish));
-        $items = Arr::consolidateArray($items);
-        $items[] = current_user_can(get_post_type_object(Application::POST_TYPE)->cap->edit_posts)
-            ? glsr(Builder::class)->a($text, [
+        $text = _n( '%s Review', '%s Reviews', $postCount->publish, 'site-reviews' );
+        $text = sprintf( $text, number_format_i18n( $postCount->publish ) );
+        $items = Arr::consolidateArray( $items );
+        $items[] = current_user_can( get_post_type_object( Application::POST_TYPE )->cap->edit_posts )
+            ? glsr( Builder::class )->a( $text, [
                 'class' => 'glsr-review-count',
                 'href' => 'edit.php?post_type='.Application::POST_TYPE,
-            ])
-            : glsr(Builder::class)->span($text, [
+            ] )
+            : glsr( Builder::class )->span( $text, [
                 'class' => 'glsr-review-count',
-            ]);
+            ] );
         return $items;
     }
 
@@ -97,11 +97,11 @@ 
      * @return array
      * @filter mce_external_plugins
      */
-    public function filterTinymcePlugins($plugins)
+    public function filterTinymcePlugins( $plugins )
     {
-        if (current_user_can('edit_posts') || current_user_can('edit_pages')) {
-            $plugins = Arr::consolidateArray($plugins);
-            $plugins['glsr_shortcode'] = glsr()->url('assets/scripts/mce-plugin.js');
+        if( current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' ) ) {
+            $plugins = Arr::consolidateArray( $plugins );
+            $plugins['glsr_shortcode'] = glsr()->url( 'assets/scripts/mce-plugin.js' );
         }
         return $plugins;
     }
@@ -112,12 +112,12 @@ 
      */
     public function registerTinymcePopups()
     {
-        $command = new RegisterTinymcePopups([
-            'site_reviews' => esc_html__('Recent Reviews', 'site-reviews'),
-            'site_reviews_form' => esc_html__('Submit a Review', 'site-reviews'),
-            'site_reviews_summary' => esc_html__('Summary of Reviews', 'site-reviews'),
-        ]);
-        $this->execute($command);
+        $command = new RegisterTinymcePopups( [
+            'site_reviews' => esc_html__( 'Recent Reviews', 'site-reviews' ),
+            'site_reviews_form' => esc_html__( 'Submit a Review', 'site-reviews' ),
+            'site_reviews_summary' => esc_html__( 'Summary of Reviews', 'site-reviews' ),
+        ] );
+        $this->execute( $command );
     }
 
     /**
@@ -125,22 +125,22 @@ 
      * @return void|null
      * @action media_buttons
      */
-    public function renderTinymceButton($editorId)
+    public function renderTinymceButton( $editorId )
     {
-        $allowedEditors = apply_filters('site-reviews/tinymce/editor-ids', ['content'], $editorId);
-        if ('post' != glsr_current_screen()->base || !in_array($editorId, $allowedEditors)) {
+        $allowedEditors = apply_filters( 'site-reviews/tinymce/editor-ids', ['content'], $editorId );
+        if( 'post' != glsr_current_screen()->base || !in_array( $editorId, $allowedEditors ) ) {
             return;
         }
         $shortcodes = [];
-        foreach (glsr()->mceShortcodes as $shortcode => $values) {
+        foreach( glsr()->mceShortcodes as $shortcode => $values ) {
             $shortcodes[$shortcode] = $values;
         }
-        if (empty($shortcodes)) {
+        if( empty($shortcodes) ) {
             return;
         }
-        glsr()->render('partials/editor/tinymce', [
+        glsr()->render( 'partials/editor/tinymce', [
             'shortcodes' => $shortcodes,
-        ]);
+        ] );
     }
 
     /**
@@ -148,8 +148,8 @@ 
      */
     public function routerClearConsole()
     {
-        glsr(Console::class)->clear();
-        glsr(Notice::class)->addSuccess(__('Console cleared.', 'site-reviews'));
+        glsr( Console::class )->clear();
+        glsr( Notice::class )->addSuccess( __( 'Console cleared.', 'site-reviews' ) );
     }
 
     /**
@@ -157,19 +157,19 @@ 
      */
     public function routerFetchConsole()
     {
-        glsr(Notice::class)->addSuccess(__('Console reloaded.', 'site-reviews'));
+        glsr( Notice::class )->addSuccess( __( 'Console reloaded.', 'site-reviews' ) );
     }
 
     /**
      * @param bool $showNotice
      * @return void
      */
-    public function routerCountReviews($showNotice = true)
+    public function routerCountReviews( $showNotice = true )
     {
-        glsr(CountsManager::class)->countAll();
-        glsr(OptionManager::class)->set('last_review_count', current_time('timestamp'));
-        if ($showNotice) {
-            glsr(Notice::class)->clear()->addSuccess(__('Recalculated rating counts.', 'site-reviews'));
+        glsr( CountsManager::class )->countAll();
+        glsr( OptionManager::class )->set( 'last_review_count', current_time( 'timestamp' ) );
+        if( $showNotice ) {
+            glsr( Notice::class )->clear()->addSuccess( __( 'Recalculated rating counts.', 'site-reviews' ) );
         }
     }
 
@@ -178,7 +178,7 @@ 
      */
     public function routerDownloadConsole()
     {
-        $this->download(Application::ID.'-console.txt', glsr(Console::class)->get());
+        $this->download( Application::ID.'-console.txt', glsr( Console::class )->get() );
     }
 
     /**
@@ -186,7 +186,7 @@ 
      */
     public function routerDownloadSystemInfo()
     {
-        $this->download(Application::ID.'-system-info.txt', glsr(System::class)->get());
+        $this->download( Application::ID.'-system-info.txt', glsr( System::class )->get() );
     }
 
     /**
@@ -194,7 +194,7 @@ 
      */
     public function routerExportSettings()
     {
-        $this->download(Application::ID.'-settings.json', glsr(OptionManager::class)->json());
+        $this->download( Application::ID.'-settings.json', glsr( OptionManager::class )->json() );
     }
 
     /**
@@ -203,35 +203,35 @@ 
     public function routerImportSettings()
     {
         $file = $_FILES['import-file'];
-        if (UPLOAD_ERR_OK !== $file['error']) {
-            return glsr(Notice::class)->addError($this->getUploadError($file['error']));
+        if( UPLOAD_ERR_OK !== $file['error'] ) {
+            return glsr( Notice::class )->addError( $this->getUploadError( $file['error'] ) );
         }
-        if ('application/json' !== $file['type'] || !Str::endsWith('.json', $file['name'])) {
-            return glsr(Notice::class)->addError(__('Please use a valid Site Reviews settings file.', 'site-reviews'));
+        if( 'application/json' !== $file['type'] || !Str::endsWith( '.json', $file['name'] ) ) {
+            return glsr( Notice::class )->addError( __( 'Please use a valid Site Reviews settings file.', 'site-reviews' ) );
         }
-        $settings = json_decode(file_get_contents($file['tmp_name']), true);
-        if (empty($settings)) {
-            return glsr(Notice::class)->addWarning(__('There were no settings found to import.', 'site-reviews'));
+        $settings = json_decode( file_get_contents( $file['tmp_name'] ), true );
+        if( empty($settings) ) {
+            return glsr( Notice::class )->addWarning( __( 'There were no settings found to import.', 'site-reviews' ) );
         }
-        glsr(OptionManager::class)->set(glsr(OptionManager::class)->normalize($settings));
-        glsr(Notice::class)->addSuccess(__('Settings imported.', 'site-reviews'));
+        glsr( OptionManager::class )->set( glsr( OptionManager::class )->normalize( $settings ) );
+        glsr( Notice::class )->addSuccess( __( 'Settings imported.', 'site-reviews' ) );
     }
 
     /**
      * @param int $errorCode
      * @return string
      */
-    protected function getUploadError($errorCode)
+    protected function getUploadError( $errorCode )
     {
         $errors = [
-            UPLOAD_ERR_INI_SIZE => __('The uploaded file exceeds the upload_max_filesize directive in php.ini.', 'site-reviews'),
-            UPLOAD_ERR_FORM_SIZE => __('The uploaded file is too big.', 'site-reviews'),
-            UPLOAD_ERR_PARTIAL => __('The uploaded file was only partially uploaded.', 'site-reviews'),
-            UPLOAD_ERR_NO_FILE => __('No file was uploaded.', 'site-reviews'),
-            UPLOAD_ERR_NO_TMP_DIR => __('Missing a temporary folder.', 'site-reviews'),
-            UPLOAD_ERR_CANT_WRITE => __('Failed to write file to disk.', 'site-reviews'),
-            UPLOAD_ERR_EXTENSION => __('A PHP extension stopped the file upload.', 'site-reviews'),
+            UPLOAD_ERR_INI_SIZE => __( 'The uploaded file exceeds the upload_max_filesize directive in php.ini.', 'site-reviews' ),
+            UPLOAD_ERR_FORM_SIZE => __( 'The uploaded file is too big.', 'site-reviews' ),
+            UPLOAD_ERR_PARTIAL => __( 'The uploaded file was only partially uploaded.', 'site-reviews' ),
+            UPLOAD_ERR_NO_FILE => __( 'No file was uploaded.', 'site-reviews' ),
+            UPLOAD_ERR_NO_TMP_DIR => __( 'Missing a temporary folder.', 'site-reviews' ),
+            UPLOAD_ERR_CANT_WRITE => __( 'Failed to write file to disk.', 'site-reviews' ),
+            UPLOAD_ERR_EXTENSION => __( 'A PHP extension stopped the file upload.', 'site-reviews' ),
         ];
-        return Arr::get($errors, $errorCode, __('Unknown upload error.', 'site-reviews'));
+        return Arr::get( $errors, $errorCode, __( 'Unknown upload error.', 'site-reviews' ) );
     }
 }

plugin/Modules/ReviewLimits.php

Spacing +27 added lines, -27 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/reviews-limits/query', $parameters, $args);
+        return apply_filters( 'site-reviews/reviews-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,39 +42,39 @@ 
      * @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);
-        return 0 === count($reviews);
+        $reviews = glsr_get_reviews( $args );
+        remove_filter( 'site-reviews/get/reviews/query', [$this, 'filterReviewsQuery'], 5 );
+        return 0 === count( $reviews );
     }
 
     /**
@@ -82,8 +82,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' ) );
     }
 
     /**
@@ -91,8 +91,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' ) );
     }
 
     /**
@@ -101,10 +101,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/Rebusify.php

Spacing +39 added lines, -39 removed lines

@@ -20,25 +20,25 @@ 
     /**
      * @return mixed
      */
-    public function __get($key)
+    public function __get( $key )
     {
-        return property_exists($this, $key)
+        return property_exists( $this, $key )
             ? $this->$key
-            : Arr::get($this->response, $key, null);
+            : Arr::get( $this->response, $key, null );
     }
 
     /**
      * @return self
      */
-    public function activateKey($apiKey = '', $email = '')
+    public function activateKey( $apiKey = '', $email = '' )
     {
-        $this->send('api_key_activation.php', [
+        $this->send( 'api_key_activation.php', [
             'body' => [
                 'apikey' => $apiKey ?: 0,
                 'domain' => get_home_url(),
                 'email' => $email ?: 0,
             ],
-        ]);
+        ] );
         return $this;
     }
 
@@ -56,77 +56,77 @@ 
     /**
      * @return self
      */
-    public function sendReview(Review $review)
+    public function sendReview( Review $review )
     {
-        $this->send('index.php', [
-            'body' => $this->getBodyForReview($review),
+        $this->send( 'index.php', [
+            'body' => $this->getBodyForReview( $review ),
             'timeout' => 120,
-        ]);
+        ] );
         return $this;
     }
 
     /**
      * @return self
      */
-    public function sendReviewResponse(Review $review)
+    public function sendReviewResponse( Review $review )
     {
-        $this->send('fetch_customer_reply.php', [
-            'body' => $this->getBodyForResponse($review),
-        ]);
+        $this->send( 'fetch_customer_reply.php', [
+            'body' => $this->getBodyForResponse( $review ),
+        ] );
         return $this;
     }
 
     /**
      * @return array
      */
-    protected function getBodyForResponse(Review $review)
+    protected function getBodyForResponse( Review $review )
     {
         $rebusifyResponse = [
-            'reply' => Str::truncate($review->response, 300),
-            'review_id' => glsr(Database::class)->get($review->ID, 'rebusify'), // this is the rebusify review ID
+            'reply' => Str::truncate( $review->response, 300 ),
+            'review_id' => glsr( Database::class )->get( $review->ID, 'rebusify' ), // this is the rebusify review ID
             'review_transaction_id' => $review->review_id,
             'type' => 'M',
         ];
-        return apply_filters('site-reviews/rebusify/response', $rebusifyResponse, $review);
+        return apply_filters( 'site-reviews/rebusify/response', $rebusifyResponse, $review );
     }
 
     /**
      * @return array
      */
-    protected function getBodyForReview(Review $review)
+    protected function getBodyForReview( Review $review )
     {
         $rebusifyReview = [
             'domain' => get_home_url(),
-            'firstname' => Str::truncate(Str::convertName($review->author, 'first'), 25),
+            'firstname' => Str::truncate( Str::convertName( $review->author, 'first' ), 25 ),
             'rate' => $review->rating,
             'review_transaction_id' => $review->review_id,
-            'reviews' => Str::truncate($review->content, 280),
-            'title' => Str::truncate($review->title, 35),
+            'reviews' => Str::truncate( $review->content, 280 ),
+            'title' => Str::truncate( $review->title, 35 ),
             'transaction' => Application::ID, // woocommerce field, not needed for Site Reviews
         ];
-        return apply_filters('site-reviews/rebusify/review', $rebusifyReview, $review);
+        return apply_filters( 'site-reviews/rebusify/review', $rebusifyReview, $review );
     }
 
     /**
      * @param \WP_Error|array $response
      * @return void
      */
-    protected function handleResponse($response)
+    protected function handleResponse( $response )
     {
-        if (is_wp_error($response)) {
+        if( is_wp_error( $response ) ) {
             $this->message = $response->get_error_message();
         } else {
-            $responseBody = wp_remote_retrieve_body($response);
-            $responseCode = wp_remote_retrieve_response_code($response);
-            $responseData = (array) json_decode($responseBody, true);
-            $this->response = array_shift($responseData);
-            $this->message = Arr::get($this->response, 'msg');
-            $this->success = 'success' === Arr::get($this->response, 'result') || 'yes' === Arr::get($this->response, 'success'); // @todo remove this ugly hack!
-            if (200 !== $responseCode) {
-                glsr_log()->error('Bad response code ['.$responseCode.']');
+            $responseBody = wp_remote_retrieve_body( $response );
+            $responseCode = wp_remote_retrieve_response_code( $response );
+            $responseData = (array)json_decode( $responseBody, true );
+            $this->response = array_shift( $responseData );
+            $this->message = Arr::get( $this->response, 'msg' );
+            $this->success = 'success' === Arr::get( $this->response, 'result' ) || 'yes' === Arr::get( $this->response, 'success' ); // @todo remove this ugly hack!
+            if( 200 !== $responseCode ) {
+                glsr_log()->error( 'Bad response code ['.$responseCode.']' );
             }
-            if (!$this->success) {
-                glsr_log()->error($this->message);
+            if( !$this->success ) {
+                glsr_log()->error( $this->message );
             }
         }
     }
@@ -135,18 +135,18 @@ 
      * @param string $endpoint
      * @return void
      */
-    protected function send($endpoint, array $args = [])
+    protected function send( $endpoint, array $args = [] )
     {
-        $args = wp_parse_args($args, [
+        $args = wp_parse_args( $args, [
             'body' => null,
             'headers' => ['Content-Type' => 'application/x-www-form-urlencoded'],
             'redirection' => 5,
             'sslverify' => false,
             'timeout' => 5,
-        ]);
+        ] );
         $this->reset();
         $this->handleResponse(
-            wp_remote_post(trailingslashit(static::API_URL).$endpoint, $args)
+            wp_remote_post( trailingslashit( static::API_URL ).$endpoint, $args )
         );
     }
 }

plugin/Modules/Schema/BaseType.php

Spacing +45 added lines, -45 removed lines

@@ -37,18 +37,18 @@ 
      * @param string $method
      * @return static
      */
-    public function __call($method, array $arguments)
+    public function __call( $method, array $arguments )
     {
-        return $this->setProperty($method, Arr::get($arguments, 0));
+        return $this->setProperty( $method, Arr::get( $arguments, 0 ) );
     }
 
     /**
      * @param string $type
      */
-    public function __construct($type = null)
+    public function __construct( $type = null )
     {
-        $this->type = !is_string($type)
-            ? (new ReflectionClass($this))->getShortName()
+        $this->type = !is_string( $type )
+            ? (new ReflectionClass( $this ))->getShortName()
             : $type;
         $this->setAllowedProperties();
     }
@@ -64,10 +64,10 @@ 
     /**
      * @return static
      */
-    public function addProperties(array $properties)
+    public function addProperties( array $properties )
     {
-        foreach ($properties as $property => $value) {
-            $this->setProperty($property, $value);
+        foreach( $properties as $property => $value ) {
+            $this->setProperty( $property, $value );
         }
         return $this;
     }
@@ -93,9 +93,9 @@ 
      * @param mixed $default
      * @return mixed
      */
-    public function getProperty($property, $default = null)
+    public function getProperty( $property, $default = null )
     {
-        return Arr::get($this->properties, $property, $default);
+        return Arr::get( $this->properties, $property, $default );
     }
 
     /**
@@ -111,10 +111,10 @@ 
      * @param mixed $callback
      * @return static
      */
-    public function doIf($condition, $callback)
+    public function doIf( $condition, $callback )
     {
-        if ($condition) {
-            $callback($this);
+        if( $condition ) {
+            $callback( $this );
         }
         return $this;
     }
@@ -131,18 +131,18 @@ 
      * @param mixed $offset
      * @return bool
      */
-    public function offsetExists($offset)
+    public function offsetExists( $offset )
     {
-        return array_key_exists($offset, $this->properties);
+        return array_key_exists( $offset, $this->properties );
     }
 
     /**
      * @param string $offset
      * @return mixed
      */
-    public function offsetGet($offset)
+    public function offsetGet( $offset )
     {
-        return $this->getProperty($offset);
+        return $this->getProperty( $offset );
     }
 
     /**
@@ -150,16 +150,16 @@ 
      * @param mixed $value
      * @return void
      */
-    public function offsetSet($offset, $value)
+    public function offsetSet( $offset, $value )
     {
-        $this->setProperty($offset, $value);
+        $this->setProperty( $offset, $value );
     }
 
     /**
      * @param string $offset
      * @return void
      */
-    public function offsetUnset($offset)
+    public function offsetUnset( $offset )
     {
         unset($this->properties[$offset]);
     }
@@ -169,11 +169,11 @@ 
      * @param mixed $value
      * @return static
      */
-    public function setProperty($property, $value)
+    public function setProperty( $property, $value )
     {
-        if (!in_array($property, $this->allowed)
-            && 'UnknownType' != (new ReflectionClass($this))->getShortName()) {
-            glsr_log()->warning($this->getType().' does not allow the "'.$property.'" property');
+        if( !in_array( $property, $this->allowed )
+            && 'UnknownType' != (new ReflectionClass( $this ))->getShortName() ) {
+            glsr_log()->warning( $this->getType().' does not allow the "'.$property.'" property' );
             return $this;
         }
         $this->properties[$property] = $value;
@@ -188,7 +188,7 @@ 
         return [
             '@context' => $this->getContext(),
             '@type' => $this->getType(),
-        ] + $this->serializeProperty($this->getProperties());
+        ] + $this->serializeProperty( $this->getProperties() );
     }
 
     /**
@@ -196,8 +196,8 @@ 
      */
     public function toScript()
     {
-        return sprintf('<script type="application/ld+json">%s</script>',
-            json_encode($this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES)
+        return sprintf( '<script type="application/ld+json">%s</script>',
+            json_encode( $this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES )
         );
     }
 
@@ -205,20 +205,20 @@ 
      * @param array|null $parents
      * @return array
      */
-    protected function getParents($parents = null)
+    protected function getParents( $parents = null )
     {
-        if (!isset($parents)) {
+        if( !isset($parents) ) {
             $parents = $this->parents;
         }
         $newParents = $parents;
-        foreach ($parents as $parent) {
-            $parentClass = Helper::buildClassName($parent, __NAMESPACE__);
-            if (!class_exists($parentClass)) {
+        foreach( $parents as $parent ) {
+            $parentClass = Helper::buildClassName( $parent, __NAMESPACE__ );
+            if( !class_exists( $parentClass ) ) {
                 continue;
             }
-            $newParents = array_merge($newParents, $this->getParents((new $parentClass())->parents));
+            $newParents = array_merge( $newParents, $this->getParents( (new $parentClass())->parents ) );
         }
-        return array_values(array_unique($newParents));
+        return array_values( array_unique( $newParents ) );
     }
 
     /**
@@ -227,12 +227,12 @@ 
     protected function setAllowedProperties()
     {
         $parents = $this->getParents();
-        foreach ($parents as $parent) {
-            $parentClass = Helper::buildClassName($parent, __NAMESPACE__);
-            if (!class_exists($parentClass)) {
+        foreach( $parents as $parent ) {
+            $parentClass = Helper::buildClassName( $parent, __NAMESPACE__ );
+            if( !class_exists( $parentClass ) ) {
                 continue;
             }
-            $this->allowed = array_values(array_unique(array_merge((new $parentClass())->allowed, $this->allowed)));
+            $this->allowed = array_values( array_unique( array_merge( (new $parentClass())->allowed, $this->allowed ) ) );
         }
     }
 
@@ -240,19 +240,19 @@ 
      * @param mixed $property
      * @return array|string
      */
-    protected function serializeProperty($property)
+    protected function serializeProperty( $property )
     {
-        if (is_array($property)) {
-            return array_map([$this, 'serializeProperty'], $property);
+        if( is_array( $property ) ) {
+            return array_map( [$this, 'serializeProperty'], $property );
         }
-        if ($property instanceof Type) {
+        if( $property instanceof Type ) {
             $property = $property->toArray();
             unset($property['@context']);
         }
-        if ($property instanceof DateTimeInterface) {
-            $property = $property->format(DateTime::ATOM);
+        if( $property instanceof DateTimeInterface ) {
+            $property = $property->format( DateTime::ATOM );
         }
-        if (is_object($property)) {
+        if( is_object( $property ) ) {
             throw new InvalidProperty();
         }
         return $property;

plugin/Modules/Html/Settings.php

Spacing +67 added lines, -67 removed lines

@@ -20,31 +20,31 @@ 
      * @param string $id
      * @return string
      */
-    public function buildFields($id)
+    public function buildFields( $id )
     {
-        $this->settings = glsr(DefaultsManager::class)->settings();
-        $method = Helper::buildMethodName($id, 'getTemplateDataFor');
-        $data = !method_exists($this, $method)
-            ? $this->getTemplateData($id)
-            : $this->$method($id);
-        return glsr(Template::class)->build('pages/settings/'.$id, $data);
+        $this->settings = glsr( DefaultsManager::class )->settings();
+        $method = Helper::buildMethodName( $id, 'getTemplateDataFor' );
+        $data = !method_exists( $this, $method )
+            ? $this->getTemplateData( $id )
+            : $this->$method( $id );
+        return glsr( Template::class )->build( 'pages/settings/'.$id, $data );
     }
 
     /**
      * @return string
      */
-    protected function getFieldDefault(array $field)
+    protected function getFieldDefault( array $field )
     {
-        return Arr::get($field, 'default');
+        return Arr::get( $field, 'default' );
     }
 
     /**
      * @return string
      */
-    protected function getFieldNameForDependsOn($path)
+    protected function getFieldNameForDependsOn( $path )
     {
-        $fieldName = Str::convertPathToName($path, OptionManager::databaseKey());
-        return $this->isMultiDependency($path)
+        $fieldName = Str::convertPathToName( $path, OptionManager::databaseKey() );
+        return $this->isMultiDependency( $path )
             ? $fieldName.'[]'
             : $fieldName;
     }
@@ -52,25 +52,25 @@ 
     /**
      * @return array
      */
-    protected function getSettingFields($path)
+    protected function getSettingFields( $path )
     {
-        return array_filter($this->settings, function ($key) use ($path) {
-            return Str::startsWith($path, $key);
-        }, ARRAY_FILTER_USE_KEY);
+        return array_filter( $this->settings, function( $key ) use ($path) {
+            return Str::startsWith( $path, $key );
+        }, ARRAY_FILTER_USE_KEY );
     }
 
     /**
      * @return string
      */
-    protected function getSettingRows(array $fields)
+    protected function getSettingRows( array $fields )
     {
         $rows = '';
-        foreach ($fields as $name => $field) {
-            $field = wp_parse_args($field, [
+        foreach( $fields as $name => $field ) {
+            $field = wp_parse_args( $field, [
                 'is_setting' => true,
                 'name' => $name,
-            ]);
-            $rows.= new Field($this->normalize($field));
+            ] );
+            $rows .= new Field( $this->normalize( $field ) );
         }
         return $rows;
     }
@@ -79,12 +79,12 @@ 
      * @param string $id
      * @return array
      */
-    protected function getTemplateData($id)
+    protected function getTemplateData( $id )
     {
-        $fields = $this->getSettingFields($this->normalizeSettingPath($id));
+        $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
         return [
             'context' => [
-                'rows' => $this->getSettingRows($fields),
+                'rows' => $this->getSettingRows( $fields ),
             ],
         ];
     }
@@ -93,19 +93,19 @@ 
      * @param string $id
      * @return array
      */
-    protected function getTemplateDataForAddons($id)
+    protected function getTemplateDataForAddons( $id )
     {
-        $fields = $this->getSettingFields($this->normalizeSettingPath($id));
-        $settings = Arr::convertDotNotationArray($fields);
-        $settingKeys = array_keys($settings['settings']['addons']);
+        $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
+        $settings = Arr::convertDotNotationArray( $fields );
+        $settingKeys = array_keys( $settings['settings']['addons'] );
         $results = [];
-        foreach ($settingKeys as $key) {
-            $addonFields = array_filter($fields, function ($path) use ($key) {
-                return Str::startsWith('settings.addons.'.$key, $path);
-            }, ARRAY_FILTER_USE_KEY);
-            $results[$key] = $this->getSettingRows($addonFields);
+        foreach( $settingKeys as $key ) {
+            $addonFields = array_filter( $fields, function( $path ) use ($key) {
+                return Str::startsWith( 'settings.addons.'.$key, $path );
+            }, ARRAY_FILTER_USE_KEY );
+            $results[$key] = $this->getSettingRows( $addonFields );
         }
-        ksort($results);
+        ksort( $results );
         return [
             'settings' => $results,
         ];
@@ -115,13 +115,13 @@ 
      * @param string $id
      * @return array
      */
-    protected function getTemplateDataForLicenses($id)
+    protected function getTemplateDataForLicenses( $id )
     {
-        $fields = $this->getSettingFields($this->normalizeSettingPath($id));
-        ksort($fields);
+        $fields = $this->getSettingFields( $this->normalizeSettingPath( $id ) );
+        ksort( $fields );
         return [
             'context' => [
-                'rows' => $this->getSettingRows($fields),
+                'rows' => $this->getSettingRows( $fields ),
             ],
         ];
     }
@@ -131,7 +131,7 @@ 
      */
     protected function getTemplateDataForTranslations()
     {
-        $translations = glsr(Translation::class)->renderAll();
+        $translations = glsr( Translation::class )->renderAll();
         $class = empty($translations)
             ? 'glsr-hidden'
             : '';
@@ -149,16 +149,16 @@ 
      * @param string|array $expectedValue
      * @return bool
      */
-    protected function isFieldHidden($path, $expectedValue)
+    protected function isFieldHidden( $path, $expectedValue )
     {
-        $optionValue = glsr(OptionManager::class)->get(
+        $optionValue = glsr( OptionManager::class )->get(
             $path,
-            Arr::get(glsr()->defaults, $path)
+            Arr::get( glsr()->defaults, $path )
         );
-        if (is_array($expectedValue)) {
-            return is_array($optionValue)
-                ? 0 === count(array_intersect($optionValue, $expectedValue))
-                : !in_array($optionValue, $expectedValue);
+        if( is_array( $expectedValue ) ) {
+            return is_array( $optionValue )
+                ? 0 === count( array_intersect( $optionValue, $expectedValue ) )
+                : !in_array( $optionValue, $expectedValue );
         }
         return $optionValue != $expectedValue;
     }
@@ -166,9 +166,9 @@ 
     /**
      * @return bool
      */
-    protected function isMultiDependency($path)
+    protected function isMultiDependency( $path )
     {
-        if (isset($this->settings[$path])) {
+        if( isset($this->settings[$path]) ) {
             $field = $this->settings[$path];
             return ('checkbox' == $field['type'] && !empty($field['options']))
                 || !empty($field['multiple']);
@@ -179,32 +179,32 @@ 
     /**
      * @return array
      */
-    protected function normalize(array $field)
+    protected function normalize( array $field )
     {
-        $field = $this->normalizeDependsOn($field);
-        $field = $this->normalizeLabelAndLegend($field);
-        $field = $this->normalizeValue($field);
+        $field = $this->normalizeDependsOn( $field );
+        $field = $this->normalizeLabelAndLegend( $field );
+        $field = $this->normalizeValue( $field );
         return $field;
     }
 
     /**
      * @return array
      */
-    protected function normalizeDependsOn(array $field)
+    protected function normalizeDependsOn( array $field )
     {
-        if (!empty($field['depends_on']) && is_array($field['depends_on'])) {
+        if( !empty($field['depends_on']) && is_array( $field['depends_on'] ) ) {
             $isFieldHidden = false;
             $conditions = [];
-            foreach ($field['depends_on'] as $path => $value) {
+            foreach( $field['depends_on'] as $path => $value ) {
                 $conditions[] = [
-                    'name' => $this->getFieldNameForDependsOn($path),
+                    'name' => $this->getFieldNameForDependsOn( $path ),
                     'value' => $value,
                 ];
-                if ($this->isFieldHidden($path, $value)) {
+                if( $this->isFieldHidden( $path, $value ) ) {
                     $isFieldHidden = true;
                 }
             }
-            $field['data-depends'] = json_encode($conditions, JSON_HEX_APOS | JSON_HEX_QUOT);
+            $field['data-depends'] = json_encode( $conditions, JSON_HEX_APOS | JSON_HEX_QUOT );
             $field['is_hidden'] = $isFieldHidden;
         }
         return $field;
@@ -213,14 +213,14 @@ 
     /**
      * @return array
      */
-    protected function normalizeLabelAndLegend(array $field)
+    protected function normalizeLabelAndLegend( array $field )
     {
-        if (!empty($field['label'])) {
+        if( !empty($field['label']) ) {
             $field['legend'] = $field['label'];
             unset($field['label']);
         } else {
             $field['is_valid'] = false;
-            glsr_log()->warning('Setting field is missing a label')->debug($field);
+            glsr_log()->warning( 'Setting field is missing a label' )->debug( $field );
         }
         return $field;
     }
@@ -228,12 +228,12 @@ 
     /**
      * @return array
      */
-    protected function normalizeValue(array $field)
+    protected function normalizeValue( array $field )
     {
-        if (!isset($field['value'])) {
-            $field['value'] = glsr(OptionManager::class)->get(
+        if( !isset($field['value']) ) {
+            $field['value'] = glsr( OptionManager::class )->get(
                 $field['name'],
-                $this->getFieldDefault($field)
+                $this->getFieldDefault( $field )
             );
         }
         return $field;
@@ -242,8 +242,8 @@ 
     /**
      * @return string
      */
-    protected function normalizeSettingPath($path)
+    protected function normalizeSettingPath( $path )
     {
-        return Str::prefix('settings.', rtrim($path, '.'));
+        return Str::prefix( 'settings.', rtrim( $path, '.' ) );
     }
 }

plugin/Modules/Html/Fields/Field.php

Spacing +13 added lines, -13 removed lines

@@ -12,7 +12,7 @@ 
      */
     protected $builder;
 
-    public function __construct(Builder $builder)
+    public function __construct( Builder $builder )
     {
         $this->builder = $builder;
     }
@@ -22,7 +22,7 @@ 
      */
     public function build()
     {
-        glsr_log()->error('Build method is not implemented for '.get_class($this));
+        glsr_log()->error( 'Build method is not implemented for '.get_class( $this ) );
     }
 
     /**
@@ -36,14 +36,14 @@ 
     /**
      * @return array
      */
-    public static function merge(array $args)
+    public static function merge( array $args )
     {
         $merged = array_merge(
-            wp_parse_args($args, static::defaults()),
+            wp_parse_args( $args, static::defaults() ),
             static::required()
         );
-        $merged['class'] = implode(' ', static::mergedAttribute('class', ' ', $args));
-        $merged['style'] = implode(';', static::mergedAttribute('style', ';', $args));
+        $merged['class'] = implode( ' ', static::mergedAttribute( 'class', ' ', $args ) );
+        $merged['style'] = implode( ';', static::mergedAttribute( 'style', ';', $args ) );
         return $merged;
     }
 
@@ -52,13 +52,13 @@ 
      * @param string $key
      * @return array
      */
-    public static function mergedAttribute($key, $delimiter, array $args)
+    public static function mergedAttribute( $key, $delimiter, array $args )
     {
-        return array_filter(array_merge(
-            explode($delimiter, Arr::get($args, $key)),
-            explode($delimiter, Arr::get(static::defaults(), $key)),
-            explode($delimiter, Arr::get(static::required(), $key))
-        ));
+        return array_filter( array_merge(
+            explode( $delimiter, Arr::get( $args, $key ) ),
+            explode( $delimiter, Arr::get( static::defaults(), $key ) ),
+            explode( $delimiter, Arr::get( static::required(), $key ) )
+        ) );
     }
 
     /**
@@ -74,6 +74,6 @@ 
      */
     protected function mergeFieldArgs()
     {
-        $this->builder->args = static::merge($this->builder->args);
+        $this->builder->args = static::merge( $this->builder->args );
     }
 }