pryley / site-reviews

plugin/Helpers/Str.php

Spacing +51 added lines, -51 removed lines

@@ -9,21 +9,21 @@ 
      * @param string $search
      * @return string
      */
-    public static function afterLast($subject, $search)
+    public static function afterLast( $subject, $search )
     {
         return '' === $search
             ? $subject
-            : array_reverse(explode($search, $subject))[0];
+            : array_reverse( explode( $search, $subject ) )[0];
     }
 
     /**
      * @param string $string
      * @return string
      */
-    public static function camelCase($string)
+    public static function camelCase( $string )
     {
-        $string = ucwords(str_replace(['-', '_'], ' ', trim($string)));
-        return str_replace(' ', '', $string);
+        $string = ucwords( str_replace( ['-', '_'], ' ', trim( $string ) ) );
+        return str_replace( ' ', '', $string );
     }
 
     /**
@@ -31,9 +31,9 @@ 
      * @param string $needle
      * @return bool
      */
-    public static function contains($haystack, $needle)
+    public static function contains( $haystack, $needle )
     {
-        return false !== strpos($haystack, $needle);
+        return false !== strpos( $haystack, $needle );
     }
 
     /**
@@ -42,27 +42,27 @@ 
      * @param string $initialType period|period_space|space
      * @return string
      */
-    public static function convertName($name, $nameType = '', $initialType = '')
+    public static function convertName( $name, $nameType = '', $initialType = '' )
     {
-        $names = preg_split('/\W/', $name, 0, PREG_SPLIT_NO_EMPTY);
-        $firstName = array_shift($names);
-        $lastName = array_pop($names);
+        $names = preg_split( '/\W/', $name, 0, PREG_SPLIT_NO_EMPTY );
+        $firstName = array_shift( $names );
+        $lastName = array_pop( $names );
         $initialTypes = [
             'period' => '.',
             'period_space' => '. ',
             'space' => ' ',
         ];
-        $initialPunctuation = (string) Arr::get($initialTypes, $initialType, ' ');
-        if ('initials' == $nameType) {
-            return static::convertToInitials($name, $initialPunctuation);
+        $initialPunctuation = (string)Arr::get( $initialTypes, $initialType, ' ' );
+        if( 'initials' == $nameType ) {
+            return static::convertToInitials( $name, $initialPunctuation );
         }
         $nameTypes = [
             'first' => $firstName,
-            'first_initial' => substr($firstName, 0, 1).$initialPunctuation.$lastName,
+            'first_initial' => substr( $firstName, 0, 1 ).$initialPunctuation.$lastName,
             'last' => $lastName,
-            'last_initial' => $firstName.' '.substr($lastName, 0, 1).$initialPunctuation,
+            'last_initial' => $firstName.' '.substr( $lastName, 0, 1 ).$initialPunctuation,
         ];
-        return trim((string) Arr::get($nameTypes, $nameType, $name));
+        return trim( (string)Arr::get( $nameTypes, $nameType, $name ) );
     }
 
     /**
@@ -70,9 +70,9 @@ 
      * @param string $prefix
      * @return string
      */
-    public static function convertPathToId($path, $prefix = '')
+    public static function convertPathToId( $path, $prefix = '' )
     {
-        return str_replace(['[', ']'], ['-', ''], static::convertPathToName($path, $prefix));
+        return str_replace( ['[', ']'], ['-', ''], static::convertPathToName( $path, $prefix ) );
     }
 
     /**
@@ -80,12 +80,12 @@ 
      * @param string $prefix
      * @return string
      */
-    public static function convertPathToName($path, $prefix = '')
+    public static function convertPathToName( $path, $prefix = '' )
     {
-        $levels = explode('.', $path);
-        return array_reduce($levels, function ($result, $value) {
+        $levels = explode( '.', $path );
+        return array_reduce( $levels, function( $result, $value ) {
             return $result .= '['.$value.']';
-        }, $prefix);
+        }, $prefix );
     }
 
     /**
@@ -93,11 +93,11 @@ 
      * @param string $initialPunctuation
      * @return string
      */
-    public static function convertToInitials($name, $initialPunctuation = '')
+    public static function convertToInitials( $name, $initialPunctuation = '' )
     {
-        preg_match_all('/(?<=\s|\b)\pL/u', $name, $matches);
-        return array_reduce($matches[0], function ($carry, $word) use ($initialPunctuation) {
-            return $carry.strtoupper(substr($word, 0, 1)).$initialPunctuation;
+        preg_match_all( '/(?<=\s|\b)\pL/u', $name, $matches );
+        return array_reduce( $matches[0], function( $carry, $word ) use ($initialPunctuation) {
+            return $carry.strtoupper( substr( $word, 0, 1 ) ).$initialPunctuation;
         });
     }
 
@@ -105,9 +105,9 @@ 
      * @param string $string
      * @return string
      */
-    public static function dashCase($string)
+    public static function dashCase( $string )
     {
-        return str_replace('_', '-', static::snakeCase($string));
+        return str_replace( '_', '-', static::snakeCase( $string ) );
     }
 
     /**
@@ -115,11 +115,11 @@ 
      * @param string $haystack
      * @return bool
      */
-    public static function endsWith($needle, $haystack)
+    public static function endsWith( $needle, $haystack )
     {
-        $length = strlen($needle);
+        $length = strlen( $needle );
         return 0 != $length
-            ? substr($haystack, -$length) === $needle
+            ? substr( $haystack, -$length ) === $needle
             : true;
     }
 
@@ -129,12 +129,12 @@ 
      * @param string|null $trim
      * @return string
      */
-    public static function prefix($prefix, $string, $trim = null)
+    public static function prefix( $prefix, $string, $trim = null )
     {
-        if (null === $trim) {
+        if( null === $trim ) {
             $trim = $prefix;
         }
-        return $prefix.trim(static::removePrefix($trim, $string));
+        return $prefix.trim( static::removePrefix( $trim, $string ) );
     }
 
     /**
@@ -142,10 +142,10 @@ 
      * @param string $string
      * @return string
      */
-    public static function removePrefix($prefix, $string)
+    public static function removePrefix( $prefix, $string )
     {
-        return static::startsWith($prefix, $string)
-            ? substr($string, strlen($prefix))
+        return static::startsWith( $prefix, $string )
+            ? substr( $string, strlen( $prefix ) )
             : $string;
     }
 
@@ -153,16 +153,16 @@ 
      * @param string $string
      * @return string
      */
-    public static function snakeCase($string)
+    public static function snakeCase( $string )
     {
-        if (!ctype_lower($string)) {
-            $string = preg_replace('/\s+/u', '', $string);
-            $string = preg_replace('/(.)(?=[A-Z])/u', '$1_', $string);
-            $string = function_exists('mb_strtolower')
-                ? mb_strtolower($string, 'UTF-8')
-                : strtolower($string);
+        if( !ctype_lower( $string ) ) {
+            $string = preg_replace( '/\s+/u', '', $string );
+            $string = preg_replace( '/(.)(?=[A-Z])/u', '$1_', $string );
+            $string = function_exists( 'mb_strtolower' )
+                ? mb_strtolower( $string, 'UTF-8' )
+                : strtolower( $string );
         }
-        return str_replace('-', '_', $string);
+        return str_replace( '-', '_', $string );
     }
 
     /**
@@ -170,9 +170,9 @@ 
      * @param string $haystack
      * @return bool
      */
-    public static function startsWith($needle, $haystack)
+    public static function startsWith( $needle, $haystack )
     {
-        return substr($haystack, 0, strlen($needle)) === $needle;
+        return substr( $haystack, 0, strlen( $needle ) ) === $needle;
     }
 
     /**
@@ -180,10 +180,10 @@ 
      * @param int $length
      * @return string
      */
-    public static function truncate($string, $length)
+    public static function truncate( $string, $length )
     {
-        return strlen($string) > $length
-            ? substr($string, 0, $length)
+        return strlen( $string ) > $length
+            ? substr( $string, 0, $length )
             : $string;
     }
 }

plugin/Modules/Updater.php

Spacing +76 added lines, -76 removed lines

@@ -29,33 +29,33 @@ 
      * @param string $apiUrl
      * @param string $file
      */
-    public function __construct($apiUrl, $file, array $data = [])
+    public function __construct( $apiUrl, $file, array $data = [] )
     {
-        if (!function_exists('get_plugin_data')) {
+        if( !function_exists( 'get_plugin_data' ) ) {
             require_once ABSPATH.WPINC.'/plugin.php';
         }
-        $this->apiUrl = trailingslashit(apply_filters('site-reviews/addon/api-url', $apiUrl));
-        $this->data = wp_parse_args($data, get_plugin_data($file));
-        $this->plugin = plugin_basename($file);
-        $this->transientName = Application::PREFIX.md5(Arr::get($data, 'TextDomain'));
+        $this->apiUrl = trailingslashit( apply_filters( 'site-reviews/addon/api-url', $apiUrl ) );
+        $this->data = wp_parse_args( $data, get_plugin_data( $file ) );
+        $this->plugin = plugin_basename( $file );
+        $this->transientName = Application::PREFIX.md5( Arr::get( $data, 'TextDomain' ) );
     }
 
     /**
      * @return object
      */
-    public function activateLicense(array $data = [])
+    public function activateLicense( array $data = [] )
     {
-        return $this->request('activate_license', $data);
+        return $this->request( 'activate_license', $data );
     }
 
     /**
      * @return object
      */
-    public function checkLicense(array $data = [])
+    public function checkLicense( array $data = [] )
     {
-        $response = $this->request('check_license', $data);
-        if ('valid' === Arr::get($response, 'license')) {
-            $this->getPluginUpdate(true);
+        $response = $this->request( 'check_license', $data );
+        if( 'valid' === Arr::get( $response, 'license' ) ) {
+            $this->getPluginUpdate( true );
         }
         return $response;
     }
@@ -63,9 +63,9 @@ 
     /**
      * @return object
      */
-    public function deactivateLicense(array $data = [])
+    public function deactivateLicense( array $data = [] )
     {
-        return $this->request('deactivate_license', $data);
+        return $this->request( 'deactivate_license', $data );
     }
 
     /**
@@ -74,14 +74,14 @@ 
      * @param object $args
      * @return mixed
      */
-    public function filterPluginUpdateDetails($result, $action, $args)
+    public function filterPluginUpdateDetails( $result, $action, $args )
     {
-        if ('plugin_information' != $action
-            || Arr::get($this->data, 'TextDomain') != Arr::get($args, 'slug')) {
+        if( 'plugin_information' != $action
+            || Arr::get( $this->data, 'TextDomain' ) != Arr::get( $args, 'slug' ) ) {
             return $result;
         }
-        if ($updateInfo = $this->getPluginUpdate()) {
-            return $this->modifyUpdateDetails($updateInfo);
+        if( $updateInfo = $this->getPluginUpdate() ) {
+            return $this->modifyUpdateDetails( $updateInfo );
         }
         return $result;
     }
@@ -90,10 +90,10 @@ 
      * @param object $transient
      * @return object
      */
-    public function filterPluginUpdates($transient)
+    public function filterPluginUpdates( $transient )
     {
-        if ($updateInfo = $this->getPluginUpdate()) {
-            return $this->modifyPluginUpdates($transient, $updateInfo);
+        if( $updateInfo = $this->getPluginUpdate() ) {
+            return $this->modifyPluginUpdates( $transient, $updateInfo );
         }
         return $transient;
     }
@@ -101,9 +101,9 @@ 
     /**
      * @return object
      */
-    public function getVersion(array $data = [])
+    public function getVersion( array $data = [] )
     {
-        return $this->request('get_version', $data);
+        return $this->request( 'get_version', $data );
     }
 
     /**
@@ -111,13 +111,13 @@ 
      */
     public function init()
     {
-        if ($this->apiUrl === trailingslashit(home_url())) {
+        if( $this->apiUrl === trailingslashit( home_url() ) ) {
             return;
         }
-        add_filter('plugins_api',                             [$this, 'filterPluginUpdateDetails'], 10, 3);
-        add_filter('pre_set_site_transient_update_plugins',   [$this, 'filterPluginUpdates'], 999);
-        add_action('load-update-core.php',                    [$this, 'onForceUpdateCheck'], 9);
-        add_action('in_plugin_update_message-'.$this->plugin, [$this, 'renderLicenseMissingLink']);
+        add_filter( 'plugins_api', [$this, 'filterPluginUpdateDetails'], 10, 3 );
+        add_filter( 'pre_set_site_transient_update_plugins', [$this, 'filterPluginUpdates'], 999 );
+        add_action( 'load-update-core.php', [$this, 'onForceUpdateCheck'], 9 );
+        add_action( 'in_plugin_update_message-'.$this->plugin, [$this, 'renderLicenseMissingLink'] );
     }
 
     /**
@@ -126,7 +126,7 @@ 
     public function isLicenseValid()
     {
         $result = $this->checkLicense();
-        return 'valid' === Arr::get($result, 'license');
+        return 'valid' === Arr::get( $result, 'license' );
     }
 
     /**
@@ -134,14 +134,14 @@ 
      */
     public function onForceUpdateCheck()
     {
-        if (!filter_input(INPUT_GET, 'force-check')) {
+        if( !filter_input( INPUT_GET, 'force-check' ) ) {
             return;
         }
-        foreach (glsr()->addons as $addon) {
+        foreach( glsr()->addons as $addon ) {
             try {
-                glsr($addon)->updater->getPluginUpdate(true);
-            } catch (\Exception $e) {
-                glsr_log()->error($e->getMessage());
+                glsr( $addon )->updater->getPluginUpdate( true );
+            } catch( \Exception $e ) {
+                glsr_log()->error( $e->getMessage() );
             }
         }
     }
@@ -151,8 +151,8 @@ 
      */
     public function renderLicenseMissingLink()
     {
-        if (!$this->isLicenseValid()) {
-            glsr()->render('partials/addons/license-missing');
+        if( !$this->isLicenseValid() ) {
+            glsr()->render( 'partials/addons/license-missing' );
         }
     }
 
@@ -161,22 +161,22 @@ 
      */
     protected function getCachedVersion()
     {
-        return get_transient($this->transientName);
+        return get_transient( $this->transientName );
     }
 
     /**
      * @param bool $force
      * @return false|object
      */
-    protected function getPluginUpdate($force = false)
+    protected function getPluginUpdate( $force = false )
     {
         $version = $this->getCachedVersion();
-        if (false === $version || $force) {
+        if( false === $version || $force ) {
             $version = $this->getVersion();
-            $this->setCachedVersion($version);
+            $this->setCachedVersion( $version );
         }
-        if (isset($version->error)) {
-            glsr_log()->error($version->error);
+        if( isset($version->error) ) {
+            glsr_log()->error( $version->error );
             return false;
         }
         return $version;
@@ -187,15 +187,15 @@ 
      * @param object $updateInfo
      * @return object
      */
-    protected function modifyPluginUpdates($transient, $updateInfo)
+    protected function modifyPluginUpdates( $transient, $updateInfo )
     {
-        $updateInfo->id = Application::ID.'/'.Arr::get($this->data, 'TextDomain');
+        $updateInfo->id = Application::ID.'/'.Arr::get( $this->data, 'TextDomain' );
         $updateInfo->plugin = $this->plugin;
-        $updateInfo->requires_php = Arr::get($this->data, 'RequiresPHP');
-        $updateInfo->tested = Arr::get($this->data, 'testedTo');
-        $transient->checked[$this->plugin] = Arr::get($this->data, 'Version');
+        $updateInfo->requires_php = Arr::get( $this->data, 'RequiresPHP' );
+        $updateInfo->tested = Arr::get( $this->data, 'testedTo' );
+        $transient->checked[$this->plugin] = Arr::get( $this->data, 'Version' );
         $transient->last_checked = time();
-        if (Helper::isGreaterThan($updateInfo->new_version, Arr::get($this->data, 'Version'))) {
+        if( Helper::isGreaterThan( $updateInfo->new_version, Arr::get( $this->data, 'Version' ) ) ) {
             unset($transient->no_update[$this->plugin]);
             $updateInfo->update = true;
             $transient->response[$this->plugin] = $updateInfo;
@@ -210,13 +210,13 @@ 
      * @param object $updateInfo
      * @return object
      */
-    protected function modifyUpdateDetails($updateInfo)
+    protected function modifyUpdateDetails( $updateInfo )
     {
-        $updateInfo->author = Arr::get($this->data, 'Author');
-        $updateInfo->author_profile = Arr::get($this->data, 'AuthorURI');
-        $updateInfo->requires = Arr::get($this->data, 'RequiresWP');
-        $updateInfo->requires_php = Arr::get($this->data, 'RequiresPHP');
-        $updateInfo->tested = Arr::get($this->data, 'testedTo');
+        $updateInfo->author = Arr::get( $this->data, 'Author' );
+        $updateInfo->author_profile = Arr::get( $this->data, 'AuthorURI' );
+        $updateInfo->requires = Arr::get( $this->data, 'RequiresWP' );
+        $updateInfo->requires_php = Arr::get( $this->data, 'RequiresPHP' );
+        $updateInfo->tested = Arr::get( $this->data, 'testedTo' );
         $updateInfo->version = $updateInfo->new_version;
         return $updateInfo;
     }
@@ -225,49 +225,49 @@ 
      * @param \WP_Error|array $response
      * @return object
      */
-    protected function normalizeResponse($response)
+    protected function normalizeResponse( $response )
     {
-        $body = wp_remote_retrieve_body($response);
-        if ($data = json_decode($body)) {
-            $data = array_map('maybe_unserialize', (array) $data);
-            return (object) $data;
+        $body = wp_remote_retrieve_body( $response );
+        if( $data = json_decode( $body ) ) {
+            $data = array_map( 'maybe_unserialize', (array)$data );
+            return (object)$data;
         }
-        $error = is_wp_error($response)
+        $error = is_wp_error( $response )
             ? $response->get_error_message()
-            : 'Update server not responding ('.Arr::get($this->data, 'TextDomain').')';
-        return (object) ['error' => $error];
+            : 'Update server not responding ('.Arr::get( $this->data, 'TextDomain' ).')';
+        return (object)['error' => $error];
     }
 
     /**
      * @param string $action activate_license|check_license|deactivate_license|get_version
      * @return object
      */
-    protected function request($action, array $data = [])
+    protected function request( $action, array $data = [] )
     {
-        $data = wp_parse_args($data, $this->data);
-        $response = wp_remote_post($this->apiUrl, [
+        $data = wp_parse_args( $data, $this->data );
+        $response = wp_remote_post( $this->apiUrl, [
             'body' => [
                 'edd_action' => $action,
-                'item_id' => Arr::get($data, 'item_id'),
-                'item_name' => Arr::get($data, 'Name'),
-                'license' => Arr::get($data, 'license'),
-                'slug' => Arr::get($data, 'TextDomain'),
+                'item_id' => Arr::get( $data, 'item_id' ),
+                'item_name' => Arr::get( $data, 'Name' ),
+                'license' => Arr::get( $data, 'license' ),
+                'slug' => Arr::get( $data, 'TextDomain' ),
                 'url' => home_url(),
             ],
-            'sslverify' => apply_filters('site-reviews/sslverify/post', false),
+            'sslverify' => apply_filters( 'site-reviews/sslverify/post', false ),
             'timeout' => 15,
-        ]);
-        return $this->normalizeResponse($response);
+        ] );
+        return $this->normalizeResponse( $response );
     }
 
     /**
      * @param object $version
      * @return void
      */
-    protected function setCachedVersion($version)
+    protected function setCachedVersion( $version )
     {
-        if (!isset($version->error)) {
-            set_transient($this->transientName, $version, 3 * HOUR_IN_SECONDS);
+        if( !isset($version->error) ) {
+            set_transient( $this->transientName, $version, 3 * HOUR_IN_SECONDS );
         }
     }
 }

plugin/Modules/Migrations/Migrate_3_0_0.php

Spacing +18 added lines, -18 removed lines

@@ -65,11 +65,11 @@ 
     {
         $this->newSettings = $this->getNewSettings();
         $this->oldSettings = $this->getOldSettings();
-        if (empty($this->oldSettings) || empty($this->newSettings)) {
+        if( empty($this->oldSettings) || empty($this->newSettings) ) {
             return;
         }
-        foreach (static::MAPPED_SETTINGS as $old => $new) {
-            if (empty($this->oldSettings[$old])) {
+        foreach( static::MAPPED_SETTINGS as $old => $new ) {
+            if( empty($this->oldSettings[$old]) ) {
                 continue;
             }
             $this->newSettings[$new] = $this->oldSettings[$old];
@@ -77,12 +77,12 @@ 
         $this->migrateNotificationSettings();
         $this->migrateRecaptchaSettings();
         $this->migrateRequiredSettings();
-        $oldSettings = Arr::convertDotNotationArray($this->oldSettings);
-        $newSettings = Arr::convertDotNotationArray($this->newSettings);
-        if (isset($oldSettings['settings']['strings']) && is_array($oldSettings['settings']['strings'])) {
+        $oldSettings = Arr::convertDotNotationArray( $this->oldSettings );
+        $newSettings = Arr::convertDotNotationArray( $this->newSettings );
+        if( isset($oldSettings['settings']['strings']) && is_array( $oldSettings['settings']['strings'] ) ) {
             $newSettings['settings']['strings'] = $oldSettings['settings']['strings'];
         }
-        update_option(OptionManager::databaseKey(3), $newSettings);
+        update_option( OptionManager::databaseKey( 3 ), $newSettings );
     }
 
     /**
@@ -98,7 +98,7 @@ 
      */
     protected function getNewSettings()
     {
-        return  Arr::flattenArray(Arr::consolidateArray(OptionManager::databaseKey(3)));
+        return  Arr::flattenArray( Arr::consolidateArray( OptionManager::databaseKey( 3 ) ) );
     }
 
     /**
@@ -106,11 +106,11 @@ 
      */
     protected function getOldSettings()
     {
-        $defaults = array_fill_keys(array_keys(static::MAPPED_SETTINGS), '');
-        $settings = Arr::consolidateArray(get_option(OptionManager::databaseKey(2)));
-        $settings = Arr::flattenArray($settings);
+        $defaults = array_fill_keys( array_keys( static::MAPPED_SETTINGS ), '' );
+        $settings = Arr::consolidateArray( get_option( OptionManager::databaseKey( 2 ) ) );
+        $settings = Arr::flattenArray( $settings );
         return !empty($settings)
-            ? wp_parse_args($settings, $defaults)
+            ? wp_parse_args( $settings, $defaults )
             : [];
     }
 
@@ -125,8 +125,8 @@ 
             'webhook' => 'slack',
         ];
         $this->newSettings['settings.general.notifications'] = [];
-        foreach ($notifications as $old => $new) {
-            if ($this->oldSettings['settings.general.notification'] != $old) {
+        foreach( $notifications as $old => $new ) {
+            if( $this->oldSettings['settings.general.notification'] != $old ) {
                 continue;
             }
             $this->newSettings['settings.general.notifications'][] = $new;
@@ -143,11 +143,11 @@ 
             'SecretKey' => $this->oldSettings['settings.reviews-form.recaptcha.secret'],
             'SiteKey' => $this->oldSettings['settings.reviews-form.recaptcha.key'],
         ];
-        if (in_array($this->oldSettings['settings.reviews-form.recaptcha.integration'], ['custom', 'invisible-recaptcha'])) {
+        if( in_array( $this->oldSettings['settings.reviews-form.recaptcha.integration'], ['custom', 'invisible-recaptcha'] ) ) {
             $this->newSettings['settings.submissions.recaptcha.integration'] = 'all';
         }
-        if ('invisible-recaptcha' == $this->oldSettings['settings.reviews-form.recaptcha.integration']) {
-            $recaptcha = wp_parse_args((array) get_site_option('ic-settings', [], false), $recaptcha);
+        if( 'invisible-recaptcha' == $this->oldSettings['settings.reviews-form.recaptcha.integration'] ) {
+            $recaptcha = wp_parse_args( (array)get_site_option( 'ic-settings', [], false ), $recaptcha );
         }
         $this->newSettings['settings.submissions.recaptcha.key'] = $recaptcha['SiteKey'];
         $this->newSettings['settings.submissions.recaptcha.secret'] = $recaptcha['SecretKey'];
@@ -159,7 +159,7 @@ 
      */
     protected function migrateRequiredSettings()
     {
-        $this->newSettings['settings.submissions.required'] = array_filter((array) $this->oldSettings['settings.reviews-form.required']);
+        $this->newSettings['settings.submissions.required'] = array_filter( (array)$this->oldSettings['settings.reviews-form.required'] );
         $this->newSettings['settings.submissions.required'][] = 'rating';
         $this->newSettings['settings.submissions.required'][] = 'terms';
     }

plugin/Modules/Multilingual/Wpml.php

Spacing +19 added lines, -19 removed lines

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

plugin/Modules/Multilingual/Polylang.php

Spacing +19 added lines, -19 removed lines

@@ -15,37 +15,37 @@ 
     /**
      * {@inheritdoc}
      */
-    public function getPostId($postId)
+    public function getPostId( $postId )
     {
-        $postId = trim($postId);
-        if (!is_numeric($postId)) {
+        $postId = trim( $postId );
+        if( !is_numeric( $postId ) ) {
             return 0;
         }
-        if ($this->isEnabled()) {
-            $polylangPostId = pll_get_post($postId, pll_get_post_language(get_the_ID()));
+        if( $this->isEnabled() ) {
+            $polylangPostId = pll_get_post( $postId, pll_get_post_language( get_the_ID() ) );
         }
-        if (!empty($polylangPostId)) {
+        if( !empty($polylangPostId) ) {
             $postId = $polylangPostId;
         }
-        return intval($postId);
+        return intval( $postId );
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getPostIds(array $postIds)
+    public function getPostIds( array $postIds )
     {
-        if (!$this->isEnabled()) {
+        if( !$this->isEnabled() ) {
             return $postIds;
         }
         $newPostIds = [];
-        foreach (Arr::unique($postIds) as $postId) {
+        foreach( Arr::unique( $postIds ) as $postId ) {
             $newPostIds = array_merge(
                 $newPostIds,
-                array_values(pll_get_post_translations($postId))
+                array_values( pll_get_post_translations( $postId ) )
             );
         }
-        return Arr::unique($newPostIds);
+        return Arr::unique( $newPostIds );
     }
 
     /**
@@ -53,10 +53,10 @@ 
      */
     public function isActive()
     {
-        return function_exists('PLL')
-            && function_exists('pll_get_post')
-            && function_exists('pll_get_post_language')
-            && function_exists('pll_get_post_translations');
+        return function_exists( 'PLL' )
+            && function_exists( 'pll_get_post' )
+            && function_exists( 'pll_get_post_language' )
+            && function_exists( 'pll_get_post_translations' );
     }
 
     /**
@@ -65,7 +65,7 @@ 
     public function isEnabled()
     {
         return $this->isActive()
-            && 'polylang' == glsr(OptionManager::class)->get('settings.general.multilingual');
+            && 'polylang' == glsr( OptionManager::class )->get( 'settings.general.multilingual' );
     }
 
     /**
@@ -73,7 +73,7 @@ 
      */
     public function isSupported()
     {
-        return defined('POLYLANG_VERSION')
-            && Helper::isGreaterThanOrEqual(POLYLANG_VERSION, $this->supportedVersion);
+        return defined( 'POLYLANG_VERSION' )
+            && Helper::isGreaterThanOrEqual( POLYLANG_VERSION, $this->supportedVersion );
     }
 }

plugin/Helper.php

Spacing +59 added lines, -59 removed lines

@@ -13,10 +13,10 @@ 
      * @param string $path
      * @return string
      */
-    public static function buildClassName($name, $path = '')
+    public static function buildClassName( $name, $path = '' )
     {
-        $className = Str::camelCase($name);
-        $path = ltrim(str_replace(__NAMESPACE__, '', $path), '\\');
+        $className = Str::camelCase( $name );
+        $path = ltrim( str_replace( __NAMESPACE__, '', $path ), '\\' );
         return !empty($path)
             ? __NAMESPACE__.'\\'.$path.'\\'.$className
             : $className;
@@ -27,18 +27,18 @@ 
      * @param string $prefix
      * @return string
      */
-    public static function buildMethodName($name, $prefix = '')
+    public static function buildMethodName( $name, $prefix = '' )
     {
-        return lcfirst($prefix.static::buildClassName($name));
+        return lcfirst( $prefix.static::buildClassName( $name ) );
     }
 
     /**
      * @param string $name
      * @return string
      */
-    public static function buildPropertyName($name)
+    public static function buildPropertyName( $name )
     {
-        return static::buildMethodName($name);
+        return static::buildMethodName( $name );
     }
 
     /**
@@ -46,11 +46,11 @@ 
      * @param mixed $value
      * @return mixed
      */
-    public static function castTo($cast = '', $value)
+    public static function castTo( $cast = '', $value )
     {
-        $method = static::buildMethodName($cast, 'castTo');
-        return !empty($cast) && method_exists(__CLASS__, $method)
-            ? static::$method($value)
+        $method = static::buildMethodName( $cast, 'castTo' );
+        return !empty($cast) && method_exists( __CLASS__, $method )
+            ? static::$method( $value )
             : $value;
     }
 
@@ -58,73 +58,73 @@ 
      * @param mixed $value
      * @return array
      */
-    public static function castToArray($value)
+    public static function castToArray( $value )
     {
-        return (array) $value;
+        return (array)$value;
     }
 
     /**
      * @param mixed $value
      * @return bool
      */
-    public static function castToBool($value)
+    public static function castToBool( $value )
     {
-        return filter_var($value, FILTER_VALIDATE_BOOLEAN);
+        return filter_var( $value, FILTER_VALIDATE_BOOLEAN );
     }
 
     /**
      * @param mixed $value
      * @return float
      */
-    public static function castToFloat($value)
+    public static function castToFloat( $value )
     {
-        return (float) filter_var($value, FILTER_VALIDATE_FLOAT, FILTER_FLAG_ALLOW_THOUSAND);
+        return (float)filter_var( $value, FILTER_VALIDATE_FLOAT, FILTER_FLAG_ALLOW_THOUSAND );
     }
 
     /**
      * @param mixed $value
      * @return int
      */
-    public static function castToInt($value)
+    public static function castToInt( $value )
     {
-        return (int) filter_var($value, FILTER_VALIDATE_INT);
+        return (int)filter_var( $value, FILTER_VALIDATE_INT );
     }
 
     /**
      * @param mixed $value
      * @return object
      */
-    public static function castToObject($value)
+    public static function castToObject( $value )
     {
-        return (object) (array) $value;
+        return (object)(array)$value;
     }
 
     /**
      * @param mixed $value
      * @return string
      */
-    public static function castToString($value)
+    public static function castToString( $value )
     {
-        if (is_object($value) && in_array('__toString', get_class_methods($value))) {
-            return (string) $value->__toString();
+        if( is_object( $value ) && in_array( '__toString', get_class_methods( $value ) ) ) {
+            return (string)$value->__toString();
         }
-        if (is_array($value) || is_object($value)) {
-            return serialize($value);
+        if( is_array( $value ) || is_object( $value ) ) {
+            return serialize( $value );
         }
-        return (string) $value;
+        return (string)$value;
     }
 
     /**
      * @param string $key
      * @return mixed
      */
-    public static function filterInput($key, array $request = [])
+    public static function filterInput( $key, array $request = [] )
     {
-        if (isset($request[$key])) {
+        if( isset($request[$key]) ) {
             return $request[$key];
         }
-        $variable = filter_input(INPUT_POST, $key);
-        if (is_null($variable) && isset($_POST[$key])) {
+        $variable = filter_input( INPUT_POST, $key );
+        if( is_null( $variable ) && isset($_POST[$key]) ) {
             $variable = $_POST[$key];
         }
         return $variable;
@@ -134,13 +134,13 @@ 
      * @param string $key
      * @return array
      */
-    public static function filterInputArray($key)
+    public static function filterInputArray( $key )
     {
-        $variable = filter_input(INPUT_POST, $key, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY);
-        if (empty($variable) && !empty($_POST[$key]) && is_array($_POST[$key])) {
+        $variable = filter_input( INPUT_POST, $key, FILTER_DEFAULT, FILTER_REQUIRE_ARRAY );
+        if( empty($variable) && !empty($_POST[$key]) && is_array( $_POST[$key] ) ) {
             $variable = $_POST[$key];
         }
-        return (array) $variable;
+        return (array)$variable;
     }
 
     /**
@@ -149,22 +149,22 @@ 
     public static function getIpAddress()
     {
         $whitelist = [];
-        $isUsingCloudflare = !empty(filter_input(INPUT_SERVER, 'CF-Connecting-IP'));
-        if (apply_filters('site-reviews/whip/whitelist/cloudflare', $isUsingCloudflare)) {
-            $cloudflareIps = glsr(Cache::class)->getCloudflareIps();
+        $isUsingCloudflare = !empty(filter_input( INPUT_SERVER, 'CF-Connecting-IP' ));
+        if( apply_filters( 'site-reviews/whip/whitelist/cloudflare', $isUsingCloudflare ) ) {
+            $cloudflareIps = glsr( Cache::class )->getCloudflareIps();
             $whitelist[Whip::CLOUDFLARE_HEADERS] = [Whip::IPV4 => $cloudflareIps['v4']];
-            if (defined('AF_INET6')) {
+            if( defined( 'AF_INET6' ) ) {
                 $whitelist[Whip::CLOUDFLARE_HEADERS][Whip::IPV6] = $cloudflareIps['v6'];
             }
         }
-        $whitelist = apply_filters('site-reviews/whip/whitelist', $whitelist);
-        $methods = apply_filters('site-reviews/whip/methods', Whip::ALL_METHODS);
-        $whip = new Whip($methods, $whitelist);
-        do_action_ref_array('site-reviews/whip', [$whip]);
-        if (false !== ($clientAddress = $whip->getValidIpAddress())) {
-            return (string) $clientAddress;
+        $whitelist = apply_filters( 'site-reviews/whip/whitelist', $whitelist );
+        $methods = apply_filters( 'site-reviews/whip/methods', Whip::ALL_METHODS );
+        $whip = new Whip( $methods, $whitelist );
+        do_action_ref_array( 'site-reviews/whip', [$whip] );
+        if( false !== ($clientAddress = $whip->getValidIpAddress()) ) {
+            return (string)$clientAddress;
         }
-        glsr_log()->error('Unable to detect IP address.');
+        glsr_log()->error( 'Unable to detect IP address.' );
         return 'unknown';
     }
 
@@ -174,12 +174,12 @@ 
      * @param string|int $max
      * @return bool
      */
-    public static function inRange($value, $min, $max)
+    public static function inRange( $value, $min, $max )
     {
-        $inRange = filter_var($value, FILTER_VALIDATE_INT, ['options' => [
-            'min_range' => intval($min),
-            'max_range' => intval($max),
-        ]]);
+        $inRange = filter_var( $value, FILTER_VALIDATE_INT, ['options' => [
+            'min_range' => intval( $min ),
+            'max_range' => intval( $max ),
+        ]] );
         return false !== $inRange;
     }
 
@@ -188,9 +188,9 @@ 
      * @param int|string $compareWithValue
      * @return bool
      */
-    public static function isGreaterThan($value, $compareWithValue)
+    public static function isGreaterThan( $value, $compareWithValue )
     {
-        return version_compare($value, $compareWithValue, '>');
+        return version_compare( $value, $compareWithValue, '>' );
     }
 
     /**
@@ -198,9 +198,9 @@ 
      * @param int|string $compareWithValue
      * @return bool
      */
-    public static function isGreaterThanOrEqual($value, $compareWithValue)
+    public static function isGreaterThanOrEqual( $value, $compareWithValue )
     {
-        return version_compare($value, $compareWithValue, '>=');
+        return version_compare( $value, $compareWithValue, '>=' );
     }
 
     /**
@@ -208,9 +208,9 @@ 
      * @param int|string $compareWithValue
      * @return bool
      */
-    public static function isLessThan($value, $compareWithValue)
+    public static function isLessThan( $value, $compareWithValue )
     {
-        return version_compare($value, $compareWithValue, '<');
+        return version_compare( $value, $compareWithValue, '<' );
     }
 
     /**
@@ -218,8 +218,8 @@ 
      * @param int|string $compareWithValue
      * @return bool
      */
-    public static function isLessThanOrEqual($value, $compareWithValue)
+    public static function isLessThanOrEqual( $value, $compareWithValue )
     {
-        return version_compare($value, $compareWithValue, '<=');
+        return version_compare( $value, $compareWithValue, '<=' );
     }
 }

plugin/Database/OptionManager.php

Spacing +39 added lines, -39 removed lines

@@ -17,19 +17,19 @@ 
     /**
      * @return string
      */
-    public static function databaseKey($version = null)
+    public static function databaseKey( $version = null )
     {
-        if (1 == $version) {
+        if( 1 == $version ) {
             return 'geminilabs_site_reviews_settings';
         }
-        if (2 == $version) {
+        if( 2 == $version ) {
             return 'geminilabs_site_reviews-v2';
         }
-        if (null === $version) {
-            $version = explode('.', glsr()->version);
-            $version = array_shift($version);
+        if( null === $version ) {
+            $version = explode( '.', glsr()->version );
+            $version = array_shift( $version );
         }
-        return Str::snakeCase(Application::ID.'-v'.intval($version));
+        return Str::snakeCase( Application::ID.'-v'.intval( $version ) );
     }
 
     /**
@@ -37,7 +37,7 @@ 
      */
     public function all()
     {
-        if (empty($this->options)) {
+        if( empty($this->options) ) {
             $this->reset();
         }
         return $this->options;
@@ -47,20 +47,20 @@ 
      * @param string $path
      * @return bool
      */
-    public function delete($path)
+    public function delete( $path )
     {
-        $keys = explode('.', $path);
-        $last = array_pop($keys);
+        $keys = explode( '.', $path );
+        $last = array_pop( $keys );
         $options = $this->all();
         $pointer = &$options;
-        foreach ($keys as $key) {
-            if (!isset($pointer[$key]) || !is_array($pointer[$key])) {
+        foreach( $keys as $key ) {
+            if( !isset($pointer[$key]) || !is_array( $pointer[$key] ) ) {
                 continue;
             }
             $pointer = &$pointer[$key];
         }
         unset($pointer[$last]);
-        return $this->set($options);
+        return $this->set( $options );
     }
 
     /**
@@ -69,19 +69,19 @@ 
      * @param string $cast
      * @return mixed
      */
-    public function get($path = '', $fallback = '', $cast = '')
+    public function get( $path = '', $fallback = '', $cast = '' )
     {
-        $result = Arr::get($this->all(), $path, $fallback);
-        return Helper::castTo($cast, $result);
+        $result = Arr::get( $this->all(), $path, $fallback );
+        return Helper::castTo( $cast, $result );
     }
 
     /**
      * @param string $path
      * @return bool
      */
-    public function getBool($path)
+    public function getBool( $path )
     {
-        return Helper::castToBool($this->get($path));
+        return Helper::castToBool( $this->get( $path ) );
     }
 
     /**
@@ -90,13 +90,13 @@ 
      * @param string $cast
      * @return mixed
      */
-    public function getWP($path, $fallback = '', $cast = '')
+    public function getWP( $path, $fallback = '', $cast = '' )
     {
-        $option = get_option($path, $fallback);
-        if (empty($option)) {
+        $option = get_option( $path, $fallback );
+        if( empty($option) ) {
             $option = $fallback;
         }
-        return Helper::castTo($cast, $option);
+        return Helper::castTo( $cast, $option );
     }
 
     /**
@@ -104,25 +104,25 @@ 
      */
     public function json()
     {
-        return json_encode($this->all());
+        return json_encode( $this->all() );
     }
 
     /**
      * @return array
      */
-    public function normalize(array $options = [])
+    public function normalize( array $options = [] )
     {
         $options = wp_parse_args(
-            Arr::flattenArray($options),
-            glsr(DefaultsManager::class)->defaults()
+            Arr::flattenArray( $options ),
+            glsr( DefaultsManager::class )->defaults()
         );
-        array_walk($options, function (&$value) {
-            if (!is_string($value)) {
+        array_walk( $options, function( &$value ) {
+            if( !is_string( $value ) ) {
                 return;
             }
-            $value = wp_kses($value, wp_kses_allowed_html('post'));
+            $value = wp_kses( $value, wp_kses_allowed_html( 'post' ) );
         });
-        return Arr::convertDotNotationArray($options);
+        return Arr::convertDotNotationArray( $options );
     }
 
     /**
@@ -130,7 +130,7 @@ 
      */
     public function isRecaptchaEnabled()
     {
-        $integration = $this->get('settings.submissions.recaptcha.integration');
+        $integration = $this->get( 'settings.submissions.recaptcha.integration' );
         return 'all' == $integration || ('guest' == $integration && !is_user_logged_in());
     }
 
@@ -139,9 +139,9 @@ 
      */
     public function reset()
     {
-        $options = $this->getWP(static::databaseKey(), []);
-        if (!is_array($options) || empty($options)) {
-            delete_option(static::databaseKey());
+        $options = $this->getWP( static::databaseKey(), [] );
+        if( !is_array( $options ) || empty($options) ) {
+            delete_option( static::databaseKey() );
             $options = glsr()->defaults ?: [];
         }
         $this->options = $options;
@@ -152,12 +152,12 @@ 
      * @param mixed $value
      * @return bool
      */
-    public function set($pathOrOptions, $value = '')
+    public function set( $pathOrOptions, $value = '' )
     {
-        if (is_string($pathOrOptions)) {
-            $pathOrOptions = Arr::set($this->all(), $pathOrOptions, $value);
+        if( is_string( $pathOrOptions ) ) {
+            $pathOrOptions = Arr::set( $this->all(), $pathOrOptions, $value );
         }
-        if ($result = update_option(static::databaseKey(), (array) $pathOrOptions)) {
+        if( $result = update_option( static::databaseKey(), (array)$pathOrOptions ) ) {
             $this->reset();
         }
         return $result;

helpers.php

Spacing +46 added lines, -46 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-defined('WPINC') || die;
+defined( 'WPINC' ) || die;
 
 /*
  * Alternate method of using the functions without having to use `function_exists()`
@@ -8,7 +8,7 @@ 
  * @param mixed ...
  * @return mixed
  */
-add_filter('plugins_loaded', function () {
+add_filter( 'plugins_loaded', function() {
     $hooks = array(
         'glsr_calculate_ratings' => 1,
         'glsr_create_review' => 2,
@@ -21,23 +21,23 @@ 
         'glsr_log' => 3,
         'glsr_star_rating' => 2,
     );
-    foreach ($hooks as $function => $acceptedArgs) {
-        add_filter($function, function () use ($function) {
+    foreach( $hooks as $function => $acceptedArgs ) {
+        add_filter( $function, function() use ($function) {
             $args = func_get_args();
-            array_shift($args); // remove the fallback value
-            return call_user_func_array($function, $args);
-        }, 10, $acceptedArgs);
+            array_shift( $args ); // remove the fallback value
+            return call_user_func_array( $function, $args );
+        }, 10, $acceptedArgs );
     }
 });
 
 /**
  * @return mixed
  */
-function glsr($alias = null)
+function glsr( $alias = null )
 {
     $app = \GeminiLabs\SiteReviews\Application::load();
-    return !is_null($alias)
-        ? $app->make($alias)
+    return !is_null( $alias )
+        ? $app->make( $alias )
         : $app;
 }
 
@@ -46,12 +46,12 @@ 
  * @param $column string
  * @return array
  */
-function glsr_array_column(array $array, $column)
+function glsr_array_column( array $array, $column )
 {
     $result = array();
-    foreach ($array as $subarray) {
-        $subarray = (array) $subarray;
-        if (!isset($subarray[$column])) {
+    foreach( $array as $subarray ) {
+        $subarray = (array)$subarray;
+        if( !isset($subarray[$column]) ) {
             continue;
         }
         $result[] = $subarray[$column];
@@ -64,19 +64,19 @@ 
  */
 function glsr_calculate_ratings()
 {
-    glsr('Database\CountsManager')->updateAll();
-    glsr_log()->notice(__('Recalculated rating counts.', 'site-reviews'));
+    glsr( 'Database\CountsManager' )->updateAll();
+    glsr_log()->notice( __( 'Recalculated rating counts.', 'site-reviews' ) );
 }
 
 /**
  * @return \GeminiLabs\SiteReviews\Review|false
  */
-function glsr_create_review($reviewValues = array())
+function glsr_create_review( $reviewValues = array() )
 {
     $review = new \GeminiLabs\SiteReviews\Commands\CreateReview(
-        \GeminiLabs\SiteReviews\Helpers\Arr::consolidateArray($reviewValues)
+        \GeminiLabs\SiteReviews\Helpers\Arr::consolidateArray( $reviewValues )
     );
-    return glsr('Database\ReviewManager')->create($review);
+    return glsr( 'Database\ReviewManager' )->create( $review );
 }
 
 /**
@@ -84,11 +84,11 @@ 
  */
 function glsr_current_screen()
 {
-    if (function_exists('get_current_screen')) {
+    if( function_exists( 'get_current_screen' ) ) {
         $screen = get_current_screen();
     }
     return empty($screen)
-        ? (object) array_fill_keys(['base', 'id', 'post_type'], null)
+        ? (object)array_fill_keys( ['base', 'id', 'post_type'], null )
         : $screen;
 }
 
@@ -96,15 +96,15 @@ 
  * @param mixed ...$vars
  * @return void
  */
-function glsr_debug(...$vars)
+function glsr_debug( ...$vars )
 {
-    if (1 == count($vars)) {
-        $value = htmlspecialchars(print_r($vars[0], true), ENT_QUOTES, 'UTF-8');
-        printf('<div class="glsr-debug"><pre>%s</pre></div>', $value);
+    if( 1 == count( $vars ) ) {
+        $value = htmlspecialchars( print_r( $vars[0], true ), ENT_QUOTES, 'UTF-8' );
+        printf( '<div class="glsr-debug"><pre>%s</pre></div>', $value );
     } else {
         echo '<div class="glsr-debug-group">';
-        foreach ($vars as $var) {
-            glsr_debug($var);
+        foreach( $vars as $var ) {
+            glsr_debug( $var );
         }
         echo '</div>';
     }
@@ -116,9 +116,9 @@ 
  * @param mixed $fallback
  * @return mixed
  */
-function glsr_get($array, $path = '', $fallback = '')
+function glsr_get( $array, $path = '', $fallback = '' )
 {
-    return \GeminiLabs\SiteReviews\Helpers\Arr::get($array, $path, $fallback);
+    return \GeminiLabs\SiteReviews\Helpers\Arr::get( $array, $path, $fallback );
 }
 
 /**
@@ -127,10 +127,10 @@ 
  * @param string $cast
  * @return string|array
  */
-function glsr_get_option($path = '', $fallback = '', $cast = '')
+function glsr_get_option( $path = '', $fallback = '', $cast = '' )
 {
-    return is_string($path)
-        ? glsr('Database\OptionManager')->get(\GeminiLabs\SiteReviews\Helpers\Str::prefix('settings.', $path), $fallback, $cast)
+    return is_string( $path )
+        ? glsr( 'Database\OptionManager' )->get( \GeminiLabs\SiteReviews\Helpers\Str::prefix( 'settings.', $path ), $fallback, $cast )
         : $fallback;
 }
 
@@ -139,30 +139,30 @@ 
  */
 function glsr_get_options()
 {
-    return glsr('Database\OptionManager')->get('settings');
+    return glsr( 'Database\OptionManager' )->get( 'settings' );
 }
 
 /**
  * @param \WP_Post|int $post
  * @return \GeminiLabs\SiteReviews\Review
  */
-function glsr_get_review($post)
+function glsr_get_review( $post )
 {
-    if (is_numeric($post)) {
-        $post = get_post($post);
+    if( is_numeric( $post ) ) {
+        $post = get_post( $post );
     }
-    if (!($post instanceof WP_Post)) {
-        $post = new WP_Post((object) []);
+    if( !($post instanceof WP_Post) ) {
+        $post = new WP_Post( (object)[] );
     }
-    return glsr('Database\ReviewManager')->single($post);
+    return glsr( 'Database\ReviewManager' )->single( $post );
 }
 
 /**
  * @return array
  */
-function glsr_get_reviews($args = array())
+function glsr_get_reviews( $args = array() )
 {
-    return glsr('Database\ReviewManager')->get(\GeminiLabs\SiteReviews\Helpers\Arr::consolidateArray($args));
+    return glsr( 'Database\ReviewManager' )->get( \GeminiLabs\SiteReviews\Helpers\Arr::consolidateArray( $args ) );
 }
 
 /**
@@ -171,9 +171,9 @@ 
 function glsr_log()
 {
     $args = func_get_args();
-    $console = glsr('Modules\Console');
-    if ($value = \GeminiLabs\SiteReviews\Helpers\Arr::get($args, '0')) {
-        return $console->debug($value, \GeminiLabs\SiteReviews\Helpers\Arr::get($args, '1', []));
+    $console = glsr( 'Modules\Console' );
+    if( $value = \GeminiLabs\SiteReviews\Helpers\Arr::get( $args, '0' ) ) {
+        return $console->debug( $value, \GeminiLabs\SiteReviews\Helpers\Arr::get( $args, '1', [] ) );
     }
     return $console;
 }
@@ -181,7 +181,7 @@ 
 /**
  * @return string
  */
-function glsr_star_rating($rating)
+function glsr_star_rating( $rating )
 {
-    return glsr('Modules\Html\Partial')->build('star-rating', ['rating' => $rating]);
+    return glsr( 'Modules\Html\Partial' )->build( 'star-rating', ['rating' => $rating] );
 }

plugin/Commands/CreateReview.php

Spacing +37 added lines, -37 removed lines

@@ -27,37 +27,37 @@ 
     public $title;
     public $url;
 
-    public function __construct($input)
+    public function __construct( $input )
     {
         $this->request = $input;
         $this->ajax_request = isset($input['_ajax_request']);
-        $this->assigned_to = $this->getNumeric('assign_to');
-        $this->author = sanitize_text_field($this->getUser('name'));
+        $this->assigned_to = $this->getNumeric( 'assign_to' );
+        $this->author = sanitize_text_field( $this->getUser( 'name' ) );
         $this->avatar = $this->getAvatar();
         $this->blacklisted = isset($input['blacklisted']);
         $this->category = $this->getCategory();
-        $this->content = sanitize_textarea_field($this->get('content'));
+        $this->content = sanitize_textarea_field( $this->get( 'content' ) );
         $this->custom = $this->getCustom();
-        $this->date = $this->getDate('date');
-        $this->email = sanitize_email($this->getUser('email'));
-        $this->form_id = sanitize_key($this->get('form_id'));
-        $this->ip_address = $this->get('ip_address');
-        $this->post_id = intval($this->get('_post_id'));
-        $this->rating = intval($this->get('rating'));
-        $this->referer = sanitize_text_field($this->get('_referer'));
-        $this->response = sanitize_textarea_field($this->get('response'));
+        $this->date = $this->getDate( 'date' );
+        $this->email = sanitize_email( $this->getUser( 'email' ) );
+        $this->form_id = sanitize_key( $this->get( 'form_id' ) );
+        $this->ip_address = $this->get( 'ip_address' );
+        $this->post_id = intval( $this->get( '_post_id' ) );
+        $this->rating = intval( $this->get( 'rating' ) );
+        $this->referer = sanitize_text_field( $this->get( '_referer' ) );
+        $this->response = sanitize_textarea_field( $this->get( 'response' ) );
         $this->terms = !empty($input['terms']);
-        $this->title = sanitize_text_field($this->get('title'));
-        $this->url = esc_url_raw(sanitize_text_field($this->get('url')));
+        $this->title = sanitize_text_field( $this->get( 'title' ) );
+        $this->url = esc_url_raw( sanitize_text_field( $this->get( 'url' ) ) );
     }
 
     /**
      * @param string $key
      * @return string
      */
-    protected function get($key)
+    protected function get( $key )
     {
-        return (string) Arr::get($this->request, $key);
+        return (string)Arr::get( $this->request, $key );
     }
 
     /**
@@ -65,9 +65,9 @@ 
      */
     protected function getAvatar()
     {
-        $avatar = $this->get('avatar');
-        return !filter_var($avatar, FILTER_VALIDATE_URL, FILTER_FLAG_PATH_REQUIRED)
-            ? (string) get_avatar_url($this->get('email'))
+        $avatar = $this->get( 'avatar' );
+        return !filter_var( $avatar, FILTER_VALIDATE_URL, FILTER_FLAG_PATH_REQUIRED )
+            ? (string)get_avatar_url( $this->get( 'email' ) )
             : $avatar;
     }
 
@@ -76,8 +76,8 @@ 
      */
     protected function getCategory()
     {
-        $categories = Arr::convertStringToArray($this->get('category'));
-        return sanitize_key(Arr::get($categories, 0));
+        $categories = Arr::convertStringToArray( $this->get( 'category' ) );
+        return sanitize_key( Arr::get( $categories, 0 ) );
     }
 
     /**
@@ -90,14 +90,14 @@ 
             '_referer', 'assign_to', 'category', 'content', 'date', 'email', 'excluded', 'form_id',
             'gotcha', 'ip_address', 'name', 'rating', 'response', 'terms', 'title', 'url',
         ];
-        $unset = apply_filters('site-reviews/create/unset-keys-from-custom', $unset);
+        $unset = apply_filters( 'site-reviews/create/unset-keys-from-custom', $unset );
         $custom = $this->request;
-        foreach ($unset as $key) {
+        foreach( $unset as $key ) {
             unset($custom[$key]);
         }
-        foreach ($custom as $key => $value) {
-            if (is_string($value)) {
-                $custom[$key] = sanitize_text_field($value);
+        foreach( $custom as $key => $value ) {
+            if( is_string( $value ) ) {
+                $custom[$key] = sanitize_text_field( $value );
             }
         }
         return $custom;
@@ -107,29 +107,29 @@ 
      * @param string $key
      * @return string
      */
-    protected function getDate($key)
+    protected function getDate( $key )
     {
-        $date = strtotime($this->get($key));
-        if (false === $date) {
+        $date = strtotime( $this->get( $key ) );
+        if( false === $date ) {
             $date = time();
         }
-        return get_date_from_gmt(gmdate('Y-m-d H:i:s', $date));
+        return get_date_from_gmt( gmdate( 'Y-m-d H:i:s', $date ) );
     }
 
     /**
      * @param string $key
      * @return string
      */
-    protected function getUser($key)
+    protected function getUser( $key )
     {
-        $value = $this->get($key);
-        if (empty($value)) {
+        $value = $this->get( $key );
+        if( empty($value) ) {
             $user = wp_get_current_user();
             $userValues = [
                 'email' => 'user_email',
                 'name' => 'display_name',
             ];
-            if ($user->exists() && array_key_exists($key, $userValues)) {
+            if( $user->exists() && array_key_exists( $key, $userValues ) ) {
                 return $user->{$userValues[$key]};
             }
         }
@@ -140,10 +140,10 @@ 
      * @param string $key
      * @return string
      */
-    protected function getNumeric($key)
+    protected function getNumeric( $key )
     {
-        $value = $this->get($key);
-        return is_numeric($value)
+        $value = $this->get( $key );
+        return is_numeric( $value )
             ? $value
             : '';
     }