pryley / site-reviews

plugin/Modules/Html/Attributes.php

Spacing +51 added lines, -51 removed lines

@@ -82,15 +82,15 @@ 
      * @param array $args
      * @return static
      */
-    public function __call($method, $args)
+    public function __call( $method, $args )
     {
         $args += [[], false];
-        $constant = 'static::ATTRIBUTES_'.strtoupper($method);
-        $allowedAttributeKeys = defined($constant)
-            ? constant($constant)
+        $constant = 'static::ATTRIBUTES_'.strtoupper( $method );
+        $allowedAttributeKeys = defined( $constant )
+            ? constant( $constant )
             : [];
-        $this->normalize((array) $args[0], $allowedAttributeKeys);
-        $this->normalizeInputType($method);
+        $this->normalize( (array)$args[0], $allowedAttributeKeys );
+        $this->normalizeInputType( $method );
         return $this;
     }
 
@@ -108,21 +108,21 @@ 
     public function toString()
     {
         $attributes = [];
-        foreach ($this->attributes as $attribute => $value) {
-            $quote = $this->getQuoteChar($attribute);
-            $attributes[] = in_array($attribute, static::BOOLEAN_ATTRIBUTES)
+        foreach( $this->attributes as $attribute => $value ) {
+            $quote = $this->getQuoteChar( $attribute );
+            $attributes[] = in_array( $attribute, static::BOOLEAN_ATTRIBUTES )
                 ? $attribute
-                : $attribute.'='.$quote.implode(',', (array) $value).$quote;
+                : $attribute.'='.$quote.implode( ',', (array)$value ).$quote;
         }
-        return implode(' ', $attributes);
+        return implode( ' ', $attributes );
     }
 
     /**
      * @return array
      */
-    protected function filterAttributes(array $allowedAttributeKeys)
+    protected function filterAttributes( array $allowedAttributeKeys )
     {
-        return array_intersect_key($this->attributes, array_flip($allowedAttributeKeys));
+        return array_intersect_key( $this->attributes, array_flip( $allowedAttributeKeys ) );
     }
 
     /**
@@ -130,15 +130,15 @@ 
      */
     protected function filterGlobalAttributes()
     {
-        $globalAttributes = $this->filterAttributes(static::GLOBAL_ATTRIBUTES);
+        $globalAttributes = $this->filterAttributes( static::GLOBAL_ATTRIBUTES );
         $wildcards = [];
-        foreach (static::GLOBAL_WILDCARD_ATTRIBUTES as $wildcard) {
-            $newWildcards = array_filter($this->attributes, function ($key) use ($wildcard) {
-                return glsr(Helper::class)->startsWith($wildcard, $key);
-            }, ARRAY_FILTER_USE_KEY);
-            $wildcards = array_merge($wildcards, $newWildcards);
+        foreach( static::GLOBAL_WILDCARD_ATTRIBUTES as $wildcard ) {
+            $newWildcards = array_filter( $this->attributes, function( $key ) use ($wildcard) {
+                return glsr( Helper::class )->startsWith( $wildcard, $key );
+            }, ARRAY_FILTER_USE_KEY );
+            $wildcards = array_merge( $wildcards, $newWildcards );
         }
-        return array_merge($globalAttributes, $wildcards);
+        return array_merge( $globalAttributes, $wildcards );
     }
 
     /**
@@ -147,7 +147,7 @@ 
     protected function getPermanentAttributes()
     {
         $permanentAttributes = [];
-        if (array_key_exists('value', $this->attributes)) {
+        if( array_key_exists( 'value', $this->attributes ) ) {
             $permanentAttributes['value'] = $this->attributes['value'];
         }
         return $permanentAttributes;
@@ -157,9 +157,9 @@ 
      * @param string $attribute
      * @return string
      */
-    protected function getQuoteChar($attribute)
+    protected function getQuoteChar( $attribute )
     {
-        return glsr(Helper::class)->startsWith('data-', $attribute)
+        return glsr( Helper::class )->startsWith( 'data-', $attribute )
             ? '\''
             : '"';
     }
@@ -169,19 +169,19 @@ 
      * @param mixed $value
      * @return bool
      */
-    protected function isAttributeKeyNumeric($key, $value)
+    protected function isAttributeKeyNumeric( $key, $value )
     {
-        return is_string($value)
-            && is_numeric($key)
-            && !array_key_exists($value, $this->attributes);
+        return is_string( $value )
+            && is_numeric( $key )
+            && !array_key_exists( $value, $this->attributes );
     }
 
     /**
      * @return void
      */
-    protected function normalize(array $args, array $allowedAttributeKeys)
+    protected function normalize( array $args, array $allowedAttributeKeys )
     {
-        $this->attributes = array_change_key_case($args, CASE_LOWER);
+        $this->attributes = array_change_key_case( $args, CASE_LOWER );
         $this->normalizeBooleanAttributes();
         $this->normalizeDataAttributes();
         $this->normalizeStringAttributes();
@@ -189,7 +189,7 @@ 
         $this->removeIndexedAttributes();
         $this->attributes = array_merge(
             $this->filterGlobalAttributes(),
-            $this->filterAttributes($allowedAttributeKeys)
+            $this->filterAttributes( $allowedAttributeKeys )
         );
     }
 
@@ -198,15 +198,15 @@ 
      */
     protected function normalizeBooleanAttributes()
     {
-        foreach ($this->attributes as $key => $value) {
-            if ($this->isAttributeKeyNumeric($key, $value)) {
+        foreach( $this->attributes as $key => $value ) {
+            if( $this->isAttributeKeyNumeric( $key, $value ) ) {
                 $key = $value;
                 $value = true;
             }
-            if (!in_array($key, static::BOOLEAN_ATTRIBUTES)) {
+            if( !in_array( $key, static::BOOLEAN_ATTRIBUTES ) ) {
                 continue;
             }
-            $this->attributes[$key] = wp_validate_boolean($value);
+            $this->attributes[$key] = wp_validate_boolean( $value );
         }
     }
 
@@ -215,16 +215,16 @@ 
      */
     protected function normalizeDataAttributes()
     {
-        foreach ($this->attributes as $key => $value) {
-            if ($this->isAttributeKeyNumeric($key, $value)) {
+        foreach( $this->attributes as $key => $value ) {
+            if( $this->isAttributeKeyNumeric( $key, $value ) ) {
                 $key = $value;
                 $value = '';
             }
-            if (!glsr(Helper::class)->startsWith('data-', $key)) {
+            if( !glsr( Helper::class )->startsWith( 'data-', $key ) ) {
                 continue;
             }
-            if (is_array($value)) {
-                $value = json_encode($value, JSON_HEX_APOS | JSON_NUMERIC_CHECK | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE);
+            if( is_array( $value ) ) {
+                $value = json_encode( $value, JSON_HEX_APOS | JSON_NUMERIC_CHECK | JSON_UNESCAPED_SLASHES | JSON_UNESCAPED_UNICODE );
             }
             $this->attributes[$key] = $value;
         }
@@ -235,11 +235,11 @@ 
      */
     protected function normalizeStringAttributes()
     {
-        foreach ($this->attributes as $key => $value) {
-            if (!is_string($value)) {
+        foreach( $this->attributes as $key => $value ) {
+            if( !is_string( $value ) ) {
                 continue;
             }
-            $this->attributes[$key] = trim($value);
+            $this->attributes[$key] = trim( $value );
         }
     }
 
@@ -247,13 +247,13 @@ 
      * @param string $method
      * @return void
      */
-    protected function normalizeInputType($method)
+    protected function normalizeInputType( $method )
     {
-        if ('input' != $method) {
+        if( 'input' != $method ) {
             return;
         }
-        $attributes = wp_parse_args($this->attributes, ['type' => '']);
-        if (!in_array($attributes['type'], static::INPUT_TYPES)) {
+        $attributes = wp_parse_args( $this->attributes, ['type' => ''] );
+        if( !in_array( $attributes['type'], static::INPUT_TYPES ) ) {
             $this->attributes['type'] = 'text';
         }
     }
@@ -265,16 +265,16 @@ 
     {
         $attributes = $this->attributes;
         $permanentAttributes = $this->getPermanentAttributes();
-        foreach ($this->attributes as $key => $value) {
-            if (in_array($key, static::BOOLEAN_ATTRIBUTES) && !$value) {
+        foreach( $this->attributes as $key => $value ) {
+            if( in_array( $key, static::BOOLEAN_ATTRIBUTES ) && !$value ) {
                 unset($attributes[$key]);
             }
-            if (glsr(Helper::class)->startsWith('data-', $key)) {
+            if( glsr( Helper::class )->startsWith( 'data-', $key ) ) {
                 $permanentAttributes[$key] = $value;
                 unset($attributes[$key]);
             }
         }
-        $this->attributes = array_merge(array_filter($attributes), $permanentAttributes);
+        $this->attributes = array_merge( array_filter( $attributes ), $permanentAttributes );
     }
 
     /**
@@ -284,7 +284,7 @@ 
     {
         $this->attributes = array_diff_key(
             $this->attributes,
-            array_filter($this->attributes, 'is_numeric', ARRAY_FILTER_USE_KEY)
+            array_filter( $this->attributes, 'is_numeric', ARRAY_FILTER_USE_KEY )
         );
     }
 }

plugin/Modules/Html/Fields/Honeypot.php

Spacing +2 added lines, -2 removed lines

@@ -9,9 +9,9 @@
      */
     public function build()
     {
-        $this->builder->args = wp_parse_args($this->builder->args, [
+        $this->builder->args = wp_parse_args( $this->builder->args, [
             'name' => $this->builder->args['text'],
-        ]);
+        ] );
         $this->builder->tag = 'input';
         $this->mergeFieldArgs();
         return $this->builder->getOpeningTag();

plugin/Modules/Html/Fields/YesNo.php

Spacing +2 added lines, -2 removed lines

@@ -32,8 +32,8 @@
         return [
             'is_multi' => true,
             'options' => [
-                'no' => __('No', 'site-reviews'),
-                'yes' => __('Yes', 'site-reviews'),
+                'no' => __( 'No', 'site-reviews' ),
+                'yes' => __( 'Yes', 'site-reviews' ),
             ],
             'type' => 'radio',
         ];

plugin/Modules/Html/Fields/Field.php

Spacing +13 added lines, -13 removed lines

@@ -11,7 +11,7 @@ 
      */
     protected $builder;
 
-    public function __construct(Builder $builder)
+    public function __construct( Builder $builder )
     {
         $this->builder = $builder;
     }
@@ -21,7 +21,7 @@ 
      */
     public function build()
     {
-        glsr_log()->error('Build method is not implemented for '.get_class($this));
+        glsr_log()->error( 'Build method is not implemented for '.get_class( $this ) );
     }
 
     /**
@@ -35,14 +35,14 @@ 
     /**
      * @return array
      */
-    public static function merge(array $args)
+    public static function merge( array $args )
     {
         $merged = array_merge(
-            wp_parse_args($args, static::defaults()),
+            wp_parse_args( $args, static::defaults() ),
             static::required()
         );
-        $merged['class'] = implode(' ', static::mergedAttribute('class', ' ', $args));
-        $merged['style'] = implode(';', static::mergedAttribute('style', ';', $args));
+        $merged['class'] = implode( ' ', static::mergedAttribute( 'class', ' ', $args ) );
+        $merged['style'] = implode( ';', static::mergedAttribute( 'style', ';', $args ) );
         return $merged;
     }
 
@@ -51,13 +51,13 @@ 
      * @param string $key
      * @return array
      */
-    public static function mergedAttribute($key, $delimiter, array $args)
+    public static function mergedAttribute( $key, $delimiter, array $args )
     {
-        return array_filter(array_merge(
-            explode($delimiter, glsr_get($args, $key)),
-            explode($delimiter, glsr_get(static::defaults(), $key)),
-            explode($delimiter, glsr_get(static::required(), $key))
-        ));
+        return array_filter( array_merge(
+            explode( $delimiter, glsr_get( $args, $key ) ),
+            explode( $delimiter, glsr_get( static::defaults(), $key ) ),
+            explode( $delimiter, glsr_get( static::required(), $key ) )
+        ) );
     }
 
     /**
@@ -73,6 +73,6 @@ 
      */
     protected function mergeFieldArgs()
     {
-        $this->builder->args = static::merge($this->builder->args);
+        $this->builder->args = static::merge( $this->builder->args );
     }
 }

plugin/Modules/Html/Fields/Rating.php

Spacing +3 added lines, -3 removed lines

@@ -21,9 +21,9 @@
      */
     public static function required()
     {
-        $options = ['' => __('Select a Rating', 'site-reviews')];
-        foreach (range(glsr()->constant('MAX_RATING', RatingModule::class), 1) as $rating) {
-            $options[$rating] = sprintf(_n('%s Star', '%s Stars', $rating, 'site-reviews'), $rating);
+        $options = ['' => __( 'Select a Rating', 'site-reviews' )];
+        foreach( range( glsr()->constant( 'MAX_RATING', RatingModule::class ), 1 ) as $rating ) {
+            $options[$rating] = sprintf( _n( '%s Star', '%s Stars', $rating, 'site-reviews' ), $rating );
         }
         return [
             'class' => 'glsr-star-rating',

plugin/Modules/Html/Partial.php

Spacing +10 added lines, -10 removed lines

@@ -10,17 +10,17 @@ 
      * @param string $partialPath
      * @return string
      */
-    public function build($partialPath, array $args = [])
+    public function build( $partialPath, array $args = [] )
     {
-        $className = glsr(Helper::class)->buildClassName($partialPath, 'Modules\Html\Partials');
-        if (!class_exists($className)) {
-            glsr_log()->error('Partial missing: '.$className);
+        $className = glsr( Helper::class )->buildClassName( $partialPath, 'Modules\Html\Partials' );
+        if( !class_exists( $className ) ) {
+            glsr_log()->error( 'Partial missing: '.$className );
             return;
         }
-        $args = apply_filters('site-reviews/partial/args/'.$partialPath, $args);
-        $partial = glsr($className)->build($args);
-        $partial = apply_filters('site-reviews/rendered/partial', $partial, $partialPath, $args);
-        $partial = apply_filters('site-reviews/rendered/partial/'.$partialPath, $partial, $args);
+        $args = apply_filters( 'site-reviews/partial/args/'.$partialPath, $args );
+        $partial = glsr( $className )->build( $args );
+        $partial = apply_filters( 'site-reviews/rendered/partial', $partial, $partialPath, $args );
+        $partial = apply_filters( 'site-reviews/rendered/partial/'.$partialPath, $partial, $args );
         return $partial;
     }
 
@@ -28,8 +28,8 @@ 
      * @param string $partialPath
      * @return void
      */
-    public function render($partialPath, array $args = [])
+    public function render( $partialPath, array $args = [] )
     {
-        echo $this->build($partialPath, $args);
+        echo $this->build( $partialPath, $args );
     }
 }

plugin/Modules/Html/Form.php

Spacing +7 added lines, -7 removed lines

@@ -8,9 +8,9 @@ 
      * @param string $id
      * @return string
      */
-    public function buildFields($id)
+    public function buildFields( $id )
     {
-        return array_reduce($this->getFields($id), function ($carry, $field) {
+        return array_reduce( $this->getFields( $id ), function( $carry, $field ) {
             return $carry.$field;
         });
     }
@@ -19,11 +19,11 @@ 
      * @param string $id
      * @return array
      */
-    public function getFields($id)
+    public function getFields( $id )
     {
         $fields = [];
-        foreach (glsr()->config('forms/'.$id) as $name => $field) {
-            $fields[] = new Field(wp_parse_args($field, ['name' => $name]));
+        foreach( glsr()->config( 'forms/'.$id ) as $name => $field ) {
+            $fields[] = new Field( wp_parse_args( $field, ['name' => $name] ) );
         }
         return $fields;
     }
@@ -32,8 +32,8 @@ 
      * @param string $id
      * @return void
      */
-    public function renderFields($id)
+    public function renderFields( $id )
     {
-        echo $this->buildFields($id);
+        echo $this->buildFields( $id );
     }
 }

plugin/Modules/Html/Partials/StarRating.php

Spacing +3 added lines, -3 removed lines

@@ -9,14 +9,14 @@
     /**
      * @return void|string
      */
-    public function build(array $args = [])
+    public function build( array $args = [] )
     {
         require_once ABSPATH.'wp-admin/includes/template.php';
         ob_start();
-        wp_star_rating($args);
+        wp_star_rating( $args );
         $stars = ob_get_clean();
         return !glsr()->isAdmin()
-            ? str_replace(['star-rating', 'star star'], ['glsr-stars', 'glsr-star glsr-star'], $stars)
+            ? str_replace( ['star-rating', 'star star'], ['glsr-stars', 'glsr-star glsr-star'], $stars )
             : $stars;
     }
 }

plugin/Modules/Html/Partials/SiteReviewsSummary.php

Spacing +40 added lines, -40 removed lines

@@ -28,16 +28,16 @@ 
     /**
      * @return void|string
      */
-    public function build(array $args = [])
+    public function build( array $args = [] )
     {
         $this->args = $args;
-        $this->ratingCounts = glsr(ReviewManager::class)->getRatingCounts($args);
-        if (!array_sum($this->ratingCounts) && $this->isHidden('if_empty')) {
+        $this->ratingCounts = glsr( ReviewManager::class )->getRatingCounts( $args );
+        if( !array_sum( $this->ratingCounts ) && $this->isHidden( 'if_empty' ) ) {
             return;
         }
-        $this->averageRating = glsr(Rating::class)->getAverage($this->ratingCounts);
+        $this->averageRating = glsr( Rating::class )->getAverage( $this->ratingCounts );
         $this->generateSchema();
-        return glsr(Template::class)->build('templates/reviews-summary', [
+        return glsr( Template::class )->build( 'templates/reviews-summary', [
             'context' => [
                 'assigned_to' => $this->args['assigned_to'],
                 'category' => $this->args['category'],
@@ -48,7 +48,7 @@ 
                 'stars' => $this->buildStars(),
                 'text' => $this->buildText(),
             ],
-        ]);
+        ] );
     }
 
     /**
@@ -56,35 +56,35 @@ 
      */
     protected function buildPercentage()
     {
-        if ($this->isHidden('bars')) {
+        if( $this->isHidden( 'bars' ) ) {
             return;
         }
-        $percentages = preg_filter('/$/', '%', glsr(Rating::class)->getPercentages($this->ratingCounts));
-        $bars = array_reduce(range(glsr()->constant('MAX_RATING', Rating::class), 1), function ($carry, $level) use ($percentages) {
-            $label = $this->buildPercentageLabel($this->args['labels'][$level]);
-            $background = $this->buildPercentageBackground($percentages[$level]);
-            $count = apply_filters('site-reviews/summary/counts',
+        $percentages = preg_filter( '/$/', '%', glsr( Rating::class )->getPercentages( $this->ratingCounts ) );
+        $bars = array_reduce( range( glsr()->constant( 'MAX_RATING', Rating::class ), 1 ), function( $carry, $level ) use ($percentages) {
+            $label = $this->buildPercentageLabel( $this->args['labels'][$level] );
+            $background = $this->buildPercentageBackground( $percentages[$level] );
+            $count = apply_filters( 'site-reviews/summary/counts',
                 $percentages[$level],
                 $this->ratingCounts[$level]
             );
-            $percent = $this->buildPercentageCount($count);
-            return $carry.glsr(Builder::class)->div($label.$background.$percent, [
+            $percent = $this->buildPercentageCount( $count );
+            return $carry.glsr( Builder::class )->div( $label.$background.$percent, [
                 'class' => 'glsr-bar',
-            ]);
+            ] );
         });
-        return $this->wrap('percentage', $bars);
+        return $this->wrap( 'percentage', $bars );
     }
 
     /**
      * @param string $percent
      * @return string
      */
-    protected function buildPercentageBackground($percent)
+    protected function buildPercentageBackground( $percent )
     {
-        $backgroundPercent = glsr(Builder::class)->span([
+        $backgroundPercent = glsr( Builder::class )->span( [
             'class' => 'glsr-bar-background-percent',
             'style' => 'width:'.$percent,
-        ]);
+        ] );
         return '<span class="glsr-bar-background">'.$backgroundPercent.'</span>';
     }
 
@@ -92,7 +92,7 @@ 
      * @param string $count
      * @return string
      */
-    protected function buildPercentageCount($count)
+    protected function buildPercentageCount( $count )
     {
         return '<span class="glsr-bar-percent">'.$count.'</span>';
     }
@@ -101,7 +101,7 @@ 
      * @param string $label
      * @return string
      */
-    protected function buildPercentageLabel($label)
+    protected function buildPercentageLabel( $label )
     {
         return '<span class="glsr-bar-label">'.$label.'</span>';
     }
@@ -111,10 +111,10 @@ 
      */
     protected function buildRating()
     {
-        if ($this->isHidden('rating')) {
+        if( $this->isHidden( 'rating' ) ) {
             return;
         }
-        return $this->wrap('rating', '<span>'.$this->averageRating.'</span>');
+        return $this->wrap( 'rating', '<span>'.$this->averageRating.'</span>' );
     }
 
     /**
@@ -122,11 +122,11 @@ 
      */
     protected function buildStars()
     {
-        if ($this->isHidden('stars')) {
+        if( $this->isHidden( 'stars' ) ) {
             return;
         }
-        $stars = glsr_star_rating($this->averageRating);
-        return $this->wrap('stars', $stars);
+        $stars = glsr_star_rating( $this->averageRating );
+        return $this->wrap( 'stars', $stars );
     }
 
     /**
@@ -134,11 +134,11 @@ 
      */
     protected function buildText()
     {
-        if ($this->isHidden('summary')) {
+        if( $this->isHidden( 'summary' ) ) {
             return;
         }
-        $count = intval(array_sum($this->ratingCounts));
-        if (empty($this->args['text'])) {
+        $count = intval( array_sum( $this->ratingCounts ) );
+        if( empty($this->args['text']) ) {
             // @todo document this change
             $this->args['text'] = _nx(
                 '{rating} out of {max} stars (based on {num} review)',
@@ -150,10 +150,10 @@ 
         }
         $summary = str_replace(
             ['{rating}', '{max}', '{num}'],
-            [$this->averageRating, glsr()->constant('MAX_RATING', Rating::class), $count],
+            [$this->averageRating, glsr()->constant( 'MAX_RATING', Rating::class ), $count],
             $this->args['text']
         );
-        return $this->wrap('text', '<span>'.$summary.'</span>');
+        return $this->wrap( 'text', '<span>'.$summary.'</span>' );
     }
 
     /**
@@ -161,11 +161,11 @@ 
      */
     protected function generateSchema()
     {
-        if (!wp_validate_boolean($this->args['schema'])) {
+        if( !wp_validate_boolean( $this->args['schema'] ) ) {
             return;
         }
-        glsr(Schema::class)->store(
-            glsr(Schema::class)->buildSummary($this->args)
+        glsr( Schema::class )->store(
+            glsr( Schema::class )->buildSummary( $this->args )
         );
     }
 
@@ -174,16 +174,16 @@ 
      */
     protected function getClass()
     {
-        return trim('glsr-summary glsr-default '.$this->args['class']);
+        return trim( 'glsr-summary glsr-default '.$this->args['class'] );
     }
 
     /**
      * @param string $key
      * @return bool
      */
-    protected function isHidden($key)
+    protected function isHidden( $key )
     {
-        return in_array($key, $this->args['hide']);
+        return in_array( $key, $this->args['hide'] );
     }
 
     /**
@@ -191,10 +191,10 @@ 
      * @param string $value
      * @return string
      */
-    protected function wrap($key, $value)
+    protected function wrap( $key, $value )
     {
-        return glsr(Builder::class)->div($value, [
+        return glsr( Builder::class )->div( $value, [
             'class' => 'glsr-summary-'.$key,
-        ]);
+        ] );
     }
 }