From 04c961b163e90a9da2bad07d1ac241ea57a61094 Mon Sep 17 00:00:00 2001
From: bobzel <zzzman@gmail.com>
Date: Fri, 23 Dec 2022 17:12:45 -0500
Subject: fixed list lastElement() to return real field value, not proxy. 
 fixed getField to only query proto if value is undefined, not just falsy

---
 src/fields/List.ts | 2 +-
 src/fields/util.ts | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/fields/List.ts b/src/fields/List.ts
index 809173ddf..9c7794813 100644
--- a/src/fields/List.ts
+++ b/src/fields/List.ts
@@ -128,7 +128,7 @@ const listHandlers: any = {
         return this[Self].__fields.map(toRealField).join(separator);
     },
     lastElement() {
-        return this[Self].__fields.lastElement();
+        return this[Self].__realFields().lastElement();
     },
     lastIndexOf(valueToFind: any, fromIndex: number) {
         if (valueToFind instanceof RefField) {
diff --git a/src/fields/util.ts b/src/fields/util.ts
index 285cbb4c6..dc0b41276 100644
--- a/src/fields/util.ts
+++ b/src/fields/util.ts
@@ -368,7 +368,7 @@ function getFieldImpl(target: any, prop: string | number, proxy: any, ignoreProt
     const field = target.__fields[prop];
     const value = field?.[ToValue]?.(proxy); // converts ComputedFields to values, or unpacks ProxyFields into Proxys
     if (value) return value.value;
-    if (!field && !ignoreProto && prop !== 'proto') {
+    if (field === undefined && !ignoreProto && prop !== 'proto') {
         const proto = getFieldImpl(target, 'proto', proxy, true); //TODO tfs: instead of proxy we could use target[SelfProxy]... I don't which semantics we want or if it really matters
         if (proto instanceof Doc && GetEffectiveAcl(proto) !== AclPrivate) {
             return getFieldImpl(proto, prop, proxy, ignoreProto);
-- 
cgit v1.2.3-70-g09d2