Fixed issue with updating nested properties via binding.

Nedyalko Nikolov · ErjanGavalji · commit ffc1fcb030cd · 2015-06-10T09:36:33.000+03:00
diff --git a/apps/tests/ui/bindable-tests.ts b/apps/tests/ui/bindable-tests.ts
@@ -541,4 +541,36 @@ export function test_BindingToDictionaryAtAppLevel() {
     }
 
     helper.buildUIAndRunTest(createLabel(), testFunc);
+}
+
+export function test_UpdatingNestedPropertyViaBinding() {
+    var expectedValue1 = "Alabala";
+    var expectedValue2 = "Tralala";
+    var viewModel = new observable.Observable();
+    var parentViewModel = new observable.Observable();
+    viewModel.set("parentView", parentViewModel);
+    parentViewModel.set("name", expectedValue1);
+
+    var testElement: bindable.Bindable = new bindable.Bindable();
+
+    testElement.bind({
+        sourceProperty: "parentView.name",
+        targetProperty: "targetName",
+        twoWay: true
+    }, viewModel);
+
+    var testElement2: bindable.Bindable = new bindable.Bindable();
+
+    testElement2.bind({
+        sourceProperty: "parentView.name",
+        targetProperty: "targetProperty",
+        twoWay: true
+    }, viewModel);
+
+    TKUnit.assertEqual(testElement.get("targetName"), expectedValue1);
+
+    testElement.set("targetName", expectedValue2);
+
+    TKUnit.assertEqual(parentViewModel.get("name"), expectedValue2);
+    TKUnit.assertEqual(testElement2.get("targetProperty"), expectedValue2);
 }
diff --git a/ui/core/bindable.ts b/ui/core/bindable.ts
@@ -261,7 +261,7 @@ export class Binding {
             else {
                 this.updateTarget(expressionValue);
             }
-        } else if (data.propertyName === this.options.sourceProperty) {
+        } else if (data.propertyName === this.sourceOptions.property) {
             this.updateTarget(data.value);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -261,7 +261,7 @@ export class Binding {`
`261`	`261`	`else {`
`262`	`262`	`this.updateTarget(expressionValue);`
`263`	`263`	`}`
`264`		`- } else if (data.propertyName === this.options.sourceProperty) {`
	`264`	`+ } else if (data.propertyName === this.sourceOptions.property) {`
`265`	`265`	`this.updateTarget(data.value);`
`266`	`266`	`}`
`267`	`267`	`}`