pryley / site-reviews

views/pages/welcome/upgrade-guide.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <p class="about-description">If you are using any Site Reviews code snippets or have built any custom integrations with Site Reviews, please make sure to read this upgrade guide.</p>
 <div class="is-fullwidth">

views/pages/settings/index.php

Spacing +6 added lines, -6 removed lines

@@ -1,21 +1,21 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <div class="wrap">
-    <h1 class="wp-heading-inline"><?= esc_html(get_admin_page_title()); ?></h1>
+    <h1 class="wp-heading-inline"><?= esc_html( get_admin_page_title() ); ?></h1>
     <?= $notices; ?>
     <h2 class="glsr-nav-tab-wrapper nav-tab-wrapper">
-        <?php foreach ($tabs as $id => $title) : ?>
+        <?php foreach( $tabs as $id => $title ) : ?>
         <a class="glsr-nav-tab nav-tab" href="#<?= $id; ?>"><?= $title; ?></a>
         <?php endforeach; ?>
     </h2>
     <form class="glsr-form" action="options.php" enctype="multipart/form-data" method="post">
-        <?php foreach ($tabs as $id => $title) : ?>
+        <?php foreach( $tabs as $id => $title ) : ?>
         <div class="glsr-nav-view ui-tabs-hide" id="<?= $id; ?>">
-            <?= $settings->buildFields($id); ?>
+            <?= $settings->buildFields( $id ); ?>
         </div>
         <?php endforeach; ?>
         <input type="hidden" name="_active_tab">
-        <?php settings_fields(glsr()->id.'-settings'); ?>
+        <?php settings_fields( glsr()->id.'-settings' ); ?>
         <?php submit_button(); ?>
     </form>
 </div>

views/pages/tools/index.php

Spacing +5 added lines, -5 removed lines

@@ -1,16 +1,16 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <div class="wrap">
-    <h1 class="wp-heading-inline"><?= esc_html(get_admin_page_title()); ?></h1>
+    <h1 class="wp-heading-inline"><?= esc_html( get_admin_page_title() ); ?></h1>
     <?= $notices; ?>
     <h2 class="glsr-nav-tab-wrapper nav-tab-wrapper">
-        <?php foreach ($tabs as $id => $title) : ?>
+        <?php foreach( $tabs as $id => $title ) : ?>
         <a class="glsr-nav-tab nav-tab" href="#<?= $id; ?>"><?= $title; ?></a>
         <?php endforeach; ?>
     </h2>
-    <?php foreach ($tabs as $id => $title) : ?>
+    <?php foreach( $tabs as $id => $title ) : ?>
     <div class="glsr-nav-view ui-tabs-hide" id="<?= $id; ?>">
-        <?php $template->render('pages/tools/'.$id, $data); ?>
+        <?php $template->render( 'pages/tools/'.$id, $data ); ?>
     </div>
     <?php endforeach; ?>
     <input type="hidden" name="_active_tab">

plugin/Modules/Html/Field.php

Spacing +43 added lines, -43 removed lines

@@ -14,9 +14,9 @@ 
      */
     public $field;
 
-    public function __construct(array $field = [])
+    public function __construct( array $field = [] )
     {
-        $this->field = wp_parse_args($field, [
+        $this->field = wp_parse_args( $field, [
             'errors' => false,
             'is_hidden' => false,
             'is_multi' => false,
@@ -26,7 +26,7 @@ 
             'is_valid' => true,
             'is_widget' => false,
             'path' => '',
-        ]);
+        ] );
         $this->normalize();
     }
 
@@ -35,7 +35,7 @@ 
      */
     public function __toString()
     {
-        return (string) $this->build();
+        return (string)$this->build();
     }
 
     /**
@@ -43,16 +43,16 @@ 
      */
     public function build()
     {
-        if (!$this->field['is_valid']) {
+        if( !$this->field['is_valid'] ) {
             return;
         }
-        if ($this->field['is_raw']) {
-            return glsr(Builder::class)->{$this->field['type']}($this->field);
+        if( $this->field['is_raw'] ) {
+            return glsr( Builder::class )->{$this->field['type']}($this->field);
         }
-        if (!$this->field['is_setting']) {
+        if( !$this->field['is_setting'] ) {
             return $this->buildField();
         }
-        if (!$this->field['is_multi']) {
+        if( !$this->field['is_multi'] ) {
             return $this->buildSettingField();
         }
         return $this->buildSettingMultiField();
@@ -71,22 +71,22 @@ 
      */
     protected function buildField()
     {
-        $field = glsr(Template::class)->build('templates/form/field_'.$this->field['type'], [
+        $field = glsr( Template::class )->build( 'templates/form/field_'.$this->field['type'], [
             'context' => [
                 'class' => $this->getFieldClass(),
                 'errors' => $this->getFieldErrors(),
-                'field' => glsr(Builder::class)->raw($this->field),
-                'label' => glsr(Builder::class)->label([
+                'field' => glsr( Builder::class )->raw( $this->field ),
+                'label' => glsr( Builder::class )->label( [
                     'class' => 'glsr-'.$this->field['type'].'-label',
                     'for' => $this->field['id'],
                     'is_public' => $this->field['is_public'],
                     'text' => $this->field['label'].'<span></span>',
                     'type' => $this->field['type'],
-                ]),
+                ] ),
             ],
             'field' => $this->field,
-        ]);
-        return apply_filters('site-reviews/rendered/field', $field, $this->field['type'], $this->field);
+        ] );
+        return apply_filters( 'site-reviews/rendered/field', $field, $this->field['type'], $this->field );
     }
 
     /**
@@ -94,14 +94,14 @@ 
      */
     protected function buildSettingField()
     {
-        return glsr(Template::class)->build('partials/form/table-row', [
+        return glsr( Template::class )->build( 'partials/form/table-row', [
             'context' => [
                 'class' => $this->getFieldClass(),
-                'field' => glsr(Builder::class)->{$this->field['type']}($this->field),
-                'label' => glsr(Builder::class)->label($this->field['legend'], ['for' => $this->field['id']]),
+                'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
+                'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
             ],
             'field' => $this->field,
-        ]);
+        ] );
     }
 
     /**
@@ -111,16 +111,16 @@ 
     {
         $dependsOn = $this->getFieldDependsOn();
         unset($this->field['data-depends']);
-        return glsr(Template::class)->build('partials/form/table-row-multiple', [
+        return glsr( Template::class )->build( 'partials/form/table-row-multiple', [
             'context' => [
                 'class' => $this->getFieldClass(),
                 'depends_on' => $dependsOn,
-                'field' => glsr(Builder::class)->{$this->field['type']}($this->field),
-                'label' => glsr(Builder::class)->label($this->field['legend'], ['for' => $this->field['id']]),
+                'field' => glsr( Builder::class )->{$this->field['type']}($this->field),
+                'label' => glsr( Builder::class )->label( $this->field['legend'], ['for' => $this->field['id']] ),
                 'legend' => $this->field['legend'],
             ],
             'field' => $this->field,
-        ]);
+        ] );
     }
 
     /**
@@ -129,17 +129,17 @@ 
     protected function getFieldClass()
     {
         $classes = [];
-        if (!empty($this->field['errors'])) {
+        if( !empty($this->field['errors']) ) {
             $classes[] = 'glsr-has-error';
         }
-        if ($this->field['is_hidden']) {
+        if( $this->field['is_hidden'] ) {
             $classes[] = 'hidden';
         }
-        if (!empty($this->field['required'])) {
+        if( !empty($this->field['required']) ) {
             $classes[] = 'glsr-required';
         }
-        $classes = apply_filters('site-reviews/rendered/field/classes', $classes, $this->field);
-        return implode(' ', $classes);
+        $classes = apply_filters( 'site-reviews/rendered/field/classes', $classes, $this->field );
+        return implode( ' ', $classes );
     }
 
     /**
@@ -157,18 +157,18 @@ 
      */
     protected function getFieldErrors()
     {
-        if (empty($this->field['errors']) || !is_array($this->field['errors'])) {
+        if( empty($this->field['errors']) || !is_array( $this->field['errors'] ) ) {
             return;
         }
-        $errors = array_reduce($this->field['errors'], function ($carry, $error) {
-            return $carry.glsr(Builder::class)->span($error, ['class' => 'glsr-field-error']);
+        $errors = array_reduce( $this->field['errors'], function( $carry, $error ) {
+            return $carry.glsr( Builder::class )->span( $error, ['class' => 'glsr-field-error'] );
         });
-        return glsr(Template::class)->build('templates/form/field-errors', [
+        return glsr( Template::class )->build( 'templates/form/field-errors', [
             'context' => [
                 'errors' => $errors,
             ],
             'field' => $this->field,
-        ]);
+        ] );
     }
 
     /**
@@ -190,17 +190,17 @@ 
         $requiredValues = [
             'name', 'type',
         ];
-        foreach ($requiredValues as $value) {
-            if (isset($this->field[$value])) {
+        foreach( $requiredValues as $value ) {
+            if( isset($this->field[$value]) ) {
                 continue;
             }
             $missingValues[] = $value;
             $this->field['is_valid'] = false;
         }
-        if (!empty($missingValues)) {
+        if( !empty($missingValues) ) {
             glsr_log()
-                ->warning('Field is missing: '.implode(', ', $missingValues))
-                ->debug($this->field);
+                ->warning( 'Field is missing: '.implode( ', ', $missingValues ) )
+                ->debug( $this->field );
         }
         return $this->field['is_valid'];
     }
@@ -210,13 +210,13 @@ 
      */
     protected function normalize()
     {
-        if (!$this->isFieldValid()) {
+        if( !$this->isFieldValid() ) {
             return;
         }
         $this->field['path'] = $this->field['name'];
-        $className = Helper::buildClassName($this->field['type'], __NAMESPACE__.'\Fields');
-        if (class_exists($className)) {
-            $this->field = $className::merge($this->field);
+        $className = Helper::buildClassName( $this->field['type'], __NAMESPACE__.'\Fields' );
+        if( class_exists( $className ) ) {
+            $this->field = $className::merge( $this->field );
         }
         $this->normalizeFieldId();
         $this->normalizeFieldName();
@@ -227,7 +227,7 @@ 
      */
     protected function normalizeFieldId()
     {
-        if (isset($this->field['id']) || $this->field['is_raw']) {
+        if( isset($this->field['id']) || $this->field['is_raw'] ) {
             return;
         }
         $this->field['id'] = Str::convertPathToId(

plugin/Controllers/EditorController/Metaboxes.php

Spacing +9 added lines, -9 removed lines

@@ -11,29 +11,29 @@
      * @param int $postId
      * @return void
      */
-    public function saveAssignedToMetabox($postId)
+    public function saveAssignedToMetabox( $postId )
     {
-        if (!wp_verify_nonce(Helper::filterInput('_nonce-assigned-to'), 'assigned_to')) {
+        if( !wp_verify_nonce( Helper::filterInput( '_nonce-assigned-to' ), 'assigned_to' ) ) {
             return;
         }
-        $assignedTo = strval(Helper::filterInput('assigned_to'));
-        glsr(Database::class)->update($postId, 'assigned_to', $assignedTo);
+        $assignedTo = strval( Helper::filterInput( 'assigned_to' ) );
+        glsr( Database::class )->update( $postId, 'assigned_to', $assignedTo );
     }
 
     /**
      * @param int $postId
      * @return mixed
      */
-    public function saveResponseMetabox($postId)
+    public function saveResponseMetabox( $postId )
     {
-        if (!wp_verify_nonce(Helper::filterInput('_nonce-response'), 'response')) {
+        if( !wp_verify_nonce( Helper::filterInput( '_nonce-response' ), 'response' ) ) {
             return;
         }
-        $response = strval(Helper::filterInput('response'));
-        glsr(Database::class)->update($postId, 'response', trim(wp_kses($response, [
+        $response = strval( Helper::filterInput( 'response' ) );
+        glsr( Database::class )->update( $postId, 'response', trim( wp_kses( $response, [
             'a' => ['href' => [], 'title' => []],
             'em' => [],
             'strong' => [],
-        ])));
+        ] ) ) );
     }
 }

plugin/Commands/RegisterPostType.php

Spacing +28 added lines, -28 removed lines

@@ -15,10 +15,10 @@ 
     public $postType;
     public $single;
 
-    public function __construct($input)
+    public function __construct( $input )
     {
-        $args = glsr(PostTypeDefaults::class)->merge($input);
-        $this->normalize($args);
+        $args = glsr( PostTypeDefaults::class )->merge( $input );
+        $this->normalize( $args );
         $this->normalizeColumns();
         $this->normalizeLabels();
     }
@@ -26,19 +26,19 @@ 
     /**
      * @return void
      */
-    protected function normalize(array $args)
+    protected function normalize( array $args )
     {
-        foreach ($args as $key => $value) {
-            $property = Helper::buildPropertyName($key);
-            if (!property_exists($this, $property)) {
+        foreach( $args as $key => $value ) {
+            $property = Helper::buildPropertyName( $key );
+            if( !property_exists( $this, $property ) ) {
                 continue;
             }
             $this->$property = $value;
             unset($args[$key]);
         }
-        $this->args = wp_parse_args($args, [
+        $this->args = wp_parse_args( $args, [
             'menu_name' => $this->plural,
-        ]);
+        ] );
     }
 
     /**
@@ -46,22 +46,22 @@ 
      */
     protected function normalizeLabels()
     {
-        $this->args['labels'] = wp_parse_args($this->args['labels'], [
-            'add_new_item' => sprintf(_x('Add New %s', 'Add New Post', 'site-reviews'), $this->plural),
-            'all_items' => sprintf(_x('All %s', 'All Posts', 'site-reviews'), $this->plural),
-            'archives' => sprintf(_x('%s Archives', 'Post Archives', 'site-reviews'), $this->single),
-            'edit_item' => sprintf(_x('Edit %s', 'Edit Post', 'site-reviews'), $this->single),
-            'insert_into_item' => sprintf(_x('Insert into %s', 'Insert into Post', 'site-reviews'), $this->single),
+        $this->args['labels'] = wp_parse_args( $this->args['labels'], [
+            'add_new_item' => sprintf( _x( 'Add New %s', 'Add New Post', 'site-reviews' ), $this->plural ),
+            'all_items' => sprintf( _x( 'All %s', 'All Posts', 'site-reviews' ), $this->plural ),
+            'archives' => sprintf( _x( '%s Archives', 'Post Archives', 'site-reviews' ), $this->single ),
+            'edit_item' => sprintf( _x( 'Edit %s', 'Edit Post', 'site-reviews' ), $this->single ),
+            'insert_into_item' => sprintf( _x( 'Insert into %s', 'Insert into Post', 'site-reviews' ), $this->single ),
             'menu_name' => $this->args['menu_name'],
             'name' => $this->plural,
-            'new_item' => sprintf(_x('New %s', 'New Post', 'site-reviews'), $this->single),
-            'not_found' => sprintf(_x('No %s found', 'No Posts found', 'site-reviews'), $this->plural),
-            'not_found_in_trash' => sprintf(_x('No %s found in Trash', 'No Posts found in Trash', 'site-reviews'), $this->plural),
-            'search_items' => sprintf(_x('Search %s', 'Search Posts', 'site-reviews'), $this->plural),
+            'new_item' => sprintf( _x( 'New %s', 'New Post', 'site-reviews' ), $this->single ),
+            'not_found' => sprintf( _x( 'No %s found', 'No Posts found', 'site-reviews' ), $this->plural ),
+            'not_found_in_trash' => sprintf( _x( 'No %s found in Trash', 'No Posts found in Trash', 'site-reviews' ), $this->plural ),
+            'search_items' => sprintf( _x( 'Search %s', 'Search Posts', 'site-reviews' ), $this->plural ),
             'singular_name' => $this->single,
-            'uploaded_to_this_item' => sprintf(_x('Uploaded to this %s', 'Uploaded to this Post', 'site-reviews'), $this->single),
-            'view_item' => sprintf(_x('View %s', 'View Post', 'site-reviews'), $this->single),
-        ]);
+            'uploaded_to_this_item' => sprintf( _x( 'Uploaded to this %s', 'Uploaded to this Post', 'site-reviews' ), $this->single ),
+            'view_item' => sprintf( _x( 'View %s', 'View Post', 'site-reviews' ), $this->single ),
+        ] );
         unset($this->args['menu_name']);
     }
 
@@ -71,13 +71,13 @@ 
     protected function normalizeColumns()
     {
         $this->columns = ['cb' => ''] + $this->columns;
-        if (array_key_exists('category', $this->columns)) {
-            $keys = array_keys($this->columns);
-            $keys[array_search('category', $keys)] = 'taxonomy-'.Application::TAXONOMY;
-            $this->columns = array_combine($keys, $this->columns);
+        if( array_key_exists( 'category', $this->columns ) ) {
+            $keys = array_keys( $this->columns );
+            $keys[array_search( 'category', $keys )] = 'taxonomy-'.Application::TAXONOMY;
+            $this->columns = array_combine( $keys, $this->columns );
         }
-        if (array_key_exists('pinned', $this->columns)) {
-            $this->columns['pinned'] = glsr(Builder::class)->span('<span>'.$this->columns['pinned'].'</span>',
+        if( array_key_exists( 'pinned', $this->columns ) ) {
+            $this->columns['pinned'] = glsr( Builder::class )->span( '<span>'.$this->columns['pinned'].'</span>',
                 ['class' => 'pinned-icon']
             );
         }

plugin/Router.php

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

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

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;