dobo 2.3.0 → 2.3.1

package/lib/collect-models.js

@@ -126,7 +126,7 @@ async function findAllIndexes (model, inputs = []) {
  * @param {Object} model - Model
  * @param {Array} [models] - All model match agaist. Defaults to ```dobo.models```
  */
-export async function sanitizeRef (model, models, fatal) {
+export async function sanitizeRef (model, models) {
   const { find, isString, pullAt } = this.app.lib._
   if (!models) models = this.models
   for (const prop of model.properties) {
@@ -139,13 +139,15 @@ export async function sanitizeRef (model, models, fatal) {
       ref.type = ref.type ?? '1:1'
       const rModel = find(models, { name: ref.model })
       if (!rModel) {
-        if (fatal) this.fatal('unknownModelForRef%s%s%s', ref.model, model.name, prop.name)
-        else ignored.push(ref.model)
+        ignored.push(key)
+        this.log.warn('notFound%s%s', this.t('model'), ref.model)
+        continue
       }
       const rProp = find(rModel.properties, { name: ref.propName })
       if (!rProp) {
-        if (fatal) this.fatal('unknownPropForRef%s%s%s%s', ref.model, ref.propName, model.name, prop.name)
-        else ignored.push(ref.model)
+        ignored.push(key)
+        this.log.warn('notFound%s%s', this.t('property'), `${ref.propName}@${ref.model}`)
+        continue
       }
       ref.fields = ref.fields ?? '*'
       if (['*', 'all'].includes(ref.fields)) ref.fields = rModel.properties.map(p => p.name)

package/lib/factory/model/_util.js

@@ -189,7 +189,7 @@ export async function getSingleRef (record = {}, options = {}) {
 
 export async function getMultiRefs (records = [], options = {}) {
   const { isSet } = this.app.lib.aneka
-  const { uniq, map, get } = this.app.lib._
+  const { uniq, without, get } = this.app.lib._
   const props = this.properties.filter(p => isSet(p.ref) && !(options.hidden ?? []).includes(p.name))
   options.refs = options.refs ?? []
   if (props.length > 0) {
@@ -201,15 +201,16 @@ export async function getMultiRefs (records = [], options = {}) {
         if (ref.type !== '1:1') continue
         if (get(records, `0._ref.${key}`)) continue
         const rModel = this.app.dobo.getModel(ref.model)
-        const matches = uniq(map(records, r => {
-          let v = r[prop.name]
-          if (ref.propName === 'id') v = this.sanitizeId(v)
-          return v
-        }))
+        let matches = []
+        for (const r of records) {
+          matches.push(rModel.sanitizeId(r[prop.name]))
+        }
+        matches = uniq(without(matches, undefined, null, NaN))
         const query = {}
         query[ref.propName] = { $in: matches }
         const rFilter = { query, limit: matches.length }
         const rOptions = { dataOnly: true, refs: [] }
+        this.app.dump(this.name, rModel.name, rFilter, records)
         const results = await rModel.findRecord(rFilter, rOptions)
         const fields = [...ref.fields]
         if (!fields.includes(prop.name)) fields.push(prop.name)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dobo",
-  "version": "2.3.0",
+  "version": "2.3.1",
   "description": "DBMS for Bajo Framework",
   "main": "index.js",
   "scripts": {

package/wiki/CHANGES.md

@@ -1,5 +1,10 @@
 # Changes
 
+## 2026-01-26
+
+- [2.3.1] Bug fix if reference model isn't loaded only yield warning
+- [2.3.1] Bug fix on fetching multi references
+
 ## 2026-01-24
 
 - [2.3.0] Add ```dynHooks``` to model's options