pryley / site-reviews

views/pages/settings/general.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <table class="form-table">
     <tbody>

views/pages/settings/reviews.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <table class="form-table">
     <tbody>

activate.php

Spacing +41 added lines, -41 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-defined('WPINC') || die;
+defined( 'WPINC' ) || die;
 
 /**
  * Check for minimum system requirments on plugin activation.
@@ -24,13 +24,13 @@ 
     /**
      * @param string $file
      */
-    public function __construct($file, array $versions = array())
+    public function __construct( $file, array $versions = array() )
     {
-        $this->file = realpath($file);
-        $this->versions = wp_parse_args($versions, array(
+        $this->file = realpath( $file );
+        $this->versions = wp_parse_args( $versions, array(
             'php' => static::MIN_PHP_VERSION,
             'wordpress' => static::MIN_WORDPRESS_VERSION,
-        ));
+        ) );
     }
 
     /**
@@ -38,11 +38,11 @@ 
      */
     public function canProceed()
     {
-        if ($this->isValid()) {
+        if( $this->isValid() ) {
             return true;
         }
-        add_action('activated_plugin', array($this, 'deactivate'));
-        add_action('admin_notices', array($this, 'deactivate'));
+        add_action( 'activated_plugin', array( $this, 'deactivate' ) );
+        add_action( 'admin_notices', array( $this, 'deactivate' ) );
         return false;
     }
 
@@ -51,7 +51,7 @@ 
      */
     public function isPhpValid()
     {
-        return !version_compare(PHP_VERSION, $this->versions['php'], '<');
+        return !version_compare( PHP_VERSION, $this->versions['php'], '<' );
     }
 
     /**
@@ -68,25 +68,25 @@ 
     public function isWpValid()
     {
         global $wp_version;
-        return !version_compare($wp_version, $this->versions['wordpress'], '<');
+        return !version_compare( $wp_version, $this->versions['wordpress'], '<' );
     }
 
     /**
      * @param string $plugin
      * @return void
      */
-    public function deactivate($plugin)
+    public function deactivate( $plugin )
     {
-        if ($this->isValid()) {
+        if( $this->isValid() ) {
             return;
         }
-        $pluginSlug = plugin_basename($this->file);
-        if ($plugin == $pluginSlug) {
+        $pluginSlug = plugin_basename( $this->file );
+        if( $plugin == $pluginSlug ) {
             $this->redirect(); //exit
         }
-        $pluginData = get_file_data($this->file, array('name' => 'Plugin Name'), 'plugin');
-        deactivate_plugins($pluginSlug);
-        $this->printNotice($pluginData['name']);
+        $pluginData = get_file_data( $this->file, array( 'name' => 'Plugin Name' ), 'plugin' );
+        deactivate_plugins( $pluginSlug );
+        $this->printNotice( $pluginData['name'] );
     }
 
     /**
@@ -95,13 +95,13 @@ 
     protected function getMessages()
     {
         return array(
-            __('The %s plugin was deactivated.', 'site-reviews'),
-            __('This plugin requires %s or greater in order to work properly.', 'site-reviews'),
-            __('Please contact your hosting provider or server administrator to upgrade the version of PHP on your server (your server is running PHP version %s), or try to find an alternative plugin.', 'site-reviews'),
-            __('PHP version', 'site-reviews'),
-            __('WordPress version', 'site-reviews'),
-            __('Update WordPress', 'site-reviews'),
-            __('You can use the %s plugin to restore %s to the previous version.', 'site-reviews'),
+            __( 'The %s plugin was deactivated.', 'site-reviews' ),
+            __( 'This plugin requires %s or greater in order to work properly.', 'site-reviews' ),
+            __( 'Please contact your hosting provider or server administrator to upgrade the version of PHP on your server (your server is running PHP version %s), or try to find an alternative plugin.', 'site-reviews' ),
+            __( 'PHP version', 'site-reviews' ),
+            __( 'WordPress version', 'site-reviews' ),
+            __( 'Update WordPress', 'site-reviews' ),
+            __( 'You can use the %s plugin to restore %s to the previous version.', 'site-reviews' ),
         );
     }
 
@@ -109,22 +109,22 @@ 
      * @param string $pluginName
      * @return void
      */
-    protected function printNotice($pluginName)
+    protected function printNotice( $pluginName )
     {
         $noticeTemplate = '<div id="message" class="notice notice-error error is-dismissible"><p><strong>%s</strong></p><p>%s</p><p>%s</p></div>';
         $messages = $this->getMessages();
-        $rollbackMessage = sprintf('<strong>'.$messages[6].'</strong>', '<a href="https://wordpress.org/plugins/wp-rollback/">WP Rollback</a>', $pluginName);
-        if (!$this->isPhpValid()) {
-            printf($noticeTemplate,
-                sprintf($messages[0], $pluginName),
-                sprintf($messages[1], $messages[3].' '.$this->versions['php']),
-                sprintf($messages[2], PHP_VERSION).'</p><p>'.$rollbackMessage
+        $rollbackMessage = sprintf( '<strong>'.$messages[6].'</strong>', '<a href="https://wordpress.org/plugins/wp-rollback/">WP Rollback</a>', $pluginName );
+        if( !$this->isPhpValid() ) {
+            printf( $noticeTemplate,
+                sprintf( $messages[0], $pluginName ),
+                sprintf( $messages[1], $messages[3].' '.$this->versions['php'] ),
+                sprintf( $messages[2], PHP_VERSION ).'</p><p>'.$rollbackMessage
             );
-        } elseif (!$this->isWpValid()) {
-            printf($noticeTemplate,
-                sprintf($messages[0], $pluginName),
-                sprintf($messages[1], $messages[4].' '.$this->versions['wordpress']),
-                $rollbackMessage.'</p><p>'.sprintf('<a href="%s">%s</a>', admin_url('update-core.php'), $messages[5])
+        } elseif( !$this->isWpValid() ) {
+            printf( $noticeTemplate,
+                sprintf( $messages[0], $pluginName ),
+                sprintf( $messages[1], $messages[4].' '.$this->versions['wordpress'] ),
+                $rollbackMessage.'</p><p>'.sprintf( '<a href="%s">%s</a>', admin_url( 'update-core.php' ), $messages[5] )
             );
         }
     }
@@ -134,11 +134,11 @@ 
      */
     protected function redirect()
     {
-        wp_safe_redirect(self_admin_url(sprintf('plugins.php?plugin_status=%s&paged=%s&s=%s',
-            filter_input(INPUT_GET, 'plugin_status'),
-            filter_input(INPUT_GET, 'paged'),
-            filter_input(INPUT_GET, 's')
-        )));
+        wp_safe_redirect( self_admin_url( sprintf( 'plugins.php?plugin_status=%s&paged=%s&s=%s',
+            filter_input( INPUT_GET, 'plugin_status' ),
+            filter_input( INPUT_GET, 'paged' ),
+            filter_input( INPUT_GET, 's' )
+        ) ) );
         exit;
     }
 }

templates/reviews-form.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <div class="glsr-form-wrap">
     <form class="{{ class }}" id="{{ id }}" method="post" enctype="multipart/form-data">

templates/form/response.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <div class="glsr-form-message {{ class }}">
     {{ message }}

plugin/Modules/Html/Settings.php

Spacing +67 added lines, -67 removed lines

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

plugin/Modules/Html/Builder.php

Spacing +70 added lines, -70 removed lines

@@ -61,18 +61,18 @@ 
      * @param array $args
      * @return string|void
      */
-    public function __call($method, $args)
+    public function __call( $method, $args )
     {
         $instance = new static();
-        $instance->setTagFromMethod($method);
-        call_user_func_array([$instance, 'normalize'], $args += ['', '']);
-        $tags = array_merge(static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT);
-        do_action_ref_array('site-reviews/builder', [$instance]);
-        $generatedTag = in_array($instance->tag, $tags)
+        $instance->setTagFromMethod( $method );
+        call_user_func_array( [$instance, 'normalize'], $args += ['', ''] );
+        $tags = array_merge( static::TAGS_FORM, static::TAGS_SINGLE, static::TAGS_STRUCTURE, static::TAGS_TEXT );
+        do_action_ref_array( 'site-reviews/builder', [$instance] );
+        $generatedTag = in_array( $instance->tag, $tags )
             ? $instance->buildTag()
             : $instance->buildCustomField();
-        $generatedTag = apply_filters('site-reviews/builder/result', $generatedTag, $instance);
-        if (!$this->render) {
+        $generatedTag = apply_filters( 'site-reviews/builder/result', $generatedTag, $instance );
+        if( !$this->render ) {
             return $generatedTag;
         }
         echo $generatedTag;
@@ -83,15 +83,15 @@ 
      * @param mixed $value
      * @return void
      */
-    public function __set($property, $value)
+    public function __set( $property, $value )
     {
         $properties = [
             'args' => 'is_array',
             'render' => 'is_bool',
             'tag' => 'is_string',
         ];
-        if (!isset($properties[$property])
-            || empty(array_filter([$value], $properties[$property]))
+        if( !isset($properties[$property])
+            || empty(array_filter( [$value], $properties[$property] ))
         ) {
             return;
         }
@@ -103,7 +103,7 @@ 
      */
     public function getClosingTag()
     {
-        if (empty($this->tag)) {
+        if( empty($this->tag) ) {
             return;
         }
         return '</'.$this->tag.'>';
@@ -114,11 +114,11 @@ 
      */
     public function getOpeningTag()
     {
-        if (empty($this->tag)) {
+        if( empty($this->tag) ) {
             return;
         }
-        $attributes = glsr(Attributes::class)->{$this->tag}($this->args)->toString();
-        return '<'.trim($this->tag.' '.$attributes).'>';
+        $attributes = glsr( Attributes::class )->{$this->tag}($this->args)->toString();
+        return '<'.trim( $this->tag.' '.$attributes ).'>';
     }
 
     /**
@@ -126,19 +126,19 @@ 
      */
     public function getTag()
     {
-        if (in_array($this->tag, static::TAGS_SINGLE)) {
+        if( in_array( $this->tag, static::TAGS_SINGLE ) ) {
             return $this->getOpeningTag();
         }
-        if (!in_array($this->tag, static::TAGS_FORM)) {
+        if( !in_array( $this->tag, static::TAGS_FORM ) ) {
             return $this->buildDefaultTag();
         }
-        return call_user_func([$this, 'buildForm'.ucfirst($this->tag)]).$this->buildFieldDescription();
+        return call_user_func( [$this, 'buildForm'.ucfirst( $this->tag )] ).$this->buildFieldDescription();
     }
 
     /**
      * @return string
      */
-    public function raw(array $field)
+    public function raw( array $field )
     {
         unset($field['label']);
         return $this->{$field['type']}($field);
@@ -150,18 +150,18 @@ 
     protected function buildCustomField()
     {
         $className = $this->getCustomFieldClassName();
-        if (class_exists($className)) {
-            return (new $className($this))->build();
+        if( class_exists( $className ) ) {
+            return (new $className( $this ))->build();
         }
-        glsr_log()->error('Field missing: '.$className);
+        glsr_log()->error( 'Field missing: '.$className );
     }
 
     /**
      * @return string|void
      */
-    protected function buildDefaultTag($text = '')
+    protected function buildDefaultTag( $text = '' )
     {
-        if (empty($text)) {
+        if( empty($text) ) {
             $text = $this->args['text'];
         }
         return $this->getOpeningTag().$text.$this->getClosingTag();
@@ -172,13 +172,13 @@ 
      */
     protected function buildFieldDescription()
     {
-        if (empty($this->args['description'])) {
+        if( empty($this->args['description']) ) {
             return;
         }
-        if ($this->args['is_widget']) {
-            return $this->small($this->args['description']);
+        if( $this->args['is_widget'] ) {
+            return $this->small( $this->args['description'] );
         }
-        return $this->p($this->args['description'], ['class' => 'description']);
+        return $this->p( $this->args['description'], ['class' => 'description'] );
     }
 
     /**
@@ -186,9 +186,9 @@ 
      */
     protected function buildFormInput()
     {
-        if (!in_array($this->args['type'], ['checkbox', 'radio'])) {
-            if (isset($this->args['multiple'])) {
-                $this->args['name'].= '[]';
+        if( !in_array( $this->args['type'], ['checkbox', 'radio'] ) ) {
+            if( isset($this->args['multiple']) ) {
+                $this->args['name'] .= '[]';
             }
             return $this->buildFormLabel().$this->getOpeningTag();
         }
@@ -202,19 +202,19 @@ 
      */
     protected function buildFormInputChoice()
     {
-        if (!empty($this->args['text'])) {
+        if( !empty($this->args['text']) ) {
             $this->args['label'] = $this->args['text'];
         }
-        if (!$this->args['is_public']) {
-            return $this->buildFormLabel([
+        if( !$this->args['is_public'] ) {
+            return $this->buildFormLabel( [
                 'class' => 'glsr-'.$this->args['type'].'-label',
                 'text' => $this->getOpeningTag().' '.$this->args['label'].'<span></span>',
-            ]);
+            ] );
         }
-        return $this->getOpeningTag().$this->buildFormLabel([
+        return $this->getOpeningTag().$this->buildFormLabel( [
             'class' => 'glsr-'.$this->args['type'].'-label',
             'text' => $this->args['label'].'<span></span>',
-        ]);
+        ] );
     }
 
     /**
@@ -222,39 +222,39 @@ 
      */
     protected function buildFormInputMultiChoice()
     {
-        if ('checkbox' == $this->args['type']) {
-            $this->args['name'].= '[]';
+        if( 'checkbox' == $this->args['type'] ) {
+            $this->args['name'] .= '[]';
         }
         $index = 0;
-        $options = array_reduce(array_keys($this->args['options']), function ($carry, $key) use (&$index) {
-            return $carry.$this->li($this->{$this->args['type']}([
-                'checked' => in_array($key, (array) $this->args['value']),
+        $options = array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use (&$index) {
+            return $carry.$this->li( $this->{$this->args['type']}([
+                'checked' => in_array( $key, (array)$this->args['value'] ),
                 'id' => $this->args['id'].'-'.$index++,
                 'name' => $this->args['name'],
                 'text' => $this->args['options'][$key],
                 'value' => $key,
-            ]));
+            ]) );
         });
-        return $this->ul($options, [
+        return $this->ul( $options, [
             'class' => $this->args['class'],
             'id' => $this->args['id'],
-        ]);
+        ] );
     }
 
     /**
      * @return void|string
      */
-    protected function buildFormLabel(array $customArgs = [])
+    protected function buildFormLabel( array $customArgs = [] )
     {
-        if (empty($this->args['label']) || 'hidden' == $this->args['type']) {
+        if( empty($this->args['label']) || 'hidden' == $this->args['type'] ) {
             return;
         }
-        return $this->label(wp_parse_args($customArgs, [
+        return $this->label( wp_parse_args( $customArgs, [
             'for' => $this->args['id'],
             'is_public' => $this->args['is_public'],
             'text' => $this->args['label'],
             'type' => $this->args['type'],
-        ]));
+        ] ) );
     }
 
     /**
@@ -262,7 +262,7 @@ 
      */
     protected function buildFormSelect()
     {
-        return $this->buildFormLabel().$this->buildDefaultTag($this->buildFormSelectOptions());
+        return $this->buildFormLabel().$this->buildDefaultTag( $this->buildFormSelectOptions() );
     }
 
     /**
@@ -270,12 +270,12 @@ 
      */
     protected function buildFormSelectOptions()
     {
-        return array_reduce(array_keys($this->args['options']), function ($carry, $key) {
-            return $carry.$this->option([
+        return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) {
+            return $carry.$this->option( [
                 'selected' => $this->args['value'] == $key,
                 'text' => $this->args['options'][$key],
                 'value' => $key,
-            ]);
+            ] );
         });
     }
 
@@ -284,7 +284,7 @@ 
      */
     protected function buildFormTextarea()
     {
-        return $this->buildFormLabel().$this->buildDefaultTag($this->args['value']);
+        return $this->buildFormLabel().$this->buildDefaultTag( $this->args['value'] );
     }
 
     /**
@@ -301,8 +301,8 @@ 
      */
     protected function getCustomFieldClassName()
     {
-        $classname = glsr(Helper::class)->buildClassName($this->tag, __NAMESPACE__.'\Fields');
-        return apply_filters('site-reviews/builder/field/'.$this->tag, $classname);
+        $classname = glsr( Helper::class )->buildClassName( $this->tag, __NAMESPACE__.'\Fields' );
+        return apply_filters( 'site-reviews/builder/field/'.$this->tag, $classname );
     }
 
     /**
@@ -311,27 +311,27 @@ 
     protected function mergeArgsWithRequiredDefaults()
     {
         $className = $this->getCustomFieldClassName();
-        if (class_exists($className)) {
-            $this->args = $className::merge($this->args);
+        if( class_exists( $className ) ) {
+            $this->args = $className::merge( $this->args );
         }
-        $this->args = glsr(BuilderDefaults::class)->merge($this->args);
+        $this->args = glsr( BuilderDefaults::class )->merge( $this->args );
     }
 
     /**
      * @param string|array ...$params
      * @return void
      */
-    protected function normalize(...$params)
+    protected function normalize( ...$params )
     {
-        if (is_string($params[0]) || is_numeric($params[0])) {
-            $this->setNameOrTextAttributeForTag($params[0]);
+        if( is_string( $params[0] ) || is_numeric( $params[0] ) ) {
+            $this->setNameOrTextAttributeForTag( $params[0] );
         }
-        if (is_array($params[0])) {
+        if( is_array( $params[0] ) ) {
             $this->args += $params[0];
-        } elseif (is_array($params[1])) {
+        } elseif( is_array( $params[1] ) ) {
             $this->args += $params[1];
         }
-        if (!isset($this->args['is_public'])) {
+        if( !isset($this->args['is_public']) ) {
             $this->args['is_public'] = false;
         }
     }
@@ -340,9 +340,9 @@ 
      * @param string $value
      * @return void
      */
-    protected function setNameOrTextAttributeForTag($value)
+    protected function setNameOrTextAttributeForTag( $value )
     {
-        $attribute = in_array($this->tag, static::TAGS_FORM)
+        $attribute = in_array( $this->tag, static::TAGS_FORM )
             ? 'name'
             : 'text';
         $this->args[$attribute] = $value;
@@ -352,10 +352,10 @@ 
      * @param string $method
      * @return void
      */
-    protected function setTagFromMethod($method)
+    protected function setTagFromMethod( $method )
     {
-        $this->tag = strtolower($method);
-        if (in_array($this->tag, static::INPUT_TYPES)) {
+        $this->tag = strtolower( $method );
+        if( in_array( $this->tag, static::INPUT_TYPES ) ) {
             $this->args['type'] = $this->tag;
             $this->tag = 'input';
         }

plugin/Modules/Html/ReviewsHtml.php

Spacing +26 added lines, -26 removed lines

@@ -27,13 +27,13 @@ 
      */
     public $reviews;
 
-    public function __construct(array $reviews, $maxPageCount, array $args)
+    public function __construct( array $reviews, $maxPageCount, array $args )
     {
         $this->args = $args;
         $this->max_num_pages = $maxPageCount;
         $this->reviews = $reviews;
         $this->pagination = $this->buildPagination();
-        parent::__construct($reviews, ArrayObject::STD_PROP_LIST | ArrayObject::ARRAY_AS_PROPS);
+        parent::__construct( $reviews, ArrayObject::STD_PROP_LIST | ArrayObject::ARRAY_AS_PROPS );
     }
 
     /**
@@ -41,7 +41,7 @@ 
      */
     public function __toString()
     {
-        return glsr(Template::class)->build('templates/reviews', [
+        return glsr( Template::class )->build( 'templates/reviews', [
             'args' => $this->args,
             'context' => [
                 'assigned_to' => $this->args['assigned_to'],
@@ -51,7 +51,7 @@ 
                 'pagination' => $this->getPagination(),
                 'reviews' => $this->getReviews(),
             ],
-        ]);
+        ] );
     }
 
     /**
@@ -59,7 +59,7 @@ 
      */
     public function getPagination()
     {
-        return wp_validate_boolean($this->args['pagination'])
+        return wp_validate_boolean( $this->args['pagination'] )
             ? $this->pagination
             : '';
     }
@@ -71,26 +71,26 @@ 
     {
         $html = empty($this->reviews)
             ? $this->getReviewsFallback()
-            : implode(PHP_EOL, $this->reviews);
+            : implode( PHP_EOL, $this->reviews );
         $wrapper = '<div class="glsr-reviews">%s</div>';
-        $wrapper = apply_filters('site-reviews/reviews/reviews-wrapper', $wrapper);
-        return sprintf($wrapper, $html);
+        $wrapper = apply_filters( 'site-reviews/reviews/reviews-wrapper', $wrapper );
+        return sprintf( $wrapper, $html );
     }
 
     /**
      * @param mixed $key
      * @return mixed
      */
-    public function offsetGet($key)
+    public function offsetGet( $key )
     {
-        if ('navigation' == $key) {
+        if( 'navigation' == $key ) {
             glsr()->deprecated[] = 'The $reviewsHtml->navigation property has been been deprecated. Please use the $reviewsHtml->pagination property instead.';
             return $this->pagination;
         }
-        if (property_exists($this, $key)) {
+        if( property_exists( $this, $key ) ) {
             return $this->{$key};
         }
-        return array_key_exists($key, $this->reviews)
+        return array_key_exists( $key, $this->reviews )
             ? $this->reviews[$key]
             : null;
     }
@@ -100,15 +100,15 @@ 
      */
     protected function buildPagination()
     {
-        $html = glsr(Partial::class)->build('pagination', [
-            'baseUrl' => glsr_get($this->args, 'pagedUrl'),
-            'current' => glsr_get($this->args, 'paged'),
+        $html = glsr( Partial::class )->build( 'pagination', [
+            'baseUrl' => glsr_get( $this->args, 'pagedUrl' ),
+            'current' => glsr_get( $this->args, 'paged' ),
             'total' => $this->max_num_pages,
-        ]);
-        $html.= sprintf('<glsr-pagination hidden data-atts=\'%s\'></glsr-pagination>', $this->args['json']);
+        ] );
+        $html .= sprintf( '<glsr-pagination hidden data-atts=\'%s\'></glsr-pagination>', $this->args['json'] );
         $wrapper = '<div class="glsr-pagination">%s</div>';
-        $wrapper = apply_filters('site-reviews/reviews/pagination-wrapper', $wrapper);
-        return sprintf($wrapper, $html);
+        $wrapper = apply_filters( 'site-reviews/reviews/pagination-wrapper', $wrapper );
+        return sprintf( $wrapper, $html );
     }
 
     /**
@@ -119,12 +119,12 @@ 
         $defaults = [
             'glsr-default',
         ];
-        if ('ajax' == $this->args['pagination']) {
+        if( 'ajax' == $this->args['pagination'] ) {
             $defaults[] = 'glsr-ajax-pagination';
         }
-        $classes = explode(' ', $this->args['class']);
-        $classes = array_unique(array_merge($defaults, array_filter($classes)));
-        return implode(' ', $classes);
+        $classes = explode( ' ', $this->args['class'] );
+        $classes = array_unique( array_merge( $defaults, array_filter( $classes ) ) );
+        return implode( ' ', $classes );
     }
 
     /**
@@ -132,10 +132,10 @@ 
      */
     protected function getReviewsFallback()
     {
-        if (empty($this->args['fallback']) && glsr(OptionManager::class)->getBool('settings.reviews.fallback')) {
-            $this->args['fallback'] = __('There are no reviews yet. Be the first one to write one.', 'site-reviews');
+        if( empty($this->args['fallback']) && glsr( OptionManager::class )->getBool( 'settings.reviews.fallback' ) ) {
+            $this->args['fallback'] = __( 'There are no reviews yet. Be the first one to write one.', 'site-reviews' );
         }
         $fallback = '<p class="glsr-no-margins">'.$this->args['fallback'].'</p>';
-        return apply_filters('site-reviews/reviews/fallback', $fallback, $this->args);
+        return apply_filters( 'site-reviews/reviews/fallback', $fallback, $this->args );
     }
 }

plugin/Modules/Notice.php

Spacing +30 added lines, -30 removed lines

@@ -13,41 +13,41 @@ 
      * @param string|array|WP_Error $message
      * @return void
      */
-    public function add($type, $message, array $args = [])
+    public function add( $type, $message, array $args = [] )
     {
-        if (empty(array_filter([$message, $type]))) {
+        if( empty(array_filter( [$message, $type] )) ) {
             return;
         }
         $args['message'] = $message;
         $args['type'] = $type;
-        add_settings_error(Application::ID, '', json_encode($this->normalize($args)));
+        add_settings_error( Application::ID, '', json_encode( $this->normalize( $args ) ) );
     }
 
     /**
      * @param string|array|WP_Error $message
      * @return void
      */
-    public function addError($message, array $args = [])
+    public function addError( $message, array $args = [] )
     {
-        $this->add('error', $message, $args);
+        $this->add( 'error', $message, $args );
     }
 
     /**
      * @param string|array|WP_Error $message
      * @return void
      */
-    public function addSuccess($message, array $args = [])
+    public function addSuccess( $message, array $args = [] )
     {
-        $this->add('success', $message, $args);
+        $this->add( 'success', $message, $args );
     }
 
     /**
      * @param string|array|WP_Error $message
      * @return void
      */
-    public function addWarning($message, array $args = [])
+    public function addWarning( $message, array $args = [] )
     {
-        $this->add('warning', $message, $args);
+        $this->add( 'warning', $message, $args );
     }
 
     /**
@@ -57,7 +57,7 @@ 
     {
         global $wp_settings_errors;
         $wp_settings_errors = [];
-        delete_transient('settings_errors');
+        delete_transient( 'settings_errors' );
         return $this;
     }
 
@@ -66,41 +66,41 @@ 
      */
     public function get()
     {
-        $notices = array_map('unserialize',
-            array_unique(array_map('serialize', get_settings_errors(Application::ID)))
+        $notices = array_map( 'unserialize',
+            array_unique( array_map( 'serialize', get_settings_errors( Application::ID ) ) )
         );
-        if (empty($notices)) {
+        if( empty($notices) ) {
             return;
         }
-        return array_reduce($notices, function ($carry, $notice) {
-            return $carry.$this->buildNotice(json_decode($notice['message'], true));
+        return array_reduce( $notices, function( $carry, $notice ) {
+            return $carry.$this->buildNotice( json_decode( $notice['message'], true ) );
         });
     }
 
     /**
      * @return string
      */
-    protected function buildNotice(array $args)
+    protected function buildNotice( array $args )
     {
-        $messages = array_reduce($args['messages'], function ($carry, $message) {
-            return $carry.glsr(Builder::class)->p($message);
+        $messages = array_reduce( $args['messages'], function( $carry, $message ) {
+            return $carry.glsr( Builder::class )->p( $message );
         });
         $class = 'notice notice-'.$args['type'];
-        if ($args['inline']) {
-            $class.= ' inline';
+        if( $args['inline'] ) {
+            $class .= ' inline';
         }
-        if ($args['dismissible']) {
-            $class.= ' is-dismissible';
+        if( $args['dismissible'] ) {
+            $class .= ' is-dismissible';
         }
-        return glsr(Builder::class)->div($messages, [
+        return glsr( Builder::class )->div( $messages, [
             'class' => $class,
-        ]);
+        ] );
     }
 
     /**
      * @return array
      */
-    protected function normalize(array $args)
+    protected function normalize( array $args )
     {
         $defaults = [
             'dismissible' => true,
@@ -108,13 +108,13 @@ 
             'message' => '',
             'type' => '',
         ];
-        $args = shortcode_atts($defaults, $args);
-        if (!in_array($args['type'], ['error', 'warning', 'success'])) {
+        $args = shortcode_atts( $defaults, $args );
+        if( !in_array( $args['type'], ['error', 'warning', 'success'] ) ) {
             $args['type'] = 'success';
         }
-        $args['messages'] = is_wp_error($args['message'])
-            ? (array) $args['message']->get_error_message()
-            : (array) $args['message'];
+        $args['messages'] = is_wp_error( $args['message'] )
+            ? (array)$args['message']->get_error_message()
+            : (array)$args['message'];
         unset($args['message']);
         return $args;
     }