fix: ensure calling env command with options only would not throw (#10114)

Author: paulbalandan

system/Commands/Utilities/Environment.php

@@ -85,7 +85,7 @@ final class Environment extends BaseCommand
      */
     public function run(array $params)
     {
-        if ($params === []) {
+        if (! isset($params[0])) {
             CLI::write(sprintf('Your environment is currently set as %s.', CLI::color(service('superglobals')->server('CI_ENVIRONMENT', ENVIRONMENT), 'green')));
             CLI::newLine();
 

tests/system/Commands/Utilities/EnvironmentCommandTest.php

@@ -64,6 +64,13 @@ public function testUsingCommandWithNoArgumentsGivesCurrentEnvironment(): void
         $this->assertStringContainsString(ENVIRONMENT, $this->getStreamFilterBuffer());
     }
 
+    public function testUsingCommandWithOptionsOnlyGivesCurrentEnvironment(): void
+    {
+        command('env --foo');
+        $this->assertStringContainsString('testing', $this->getStreamFilterBuffer());
+        $this->assertStringContainsString(ENVIRONMENT, $this->getStreamFilterBuffer());
+    }
+
     public function testProvidingTestingAsEnvGivesErrorMessage(): void
     {
         command('env testing');

user_guide_src/source/changelogs/v4.7.3.rst

@@ -37,6 +37,7 @@ Bugs Fixed
 **********
 
 - **Autoloader:** Fixed a bug where ``Autoloader::unregister()`` (used during tests) silently failed to remove handlers from the SPL autoload stack, causing closures to accumulate permanently.
+- **Commands:** Fixed a bug in the ``env`` command where passing options only would cause the command to throw a ``TypeError`` instead of showing the current environment.
 - **Common:** Fixed a bug where the ``command()`` helper function did not properly clean up output buffers, which could lead to risky tests when exceptions were thrown.
 - **Validation:** Fixed a bug where ``Validation::getValidated()`` dropped fields whose validated value was explicitly ``null``.