pryley / site-reviews

plugin/Modules/Translator.php

Spacing +25 added lines, -25 removed lines

@@ -11,31 +11,31 @@ 
      * @param string $domain
      * @return string
      */
-    public function translate($original, $domain, array $args)
+    public function translate( $original, $domain, array $args )
     {
-        $domains = apply_filters('site-reviews/translator/domains', [Application::ID]);
-        if (!in_array($domain, $domains)) {
+        $domains = apply_filters( 'site-reviews/translator/domains', [Application::ID] );
+        if( !in_array( $domain, $domains ) ) {
             return $original;
         }
-        $args = $this->normalizeTranslationArgs($args);
-        $strings = $this->getTranslationStrings($args['single'], $args['plural']);
-        if (empty($strings)) {
+        $args = $this->normalizeTranslationArgs( $args );
+        $strings = $this->getTranslationStrings( $args['single'], $args['plural'] );
+        if( empty($strings) ) {
             return $original;
         }
-        $string = current($strings);
+        $string = current( $strings );
         return 'plural' == $string['type']
-            ? $this->translatePlural($domain, $string, $args)
-            : $this->translateSingle($domain, $string, $args);
+            ? $this->translatePlural( $domain, $string, $args )
+            : $this->translateSingle( $domain, $string, $args );
     }
 
     /**
      * Used when search/replacing a default text-domain translation
      * @return string
      */
-    public function getTranslation(array $args)
+    public function getTranslation( array $args )
     {
-        $args = $this->normalizeTranslationArgs($args);
-        return get_translations_for_domain(Application::ID)->translate_plural($args['single'], $args['plural'], $args['number']);
+        $args = $this->normalizeTranslationArgs( $args );
+        return get_translations_for_domain( Application::ID )->translate_plural( $args['single'], $args['plural'], $args['number'] );
     }
 
     /**
@@ -43,18 +43,18 @@ 
      * @param string $plural
      * @return array
      */
-    protected function getTranslationStrings($single, $plural)
+    protected function getTranslationStrings( $single, $plural )
     {
-        return array_filter(glsr(Translation::class)->translations(), function ($string) use ($single, $plural) {
-            return $string['s1'] == html_entity_decode($single, ENT_COMPAT, 'UTF-8')
-                && $string['p1'] == html_entity_decode($plural, ENT_COMPAT, 'UTF-8');
+        return array_filter( glsr( Translation::class )->translations(), function( $string ) use ($single, $plural) {
+            return $string['s1'] == html_entity_decode( $single, ENT_COMPAT, 'UTF-8' )
+                && $string['p1'] == html_entity_decode( $plural, ENT_COMPAT, 'UTF-8' );
         });
     }
 
     /**
      * @return array
      */
-    protected function normalizeTranslationArgs(array $args)
+    protected function normalizeTranslationArgs( array $args )
     {
         $defaults = [
             'context' => '',
@@ -62,22 +62,22 @@ 
             'plural' => '',
             'single' => '',
         ];
-        return shortcode_atts($defaults, $args);
+        return shortcode_atts( $defaults, $args );
     }
 
     /**
      * @param string $domain
      * @return string
      */
-    protected function translatePlural($domain, array $string, array $args)
+    protected function translatePlural( $domain, array $string, array $args )
     {
-        if (!empty($string['s2'])) {
+        if( !empty($string['s2']) ) {
             $args['single'] = $string['s2'];
         }
-        if (!empty($string['p2'])) {
+        if( !empty($string['p2']) ) {
             $args['plural'] = $string['p2'];
         }
-        return get_translations_for_domain($domain)->translate_plural(
+        return get_translations_for_domain( $domain )->translate_plural(
             $args['single'],
             $args['plural'],
             $args['number'],
@@ -89,12 +89,12 @@ 
      * @param string $domain
      * @return string
      */
-    protected function translateSingle($domain, array $string, array $args)
+    protected function translateSingle( $domain, array $string, array $args )
     {
-        if (!empty($string['s2'])) {
+        if( !empty($string['s2']) ) {
             $args['single'] = $string['s2'];
         }
-        return get_translations_for_domain($domain)->translate(
+        return get_translations_for_domain( $domain )->translate(
             $args['single'],
             $args['context']
         );

plugin/Controllers/PublicController.php

Spacing +18 added lines, -18 removed lines

@@ -28,14 +28,14 @@ 
      * @return string
      * @filter script_loader_tag
      */
-    public function filterEnqueuedScripts($tag, $handle)
+    public function filterEnqueuedScripts( $tag, $handle )
     {
         $scripts = [Application::ID.'/google-recaptcha'];
-        if (in_array($handle, apply_filters('site-reviews/async-scripts', $scripts))) {
-            $tag = str_replace(' src=', ' async src=', $tag);
+        if( in_array( $handle, apply_filters( 'site-reviews/async-scripts', $scripts ) ) ) {
+            $tag = str_replace( ' src=', ' async src=', $tag );
         }
-        if (in_array($handle, apply_filters('site-reviews/defer-scripts', $scripts))) {
-            $tag = str_replace(' src=', ' defer src=', $tag);
+        if( in_array( $handle, apply_filters( 'site-reviews/defer-scripts', $scripts ) ) ) {
+            $tag = str_replace( ' src=', ' defer src=', $tag );
         }
         return $tag;
     }
@@ -44,10 +44,10 @@ 
      * @return array
      * @filter site-reviews/config/forms/submission-form
      */
-    public function filterFieldOrder(array $config)
+    public function filterFieldOrder( array $config )
     {
-        $order = (array) apply_filters('site-reviews/submission-form/order', array_keys($config));
-        return array_intersect_key(array_merge(array_flip($order), $config), $config);
+        $order = (array)apply_filters( 'site-reviews/submission-form/order', array_keys( $config ) );
+        return array_intersect_key( array_merge( array_flip( $order ), $config ), $config );
     }
 
     /**
@@ -55,18 +55,18 @@ 
      * @return string
      * @filter site-reviews/render/view
      */
-    public function filterRenderView($view)
+    public function filterRenderView( $view )
     {
-        return glsr(Style::class)->filterView($view);
+        return glsr( Style::class )->filterView( $view );
     }
 
     /**
      * @return void
      * @action site-reviews/builder
      */
-    public function modifyBuilder(Builder $instance)
+    public function modifyBuilder( Builder $instance )
     {
-        call_user_func_array([glsr(Style::class), 'modifyField'], [$instance]);
+        call_user_func_array( [glsr( Style::class ), 'modifyField'], [$instance] );
     }
 
     /**
@@ -75,18 +75,18 @@ 
      */
     public function renderSchema()
     {
-        glsr(Schema::class)->render();
+        glsr( Schema::class )->render();
     }
 
     /**
      * @return CreateReview
      */
-    public function routerSubmitReview(array $request)
+    public function routerSubmitReview( array $request )
     {
-        $validated = glsr(ValidateReview::class)->validate($request);
-        $command = new CreateReview($validated->request);
-        if (empty($validated->error) && !$validated->recaptchaIsUnset) {
-            $this->execute($command);
+        $validated = glsr( ValidateReview::class )->validate( $request );
+        $command = new CreateReview( $validated->request );
+        if( empty($validated->error) && !$validated->recaptchaIsUnset ) {
+            $this->execute( $command );
         }
         return $command;
     }

plugin/Database/Cache.php

Spacing +20 added lines, -20 removed lines

@@ -11,24 +11,24 @@ 
      */
     public function getCloudflareIps()
     {
-        if (false === ($ipAddresses = get_transient(Application::ID.'_cloudflare_ips'))) {
-            $ipAddresses = array_fill_keys(['v4', 'v6'], []);
-            foreach (array_keys($ipAddresses) as $version) {
+        if( false === ($ipAddresses = get_transient( Application::ID.'_cloudflare_ips' )) ) {
+            $ipAddresses = array_fill_keys( ['v4', 'v6'], [] );
+            foreach( array_keys( $ipAddresses ) as $version ) {
                 $url = 'https://www.cloudflare.com/ips-'.$version;
-                $response = wp_remote_get($url, ['sslverify' => false]);
-                if (is_wp_error($response)) {
-                    glsr_log()->error($response->get_error_message());
+                $response = wp_remote_get( $url, ['sslverify' => false] );
+                if( is_wp_error( $response ) ) {
+                    glsr_log()->error( $response->get_error_message() );
                     continue;
                 }
-                if ('200' != ($statusCode = wp_remote_retrieve_response_code($response))) {
-                    glsr_log()->error('Unable to connect to '.$url.' ['.$statusCode.']');
+                if( '200' != ($statusCode = wp_remote_retrieve_response_code( $response )) ) {
+                    glsr_log()->error( 'Unable to connect to '.$url.' ['.$statusCode.']' );
                     continue;
                 }
                 $ipAddresses[$version] = array_filter(
-                    (array) preg_split('/\R/', wp_remote_retrieve_body($response))
+                    (array)preg_split( '/\R/', wp_remote_retrieve_body( $response ) )
                 );
             }
-            set_transient(Application::ID.'_cloudflare_ips', $ipAddresses, WEEK_IN_SECONDS);
+            set_transient( Application::ID.'_cloudflare_ips', $ipAddresses, WEEK_IN_SECONDS );
         }
         return $ipAddresses;
     }
@@ -37,16 +37,16 @@ 
      * @param string $metaKey
      * @return array
      */
-    public function getReviewCountsFor($metaKey)
+    public function getReviewCountsFor( $metaKey )
     {
-        $counts = wp_cache_get(Application::ID, $metaKey.'_count');
-        if (false === $counts) {
+        $counts = wp_cache_get( Application::ID, $metaKey.'_count' );
+        if( false === $counts ) {
             $counts = [];
-            $results = glsr(SqlQueries::class)->getReviewCountsFor($metaKey);
-            foreach ($results as $result) {
+            $results = glsr( SqlQueries::class )->getReviewCountsFor( $metaKey );
+            foreach( $results as $result ) {
                 $counts[$result->name] = $result->num_posts;
             }
-            wp_cache_set(Application::ID, $counts, $metaKey.'_count');
+            wp_cache_set( Application::ID, $counts, $metaKey.'_count' );
         }
         return $counts;
     }
@@ -56,12 +56,12 @@ 
      */
     public function getRemotePostTest()
     {
-        if (false === ($test = get_transient(Application::ID.'_remote_post_test'))) {
-            $response = wp_remote_post('https://api.wordpress.org/stats/php/1.0/');
-            $test = !is_wp_error($response) && in_array($response['response']['code'], range(200, 299))
+        if( false === ($test = get_transient( Application::ID.'_remote_post_test' )) ) {
+            $response = wp_remote_post( 'https://api.wordpress.org/stats/php/1.0/' );
+            $test = !is_wp_error( $response ) && in_array( $response['response']['code'], range( 200, 299 ) )
                 ? 'Works'
                 : 'Does not work';
-            set_transient(Application::ID.'_remote_post_test', $test, WEEK_IN_SECONDS);
+            set_transient( Application::ID.'_remote_post_test', $test, WEEK_IN_SECONDS );
         }
         return $test;
     }

plugin/Defaults/CreateReviewDefaults.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
             'pinned' => false,
             'rating' => '',
             'response' => '',
-            'review_id' => md5(time().mt_rand()),
+            'review_id' => md5( time().mt_rand() ),
             'review_type' => 'local',
             'title' => '',
             'url' => '',

plugin/Modules/Console.php

Spacing +114 added lines, -114 removed lines

@@ -10,13 +10,13 @@ 
 
 class Console
 {
-    const DEBUG = 0;      // Detailed debug information
-    const INFO = 1;       // Interesting events
-    const NOTICE = 2;     // Normal but significant events
-    const WARNING = 4;    // Exceptional occurrences that are not errors
-    const ERROR = 8;      // Runtime errors that do not require immediate action
-    const CRITICAL = 16;  // Critical conditions
-    const ALERT = 32;     // Action must be taken immediately
+    const DEBUG = 0; // Detailed debug information
+    const INFO = 1; // Interesting events
+    const NOTICE = 2; // Normal but significant events
+    const WARNING = 4; // Exceptional occurrences that are not errors
+    const ERROR = 8; // Runtime errors that do not require immediate action
+    const CRITICAL = 16; // Critical conditions
+    const ALERT = 32; // Action must be taken immediately
     const EMERGENCY = 64; // System is unusable
 
     protected $file;
@@ -25,9 +25,9 @@ 
 
     public function __construct()
     {
-        $this->file = glsr()->path('console.log');
-        $this->log = file_exists($this->file)
-            ? file_get_contents($this->file)
+        $this->file = glsr()->path( 'console.log' );
+        $this->log = file_exists( $this->file )
+            ? file_get_contents( $this->file )
             : '';
         $this->reset();
     }
@@ -47,9 +47,9 @@ 
      * @param array $context
      * @return static
      */
-    public function alert($message, array $context = [])
+    public function alert( $message, array $context = [] )
     {
-        return $this->log(static::ALERT, $message, $context);
+        return $this->log( static::ALERT, $message, $context );
     }
 
     /**
@@ -58,7 +58,7 @@ 
     public function clear()
     {
         $this->log = '';
-        file_put_contents($this->file, $this->log);
+        file_put_contents( $this->file, $this->log );
     }
 
     /**
@@ -68,9 +68,9 @@ 
      * @param array $context
      * @return static
      */
-    public function critical($message, array $context = [])
+    public function critical( $message, array $context = [] )
     {
-        return $this->log(static::CRITICAL, $message, $context);
+        return $this->log( static::CRITICAL, $message, $context );
     }
 
     /**
@@ -79,9 +79,9 @@ 
      * @param array $context
      * @return static
      */
-    public function debug($message, array $context = [])
+    public function debug( $message, array $context = [] )
     {
-        return $this->log(static::DEBUG, $message, $context);
+        return $this->log( static::DEBUG, $message, $context );
     }
 
     /**
@@ -90,9 +90,9 @@ 
      * @param array $context
      * @return static
      */
-    public function emergency($message, array $context = [])
+    public function emergency( $message, array $context = [] )
     {
-        return $this->log(static::EMERGENCY, $message, $context);
+        return $this->log( static::EMERGENCY, $message, $context );
     }
 
     /**
@@ -101,9 +101,9 @@ 
      * @param array $context
      * @return static
      */
-    public function error($message, array $context = [])
+    public function error( $message, array $context = [] )
     {
-        return $this->log(static::ERROR, $message, $context);
+        return $this->log( static::ERROR, $message, $context );
     }
 
     /**
@@ -112,7 +112,7 @@ 
     public function get()
     {
         return empty($this->log)
-            ? __('Console is empty', 'site-reviews')
+            ? __( 'Console is empty', 'site-reviews' )
             : $this->log;
     }
 
@@ -121,7 +121,7 @@ 
      */
     public function getLevel()
     {
-        return intval(apply_filters('site-reviews/console/level', static::INFO));
+        return intval( apply_filters( 'site-reviews/console/level', static::INFO ) );
     }
 
     /**
@@ -129,8 +129,8 @@ 
      */
     public function getLevels()
     {
-        $constants = (new ReflectionClass(__CLASS__))->getConstants();
-        return array_map('strtolower', array_flip($constants));
+        $constants = (new ReflectionClass( __CLASS__ ))->getConstants();
+        return array_map( 'strtolower', array_flip( $constants ) );
     }
 
     /**
@@ -139,21 +139,21 @@ 
     public function humanLevel()
     {
         $level = $this->getLevel();
-        return sprintf('%s (%d)', strtoupper(Arr::get($this->getLevels(), $level, 'unknown')), $level);
+        return sprintf( '%s (%d)', strtoupper( Arr::get( $this->getLevels(), $level, 'unknown' ) ), $level );
     }
 
     /**
      * @param string|null $valueIfEmpty
      * @return string
      */
-    public function humanSize($valueIfEmpty = null)
+    public function humanSize( $valueIfEmpty = null )
     {
         $bytes = $this->size();
-        if (empty($bytes) && is_string($valueIfEmpty)) {
+        if( empty($bytes) && is_string( $valueIfEmpty ) ) {
             return $valueIfEmpty;
         }
-        $exponent = floor(log(max($bytes, 1), 1024));
-        return round($bytes / pow(1024, $exponent), 2).' '.['bytes', 'KB', 'MB', 'GB'][$exponent];
+        $exponent = floor( log( max( $bytes, 1 ), 1024 ) );
+        return round( $bytes / pow( 1024, $exponent ), 2 ).' '.['bytes', 'KB', 'MB', 'GB'][$exponent];
     }
 
     /**
@@ -163,9 +163,9 @@ 
      * @param array $context
      * @return static
      */
-    public function info($message, array $context = [])
+    public function info( $message, array $context = [] )
     {
-        return $this->log(static::INFO, $message, $context);
+        return $this->log( static::INFO, $message, $context );
     }
 
     /**
@@ -175,19 +175,19 @@ 
      * @param string $backtraceLine
      * @return static
      */
-    public function log($level, $message, $context = [], $backtraceLine = '')
+    public function log( $level, $message, $context = [], $backtraceLine = '' )
     {
-        if (empty($backtraceLine)) {
+        if( empty($backtraceLine) ) {
             $backtraceLine = $this->getBacktraceLine();
         }
-        if ($this->canLogEntry($level, $backtraceLine)) {
-            $levelName = Arr::get($this->getLevels(), $level);
-            $context = Arr::consolidateArray($context);
-            $backtraceLine = $this->normalizeBacktraceLine($backtraceLine);
-            $message = $this->interpolate($message, $context);
-            $entry = $this->buildLogEntry($levelName, $message, $backtraceLine);
-            file_put_contents($this->file, $entry.PHP_EOL, FILE_APPEND | LOCK_EX);
-            apply_filters('console', $message, $levelName, $backtraceLine); // Show in Blackbar plugin if installed
+        if( $this->canLogEntry( $level, $backtraceLine ) ) {
+            $levelName = Arr::get( $this->getLevels(), $level );
+            $context = Arr::consolidateArray( $context );
+            $backtraceLine = $this->normalizeBacktraceLine( $backtraceLine );
+            $message = $this->interpolate( $message, $context );
+            $entry = $this->buildLogEntry( $levelName, $message, $backtraceLine );
+            file_put_contents( $this->file, $entry.PHP_EOL, FILE_APPEND | LOCK_EX );
+            apply_filters( 'console', $message, $levelName, $backtraceLine ); // Show in Blackbar plugin if installed
             $this->reset();
         }
         return $this;
@@ -198,17 +198,17 @@ 
      */
     public function logOnce()
     {
-        $once = Arr::consolidateArray(glsr()->{$this->logOnceKey});
+        $once = Arr::consolidateArray( glsr()->{$this->logOnceKey});
         $levels = $this->getLevels();
-        foreach ($once as $entry) {
-            $levelName = Arr::get($entry, 'level');
-            if (!in_array($levelName, $levels)) {
+        foreach( $once as $entry ) {
+            $levelName = Arr::get( $entry, 'level' );
+            if( !in_array( $levelName, $levels ) ) {
                 continue;
             }
-            $level = Arr::get(array_flip($levels), $levelName);
-            $message = Arr::get($entry, 'message');
-            $backtraceLine = Arr::get($entry, 'backtrace');
-            $this->log($level, $message, [], $backtraceLine);
+            $level = Arr::get( array_flip( $levels ), $levelName );
+            $message = Arr::get( $entry, 'message' );
+            $backtraceLine = Arr::get( $entry, 'backtrace' );
+            $this->log( $level, $message, [], $backtraceLine );
         }
         glsr()->{$this->logOnceKey} = [];
     }
@@ -219,9 +219,9 @@ 
      * @param array $context
      * @return static
      */
-    public function notice($message, array $context = [])
+    public function notice( $message, array $context = [] )
     {
-        return $this->log(static::NOTICE, $message, $context);
+        return $this->log( static::NOTICE, $message, $context );
     }
 
     /**
@@ -230,21 +230,21 @@ 
      * @param mixed $data
      * @return void
      */
-    public function once($levelName, $handle, $data)
+    public function once( $levelName, $handle, $data )
     {
-        $once = Arr::consolidateArray(glsr()->{$this->logOnceKey});
-        $filtered = array_filter($once, function ($entry) use ($levelName, $handle) {
-            return Arr::get($entry, 'level') == $levelName
-                && Arr::get($entry, 'handle') == $handle;
+        $once = Arr::consolidateArray( glsr()->{$this->logOnceKey});
+        $filtered = array_filter( $once, function( $entry ) use ($levelName, $handle) {
+            return Arr::get( $entry, 'level' ) == $levelName
+                && Arr::get( $entry, 'handle' ) == $handle;
         });
-        if (!empty($filtered)) {
+        if( !empty($filtered) ) {
             return;
         }
         $once[] = [
-            'backtrace' => $this->getBacktraceLineFromData($data),
+            'backtrace' => $this->getBacktraceLineFromData( $data ),
             'handle' => $handle,
             'level' => $levelName,
-            'message' => '[RECURRING] '.$this->getMessageFromData($data),
+            'message' => '[RECURRING] '.$this->getMessageFromData( $data ),
         ];
         glsr()->{$this->logOnceKey} = $once;
     }
@@ -254,8 +254,8 @@ 
      */
     public function size()
     {
-        return file_exists($this->file)
-            ? filesize($this->file)
+        return file_exists( $this->file )
+            ? filesize( $this->file )
             : 0;
     }
 
@@ -266,9 +266,9 @@ 
      * @param array $context
      * @return static
      */
-    public function warning($message, array $context = [])
+    public function warning( $message, array $context = [] )
     {
-        return $this->log(static::WARNING, $message, $context);
+        return $this->log( static::WARNING, $message, $context );
     }
 
     /**
@@ -276,11 +276,11 @@ 
      * @param int $index
      * @return string
      */
-    protected function buildBacktraceLine($backtrace, $index)
+    protected function buildBacktraceLine( $backtrace, $index )
     {
-        return sprintf('%s:%s',
-            Arr::get($backtrace, $index.'.file'), // realpath
-            Arr::get($backtrace, $index.'.line')
+        return sprintf( '%s:%s',
+            Arr::get( $backtrace, $index.'.file' ), // realpath
+            Arr::get( $backtrace, $index.'.line' )
         );
     }
 
@@ -290,11 +290,11 @@ 
      * @param string $backtraceLine
      * @return string
      */
-    protected function buildLogEntry($levelName, $message, $backtraceLine = '')
+    protected function buildLogEntry( $levelName, $message, $backtraceLine = '' )
     {
-        return sprintf('[%s] %s [%s] %s',
-            current_time('mysql'),
-            strtoupper($levelName),
+        return sprintf( '[%s] %s [%s] %s',
+            current_time( 'mysql' ),
+            strtoupper( $levelName ),
             $backtraceLine,
             $message
         );
@@ -304,10 +304,10 @@ 
      * @param int $level
      * @return bool
      */
-    protected function canLogEntry($level, $backtraceLine)
+    protected function canLogEntry( $level, $backtraceLine )
     {
-        $levelExists = array_key_exists($level, $this->getLevels());
-        if (!Str::contains($backtraceLine, glsr()->path())) {
+        $levelExists = array_key_exists( $level, $this->getLevels() );
+        if( !Str::contains( $backtraceLine, glsr()->path() ) ) {
             return $levelExists; // ignore level restriction if triggered outside of the plugin
         }
         return $levelExists && $level >= $this->getLevel();
@@ -318,17 +318,17 @@ 
      */
     protected function getBacktraceLine()
     {
-        $backtrace = debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 6);
-        $search = array_search('glsr_log', glsr_array_column($backtrace, 'function'));
-        if (false !== $search) {
-            return $this->buildBacktraceLine($backtrace, (int) $search);
+        $backtrace = debug_backtrace( DEBUG_BACKTRACE_IGNORE_ARGS, 6 );
+        $search = array_search( 'glsr_log', glsr_array_column( $backtrace, 'function' ) );
+        if( false !== $search ) {
+            return $this->buildBacktraceLine( $backtrace, (int)$search );
         }
-        $search = array_search('log', glsr_array_column($backtrace, 'function'));
-        if (false !== $search) {
-            $index = '{closure}' == Arr::get($backtrace, ($search + 2).'.function')
+        $search = array_search( 'log', glsr_array_column( $backtrace, 'function' ) );
+        if( false !== $search ) {
+            $index = '{closure}' == Arr::get( $backtrace, ($search + 2).'.function' )
                 ? $search + 4
                 : $search + 1;
-            return $this->buildBacktraceLine($backtrace, $index);
+            return $this->buildBacktraceLine( $backtrace, $index );
         }
         return 'Unknown';
     }
@@ -337,23 +337,23 @@ 
      * @param mixed $data
      * @return string
      */
-    protected function getBacktraceLineFromData($data)
+    protected function getBacktraceLineFromData( $data )
     {
         $backtrace = $data instanceof Throwable
             ? $data->getTrace()
-            : debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS, 1);
-        return $this->buildBacktraceLine($backtrace, 0);
+            : debug_backtrace( DEBUG_BACKTRACE_IGNORE_ARGS, 1 );
+        return $this->buildBacktraceLine( $backtrace, 0 );
     }
 
     /**
      * @param mixed $data
      * @return string
      */
-    protected function getMessageFromData($data)
+    protected function getMessageFromData( $data )
     {
         return $data instanceof Throwable
-            ? $this->normalizeThrowableMessage($data->getMessage())
-            : print_r($data, 1);
+            ? $this->normalizeThrowableMessage( $data->getMessage() )
+            : print_r( $data, 1 );
     }
 
     /**
@@ -362,53 +362,53 @@ 
      * @param array $context
      * @return string
      */
-    protected function interpolate($message, $context = [])
+    protected function interpolate( $message, $context = [] )
     {
-        if ($this->isObjectOrArray($message) || !is_array($context)) {
-            return print_r($message, true);
+        if( $this->isObjectOrArray( $message ) || !is_array( $context ) ) {
+            return print_r( $message, true );
         }
         $replace = [];
-        foreach ($context as $key => $value) {
-            $replace['{'.$key.'}'] = $this->normalizeValue($value);
+        foreach( $context as $key => $value ) {
+            $replace['{'.$key.'}'] = $this->normalizeValue( $value );
         }
-        return strtr($message, $replace);
+        return strtr( $message, $replace );
     }
 
     /**
      * @param mixed $value
      * @return bool
      */
-    protected function isObjectOrArray($value)
+    protected function isObjectOrArray( $value )
     {
-        return is_object($value) || is_array($value);
+        return is_object( $value ) || is_array( $value );
     }
 
     /**
      * @param string $backtraceLine
      * @return string
      */
-    protected function normalizeBacktraceLine($backtraceLine)
+    protected function normalizeBacktraceLine( $backtraceLine )
     {
         $search = [
-            glsr()->path('plugin/'),
-            glsr()->path('plugin/', false),
-            trailingslashit(glsr()->path()),
-            trailingslashit(glsr()->path('', false)),
+            glsr()->path( 'plugin/' ),
+            glsr()->path( 'plugin/', false ),
+            trailingslashit( glsr()->path() ),
+            trailingslashit( glsr()->path( '', false ) ),
             WP_CONTENT_DIR,
             ABSPATH,
         ];
-        return str_replace(array_unique($search), '', $backtraceLine);
+        return str_replace( array_unique( $search ), '', $backtraceLine );
     }
 
     /**
      * @param string $message
      * @return string
      */
-    protected function normalizeThrowableMessage($message)
+    protected function normalizeThrowableMessage( $message )
     {
-        $calledIn = strpos($message, ', called in');
+        $calledIn = strpos( $message, ', called in' );
         return false !== $calledIn
-            ? substr($message, 0, $calledIn)
+            ? substr( $message, 0, $calledIn )
             : $message;
     }
 
@@ -416,14 +416,14 @@ 
      * @param mixed $value
      * @return string
      */
-    protected function normalizeValue($value)
+    protected function normalizeValue( $value )
     {
-        if ($value instanceof DateTime) {
-            $value = $value->format('Y-m-d H:i:s');
-        } elseif ($this->isObjectOrArray($value)) {
-            $value = json_encode($value);
+        if( $value instanceof DateTime ) {
+            $value = $value->format( 'Y-m-d H:i:s' );
+        } elseif( $this->isObjectOrArray( $value ) ) {
+            $value = json_encode( $value );
         }
-        return (string) $value;
+        return (string)$value;
     }
 
     /**
@@ -431,7 +431,7 @@ 
      */
     protected function reset()
     {
-        if ($this->size() <= pow(1024, 2) / 8) {
+        if( $this->size() <= pow( 1024, 2 ) / 8 ) {
             return;
         }
         $this->clear();
@@ -439,7 +439,7 @@ 
             $this->file,
             $this->buildLogEntry(
                 static::NOTICE,
-                __('Console was automatically cleared (128 KB maximum size)', 'site-reviews')
+                __( 'Console was automatically cleared (128 KB maximum size)', 'site-reviews' )
             )
         );
     }

deprecated.php

Spacing +58 added lines, -58 removed lines

@@ -1,136 +1,136 @@
 <?php
 
-defined('WPINC') || die;
+defined( 'WPINC' ) || die;
 
-if (apply_filters('site-reviews/support/deprecated/v4', true)) {
+if( apply_filters( 'site-reviews/support/deprecated/v4', true ) ) {
     // Unprotected review meta has been deprecated
-    add_filter('get_post_metadata', function ($data, $postId, $metaKey, $single) {
-        $metaKeys = array_keys(glsr('Defaults\CreateReviewDefaults')->defaults());
-        if (!in_array($metaKey, $metaKeys) || glsr()->post_type != get_post_type($postId)) {
+    add_filter( 'get_post_metadata', function( $data, $postId, $metaKey, $single ) {
+        $metaKeys = array_keys( glsr( 'Defaults\CreateReviewDefaults' )->defaults() );
+        if( !in_array( $metaKey, $metaKeys ) || glsr()->post_type != get_post_type( $postId ) ) {
             return $data;
         }
         glsr()->deprecated[] = sprintf(
             'The "%1$s" meta_key has been deprecated for Reviews. Please use the protected "_%1$s" meta_key instead.',
             $metaKey
         );
-        return get_post_meta($postId, '_'.$metaKey, $single);
-    }, 10, 4);
+        return get_post_meta( $postId, '_'.$metaKey, $single );
+    }, 10, 4 );
 
     // Modules/Html/Template.php
-    add_filter('site-reviews/interpolate/reviews', function ($context, $template) {
+    add_filter( 'site-reviews/interpolate/reviews', function( $context, $template ) {
         $search = '{{ navigation }}';
-        if (false !== strpos($template, $search)) {
+        if( false !== strpos( $template, $search ) ) {
             $context['navigation'] = $context['pagination'];
             glsr()->deprecated[] = 'The {{ navigation }} template key in "YOUR_THEME/site-reviews/reviews.php" has been deprecated. Please use the {{ pagination }} template key instead.';
         }
         return $context;
-    }, 10, 2);
+    }, 10, 2 );
 
     // Database/ReviewManager.php
-    add_action('site-reviews/review/created', function ($review) {
-        if (has_action('site-reviews/local/review/create')) {
+    add_action( 'site-reviews/review/created', function( $review ) {
+        if( has_action( 'site-reviews/local/review/create' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/local/review/create" hook has been deprecated. Please use the "site-reviews/review/created" hook instead.';
-            do_action('site-reviews/local/review/create', (array) get_post($review->ID), (array) $review, $review->ID);
+            do_action( 'site-reviews/local/review/create', (array)get_post( $review->ID ), (array)$review, $review->ID );
         }
-    }, 9);
+    }, 9 );
 
     // Handlers/CreateReview.php
-    add_action('site-reviews/review/submitted', function ($review) {
-        if (has_action('site-reviews/local/review/submitted')) {
+    add_action( 'site-reviews/review/submitted', function( $review ) {
+        if( has_action( 'site-reviews/local/review/submitted' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/local/review/submitted" hook has been deprecated. Please use the "site-reviews/review/submitted" hook instead.';
-            do_action('site-reviews/local/review/submitted', null, $review);
+            do_action( 'site-reviews/local/review/submitted', null, $review );
         }
-        if (has_filter('site-reviews/local/review/submitted/message')) {
+        if( has_filter( 'site-reviews/local/review/submitted/message' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/local/review/submitted/message" hook has been deprecated.';
         }
-    }, 9);
+    }, 9 );
 
     // Database/ReviewManager.php
-    add_filter('site-reviews/create/review-values', function ($values, $command) {
-        if (has_filter('site-reviews/local/review')) {
+    add_filter( 'site-reviews/create/review-values', function( $values, $command ) {
+        if( has_filter( 'site-reviews/local/review' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/local/review" hook has been deprecated. Please use the "site-reviews/create/review-values" hook instead.';
-            return apply_filters('site-reviews/local/review', $values, $command);
+            return apply_filters( 'site-reviews/local/review', $values, $command );
         }
         return $values;
-    }, 9, 2);
+    }, 9, 2 );
 
     // Handlers/EnqueuePublicAssets.php
-    add_filter('site-reviews/enqueue/public/localize', function ($variables) {
-        if (has_filter('site-reviews/enqueue/localize')) {
+    add_filter( 'site-reviews/enqueue/public/localize', function( $variables ) {
+        if( has_filter( 'site-reviews/enqueue/localize' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/enqueue/localize" hook has been deprecated. Please use the "site-reviews/enqueue/public/localize" hook instead.';
-            return apply_filters('site-reviews/enqueue/localize', $variables);
+            return apply_filters( 'site-reviews/enqueue/localize', $variables );
         }
         return $variables;
-    }, 9);
+    }, 9 );
 
     // Modules/Rating.php
-    add_filter('site-reviews/rating/average', function ($average) {
-        if (has_filter('site-reviews/average/rating')) {
+    add_filter( 'site-reviews/rating/average', function( $average ) {
+        if( has_filter( 'site-reviews/average/rating' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/average/rating" hook has been deprecated. Please use the "site-reviews/rating/average" hook instead.';
         }
         return $average;
-    }, 9);
+    }, 9 );
 
     // Modules/Rating.php
-    add_filter('site-reviews/rating/ranking', function ($ranking) {
-        if (has_filter('site-reviews/bayesian/ranking')) {
+    add_filter( 'site-reviews/rating/ranking', function( $ranking ) {
+        if( has_filter( 'site-reviews/bayesian/ranking' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/bayesian/ranking" hook has been deprecated. Please use the "site-reviews/rating/ranking" hook instead.';
         }
         return $ranking;
-    }, 9);
+    }, 9 );
 
     // Modules/Html/Partials/SiteReviews.php
-    add_filter('site-reviews/review/build/after', function ($renderedFields) {
-        if (has_filter('site-reviews/reviews/review/text')) {
+    add_filter( 'site-reviews/review/build/after', function( $renderedFields ) {
+        if( has_filter( 'site-reviews/reviews/review/text' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/reviews/review/text" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.';
         }
-        if (has_filter('site-reviews/reviews/review/title')) {
+        if( has_filter( 'site-reviews/reviews/review/title' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/reviews/review/title" hook has been deprecated. Please use the "site-reviews/review/build/after" hook instead.';
         }
         return $renderedFields;
-    }, 9);
+    }, 9 );
 
     // Modules/Html/Partials/SiteReviews.php
-    add_filter('site-reviews/review/build/before', function ($review) {
-        if (has_filter('site-reviews/rendered/review')) {
+    add_filter( 'site-reviews/review/build/before', function( $review ) {
+        if( has_filter( 'site-reviews/rendered/review' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/rendered/review" hook has been deprecated. Please either use a custom "review.php" template (refer to the documentation), or use the "site-reviews/review/build/after" hook instead.';
         }
-        if (has_filter('site-reviews/rendered/review/meta/order')) {
+        if( has_filter( 'site-reviews/rendered/review/meta/order' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/rendered/review/meta/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).';
         }
-        if (has_filter('site-reviews/rendered/review/order')) {
+        if( has_filter( 'site-reviews/rendered/review/order' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/rendered/review/order" hook has been deprecated. Please use a custom "review.php" template instead (refer to the documentation).';
         }
-        if (has_filter('site-reviews/rendered/review-form/login-register')) {
+        if( has_filter( 'site-reviews/rendered/review-form/login-register' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/rendered/review-form/login-register" hook has been deprecated. Please use a custom "login-register.php" template instead (refer to the documentation).';
         }
-        if (has_filter('site-reviews/reviews/navigation_links')) {
+        if( has_filter( 'site-reviews/reviews/navigation_links' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/reviews/navigation_links" hook has been deprecated. Please use a custom "pagination.php" template instead (refer to the documentation).';
         }
         return $review;
-    }, 9);
+    }, 9 );
 
-    add_filter('site-reviews/validate/custom', function ($result, $request) {
-        if (has_filter('site-reviews/validate/review/submission')) {
-            glsr_log()->warning('The "site-reviews/validate/review/submission" hook has been deprecated. Please use the "site-reviews/validate/custom" hook instead.');
-            return apply_filters('site-reviews/validate/review/submission', $result, $request);
+    add_filter( 'site-reviews/validate/custom', function( $result, $request ) {
+        if( has_filter( 'site-reviews/validate/review/submission' ) ) {
+            glsr_log()->warning( 'The "site-reviews/validate/review/submission" hook has been deprecated. Please use the "site-reviews/validate/custom" hook instead.' );
+            return apply_filters( 'site-reviews/validate/review/submission', $result, $request );
         }
         return $result;
-    }, 9, 2);
+    }, 9, 2 );
 
-    add_filter('site-reviews/views/file', function ($file, $view, $data) {
-        if (has_filter('site-reviews/addon/views/file')) {
+    add_filter( 'site-reviews/views/file', function( $file, $view, $data ) {
+        if( has_filter( 'site-reviews/addon/views/file' ) ) {
             glsr()->deprecated[] = 'The "site-reviews/addon/views/file" hook has been deprecated. Please use the "site-reviews/views/file" hook instead.';
-            $file = apply_filters('site-reviews/addon/views/file', $file, $view, $data);
+            $file = apply_filters( 'site-reviews/addon/views/file', $file, $view, $data );
         }
         return $file;
-    }, 9, 3);
+    }, 9, 3 );
 }
 
-add_action('wp_footer', function () {
-    $notices = array_keys(array_flip(glsr()->deprecated));
-    natsort($notices);
-    foreach ($notices as $notice) {
-        glsr_log()->warning($notice);
+add_action( 'wp_footer', function() {
+    $notices = array_keys( array_flip( glsr()->deprecated ) );
+    natsort( $notices );
+    foreach( $notices as $notice ) {
+        glsr_log()->warning( $notice );
     }
 });

views/partials/editor/review.php

Spacing +7 added lines, -7 removed lines

@@ -1,26 +1,26 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <div id="titlediv">
-    <input type="text" id="title" value="<?= $post->post_title ? esc_attr($post->post_title) : sprintf('(%s)', __('no title', 'site-reviews')); ?>" readonly>
+    <input type="text" id="title" value="<?= $post->post_title ? esc_attr( $post->post_title ) : sprintf( '(%s)', __( 'no title', 'site-reviews' ) ); ?>" readonly>
 </div>
 
 <div id="contentdiv" class="wp-editor-container">
-    <textarea readonly><?= esc_attr($post->post_content); ?></textarea>
+    <textarea readonly><?= esc_attr( $post->post_content ); ?></textarea>
 </div>
 
-<?php if (empty($response)) {
+<?php if( empty($response) ) {
     return;
 } ?>
 
 <div class="postbox glsr-response-postbox">
     <button type="button" class="handlediv" aria-expanded="true">
-        <span class="screen-reader-text"><?= __('Toggle panel: Public Response', 'site-reviews'); ?></span>
+        <span class="screen-reader-text"><?= __( 'Toggle panel: Public Response', 'site-reviews' ); ?></span>
         <span class="toggle-indicator" aria-hidden="true"></span>
     </button>
     <h2 class="hndle">
-        <span><?= __('Public Response', 'site-reviews'); ?></span>
+        <span><?= __( 'Public Response', 'site-reviews' ); ?></span>
     </h2>
     <div class="inside">
-        <?= wpautop(esc_attr($response)); ?>
+        <?= wpautop( esc_attr( $response ) ); ?>
     </div>
 </div>

views/pages/tools/console.php

Spacing +7 added lines, -7 removed lines

@@ -1,22 +1,22 @@
-<?php defined('WPINC') || die; ?>
+<?php defined( 'WPINC' ) || die; ?>
 
 <textarea id="log-file" class="large-text code glsr-code glsr-code-large" rows="20" readonly>{{ console }}</textarea>
 <form method="post" class="glsr-float-left">
     <input type="hidden" name="{{ id }}[_action]" value="download-console">
-    <?php wp_nonce_field('download-console'); ?>
-    <?php submit_button(__('Download Console', 'site-reviews'), 'primary', '', false); ?>
+    <?php wp_nonce_field( 'download-console' ); ?>
+    <?php submit_button( __( 'Download Console', 'site-reviews' ), 'primary', '', false ); ?>
 </form>
 <form method="post" class="glsr-float-left">
     <input type="hidden" name="{{ id }}[_action]" value="fetch-console">
-    <?php wp_nonce_field('fetch-console'); ?>
+    <?php wp_nonce_field( 'fetch-console' ); ?>
     <button type="submit" class="glsr-button button" id="fetch-console">
-        <span data-loading="<?= __('Reloading...', 'site-reviews'); ?>"><?= __('Reload', 'site-reviews'); ?></span>
+        <span data-loading="<?= __( 'Reloading...', 'site-reviews' ); ?>"><?= __( 'Reload', 'site-reviews' ); ?></span>
     </button>
 </form>
 <form method="post">
     <input type="hidden" name="{{ id }}[_action]" value="clear-console">
-    <?php wp_nonce_field('clear-console'); ?>
+    <?php wp_nonce_field( 'clear-console' ); ?>
     <button type="submit" class="glsr-button button" id="clear-console">
-        <span data-loading="<?= __('Clearing...', 'site-reviews'); ?>"><?= __('Clear', 'site-reviews'); ?></span>
+        <span data-loading="<?= __( 'Clearing...', 'site-reviews' ); ?>"><?= __( 'Clear', 'site-reviews' ); ?></span>
     </button>
 </form>

plugin/Handlers/EnqueueAdminAssets.php

Spacing +36 added lines, -36 removed lines

@@ -18,9 +18,9 @@ 
     /**
      * @return void
      */
-    public function handle(Command $command)
+    public function handle( Command $command )
     {
-        $this->generatePointers($command->pointers);
+        $this->generatePointers( $command->pointers );
         $this->enqueueAssets();
         $this->localizeAssets();
     }
@@ -30,25 +30,25 @@ 
      */
     public function enqueueAssets()
     {
-        if (!$this->isCurrentScreen()) {
+        if( !$this->isCurrentScreen() ) {
             return;
         }
         wp_enqueue_style(
             Application::ID,
-            glsr()->url('assets/styles/'.Application::ID.'-admin.css'),
+            glsr()->url( 'assets/styles/'.Application::ID.'-admin.css' ),
             [],
             glsr()->version
         );
         wp_enqueue_script(
             Application::ID,
-            glsr()->url('assets/scripts/'.Application::ID.'-admin.js'),
+            glsr()->url( 'assets/scripts/'.Application::ID.'-admin.js' ),
             $this->getDependencies(),
             glsr()->version,
             true
         );
-        if (!empty($this->pointers)) {
-            wp_enqueue_style('wp-pointer');
-            wp_enqueue_script('wp-pointer');
+        if( !empty($this->pointers) ) {
+            wp_enqueue_style( 'wp-pointer' );
+            wp_enqueue_script( 'wp-pointer' );
         }
     }
 
@@ -60,33 +60,33 @@ 
         $variables = [
             'action' => Application::PREFIX.'action',
             'addons' => [],
-            'ajaxurl' => admin_url('admin-ajax.php'),
+            'ajaxurl' => admin_url( 'admin-ajax.php' ),
             'hideoptions' => [
-                'site_reviews' => glsr(SiteReviewsShortcode::class)->getHideOptions(),
-                'site_reviews_form' => glsr(SiteReviewsFormShortcode::class)->getHideOptions(),
-                'site_reviews_summary' => glsr(SiteReviewsSummaryShortcode::class)->getHideOptions(),
+                'site_reviews' => glsr( SiteReviewsShortcode::class )->getHideOptions(),
+                'site_reviews_form' => glsr( SiteReviewsFormShortcode::class )->getHideOptions(),
+                'site_reviews_summary' => glsr( SiteReviewsSummaryShortcode::class )->getHideOptions(),
             ],
             'nameprefix' => Application::ID,
             'nonce' => [
-                'change-status' => wp_create_nonce('change-status'),
-                'clear-console' => wp_create_nonce('clear-console'),
-                'count-reviews' => wp_create_nonce('count-reviews'),
-                'fetch-console' => wp_create_nonce('fetch-console'),
-                'mce-shortcode' => wp_create_nonce('mce-shortcode'),
-                'sync-reviews' => wp_create_nonce('sync-reviews'),
-                'toggle-pinned' => wp_create_nonce('toggle-pinned'),
+                'change-status' => wp_create_nonce( 'change-status' ),
+                'clear-console' => wp_create_nonce( 'clear-console' ),
+                'count-reviews' => wp_create_nonce( 'count-reviews' ),
+                'fetch-console' => wp_create_nonce( 'fetch-console' ),
+                'mce-shortcode' => wp_create_nonce( 'mce-shortcode' ),
+                'sync-reviews' => wp_create_nonce( 'sync-reviews' ),
+                'toggle-pinned' => wp_create_nonce( 'toggle-pinned' ),
             ],
             'pointers' => $this->pointers,
             'shortcodes' => [],
             'tinymce' => [
-                'glsr_shortcode' => glsr()->url('assets/scripts/mce-plugin.js'),
+                'glsr_shortcode' => glsr()->url( 'assets/scripts/mce-plugin.js' ),
             ],
         ];
-        if (user_can_richedit()) {
+        if( user_can_richedit() ) {
             $variables['shortcodes'] = $this->localizeShortcodes();
         }
-        $variables = apply_filters('site-reviews/enqueue/admin/localize', $variables);
-        wp_localize_script(Application::ID, 'GLSR', $variables);
+        $variables = apply_filters( 'site-reviews/enqueue/admin/localize', $variables );
+        wp_localize_script( Application::ID, 'GLSR', $variables );
     }
 
     /**
@@ -94,17 +94,17 @@ 
      */
     protected function getDependencies()
     {
-        $dependencies = apply_filters('site-reviews/enqueue/admin/dependencies', []);
-        $dependencies = array_merge($dependencies, [
+        $dependencies = apply_filters( 'site-reviews/enqueue/admin/dependencies', [] );
+        $dependencies = array_merge( $dependencies, [
             'jquery', 'jquery-ui-sortable', 'underscore', 'wp-util',
-        ]);
+        ] );
         return $dependencies;
     }
 
     /**
      * @return array
      */
-    protected function generatePointer(array $pointer)
+    protected function generatePointer( array $pointer )
     {
         return [
             'id' => $pointer['id'],
@@ -120,19 +120,19 @@ 
     /**
      * @return void
      */
-    protected function generatePointers(array $pointers)
+    protected function generatePointers( array $pointers )
     {
-        $dismissedPointers = get_user_meta(get_current_user_id(), 'dismissed_wp_pointers', true);
-        $dismissedPointers = explode(',', (string) $dismissedPointers);
+        $dismissedPointers = get_user_meta( get_current_user_id(), 'dismissed_wp_pointers', true );
+        $dismissedPointers = explode( ',', (string)$dismissedPointers );
         $generatedPointers = [];
-        foreach ($pointers as $pointer) {
-            if ($pointer['screen'] != glsr_current_screen()->id) {
+        foreach( $pointers as $pointer ) {
+            if( $pointer['screen'] != glsr_current_screen()->id ) {
                 continue;
             }
-            if (in_array($pointer['id'], $dismissedPointers)) {
+            if( in_array( $pointer['id'], $dismissedPointers ) ) {
                 continue;
             }
-            $generatedPointers[] = $this->generatePointer($pointer);
+            $generatedPointers[] = $this->generatePointer( $pointer );
         }
         $this->pointers = $generatedPointers;
     }
@@ -156,8 +156,8 @@ 
     protected function localizeShortcodes()
     {
         $variables = [];
-        foreach (glsr()->mceShortcodes as $tag => $args) {
-            if (empty($args['required'])) {
+        foreach( glsr()->mceShortcodes as $tag => $args ) {
+            if( empty($args['required']) ) {
                 continue;
             }
             $variables[$tag] = $args['required'];