From 1359050efd7053fe9f162759041734102ba68345 Mon Sep 17 00:00:00 2001
From: Steffen Vogel <info@steffenvogel.de>
Date: Tue, 19 Oct 2010 18:57:08 +0200
Subject: [PATCH] improved cookie functions

---
 frontend/javascripts/jquery-extensions.js | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/frontend/javascripts/jquery-extensions.js b/frontend/javascripts/jquery-extensions.js
index a4957f9..247379e 100644
--- a/frontend/javascripts/jquery-extensions.js
+++ b/frontend/javascripts/jquery-extensions.js
@@ -58,7 +58,11 @@ $.extend( {
  */
 $.extend({
 	setCookie: function(name, value, options) {
-		options = options || {};
+		// defaults
+		options = options || {
+			expires: null,
+			
+		};
 		
 		if (value === null) {
 			value = '';
@@ -66,10 +70,9 @@ $.extend({
 		}
 		
 		var expires = '';
-		
 		if (options.expires && (typeof options.expires == 'number' || options.expires.toUTCString)) {
 			var date;
-			if (typeof options.expires == 'number') {
+			if (typeof options.expires == 'number') { // expires x seconds in the future
 				date = new Date();
 				date.setTime(date.getTime()
 						+ (options.expires * 24 * 60 * 60 * 1000));
@@ -89,22 +92,15 @@ $.extend({
 		document.cookie = name + '=' + encodeURIComponent(value) + expires + path + domain + secure;
 	},
 	getCookie: function(name) {
-		var value = null;
-		
 		if (document.cookie && document.cookie != '') {
 			var cookies = document.cookie.split(';');
-			
 			for (var i = 0; i < cookies.length; i++) {
 				var cookie = $.trim(cookies[i]);
-				
 				// Does this cookie string begin with the name we want?
 				if (cookie.substring(0, name.length + 1) == (name + '=')) {
-					value = decodeURIComponent(cookie.substring(name.length + 1));
-					break;
+					return decodeURIComponent(cookie.substring(name.length + 1));
 				}
 			}
 		}
-		
-		return value;
 	}
-});
\ No newline at end of file
+});