Calling $.cookie without arguments returns all available cookies as object...

carhartl · carhartl · commit c32ed09760ae · 2013-01-24T16:20:01.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,8 @@ HEAD
 
 - Properly handle RFC 2068 quoted cookie values.
 
+- `$.cookie()` returns all available cookies.
+
 1.2.0
 -----
 - Adding `$.removeCookie('foo')` for deleting a cookie, using `$.cookie('foo', null)` is now deprecated.
diff --git a/README.md b/README.md
@@ -29,6 +29,10 @@ Read cookie:
 
     $.cookie('the_cookie'); // => "the_value"
     $.cookie('not_existing'); // => null
+    
+Read all available cookies:
+
+    $.cookie(); // => { "the_cookie": "the_value", "...remaining": "cookies" }
 
 Delete cookie:
 
diff --git a/jquery.cookie.js b/jquery.cookie.js
@@ -54,15 +54,27 @@
 		// read
 		var decode = config.raw ? raw : decoded;
 		var cookies = document.cookie.split('; ');
+		var result = key ? null : {};
 		for (var i = 0, l = cookies.length; i < l; i++) {
 			var parts = cookies[i].split('=');
-			if (decode(parts.shift()) === key) {
-				var cookie = decode(parts.join('='));
-				return config.json ? JSON.parse(cookie) : cookie;
+			var name = decode(parts.shift());
+			var cookie = decode(parts.join('='));
+			
+			if (config.json) {
+				cookie = JSON.parse(cookie);
+			}
+			
+			if (key && key === name) {
+				result = cookie;
+				break;
+			}
+			
+			if (!key) {
+				result[name] = cookie;
 			}
 		}
 
-		return null;
+		return result;
 	};
 
 	config.defaults = {};
diff --git a/test.js b/test.js
@@ -81,6 +81,17 @@ test('json: true', function () {
 	}
 });
 
++test('no arguments', function () {
+	document.cookie = 'x=y';
+	var count = document.cookie.split(';').length;
+	var cookies = $.cookie();
+	equal(typeof cookies, 'object', 'should return object');
+	var found = 0;
+	for (var key in cookies) { found++; }
+	equal(found, count, 'should contain all cookies');
+	equal(cookies.x, 'y', 'should contain cookie values');
+});
+
 asyncTest('malformed cookie value in IE (#88, #117)', function() {
 	expect(1);
 	// Sandbox in an iframe so that we can poke around with document.cookie.
