v 1.0.4

code-quality/phpcs.xml

@@ -4,6 +4,18 @@
 
     <rule ref="PSR12"/>
 
+    <rule ref="Generic.Formatting.SpaceAfterNot">
+        <properties>
+            <property name="spacing" value="1"/>
+        </properties>
+    </rule>
+
+    <rule ref="Generic.Formatting.SpaceAfterCast">
+        <properties>
+            <property name="spacing" value="1"/>
+        </properties>
+    </rule>
+
     <rule ref="SlevomatCodingStandard.Classes.ClassStructure">
         <properties>
             <property name="groups" type="array">

code-quality/phpstan.neon

@@ -2,4 +2,6 @@ parameters:
     level: 9
     paths:
         - ../src
-        - ../private-classes
+        - ../private-classes
+includes:
+    - vendor/phpstan/phpstan-strict-rules/rules.neon

private-classes/Blade/BladeCompiler.php

@@ -70,7 +70,7 @@ protected function applyExtensionCallback(string $template): string
     // Removes all parts like: "{{--Comment--}}".
     protected function removeComments(string $template): string
     {
-        return (string)preg_replace('/{{--[\s\S]*?--}}/', '', $template);
+        return (string) preg_replace('/{{--[\s\S]*?--}}/', '', $template);
     }
 
     protected function replaceOpeningEcho(string $template, string $escape_callback_name): string
@@ -100,7 +100,7 @@ protected function replaceOpeningTagWithBrackets(string $tag, string $template):
         $regex = $this->getRegexForTagWithBrackets($tag);
         $replacement = sprintf('<?php %s( $1 ): ?>', $tag);
 
-        return (string)preg_replace($regex, $replacement, $template);
+        return (string) preg_replace($regex, $replacement, $template);
     }
 
     protected function replaceClosingLoops(string $template): string
@@ -133,7 +133,7 @@ protected function replaceClosingPhp(string $template): string
 
     protected function replaceUseDirective(string $template): string
     {
-        return (string)preg_replace('/@use\s*\((["\'])(.*?)\1\)/s', '<?php use $2; ?>', $template);
+        return (string) preg_replace('/@use\s*\((["\'])(.*?)\1\)/s', '<?php use $2; ?>', $template);
     }
 
     protected function replaceSelectedDirective(string $template): string
@@ -142,7 +142,7 @@ protected function replaceSelectedDirective(string $template): string
 
         $replacement = '<?php if ( $1 ) echo "selected=\"\""; ?>';
 
-        return (string)preg_replace($regex, $replacement, $template);
+        return (string) preg_replace($regex, $replacement, $template);
     }
 
     protected function replaceCheckedDirective(string $template): string
@@ -151,7 +151,7 @@ protected function replaceCheckedDirective(string $template): string
 
         $replacement = '<?php if ( $1 ) echo "checked=\"\""; ?>';
 
-        return (string)preg_replace($regex, $replacement, $template);
+        return (string) preg_replace($regex, $replacement, $template);
     }
 
     protected function replaceSwitchDirectives(string $template): string
@@ -161,7 +161,7 @@ protected function replaceSwitchDirectives(string $template): string
         // 1. remove space between @switch and the first "case",
         // otherwise it'll case an error (spaces are threat as unexpected HTML).
         $regex = '/@switch\s*\((.*)\)\s*<\?php/';
-        $template = (string)preg_replace($regex, '<?php switch($1): ?><?php', $template);
+        $template = (string) preg_replace($regex, '<?php switch($1): ?><?php', $template);
 
         $template = str_replace('@break', '<?php break; ?>', $template);
         $template = str_replace('@default', '<?php default: ?>', $template);
@@ -176,7 +176,7 @@ protected function replaceClassDirective(string $template, string $escape_callba
         $regex = '/@class\s*\((\[.*])\)/s';
         $replacement = $this->getCodeForConditionClasses($escape_callback_name);
 
-        return (string)preg_replace($regex, $replacement, $template);
+        return (string) preg_replace($regex, $replacement, $template);
     }
 
     protected function getCodeForConditionClasses(string $escape_callback_name): string

private-classes/EventDispatcher.php

@@ -83,7 +83,7 @@ public function unregisterEventDetails(string $eventName, array $eventDetails):
      */
     protected function getEventDetails(string $eventName): array
     {
-        return true === key_exists($eventName, $this->eventDetails) ?
+        return key_exists($eventName, $this->eventDetails) ?
             $this->eventDetails[$eventName] :
             [];
     }
@@ -93,7 +93,7 @@ protected function getEventDetails(string $eventName): array
      */
     protected function getEventListeners(string $eventName): array
     {
-        return true === key_exists($eventName, $this->eventListeners) ?
+        return key_exists($eventName, $this->eventListeners) ?
             $this->eventListeners[$eventName] :
             [];
     }

private-classes/Model/ModelFactoryWithDefaultsManagement.php

@@ -34,7 +34,7 @@ public function createModel(string $modelClass, ?Closure $setupModelCallback = n
     {
         $model = $this->modelFactory->createModel($modelClass);
 
-        if (true === ($model instanceof TemplateModelWithDefaultsInterface)) {
+        if ($model instanceof TemplateModelWithDefaultsInterface) {
             $this->setDefaultValuesRecursively($model);
         }
 
@@ -64,7 +64,7 @@ protected function setDefaultValuesRecursively(TemplateModelWithDefaultsInterfac
     protected function getInnerModels(array $variables): array
     {
         return array_filter($variables, function ($item) {
-            return true === ($item instanceof TemplateModelWithDefaultsInterface) ;
+            return $item instanceof TemplateModelWithDefaultsInterface ;
         });
     }
 }

private-classes/Model/ModelNamespaceResolver.php

@@ -22,7 +22,7 @@ public function __construct(ObjectClassReader $objectClassReader)
 
     public function resolveModelNamespace($modelOrClass): string
     {
-        $modelNamespaceWithClassName = false === is_string($modelOrClass) ?
+        $modelNamespaceWithClassName = ! is_string($modelOrClass) ?
             $this->objectClassReader->getObjectClass($modelOrClass) :
             $modelOrClass;
 

private-classes/Model/ModelRenderer.php

@@ -32,7 +32,7 @@ public function __construct(
 
     public function renderModel($modelOrClass, ?Closure $setupModelCallback = null): string
     {
-        $model = true === is_string($modelOrClass) ?
+        $model =  is_string($modelOrClass) ?
             $this->viewFactory->createModel($modelOrClass) :
             $modelOrClass;
 

private-classes/Model/ModelRendererWithEventDetails.php

@@ -30,7 +30,7 @@ public function __construct(
 
     public function renderModel($modelOrClass, ?Closure $setupModelCallback = null): string
     {
-        $modelClass = true === is_string($modelOrClass) ?
+        $modelClass =  is_string($modelOrClass) ?
             $modelOrClass :
             get_class($modelOrClass);
 

private-classes/Object/ObjectPropertyWriter.php

@@ -24,7 +24,7 @@ public function assignPropertyValues(
 
         array_map(
             function (ReflectionProperty $reflectionProperty) use ($instance, $propertyValueProvider) {
-                if (true === $reflectionProperty->isInitialized($instance)) {
+                if ($reflectionProperty->isInitialized($instance)) {
                     return;
                 }
 
@@ -55,7 +55,7 @@ protected function setDefaultValueForSupportedType(
         PropertyValueProviderInterface $propertyValueProvider,
         ReflectionProperty $reflectionProperty
     ): bool {
-        if (false === $propertyValueProvider->supportsProperty($reflectionProperty)) {
+        if (! $propertyValueProvider->supportsProperty($reflectionProperty)) {
             return false;
         }
 

private-classes/Object/ObjectReader.php

@@ -100,7 +100,7 @@ protected function getPropertyValues(object $instance, array $reflectionProperti
             function (array $variableValues, ReflectionProperty $reflection_property) use ($instance) {
                 // make sure the property is initialized.
                 // Otherwise, we'll get "must not be accessed before initialization" error.
-                if (true === $reflection_property->isInitialized($instance)) {
+                if ($reflection_property->isInitialized($instance)) {
                     $variableValues[ $reflection_property->getName() ] = $reflection_property->getValue($instance);
                 }
 

private-classes/Object/PropertyValueProviderByTypes.php

@@ -30,24 +30,24 @@ public function __construct(PropertyValueProviderInterface $propertyValueProvide
 
     public function supportsProperty(ReflectionProperty $property): bool
     {
-        if (true === $this->propertyValueProvider->supportsProperty($property)) {
+        if ($this->propertyValueProvider->supportsProperty($property)) {
             return true;
         }
 
         $type = $this->getPropertyType($property);
 
-        return true === key_exists($type, $this->valuesByType);
+        return key_exists($type, $this->valuesByType);
     }
 
     public function getPropertyValue(ReflectionProperty $property)
     {
-        if (true === $this->propertyValueProvider->supportsProperty($property)) {
+        if ($this->propertyValueProvider->supportsProperty($property)) {
             return $this->propertyValueProvider->getPropertyValue($property);
         }
 
         $type = $this->getPropertyType($property);
 
-        return true === key_exists($type, $this->valuesByType) ?
+        return key_exists($type, $this->valuesByType) ?
         $this->valuesByType[$type] :
         null;
     }

private-classes/Object/PropertyValueProviderForModels.php

@@ -28,7 +28,7 @@ public function __construct(
 
     public function supportsProperty(ReflectionProperty $property): bool
     {
-        if (true === $this->propertyValueProvider->supportsProperty($property)) {
+        if ($this->propertyValueProvider->supportsProperty($property)) {
             return true;
         }
 
@@ -39,7 +39,7 @@ public function supportsProperty(ReflectionProperty $property): bool
 
     public function getPropertyValue(ReflectionProperty $property)
     {
-        if (true === $this->propertyValueProvider->supportsProperty($property)) {
+        if ($this->propertyValueProvider->supportsProperty($property)) {
             return $this->propertyValueProvider->getPropertyValue($property);
         }
 
@@ -58,8 +58,8 @@ public function getPropertyValue(ReflectionProperty $property)
      */
     protected function getValidModelClass(string $propertyType)
     {
-        return true === class_exists($propertyType) &&
-        true === is_a($propertyType, TemplateModelInterface::class, true) ?
+        return  class_exists($propertyType) &&
+         is_a($propertyType, TemplateModelInterface::class, true) ?
             $propertyType :
             null;
     }

private-classes/Object/PropertyValueProviderForNullable.php

@@ -22,18 +22,19 @@ public function __construct(PropertyValueProviderInterface $propertyValueProvide
 
     public function supportsProperty(ReflectionProperty $property): bool
     {
-        if (true === $this->propertyValueProvider->supportsProperty($property)) {
+        if ($this->propertyValueProvider->supportsProperty($property)) {
             return true;
         }
 
         $type = $property->getType();
 
-        return null !== $type && true === $type->allowsNull();
+        return null !== $type &&
+            $type->allowsNull();
     }
 
     public function getPropertyValue(ReflectionProperty $property)
     {
-        if (true === $this->propertyValueProvider->supportsProperty($property)) {
+        if ($this->propertyValueProvider->supportsProperty($property)) {
             return $this->propertyValueProvider->getPropertyValue($property);
         }
 

private-classes/Template/FileModelTemplateResolver.php

@@ -52,14 +52,14 @@ public function resolveModelTemplate(TemplateModelInterface $model): string
 
         $absoluteTemplatePath = $this->getAbsoluteTemplatePath($relativeTemplatePath);
 
-        return true === $this->isFileBasedTemplate ?
+        return  $this->isFileBasedTemplate ?
             $absoluteTemplatePath :
             $this->getFileContent($absoluteTemplatePath);
     }
 
     protected function getFileContent(string $file): string
     {
-        if (false === file_exists($file)) {
+        if (! file_exists($file)) {
             return '';
         }
 
@@ -75,13 +75,13 @@ protected function getAbsoluteTemplatePath(string $relativeTemplatePath): string
     protected function getRelativeTemplatePath(string $relativeModelNamespace, string $modelName): string
     {
         $relativeModelPath = str_replace('\\', DIRECTORY_SEPARATOR, $relativeModelNamespace);
-        $modelName = (string)preg_replace('/([a-z])([A-Z])/', '$1-$2', $modelName);
+        $modelName = (string) preg_replace('/([a-z])([A-Z])/', '$1-$2', $modelName);
 
         $relativeTemplatePath = $relativeModelPath;
         $relativeTemplatePath .= '' !== $relativeTemplatePath ? DIRECTORY_SEPARATOR
             : '';
-        $relativeTemplatePath .= $modelName;
+        $relativeTemplatePath .= strtolower($modelName);
 
-        return strtolower($relativeTemplatePath);
+        return $relativeTemplatePath;
     }
 }

private-classes/Template/TemplateRenderer.php

@@ -26,6 +26,6 @@ public function renderTemplate(string $template, array $variables = []): string
 
         $this->codeExecutor->runCode($template, $variables);
 
-        return (string)ob_get_clean();
+        return (string) ob_get_clean();
     }
 }

private-classes/Template/TemplateRendererWithCustomEscape.php

@@ -80,17 +80,17 @@ protected function setOutputEscapeCallback(
     protected function caseToString($value): string
     {
         if (
-            true === is_string($value) ||
-            true === is_numeric($value)
+             is_string($value) ||
+            is_numeric($value)
         ) {
-            return (string)$value;
+            return (string) $value;
         }
 
         if (
-            true === is_object($value) &&
+             is_object($value) &&
             method_exists($value, '__toString')
         ) {
-            return (string)$value;
+            return (string) $value;
         }
 
         return '';

private-classes/Template/TemplateRendererWithFileTemplate.php

@@ -28,10 +28,10 @@ public function renderTemplate(string $template, array $variables = []): string
 
     protected function getFileContent(string $file): string
     {
-        if (false === file_exists($file)) {
+        if (! file_exists($file)) {
             return '';
         }
 
-        return (string)file_get_contents($file);
+        return (string) file_get_contents($file);
     }
 }

private-classes/Template/TemplateRendererWithModelsRender.php

@@ -55,11 +55,11 @@ function ($item) {
      */
     protected function renderIfModel($item)
     {
-        if (true === ( $item instanceof TemplateModelInterface )) {
+        if ($item instanceof TemplateModelInterface) {
             $item = $this->modelRenderer->renderModel($item);
         } elseif (
-            true === is_array($item) &&
-            false === is_callable($item)
+             is_array($item) &&
+            ! is_callable($item)
         ) {
             // @phpstan-ignore-next-line
             $item = $this->renderModels($item);

private-classes/View/ViewNamespace.php

@@ -117,7 +117,7 @@ public function __construct(
             $modelRendererWithNamespace
         );
 
-        if (true === $config->modelsAsStringsInTemplates()) {
+        if ($config->modelsAsStringsInTemplates()) {
             $templateRenderer = $templateRendererWithModelsRender;
         }
 

src/View/ViewTemplateRenderer.php

@@ -65,7 +65,7 @@ public function __construct(?ViewTemplateRendererConfig $config = null)
             new TemplateRenderer($codeExecutor) :
             $templateRenderer;
 
-        if (true === $config->fileBasedTemplates()) {
+        if ($config->fileBasedTemplates()) {
             $templateRenderer = new TemplateRendererWithFileTemplate($templateRenderer);
         }