fix: Compare env to false instead of using empty

exaby73 · web-flow · commit d281abcffded · 2025-02-22T11:31:56.000+05:30
diff --git a/src/Bolt/ProtocolFactory.php b/src/Bolt/ProtocolFactory.php
@@ -34,7 +34,7 @@ public function createProtocol(IConnection $connection, AuthenticateInterface $a
     {
         $boltOptoutEnv = getenv('BOLT_ANALYTICS_OPTOUT');
         /** @psalm-suppress RiskyTruthyFalsyComparison */
-        if (empty($boltOptoutEnv)) {
+        if ($boltOptoutEnv === false) {
             putenv('BOLT_ANALYTICS_OPTOUT=1');
         }
 

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ public function createProtocol(IConnection $connection, AuthenticateInterface $a`
`34`	`34`	`{`
`35`	`35`	`$boltOptoutEnv = getenv('BOLT_ANALYTICS_OPTOUT');`
`36`	`36`	`/** @psalm-suppress RiskyTruthyFalsyComparison */`
`37`		`- if (empty($boltOptoutEnv)) {`
	`37`	`+ if ($boltOptoutEnv === false) {`
`38`	`38`	`putenv('BOLT_ANALYTICS_OPTOUT=1');`
`39`	`39`	`}`
`40`	`40`