Add the "Cookies" API without changing the jquery dependency requirement

FagnerMartinsBrack · FagnerMartinsBrack · commit 25ced9c78c57 · 2015-03-23T20:35:36.000-03:00
Adjust the tests to use the new API. The new API behaves exactly how
"$.cookie" does.

I opted for "Cookies" instead of "Cookie" because it would be
semantically correct to call "Cookies.get()" to get all cookies instead
of "Cookie.get()", otherwise it would mean bloating the codebase by
doing something like "Cookie.getAll()", "Cookie.getCookies()" or
something like that.

+31 bytes gzipped
diff --git a/.jshintrc b/.jshintrc
@@ -8,5 +8,8 @@
 	"nonbsp": true,
 	"trailing": true,
 	"undef": true,
-	"unused": true
+	"unused": true,
+	"globals": {
+		"Cookies": true
+	}
 }
diff --git a/src/jquery.cookie.js b/src/jquery.cookie.js
@@ -14,7 +14,7 @@
 		module.exports = factory(require('jquery'));
 	} else {
 		// Browser globals
-		factory(jQuery);
+		window.Cookies = factory(jQuery);
 	}
 }(function ($) {
 
@@ -111,4 +111,9 @@
 		return !$.cookie(key);
 	};
 
+	$.cookie.get = $.cookie;
+	$.cookie.set = $.cookie;
+	$.cookie.remove = $.removeCookie;
+
+	return $.cookie;
 }));
diff --git a/test/malformed_cookie.html b/test/malformed_cookie.html
@@ -7,7 +7,7 @@
 		<script>
 			try {
 				Object.defineProperty(document, "cookie", { get: function() { return "first=one; ; second=two"; } });
-				window.testValue = $.cookie("second");
+				window.testValue = Cookies.get("second");
 				window.ok = true;
 			} catch (er) {
 			}
diff --git a/test/tests.js b/test/tests.js

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`<script>`
`8`	`8`	`try {`
`9`	`9`	`Object.defineProperty(document, "cookie", { get: function() { return "first=one; ; second=two"; } });`
`10`		`- window.testValue = $.cookie("second");`
	`10`	`+ window.testValue = Cookies.get("second");`
`11`	`11`	`window.ok = true;`
`12`	`12`	`} catch (er) {`
`13`	`13`	`}`