From: John Resig <jeresig@gmail.com>
Date: Mon, 20 Sep 2010 21:47:41 +0000 (-0400)
Subject: Pull data-* attributes into .data(). Original code by Andrée Hasson and Paul Irish... 
X-Git-Url: http://git.asbjorn.biz/?p=jquery.git;a=commitdiff_plain;h=20673d7e5836dda504b66730b528a8dae9787493

Pull data-* attributes into .data(). Original code by Andrée Hasson and Paul Irish. Fixes #6921.
---

diff --git a/src/data.js b/src/data.js
index 5404c93..f2cfc81 100644
--- a/src/data.js
+++ b/src/data.js
@@ -1,6 +1,7 @@
 (function( jQuery ) {
 
-var windowData = {};
+var windowData = {},
+	rnum = /^-?[0-9.]$/;
 
 jQuery.extend({
 	cache: {},
@@ -142,8 +143,23 @@ jQuery.fn.extend({
 		if ( value === undefined ) {
 			var data = this.triggerHandler("getData" + parts[1] + "!", [parts[0]]);
 
+			// Try to fetch any internally stored data first
 			if ( data === undefined && this.length ) {
 				data = jQuery.data( this[0], key );
+
+				// If nothing was found internally, try to fetch any
+				// data from the HTML5 data-* attribute
+				if ( data === undefined && this[0].nodeType === 1 ) {
+					data = this[0].getAttribute( "data-" + key );
+
+					if ( data != null ) {
+						data = data === "true" ? true :
+							data === "false" ? false :
+							data === "null" ? null :
+							rnum.test( data ) ? parseFloat( data ) :
+							data;
+					}
+				}
 			}
 
 			return data === undefined && parts[1] ?
diff --git a/test/unit/data.js b/test/unit/data.js
index 872bbb7..c7307bf 100644
--- a/test/unit/data.js
+++ b/test/unit/data.js
@@ -157,6 +157,74 @@ test(".data(String) and .data(String, Object)", function() {
 	$elem.removeData();
 });
 
+test("data-* attributes", function() {
+	expect(19);
+	var div = jQuery("<div>"),
+		child = jQuery("<div data-myobj='old data' data-ignored=\"DOM\"></div>");
+		
+	equals( div.data("attr"), undefined, "Check for non-existing data-attr attribute" );
+
+	div.attr("data-attr", "exists");
+	equals( div.data("attr"), "exists", "Check for existing data-attr attribute" );
+		
+	div.data("attr", "internal").attr("data-attr", "external");
+	equals( div.data("attr"), "internal", "Check for .data('attr') precedence (internal > external data-* attribute)" );
+	
+	child.appendTo('#main');
+	equals( child.data("myobj"), "old data", "Value accessed from data-* attribute");
+
+	child.data("myobj", "replaced");
+	equals( child.data("myobj"), "replaced", "Original data overwritten");
+
+	child.data("ignored", "cache");
+	equals( child.data("ignored"), "cache", "Cached data used before DOM data-* fallback");
+
+	child
+		.attr("data-true", "true")
+		.attr("data-false", "false")
+		.attr("data-five", "5")
+		.attr("data-null", "null")
+		.attr("data-string", "test");
+	
+	equals( child.data('true'), true, "Primitive true read from attribute");
+	equals( child.data('false'), false, "Primitive false read from attribute");
+	equals( child.data('five'), 5, "Primitive number read from attribute");
+	equals( child.data('null'), null, "Primitive null read from attribute");
+	equals( child.data('string'), "test", "Typical string read from attribute");
+
+	child.remove();
+	
+	// tests from metadata plugin
+	function testData(index, elem) {
+		switch (index) {
+		case 0:
+			equals(jQuery(elem).data("foo"), "bar", "Check foo property");
+			equals(jQuery(elem).data("bar"), "baz", "Check baz property");
+			break;
+		case 1:
+			equals(jQuery(elem).data("test"), "bar", "Check test property");
+			equals(jQuery(elem).data("bar"), "baz", "Check bar property");
+			break;
+		case 2:
+			equals(jQuery(elem).data("zoooo"), "bar", "Check zoooo property");
+			equals(jQuery(elem).data("bar"), '{"test":"baz"}', "Check bar property");
+			break;
+		case 3:
+			equals(jQuery(elem).data("number"), true, "Check number property");
+			equals(jQuery(elem).data("stuff"), "[2,8]", "Check stuff property");
+			break;
+		default:
+			ok(false, ["Assertion failed on index ", index, ", with data ", data].join(''));
+		}
+	}
+	
+	var metadata = '<ol><li class="test test2" data-foo="bar" data-bar="baz" data-arr="[1,2]">Some stuff</li><li class="test test2" data-test="bar" data-bar="baz">Some stuff</li><li class="test test2" data-zoooo="bar" data-bar=\'{"test":"baz"}\'>Some stuff</li><li class="test test2" data-number=true data-stuff="[2,8]">Some stuff</li></ol>',
+		elem = jQuery(metadata).appendTo('#main');
+	
+	elem.find("li").each(testData);
+	elem.remove();
+});
+
 test(".data(Object)", function() {
 	expect(2);