minor #41009 [Config ]Remove log message when no configuration is found in ConfigBuilder (alexander-schranz)

fabpot · fabpot · commit 610c26318113 · 2021-05-01T16:15:21.000+02:00
This PR was merged into the 5.3-dev branch. Discussion ---------- [Config ]Remove log message when no configuration is found in ConfigBuilder | Q | A | ------------- | --- | Branch? | 5.x for features / 4.4 or 5.2 for bug fixes  | Bug fix? | - | New feature? | no  | Deprecations? | no  | Tickets | Fix #...  | License | MIT | Doc PR | symfony/symfony-docs#...   I think when a bundle has no configuration we should also not log an warning. Commits ------- 20c03b5 Remove log message when not configuration is found
diff --git a/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ConfigBuilderCacheWarmer.php b/src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ConfigBuilderCacheWarmer.php
@@ -77,10 +77,6 @@ private function dumpExtension(ExtensionInterface $extension, ConfigBuilderGener
         }
 
         if (!$configuration) {
-            if ($this->logger) {
-                $this->logger->info('No configuration found for extension {extensionClass}.', ['extensionClass' => \get_class($extension)]);
-            }
-
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -77,10 +77,6 @@ private function dumpExtension(ExtensionInterface $extension, ConfigBuilderGener`
`77`	`77`	`}`
`78`	`78`
`79`	`79`	`if (!$configuration) {`
`80`		`- if ($this->logger) {`
`81`		`- $this->logger->info('No configuration found for extension {extensionClass}.', ['extensionClass' => \get_class($extension)]);`
`82`		`- }`
`83`		`-`
`84`	`80`	`return;`
`85`	`81`	`}`
`86`	`82`