Merge pull request #32 from canjs/key-value

Add a value property getter & setter to the key module/observable

key/key-test.js

@@ -20,6 +20,19 @@ QUnit.test("basics", function(assert) {
 	assert.equal(canReflect.getValue(observable), "aloha", "getValue unbound");
 });
 
+QUnit.test("value property is a getter and setter", function(assert) {
+	var outer = {inner: {key: "hello"}};
+	var observable = keyObservable(outer, "inner.key");
+
+	// Check getting the value
+	assert.equal(observable.value, "hello", "value getter works");
+
+	// Check setting the value
+	observable.value = "aloha";
+	assert.equal(canReflect.getValue(outer).inner.key, "aloha", "value setter sets other object");
+	assert.equal(canReflect.getValue(observable), "aloha", "value getter works after set");
+});
+
 QUnit.test("get and set Priority", function(assert) {
 	var outer = {inner: {key: "hello"}};
 	var observable = keyObservable(outer, "inner.key");

key/key.js

@@ -46,10 +46,19 @@ module.exports = function keyObservable(root, keyPath) {
 		return value;
 	});
 
+	// Function for setting the value
+	var valueSetter = function(newVal) {
+		canKey.set(root, keyPathParts, newVal);
+	};
+
+	// The `value` property getter & setter
+	Object.defineProperty(observation, "value", {
+		get: observation.get,
+		set: valueSetter
+	});
+
 	var symbolsToAssign = {
-		"can.setValue": function(newVal) {
-			canKey.set(root, keyPathParts, newVal);
-		}
+		"can.setValue": valueSetter
 	};
 
 	//!steal-remove-start

package.json

@@ -42,7 +42,7 @@
     "can-key-tree": "^1.0.0",
     "can-log": "^1.0.0",
     "can-namespace": "1.0.0",
-    "can-observation": "^4.0.0",
+    "can-observation": "^4.1.0",
     "can-observation-recorder": "^1.0.0",
     "can-queues": "^1.0.0",
     "can-reflect": "^1.10.1",