pryley / site-reviews

plugin/Router.php

Indentation +147 added lines, -147 removed lines

@@ -7,162 +7,162 @@
 
 class Router
 {
-    /**
-     * @var array
-     */
-    protected $unguardedActions = [];
+	/**
+	 * @var array
+	 */
+	protected $unguardedActions = [];
 
-    public function __construct()
-    {
-        $this->unguardedActions = apply_filters('site-reviews/router/unguarded-actions', [
-            'dismiss-notice',
-            'fetch-paged-reviews',
-        ]);
-    }
+	public function __construct()
+	{
+		$this->unguardedActions = apply_filters('site-reviews/router/unguarded-actions', [
+			'dismiss-notice',
+			'fetch-paged-reviews',
+		]);
+	}
 
-    /**
-     * @return void
-     */
-    public function routeAdminPostRequest()
-    {
-        $request = $this->getRequest();
-        if (!$this->isValidPostRequest($request)) {
-            return;
-        }
-        check_admin_referer($request['_action']);
-        $this->routeRequest('admin', $request['_action'], $request);
-    }
+	/**
+	 * @return void
+	 */
+	public function routeAdminPostRequest()
+	{
+		$request = $this->getRequest();
+		if (!$this->isValidPostRequest($request)) {
+			return;
+		}
+		check_admin_referer($request['_action']);
+		$this->routeRequest('admin', $request['_action'], $request);
+	}
 
-    /**
-     * @return void
-     */
-    public function routeAjaxRequest()
-    {
-        $request = $this->getRequest();
-        $this->checkAjaxRequest($request);
-        $this->checkAjaxNonce($request);
-        $this->routeRequest('ajax', $request['_action'], $request);
-        wp_die();
-    }
+	/**
+	 * @return void
+	 */
+	public function routeAjaxRequest()
+	{
+		$request = $this->getRequest();
+		$this->checkAjaxRequest($request);
+		$this->checkAjaxNonce($request);
+		$this->routeRequest('ajax', $request['_action'], $request);
+		wp_die();
+	}
 
-    /**
-     * @return void
-     */
-    public function routePublicPostRequest()
-    {
-        if (is_admin()) {
-            return;
-        }
-        $request = $this->getRequest();
-        if (!$this->isValidPostRequest($request)) {
-            return;
-        }
-        if (!$this->isValidPublicNonce($request)) {
-            return;
-        }
-        $this->routeRequest('public', $request['_action'], $request);
-    }
+	/**
+	 * @return void
+	 */
+	public function routePublicPostRequest()
+	{
+		if (is_admin()) {
+			return;
+		}
+		$request = $this->getRequest();
+		if (!$this->isValidPostRequest($request)) {
+			return;
+		}
+		if (!$this->isValidPublicNonce($request)) {
+			return;
+		}
+		$this->routeRequest('public', $request['_action'], $request);
+	}
 
-    /**
-     * @return void
-     */
-    protected function checkAjaxNonce(array $request)
-    {
-        if (!is_user_logged_in() || in_array(Arr::get($request, '_action'), $this->unguardedActions)) {
-            return;
-        }
-        if (!isset($request['_nonce'])) {
-            $this->sendAjaxError('request is missing a nonce', $request);
-        }
-        if (!wp_verify_nonce($request['_nonce'], $request['_action'])) {
-            $this->sendAjaxError('request failed the nonce check', $request, 403);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	protected function checkAjaxNonce(array $request)
+	{
+		if (!is_user_logged_in() || in_array(Arr::get($request, '_action'), $this->unguardedActions)) {
+			return;
+		}
+		if (!isset($request['_nonce'])) {
+			$this->sendAjaxError('request is missing a nonce', $request);
+		}
+		if (!wp_verify_nonce($request['_nonce'], $request['_action'])) {
+			$this->sendAjaxError('request failed the nonce check', $request, 403);
+		}
+	}
 
-    /**
-     * @return void
-     */
-    protected function checkAjaxRequest(array $request)
-    {
-        if (!isset($request['_action'])) {
-            $this->sendAjaxError('request must include an action', $request);
-        }
-        if (empty($request['_ajax_request'])) {
-            $this->sendAjaxError('request is invalid', $request);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	protected function checkAjaxRequest(array $request)
+	{
+		if (!isset($request['_action'])) {
+			$this->sendAjaxError('request must include an action', $request);
+		}
+		if (empty($request['_ajax_request'])) {
+			$this->sendAjaxError('request is invalid', $request);
+		}
+	}
 
-    /**
-     * All ajax requests in the plugin are triggered by a single action hook: glsr_action,
-     * while each ajax route is determined by $_POST[request][_action].
-     * @return array
-     */
-    protected function getRequest()
-    {
-        $request = Helper::filterInputArray(Application::ID);
-        if (Helper::filterInput('action') == Application::PREFIX.'action') {
-            $request['_ajax_request'] = true;
-        }
-        if ('submit-review' == Helper::filterInput('_action', $request)) {
-            $request['_recaptcha-token'] = Helper::filterInput('g-recaptcha-response');
-        }
-        return $request;
-    }
+	/**
+	 * All ajax requests in the plugin are triggered by a single action hook: glsr_action,
+	 * while each ajax route is determined by $_POST[request][_action].
+	 * @return array
+	 */
+	protected function getRequest()
+	{
+		$request = Helper::filterInputArray(Application::ID);
+		if (Helper::filterInput('action') == Application::PREFIX.'action') {
+			$request['_ajax_request'] = true;
+		}
+		if ('submit-review' == Helper::filterInput('_action', $request)) {
+			$request['_recaptcha-token'] = Helper::filterInput('g-recaptcha-response');
+		}
+		return $request;
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isValidPostRequest(array $request = [])
-    {
-        return !empty($request['_action']) && empty($request['_ajax_request']);
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isValidPostRequest(array $request = [])
+	{
+		return !empty($request['_action']) && empty($request['_ajax_request']);
+	}
 
-    /**
-     * @return bool
-     */
-    protected function isValidPublicNonce(array $request)
-    {
-        if (is_user_logged_in() && !wp_verify_nonce($request['_nonce'], $request['_action'])) {
-            glsr_log()->error('nonce check failed for public request')->debug($request);
-            return false;
-        }
-        return true;
-    }
+	/**
+	 * @return bool
+	 */
+	protected function isValidPublicNonce(array $request)
+	{
+		if (is_user_logged_in() && !wp_verify_nonce($request['_nonce'], $request['_action'])) {
+			glsr_log()->error('nonce check failed for public request')->debug($request);
+			return false;
+		}
+		return true;
+	}
 
-    /**
-     * @param string $type
-     * @param string $action
-     * @return void
-     */
-    protected function routeRequest($type, $action, array $request = [])
-    {
-        $actionHook = 'site-reviews/route/'.$type.'/request';
-        $controller = glsr(Helper::buildClassName($type.'-controller', 'Controllers'));
-        $method = Helper::buildMethodName($action, 'router');
-        $request = apply_filters('site-reviews/route/request', $request, $action, $type);
-        do_action($actionHook, $action, $request);
-        if (is_callable([$controller, $method])) {
-            call_user_func([$controller, $method], $request);
-            return;
-        }
-        if (0 === did_action($actionHook)) {
-            glsr_log('Unknown '.$type.' router request: '.$action);
-        }
-    }
+	/**
+	 * @param string $type
+	 * @param string $action
+	 * @return void
+	 */
+	protected function routeRequest($type, $action, array $request = [])
+	{
+		$actionHook = 'site-reviews/route/'.$type.'/request';
+		$controller = glsr(Helper::buildClassName($type.'-controller', 'Controllers'));
+		$method = Helper::buildMethodName($action, 'router');
+		$request = apply_filters('site-reviews/route/request', $request, $action, $type);
+		do_action($actionHook, $action, $request);
+		if (is_callable([$controller, $method])) {
+			call_user_func([$controller, $method], $request);
+			return;
+		}
+		if (0 === did_action($actionHook)) {
+			glsr_log('Unknown '.$type.' router request: '.$action);
+		}
+	}
 
-    /**
-     * @param string $error
-     * @param int $statusCode
-     * @return void
-     */
-    protected function sendAjaxError($error, array $request, $statusCode = 400)
-    {
-        glsr_log()->error($error)->debug($request);
-        glsr(Notice::class)->addError(__('There was an error (try reloading the page).', 'site-reviews').' <code>'.$error.'</code>');
-        wp_send_json_error([
-            'message' => __('The form could not be submitted. Please notify the site administrator.', 'site-reviews'),
-            'notices' => glsr(Notice::class)->get(),
-            'error' => $error,
-        ]);
-    }
+	/**
+	 * @param string $error
+	 * @param int $statusCode
+	 * @return void
+	 */
+	protected function sendAjaxError($error, array $request, $statusCode = 400)
+	{
+		glsr_log()->error($error)->debug($request);
+		glsr(Notice::class)->addError(__('There was an error (try reloading the page).', 'site-reviews').' <code>'.$error.'</code>');
+		wp_send_json_error([
+			'message' => __('The form could not be submitted. Please notify the site administrator.', 'site-reviews'),
+			'notices' => glsr(Notice::class)->get(),
+			'error' => $error,
+		]);
+	}
 }

Spacing +47 added lines, -47 removed lines

@@ -14,10 +14,10 @@ 
 
     public function __construct()
     {
-        $this->unguardedActions = apply_filters('site-reviews/router/unguarded-actions', [
+        $this->unguardedActions = apply_filters( 'site-reviews/router/unguarded-actions', [
             'dismiss-notice',
             'fetch-paged-reviews',
-        ]);
+        ] );
     }
 
     /**
@@ -26,11 +26,11 @@ 
     public function routeAdminPostRequest()
     {
         $request = $this->getRequest();
-        if (!$this->isValidPostRequest($request)) {
+        if( !$this->isValidPostRequest( $request ) ) {
             return;
         }
-        check_admin_referer($request['_action']);
-        $this->routeRequest('admin', $request['_action'], $request);
+        check_admin_referer( $request['_action'] );
+        $this->routeRequest( 'admin', $request['_action'], $request );
     }
 
     /**
@@ -39,9 +39,9 @@ 
     public function routeAjaxRequest()
     {
         $request = $this->getRequest();
-        $this->checkAjaxRequest($request);
-        $this->checkAjaxNonce($request);
-        $this->routeRequest('ajax', $request['_action'], $request);
+        $this->checkAjaxRequest( $request );
+        $this->checkAjaxNonce( $request );
+        $this->routeRequest( 'ajax', $request['_action'], $request );
         wp_die();
     }
 
@@ -50,45 +50,45 @@ 
      */
     public function routePublicPostRequest()
     {
-        if (is_admin()) {
+        if( is_admin() ) {
             return;
         }
         $request = $this->getRequest();
-        if (!$this->isValidPostRequest($request)) {
+        if( !$this->isValidPostRequest( $request ) ) {
             return;
         }
-        if (!$this->isValidPublicNonce($request)) {
+        if( !$this->isValidPublicNonce( $request ) ) {
             return;
         }
-        $this->routeRequest('public', $request['_action'], $request);
+        $this->routeRequest( 'public', $request['_action'], $request );
     }
 
     /**
      * @return void
      */
-    protected function checkAjaxNonce(array $request)
+    protected function checkAjaxNonce( array $request )
     {
-        if (!is_user_logged_in() || in_array(Arr::get($request, '_action'), $this->unguardedActions)) {
+        if( !is_user_logged_in() || in_array( Arr::get( $request, '_action' ), $this->unguardedActions ) ) {
             return;
         }
-        if (!isset($request['_nonce'])) {
-            $this->sendAjaxError('request is missing a nonce', $request);
+        if( !isset($request['_nonce']) ) {
+            $this->sendAjaxError( 'request is missing a nonce', $request );
         }
-        if (!wp_verify_nonce($request['_nonce'], $request['_action'])) {
-            $this->sendAjaxError('request failed the nonce check', $request, 403);
+        if( !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
+            $this->sendAjaxError( 'request failed the nonce check', $request, 403 );
         }
     }
 
     /**
      * @return void
      */
-    protected function checkAjaxRequest(array $request)
+    protected function checkAjaxRequest( array $request )
     {
-        if (!isset($request['_action'])) {
-            $this->sendAjaxError('request must include an action', $request);
+        if( !isset($request['_action']) ) {
+            $this->sendAjaxError( 'request must include an action', $request );
         }
-        if (empty($request['_ajax_request'])) {
-            $this->sendAjaxError('request is invalid', $request);
+        if( empty($request['_ajax_request']) ) {
+            $this->sendAjaxError( 'request is invalid', $request );
         }
     }
 
@@ -99,12 +99,12 @@ 
      */
     protected function getRequest()
     {
-        $request = Helper::filterInputArray(Application::ID);
-        if (Helper::filterInput('action') == Application::PREFIX.'action') {
+        $request = Helper::filterInputArray( Application::ID );
+        if( Helper::filterInput( 'action' ) == Application::PREFIX.'action' ) {
             $request['_ajax_request'] = true;
         }
-        if ('submit-review' == Helper::filterInput('_action', $request)) {
-            $request['_recaptcha-token'] = Helper::filterInput('g-recaptcha-response');
+        if( 'submit-review' == Helper::filterInput( '_action', $request ) ) {
+            $request['_recaptcha-token'] = Helper::filterInput( 'g-recaptcha-response' );
         }
         return $request;
     }
@@ -112,7 +112,7 @@ 
     /**
      * @return bool
      */
-    protected function isValidPostRequest(array $request = [])
+    protected function isValidPostRequest( array $request = [] )
     {
         return !empty($request['_action']) && empty($request['_ajax_request']);
     }
@@ -120,10 +120,10 @@ 
     /**
      * @return bool
      */
-    protected function isValidPublicNonce(array $request)
+    protected function isValidPublicNonce( array $request )
     {
-        if (is_user_logged_in() && !wp_verify_nonce($request['_nonce'], $request['_action'])) {
-            glsr_log()->error('nonce check failed for public request')->debug($request);
+        if( is_user_logged_in() && !wp_verify_nonce( $request['_nonce'], $request['_action'] ) ) {
+            glsr_log()->error( 'nonce check failed for public request' )->debug( $request );
             return false;
         }
         return true;
@@ -134,19 +134,19 @@ 
      * @param string $action
      * @return void
      */
-    protected function routeRequest($type, $action, array $request = [])
+    protected function routeRequest( $type, $action, array $request = [] )
     {
         $actionHook = 'site-reviews/route/'.$type.'/request';
-        $controller = glsr(Helper::buildClassName($type.'-controller', 'Controllers'));
-        $method = Helper::buildMethodName($action, 'router');
-        $request = apply_filters('site-reviews/route/request', $request, $action, $type);
-        do_action($actionHook, $action, $request);
-        if (is_callable([$controller, $method])) {
-            call_user_func([$controller, $method], $request);
+        $controller = glsr( Helper::buildClassName( $type.'-controller', 'Controllers' ) );
+        $method = Helper::buildMethodName( $action, 'router' );
+        $request = apply_filters( 'site-reviews/route/request', $request, $action, $type );
+        do_action( $actionHook, $action, $request );
+        if( is_callable( [$controller, $method] ) ) {
+            call_user_func( [$controller, $method], $request );
             return;
         }
-        if (0 === did_action($actionHook)) {
-            glsr_log('Unknown '.$type.' router request: '.$action);
+        if( 0 === did_action( $actionHook ) ) {
+            glsr_log( 'Unknown '.$type.' router request: '.$action );
         }
     }
 
@@ -155,14 +155,14 @@ 
      * @param int $statusCode
      * @return void
      */
-    protected function sendAjaxError($error, array $request, $statusCode = 400)
+    protected function sendAjaxError( $error, array $request, $statusCode = 400 )
     {
-        glsr_log()->error($error)->debug($request);
-        glsr(Notice::class)->addError(__('There was an error (try reloading the page).', 'site-reviews').' <code>'.$error.'</code>');
-        wp_send_json_error([
-            'message' => __('The form could not be submitted. Please notify the site administrator.', 'site-reviews'),
-            'notices' => glsr(Notice::class)->get(),
+        glsr_log()->error( $error )->debug( $request );
+        glsr( Notice::class )->addError( __( 'There was an error (try reloading the page).', 'site-reviews' ).' <code>'.$error.'</code>' );
+        wp_send_json_error( [
+            'message' => __( 'The form could not be submitted. Please notify the site administrator.', 'site-reviews' ),
+            'notices' => glsr( Notice::class )->get(),
             'error' => $error,
-        ]);
+        ] );
     }
 }

plugin/Controllers/TaxonomyController.php

Indentation +88 added lines, -88 removed lines

@@ -8,97 +8,97 @@
 
 class TaxonomyController
 {
-    /**
-     * @return void
-     * @action Application::TAXONOMY._add_form_fields
-     * @action Application::TAXONOMY._edit_form
-     */
-    public function disableParents()
-    {
-        global $wp_taxonomies;
-        $wp_taxonomies[Application::TAXONOMY]->hierarchical = false;
-    }
+	/**
+	 * @return void
+	 * @action Application::TAXONOMY._add_form_fields
+	 * @action Application::TAXONOMY._edit_form
+	 */
+	public function disableParents()
+	{
+		global $wp_taxonomies;
+		$wp_taxonomies[Application::TAXONOMY]->hierarchical = false;
+	}
 
-    /**
-     * @return void
-     * @action Application::TAXONOMY._term_edit_form_top
-     * @action Application::TAXONOMY._term_new_form_tag
-     */
-    public function enableParents()
-    {
-        global $wp_taxonomies;
-        $wp_taxonomies[Application::TAXONOMY]->hierarchical = true;
-    }
+	/**
+	 * @return void
+	 * @action Application::TAXONOMY._term_edit_form_top
+	 * @action Application::TAXONOMY._term_new_form_tag
+	 */
+	public function enableParents()
+	{
+		global $wp_taxonomies;
+		$wp_taxonomies[Application::TAXONOMY]->hierarchical = true;
+	}
 
-    /**
-     * @return void
-     * @action restrict_manage_posts
-     */
-    public function renderTaxonomyFilter()
-    {
-        if (!is_object_in_taxonomy(glsr_current_screen()->post_type, Application::TAXONOMY)) {
-            return;
-        }
-        echo glsr(Builder::class)->label(__('Filter by category', 'site-reviews'), [
-            'class' => 'screen-reader-text',
-            'for' => Application::TAXONOMY,
-        ]);
-        wp_dropdown_categories([
-            'depth' => 3,
-            'hide_empty' => true,
-            'hide_if_empty' => true,
-            'hierarchical' => true,
-            'name' => Application::TAXONOMY,
-            'orderby' => 'name',
-            'selected' => $this->getSelected(),
-            'show_count' => false,
-            'show_option_all' => $this->getShowOptionAll(),
-            'taxonomy' => Application::TAXONOMY,
-            'value_field' => 'slug',
-        ]);
-    }
+	/**
+	 * @return void
+	 * @action restrict_manage_posts
+	 */
+	public function renderTaxonomyFilter()
+	{
+		if (!is_object_in_taxonomy(glsr_current_screen()->post_type, Application::TAXONOMY)) {
+			return;
+		}
+		echo glsr(Builder::class)->label(__('Filter by category', 'site-reviews'), [
+			'class' => 'screen-reader-text',
+			'for' => Application::TAXONOMY,
+		]);
+		wp_dropdown_categories([
+			'depth' => 3,
+			'hide_empty' => true,
+			'hide_if_empty' => true,
+			'hierarchical' => true,
+			'name' => Application::TAXONOMY,
+			'orderby' => 'name',
+			'selected' => $this->getSelected(),
+			'show_count' => false,
+			'show_option_all' => $this->getShowOptionAll(),
+			'taxonomy' => Application::TAXONOMY,
+			'value_field' => 'slug',
+		]);
+	}
 
-    /**
-     * @param int $postId
-     * @param array $terms
-     * @param array $newTTIds
-     * @param string $taxonomy
-     * @param bool $append
-     * @param array $oldTTIds
-     * @return void
-     * @action set_object_terms
-     */
-    public function restrictTermSelection($postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds)
-    {
-        if (Application::TAXONOMY != $taxonomy || count($newTTIds) <= 1) {
-            return;
-        }
-        $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);
-        }
-    }
+	/**
+	 * @param int $postId
+	 * @param array $terms
+	 * @param array $newTTIds
+	 * @param string $taxonomy
+	 * @param bool $append
+	 * @param array $oldTTIds
+	 * @return void
+	 * @action set_object_terms
+	 */
+	public function restrictTermSelection($postId, $terms, $newTTIds, $taxonomy, $append, $oldTTIds)
+	{
+		if (Application::TAXONOMY != $taxonomy || count($newTTIds) <= 1) {
+			return;
+		}
+		$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);
+		}
+	}
 
-    /**
-     * @return string
-     */
-    protected function getSelected()
-    {
-        global $wp_query;
-        return Arr::get($wp_query->query, Application::TAXONOMY);
-    }
+	/**
+	 * @return string
+	 */
+	protected function getSelected()
+	{
+		global $wp_query;
+		return Arr::get($wp_query->query, Application::TAXONOMY);
+	}
 
-    /**
-     * @return string
-     */
-    protected function getShowOptionAll()
-    {
-        $taxonomy = get_taxonomy(Application::TAXONOMY);
-        return $taxonomy
-            ? ucfirst(strtolower($taxonomy->labels->all_items))
-            : '';
-    }
+	/**
+	 * @return string
+	 */
+	protected function getShowOptionAll()
+	{
+		$taxonomy = get_taxonomy(Application::TAXONOMY);
+		return $taxonomy
+			? ucfirst(strtolower($taxonomy->labels->all_items))
+			: '';
+	}
 }

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/Modules/Schema/BaseType.php

Indentation +221 added lines, -221 removed lines

@@ -13,248 +13,248 @@
 
 abstract class BaseType implements ArrayAccess, JsonSerializable, Type
 {
-    /**
-     * @var array
-     */
-    public $allowed = [];
+	/**
+	 * @var array
+	 */
+	public $allowed = [];
 
-    /**
-     * @var array
-     */
-    public $parents = [];
+	/**
+	 * @var array
+	 */
+	public $parents = [];
 
-    /**
-     * @var array
-     */
-    protected $properties = [];
+	/**
+	 * @var array
+	 */
+	protected $properties = [];
 
-    /**
-     * @var string
-     */
-    protected $type;
+	/**
+	 * @var string
+	 */
+	protected $type;
 
-    /**
-     * @param string $method
-     * @return static
-     */
-    public function __call($method, array $arguments)
-    {
-        return $this->setProperty($method, Arr::get($arguments, 0));
-    }
+	/**
+	 * @param string $method
+	 * @return static
+	 */
+	public function __call($method, array $arguments)
+	{
+		return $this->setProperty($method, Arr::get($arguments, 0));
+	}
 
-    /**
-     * @param string $type
-     */
-    public function __construct($type = null)
-    {
-        $this->type = !is_string($type)
-            ? (new ReflectionClass($this))->getShortName()
-            : $type;
-        $this->setAllowedProperties();
-    }
+	/**
+	 * @param string $type
+	 */
+	public function __construct($type = null)
+	{
+		$this->type = !is_string($type)
+			? (new ReflectionClass($this))->getShortName()
+			: $type;
+		$this->setAllowedProperties();
+	}
 
-    /**
-     * @return string
-     */
-    public function __toString()
-    {
-        return $this->toScript();
-    }
+	/**
+	 * @return string
+	 */
+	public function __toString()
+	{
+		return $this->toScript();
+	}
 
-    /**
-     * @return static
-     */
-    public function addProperties(array $properties)
-    {
-        foreach ($properties as $property => $value) {
-            $this->setProperty($property, $value);
-        }
-        return $this;
-    }
+	/**
+	 * @return static
+	 */
+	public function addProperties(array $properties)
+	{
+		foreach ($properties as $property => $value) {
+			$this->setProperty($property, $value);
+		}
+		return $this;
+	}
 
-    /**
-     * @return string
-     */
-    public function getContext()
-    {
-        return 'https://schema.org';
-    }
+	/**
+	 * @return string
+	 */
+	public function getContext()
+	{
+		return 'https://schema.org';
+	}
 
-    /**
-     * @return array
-     */
-    public function getProperties()
-    {
-        return $this->properties;
-    }
+	/**
+	 * @return array
+	 */
+	public function getProperties()
+	{
+		return $this->properties;
+	}
 
-    /**
-     * @param string $property
-     * @param mixed $default
-     * @return mixed
-     */
-    public function getProperty($property, $default = null)
-    {
-        return Arr::get($this->properties, $property, $default);
-    }
+	/**
+	 * @param string $property
+	 * @param mixed $default
+	 * @return mixed
+	 */
+	public function getProperty($property, $default = null)
+	{
+		return Arr::get($this->properties, $property, $default);
+	}
 
-    /**
-     * @return string
-     */
-    public function getType()
-    {
-        return $this->type;
-    }
+	/**
+	 * @return string
+	 */
+	public function getType()
+	{
+		return $this->type;
+	}
 
-    /**
-     * @param bool $condition
-     * @param mixed $callback
-     * @return static
-     */
-    public function doIf($condition, $callback)
-    {
-        if ($condition) {
-            $callback($this);
-        }
-        return $this;
-    }
+	/**
+	 * @param bool $condition
+	 * @param mixed $callback
+	 * @return static
+	 */
+	public function doIf($condition, $callback)
+	{
+		if ($condition) {
+			$callback($this);
+		}
+		return $this;
+	}
 
-    /**
-     * @return array
-     */
-    public function jsonSerialize()
-    {
-        return $this->toArray();
-    }
+	/**
+	 * @return array
+	 */
+	public function jsonSerialize()
+	{
+		return $this->toArray();
+	}
 
-    /**
-     * @param mixed $offset
-     * @return bool
-     */
-    public function offsetExists($offset)
-    {
-        return array_key_exists($offset, $this->properties);
-    }
+	/**
+	 * @param mixed $offset
+	 * @return bool
+	 */
+	public function offsetExists($offset)
+	{
+		return array_key_exists($offset, $this->properties);
+	}
 
-    /**
-     * @param string $offset
-     * @return mixed
-     */
-    public function offsetGet($offset)
-    {
-        return $this->getProperty($offset);
-    }
+	/**
+	 * @param string $offset
+	 * @return mixed
+	 */
+	public function offsetGet($offset)
+	{
+		return $this->getProperty($offset);
+	}
 
-    /**
-     * @param string $offset
-     * @param mixed $value
-     * @return void
-     */
-    public function offsetSet($offset, $value)
-    {
-        $this->setProperty($offset, $value);
-    }
+	/**
+	 * @param string $offset
+	 * @param mixed $value
+	 * @return void
+	 */
+	public function offsetSet($offset, $value)
+	{
+		$this->setProperty($offset, $value);
+	}
 
-    /**
-     * @param string $offset
-     * @return void
-     */
-    public function offsetUnset($offset)
-    {
-        unset($this->properties[$offset]);
-    }
+	/**
+	 * @param string $offset
+	 * @return void
+	 */
+	public function offsetUnset($offset)
+	{
+		unset($this->properties[$offset]);
+	}
 
-    /**
-     * @param string $property
-     * @param mixed $value
-     * @return static
-     */
-    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');
-            return $this;
-        }
-        $this->properties[$property] = $value;
-        return $this;
-    }
+	/**
+	 * @param string $property
+	 * @param mixed $value
+	 * @return static
+	 */
+	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');
+			return $this;
+		}
+		$this->properties[$property] = $value;
+		return $this;
+	}
 
-    /**
-     * @return array
-     */
-    public function toArray()
-    {
-        return [
-            '@context' => $this->getContext(),
-            '@type' => $this->getType(),
-        ] + $this->serializeProperty($this->getProperties());
-    }
+	/**
+	 * @return array
+	 */
+	public function toArray()
+	{
+		return [
+			'@context' => $this->getContext(),
+			'@type' => $this->getType(),
+		] + $this->serializeProperty($this->getProperties());
+	}
 
-    /**
-     * @return string
-     */
-    public function toScript()
-    {
-        return sprintf('<script type="application/ld+json">%s</script>',
-            json_encode($this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES)
-        );
-    }
+	/**
+	 * @return string
+	 */
+	public function toScript()
+	{
+		return sprintf('<script type="application/ld+json">%s</script>',
+			json_encode($this->toArray(), JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES)
+		);
+	}
 
-    /**
-     * @param array|null $parents
-     * @return array
-     */
-    protected function getParents($parents = null)
-    {
-        if (!isset($parents)) {
-            $parents = $this->parents;
-        }
-        $newParents = $parents;
-        foreach ($parents as $parent) {
-            $parentClass = Helper::buildClassName($parent, __NAMESPACE__);
-            if (!class_exists($parentClass)) {
-                continue;
-            }
-            $newParents = array_merge($newParents, $this->getParents((new $parentClass())->parents));
-        }
-        return array_values(array_unique($newParents));
-    }
+	/**
+	 * @param array|null $parents
+	 * @return array
+	 */
+	protected function getParents($parents = null)
+	{
+		if (!isset($parents)) {
+			$parents = $this->parents;
+		}
+		$newParents = $parents;
+		foreach ($parents as $parent) {
+			$parentClass = Helper::buildClassName($parent, __NAMESPACE__);
+			if (!class_exists($parentClass)) {
+				continue;
+			}
+			$newParents = array_merge($newParents, $this->getParents((new $parentClass())->parents));
+		}
+		return array_values(array_unique($newParents));
+	}
 
-    /**
-     * @return void
-     */
-    protected function setAllowedProperties()
-    {
-        $parents = $this->getParents();
-        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)));
-        }
-    }
+	/**
+	 * @return void
+	 */
+	protected function setAllowedProperties()
+	{
+		$parents = $this->getParents();
+		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)));
+		}
+	}
 
-    /**
-     * @param mixed $property
-     * @return array|string
-     */
-    protected function serializeProperty($property)
-    {
-        if (is_array($property)) {
-            return array_map([$this, 'serializeProperty'], $property);
-        }
-        if ($property instanceof Type) {
-            $property = $property->toArray();
-            unset($property['@context']);
-        }
-        if ($property instanceof DateTimeInterface) {
-            $property = $property->format(DateTime::ATOM);
-        }
-        if (is_object($property)) {
-            throw new InvalidProperty();
-        }
-        return $property;
-    }
+	/**
+	 * @param mixed $property
+	 * @return array|string
+	 */
+	protected function serializeProperty($property)
+	{
+		if (is_array($property)) {
+			return array_map([$this, 'serializeProperty'], $property);
+		}
+		if ($property instanceof Type) {
+			$property = $property->toArray();
+			unset($property['@context']);
+		}
+		if ($property instanceof DateTimeInterface) {
+			$property = $property->format(DateTime::ATOM);
+		}
+		if (is_object($property)) {
+			throw new InvalidProperty();
+		}
+		return $property;
+	}
 }

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/Fields/Field.php

Indentation +62 added lines, -62 removed lines

@@ -7,73 +7,73 @@
 
 abstract class Field
 {
-    /**
-     * @var Builder
-     */
-    protected $builder;
+	/**
+	 * @var Builder
+	 */
+	protected $builder;
 
-    public function __construct(Builder $builder)
-    {
-        $this->builder = $builder;
-    }
+	public function __construct(Builder $builder)
+	{
+		$this->builder = $builder;
+	}
 
-    /**
-     * @return string|void
-     */
-    public function build()
-    {
-        glsr_log()->error('Build method is not implemented for '.get_class($this));
-    }
+	/**
+	 * @return string|void
+	 */
+	public function build()
+	{
+		glsr_log()->error('Build method is not implemented for '.get_class($this));
+	}
 
-    /**
-     * @return array
-     */
-    public static function defaults()
-    {
-        return [];
-    }
+	/**
+	 * @return array
+	 */
+	public static function defaults()
+	{
+		return [];
+	}
 
-    /**
-     * @return array
-     */
-    public static function merge(array $args)
-    {
-        $merged = array_merge(
-            wp_parse_args($args, static::defaults()),
-            static::required()
-        );
-        $merged['class'] = implode(' ', static::mergedAttribute('class', ' ', $args));
-        $merged['style'] = implode(';', static::mergedAttribute('style', ';', $args));
-        return $merged;
-    }
+	/**
+	 * @return array
+	 */
+	public static function merge(array $args)
+	{
+		$merged = array_merge(
+			wp_parse_args($args, static::defaults()),
+			static::required()
+		);
+		$merged['class'] = implode(' ', static::mergedAttribute('class', ' ', $args));
+		$merged['style'] = implode(';', static::mergedAttribute('style', ';', $args));
+		return $merged;
+	}
 
-    /**
-     * @param string $delimiter
-     * @param string $key
-     * @return array
-     */
-    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))
-        ));
-    }
+	/**
+	 * @param string $delimiter
+	 * @param string $key
+	 * @return array
+	 */
+	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
-     */
-    public static function required()
-    {
-        return [];
-    }
+	/**
+	 * @return array
+	 */
+	public static function required()
+	{
+		return [];
+	}
 
-    /**
-     * @return void
-     */
-    protected function mergeFieldArgs()
-    {
-        $this->builder->args = static::merge($this->builder->args);
-    }
+	/**
+	 * @return void
+	 */
+	protected function mergeFieldArgs()
+	{
+		$this->builder->args = static::merge($this->builder->args);
+	}
 }

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 );
     }
 }

plugin/Modules/Email.php

Indentation +201 added lines, -201 removed lines

@@ -9,205 +9,205 @@
 
 class Email
 {
-    /**
-     * @var array
-     */
-    public $attachments;
-
-    /**
-     * @var array
-     */
-    public $email;
-
-    /**
-     * @var array
-     */
-    public $headers;
-
-    /**
-     * @var string
-     */
-    public $message;
-
-    /**
-     * @var string
-     */
-    public $subject;
-
-    /**
-     * @var string|array
-     */
-    public $to;
-
-    /**
-     * @return Email
-     */
-    public function compose(array $email)
-    {
-        $this->normalize($email);
-        $this->attachments = $this->email['attachments'];
-        $this->headers = $this->buildHeaders();
-        $this->message = $this->buildHtmlMessage();
-        $this->subject = $this->email['subject'];
-        $this->to = $this->email['to'];
-        add_action('phpmailer_init', [$this, 'buildPlainTextMessage']);
-        return $this;
-    }
-
-    /**
-     * @param string $format
-     * @return string|null
-     */
-    public function read($format = '')
-    {
-        if ('plaintext' == $format) {
-            $message = $this->stripHtmlTags($this->message);
-            return apply_filters('site-reviews/email/message', $message, 'text', $this);
-        }
-        return $this->message;
-    }
-
-    /**
-     * @return void|bool
-     */
-    public function send()
-    {
-        if (!$this->message || !$this->subject || !$this->to) {
-            return;
-        }
-        add_action('wp_mail_failed', [$this, 'logMailError']);
-        $sent = wp_mail(
-            $this->to,
-            $this->subject,
-            $this->message,
-            $this->headers,
-            $this->attachments
-        );
-        remove_action('wp_mail_failed', [$this, 'logMailError']);
-        $this->reset();
-        return $sent;
-    }
-
-    /**
-     * @return void
-     * @action phpmailer_init
-     */
-    public function buildPlainTextMessage(PHPMailer $phpmailer)
-    {
-        if (empty($this->email)) {
-            return;
-        }
-        if ('text/plain' === $phpmailer->ContentType || !empty($phpmailer->AltBody)) {
-            return;
-        }
-        $message = $this->stripHtmlTags($phpmailer->Body);
-        $phpmailer->AltBody = apply_filters('site-reviews/email/message', $message, 'text', $this);
-    }
-
-    /**
-     * @return array
-     */
-    protected function buildHeaders()
-    {
-        $allowed = [
-            'bcc', 'cc', 'from', 'reply-to',
-        ];
-        $headers = array_intersect_key($this->email, array_flip($allowed));
-        $headers = array_filter($headers);
-        foreach ($headers as $key => $value) {
-            unset($headers[$key]);
-            $headers[] = $key.': '.$value;
-        }
-        $headers[] = 'Content-Type: text/html';
-        return apply_filters('site-reviews/email/headers', $headers, $this);
-    }
-
-    /**
-     * @return string
-     */
-    protected function buildHtmlMessage()
-    {
-        $template = trim(glsr(OptionManager::class)->get('settings.general.notification_message'));
-        if (!empty($template)) {
-            $message = glsr(Template::class)->interpolate(
-                $template, 
-                ['context' => $this->email['template-tags']], 
-                $this->email['template']
-            );
-        } elseif ($this->email['template']) {
-            $message = glsr(Template::class)->build('templates/'.$this->email['template'], [
-                'context' => $this->email['template-tags'],
-            ]);
-        }
-        if (!isset($message)) {
-            $message = $this->email['message'];
-        }
-        $message = $this->email['before'].$message.$this->email['after'];
-        $message = strip_shortcodes($message);
-        $message = wptexturize($message);
-        $message = wpautop($message);
-        $message = str_replace('<> ', '', $message);
-        $message = str_replace(']]>', ']]>', $message);
-        $message = glsr(Template::class)->build('partials/email/index', [
-            'context' => ['message' => $message],
-        ]);
-        return apply_filters('site-reviews/email/message', stripslashes($message), 'html', $this);
-    }
-
-    /**
-     * @param \WP_Error $error
-     * @return void
-     */
-    protected function logMailError($error)
-    {
-        glsr_log()->error('Email was not sent (wp_mail failed)')
-            ->debug($this)
-            ->debug($error);
-    }
-
-    /**
-     * @return void
-     */
-    protected function normalize(array $email = [])
-    {
-        $email = shortcode_atts(glsr(EmailDefaults::class)->defaults(), $email);
-        if (empty($email['reply-to'])) {
-            $email['reply-to'] = $email['from'];
-        }
-        $this->email = apply_filters('site-reviews/email/compose', $email, $this);
-    }
-
-    /**
-     * @return void
-     */
-    protected function reset()
-    {
-        $this->attachments = [];
-        $this->email = [];
-        $this->headers = [];
-        $this->message = null;
-        $this->subject = null;
-        $this->to = null;
-    }
-
-    /**
-     * @return string
-     */
-    protected function stripHtmlTags($string)
-    {
-        // remove invisible elements
-        $string = preg_replace('@<(embed|head|noembed|noscript|object|script|style)[^>]*?>.*?</\\1>@siu', '', $string);
-        // replace certain elements with a line-break
-        $string = preg_replace('@</(div|h[1-9]|p|pre|tr)@iu', "\r\n\$0", $string);
-        // replace other elements with a space
-        $string = preg_replace('@</(td|th)@iu', ' $0', $string);
-        // add a placeholder for plain-text bullets to list elements
-        $string = preg_replace('@<(li)[^>]*?>@siu', '$0-o-^-o-', $string);
-        // strip all remaining HTML tags
-        $string = wp_strip_all_tags($string);
-        $string = wp_specialchars_decode($string, ENT_QUOTES);
-        $string = preg_replace('/\v(?:[\v\h]+){2,}/', "\r\n\r\n", $string);
-        $string = str_replace('-o-^-o-', ' - ', $string);
-        return html_entity_decode($string, ENT_QUOTES, 'UTF-8');
-    }
+	/**
+	 * @var array
+	 */
+	public $attachments;
+
+	/**
+	 * @var array
+	 */
+	public $email;
+
+	/**
+	 * @var array
+	 */
+	public $headers;
+
+	/**
+	 * @var string
+	 */
+	public $message;
+
+	/**
+	 * @var string
+	 */
+	public $subject;
+
+	/**
+	 * @var string|array
+	 */
+	public $to;
+
+	/**
+	 * @return Email
+	 */
+	public function compose(array $email)
+	{
+		$this->normalize($email);
+		$this->attachments = $this->email['attachments'];
+		$this->headers = $this->buildHeaders();
+		$this->message = $this->buildHtmlMessage();
+		$this->subject = $this->email['subject'];
+		$this->to = $this->email['to'];
+		add_action('phpmailer_init', [$this, 'buildPlainTextMessage']);
+		return $this;
+	}
+
+	/**
+	 * @param string $format
+	 * @return string|null
+	 */
+	public function read($format = '')
+	{
+		if ('plaintext' == $format) {
+			$message = $this->stripHtmlTags($this->message);
+			return apply_filters('site-reviews/email/message', $message, 'text', $this);
+		}
+		return $this->message;
+	}
+
+	/**
+	 * @return void|bool
+	 */
+	public function send()
+	{
+		if (!$this->message || !$this->subject || !$this->to) {
+			return;
+		}
+		add_action('wp_mail_failed', [$this, 'logMailError']);
+		$sent = wp_mail(
+			$this->to,
+			$this->subject,
+			$this->message,
+			$this->headers,
+			$this->attachments
+		);
+		remove_action('wp_mail_failed', [$this, 'logMailError']);
+		$this->reset();
+		return $sent;
+	}
+
+	/**
+	 * @return void
+	 * @action phpmailer_init
+	 */
+	public function buildPlainTextMessage(PHPMailer $phpmailer)
+	{
+		if (empty($this->email)) {
+			return;
+		}
+		if ('text/plain' === $phpmailer->ContentType || !empty($phpmailer->AltBody)) {
+			return;
+		}
+		$message = $this->stripHtmlTags($phpmailer->Body);
+		$phpmailer->AltBody = apply_filters('site-reviews/email/message', $message, 'text', $this);
+	}
+
+	/**
+	 * @return array
+	 */
+	protected function buildHeaders()
+	{
+		$allowed = [
+			'bcc', 'cc', 'from', 'reply-to',
+		];
+		$headers = array_intersect_key($this->email, array_flip($allowed));
+		$headers = array_filter($headers);
+		foreach ($headers as $key => $value) {
+			unset($headers[$key]);
+			$headers[] = $key.': '.$value;
+		}
+		$headers[] = 'Content-Type: text/html';
+		return apply_filters('site-reviews/email/headers', $headers, $this);
+	}
+
+	/**
+	 * @return string
+	 */
+	protected function buildHtmlMessage()
+	{
+		$template = trim(glsr(OptionManager::class)->get('settings.general.notification_message'));
+		if (!empty($template)) {
+			$message = glsr(Template::class)->interpolate(
+				$template, 
+				['context' => $this->email['template-tags']], 
+				$this->email['template']
+			);
+		} elseif ($this->email['template']) {
+			$message = glsr(Template::class)->build('templates/'.$this->email['template'], [
+				'context' => $this->email['template-tags'],
+			]);
+		}
+		if (!isset($message)) {
+			$message = $this->email['message'];
+		}
+		$message = $this->email['before'].$message.$this->email['after'];
+		$message = strip_shortcodes($message);
+		$message = wptexturize($message);
+		$message = wpautop($message);
+		$message = str_replace('&lt;&gt; ', '', $message);
+		$message = str_replace(']]>', ']]&gt;', $message);
+		$message = glsr(Template::class)->build('partials/email/index', [
+			'context' => ['message' => $message],
+		]);
+		return apply_filters('site-reviews/email/message', stripslashes($message), 'html', $this);
+	}
+
+	/**
+	 * @param \WP_Error $error
+	 * @return void
+	 */
+	protected function logMailError($error)
+	{
+		glsr_log()->error('Email was not sent (wp_mail failed)')
+			->debug($this)
+			->debug($error);
+	}
+
+	/**
+	 * @return void
+	 */
+	protected function normalize(array $email = [])
+	{
+		$email = shortcode_atts(glsr(EmailDefaults::class)->defaults(), $email);
+		if (empty($email['reply-to'])) {
+			$email['reply-to'] = $email['from'];
+		}
+		$this->email = apply_filters('site-reviews/email/compose', $email, $this);
+	}
+
+	/**
+	 * @return void
+	 */
+	protected function reset()
+	{
+		$this->attachments = [];
+		$this->email = [];
+		$this->headers = [];
+		$this->message = null;
+		$this->subject = null;
+		$this->to = null;
+	}
+
+	/**
+	 * @return string
+	 */
+	protected function stripHtmlTags($string)
+	{
+		// remove invisible elements
+		$string = preg_replace('@<(embed|head|noembed|noscript|object|script|style)[^>]*?>.*?</\\1>@siu', '', $string);
+		// replace certain elements with a line-break
+		$string = preg_replace('@</(div|h[1-9]|p|pre|tr)@iu', "\r\n\$0", $string);
+		// replace other elements with a space
+		$string = preg_replace('@</(td|th)@iu', ' $0', $string);
+		// add a placeholder for plain-text bullets to list elements
+		$string = preg_replace('@<(li)[^>]*?>@siu', '$0-o-^-o-', $string);
+		// strip all remaining HTML tags
+		$string = wp_strip_all_tags($string);
+		$string = wp_specialchars_decode($string, ENT_QUOTES);
+		$string = preg_replace('/\v(?:[\v\h]+){2,}/', "\r\n\r\n", $string);
+		$string = str_replace('-o-^-o-', ' - ', $string);
+		return html_entity_decode($string, ENT_QUOTES, 'UTF-8');
+	}
 }

Spacing +52 added lines, -52 removed lines

@@ -42,15 +42,15 @@ 
     /**
      * @return Email
      */
-    public function compose(array $email)
+    public function compose( array $email )
     {
-        $this->normalize($email);
+        $this->normalize( $email );
         $this->attachments = $this->email['attachments'];
         $this->headers = $this->buildHeaders();
         $this->message = $this->buildHtmlMessage();
         $this->subject = $this->email['subject'];
         $this->to = $this->email['to'];
-        add_action('phpmailer_init', [$this, 'buildPlainTextMessage']);
+        add_action( 'phpmailer_init', [$this, 'buildPlainTextMessage'] );
         return $this;
     }
 
@@ -58,11 +58,11 @@ 
      * @param string $format
      * @return string|null
      */
-    public function read($format = '')
+    public function read( $format = '' )
     {
-        if ('plaintext' == $format) {
-            $message = $this->stripHtmlTags($this->message);
-            return apply_filters('site-reviews/email/message', $message, 'text', $this);
+        if( 'plaintext' == $format ) {
+            $message = $this->stripHtmlTags( $this->message );
+            return apply_filters( 'site-reviews/email/message', $message, 'text', $this );
         }
         return $this->message;
     }
@@ -72,10 +72,10 @@ 
      */
     public function send()
     {
-        if (!$this->message || !$this->subject || !$this->to) {
+        if( !$this->message || !$this->subject || !$this->to ) {
             return;
         }
-        add_action('wp_mail_failed', [$this, 'logMailError']);
+        add_action( 'wp_mail_failed', [$this, 'logMailError'] );
         $sent = wp_mail(
             $this->to,
             $this->subject,
@@ -83,7 +83,7 @@ 
             $this->headers,
             $this->attachments
         );
-        remove_action('wp_mail_failed', [$this, 'logMailError']);
+        remove_action( 'wp_mail_failed', [$this, 'logMailError'] );
         $this->reset();
         return $sent;
     }
@@ -92,16 +92,16 @@ 
      * @return void
      * @action phpmailer_init
      */
-    public function buildPlainTextMessage(PHPMailer $phpmailer)
+    public function buildPlainTextMessage( PHPMailer $phpmailer )
     {
-        if (empty($this->email)) {
+        if( empty($this->email) ) {
             return;
         }
-        if ('text/plain' === $phpmailer->ContentType || !empty($phpmailer->AltBody)) {
+        if( 'text/plain' === $phpmailer->ContentType || !empty($phpmailer->AltBody) ) {
             return;
         }
-        $message = $this->stripHtmlTags($phpmailer->Body);
-        $phpmailer->AltBody = apply_filters('site-reviews/email/message', $message, 'text', $this);
+        $message = $this->stripHtmlTags( $phpmailer->Body );
+        $phpmailer->AltBody = apply_filters( 'site-reviews/email/message', $message, 'text', $this );
     }
 
     /**
@@ -112,14 +112,14 @@ 
         $allowed = [
             'bcc', 'cc', 'from', 'reply-to',
         ];
-        $headers = array_intersect_key($this->email, array_flip($allowed));
-        $headers = array_filter($headers);
-        foreach ($headers as $key => $value) {
+        $headers = array_intersect_key( $this->email, array_flip( $allowed ) );
+        $headers = array_filter( $headers );
+        foreach( $headers as $key => $value ) {
             unset($headers[$key]);
             $headers[] = $key.': '.$value;
         }
         $headers[] = 'Content-Type: text/html';
-        return apply_filters('site-reviews/email/headers', $headers, $this);
+        return apply_filters( 'site-reviews/email/headers', $headers, $this );
     }
 
     /**
@@ -127,54 +127,54 @@ 
      */
     protected function buildHtmlMessage()
     {
-        $template = trim(glsr(OptionManager::class)->get('settings.general.notification_message'));
-        if (!empty($template)) {
-            $message = glsr(Template::class)->interpolate(
+        $template = trim( glsr( OptionManager::class )->get( 'settings.general.notification_message' ) );
+        if( !empty($template) ) {
+            $message = glsr( Template::class )->interpolate(
                 $template, 
                 ['context' => $this->email['template-tags']], 
                 $this->email['template']
             );
-        } elseif ($this->email['template']) {
-            $message = glsr(Template::class)->build('templates/'.$this->email['template'], [
+        } elseif( $this->email['template'] ) {
+            $message = glsr( Template::class )->build( 'templates/'.$this->email['template'], [
                 'context' => $this->email['template-tags'],
-            ]);
+            ] );
         }
-        if (!isset($message)) {
+        if( !isset($message) ) {
             $message = $this->email['message'];
         }
         $message = $this->email['before'].$message.$this->email['after'];
-        $message = strip_shortcodes($message);
-        $message = wptexturize($message);
-        $message = wpautop($message);
-        $message = str_replace('<> ', '', $message);
-        $message = str_replace(']]>', ']]>', $message);
-        $message = glsr(Template::class)->build('partials/email/index', [
+        $message = strip_shortcodes( $message );
+        $message = wptexturize( $message );
+        $message = wpautop( $message );
+        $message = str_replace( '<> ', '', $message );
+        $message = str_replace( ']]>', ']]>', $message );
+        $message = glsr( Template::class )->build( 'partials/email/index', [
             'context' => ['message' => $message],
-        ]);
-        return apply_filters('site-reviews/email/message', stripslashes($message), 'html', $this);
+        ] );
+        return apply_filters( 'site-reviews/email/message', stripslashes( $message ), 'html', $this );
     }
 
     /**
      * @param \WP_Error $error
      * @return void
      */
-    protected function logMailError($error)
+    protected function logMailError( $error )
     {
-        glsr_log()->error('Email was not sent (wp_mail failed)')
-            ->debug($this)
-            ->debug($error);
+        glsr_log()->error( 'Email was not sent (wp_mail failed)' )
+            ->debug( $this )
+            ->debug( $error );
     }
 
     /**
      * @return void
      */
-    protected function normalize(array $email = [])
+    protected function normalize( array $email = [] )
     {
-        $email = shortcode_atts(glsr(EmailDefaults::class)->defaults(), $email);
-        if (empty($email['reply-to'])) {
+        $email = shortcode_atts( glsr( EmailDefaults::class )->defaults(), $email );
+        if( empty($email['reply-to']) ) {
             $email['reply-to'] = $email['from'];
         }
-        $this->email = apply_filters('site-reviews/email/compose', $email, $this);
+        $this->email = apply_filters( 'site-reviews/email/compose', $email, $this );
     }
 
     /**
@@ -193,21 +193,21 @@ 
     /**
      * @return string
      */
-    protected function stripHtmlTags($string)
+    protected function stripHtmlTags( $string )
     {
         // remove invisible elements
-        $string = preg_replace('@<(embed|head|noembed|noscript|object|script|style)[^>]*?>.*?</\\1>@siu', '', $string);
+        $string = preg_replace( '@<(embed|head|noembed|noscript|object|script|style)[^>]*?>.*?</\\1>@siu', '', $string );
         // replace certain elements with a line-break
-        $string = preg_replace('@</(div|h[1-9]|p|pre|tr)@iu', "\r\n\$0", $string);
+        $string = preg_replace( '@</(div|h[1-9]|p|pre|tr)@iu', "\r\n\$0", $string );
         // replace other elements with a space
-        $string = preg_replace('@</(td|th)@iu', ' $0', $string);
+        $string = preg_replace( '@</(td|th)@iu', ' $0', $string );
         // add a placeholder for plain-text bullets to list elements
-        $string = preg_replace('@<(li)[^>]*?>@siu', '$0-o-^-o-', $string);
+        $string = preg_replace( '@<(li)[^>]*?>@siu', '$0-o-^-o-', $string );
         // strip all remaining HTML tags
-        $string = wp_strip_all_tags($string);
-        $string = wp_specialchars_decode($string, ENT_QUOTES);
-        $string = preg_replace('/\v(?:[\v\h]+){2,}/', "\r\n\r\n", $string);
-        $string = str_replace('-o-^-o-', ' - ', $string);
-        return html_entity_decode($string, ENT_QUOTES, 'UTF-8');
+        $string = wp_strip_all_tags( $string );
+        $string = wp_specialchars_decode( $string, ENT_QUOTES );
+        $string = preg_replace( '/\v(?:[\v\h]+){2,}/', "\r\n\r\n", $string );
+        $string = str_replace( '-o-^-o-', ' - ', $string );
+        return html_entity_decode( $string, ENT_QUOTES, 'UTF-8' );
     }
 }

Braces +2 added lines, -1 removed lines

@@ -134,7 +134,8 @@
                 ['context' => $this->email['template-tags']], 
                 $this->email['template']
             );
-        } elseif ($this->email['template']) {
+        }
+        elseif ($this->email['template']) {
             $message = glsr(Template::class)->build('templates/'.$this->email['template'], [
                 'context' => $this->email['template-tags'],
             ]);

plugin/Shortcodes/SiteReviewsPopup.php

Indentation +80 added lines, -80 removed lines

@@ -4,84 +4,84 @@
 
 class SiteReviewsPopup extends TinymcePopupGenerator
 {
-    /**
-     * @return array
-     */
-    public function fields()
-    {
-        return [[
-            'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
-            'minWidth' => 320,
-            'type' => 'container',
-        ], [
-            'label' => esc_html__('Title', 'site-reviews'),
-            'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Display', 'site-reviews'),
-            'maxLength' => 5,
-            'name' => 'display',
-            'size' => 3,
-            'text' => '10',
-            'tooltip' => __('How many reviews would you like to display (default: 10)?', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Rating', 'site-reviews'),
-            'name' => 'rating',
-            'options' => [
-                '5' => esc_html(sprintf(_n('%s star', '%s stars', 5, 'site-reviews'), 5)),
-                '4' => esc_html(sprintf(_n('%s star', '%s stars', 4, 'site-reviews'), 4)),
-                '3' => esc_html(sprintf(_n('%s star', '%s stars', 3, 'site-reviews'), 3)),
-                '2' => esc_html(sprintf(_n('%s star', '%s stars', 2, 'site-reviews'), 2)),
-                '1' => esc_html(sprintf(_n('%s star', '%s stars', 1, 'site-reviews'), 1)),
-                '0' => esc_html(__('Unrated', 'site-reviews')),
-            ],
-            'tooltip' => __('What is the minimum rating to display (default: 1 star)?', 'site-reviews'),
-            'type' => 'listbox',
-        ], [
-            'label' => esc_html__('Pagination', 'site-reviews'),
-            'name' => 'pagination',
-            'options' => [
-                'true' => esc_html__('Enable', 'site-reviews'),
-                'ajax' => esc_html__('Enable (using ajax)', 'site-reviews'),
-                'false' => esc_html__('Disable', 'site-reviews'),
-            ],
-            'tooltip' => __('When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews'),
-            'type' => 'listbox',
-        ],
-        $this->getTypes(__('Which type of review would you like to display?', 'site-reviews')),
-        $this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
-        [
-            'label' => esc_html__('Assigned To', 'site-reviews'),
-            'name' => 'assigned_to',
-            'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Schema', 'site-reviews'),
-            'name' => 'schema',
-            'options' => [
-                'true' => esc_html__('Enable rich snippets', 'site-reviews'),
-                'false' => esc_html__('Disable rich snippets', 'site-reviews'),
-            ],
-            'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
-            'type' => 'listbox',
-        ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
-            'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'columns' => 2,
-            'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
-            'layout' => 'grid',
-            'spacing' => 5,
-            'type' => 'container',
-        ], [
-            'hidden' => true,
-            'name' => 'id',
-            'type' => 'textbox',
-        ], ];
-    }
+	/**
+	 * @return array
+	 */
+	public function fields()
+	{
+		return [[
+			'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
+			'minWidth' => 320,
+			'type' => 'container',
+		], [
+			'label' => esc_html__('Title', 'site-reviews'),
+			'name' => 'title',
+			'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Display', 'site-reviews'),
+			'maxLength' => 5,
+			'name' => 'display',
+			'size' => 3,
+			'text' => '10',
+			'tooltip' => __('How many reviews would you like to display (default: 10)?', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Rating', 'site-reviews'),
+			'name' => 'rating',
+			'options' => [
+				'5' => esc_html(sprintf(_n('%s star', '%s stars', 5, 'site-reviews'), 5)),
+				'4' => esc_html(sprintf(_n('%s star', '%s stars', 4, 'site-reviews'), 4)),
+				'3' => esc_html(sprintf(_n('%s star', '%s stars', 3, 'site-reviews'), 3)),
+				'2' => esc_html(sprintf(_n('%s star', '%s stars', 2, 'site-reviews'), 2)),
+				'1' => esc_html(sprintf(_n('%s star', '%s stars', 1, 'site-reviews'), 1)),
+				'0' => esc_html(__('Unrated', 'site-reviews')),
+			],
+			'tooltip' => __('What is the minimum rating to display (default: 1 star)?', 'site-reviews'),
+			'type' => 'listbox',
+		], [
+			'label' => esc_html__('Pagination', 'site-reviews'),
+			'name' => 'pagination',
+			'options' => [
+				'true' => esc_html__('Enable', 'site-reviews'),
+				'ajax' => esc_html__('Enable (using ajax)', 'site-reviews'),
+				'false' => esc_html__('Disable', 'site-reviews'),
+			],
+			'tooltip' => __('When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews'),
+			'type' => 'listbox',
+		],
+		$this->getTypes(__('Which type of review would you like to display?', 'site-reviews')),
+		$this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
+		[
+			'label' => esc_html__('Assigned To', 'site-reviews'),
+			'name' => 'assigned_to',
+			'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Schema', 'site-reviews'),
+			'name' => 'schema',
+			'options' => [
+				'true' => esc_html__('Enable rich snippets', 'site-reviews'),
+				'false' => esc_html__('Disable rich snippets', 'site-reviews'),
+			],
+			'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
+			'type' => 'listbox',
+		], [
+			'label' => esc_html__('Classes', 'site-reviews'),
+			'name' => 'class',
+			'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'columns' => 2,
+			'items' => $this->getHideOptions(),
+			'label' => esc_html__('Hide', 'site-reviews'),
+			'layout' => 'grid',
+			'spacing' => 5,
+			'type' => 'container',
+		], [
+			'hidden' => true,
+			'name' => 'id',
+			'type' => 'textbox',
+		], ];
+	}
 }

Spacing +29 added lines, -29 removed lines

@@ -10,71 +10,71 @@
     public function fields()
     {
         return [[
-            'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
+            'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
             'minWidth' => 320,
             'type' => 'container',
         ], [
-            'label' => esc_html__('Title', 'site-reviews'),
+            'label' => esc_html__( 'Title', 'site-reviews' ),
             'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+            'tooltip' => __( 'Enter a custom shortcode heading.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Display', 'site-reviews'),
+            'label' => esc_html__( 'Display', 'site-reviews' ),
             'maxLength' => 5,
             'name' => 'display',
             'size' => 3,
             'text' => '10',
-            'tooltip' => __('How many reviews would you like to display (default: 10)?', 'site-reviews'),
+            'tooltip' => __( 'How many reviews would you like to display (default: 10)?', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Rating', 'site-reviews'),
+            'label' => esc_html__( 'Rating', 'site-reviews' ),
             'name' => 'rating',
             'options' => [
-                '5' => esc_html(sprintf(_n('%s star', '%s stars', 5, 'site-reviews'), 5)),
-                '4' => esc_html(sprintf(_n('%s star', '%s stars', 4, 'site-reviews'), 4)),
-                '3' => esc_html(sprintf(_n('%s star', '%s stars', 3, 'site-reviews'), 3)),
-                '2' => esc_html(sprintf(_n('%s star', '%s stars', 2, 'site-reviews'), 2)),
-                '1' => esc_html(sprintf(_n('%s star', '%s stars', 1, 'site-reviews'), 1)),
-                '0' => esc_html(__('Unrated', 'site-reviews')),
+                '5' => esc_html( sprintf( _n( '%s star', '%s stars', 5, 'site-reviews' ), 5 ) ),
+                '4' => esc_html( sprintf( _n( '%s star', '%s stars', 4, 'site-reviews' ), 4 ) ),
+                '3' => esc_html( sprintf( _n( '%s star', '%s stars', 3, 'site-reviews' ), 3 ) ),
+                '2' => esc_html( sprintf( _n( '%s star', '%s stars', 2, 'site-reviews' ), 2 ) ),
+                '1' => esc_html( sprintf( _n( '%s star', '%s stars', 1, 'site-reviews' ), 1 ) ),
+                '0' => esc_html( __( 'Unrated', 'site-reviews' ) ),
             ],
-            'tooltip' => __('What is the minimum rating to display (default: 1 star)?', 'site-reviews'),
+            'tooltip' => __( 'What is the minimum rating to display (default: 1 star)?', 'site-reviews' ),
             'type' => 'listbox',
         ], [
-            'label' => esc_html__('Pagination', 'site-reviews'),
+            'label' => esc_html__( 'Pagination', 'site-reviews' ),
             'name' => 'pagination',
             'options' => [
-                'true' => esc_html__('Enable', 'site-reviews'),
-                'ajax' => esc_html__('Enable (using ajax)', 'site-reviews'),
-                'false' => esc_html__('Disable', 'site-reviews'),
+                'true' => esc_html__( 'Enable', 'site-reviews' ),
+                'ajax' => esc_html__( 'Enable (using ajax)', 'site-reviews' ),
+                'false' => esc_html__( 'Disable', 'site-reviews' ),
             ],
-            'tooltip' => __('When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews'),
+            'tooltip' => __( 'When using pagination this shortcode can only be used once on a page. (default: disable)', 'site-reviews' ),
             'type' => 'listbox',
         ],
-        $this->getTypes(__('Which type of review would you like to display?', 'site-reviews')),
-        $this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
+        $this->getTypes( __( 'Which type of review would you like to display?', 'site-reviews' ) ),
+        $this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' ) ),
         [
-            'label' => esc_html__('Assigned To', 'site-reviews'),
+            'label' => esc_html__( 'Assigned To', 'site-reviews' ),
             'name' => 'assigned_to',
-            'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
+            'tooltip' => __( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Schema', 'site-reviews'),
+            'label' => esc_html__( 'Schema', 'site-reviews' ),
             'name' => 'schema',
             'options' => [
-                'true' => esc_html__('Enable rich snippets', 'site-reviews'),
-                'false' => esc_html__('Disable rich snippets', 'site-reviews'),
+                'true' => esc_html__( 'Enable rich snippets', 'site-reviews' ),
+                'false' => esc_html__( 'Disable rich snippets', 'site-reviews' ),
             ],
-            'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
+            'tooltip' => __( 'Rich snippets are disabled by default.', 'site-reviews' ),
             'type' => 'listbox',
         ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
+            'label' => esc_html__( 'Classes', 'site-reviews' ),
             'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+            'tooltip' => __( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
             'columns' => 2,
             'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
+            'label' => esc_html__( 'Hide', 'site-reviews' ),
             'layout' => 'grid',
             'spacing' => 5,
             'type' => 'container',

plugin/Controllers/WelcomeController.php

Indentation +88 added lines, -88 removed lines

@@ -8,97 +8,97 @@
 
 class WelcomeController extends Controller
 {
-    /**
-     * @return array
-     * @filter plugin_action_links_site-reviews/site-reviews.php
-     */
-    public function filterActionLinks(array $links)
-    {
-        $links['welcome'] = glsr(Builder::class)->a(__('About', 'site-reviews'), [
-            'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'),
-        ]);
-        return $links;
-    }
+	/**
+	 * @return array
+	 * @filter plugin_action_links_site-reviews/site-reviews.php
+	 */
+	public function filterActionLinks(array $links)
+	{
+		$links['welcome'] = glsr(Builder::class)->a(__('About', 'site-reviews'), [
+			'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'),
+		]);
+		return $links;
+	}
 
-    /**
-     * @return string
-     * @filter admin_title
-     */
-    public function filterAdminTitle($title)
-    {
-        return Application::POST_TYPE.'_page_welcome' == glsr_current_screen()->id
-            ? sprintf(__('Welcome to %s — WordPress', 'site-reviews'), glsr()->name)
-            : $title;
-    }
+	/**
+	 * @return string
+	 * @filter admin_title
+	 */
+	public function filterAdminTitle($title)
+	{
+		return Application::POST_TYPE.'_page_welcome' == glsr_current_screen()->id
+			? sprintf(__('Welcome to %s — WordPress', 'site-reviews'), glsr()->name)
+			: $title;
+	}
 
-    /**
-     * @param string $text
-     * @return string
-     * @filter admin_footer_text
-     */
-    public function filterFooterText($text)
-    {
-        if (Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id) {
-            return $text;
-        }
-        $url = 'https://wordpress.org/support/view/plugin-reviews/site-reviews?filter=5#new-post';
-        return wp_kses_post(sprintf(
-            __('Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews'),
-            '<strong>'.glsr()->name.'</strong> <a href="'.$url.'" target="_blank">&#9733;&#9733;&#9733;&#9733;&#9733;</a>',
-            '<a href="'.$url.'" target="_blank">wordpress.org</a>'
-        ));
-    }
+	/**
+	 * @param string $text
+	 * @return string
+	 * @filter admin_footer_text
+	 */
+	public function filterFooterText($text)
+	{
+		if (Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id) {
+			return $text;
+		}
+		$url = 'https://wordpress.org/support/view/plugin-reviews/site-reviews?filter=5#new-post';
+		return wp_kses_post(sprintf(
+			__('Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews'),
+			'<strong>'.glsr()->name.'</strong> <a href="'.$url.'" target="_blank">&#9733;&#9733;&#9733;&#9733;&#9733;</a>',
+			'<a href="'.$url.'" target="_blank">wordpress.org</a>'
+		));
+	}
 
-    /**
-     * @param string $plugin
-     * @param bool $isNetworkActivation
-     * @return void
-     * @action activated_plugin
-     */
-    public function redirectOnActivation($plugin, $isNetworkActivation)
-    {
-        if (!$isNetworkActivation
-            && 'cli' !== php_sapi_name()
-            && $plugin === plugin_basename(glsr()->file)) {
-            wp_safe_redirect(admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'));
-            exit;
-        }
-    }
+	/**
+	 * @param string $plugin
+	 * @param bool $isNetworkActivation
+	 * @return void
+	 * @action activated_plugin
+	 */
+	public function redirectOnActivation($plugin, $isNetworkActivation)
+	{
+		if (!$isNetworkActivation
+			&& 'cli' !== php_sapi_name()
+			&& $plugin === plugin_basename(glsr()->file)) {
+			wp_safe_redirect(admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'));
+			exit;
+		}
+	}
 
-    /**
-     * @return void
-     * @action admin_menu
-     */
-    public function registerPage()
-    {
-        add_submenu_page('edit.php?post_type='.Application::POST_TYPE,
-            sprintf(__('Welcome to %s', 'site-reviews'), glsr()->name),
-            glsr()->name,
-            glsr()->getPermission('welcome'),
-            'welcome',
-            [$this, 'renderPage']
-        );
-        remove_submenu_page('edit.php?post_type='.Application::POST_TYPE, 'welcome');
-    }
+	/**
+	 * @return void
+	 * @action admin_menu
+	 */
+	public function registerPage()
+	{
+		add_submenu_page('edit.php?post_type='.Application::POST_TYPE,
+			sprintf(__('Welcome to %s', 'site-reviews'), glsr()->name),
+			glsr()->name,
+			glsr()->getPermission('welcome'),
+			'welcome',
+			[$this, 'renderPage']
+		);
+		remove_submenu_page('edit.php?post_type='.Application::POST_TYPE, 'welcome');
+	}
 
-    /**
-     * @return void
-     * @see $this->registerPage()
-     * @callback add_submenu_page
-     */
-    public function renderPage()
-    {
-        $tabs = apply_filters('site-reviews/addon/welcome/tabs', [
-            'getting-started' => __('Getting Started', 'site-reviews'),
-            'whatsnew' => __('What\'s New', 'site-reviews'),
-            'upgrade-guide' => __('Upgrade Guide', 'site-reviews'),
-            'support' => __('Support', 'site-reviews'),
-        ]);
-        glsr()->render('pages/welcome/index', [
-            'data' => ['context' => []],
-            'http_referer' => (string) wp_get_referer(),
-            'tabs' => $tabs,
-            'template' => glsr(Template::class),
-        ]);
-    }
+	/**
+	 * @return void
+	 * @see $this->registerPage()
+	 * @callback add_submenu_page
+	 */
+	public function renderPage()
+	{
+		$tabs = apply_filters('site-reviews/addon/welcome/tabs', [
+			'getting-started' => __('Getting Started', 'site-reviews'),
+			'whatsnew' => __('What\'s New', 'site-reviews'),
+			'upgrade-guide' => __('Upgrade Guide', 'site-reviews'),
+			'support' => __('Support', 'site-reviews'),
+		]);
+		glsr()->render('pages/welcome/index', [
+			'data' => ['context' => []],
+			'http_referer' => (string) wp_get_referer(),
+			'tabs' => $tabs,
+			'template' => glsr(Template::class),
+		]);
+	}
 }

Spacing +29 added lines, -29 removed lines

@@ -12,11 +12,11 @@ 
      * @return array
      * @filter plugin_action_links_site-reviews/site-reviews.php
      */
-    public function filterActionLinks(array $links)
+    public function filterActionLinks( array $links )
     {
-        $links['welcome'] = glsr(Builder::class)->a(__('About', 'site-reviews'), [
-            'href' => admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'),
-        ]);
+        $links['welcome'] = glsr( Builder::class )->a( __( 'About', 'site-reviews' ), [
+            'href' => admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=welcome' ),
+        ] );
         return $links;
     }
 
@@ -24,10 +24,10 @@ 
      * @return string
      * @filter admin_title
      */
-    public function filterAdminTitle($title)
+    public function filterAdminTitle( $title )
     {
         return Application::POST_TYPE.'_page_welcome' == glsr_current_screen()->id
-            ? sprintf(__('Welcome to %s — WordPress', 'site-reviews'), glsr()->name)
+            ? sprintf( __( 'Welcome to %s — WordPress', 'site-reviews' ), glsr()->name )
             : $title;
     }
 
@@ -36,17 +36,17 @@ 
      * @return string
      * @filter admin_footer_text
      */
-    public function filterFooterText($text)
+    public function filterFooterText( $text )
     {
-        if (Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id) {
+        if( Application::POST_TYPE.'_page_welcome' != glsr_current_screen()->id ) {
             return $text;
         }
         $url = 'https://wordpress.org/support/view/plugin-reviews/site-reviews?filter=5#new-post';
-        return wp_kses_post(sprintf(
-            __('Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews'),
+        return wp_kses_post( sprintf(
+            __( 'Please rate %s on %s and help us spread the word. Thank you so much!', 'site-reviews' ),
             '<strong>'.glsr()->name.'</strong> <a href="'.$url.'" target="_blank">&#9733;&#9733;&#9733;&#9733;&#9733;</a>',
             '<a href="'.$url.'" target="_blank">wordpress.org</a>'
-        ));
+        ) );
     }
 
     /**
@@ -55,12 +55,12 @@ 
      * @return void
      * @action activated_plugin
      */
-    public function redirectOnActivation($plugin, $isNetworkActivation)
+    public function redirectOnActivation( $plugin, $isNetworkActivation )
     {
-        if (!$isNetworkActivation
+        if( !$isNetworkActivation
             && 'cli' !== php_sapi_name()
-            && $plugin === plugin_basename(glsr()->file)) {
-            wp_safe_redirect(admin_url('edit.php?post_type='.Application::POST_TYPE.'&page=welcome'));
+            && $plugin === plugin_basename( glsr()->file ) ) {
+            wp_safe_redirect( admin_url( 'edit.php?post_type='.Application::POST_TYPE.'&page=welcome' ) );
             exit;
         }
     }
@@ -71,14 +71,14 @@ 
      */
     public function registerPage()
     {
-        add_submenu_page('edit.php?post_type='.Application::POST_TYPE,
-            sprintf(__('Welcome to %s', 'site-reviews'), glsr()->name),
+        add_submenu_page( 'edit.php?post_type='.Application::POST_TYPE,
+            sprintf( __( 'Welcome to %s', 'site-reviews' ), glsr()->name ),
             glsr()->name,
-            glsr()->getPermission('welcome'),
+            glsr()->getPermission( 'welcome' ),
             'welcome',
             [$this, 'renderPage']
         );
-        remove_submenu_page('edit.php?post_type='.Application::POST_TYPE, 'welcome');
+        remove_submenu_page( 'edit.php?post_type='.Application::POST_TYPE, 'welcome' );
     }
 
     /**
@@ -88,17 +88,17 @@ 
      */
     public function renderPage()
     {
-        $tabs = apply_filters('site-reviews/addon/welcome/tabs', [
-            'getting-started' => __('Getting Started', 'site-reviews'),
-            'whatsnew' => __('What\'s New', 'site-reviews'),
-            'upgrade-guide' => __('Upgrade Guide', 'site-reviews'),
-            'support' => __('Support', 'site-reviews'),
-        ]);
-        glsr()->render('pages/welcome/index', [
+        $tabs = apply_filters( 'site-reviews/addon/welcome/tabs', [
+            'getting-started' => __( 'Getting Started', 'site-reviews' ),
+            'whatsnew' => __( 'What\'s New', 'site-reviews' ),
+            'upgrade-guide' => __( 'Upgrade Guide', 'site-reviews' ),
+            'support' => __( 'Support', 'site-reviews' ),
+        ] );
+        glsr()->render( 'pages/welcome/index', [
             'data' => ['context' => []],
-            'http_referer' => (string) wp_get_referer(),
+            'http_referer' => (string)wp_get_referer(),
             'tabs' => $tabs,
-            'template' => glsr(Template::class),
-        ]);
+            'template' => glsr( Template::class ),
+        ] );
     }
 }

plugin/Defaults/SiteReviewsDefaults.php

Indentation +35 added lines, -35 removed lines

@@ -6,41 +6,41 @@
 
 class SiteReviewsDefaults extends Defaults
 {
-    /**
-     * @var array
-     */
-    protected $guarded = [
-        'fallback',
-        'title',
-    ];
+	/**
+	 * @var array
+	 */
+	protected $guarded = [
+		'fallback',
+		'title',
+	];
 
-    /**
-     * @var array
-     */
-    protected $mapped = [
-        'count' => 'display', // @deprecated since v4.1.0
-        'per_page' => 'display',
-    ];
+	/**
+	 * @var array
+	 */
+	protected $mapped = [
+		'count' => 'display', // @deprecated since v4.1.0
+		'per_page' => 'display',
+	];
 
-    /**
-     * @return array
-     */
-    protected function defaults()
-    {
-        return [
-            'assigned_to' => '',
-            'category' => '',
-            'class' => '',
-            'display' => 5,
-            'fallback' => '',
-            'hide' => [],
-            'id' => '',
-            'offset' => '',
-            'pagination' => false,
-            'rating' => 0,
-            'schema' => false,
-            'title' => '',
-            'type' => 'local',
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function defaults()
+	{
+		return [
+			'assigned_to' => '',
+			'category' => '',
+			'class' => '',
+			'display' => 5,
+			'fallback' => '',
+			'hide' => [],
+			'id' => '',
+			'offset' => '',
+			'pagination' => false,
+			'rating' => 0,
+			'schema' => false,
+			'title' => '',
+			'type' => 'local',
+		];
+	}
 }

plugin/Defaults/ReviewsDefaults.php

Indentation +26 added lines, -26 removed lines

@@ -6,31 +6,31 @@
 
 class ReviewsDefaults extends Defaults
 {
-    /**
-     * @var array
-     */
-    protected $mapped = [
-        'count' => 'per_page', // @deprecated since v4.1.0
-        'display' => 'per_page',
-    ];
+	/**
+	 * @var array
+	 */
+	protected $mapped = [
+		'count' => 'per_page', // @deprecated since v4.1.0
+		'display' => 'per_page',
+	];
 
-    /**
-     * @return array
-     */
-    protected function defaults()
-    {
-        return [
-            'assigned_to' => '',
-            'category' => '',
-            'offset' => '',
-            'order' => 'DESC',
-            'orderby' => 'date',
-            'pagination' => false,
-            'per_page' => 10,
-            'post__in' => [],
-            'post__not_in' => [],
-            'rating' => '',
-            'type' => '',
-        ];
-    }
+	/**
+	 * @return array
+	 */
+	protected function defaults()
+	{
+		return [
+			'assigned_to' => '',
+			'category' => '',
+			'offset' => '',
+			'order' => 'DESC',
+			'orderby' => 'date',
+			'pagination' => false,
+			'per_page' => 10,
+			'post__in' => [],
+			'post__not_in' => [],
+			'rating' => '',
+			'type' => '',
+		];
+	}
 }