enable templates read a property from a function

packages/ember-glimmer/lib/utils/references.js

@@ -398,7 +398,7 @@ export class InternalHelperReference extends CachedReference {
 // @implements PathReference
 export class UnboundReference extends ConstReference {
   static create(value) {
-    if (typeof value === 'object' && value !== null || typeof result === 'function') {
+    if (typeof value === 'object' && value !== null || typeof value === 'function') {
       return new UnboundReference(value);
     } else {
       return PrimitiveReference.create(value);

packages/ember-glimmer/tests/integration/content-test.js

@@ -495,17 +495,17 @@ class DynamicContentTest extends RenderingTest {
 
     this.assertStableRerender();
 
-    // this.runTask(() => set(func, 'aProp', 'still a property on a function'));
-    // this.assertContent('still a property on a function');
-    // this.assertInvariants();
+    this.runTask(() => set(func, 'aProp', 'still a property on a function'));
+    this.assertContent('still a property on a function');
+    this.assertInvariants();
 
-    // func = () => {};
-    // func.aProp = 'a prop on a new function';
+    func = () => {};
+    func.aProp = 'a prop on a new function';
 
-    // this.runTask(() => set(this.context, 'func', func));
+    this.runTask(() => set(this.context, 'func', func));
 
-    // this.assertContent('a prop on a new function');
-    // this.assertInvariants();
+    this.assertContent('a prop on a new function');
+    this.assertInvariants();
   }
 }
 

packages/ember-glimmer/tests/integration/helpers/unbound-test.js

@@ -142,6 +142,23 @@ moduleFor('Helpers test: {{unbound}}', class extends RenderingTest {
     this.assertHTML(escapedHtml);
   }
 
+  ['@test unbound works with function properties']() {
+    let func = function() {}
+    func.foo = 'WOOT';
+    let parent = { func };
+    this.render(`{{unbound func.foo}}`, parent);
+
+    this.assertText('WOOT');
+
+    this.runTask(() => this.rerender());
+
+    this.assertText('WOOT');
+
+    this.runTask(() => set(this.context, 'func.foo', 'OOF'));
+
+    this.assertText('WOOT');
+  }
+
   ['@skip helper form updates on parent re-render']() {
     this.render(`{{unbound foo}}`, {
       foo: 'BORK'

packages/ember-metal/lib/property_set.js

@@ -67,7 +67,6 @@ export function set(obj, keyName, value, tolerant) {
 
     propertyDidChange(obj, keyName, meta);
   }
-
   return value;
 }
 

packages/ember-metal/lib/tags.js

@@ -13,7 +13,7 @@ function makeTag() {
 }
 
 export function tagForProperty(object, propertyKey, _meta) {
-  if (typeof object !== 'object' || object === null) { return CONSTANT_TAG; }
+  if ((typeof object !== 'object' && typeof object !== 'function') || object === null) { return CONSTANT_TAG; }
 
   let meta = _meta || metaFor(object);
   if (meta.isProxy()) {

packages/ember-metal/lib/watch_key.js

@@ -14,7 +14,7 @@ import {
 let handleMandatorySetter;
 
 export function watchKey(obj, keyName, meta) {
-  if (typeof obj !== 'object' || obj === null) { return; }
+  if (typeof obj !== 'object' && typeof obj !== 'function' || obj === null) { return; }
 
   let m = meta || metaFor(obj);
   let count = m.peekWatching(keyName) || 0;
@@ -81,9 +81,8 @@ if (MANDATORY_SETTER) {
 }
 
 export function unwatchKey(obj, keyName, _meta) {
-  if (typeof obj !== 'object' || obj === null) {
-    return;
-  }
+  if (typeof obj !== 'object' && typeof obj !== 'function' || obj === null) { return; }
+
   let meta = _meta || peekMeta(obj);
 
   // do nothing of this object has already been destroyed