pryley / site-reviews

plugin/Controllers/ListTableController.php

Spacing +71 added lines, -71 removed lines

@@ -20,15 +20,15 @@ 
      */
     public function approve()
     {
-        if (Application::ID != filter_input(INPUT_GET, 'plugin')) {
+        if( Application::ID != filter_input( INPUT_GET, 'plugin' ) ) {
             return;
         }
-        check_admin_referer('approve-review_'.($postId = $this->getPostId()));
-        wp_update_post([
+        check_admin_referer( 'approve-review_'.($postId = $this->getPostId()) );
+        wp_update_post( [
             'ID' => $postId,
             'post_status' => 'publish',
-        ]);
-        wp_safe_redirect(wp_get_referer());
+        ] );
+        wp_safe_redirect( wp_get_referer() );
         exit;
     }
 
@@ -37,20 +37,20 @@ 
      * @return array
      * @filter manage_.Application::POST_TYPE._posts_columns
      */
-    public function filterColumnsForPostType($columns)
+    public function filterColumnsForPostType( $columns )
     {
-        $columns = Arr::consolidateArray($columns);
+        $columns = Arr::consolidateArray( $columns );
         $postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE];
-        foreach ($postTypeColumns as $key => &$value) {
-            if (!array_key_exists($key, $columns) || !empty($value)) {
+        foreach( $postTypeColumns as $key => &$value ) {
+            if( !array_key_exists( $key, $columns ) || !empty($value) ) {
                 continue;
             }
             $value = $columns[$key];
         }
-        if (count(glsr(Database::class)->getReviewsMeta('review_type')) < 2) {
+        if( count( glsr( Database::class )->getReviewsMeta( 'review_type' ) ) < 2 ) {
             unset($postTypeColumns['review_type']);
         }
-        return array_filter($postTypeColumns, 'strlen');
+        return array_filter( $postTypeColumns, 'strlen' );
     }
 
     /**
@@ -59,10 +59,10 @@ 
      * @return string
      * @filter post_date_column_status
      */
-    public function filterDateColumnStatus($status, $post)
+    public function filterDateColumnStatus( $status, $post )
     {
-        if (Application::POST_TYPE == Arr::get($post, 'post_type')) {
-            $status = __('Submitted', 'site-reviews');
+        if( Application::POST_TYPE == Arr::get( $post, 'post_type' ) ) {
+            $status = __( 'Submitted', 'site-reviews' );
         }
         return $status;
     }
@@ -73,13 +73,13 @@ 
      * @return array
      * @filter default_hidden_columns
      */
-    public function filterDefaultHiddenColumns($hidden, $screen)
+    public function filterDefaultHiddenColumns( $hidden, $screen )
     {
-        if (Arr::get($screen, 'id') == 'edit-'.Application::POST_TYPE) {
-            $hidden = Arr::consolidateArray($hidden);
-            $hidden = array_unique(array_merge($hidden, [
+        if( Arr::get( $screen, 'id' ) == 'edit-'.Application::POST_TYPE ) {
+            $hidden = Arr::consolidateArray( $hidden );
+            $hidden = array_unique( array_merge( $hidden, [
                 'email', 'ip_address', 'response', 'reviewer',
-            ]));
+            ] ) );
         }
         return $hidden;
     }
@@ -90,28 +90,28 @@ 
      * @return array
      * @filter post_row_actions
      */
-    public function filterRowActions($actions, $post)
+    public function filterRowActions( $actions, $post )
     {
-        if (Application::POST_TYPE != Arr::get($post, 'post_type') || 'trash' == $post->post_status) {
+        if( Application::POST_TYPE != Arr::get( $post, 'post_type' ) || 'trash' == $post->post_status ) {
             return $actions;
         }
         unset($actions['inline hide-if-no-js']); //Remove Quick-edit
         $rowActions = [
-            'approve' => esc_attr__('Approve', 'site-reviews'),
-            'unapprove' => esc_attr__('Unapprove', 'site-reviews'),
+            'approve' => esc_attr__( 'Approve', 'site-reviews' ),
+            'unapprove' => esc_attr__( 'Unapprove', 'site-reviews' ),
         ];
         $newActions = [];
-        foreach ($rowActions as $key => $text) {
-            $newActions[$key] = glsr(Builder::class)->a($text, [
-                'aria-label' => sprintf(esc_attr_x('%s this review', 'Approve the review', 'site-reviews'), $text),
+        foreach( $rowActions as $key => $text ) {
+            $newActions[$key] = glsr( Builder::class )->a( $text, [
+                'aria-label' => sprintf( esc_attr_x( '%s this review', 'Approve the review', 'site-reviews' ), $text ),
                 'class' => 'glsr-change-status',
                 'href' => wp_nonce_url(
-                    admin_url('post.php?post='.$post->ID.'&action='.$key.'&plugin='.Application::ID),
+                    admin_url( 'post.php?post='.$post->ID.'&action='.$key.'&plugin='.Application::ID ),
                     $key.'-review_'.$post->ID
                 ),
-            ]);
+            ] );
         }
-        return $newActions + Arr::consolidateArray($actions);
+        return $newActions + Arr::consolidateArray( $actions );
     }
 
     /**
@@ -119,13 +119,13 @@ 
      * @return array
      * @filter manage_edit-.Application::POST_TYPE._sortable_columns
      */
-    public function filterSortableColumns($columns)
+    public function filterSortableColumns( $columns )
     {
-        $columns = Arr::consolidateArray($columns);
+        $columns = Arr::consolidateArray( $columns );
         $postTypeColumns = glsr()->postTypeColumns[Application::POST_TYPE];
         unset($postTypeColumns['cb']);
-        foreach ($postTypeColumns as $key => $value) {
-            if (Str::startsWith('taxonomy', $key)) {
+        foreach( $postTypeColumns as $key => $value ) {
+            if( Str::startsWith( 'taxonomy', $key ) ) {
                 continue;
             }
             $columns[$key] = $key;
@@ -139,10 +139,10 @@ 
      * @return void
      * @action bulk_edit_custom_box
      */
-    public function renderBulkEditFields($columnName, $postType)
+    public function renderBulkEditFields( $columnName, $postType )
     {
-        if ('assigned_to' == $columnName && Application::POST_TYPE == $postType) {
-            glsr()->render('partials/editor/bulk-edit-assigned-to');
+        if( 'assigned_to' == $columnName && Application::POST_TYPE == $postType ) {
+            glsr()->render( 'partials/editor/bulk-edit-assigned-to' );
         }
     }
 
@@ -151,9 +151,9 @@ 
      * @return void
      * @action restrict_manage_posts
      */
-    public function renderColumnFilters($postType)
+    public function renderColumnFilters( $postType )
     {
-        glsr(Columns::class)->renderFilters($postType);
+        glsr( Columns::class )->renderFilters( $postType );
     }
 
     /**
@@ -162,9 +162,9 @@ 
      * @return void
      * @action manage_posts_custom_column
      */
-    public function renderColumnValues($column, $postId)
+    public function renderColumnValues( $column, $postId )
     {
-        glsr(Columns::class)->renderValues($column, $postId);
+        glsr( Columns::class )->renderValues( $column, $postId );
     }
 
     /**
@@ -172,14 +172,14 @@ 
      * @return void
      * @action save_post_.Application::POST_TYPE
      */
-    public function saveBulkEditFields($postId)
+    public function saveBulkEditFields( $postId )
     {
-        if (!current_user_can('edit_posts')) {
+        if( !current_user_can( 'edit_posts' ) ) {
             return;
         }
-        $assignedTo = filter_input(INPUT_GET, 'assigned_to');
-        if ($assignedTo && get_post($assignedTo)) {
-            glsr(Database::class)->update($postId, 'assigned_to', $assignedTo);
+        $assignedTo = filter_input( INPUT_GET, 'assigned_to' );
+        if( $assignedTo && get_post( $assignedTo ) ) {
+            glsr( Database::class )->update( $postId, 'assigned_to', $assignedTo );
         }
     }
 
@@ -187,15 +187,15 @@ 
      * @return void
      * @action pre_get_posts
      */
-    public function setQueryForColumn(WP_Query $query)
+    public function setQueryForColumn( WP_Query $query )
     {
-        if (!$this->hasPermission($query)) {
+        if( !$this->hasPermission( $query ) ) {
             return;
         }
-        $this->setMetaQuery($query, [
+        $this->setMetaQuery( $query, [
             'rating', 'review_type',
-        ]);
-        $this->setOrderby($query);
+        ] );
+        $this->setOrderby( $query );
     }
 
     /**
@@ -204,15 +204,15 @@ 
      */
     public function unapprove()
     {
-        if (Application::ID != filter_input(INPUT_GET, 'plugin')) {
+        if( Application::ID != filter_input( INPUT_GET, 'plugin' ) ) {
             return;
         }
-        check_admin_referer('unapprove-review_'.($postId = $this->getPostId()));
-        wp_update_post([
+        check_admin_referer( 'unapprove-review_'.($postId = $this->getPostId()) );
+        wp_update_post( [
             'ID' => $postId,
             'post_status' => 'pending',
-        ]);
-        wp_safe_redirect(wp_get_referer());
+        ] );
+        wp_safe_redirect( wp_get_referer() );
         exit;
     }
 
@@ -221,7 +221,7 @@ 
      * @param string $domain
      * @return bool
      */
-    protected function canModifyTranslation($domain = 'default')
+    protected function canModifyTranslation( $domain = 'default' )
     {
         $screen = glsr_current_screen();
         return 'default' == $domain
@@ -232,48 +232,48 @@ 
     /**
      * @return bool
      */
-    protected function hasPermission(WP_Query $query)
+    protected function hasPermission( WP_Query $query )
     {
         global $pagenow;
         return is_admin()
             && $query->is_main_query()
-            && Application::POST_TYPE == $query->get('post_type')
+            && Application::POST_TYPE == $query->get( 'post_type' )
             && 'edit.php' == $pagenow;
     }
 
     /**
      * @return void
      */
-    protected function setMetaQuery(WP_Query $query, array $metaKeys)
+    protected function setMetaQuery( WP_Query $query, array $metaKeys )
     {
-        foreach ($metaKeys as $key) {
-            $value = (string) filter_input(INPUT_GET, $key);
-            if ('' === $value) {
+        foreach( $metaKeys as $key ) {
+            $value = (string)filter_input( INPUT_GET, $key );
+            if( '' === $value ) {
                 continue;
             }
-            $metaQuery = (array) $query->get('meta_query');
+            $metaQuery = (array)$query->get( 'meta_query' );
             $metaQuery[] = [
-                'key' => Str::prefix('_', $key, '_'),
+                'key' => Str::prefix( '_', $key, '_' ),
                 'value' => $value,
             ];
-            $query->set('meta_query', array_filter($metaQuery));
+            $query->set( 'meta_query', array_filter( $metaQuery ) );
         }
     }
 
     /**
      * @return void
      */
-    protected function setOrderby(WP_Query $query)
+    protected function setOrderby( WP_Query $query )
     {
-        $orderby = $query->get('orderby');
+        $orderby = $query->get( 'orderby' );
         $columns = glsr()->postTypeColumns[Application::POST_TYPE];
         unset($columns['cb'], $columns['title'], $columns['date']);
-        if (in_array($orderby, array_keys($columns))) {
-            if ('reviewer' == $orderby) {
+        if( in_array( $orderby, array_keys( $columns ) ) ) {
+            if( 'reviewer' == $orderby ) {
                 $orderby = 'author';
             }
-            $query->set('meta_key', Str::prefix('_', $orderby, '_'));
-            $query->set('orderby', 'meta_value');
+            $query->set( 'meta_key', Str::prefix( '_', $orderby, '_' ) );
+            $query->set( 'orderby', 'meta_value' );
         }
     }
 }

plugin/Database.php

Spacing +47 added lines, -47 removed lines

@@ -18,10 +18,10 @@ 
      * @param bool $single
      * @return mixed
      */
-    public function get($postId, $key, $single = true)
+    public function get( $postId, $key, $single = true )
     {
-        $key = Str::prefix('_', $key);
-        return get_post_meta(intval($postId), $key, $single);
+        $key = Str::prefix( '_', $key );
+        return get_post_meta( intval( $postId ), $key, $single );
     }
 
     /**
@@ -29,16 +29,16 @@ 
      * @param string $assignedTo
      * @return void|WP_Post
      */
-    public function getAssignedToPost($postId, $assignedTo = '')
+    public function getAssignedToPost( $postId, $assignedTo = '' )
     {
-        if (empty($assignedTo)) {
-            $assignedTo = $this->get($postId, 'assigned_to');
+        if( empty($assignedTo) ) {
+            $assignedTo = $this->get( $postId, 'assigned_to' );
         }
-        if (empty($assignedTo)) {
+        if( empty($assignedTo) ) {
             return;
         }
-        $assignedPost = get_post($assignedTo);
-        if ($assignedPost instanceof WP_Post && $assignedPost->ID != $postId) {
+        $assignedPost = get_post( $assignedTo );
+        if( $assignedPost instanceof WP_Post && $assignedPost->ID != $postId ) {
             return $assignedPost;
         }
     }
@@ -48,25 +48,25 @@ 
      * @param string $metaValue
      * @return array|int
      */
-    public function getReviewCount($metaKey = '', $metaValue = '')
+    public function getReviewCount( $metaKey = '', $metaValue = '' )
     {
-        if (!$metaKey) {
-            return (array) wp_count_posts(Application::POST_TYPE);
+        if( !$metaKey ) {
+            return (array)wp_count_posts( Application::POST_TYPE );
         }
-        $counts = glsr(Cache::class)->getReviewCountsFor($metaKey);
-        if (!$metaValue) {
+        $counts = glsr( Cache::class )->getReviewCountsFor( $metaKey );
+        if( !$metaValue ) {
             return $counts;
         }
-        return Arr::get($counts, $metaValue, 0);
+        return Arr::get( $counts, $metaValue, 0 );
     }
 
     /**
      * @param string $metaReviewType
      * @return array
      */
-    public function getReviewIdsByType($metaReviewType)
+    public function getReviewIdsByType( $metaReviewType )
     {
-        return glsr(SqlQueries::class)->getReviewIdsByType($metaReviewType);
+        return glsr( SqlQueries::class )->getReviewIdsByType( $metaReviewType );
     }
 
     /**
@@ -74,21 +74,21 @@ 
      * @param string $status
      * @return array
      */
-    public function getReviewsMeta($key, $status = 'publish')
+    public function getReviewsMeta( $key, $status = 'publish' )
     {
-        return glsr(SqlQueries::class)->getReviewsMeta($key, $status);
+        return glsr( SqlQueries::class )->getReviewsMeta( $key, $status );
     }
 
     /**
      * @param string $field
      * @return array
      */
-    public function getTermIds(array $values, $field)
+    public function getTermIds( array $values, $field )
     {
         $termIds = [];
-        foreach ($values as $value) {
-            $term = get_term_by($field, $value, Application::TAXONOMY);
-            if (!isset($term->term_id)) {
+        foreach( $values as $value ) {
+            $term = get_term_by( $field, $value, Application::TAXONOMY );
+            if( !isset($term->term_id) ) {
                 continue;
             }
             $termIds[] = $term->term_id;
@@ -99,17 +99,17 @@ 
     /**
      * @return array
      */
-    public function getTerms(array $args = [])
+    public function getTerms( array $args = [] )
     {
-        $args = wp_parse_args($args, [
+        $args = wp_parse_args( $args, [
             'count' => false,
             'fields' => 'id=>name',
             'hide_empty' => false,
             'taxonomy' => Application::TAXONOMY,
-        ]);
-        $terms = get_terms($args);
-        if (is_wp_error($terms)) {
-            glsr_log()->error($terms->get_error_message());
+        ] );
+        $terms = get_terms( $args );
+        if( is_wp_error( $terms ) ) {
+            glsr_log()->error( $terms->get_error_message() );
             return [];
         }
         return $terms;
@@ -119,36 +119,36 @@ 
      * @param string $searchTerm
      * @return void|string
      */
-    public function searchPosts($searchTerm)
+    public function searchPosts( $searchTerm )
     {
         $args = [
             'post_status' => 'publish',
             'post_type' => 'any',
         ];
-        if (is_numeric($searchTerm)) {
+        if( is_numeric( $searchTerm ) ) {
             $args['post__in'] = [$searchTerm];
         } else {
             $args['orderby'] = 'relevance';
             $args['posts_per_page'] = 10;
             $args['s'] = $searchTerm;
         }
-        $queryBuilder = glsr(QueryBuilder::class);
-        add_filter('posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2);
-        $search = new WP_Query($args);
-        remove_filter('posts_search', [$queryBuilder, 'filterSearchByTitle'], 500);
-        if (!$search->have_posts()) {
+        $queryBuilder = glsr( QueryBuilder::class );
+        add_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500, 2 );
+        $search = new WP_Query( $args );
+        remove_filter( 'posts_search', [$queryBuilder, 'filterSearchByTitle'], 500 );
+        if( !$search->have_posts() ) {
             return;
         }
         $results = '';
-        while ($search->have_posts()) {
+        while( $search->have_posts() ) {
             $search->the_post();
             ob_start();
-            glsr()->render('partials/editor/search-result', [
+            glsr()->render( 'partials/editor/search-result', [
                 'ID' => get_the_ID(),
-                'permalink' => esc_url((string) get_permalink()),
-                'title' => esc_attr(get_the_title()),
-            ]);
-            $results.= ob_get_clean();
+                'permalink' => esc_url( (string)get_permalink() ),
+                'title' => esc_attr( get_the_title() ),
+            ] );
+            $results .= ob_get_clean();
         }
         wp_reset_postdata();
         return $results;
@@ -160,10 +160,10 @@ 
      * @param mixed $value
      * @return int|bool
      */
-    public function set($postId, $key, $value)
+    public function set( $postId, $key, $value )
     {
-        $key = Str::prefix('_', $key);
-        return update_post_meta($postId, $key, $value);
+        $key = Str::prefix( '_', $key );
+        return update_post_meta( $postId, $key, $value );
     }
 
     /**
@@ -172,8 +172,8 @@ 
      * @param mixed $value
      * @return int|bool
      */
-    public function update($postId, $key, $value)
+    public function update( $postId, $key, $value )
     {
-        return $this->set($postId, $key, $value);
+        return $this->set( $postId, $key, $value );
     }
 }

plugin/Shortcodes/SiteReviewsFormShortcode.php

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

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/Helpers/Arr.php

Spacing +56 added lines, -56 removed lines

@@ -7,10 +7,10 @@ 
     /**
      * @return bool
      */
-    public static function compareArrays(array $arr1, array $arr2)
+    public static function compareArrays( array $arr1, array $arr2 )
     {
-        sort($arr1);
-        sort($arr2);
+        sort( $arr1 );
+        sort( $arr2 );
         return $arr1 == $arr2;
     }
 
@@ -18,21 +18,21 @@ 
      * @param mixed $array
      * @return array
      */
-    public static function consolidateArray($array)
+    public static function consolidateArray( $array )
     {
-        return is_array($array) || is_object($array)
-            ? (array) $array
+        return is_array( $array ) || is_object( $array )
+            ? (array)$array
             : [];
     }
 
     /**
      * @return array
      */
-    public static function convertDotNotationArray(array $array)
+    public static function convertDotNotationArray( array $array )
     {
         $results = [];
-        foreach ($array as $path => $value) {
-            $results = static::set($results, $path, $value);
+        foreach( $array as $path => $value ) {
+            $results = static::set( $results, $path, $value );
         }
         return $results;
     }
@@ -42,12 +42,12 @@ 
      * @param mixed $callback
      * @return array
      */
-    public static function convertStringToArray($string, $callback = null)
+    public static function convertStringToArray( $string, $callback = null )
     {
-        $array = array_map('trim', explode(',', $string));
+        $array = array_map( 'trim', explode( ',', $string ) );
         return $callback
-            ? array_filter($array, $callback)
-            : array_filter($array);
+            ? array_filter( $array, $callback )
+            : array_filter( $array );
     }
 
     /**
@@ -55,17 +55,17 @@ 
      * @param string $prefix
      * @return array
      */
-    public static function flattenArray(array $array, $flattenValue = false, $prefix = '')
+    public static function flattenArray( array $array, $flattenValue = false, $prefix = '' )
     {
         $result = [];
-        foreach ($array as $key => $value) {
-            $newKey = ltrim($prefix.'.'.$key, '.');
-            if (static::isIndexedFlatArray($value)) {
-                if ($flattenValue) {
-                    $value = '['.implode(', ', $value).']';
+        foreach( $array as $key => $value ) {
+            $newKey = ltrim( $prefix.'.'.$key, '.' );
+            if( static::isIndexedFlatArray( $value ) ) {
+                if( $flattenValue ) {
+                    $value = '['.implode( ', ', $value ).']';
                 }
-            } elseif (is_array($value)) {
-                $result = array_merge($result, static::flattenArray($value, $flattenValue, $newKey));
+            } elseif( is_array( $value ) ) {
+                $result = array_merge( $result, static::flattenArray( $value, $flattenValue, $newKey ) );
                 continue;
             }
             $result[$newKey] = $value;
@@ -80,12 +80,12 @@ 
      * @param mixed $fallback
      * @return mixed
      */
-    public static function get($data, $path = '', $fallback = '')
+    public static function get( $data, $path = '', $fallback = '' )
     {
-        $data = static::consolidateArray($data);
-        $keys = explode('.', $path);
-        foreach ($keys as $key) {
-            if (!isset($data[$key])) {
+        $data = static::consolidateArray( $data );
+        $keys = explode( '.', $path );
+        foreach( $keys as $key ) {
+            if( !isset($data[$key]) ) {
                 return $fallback;
             }
             $data = $data[$key];
@@ -98,47 +98,47 @@ 
      * @param string $position
      * @return array
      */
-    public static function insertInArray(array $array, array $insert, $key, $position = 'before')
+    public static function insertInArray( array $array, array $insert, $key, $position = 'before' )
     {
-        $keyPosition = intval(array_search($key, array_keys($array)));
-        if ('after' == $position) {
+        $keyPosition = intval( array_search( $key, array_keys( $array ) ) );
+        if( 'after' == $position ) {
             ++$keyPosition;
         }
-        if (false !== $keyPosition) {
-            $result = array_slice($array, 0, $keyPosition);
-            $result = array_merge($result, $insert);
-            return array_merge($result, array_slice($array, $keyPosition));
+        if( false !== $keyPosition ) {
+            $result = array_slice( $array, 0, $keyPosition );
+            $result = array_merge( $result, $insert );
+            return array_merge( $result, array_slice( $array, $keyPosition ) );
         }
-        return array_merge($array, $insert);
+        return array_merge( $array, $insert );
     }
 
     /**
      * @param mixed $array
      * @return bool
      */
-    public static function isIndexedFlatArray($array)
+    public static function isIndexedFlatArray( $array )
     {
-        if (!is_array($array) || array_filter($array, 'is_array')) {
+        if( !is_array( $array ) || array_filter( $array, 'is_array' ) ) {
             return false;
         }
-        return wp_is_numeric_array($array);
+        return wp_is_numeric_array( $array );
     }
 
     /**
      * @param bool $prefixed
      * @return array
      */
-    public static function prefixArrayKeys(array $values, $prefixed = true)
+    public static function prefixArrayKeys( array $values, $prefixed = true )
     {
         $trim = '_';
         $prefix = $prefixed
             ? $trim
             : '';
         $prefixed = [];
-        foreach ($values as $key => $value) {
-            $key = trim($key);
-            if (0 === strpos($key, $trim)) {
-                $key = substr($key, strlen($trim));
+        foreach( $values as $key => $value ) {
+            $key = trim( $key );
+            if( 0 === strpos( $key, $trim ) ) {
+                $key = substr( $key, strlen( $trim ) );
             }
             $prefixed[$prefix.$key] = $value;
         }
@@ -148,15 +148,15 @@ 
     /**
      * @return array
      */
-    public static function removeEmptyArrayValues(array $array)
+    public static function removeEmptyArrayValues( array $array )
     {
         $result = [];
-        foreach ($array as $key => $value) {
-            if (!$value) {
+        foreach( $array as $key => $value ) {
+            if( !$value ) {
                 continue;
             }
-            $result[$key] = is_array($value)
-                ? static::removeEmptyArrayValues($value)
+            $result[$key] = is_array( $value )
+                ? static::removeEmptyArrayValues( $value )
                 : $value;
         }
         return $result;
@@ -169,14 +169,14 @@ 
      * @param mixed $value
      * @return array
      */
-    public static function set(array $data, $path, $value)
+    public static function set( array $data, $path, $value )
     {
-        $token = strtok($path, '.');
+        $token = strtok( $path, '.' );
         $ref = &$data;
-        while (false !== $token) {
-            $ref = static::consolidateArray($ref);
+        while( false !== $token ) {
+            $ref = static::consolidateArray( $ref );
             $ref = &$ref[$token];
-            $token = strtok('.');
+            $token = strtok( '.' );
         }
         $ref = $value;
         return $data;
@@ -185,16 +185,16 @@ 
     /**
      * @return array
      */
-    public static function unique(array $values)
+    public static function unique( array $values )
     {
-        return array_filter(array_unique($values));
+        return array_filter( array_unique( $values ) );
     }
 
     /**
      * @return array
      */
-    public static function unprefixArrayKeys(array $values)
+    public static function unprefixArrayKeys( array $values )
     {
-        return static::prefixArrayKeys($values, false);
+        return static::prefixArrayKeys( $values, false );
     }
 }

plugin/Modules/Multilingual.php

Spacing +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

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/Multilingual/Wpml.php

Spacing +19 added lines, -19 removed lines

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