Fix non-stored fields appearing as nulls in JSON search response (#100)

* Fix non-stored fields being returned as nulls
lnx-search · Jul 14, 2022 · 95280de · 95280de
1 parent 6437423
commit 95280de
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/lnx-engine/search-index/src/schema.rs b/lnx-engine/search-index/src/schema.rs
@@ -558,4 +558,17 @@ impl FieldDeclaration {
             FieldDeclaration::Facet { .. } => true,
         }
     }
+
+    #[inline]
+    pub fn is_stored(&self) -> bool {
+        match self {
+            FieldDeclaration::F64 { opts } => opts.base.stored,
+            FieldDeclaration::U64 { opts } => opts.base.stored,
+            FieldDeclaration::I64 { opts } => opts.base.stored,
+            FieldDeclaration::Date { opts } => opts.base.stored,
+            FieldDeclaration::Text { opts } => opts.stored,
+            FieldDeclaration::String { opts } => opts.stored,
+            FieldDeclaration::Facet { opts } => opts.stored,
+        }
+    }
 }
diff --git a/lnx-engine/search-index/src/structures.rs b/lnx-engine/search-index/src/structures.rs
@@ -852,6 +852,10 @@ impl DocumentHit {
     ) -> Self {
         let mut compliant = HashMap::with_capacity(doc.0.len());
         for (name, info) in ctx.fields() {
+            if !info.is_stored() {
+                continue;
+            }
+
             let val = match doc.0.remove(name) {
                 Some(mut val) => {
                     if info.is_multi() {