pryley / site-reviews

plugin/Modules/Style.php

Spacing +35 added lines, -35 removed lines

@@ -34,7 +34,7 @@ 
 
     public function __construct()
     {
-        $this->style = glsr(OptionManager::class)->get('settings.general.style', 'default');
+        $this->style = glsr( OptionManager::class )->get( 'settings.general.style', 'default' );
         $this->setConfig();
     }
 
@@ -42,23 +42,23 @@ 
      * @param string $view
      * @return string
      */
-    public function filterView($view)
+    public function filterView( $view )
     {
-        $styledViews = apply_filters('site-reviews/style/views', [
+        $styledViews = apply_filters( 'site-reviews/style/views', [
             'templates/form/field',
             'templates/form/response',
             'templates/form/submit-button',
             'templates/reviews-form',
-        ]);
-        if (!preg_match('('.implode('|', $styledViews).')', $view)) {
+        ] );
+        if( !preg_match( '('.implode( '|', $styledViews ).')', $view ) ) {
             return $view;
         }
-        $views = $this->generatePossibleViews($view);
-        foreach ($views as $possibleView) {
-            if (!file_exists(glsr()->file($possibleView))) {
+        $views = $this->generatePossibleViews( $view );
+        foreach( $views as $possibleView ) {
+            if( !file_exists( glsr()->file( $possibleView ) ) ) {
                 continue;
             }
-            return Str::removePrefix('views/', $possibleView);
+            return Str::removePrefix( 'views/', $possibleView );
         }
         return $view;
     }
@@ -68,7 +68,7 @@ 
      */
     public function get()
     {
-        return apply_filters('site-reviews/style', $this->style);
+        return apply_filters( 'site-reviews/style', $this->style );
     }
 
     /**
@@ -77,77 +77,77 @@ 
     public function setConfig()
     {
         $config = shortcode_atts(
-            array_fill_keys(['fields', 'pagination', 'validation'], []),
-            glsr()->config('styles/'.$this->style)
+            array_fill_keys( ['fields', 'pagination', 'validation'], [] ),
+            glsr()->config( 'styles/'.$this->style )
         );
-        $this->fields = glsr(StyleFieldsDefaults::class)->restrict($config['fields']);
-        $this->pagination = glsr(PaginationDefaults::class)->restrict($config['pagination']);
-        $this->validation = glsr(StyleValidationDefaults::class)->restrict($config['validation']);
+        $this->fields = glsr( StyleFieldsDefaults::class )->restrict( $config['fields'] );
+        $this->pagination = glsr( PaginationDefaults::class )->restrict( $config['pagination'] );
+        $this->validation = glsr( StyleValidationDefaults::class )->restrict( $config['validation'] );
     }
 
     /**
      * @return void
      */
-    public function modifyField(Builder $instance)
+    public function modifyField( Builder $instance )
     {
-        if (!$this->isPublicInstance($instance) || empty(array_filter($this->fields))) {
+        if( !$this->isPublicInstance( $instance ) || empty(array_filter( $this->fields )) ) {
             return;
         }
-        call_user_func_array([$this, 'customize'], [$instance]);
+        call_user_func_array( [$this, 'customize'], [$instance] );
     }
 
     /**
      * @return array
      */
-    public function paginationArgs(array $args)
+    public function paginationArgs( array $args )
     {
-        return wp_parse_args($args, $this->pagination);
+        return wp_parse_args( $args, $this->pagination );
     }
 
     /**
      * @return void
      */
-    protected function customize(Builder $instance)
+    protected function customize( Builder $instance )
     {
-        if (!array_key_exists($instance->tag, $this->fields)) {
+        if( !array_key_exists( $instance->tag, $this->fields ) ) {
             return;
         }
-        $args = wp_parse_args($instance->args, array_fill_keys(['class', 'type'], ''));
+        $args = wp_parse_args( $instance->args, array_fill_keys( ['class', 'type'], '' ) );
         $key = $instance->tag.'_'.$args['type'];
-        $classes = Arr::get($this->fields, $key, Arr::get($this->fields, $instance->tag));
-        $instance->args['class'] = trim($args['class'].' '.$classes);
-        do_action_ref_array('site-reviews/customize/'.$this->style, [$instance]);
+        $classes = Arr::get( $this->fields, $key, Arr::get( $this->fields, $instance->tag ) );
+        $instance->args['class'] = trim( $args['class'].' '.$classes );
+        do_action_ref_array( 'site-reviews/customize/'.$this->style, [$instance] );
     }
 
     /**
      * @param string $view
      * @return array
      */
-    protected function generatePossibleViews($view)
+    protected function generatePossibleViews( $view )
     {
-        $basename = basename($view);
-        $basepath = rtrim($view, $basename);
+        $basename = basename( $view );
+        $basepath = rtrim( $view, $basename );
         $customPath = 'views/partials/styles/'.$this->style.'/';
-        $parts = explode('_', $basename);
+        $parts = explode( '_', $basename );
         $views = [
             $customPath.$basename,
             $customPath.$parts[0],
             $view,
             $basepath.$parts[0],
         ];
-        return array_filter($views);
+        return array_filter( $views );
     }
 
     /**
      * @return bool
      */
-    protected function isPublicInstance(Builder $instance)
+    protected function isPublicInstance( Builder $instance )
     {
-        $args = wp_parse_args($instance->args, [
+        $args = wp_parse_args( $instance->args, [
             'is_public' => false,
             'is_raw' => false,
-        ]);
-        if (is_admin() || !$args['is_public'] || $args['is_raw']) {
+        ] );
+        if( is_admin() || !$args['is_public'] || $args['is_raw'] ) {
             return false;
         }
         return true;

plugin/Modules/Migrate.php

Spacing +35 added lines, -35 removed lines

@@ -31,14 +31,14 @@ 
      */
     public function isMigrationNeeded()
     {
-        $transient = get_transient($this->transientKey);
-        if (false === $transient || !isset($transient[glsr()->version])) {
+        $transient = get_transient( $this->transientKey );
+        if( false === $transient || !isset($transient[glsr()->version]) ) {
             $transient = [
                 glsr()->version => !empty($this->getNewMigrationFiles()),
             ];
-            set_transient($this->transientKey, $transient);
+            set_transient( $this->transientKey, $transient );
         }
-        return Helper::castToBool($transient[glsr()->version]);
+        return Helper::castToBool( $transient[glsr()->version] );
     }
 
     /**
@@ -46,7 +46,7 @@ 
      */
     public function run()
     {
-        return $this->runMigrations($this->getNewMigrationFiles());
+        return $this->runMigrations( $this->getNewMigrationFiles() );
     }
 
     /**
@@ -54,7 +54,7 @@ 
      */
     public function runAll()
     {
-        return $this->runMigrations($this->getMigrationFiles());
+        return $this->runMigrations( $this->getMigrationFiles() );
     }
 
     /**
@@ -64,10 +64,10 @@ 
     {
         $fallback = '0.0.0';
         $majorVersions = [4, 3, 2, 1];
-        foreach ($majorVersions as $majorVersion) {
-            $settings = get_option(OptionManager::databaseKey($majorVersion));
-            $version = Arr::get($settings, 'version', $fallback);
-            if (Helper::isGreaterThan($version, $fallback)) {
+        foreach( $majorVersions as $majorVersion ) {
+            $settings = get_option( OptionManager::databaseKey( $majorVersion ) );
+            $version = Arr::get( $settings, 'version', $fallback );
+            if( Helper::isGreaterThan( $version, $fallback ) ) {
                 return $version;
             }
         }
@@ -80,15 +80,15 @@ 
     protected function getMigrationFiles()
     {
         $files = [];
-        $dir = glsr()->path('plugin/Modules/Migrations');
-        if (is_dir($dir)) {
-            $iterator = new DirectoryIterator($dir);
-            foreach ($iterator as $fileinfo) {
-                if ($fileinfo->isFile()) {
+        $dir = glsr()->path( 'plugin/Modules/Migrations' );
+        if( is_dir( $dir ) ) {
+            $iterator = new DirectoryIterator( $dir );
+            foreach( $iterator as $fileinfo ) {
+                if( $fileinfo->isFile() ) {
                     $files[] = $fileinfo->getFilename();
                 }
             }
-            natsort($files);
+            natsort( $files );
         }
         return $files;
     }
@@ -99,11 +99,11 @@ 
     protected function getNewMigrationFiles()
     {
         $files = $this->getMigrationFiles();
-        foreach ($files as $index => $file) {
-            $className = str_replace('.php', '', $file);
-            $migrationVersion = str_replace(['Migrate_', '_'], ['', '.'], $className);
-            $suffix = preg_replace('/[\d.]+(.+)?/', '${1}', glsr()->version); // allow alpha/beta versions
-            if (Helper::isGreaterThanOrEqual($this->currentVersion, $migrationVersion.$suffix)) {
+        foreach( $files as $index => $file ) {
+            $className = str_replace( '.php', '', $file );
+            $migrationVersion = str_replace( ['Migrate_', '_'], ['', '.'], $className );
+            $suffix = preg_replace( '/[\d.]+(.+)?/', '${1}', glsr()->version ); // allow alpha/beta versions
+            if( Helper::isGreaterThanOrEqual( $this->currentVersion, $migrationVersion.$suffix ) ) {
                 unset($files[$index]);
             }
         }
@@ -113,22 +113,22 @@ 
     /**
      * @return bool
      */
-    protected function runMigrations(array $files)
+    protected function runMigrations( array $files )
     {
-        if (empty($files)) {
+        if( empty($files) ) {
             return false;
         }
-        array_walk($files, function ($file) {
-            $className = str_replace('.php', '', $file);
-            glsr('Modules\\Migrations\\'.$className)->run();
-            $versionMigrated = str_replace(['Migrate_', '_'], ['v','.'], $className);
-            glsr_log()->debug('migration completed for '.$versionMigrated);
+        array_walk( $files, function( $file ) {
+            $className = str_replace( '.php', '', $file );
+            glsr( 'Modules\\Migrations\\'.$className )->run();
+            $versionMigrated = str_replace( ['Migrate_', '_'], ['v', '.'], $className );
+            glsr_log()->debug( 'migration completed for '.$versionMigrated );
         });
-        if ($this->currentVersion !== glsr()->version) {
-            $this->updateVersionFrom($this->currentVersion);
+        if( $this->currentVersion !== glsr()->version ) {
+            $this->updateVersionFrom( $this->currentVersion );
         }
-        glsr(OptionManager::class)->set('last_migration_run', current_time('timestamp'));
-        delete_transient($this->transientKey);
+        glsr( OptionManager::class )->set( 'last_migration_run', current_time( 'timestamp' ) );
+        delete_transient( $this->transientKey );
         return true;
     }
 
@@ -136,9 +136,9 @@ 
      * @param string $previousVersion
      * @return void
      */
-    protected function updateVersionFrom($previousVersion)
+    protected function updateVersionFrom( $previousVersion )
     {
-        glsr(OptionManager::class)->set('version', glsr()->version);
-        glsr(OptionManager::class)->set('version_upgraded_from', $previousVersion);
+        glsr( OptionManager::class )->set( 'version', glsr()->version );
+        glsr( OptionManager::class )->set( 'version_upgraded_from', $previousVersion );
     }
 }

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::consolidate($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::consolidate( $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::consolidate(glsr()->{$this->logOnceKey});
+        $once = Arr::consolidate( 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::consolidate(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::consolidate( 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' )
             )
         );
     }

plugin/Application.php

Spacing +82 added lines, -82 removed lines

@@ -41,13 +41,13 @@ 
     public function __construct()
     {
         static::$instance = $this;
-        $this->file = str_replace('plugin/Application', static::ID, (new ReflectionClass($this))->getFileName());
-        $plugin = get_file_data($this->file, [
+        $this->file = str_replace( 'plugin/Application', static::ID, (new ReflectionClass( $this ))->getFileName() );
+        $plugin = get_file_data( $this->file, [
             'languages' => 'Domain Path',
             'name' => 'Plugin Name',
             'version' => 'Version',
-        ], 'plugin');
-        array_walk($plugin, function ($value, $key) {
+        ], 'plugin' );
+        array_walk( $plugin, function( $value, $key ) {
             $this->$key = $value;
         });
     }
@@ -58,17 +58,17 @@ 
     public function activate()
     {
         $this->scheduleCronJob();
-        add_option(static::ID.'activated', true);
+        add_option( static::ID.'activated', true );
     }
 
     /**
      * @param string $view
      * @return string
      */
-    public function build($view, array $data = [])
+    public function build( $view, array $data = [] )
     {
         ob_start();
-        $this->render($view, $data);
+        $this->render( $view, $data );
         return ob_get_clean();
     }
 
@@ -76,9 +76,9 @@ 
      * @param string $capability
      * @return bool
      */
-    public function can($capability)
+    public function can( $capability )
     {
-        return $this->make(Role::class)->can($capability);
+        return $this->make( Role::class )->can( $capability );
     }
 
     /**
@@ -87,35 +87,35 @@ 
     public function catchFatalError()
     {
         $error = error_get_last();
-        if (E_ERROR !== $error['type'] || !Str::contains($error['message'], $this->path())) {
+        if( E_ERROR !== $error['type'] || !Str::contains( $error['message'], $this->path() ) ) {
             return;
         }
-        glsr_log()->error($error['message']);
+        glsr_log()->error( $error['message'] );
     }
 
     /**
      * @param string $name
      * @return array
      */
-    public function config($name)
+    public function config( $name )
     {
-        $path = apply_filters('site-reviews/config', 'config/'.$name.'.php');
-        $configFile = $this->path($path);
-        $config = file_exists($configFile)
+        $path = apply_filters( 'site-reviews/config', 'config/'.$name.'.php' );
+        $configFile = $this->path( $path );
+        $config = file_exists( $configFile )
             ? include $configFile
             : [];
-        return apply_filters('site-reviews/config/'.$name, $config);
+        return apply_filters( 'site-reviews/config/'.$name, $config );
     }
 
     /**
      * @param string $property
      * @return string
      */
-    public function constant($property, $className = 'static')
+    public function constant( $property, $className = 'static' )
     {
         $constant = $className.'::'.$property;
-        return defined($constant)
-            ? apply_filters('site-reviews/const/'.$property, constant($constant))
+        return defined( $constant )
+            ? apply_filters( 'site-reviews/const/'.$property, constant( $constant ) )
             : '';
     }
 
@@ -131,17 +131,17 @@ 
      * @param string $view
      * @return void|string
      */
-    public function file($view)
+    public function file( $view )
     {
-        $view.= '.php';
+        $view .= '.php';
         $filePaths = [];
-        if (Str::startsWith('templates/', $view)) {
-            $filePaths[] = $this->themePath(Str::removePrefix('templates/', $view));
+        if( Str::startsWith( 'templates/', $view ) ) {
+            $filePaths[] = $this->themePath( Str::removePrefix( 'templates/', $view ) );
         }
-        $filePaths[] = $this->path($view);
-        $filePaths[] = $this->path('views/'.$view);
-        foreach ($filePaths as $file) {
-            if (!file_exists($file)) {
+        $filePaths[] = $this->path( $view );
+        $filePaths[] = $this->path( 'views/'.$view );
+        foreach( $filePaths as $file ) {
+            if( !file_exists( $file ) ) {
                 continue;
             }
             return $file;
@@ -153,10 +153,10 @@ 
      */
     public function getDefaults()
     {
-        if (empty($this->defaults)) {
-            $this->defaults = $this->make(DefaultsManager::class)->get();
+        if( empty($this->defaults) ) {
+            $this->defaults = $this->make( DefaultsManager::class )->get();
         }
-        return apply_filters('site-reviews/get/defaults', $this->defaults);
+        return apply_filters( 'site-reviews/get/defaults', $this->defaults );
     }
 
     /**
@@ -164,7 +164,7 @@ 
      * @param string $tab
      * @return string
      */
-    public function getPermission($page = '', $tab = 'index')
+    public function getPermission( $page = '', $tab = 'index' )
     {
         $fallback = 'edit_posts';
         $permissions = [
@@ -185,11 +185,11 @@ 
                 'system-info' => 'edit_others_posts',
             ]
         ];
-        $permission = Arr::get($permissions, $page, $fallback);
-        if (is_array($permission)) {
-            $permission = Arr::get($permission, $tab, $fallback);
+        $permission = Arr::get( $permissions, $page, $fallback );
+        if( is_array( $permission ) ) {
+            $permission = Arr::get( $permission, $tab, $fallback );
         }
-        return empty($permission) || !is_string($permission)
+        return empty($permission) || !is_string( $permission )
             ? $fallback
             : $permission;
     }
@@ -199,10 +199,10 @@ 
      * @param string $tab
      * @return bool
      */
-    public function hasPermission($page = '', $tab = 'index')
+    public function hasPermission( $page = '', $tab = 'index' )
     {
         $isAdmin = $this->isAdmin();
-        return !$isAdmin || ($isAdmin && $this->can($this->getPermission($page, $tab)));
+        return !$isAdmin || ($isAdmin && $this->can( $this->getPermission( $page, $tab ) ));
     }
 
     /**
@@ -210,8 +210,8 @@ 
      */
     public function init()
     {
-        $this->make(Actions::class)->run();
-        $this->make(Filters::class)->run();
+        $this->make( Actions::class )->run();
+        $this->make( Filters::class )->run();
     }
 
     /**
@@ -226,31 +226,31 @@ 
      * @param string $file
      * @return string
      */
-    public function path($file = '', $realpath = true)
+    public function path( $file = '', $realpath = true )
     {
-        $path = plugin_dir_path($this->file);
-        if (!$realpath) {
-            $path = trailingslashit(WP_PLUGIN_DIR).basename(dirname($this->file));
+        $path = plugin_dir_path( $this->file );
+        if( !$realpath ) {
+            $path = trailingslashit( WP_PLUGIN_DIR ).basename( dirname( $this->file ) );
         }
-        $path = trailingslashit($path).ltrim(trim($file), '/');
-        return apply_filters('site-reviews/path', $path, $file);
+        $path = trailingslashit( $path ).ltrim( trim( $file ), '/' );
+        return apply_filters( 'site-reviews/path', $path, $file );
     }
 
     /**
      * @param object $addon
      * @return void
      */
-    public function register($addon)
+    public function register( $addon )
     {
         try {
-            $reflection = new \ReflectionClass($addon);
-            if ($id = $reflection->getConstant('ID')) {
+            $reflection = new \ReflectionClass( $addon );
+            if( $id = $reflection->getConstant( 'ID' ) ) {
                 $this->addons[] = $id;
-                $this->bind($id, $addon);
+                $this->bind( $id, $addon );
                 $addon->init();
             }
-        } catch(\ReflectionException $e) {
-            glsr_log()->error('Attempted to register an invalid addon.');
+        } catch( \ReflectionException $e ) {
+            glsr_log()->error( 'Attempted to register an invalid addon.' );
         }
     }
 
@@ -259,7 +259,7 @@ 
      */
     public function registerAddons()
     {
-        do_action('site-reviews/addon/register', $this);
+        do_action( 'site-reviews/addon/register', $this );
     }
 
     /**
@@ -267,8 +267,8 @@ 
      */
     public function registerLanguages()
     {
-        load_plugin_textdomain(static::ID, false,
-            trailingslashit(plugin_basename($this->path()).'/'.$this->languages)
+        load_plugin_textdomain( static::ID, false,
+            trailingslashit( plugin_basename( $this->path() ).'/'.$this->languages )
         );
     }
 
@@ -277,26 +277,26 @@ 
      */
     public function registerReviewTypes()
     {
-        $types = apply_filters('site-reviews/addon/types', []);
-        $this->reviewTypes = wp_parse_args($types, [
-            'local' => __('Local', 'site-reviews'),
-        ]);
+        $types = apply_filters( 'site-reviews/addon/types', [] );
+        $this->reviewTypes = wp_parse_args( $types, [
+            'local' => __( 'Local', 'site-reviews' ),
+        ] );
     }
 
     /**
      * @param string $view
      * @return void
      */
-    public function render($view, array $data = [])
+    public function render( $view, array $data = [] )
     {
-        $view = apply_filters('site-reviews/render/view', $view, $data);
-        $file = apply_filters('site-reviews/views/file', $this->file($view), $view, $data);
-        if (!file_exists($file)) {
-            glsr_log()->error(sprintf('File not found: (%s) %s', $view, $file));
+        $view = apply_filters( 'site-reviews/render/view', $view, $data );
+        $file = apply_filters( 'site-reviews/views/file', $this->file( $view ), $view, $data );
+        if( !file_exists( $file ) ) {
+            glsr_log()->error( sprintf( 'File not found: (%s) %s', $view, $file ) );
             return;
         }
-        $data = apply_filters('site-reviews/views/data', $data, $view);
-        extract($data);
+        $data = apply_filters( 'site-reviews/views/data', $data, $view );
+        extract( $data );
         include $file;
     }
 
@@ -305,8 +305,8 @@ 
      */
     public function scheduleCronJob()
     {
-        if (false === wp_next_scheduled(static::CRON_EVENT)) {
-            wp_schedule_event(time(), 'twicedaily', static::CRON_EVENT);
+        if( false === wp_next_scheduled( static::CRON_EVENT ) ) {
+            wp_schedule_event( time(), 'twicedaily', static::CRON_EVENT );
         }
     }
 
@@ -315,9 +315,9 @@ 
      */
     public function setDefaults()
     {
-        if (get_option(static::ID.'activated')) {
-            $this->make(DefaultsManager::class)->set();
-            delete_option(static::ID.'activated');
+        if( get_option( static::ID.'activated' ) ) {
+            $this->make( DefaultsManager::class )->set();
+            delete_option( static::ID.'activated' );
         }
     }
 
@@ -325,9 +325,9 @@ 
      * @param string $file
      * @return string
      */
-    public function themePath($file = '')
+    public function themePath( $file = '' )
     {
-        return get_stylesheet_directory().'/'.static::ID.'/'.ltrim(trim($file), '/');
+        return get_stylesheet_directory().'/'.static::ID.'/'.ltrim( trim( $file ), '/' );
     }
 
     /**
@@ -335,36 +335,36 @@ 
      */
     public function unscheduleCronJob()
     {
-        wp_unschedule_event(intval(wp_next_scheduled(static::CRON_EVENT)), static::CRON_EVENT);
+        wp_unschedule_event( intval( wp_next_scheduled( static::CRON_EVENT ) ), static::CRON_EVENT );
     }
 
     /**
      * @param string $path
      * @return string
      */
-    public function url($path = '')
+    public function url( $path = '' )
     {
-        $url = esc_url(plugin_dir_url($this->file).ltrim(trim($path), '/'));
-        return apply_filters('site-reviews/url', $url, $path);
+        $url = esc_url( plugin_dir_url( $this->file ).ltrim( trim( $path ), '/' ) );
+        return apply_filters( 'site-reviews/url', $url, $path );
     }
 
     /**
      * @param string $versionLevel
      * @return string
      */
-    public function version($versionLevel = '')
+    public function version( $versionLevel = '' )
     {
         $pattern = '/^v?(\d{1,5})(\.\d++)?(\.\d++)?(.+)?$/i';
-        preg_match($pattern, $this->version, $matches);
-        switch ($versionLevel) {
+        preg_match( $pattern, $this->version, $matches );
+        switch( $versionLevel ) {
             case 'major':
-                $version = Arr::get($matches, 1);
+                $version = Arr::get( $matches, 1 );
                 break;
             case 'minor':
-                $version = Arr::get($matches, 1).Arr::get($matches, 2);
+                $version = Arr::get( $matches, 1 ).Arr::get( $matches, 2 );
                 break;
             case 'patch':
-                $version = Arr::get($matches, 1).Arr::get($matches, 2).Arr::get($matches, 3);
+                $version = Arr::get( $matches, 1 ).Arr::get( $matches, 2 ).Arr::get( $matches, 3 );
                 break;
         }
         return empty($version)

plugin/Handlers/RegisterWidgets.php

Spacing +6 added lines, -6 removed lines

@@ -11,15 +11,15 @@
     /**
      * @return void
      */
-    public function handle(Command $command)
+    public function handle( Command $command )
     {
-        foreach ($command->widgets as $baseId => $args) {
-            $widgetClass = Helper::buildClassName($baseId.'-widget', 'Widgets');
-            if (!class_exists($widgetClass)) {
-                glsr_log()->error(sprintf('Widget class missing (%s)', $widgetClass));
+        foreach( $command->widgets as $baseId => $args ) {
+            $widgetClass = Helper::buildClassName( $baseId.'-widget', 'Widgets' );
+            if( !class_exists( $widgetClass ) ) {
+                glsr_log()->error( sprintf( 'Widget class missing (%s)', $widgetClass ) );
                 continue;
             }
-            register_widget(new $widgetClass(Application::PREFIX.$baseId, $args['name'], $args));
+            register_widget( new $widgetClass( Application::PREFIX.$baseId, $args['name'], $args ) );
         }
     }
 }

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.'/admin',
-            glsr()->url('assets/styles/'.Application::ID.'-admin.css'),
+            glsr()->url( 'assets/styles/'.Application::ID.'-admin.css' ),
             [],
             glsr()->version
         );
         wp_enqueue_script(
             Application::ID.'/admin',
-            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.'/admin', 'GLSR', $variables);
+        $variables = apply_filters( 'site-reviews/enqueue/admin/localize', $variables );
+        wp_localize_script( Application::ID.'/admin', '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'];

plugin/Handlers/RegisterShortcodes.php

Spacing +6 added lines, -6 removed lines

@@ -10,15 +10,15 @@
     /**
      * @return void
      */
-    public function handle(Command $command)
+    public function handle( Command $command )
     {
-        foreach ($command->shortcodes as $shortcode) {
-            $shortcodeClass = Helper::buildClassName($shortcode.'-shortcode', 'Shortcodes');
-            if (!class_exists($shortcodeClass)) {
-                glsr_log()->error(sprintf('Shortcode class missing (%s)', $shortcodeClass));
+        foreach( $command->shortcodes as $shortcode ) {
+            $shortcodeClass = Helper::buildClassName( $shortcode.'-shortcode', 'Shortcodes' );
+            if( !class_exists( $shortcodeClass ) ) {
+                glsr_log()->error( sprintf( 'Shortcode class missing (%s)', $shortcodeClass ) );
                 continue;
             }
-            add_shortcode($shortcode, [glsr($shortcodeClass), 'buildShortcode']);
+            add_shortcode( $shortcode, [glsr( $shortcodeClass ), 'buildShortcode'] );
         }
     }
 }

plugin/Handlers/RegisterTinymcePopups.php

Spacing +7 added lines, -7 removed lines

@@ -10,18 +10,18 @@
     /**
      * @return void
      */
-    public function handle(Command $command)
+    public function handle( Command $command )
     {
-        foreach ($command->popups as $slug => $label) {
-            $buttonClass = Helper::buildClassName($slug.'-popup', 'Shortcodes');
-            if (!class_exists($buttonClass)) {
-                glsr_log()->error(sprintf('Tinymce Popup class missing (%s)', $buttonClass));
+        foreach( $command->popups as $slug => $label ) {
+            $buttonClass = Helper::buildClassName( $slug.'-popup', 'Shortcodes' );
+            if( !class_exists( $buttonClass ) ) {
+                glsr_log()->error( sprintf( 'Tinymce Popup class missing (%s)', $buttonClass ) );
                 continue;
             }
-            $shortcode = glsr($buttonClass)->register($slug, [
+            $shortcode = glsr( $buttonClass )->register( $slug, [
                 'label' => $label,
                 'title' => $label,
-            ]);
+            ] );
             glsr()->mceShortcodes[$slug] = $shortcode->properties;
         }
     }

plugin/Review.php

Spacing +40 added lines, -40 removed lines

@@ -33,27 +33,27 @@ 
     public $url;
     public $user_id;
 
-    public function __construct(WP_Post $post)
+    public function __construct( WP_Post $post )
     {
-        if (Application::POST_TYPE != $post->post_type) {
+        if( Application::POST_TYPE != $post->post_type ) {
             return;
         }
         $this->content = $post->post_content;
         $this->date = $post->post_date;
-        $this->ID = intval($post->ID);
+        $this->ID = intval( $post->ID );
         $this->status = $post->post_status;
         $this->title = $post->post_title;
-        $this->user_id = intval($post->post_author);
-        $this->setProperties($post);
-        $this->setTermIds($post);
+        $this->user_id = intval( $post->post_author );
+        $this->setProperties( $post );
+        $this->setTermIds( $post );
     }
 
     /**
      * @return mixed
      */
-    public function __get($key)
+    public function __get( $key )
     {
-        return $this->offsetGet($key);
+        return $this->offsetGet( $key );
     }
 
     /**
@@ -61,41 +61,41 @@ 
      */
     public function __toString()
     {
-        return (string) $this->build();
+        return (string)$this->build();
     }
 
     /**
      * @return ReviewHtml
      */
-    public function build(array $args = [])
+    public function build( array $args = [] )
     {
-        if (empty($this->ID)) {
-            return new ReviewHtml($this);
+        if( empty($this->ID) ) {
+            return new ReviewHtml( $this );
         }
-        $partial = glsr(SiteReviewsPartial::class);
-        $partial->args = glsr(SiteReviewsDefaults::class)->merge($args);
-        $partial->options = Arr::flatten(glsr(OptionManager::class)->all());
-        return $partial->buildReview($this);
+        $partial = glsr( SiteReviewsPartial::class );
+        $partial->args = glsr( SiteReviewsDefaults::class )->merge( $args );
+        $partial->options = Arr::flatten( glsr( OptionManager::class )->all() );
+        return $partial->buildReview( $this );
     }
 
     /**
      * @param mixed $key
      * @return bool
      */
-    public function offsetExists($key)
+    public function offsetExists( $key )
     {
-        return property_exists($this, $key) || array_key_exists($key, (array) $this->custom);
+        return property_exists( $this, $key ) || array_key_exists( $key, (array)$this->custom );
     }
 
     /**
      * @param mixed $key
      * @return mixed
      */
-    public function offsetGet($key)
+    public function offsetGet( $key )
     {
-        return property_exists($this, $key)
+        return property_exists( $this, $key )
             ? $this->$key
-            : Arr::get($this->custom, $key, null);
+            : Arr::get( $this->custom, $key, null );
     }
 
     /**
@@ -103,14 +103,14 @@ 
      * @param mixed $value
      * @return void
      */
-    public function offsetSet($key, $value)
+    public function offsetSet( $key, $value )
     {
-        if (property_exists($this, $key)) {
+        if( property_exists( $this, $key ) ) {
             $this->$key = $value;
             return;
         }
-        if (!is_array($this->custom)) {
-            $this->custom = array_filter((array) $this->custom);
+        if( !is_array( $this->custom ) ) {
+            $this->custom = array_filter( (array)$this->custom );
         }
         $this->custom[$key] = $value;
     }
@@ -119,9 +119,9 @@ 
      * @param mixed $key
      * @return void
      */
-    public function offsetUnset($key)
+    public function offsetUnset( $key )
     {
-        $this->offsetSet($key, null);
+        $this->offsetSet( $key, null );
     }
 
     /**
@@ -135,7 +135,7 @@ 
     /**
      * @return bool
      */
-    protected function isModified(array $properties)
+    protected function isModified( array $properties )
     {
         return $this->date != $properties['date']
             || $this->content != $properties['content']
@@ -145,39 +145,39 @@ 
     /**
      * @return void
      */
-    protected function setProperties(WP_Post $post)
+    protected function setProperties( WP_Post $post )
     {
         $defaults = [
-            'author' => __('Anonymous', 'site-reviews'),
+            'author' => __( 'Anonymous', 'site-reviews' ),
             'date' => '',
             'review_id' => '',
             'review_type' => 'local',
         ];
         $meta = array_filter(
-            array_map('array_shift', array_filter((array) get_post_meta($post->ID))),
+            array_map( 'array_shift', array_filter( (array)get_post_meta( $post->ID ) ) ),
             'strlen'
         );
-        $meta = array_merge($defaults, Arr::unprefixKeys($meta));
-        $properties = glsr(CreateReviewDefaults::class)->restrict(array_merge($defaults, $meta));
-        $this->modified = $this->isModified($properties);
-        array_walk($properties, function ($value, $key) {
-            if (!property_exists($this, $key) || isset($this->$key)) {
+        $meta = array_merge( $defaults, Arr::unprefixKeys( $meta ) );
+        $properties = glsr( CreateReviewDefaults::class )->restrict( array_merge( $defaults, $meta ) );
+        $this->modified = $this->isModified( $properties );
+        array_walk( $properties, function( $value, $key ) {
+            if( !property_exists( $this, $key ) || isset($this->$key) ) {
                 return;
             }
-            $this->$key = maybe_unserialize($value);
+            $this->$key = maybe_unserialize( $value );
         });
     }
 
     /**
      * @return void
      */
-    protected function setTermIds(WP_Post $post)
+    protected function setTermIds( WP_Post $post )
     {
         $this->term_ids = [];
-        if (!is_array($terms = get_the_terms($post, Application::TAXONOMY))) {
+        if( !is_array( $terms = get_the_terms( $post, Application::TAXONOMY ) ) ) {
             return;
         }
-        foreach ($terms as $term) {
+        foreach( $terms as $term ) {
             $this->term_ids[] = $term->term_id;
         }
     }