correct cookie default arguments to avoid deprecated null assignments

phalcon · Sep 26, 2024 · 855c085 · 855c085
1 parent 04edb15
commit 855c085
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/phalcon/Http/Cookie.zep b/phalcon/Http/Cookie.zep
@@ -100,9 +100,9 @@ class Cookie extends AbstractInjectionAware implements CookieInterface
         var value = null,
         int expire = 0,
         string path = "/",
-        bool secure = null,
-        string domain = null,
-        bool httpOnly = null,
+        bool secure = false,
+        string domain = "",
+        bool httpOnly = false,
         array options = []
     ) {
         let this->name     = name,

diff --git a/phalcon/Http/Response/Cookies.zep b/phalcon/Http/Response/Cookies.zep
@@ -251,9 +251,9 @@ class Cookies extends AbstractInjectionAware implements CookiesInterface
         var value = null,
         int expire = 0,
         string path = "/",
-        bool secure = null,
-        string! domain = null,
-        bool httpOnly = null,
+        bool secure = false,
+        string! domain = "",
+        bool httpOnly = false,
         array options = []
     ) -> <CookiesInterface> {
         var cookie, encryption, container, response;

diff --git a/phalcon/Http/Response/CookiesInterface.zep b/phalcon/Http/Response/CookiesInterface.zep
@@ -58,9 +58,9 @@ interface CookiesInterface
         var value = null,
         int expire = 0,
         string path = "/",
-        bool secure = null,
-        string! domain = null,
-        bool httpOnly = null,
+        bool secure = false,
+        string! domain = "",
+        bool httpOnly = false,
         array options = []
     ) -> <CookiesInterface>;