pryley / site-reviews

plugin/Contracts/HooksContract.php

Indentation +4 added lines, -4 removed lines

@@ -4,8 +4,8 @@
 
 interface HooksContract
 {
-    /**
-     * @return void
-     */
-    public function run();
+	/**
+	 * @return void
+	 */
+	public function run();
 }

plugin/Contracts/ProviderContract.php

Indentation +4 added lines, -4 removed lines

@@ -6,8 +6,8 @@
 
 interface ProviderContract
 {
-    /**
-     * @return void
-     */
-    public function register(Application $app);
+	/**
+	 * @return void
+	 */
+	public function register(Application $app);
 }

Spacing +1 added lines, -1 removed lines

@@ -9,5 +9,5 @@
     /**
      * @return void
      */
-    public function register(Application $app);
+    public function register( Application $app );
 }

plugin/Container.php

Braces +6 added lines, -3 removed lines

@@ -68,9 +68,11 @@ 
     {
         if (!property_exists($this, $property) || in_array($property, static::PROTECTED_PROPERTIES)) {
             $this->storage[$property] = $value;
-        } elseif (!isset($this->$property)) {
+        }
+        elseif (!isset($this->$property)) {
             $this->$property = $value;
-        } else {
+        }
+        else {
             throw new Exception(sprintf('The "%s" property cannot be changed once set.', $property));
         }
     }
@@ -165,7 +167,8 @@ 
     {
         try {
             return $this->make($parameter->getClass()->name);
-        } catch (Exception $error) {
+        }
+        catch (Exception $error) {
             if ($parameter->isOptional()) {
                 return $parameter->getDefaultValue();
             }

Indentation +222 added lines, -222 removed lines

@@ -11,226 +11,226 @@
 
 abstract class Container
 {
-    const PROTECTED_PROPERTIES = [
-        'instance',
-        'services',
-        'session',
-        'storage',
-    ];
-
-    /**
-     * @var static
-     */
-    protected static $instance;
-
-    /**
-     * The container's bound services.
-     * @var array
-     */
-    protected $services = [];
-
-    /**
-     * @var array
-     */
-    protected $session = [];
-
-    /**
-     * The container's storage items.
-     * @var array
-     */
-    protected $storage = [];
-
-    /**
-     * @return static
-     */
-    public static function load()
-    {
-        if (empty(static::$instance)) {
-            static::$instance = new static();
-        }
-        return static::$instance;
-    }
-
-    /**
-     * @param string $property
-     * @return mixed
-     */
-    public function __get($property)
-    {
-        if (property_exists($this, $property) && !in_array($property, static::PROTECTED_PROPERTIES)) {
-            return $this->$property;
-        }
-        $constant = 'static::'.strtoupper(Str::snakeCase($property));
-        if (defined($constant)) {
-            return constant($constant);
-        }
-        return Arr::get($this->storage, $property, null);
-    }
-
-    /**
-     * @param string $property
-     * @param string $value
-     * @return void
-     */
-    public function __set($property, $value)
-    {
-        if (!property_exists($this, $property) || in_array($property, static::PROTECTED_PROPERTIES)) {
-            $this->storage[$property] = $value;
-        } elseif (!isset($this->$property)) {
-            $this->$property = $value;
-        } else {
-            throw new Exception(sprintf('The "%s" property cannot be changed once set.', $property));
-        }
-    }
-
-    /**
-     * Bind a service to the container.
-     * @param string $alias
-     * @param mixed $concrete
-     * @return mixed
-     */
-    public function bind($alias, $concrete)
-    {
-        $this->services[$alias] = $concrete;
-    }
-
-    /**
-     * Request a service from the container.
-     * @param mixed $abstract
-     * @return mixed
-     */
-    public function make($abstract)
-    {
-        if (!isset($this->services[$abstract])) {
-            $abstract = $this->addNamespace($abstract);
-        }
-        if (isset($this->services[$abstract])) {
-            $abstract = $this->services[$abstract];
-        }
-        if (is_callable($abstract)) {
-            return call_user_func_array($abstract, [$this]);
-        }
-        if (is_object($abstract)) {
-            return $abstract;
-        }
-        return $this->resolve($abstract);
-    }
-
-    /**
-     * @return void
-     */
-    public function sessionClear()
-    {
-        $this->session = [];
-    }
-
-    /**
-     * @return mixed
-     */
-    public function sessionGet($key, $fallback = '')
-    {
-        $value = Arr::get($this->session, $key, $fallback);
-        unset($this->session[$key]);
-        return $value;
-    }
-
-    /**
-     * @return void
-     */
-    public function sessionSet($key, $value)
-    {
-        $this->session[$key] = $value;
-    }
-
-    /**
-     * Bind a singleton instance to the container.
-     * @param string $alias
-     * @param callable|string|null $binding
-     * @return void
-     */
-    public function singleton($alias, $binding)
-    {
-        $this->bind($alias, $this->make($binding));
-    }
-
-    /**
-     * Prefix the current namespace to the abstract if absent.
-     * @param string $abstract
-     * @return string
-     */
-    protected function addNamespace($abstract)
-    {
-        if (!Str::contains($abstract, __NAMESPACE__) && !class_exists($abstract)) {
-            $abstract = __NAMESPACE__.'\\'.$abstract;
-        }
-        return $abstract;
-    }
-
-    /**
-     * Resolve a service from the container.
-     * @param mixed $concrete
-     * @return mixed
-     * @throws Exception
-     */
-    protected function resolve($concrete)
-    {
-        if ($concrete instanceof Closure) {
-            return $concrete($this);
-        }
-        $reflector = new ReflectionClass($concrete);
-        if (!$reflector->isInstantiable()) {
-            throw new Exception('Target ['.$concrete.'] is not instantiable.');
-        }
-        $constructor = $reflector->getConstructor();
-        if (empty($constructor)) {
-            return new $concrete();
-        }
-        return $reflector->newInstanceArgs(
-            $this->resolveDependencies($constructor->getParameters())
-        );
-    }
-
-    /**
-     * Resolve a class based dependency from the container.
-     * @return mixed
-     * @throws Exception
-     */
-    protected function resolveClass(ReflectionParameter $parameter)
-    {
-        try {
-            return $this->make($parameter->getClass()->name);
-        } catch (Exception $error) {
-            if ($parameter->isOptional()) {
-                return $parameter->getDefaultValue();
-            }
-            throw $error;
-        }
-    }
-
-    /**
-     * Resolve all of the dependencies from the ReflectionParameters.
-     * @return array
-     */
-    protected function resolveDependencies(array $dependencies)
-    {
-        $results = [];
-        foreach ($dependencies as $dependency) {
-            $results[] = !is_null($class = $dependency->getClass())
-                ? $this->resolveClass($dependency)
-                : $this->resolveDependency($dependency);
-        }
-        return $results;
-    }
-
-    /**
-     * Resolve a single ReflectionParameter dependency.
-     * @return array|null
-     */
-    protected function resolveDependency(ReflectionParameter $parameter)
-    {
-        if ($parameter->isArray() && $parameter->isDefaultValueAvailable()) {
-            return $parameter->getDefaultValue();
-        }
-        return null;
-    }
+	const PROTECTED_PROPERTIES = [
+		'instance',
+		'services',
+		'session',
+		'storage',
+	];
+
+	/**
+	 * @var static
+	 */
+	protected static $instance;
+
+	/**
+	 * The container's bound services.
+	 * @var array
+	 */
+	protected $services = [];
+
+	/**
+	 * @var array
+	 */
+	protected $session = [];
+
+	/**
+	 * The container's storage items.
+	 * @var array
+	 */
+	protected $storage = [];
+
+	/**
+	 * @return static
+	 */
+	public static function load()
+	{
+		if (empty(static::$instance)) {
+			static::$instance = new static();
+		}
+		return static::$instance;
+	}
+
+	/**
+	 * @param string $property
+	 * @return mixed
+	 */
+	public function __get($property)
+	{
+		if (property_exists($this, $property) && !in_array($property, static::PROTECTED_PROPERTIES)) {
+			return $this->$property;
+		}
+		$constant = 'static::'.strtoupper(Str::snakeCase($property));
+		if (defined($constant)) {
+			return constant($constant);
+		}
+		return Arr::get($this->storage, $property, null);
+	}
+
+	/**
+	 * @param string $property
+	 * @param string $value
+	 * @return void
+	 */
+	public function __set($property, $value)
+	{
+		if (!property_exists($this, $property) || in_array($property, static::PROTECTED_PROPERTIES)) {
+			$this->storage[$property] = $value;
+		} elseif (!isset($this->$property)) {
+			$this->$property = $value;
+		} else {
+			throw new Exception(sprintf('The "%s" property cannot be changed once set.', $property));
+		}
+	}
+
+	/**
+	 * Bind a service to the container.
+	 * @param string $alias
+	 * @param mixed $concrete
+	 * @return mixed
+	 */
+	public function bind($alias, $concrete)
+	{
+		$this->services[$alias] = $concrete;
+	}
+
+	/**
+	 * Request a service from the container.
+	 * @param mixed $abstract
+	 * @return mixed
+	 */
+	public function make($abstract)
+	{
+		if (!isset($this->services[$abstract])) {
+			$abstract = $this->addNamespace($abstract);
+		}
+		if (isset($this->services[$abstract])) {
+			$abstract = $this->services[$abstract];
+		}
+		if (is_callable($abstract)) {
+			return call_user_func_array($abstract, [$this]);
+		}
+		if (is_object($abstract)) {
+			return $abstract;
+		}
+		return $this->resolve($abstract);
+	}
+
+	/**
+	 * @return void
+	 */
+	public function sessionClear()
+	{
+		$this->session = [];
+	}
+
+	/**
+	 * @return mixed
+	 */
+	public function sessionGet($key, $fallback = '')
+	{
+		$value = Arr::get($this->session, $key, $fallback);
+		unset($this->session[$key]);
+		return $value;
+	}
+
+	/**
+	 * @return void
+	 */
+	public function sessionSet($key, $value)
+	{
+		$this->session[$key] = $value;
+	}
+
+	/**
+	 * Bind a singleton instance to the container.
+	 * @param string $alias
+	 * @param callable|string|null $binding
+	 * @return void
+	 */
+	public function singleton($alias, $binding)
+	{
+		$this->bind($alias, $this->make($binding));
+	}
+
+	/**
+	 * Prefix the current namespace to the abstract if absent.
+	 * @param string $abstract
+	 * @return string
+	 */
+	protected function addNamespace($abstract)
+	{
+		if (!Str::contains($abstract, __NAMESPACE__) && !class_exists($abstract)) {
+			$abstract = __NAMESPACE__.'\\'.$abstract;
+		}
+		return $abstract;
+	}
+
+	/**
+	 * Resolve a service from the container.
+	 * @param mixed $concrete
+	 * @return mixed
+	 * @throws Exception
+	 */
+	protected function resolve($concrete)
+	{
+		if ($concrete instanceof Closure) {
+			return $concrete($this);
+		}
+		$reflector = new ReflectionClass($concrete);
+		if (!$reflector->isInstantiable()) {
+			throw new Exception('Target ['.$concrete.'] is not instantiable.');
+		}
+		$constructor = $reflector->getConstructor();
+		if (empty($constructor)) {
+			return new $concrete();
+		}
+		return $reflector->newInstanceArgs(
+			$this->resolveDependencies($constructor->getParameters())
+		);
+	}
+
+	/**
+	 * Resolve a class based dependency from the container.
+	 * @return mixed
+	 * @throws Exception
+	 */
+	protected function resolveClass(ReflectionParameter $parameter)
+	{
+		try {
+			return $this->make($parameter->getClass()->name);
+		} catch (Exception $error) {
+			if ($parameter->isOptional()) {
+				return $parameter->getDefaultValue();
+			}
+			throw $error;
+		}
+	}
+
+	/**
+	 * Resolve all of the dependencies from the ReflectionParameters.
+	 * @return array
+	 */
+	protected function resolveDependencies(array $dependencies)
+	{
+		$results = [];
+		foreach ($dependencies as $dependency) {
+			$results[] = !is_null($class = $dependency->getClass())
+				? $this->resolveClass($dependency)
+				: $this->resolveDependency($dependency);
+		}
+		return $results;
+	}
+
+	/**
+	 * Resolve a single ReflectionParameter dependency.
+	 * @return array|null
+	 */
+	protected function resolveDependency(ReflectionParameter $parameter)
+	{
+		if ($parameter->isArray() && $parameter->isDefaultValueAvailable()) {
+			return $parameter->getDefaultValue();
+		}
+		return null;
+	}
 }

Spacing +46 added lines, -46 removed lines

@@ -45,7 +45,7 @@ 
      */
     public static function load()
     {
-        if (empty(static::$instance)) {
+        if( empty(static::$instance) ) {
             static::$instance = new static();
         }
         return static::$instance;
@@ -55,16 +55,16 @@ 
      * @param string $property
      * @return mixed
      */
-    public function __get($property)
+    public function __get( $property )
     {
-        if (property_exists($this, $property) && !in_array($property, static::PROTECTED_PROPERTIES)) {
+        if( property_exists( $this, $property ) && !in_array( $property, static::PROTECTED_PROPERTIES ) ) {
             return $this->$property;
         }
-        $constant = 'static::'.strtoupper(Str::snakeCase($property));
-        if (defined($constant)) {
-            return constant($constant);
+        $constant = 'static::'.strtoupper( Str::snakeCase( $property ) );
+        if( defined( $constant ) ) {
+            return constant( $constant );
         }
-        return Arr::get($this->storage, $property, null);
+        return Arr::get( $this->storage, $property, null );
     }
 
     /**
@@ -72,14 +72,14 @@ 
      * @param string $value
      * @return void
      */
-    public function __set($property, $value)
+    public function __set( $property, $value )
     {
-        if (!property_exists($this, $property) || in_array($property, static::PROTECTED_PROPERTIES)) {
+        if( !property_exists( $this, $property ) || in_array( $property, static::PROTECTED_PROPERTIES ) ) {
             $this->storage[$property] = $value;
-        } elseif (!isset($this->$property)) {
+        } elseif( !isset($this->$property) ) {
             $this->$property = $value;
         } else {
-            throw new Exception(sprintf('The "%s" property cannot be changed once set.', $property));
+            throw new Exception( sprintf( 'The "%s" property cannot be changed once set.', $property ) );
         }
     }
 
@@ -89,7 +89,7 @@ 
      * @param mixed $concrete
      * @return mixed
      */
-    public function bind($alias, $concrete)
+    public function bind( $alias, $concrete )
     {
         $this->services[$alias] = $concrete;
     }
@@ -99,21 +99,21 @@ 
      * @param mixed $abstract
      * @return mixed
      */
-    public function make($abstract)
+    public function make( $abstract )
     {
-        if (!isset($this->services[$abstract])) {
-            $abstract = $this->addNamespace($abstract);
+        if( !isset($this->services[$abstract]) ) {
+            $abstract = $this->addNamespace( $abstract );
         }
-        if (isset($this->services[$abstract])) {
+        if( isset($this->services[$abstract]) ) {
             $abstract = $this->services[$abstract];
         }
-        if (is_callable($abstract)) {
-            return call_user_func_array($abstract, [$this]);
+        if( is_callable( $abstract ) ) {
+            return call_user_func_array( $abstract, [$this] );
         }
-        if (is_object($abstract)) {
+        if( is_object( $abstract ) ) {
             return $abstract;
         }
-        return $this->resolve($abstract);
+        return $this->resolve( $abstract );
     }
 
     /**
@@ -127,9 +127,9 @@ 
     /**
      * @return mixed
      */
-    public function sessionGet($key, $fallback = '')
+    public function sessionGet( $key, $fallback = '' )
     {
-        $value = Arr::get($this->session, $key, $fallback);
+        $value = Arr::get( $this->session, $key, $fallback );
         unset($this->session[$key]);
         return $value;
     }
@@ -137,7 +137,7 @@ 
     /**
      * @return void
      */
-    public function sessionSet($key, $value)
+    public function sessionSet( $key, $value )
     {
         $this->session[$key] = $value;
     }
@@ -148,9 +148,9 @@ 
      * @param callable|string|null $binding
      * @return void
      */
-    public function singleton($alias, $binding)
+    public function singleton( $alias, $binding )
     {
-        $this->bind($alias, $this->make($binding));
+        $this->bind( $alias, $this->make( $binding ) );
     }
 
     /**
@@ -158,9 +158,9 @@ 
      * @param string $abstract
      * @return string
      */
-    protected function addNamespace($abstract)
+    protected function addNamespace( $abstract )
     {
-        if (!Str::contains($abstract, __NAMESPACE__) && !class_exists($abstract)) {
+        if( !Str::contains( $abstract, __NAMESPACE__ ) && !class_exists( $abstract ) ) {
             $abstract = __NAMESPACE__.'\\'.$abstract;
         }
         return $abstract;
@@ -172,21 +172,21 @@ 
      * @return mixed
      * @throws Exception
      */
-    protected function resolve($concrete)
+    protected function resolve( $concrete )
     {
-        if ($concrete instanceof Closure) {
-            return $concrete($this);
+        if( $concrete instanceof Closure ) {
+            return $concrete( $this );
         }
-        $reflector = new ReflectionClass($concrete);
-        if (!$reflector->isInstantiable()) {
-            throw new Exception('Target ['.$concrete.'] is not instantiable.');
+        $reflector = new ReflectionClass( $concrete );
+        if( !$reflector->isInstantiable() ) {
+            throw new Exception( 'Target ['.$concrete.'] is not instantiable.' );
         }
         $constructor = $reflector->getConstructor();
-        if (empty($constructor)) {
+        if( empty($constructor) ) {
             return new $concrete();
         }
         return $reflector->newInstanceArgs(
-            $this->resolveDependencies($constructor->getParameters())
+            $this->resolveDependencies( $constructor->getParameters() )
         );
     }
 
@@ -195,12 +195,12 @@ 
      * @return mixed
      * @throws Exception
      */
-    protected function resolveClass(ReflectionParameter $parameter)
+    protected function resolveClass( ReflectionParameter $parameter )
     {
         try {
-            return $this->make($parameter->getClass()->name);
-        } catch (Exception $error) {
-            if ($parameter->isOptional()) {
+            return $this->make( $parameter->getClass()->name );
+        } catch( Exception $error ) {
+            if( $parameter->isOptional() ) {
                 return $parameter->getDefaultValue();
             }
             throw $error;
@@ -211,13 +211,13 @@ 
      * Resolve all of the dependencies from the ReflectionParameters.
      * @return array
      */
-    protected function resolveDependencies(array $dependencies)
+    protected function resolveDependencies( array $dependencies )
     {
         $results = [];
-        foreach ($dependencies as $dependency) {
-            $results[] = !is_null($class = $dependency->getClass())
-                ? $this->resolveClass($dependency)
-                : $this->resolveDependency($dependency);
+        foreach( $dependencies as $dependency ) {
+            $results[] = !is_null( $class = $dependency->getClass() )
+                ? $this->resolveClass( $dependency )
+                : $this->resolveDependency( $dependency );
         }
         return $results;
     }
@@ -226,9 +226,9 @@ 
      * Resolve a single ReflectionParameter dependency.
      * @return array|null
      */
-    protected function resolveDependency(ReflectionParameter $parameter)
+    protected function resolveDependency( ReflectionParameter $parameter )
     {
-        if ($parameter->isArray() && $parameter->isDefaultValueAvailable()) {
+        if( $parameter->isArray() && $parameter->isDefaultValueAvailable() ) {
             return $parameter->getDefaultValue();
         }
         return null;

plugin/Handlers/RegisterTaxonomy.php

Indentation +8 added lines, -8 removed lines

@@ -7,12 +7,12 @@
 
 class RegisterTaxonomy
 {
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        register_taxonomy(Application::TAXONOMY, Application::POST_TYPE, $command->args);
-        register_taxonomy_for_object_type(Application::TAXONOMY, Application::POST_TYPE);
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		register_taxonomy(Application::TAXONOMY, Application::POST_TYPE, $command->args);
+		register_taxonomy_for_object_type(Application::TAXONOMY, Application::POST_TYPE);
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -10,9 +10,9 @@
     /**
      * @return void
      */
-    public function handle(Command $command)
+    public function handle( Command $command )
     {
-        register_taxonomy(Application::TAXONOMY, Application::POST_TYPE, $command->args);
-        register_taxonomy_for_object_type(Application::TAXONOMY, Application::POST_TYPE);
+        register_taxonomy( Application::TAXONOMY, Application::POST_TYPE, $command->args );
+        register_taxonomy_for_object_type( Application::TAXONOMY, Application::POST_TYPE );
     }
 }

plugin/Handlers/RegisterPostType.php

Indentation +12 added lines, -12 removed lines

@@ -6,16 +6,16 @@
 
 class RegisterPostType
 {
-    /**
-     * @return void
-     */
-    public function handle(Command $command)
-    {
-        if (!in_array($command->postType, get_post_types(['_builtin' => true]))) {
-            register_post_type($command->postType, $command->args);
-            glsr()->postTypeColumns = wp_parse_args(glsr()->postTypeColumns, [
-                $command->postType => $command->columns,
-            ]);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	public function handle(Command $command)
+	{
+		if (!in_array($command->postType, get_post_types(['_builtin' => true]))) {
+			register_post_type($command->postType, $command->args);
+			glsr()->postTypeColumns = wp_parse_args(glsr()->postTypeColumns, [
+				$command->postType => $command->columns,
+			]);
+		}
+	}
 }

Spacing +5 added lines, -5 removed lines

@@ -9,13 +9,13 @@
     /**
      * @return void
      */
-    public function handle(Command $command)
+    public function handle( Command $command )
     {
-        if (!in_array($command->postType, get_post_types(['_builtin' => true]))) {
-            register_post_type($command->postType, $command->args);
-            glsr()->postTypeColumns = wp_parse_args(glsr()->postTypeColumns, [
+        if( !in_array( $command->postType, get_post_types( ['_builtin' => true] ) ) ) {
+            register_post_type( $command->postType, $command->args );
+            glsr()->postTypeColumns = wp_parse_args( glsr()->postTypeColumns, [
                 $command->postType => $command->columns,
-            ]);
+            ] );
         }
     }
 }

plugin/Shortcodes/Shortcode.php

Braces +2 added lines, -1 removed lines

@@ -148,7 +148,8 @@
     {
         if ('parent_id' == $postId) {
             $postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
-        } elseif ('post_id' == $postId) {
+        }
+        elseif ('post_id' == $postId) {
             $postId = intval(get_the_ID());
         }
         return $postId;

Indentation +222 added lines, -222 removed lines

@@ -14,248 +14,248 @@
 
 abstract class Shortcode implements ShortcodeContract
 {
-    /**
-     * @var string
-     */
-    protected $partialName;
+	/**
+	 * @var string
+	 */
+	protected $partialName;
 
-    /**
-     * @var string
-     */
-    protected $shortcodeName;
+	/**
+	 * @var string
+	 */
+	protected $shortcodeName;
 
-    public function __construct()
-    {
-        $this->partialName = $this->getShortcodePartialName();
-        $this->shortcodeName = $this->getShortcodeName();
-    }
+	public function __construct()
+	{
+		$this->partialName = $this->getShortcodePartialName();
+		$this->shortcodeName = $this->getShortcodeName();
+	}
 
-    /**
-     * @param string|array $atts
-     * @param string $type
-     * @return string
-     */
-    public function build($atts, array $args = [], $type = 'shortcode')
-    {
-        $args = $this->normalizeArgs($args, $type);
-        $atts = $this->normalizeAtts($atts, $type);
-        $partial = glsr(Partial::class)->build($this->partialName, $atts);
-        if (!empty($atts['title'])) {
-            $atts = Arr::set($atts, 'title', $args['before_title'].$atts['title'].$args['after_title']);
-        }
-        $html = glsr(Builder::class)->div((string) $partial, [
-            'class' => 'glsr glsr-'.glsr(Style::class)->get(),
-            'data-shortcode' => Str::snakeCase($this->partialName),
-            'data-type' => $type,
-            'data-atts' => $atts['json'], // I prefer to have this attribute displayed last
-        ]);
-        return $args['before_widget'].$atts['title'].$html.$args['after_widget'];
-    }
+	/**
+	 * @param string|array $atts
+	 * @param string $type
+	 * @return string
+	 */
+	public function build($atts, array $args = [], $type = 'shortcode')
+	{
+		$args = $this->normalizeArgs($args, $type);
+		$atts = $this->normalizeAtts($atts, $type);
+		$partial = glsr(Partial::class)->build($this->partialName, $atts);
+		if (!empty($atts['title'])) {
+			$atts = Arr::set($atts, 'title', $args['before_title'].$atts['title'].$args['after_title']);
+		}
+		$html = glsr(Builder::class)->div((string) $partial, [
+			'class' => 'glsr glsr-'.glsr(Style::class)->get(),
+			'data-shortcode' => Str::snakeCase($this->partialName),
+			'data-type' => $type,
+			'data-atts' => $atts['json'], // I prefer to have this attribute displayed last
+		]);
+		return $args['before_widget'].$atts['title'].$html.$args['after_widget'];
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function buildBlock($atts = [])
-    {
-        return $this->build($atts, [], 'block');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function buildBlock($atts = [])
+	{
+		return $this->build($atts, [], 'block');
+	}
 
-    /**
-     * {@inheritdoc}
-     */
-    public function buildShortcode($atts = [])
-    {
-        return $this->build($atts, [], 'shortcode');
-    }
+	/**
+	 * {@inheritdoc}
+	 */
+	public function buildShortcode($atts = [])
+	{
+		return $this->build($atts, [], 'shortcode');
+	}
 
-    /**
-     * @return array
-     */
-    public function getDefaults($atts)
-    {
-        return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
-    }
+	/**
+	 * @return array
+	 */
+	public function getDefaults($atts)
+	{
+		return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
+	}
 
-    /**
-     * @return array
-     */
-    public function getHideOptions()
-    {
-        $options = $this->hideOptions();
-        return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
-    }
+	/**
+	 * @return array
+	 */
+	public function getHideOptions()
+	{
+		$options = $this->hideOptions();
+		return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
+	}
 
-    /**
-     * @return string
-     */
-    public function getShortClassName($replace = '', $search = 'Shortcode')
-    {
-        return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
-    }
+	/**
+	 * @return string
+	 */
+	public function getShortClassName($replace = '', $search = 'Shortcode')
+	{
+		return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
+	}
 
-    /**
-     * @return string
-     */
-    public function getShortcodeDefaultsClassName()
-    {
-        $className = Str::replaceLast('Shortcode', 'Defaults', get_class($this));
-        return str_replace('Shortcodes', 'Defaults', $className);
-    }
+	/**
+	 * @return string
+	 */
+	public function getShortcodeDefaultsClassName()
+	{
+		$className = Str::replaceLast('Shortcode', 'Defaults', get_class($this));
+		return str_replace('Shortcodes', 'Defaults', $className);
+	}
 
-    /**
-     * @return string
-     */
-    public function getShortcodeName()
-    {
-        return Str::snakeCase($this->getShortClassName());
-    }
+	/**
+	 * @return string
+	 */
+	public function getShortcodeName()
+	{
+		return Str::snakeCase($this->getShortClassName());
+	}
 
-    /**
-     * @return string
-     */
-    public function getShortcodePartialName()
-    {
-        return Str::dashCase($this->getShortClassName());
-    }
+	/**
+	 * @return string
+	 */
+	public function getShortcodePartialName()
+	{
+		return Str::dashCase($this->getShortClassName());
+	}
 
-    /**
-     * @param array|string $args
-     * @param string $type
-     * @return array
-     */
-    public function normalizeArgs($args, $type = 'shortcode')
-    {
-        $args = wp_parse_args($args, [
-            'before_widget' => '',
-            'after_widget' => '',
-            'before_title' => '<h2 class="glsr-title">',
-            'after_title' => '</h2>',
-        ]);
-        return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
-    }
+	/**
+	 * @param array|string $args
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeArgs($args, $type = 'shortcode')
+	{
+		$args = wp_parse_args($args, [
+			'before_widget' => '',
+			'after_widget' => '',
+			'before_title' => '<h2 class="glsr-title">',
+			'after_title' => '</h2>',
+		]);
+		return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
+	}
 
-    /**
-     * @param array|string $atts
-     * @param string $type
-     * @return array
-     */
-    public function normalizeAtts($atts, $type = 'shortcode')
-    {
-        $atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
-        $atts = $this->getDefaults($atts);
-        array_walk($atts, function (&$value, $key) {
-            $methodName = Helper::buildMethodName($key, 'normalize');
-            if (!method_exists($this, $methodName)) {
-                return;
-            }
-            $value = $this->$methodName($value);
-        });
-        $this->setId($atts);
-        return $atts;
-    }
+	/**
+	 * @param array|string $atts
+	 * @param string $type
+	 * @return array
+	 */
+	public function normalizeAtts($atts, $type = 'shortcode')
+	{
+		$atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
+		$atts = $this->getDefaults($atts);
+		array_walk($atts, function (&$value, $key) {
+			$methodName = Helper::buildMethodName($key, 'normalize');
+			if (!method_exists($this, $methodName)) {
+				return;
+			}
+			$value = $this->$methodName($value);
+		});
+		$this->setId($atts);
+		return $atts;
+	}
 
-    /**
-     * @return array
-     */
-    abstract protected function hideOptions();
+	/**
+	 * @return array
+	 */
+	abstract protected function hideOptions();
 
-    /**
-     * @param string $postId
-     * @return int|string
-     */
-    protected function normalizeAssignedTo($postId)
-    {
-        if ('parent_id' == $postId) {
-            $postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
-        } elseif ('post_id' == $postId) {
-            $postId = intval(get_the_ID());
-        }
-        return $postId;
-    }
+	/**
+	 * @param string $postId
+	 * @return int|string
+	 */
+	protected function normalizeAssignedTo($postId)
+	{
+		if ('parent_id' == $postId) {
+			$postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
+		} elseif ('post_id' == $postId) {
+			$postId = intval(get_the_ID());
+		}
+		return $postId;
+	}
 
-    /**
-     * @param string $postId
-     * @return int|string
-     */
-    protected function normalizeAssignTo($postId)
-    {
-        return $this->normalizeAssignedTo($postId);
-    }
+	/**
+	 * @param string $postId
+	 * @return int|string
+	 */
+	protected function normalizeAssignTo($postId)
+	{
+		return $this->normalizeAssignedTo($postId);
+	}
 
-    /**
-     * @param string|array $hide
-     * @return array
-     */
-    protected function normalizeHide($hide)
-    {
-        if (is_string($hide)) {
-            $hide = explode(',', $hide);
-        }
-        $hideKeys = array_keys($this->getHideOptions());
-        return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
-            return in_array($value, $hideKeys);
-        });
-    }
+	/**
+	 * @param string|array $hide
+	 * @return array
+	 */
+	protected function normalizeHide($hide)
+	{
+		if (is_string($hide)) {
+			$hide = explode(',', $hide);
+		}
+		$hideKeys = array_keys($this->getHideOptions());
+		return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
+			return in_array($value, $hideKeys);
+		});
+	}
 
-    /**
-     * @param string $id
-     * @return string
-     */
-    protected function normalizeId($id)
-    {
-        return sanitize_title($id);
-    }
+	/**
+	 * @param string $id
+	 * @return string
+	 */
+	protected function normalizeId($id)
+	{
+		return sanitize_title($id);
+	}
 
-    /**
-     * @param string $labels
-     * @return array
-     */
-    protected function normalizeLabels($labels)
-    {
-        $defaults = [
-            __('Excellent', 'site-reviews'),
-            __('Very good', 'site-reviews'),
-            __('Average', 'site-reviews'),
-            __('Poor', 'site-reviews'),
-            __('Terrible', 'site-reviews'),
-        ];
-        $maxRating = (int) glsr()->constant('MAX_RATING', Rating::class);
-        $defaults = array_pad(array_slice($defaults, 0, $maxRating), $maxRating, '');
-        $labels = array_map('trim', explode(',', $labels));
-        foreach ($defaults as $i => $label) {
-            if (empty($labels[$i])) {
-                continue;
-            }
-            $defaults[$i] = $labels[$i];
-        }
-        return array_combine(range($maxRating, 1), $defaults);
-    }
+	/**
+	 * @param string $labels
+	 * @return array
+	 */
+	protected function normalizeLabels($labels)
+	{
+		$defaults = [
+			__('Excellent', 'site-reviews'),
+			__('Very good', 'site-reviews'),
+			__('Average', 'site-reviews'),
+			__('Poor', 'site-reviews'),
+			__('Terrible', 'site-reviews'),
+		];
+		$maxRating = (int) glsr()->constant('MAX_RATING', Rating::class);
+		$defaults = array_pad(array_slice($defaults, 0, $maxRating), $maxRating, '');
+		$labels = array_map('trim', explode(',', $labels));
+		foreach ($defaults as $i => $label) {
+			if (empty($labels[$i])) {
+				continue;
+			}
+			$defaults[$i] = $labels[$i];
+		}
+		return array_combine(range($maxRating, 1), $defaults);
+	}
 
-    /**
-     * @param string $schema
-     * @return bool
-     */
-    protected function normalizeSchema($schema)
-    {
-        return wp_validate_boolean($schema);
-    }
+	/**
+	 * @param string $schema
+	 * @return bool
+	 */
+	protected function normalizeSchema($schema)
+	{
+		return wp_validate_boolean($schema);
+	}
 
-    /**
-     * @param string $text
-     * @return string
-     */
-    protected function normalizeText($text)
-    {
-        return trim($text);
-    }
+	/**
+	 * @param string $text
+	 * @return string
+	 */
+	protected function normalizeText($text)
+	{
+		return trim($text);
+	}
 
-    /**
-     * @return void
-     */
-    protected function setId(array &$atts)
-    {
-        if (empty($atts['id'])) {
-            $atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
-        }
-    }
+	/**
+	 * @return void
+	 */
+	protected function setId(array &$atts)
+	{
+		if (empty($atts['id'])) {
+			$atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
+		}
+	}
 }

Spacing +69 added lines, -69 removed lines

@@ -35,45 +35,45 @@ 
      * @param string $type
      * @return string
      */
-    public function build($atts, array $args = [], $type = 'shortcode')
+    public function build( $atts, array $args = [], $type = 'shortcode' )
     {
-        $args = $this->normalizeArgs($args, $type);
-        $atts = $this->normalizeAtts($atts, $type);
-        $partial = glsr(Partial::class)->build($this->partialName, $atts);
-        if (!empty($atts['title'])) {
-            $atts = Arr::set($atts, 'title', $args['before_title'].$atts['title'].$args['after_title']);
+        $args = $this->normalizeArgs( $args, $type );
+        $atts = $this->normalizeAtts( $atts, $type );
+        $partial = glsr( Partial::class )->build( $this->partialName, $atts );
+        if( !empty($atts['title']) ) {
+            $atts = Arr::set( $atts, 'title', $args['before_title'].$atts['title'].$args['after_title'] );
         }
-        $html = glsr(Builder::class)->div((string) $partial, [
-            'class' => 'glsr glsr-'.glsr(Style::class)->get(),
-            'data-shortcode' => Str::snakeCase($this->partialName),
+        $html = glsr( Builder::class )->div( (string)$partial, [
+            'class' => 'glsr glsr-'.glsr( Style::class )->get(),
+            'data-shortcode' => Str::snakeCase( $this->partialName ),
             'data-type' => $type,
             'data-atts' => $atts['json'], // I prefer to have this attribute displayed last
-        ]);
+        ] );
         return $args['before_widget'].$atts['title'].$html.$args['after_widget'];
     }
 
     /**
      * {@inheritdoc}
      */
-    public function buildBlock($atts = [])
+    public function buildBlock( $atts = [] )
     {
-        return $this->build($atts, [], 'block');
+        return $this->build( $atts, [], 'block' );
     }
 
     /**
      * {@inheritdoc}
      */
-    public function buildShortcode($atts = [])
+    public function buildShortcode( $atts = [] )
     {
-        return $this->build($atts, [], 'shortcode');
+        return $this->build( $atts, [], 'shortcode' );
     }
 
     /**
      * @return array
      */
-    public function getDefaults($atts)
+    public function getDefaults( $atts )
     {
-        return glsr($this->getShortcodeDefaultsClassName())->restrict(wp_parse_args($atts));
+        return glsr( $this->getShortcodeDefaultsClassName() )->restrict( wp_parse_args( $atts ) );
     }
 
     /**
@@ -82,15 +82,15 @@ 
     public function getHideOptions()
     {
         $options = $this->hideOptions();
-        return apply_filters('site-reviews/shortcode/hide-options', $options, $this->shortcodeName);
+        return apply_filters( 'site-reviews/shortcode/hide-options', $options, $this->shortcodeName );
     }
 
     /**
      * @return string
      */
-    public function getShortClassName($replace = '', $search = 'Shortcode')
+    public function getShortClassName( $replace = '', $search = 'Shortcode' )
     {
-        return str_replace($search, $replace, (new ReflectionClass($this))->getShortName());
+        return str_replace( $search, $replace, (new ReflectionClass( $this ))->getShortName() );
     }
 
     /**
@@ -98,8 +98,8 @@ 
      */
     public function getShortcodeDefaultsClassName()
     {
-        $className = Str::replaceLast('Shortcode', 'Defaults', get_class($this));
-        return str_replace('Shortcodes', 'Defaults', $className);
+        $className = Str::replaceLast( 'Shortcode', 'Defaults', get_class( $this ) );
+        return str_replace( 'Shortcodes', 'Defaults', $className );
     }
 
     /**
@@ -107,7 +107,7 @@ 
      */
     public function getShortcodeName()
     {
-        return Str::snakeCase($this->getShortClassName());
+        return Str::snakeCase( $this->getShortClassName() );
     }
 
     /**
@@ -115,7 +115,7 @@ 
      */
     public function getShortcodePartialName()
     {
-        return Str::dashCase($this->getShortClassName());
+        return Str::dashCase( $this->getShortClassName() );
     }
 
     /**
@@ -123,15 +123,15 @@ 
      * @param string $type
      * @return array
      */
-    public function normalizeArgs($args, $type = 'shortcode')
+    public function normalizeArgs( $args, $type = 'shortcode' )
     {
-        $args = wp_parse_args($args, [
+        $args = wp_parse_args( $args, [
             'before_widget' => '',
             'after_widget' => '',
             'before_title' => '<h2 class="glsr-title">',
             'after_title' => '</h2>',
-        ]);
-        return apply_filters('site-reviews/shortcode/args', $args, $type, $this->partialName);
+        ] );
+        return apply_filters( 'site-reviews/shortcode/args', $args, $type, $this->partialName );
     }
 
     /**
@@ -139,18 +139,18 @@ 
      * @param string $type
      * @return array
      */
-    public function normalizeAtts($atts, $type = 'shortcode')
+    public function normalizeAtts( $atts, $type = 'shortcode' )
     {
-        $atts = apply_filters('site-reviews/shortcode/atts', $atts, $type, $this->partialName);
-        $atts = $this->getDefaults($atts);
-        array_walk($atts, function (&$value, $key) {
-            $methodName = Helper::buildMethodName($key, 'normalize');
-            if (!method_exists($this, $methodName)) {
+        $atts = apply_filters( 'site-reviews/shortcode/atts', $atts, $type, $this->partialName );
+        $atts = $this->getDefaults( $atts );
+        array_walk( $atts, function( &$value, $key ) {
+            $methodName = Helper::buildMethodName( $key, 'normalize' );
+            if( !method_exists( $this, $methodName ) ) {
                 return;
             }
-            $value = $this->$methodName($value);
+            $value = $this->$methodName( $value );
         });
-        $this->setId($atts);
+        $this->setId( $atts );
         return $atts;
     }
 
@@ -163,12 +163,12 @@ 
      * @param string $postId
      * @return int|string
      */
-    protected function normalizeAssignedTo($postId)
+    protected function normalizeAssignedTo( $postId )
     {
-        if ('parent_id' == $postId) {
-            $postId = intval(wp_get_post_parent_id(intval(get_the_ID())));
-        } elseif ('post_id' == $postId) {
-            $postId = intval(get_the_ID());
+        if( 'parent_id' == $postId ) {
+            $postId = intval( wp_get_post_parent_id( intval( get_the_ID() ) ) );
+        } elseif( 'post_id' == $postId ) {
+            $postId = intval( get_the_ID() );
         }
         return $postId;
     }
@@ -177,23 +177,23 @@ 
      * @param string $postId
      * @return int|string
      */
-    protected function normalizeAssignTo($postId)
+    protected function normalizeAssignTo( $postId )
     {
-        return $this->normalizeAssignedTo($postId);
+        return $this->normalizeAssignedTo( $postId );
     }
 
     /**
      * @param string|array $hide
      * @return array
      */
-    protected function normalizeHide($hide)
+    protected function normalizeHide( $hide )
     {
-        if (is_string($hide)) {
-            $hide = explode(',', $hide);
+        if( is_string( $hide ) ) {
+            $hide = explode( ',', $hide );
         }
-        $hideKeys = array_keys($this->getHideOptions());
-        return array_filter(array_map('trim', $hide), function ($value) use ($hideKeys) {
-            return in_array($value, $hideKeys);
+        $hideKeys = array_keys( $this->getHideOptions() );
+        return array_filter( array_map( 'trim', $hide ), function( $value ) use ($hideKeys) {
+            return in_array( $value, $hideKeys );
         });
     }
 
@@ -201,61 +201,61 @@ 
      * @param string $id
      * @return string
      */
-    protected function normalizeId($id)
+    protected function normalizeId( $id )
     {
-        return sanitize_title($id);
+        return sanitize_title( $id );
     }
 
     /**
      * @param string $labels
      * @return array
      */
-    protected function normalizeLabels($labels)
+    protected function normalizeLabels( $labels )
     {
         $defaults = [
-            __('Excellent', 'site-reviews'),
-            __('Very good', 'site-reviews'),
-            __('Average', 'site-reviews'),
-            __('Poor', 'site-reviews'),
-            __('Terrible', 'site-reviews'),
+            __( 'Excellent', 'site-reviews' ),
+            __( 'Very good', 'site-reviews' ),
+            __( 'Average', 'site-reviews' ),
+            __( 'Poor', 'site-reviews' ),
+            __( 'Terrible', 'site-reviews' ),
         ];
-        $maxRating = (int) glsr()->constant('MAX_RATING', Rating::class);
-        $defaults = array_pad(array_slice($defaults, 0, $maxRating), $maxRating, '');
-        $labels = array_map('trim', explode(',', $labels));
-        foreach ($defaults as $i => $label) {
-            if (empty($labels[$i])) {
+        $maxRating = (int)glsr()->constant( 'MAX_RATING', Rating::class );
+        $defaults = array_pad( array_slice( $defaults, 0, $maxRating ), $maxRating, '' );
+        $labels = array_map( 'trim', explode( ',', $labels ) );
+        foreach( $defaults as $i => $label ) {
+            if( empty($labels[$i]) ) {
                 continue;
             }
             $defaults[$i] = $labels[$i];
         }
-        return array_combine(range($maxRating, 1), $defaults);
+        return array_combine( range( $maxRating, 1 ), $defaults );
     }
 
     /**
      * @param string $schema
      * @return bool
      */
-    protected function normalizeSchema($schema)
+    protected function normalizeSchema( $schema )
     {
-        return wp_validate_boolean($schema);
+        return wp_validate_boolean( $schema );
     }
 
     /**
      * @param string $text
      * @return string
      */
-    protected function normalizeText($text)
+    protected function normalizeText( $text )
     {
-        return trim($text);
+        return trim( $text );
     }
 
     /**
      * @return void
      */
-    protected function setId(array &$atts)
+    protected function setId( array &$atts )
     {
-        if (empty($atts['id'])) {
-            $atts['id'] = Application::PREFIX.substr(md5(serialize($atts)), 0, 8);
+        if( empty($atts['id']) ) {
+            $atts['id'] = Application::PREFIX.substr( md5( serialize( $atts ) ), 0, 8 );
         }
     }
 }

plugin/Shortcodes/SiteReviewsFormPopup.php

Indentation +46 added lines, -46 removed lines

@@ -4,50 +4,50 @@
 
 class SiteReviewsFormPopup extends TinymcePopupGenerator
 {
-    /**
-     * @return array
-     */
-    public function fields()
-    {
-        return [[
-            'type' => 'container',
-            'html' => '<p class="strong">'.esc_html__('All settings are optional.', 'site-reviews').'</p>',
-        ], [
-            'label' => esc_html__('Title', 'site-reviews'),
-            'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Description', 'site-reviews'),
-            'minHeight' => 60,
-            'minWidth' => 240,
-            'multiline' => true,
-            'name' => 'description',
-            'tooltip' => __('Enter a custom shortcode description.', 'site-reviews'),
-            'type' => 'textbox',
-        ],
-        $this->getCategories(__('Automatically assign a category to reviews submitted with this shortcode.', 'site-reviews')),
-        [
-            'label' => esc_html__('Assign To', 'site-reviews'),
-            'name' => 'assign_to',
-            'tooltip' => __('Assign submitted reviews to a custom page/post ID. You can also enter "post_id" to assign reviews to the ID of the current page.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
-            'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'columns' => 2,
-            'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
-            'layout' => 'grid',
-            'spacing' => 5,
-            'type' => 'container',
-        ], [
-            'hidden' => true,
-            'name' => 'id',
-            'type' => 'textbox',
-        ], ];
-    }
+	/**
+	 * @return array
+	 */
+	public function fields()
+	{
+		return [[
+			'type' => 'container',
+			'html' => '<p class="strong">'.esc_html__('All settings are optional.', 'site-reviews').'</p>',
+		], [
+			'label' => esc_html__('Title', 'site-reviews'),
+			'name' => 'title',
+			'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Description', 'site-reviews'),
+			'minHeight' => 60,
+			'minWidth' => 240,
+			'multiline' => true,
+			'name' => 'description',
+			'tooltip' => __('Enter a custom shortcode description.', 'site-reviews'),
+			'type' => 'textbox',
+		],
+		$this->getCategories(__('Automatically assign a category to reviews submitted with this shortcode.', 'site-reviews')),
+		[
+			'label' => esc_html__('Assign To', 'site-reviews'),
+			'name' => 'assign_to',
+			'tooltip' => __('Assign submitted reviews to a custom page/post ID. You can also enter "post_id" to assign reviews to the ID of the current page.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Classes', 'site-reviews'),
+			'name' => 'class',
+			'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'columns' => 2,
+			'items' => $this->getHideOptions(),
+			'label' => esc_html__('Hide', 'site-reviews'),
+			'layout' => 'grid',
+			'spacing' => 5,
+			'type' => 'container',
+		], [
+			'hidden' => true,
+			'name' => 'id',
+			'type' => 'textbox',
+		], ];
+	}
 }

Spacing +11 added lines, -11 removed lines

@@ -11,36 +11,36 @@
     {
         return [[
             'type' => 'container',
-            'html' => '<p class="strong">'.esc_html__('All settings are optional.', 'site-reviews').'</p>',
+            'html' => '<p class="strong">'.esc_html__( 'All settings are optional.', 'site-reviews' ).'</p>',
         ], [
-            'label' => esc_html__('Title', 'site-reviews'),
+            'label' => esc_html__( 'Title', 'site-reviews' ),
             'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+            'tooltip' => __( 'Enter a custom shortcode heading.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Description', 'site-reviews'),
+            'label' => esc_html__( 'Description', 'site-reviews' ),
             'minHeight' => 60,
             'minWidth' => 240,
             'multiline' => true,
             'name' => 'description',
-            'tooltip' => __('Enter a custom shortcode description.', 'site-reviews'),
+            'tooltip' => __( 'Enter a custom shortcode description.', 'site-reviews' ),
             'type' => 'textbox',
         ],
-        $this->getCategories(__('Automatically assign a category to reviews submitted with this shortcode.', 'site-reviews')),
+        $this->getCategories( __( 'Automatically assign a category to reviews submitted with this shortcode.', 'site-reviews' ) ),
         [
-            'label' => esc_html__('Assign To', 'site-reviews'),
+            'label' => esc_html__( 'Assign To', 'site-reviews' ),
             'name' => 'assign_to',
-            'tooltip' => __('Assign submitted reviews to a custom page/post ID. You can also enter "post_id" to assign reviews to the ID of the current page.', 'site-reviews'),
+            'tooltip' => __( 'Assign submitted reviews to a custom page/post ID. You can also enter "post_id" to assign reviews to the ID of the current page.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
+            'label' => esc_html__( 'Classes', 'site-reviews' ),
             'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+            'tooltip' => __( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
             'columns' => 2,
             'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
+            'label' => esc_html__( 'Hide', 'site-reviews' ),
             'layout' => 'grid',
             'spacing' => 5,
             'type' => 'container',

plugin/Shortcodes/SiteReviewsSummaryPopup.php

Indentation +45 added lines, -45 removed lines

@@ -4,49 +4,49 @@
 
 class SiteReviewsSummaryPopup extends SiteReviewsPopup
 {
-    /**
-     * @return array
-     */
-    public function fields()
-    {
-        return [[
-            'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
-            'minWidth' => 320,
-            'type' => 'container',
-        ], [
-            'label' => esc_html__('Title', 'site-reviews'),
-            'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
-            'type' => 'textbox',
-        ],
-        $this->getTypes(__('Which type of review would you like to use?', 'site-reviews')),
-        $this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
-        [
-            'label' => esc_html__('Assigned To', 'site-reviews'),
-            'name' => 'assigned_to',
-            'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'label' => esc_html__('Schema', 'site-reviews'),
-            'name' => 'schema',
-            'options' => [
-                'true' => esc_html__('Enable rich snippets', 'site-reviews'),
-                'false' => esc_html__('Disable rich snippets', 'site-reviews'),
-            ],
-            'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
-            'type' => 'listbox',
-        ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
-            'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
-            'type' => 'textbox',
-        ], [
-            'columns' => 2,
-            'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
-            'layout' => 'grid',
-            'spacing' => 5,
-            'type' => 'container',
-        ], ];
-    }
+	/**
+	 * @return array
+	 */
+	public function fields()
+	{
+		return [[
+			'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
+			'minWidth' => 320,
+			'type' => 'container',
+		], [
+			'label' => esc_html__('Title', 'site-reviews'),
+			'name' => 'title',
+			'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+			'type' => 'textbox',
+		],
+		$this->getTypes(__('Which type of review would you like to use?', 'site-reviews')),
+		$this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
+		[
+			'label' => esc_html__('Assigned To', 'site-reviews'),
+			'name' => 'assigned_to',
+			'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'label' => esc_html__('Schema', 'site-reviews'),
+			'name' => 'schema',
+			'options' => [
+				'true' => esc_html__('Enable rich snippets', 'site-reviews'),
+				'false' => esc_html__('Disable rich snippets', 'site-reviews'),
+			],
+			'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
+			'type' => 'listbox',
+		], [
+			'label' => esc_html__('Classes', 'site-reviews'),
+			'name' => 'class',
+			'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+			'type' => 'textbox',
+		], [
+			'columns' => 2,
+			'items' => $this->getHideOptions(),
+			'label' => esc_html__('Hide', 'site-reviews'),
+			'layout' => 'grid',
+			'spacing' => 5,
+			'type' => 'container',
+		], ];
+	}
 }

Spacing +14 added lines, -14 removed lines

@@ -10,40 +10,40 @@
     public function fields()
     {
         return [[
-            'html' => sprintf('<p class="strong">%s</p>', esc_html__('All settings are optional.', 'site-reviews')),
+            'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'All settings are optional.', 'site-reviews' ) ),
             'minWidth' => 320,
             'type' => 'container',
         ], [
-            'label' => esc_html__('Title', 'site-reviews'),
+            'label' => esc_html__( 'Title', 'site-reviews' ),
             'name' => 'title',
-            'tooltip' => __('Enter a custom shortcode heading.', 'site-reviews'),
+            'tooltip' => __( 'Enter a custom shortcode heading.', 'site-reviews' ),
             'type' => 'textbox',
         ],
-        $this->getTypes(__('Which type of review would you like to use?', 'site-reviews')),
-        $this->getCategories(__('Limit reviews to this category.', 'site-reviews')),
+        $this->getTypes( __( 'Which type of review would you like to use?', 'site-reviews' ) ),
+        $this->getCategories( __( 'Limit reviews to this category.', 'site-reviews' ) ),
         [
-            'label' => esc_html__('Assigned To', 'site-reviews'),
+            'label' => esc_html__( 'Assigned To', 'site-reviews' ),
             'name' => 'assigned_to',
-            'tooltip' => __('Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews'),
+            'tooltip' => __( 'Limit reviews to those assigned to this post ID (separate multiple IDs with a comma). You can also enter "post_id" to use the ID of the current page, or "parent_id" to use the ID of the parent page.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
-            'label' => esc_html__('Schema', 'site-reviews'),
+            'label' => esc_html__( 'Schema', 'site-reviews' ),
             'name' => 'schema',
             'options' => [
-                'true' => esc_html__('Enable rich snippets', 'site-reviews'),
-                'false' => esc_html__('Disable rich snippets', 'site-reviews'),
+                'true' => esc_html__( 'Enable rich snippets', 'site-reviews' ),
+                'false' => esc_html__( 'Disable rich snippets', 'site-reviews' ),
             ],
-            'tooltip' => __('Rich snippets are disabled by default.', 'site-reviews'),
+            'tooltip' => __( 'Rich snippets are disabled by default.', 'site-reviews' ),
             'type' => 'listbox',
         ], [
-            'label' => esc_html__('Classes', 'site-reviews'),
+            'label' => esc_html__( 'Classes', 'site-reviews' ),
             'name' => 'class',
-            'tooltip' => __('Add custom CSS classes to the shortcode.', 'site-reviews'),
+            'tooltip' => __( 'Add custom CSS classes to the shortcode.', 'site-reviews' ),
             'type' => 'textbox',
         ], [
             'columns' => 2,
             'items' => $this->getHideOptions(),
-            'label' => esc_html__('Hide', 'site-reviews'),
+            'label' => esc_html__( 'Hide', 'site-reviews' ),
             'layout' => 'grid',
             'spacing' => 5,
             'type' => 'container',

plugin/Shortcodes/SiteReviewsShortcode.php

Indentation +13 added lines, -13 removed lines

@@ -4,17 +4,17 @@
 
 class SiteReviewsShortcode extends Shortcode
 {
-    protected function hideOptions()
-    {
-        return [
-            'title' => __('Hide the title', 'site-reviews'),
-            'rating' => __('Hide the rating', 'site-reviews'),
-            'date' => __('Hide the date', 'site-reviews'),
-            'assigned_to' => __('Hide the assigned to link (if shown)', 'site-reviews'),
-            'content' => __('Hide the content', 'site-reviews'),
-            'avatar' => __('Hide the avatar (if shown)', 'site-reviews'),
-            'author' => __('Hide the author', 'site-reviews'),
-            'response' => __('Hide the response', 'site-reviews'),
-        ];
-    }
+	protected function hideOptions()
+	{
+		return [
+			'title' => __('Hide the title', 'site-reviews'),
+			'rating' => __('Hide the rating', 'site-reviews'),
+			'date' => __('Hide the date', 'site-reviews'),
+			'assigned_to' => __('Hide the assigned to link (if shown)', 'site-reviews'),
+			'content' => __('Hide the content', 'site-reviews'),
+			'avatar' => __('Hide the avatar (if shown)', 'site-reviews'),
+			'author' => __('Hide the author', 'site-reviews'),
+			'response' => __('Hide the response', 'site-reviews'),
+		];
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -7,14 +7,14 @@
     protected function hideOptions()
     {
         return [
-            'title' => __('Hide the title', 'site-reviews'),
-            'rating' => __('Hide the rating', 'site-reviews'),
-            'date' => __('Hide the date', 'site-reviews'),
-            'assigned_to' => __('Hide the assigned to link (if shown)', 'site-reviews'),
-            'content' => __('Hide the content', 'site-reviews'),
-            'avatar' => __('Hide the avatar (if shown)', 'site-reviews'),
-            'author' => __('Hide the author', 'site-reviews'),
-            'response' => __('Hide the response', 'site-reviews'),
+            'title' => __( 'Hide the title', 'site-reviews' ),
+            'rating' => __( 'Hide the rating', 'site-reviews' ),
+            'date' => __( 'Hide the date', 'site-reviews' ),
+            'assigned_to' => __( 'Hide the assigned to link (if shown)', 'site-reviews' ),
+            'content' => __( 'Hide the content', 'site-reviews' ),
+            'avatar' => __( 'Hide the avatar (if shown)', 'site-reviews' ),
+            'author' => __( 'Hide the author', 'site-reviews' ),
+            'response' => __( 'Hide the response', 'site-reviews' ),
         ];
     }
 }