fix setCookie (fails when options.expires is not set or is not a Date)

YakovL · YakovL · commit 633346865492 · 2019-12-03T21:54:20.000+03:00
diff --git a/6-data-storage/01-cookie/article.md b/6-data-storage/01-cookie/article.md
@@ -311,7 +311,7 @@ function setCookie(name, value, options = {}) {
     ...options
   };
 
-  if (options.expires.toUTCString) {
+  if (options.expires instanceof Date && options.expires.toUTCString) {
     options.expires = options.expires.toUTCString();
   }
 
diff --git a/6-data-storage/01-cookie/cookie.js b/6-data-storage/01-cookie/cookie.js
@@ -13,7 +13,7 @@ function setCookie(name, value, options = {}) {
     ...options
   };
 
-  if (options.expires.toUTCString) {
+  if (options.expires instanceof Date && options.expires.toUTCString) {
     options.expires = options.expires.toUTCString();
   }
 

Original file line number	Diff line number	Diff line change
`@@ -311,7 +311,7 @@ function setCookie(name, value, options = {}) {`
`311`	`311`	`...options`
`312`	`312`	`};`
`313`	`313`
`314`		`- if (options.expires.toUTCString) {`
	`314`	`+ if (options.expires instanceof Date && options.expires.toUTCString) {`
`315`	`315`	`options.expires = options.expires.toUTCString();`
`316`	`316`	`}`
`317`	`317`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ function setCookie(name, value, options = {}) {`
`13`	`13`	`...options`
`14`	`14`	`};`
`15`	`15`
`16`		`- if (options.expires.toUTCString) {`
	`16`	`+ if (options.expires instanceof Date && options.expires.toUTCString) {`
`17`	`17`	`options.expires = options.expires.toUTCString();`
`18`	`18`	`}`
`19`	`19`