From 2fad67899a8fd6b82b665e42fca24cdc029c4b11 Mon Sep 17 00:00:00 2001 From: davidspilka Date: Wed, 20 Sep 2023 14:35:06 +0200 Subject: [PATCH] =?UTF-8?q?Vypnut=C3=AD=20pravidla=20SpaceBeforeNotSniff?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Důvod vypnutí: S novým CS se objevil v určitých případech problém, kdy spolu kolidují 2 pravidla a tím pádem nefunguje cs-fix. Jedná se o případy typu: ``` $builder->addDefinition($this->prefix('proactiveConfigurationsMapProvider')) ->setFactory(ProactiveConfigurationsMapProvider::class) ->setAutowired(! $lazyConfigurationsMap) ; ``` kdy problém je ten řádek `->setAutowired(! $lazyConfigurationsMap)``. Problém je v tom, že naše pravidlo `/Sniffs/Formatting/SpaceBeforeNotSniff.php` , které psali před nedávnem kluci ze SuperZOO, vyžaduje mezeru před `!`. Po opravě by tedy vzniklo `->setAutowired( ! $lazyConfigurationsMap)` , což je ale problém pro pravidlo `PSR2.Methods.FunctionCallSignature.SpaceAfterOpenBracket` , které vyžaduje 0 mezer po otevírací závorce volání funkce. --- src/PeckaCodingStandardPSR12Based/ruleset.xml | 1 - 1 file changed, 1 deletion(-) diff --git a/src/PeckaCodingStandardPSR12Based/ruleset.xml b/src/PeckaCodingStandardPSR12Based/ruleset.xml index 9d886c8..cebc92f 100644 --- a/src/PeckaCodingStandardPSR12Based/ruleset.xml +++ b/src/PeckaCodingStandardPSR12Based/ruleset.xml @@ -152,7 +152,6 @@ -