Ignore for Latte\Runtime\Html in link if strict mode is not enabled (#…

…410)

CHANGELOG.md

@@ -8,6 +8,8 @@
 - Added type int as param for Runtime::item() method
 - Transformer for ternary condition with is_object and dynamic form fields - it removes always true / always false condition errors
 - Added nette/forms errors `Cannot call method endTag() on Nette\Utils\Html|string.` and `Cannot call method startTag() on Nette\Utils\Html|string.` to ignore list until they are fixed
+- Support for dynamic forms with known name
+- Ignore for Latte\Runtime\Html in link if strict mode is not enabled
 
 ### Removed
 - ignore-next-line for dynamic inputs - should be solved by removing ternary condition for dynamic inputs

extension.neon

@@ -205,7 +205,7 @@ services:
 
     # Error builder
     - Efabrica\PHPStanLatte\Error\LineMapper\LineMapper(%debugMode%)
-    - Efabrica\PHPStanLatte\Error\ErrorBuilder(%latte.errorPatternsToIgnore%, %latte.warningPatterns%)
+    - Efabrica\PHPStanLatte\Error\ErrorBuilder(%latte.errorPatternsToIgnore%, %latte.warningPatterns%, %latte.strictMode%)
 
     # Error transformers
     - Efabrica\PHPStanLatte\Error\Transformer\UndefinedLatteFilterErrorTransformer

src/Error/ErrorBuilder.php

@@ -69,6 +69,7 @@ final class ErrorBuilder
     public function __construct(
         array $errorPatternsToIgnore,
         array $warningPatterns,
+        bool $strictMode,
         PresenterFactoryFaker $presenterFactoryFaker,
         array $errorTransformers,
         LineMapper $lineMapper
@@ -77,6 +78,9 @@ public function __construct(
         $this->warningPatterns = $warningPatterns;
         $this->errorTransformers = $errorTransformers;
         $this->lineMapper = $lineMapper;
+        if ($strictMode === false) {
+            $this->errorPatternsToIgnore[] = '/Parameter #1 \$destination of method Nette\\\\Application\\\\UI\\\\Component::link\(\) expects string, Latte\\\\Runtime\\\\Html\|string\|false given\./'; // nette/application error https://github.com/nette/application/issues/313 found by https://github.com/efabrica-team/phpstan-latte/issues/398
+        }
         if ($presenterFactoryFaker->getPresenterFactory() === null) {
             $this->errorPatternsToIgnore[] = '/Cannot load presenter .*/';
         }

tests/Rule/LatteTemplatesRule/PresenterWithoutModule/Fixtures/templates/Links/default.latte

@@ -97,3 +97,7 @@
 <a href="{link delete!}">Missing required param in handle link</a>
 <a href="{link delete!, 'my-id', 'extra param'}">Extra param in handle link</a>
 <a href="{link delete!, id: 'my-id', extra: 'extra param'}">Extra named param in handle link</a>
+
+<a n:href="Links:create" n:attr="aa, bb">Create with presenter via n:href</a>
+{capture $linksCreate}Links:create{/capture}
+<a n:href="$linksCreate" n:attr="aa, bb">Create with presenter via n:href</a>

tests/Rule/LatteTemplatesRule/PresenterWithoutModule/LatteTemplatesRuleForPresenterWithNoMappingTest.php

@@ -15,6 +15,7 @@ protected static function additionalConfigFiles(): array
             __DIR__ . '/../../../../rules.neon',
             __DIR__ . '/../../../config.neon',
             __DIR__ . '/config.neon',
+            __DIR__ . '/strict.neon',
         ];
     }
 
@@ -32,6 +33,11 @@ public function testLinks(): void
                 98,
                 'default.latte',
             ],
+            [
+                'Parameter #1 $destination of method Nette\Application\UI\Component::link() expects string, Latte\Runtime\Html|string|false given.',
+                103,
+                'default.latte',
+            ],
         ]);
     }
 }

tests/Rule/LatteTemplatesRule/PresenterWithoutModule/strict.neon

@@ -0,0 +1,3 @@
+parameters:
+    latte:
+        strictMode: true