minor #39412 [Notifier] Only use sprintf instead of sprintf and string concat (OskarStark)

nicolas-grekas · nicolas-grekas · commit 56f04565b2e5 · 2020-12-09T19:30:38.000+01:00
This PR was submitted for the 5.x branch but it was merged into the 5.1 branch instead. Discussion ---------- [Notifier] Only use sprintf instead of sprintf and string concat | Q | A | ------------- | --- | Branch? | 5.x | Bug fix? | no | New feature? | no  | Deprecations? | no  | Tickets | --- | License | MIT | Doc PR | --- Commits ------- ca65c54 [Notifier] Only use sprintf instead of sprintf and string concat
diff --git a/src/Symfony/Component/Notifier/Exception/IncompleteDsnException.php b/src/Symfony/Component/Notifier/Exception/IncompleteDsnException.php
@@ -24,7 +24,7 @@ public function __construct(string $message, string $dsn = null, ?\Throwable $pr
     {
         $this->dsn = $dsn;
         if ($dsn) {
-            $message = sprintf('Invalid "%s" notifier DSN: ', $dsn).$message;
+            $message = sprintf('Invalid "%s" notifier DSN: %s', $dsn, $message);
         }
 
         parent::__construct($message, 0, $previous);

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ public function __construct(string $message, string $dsn = null, ?\Throwable $pr`
`24`	`24`	`{`
`25`	`25`	`$this->dsn = $dsn;`
`26`	`26`	`if ($dsn) {`
`27`		`- $message = sprintf('Invalid "%s" notifier DSN: ', $dsn).$message;`
	`27`	`+ $message = sprintf('Invalid "%s" notifier DSN: %s', $dsn, $message);`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`parent::__construct($message, 0, $previous);`