fix: Cookie class option prefix bug

When option prefix is '0', it is not set correctly.

Author: kenjis

system/Cookie/Cookie.php

@@ -119,6 +119,8 @@ class Cookie implements ArrayAccess, CloneableCookieInterface
      * Set the default attributes to a Cookie instance by injecting
      * the values from the `CookieConfig` config or an array.
      *
+     * This method is called from Response::__construct().
+     *
      * @param array<string, mixed>|CookieConfig $config
      *
      * @return array<string, mixed> The old defaults array. Useful for resetting.
@@ -209,7 +211,7 @@ final public function __construct(string $name, string $value = '', array $optio
         }
 
         // to preserve backward compatibility with array-based cookies in previous CI versions
-        $prefix = $options['prefix'] ?: self::$defaults['prefix'];
+        $prefix = ($options['prefix'] === '') ? self::$defaults['prefix'] : $options['prefix'];
         $path   = $options['path'] ?: self::$defaults['path'];
         $domain = $options['domain'] ?: self::$defaults['domain'];
 

tests/system/Cookie/CookieTest.php

@@ -16,6 +16,7 @@
 use Config\Cookie as CookieConfig;
 use DateTimeImmutable;
 use DateTimeZone;
+use Generator;
 use LogicException;
 
 /**
@@ -76,6 +77,38 @@ public function testConfigInjectionForDefaults(): void
         Cookie::setDefaults($old);
     }
 
+    /**
+     * @dataProvider prefixProvider
+     */
+    public function testConfigPrefix(string $configPrefix, string $optionPrefix, string $expected): void
+    {
+        $config         = new CookieConfig();
+        $config->prefix = $configPrefix;
+        Cookie::setDefaults($config);
+
+        $cookie = new Cookie(
+            'test',
+            'value',
+            [
+                'prefix' => $optionPrefix,
+            ]
+        );
+
+        $this->assertSame($expected, $cookie->getPrefixedName());
+    }
+
+    public function prefixProvider(): Generator
+    {
+        yield from [
+            ['prefix_', '', 'prefix_test'],
+            ['prefix_', '0', '0test'],
+            ['prefix_', 'new_', 'new_test'],
+            ['', '', 'test'],
+            ['', '0', '0test'],
+            ['', 'new_', 'new_test'],
+        ];
+    }
+
     public function testValidationOfRawCookieName(): void
     {
         $this->expectException(CookieException::class);