elgg.session.cookie() again handles missing options.expires

1 files changed, 12 insertions, 11 deletions

diff --git a/js/lib/session.js b/js/lib/session.js
index 0fc7f5c87..a8d52733c 100644
--- a/js/lib/session.js
+++ b/js/lib/session.js
@@ -14,9 +14,9 @@ elgg.provide('elgg.session');
  * 	{string} options[domain]
  * 	{boolean} options[secure]
  * 
- * @return {string} The value of the cookie, if only name is specified
+ * @return {string|undefined} The value of the cookie, if only name is specified. Undefined if no value set
  */
-elgg.session.cookie = function (name, value, options) {
+elgg.session.cookie = function(name, value, options) {
 	var cookies = [], cookie = [], i = 0, date, valid = true;
 	
 	//elgg.session.cookie()
@@ -48,17 +48,18 @@ elgg.session.cookie = function (name, value, options) {
 	
 	cookies.push(name + '=' + value);
 
-    if (elgg.isNumber(options.expires)) {
-        date = new Date();
-        date.setTime(date.getTime() + (options.expires * 24 * 60 * 60 * 1000));
-    } else if (options.expires.toUTCString) {
-        date = options.expires;
-    }
+    if (options.expires) {
+        if (elgg.isNumber(options.expires)) {
+            date = new Date();
+            date.setTime(date.getTime() + (options.expires * 24 * 60 * 60 * 1000));
+        } else if (options.expires.toUTCString) {
+            date = options.expires;
+        }
 
-    if (date) {
-        cookies.push('expires=' + date.toUTCString());
+        if (date) {
+            cookies.push('expires=' + date.toUTCString());
+        }
     }
-
 	
 	// CAUTION: Needed to parenthesize options.path and options.domain
 	// in the following expressions, otherwise they evaluate to undefined