waibu-db 2.12.1 → 2.12.4

package/index.js

@@ -157,10 +157,9 @@ async function factory (pkgName) {
       const q = query ?? set({}, field, { $in })
       const options = {
         dataOnly: true,
-        limit: -1,
         query: q
       }
-      return await this.findRecord({ model, req, options })
+      return await this.findAllRecord({ model, req, options })
     }
 
     formatRecord = async ({ data, req, schema, options = {} }) => {

package/lib/crud/add-handler.js

@@ -6,7 +6,7 @@ async function addHandler ({ req, reply, model, params = {}, template, addOnsHan
   const { pick, map, merge, omit } = this.app.lib._
   const opts = merge({}, options.modelOpts)
   model = pascalCase(model ?? req.params.model)
-  const { schema } = await getSchemaExt(model, 'add', merge({}, { params }, options))
+  const { schema } = await getSchemaExt(model, 'add', merge({ args: [req] }, { params }, options))
   if (schema.disabled.includes('create')) return await reply.view(templateDisabled, { action: 'add' })
   // req.query.attachment = true
   opts.fields = schema.view.fields

package/lib/crud/delete-handler.js

@@ -5,14 +5,14 @@ async function deleteHandler ({ req, reply, model, params = {}, templateDisabled
   const { reduce, merge } = this.app.lib._
   const opts = merge({}, options.modelOpts)
   model = pascalCase(model ?? req.params.model)
-  const { schema } = await getSchemaExt(model, 'add', merge({}, { params }, options))
+  const { schema } = await getSchemaExt(model, 'delete', merge({ args: [req] }, { params }, options))
   if (schema.disabled.includes('remove')) return await reply.view(templateDisabled, { action: 'delete' })
   opts.fields = schema.view.fields
   delete req.query.query
   const ids = (req.body.ids ?? '').split(',')
   if (ids.length > 0) {
     const result = []
-    opts.noResult = true
+    // opts.noResult = true
     opts.noFlash = true
     for (const id of ids) {
       try {

package/lib/crud/details-handler.js

@@ -6,7 +6,7 @@ async function detailsHandler ({ req, reply, model, params = {}, id, template, a
   const { merge, isEmpty } = this.app.lib._
   const opts = merge({ refs: '*' }, options.modelOpts)
   model = pascalCase(model ?? req.params.model)
-  const { schema } = await getSchemaExt(model, 'details', merge({}, { params }, options))
+  const { schema } = await getSchemaExt(model, 'details', merge({ args: [req] }, { params }, options))
   if (schema.disabled.includes('get')) return await reply.view(templateDisabled, { action: 'details' })
   // req.query.attachment = true
   opts.fields = schema.view.fields

package/lib/crud/edit-handler.js

@@ -13,7 +13,7 @@ async function editHandler ({ req, reply, model, id, params = {}, template, addO
   let resp
   let form
   model = pascalCase(model ?? req.params.model)
-  const { schema } = await getSchemaExt(model, 'edit', merge({}, { params }, options))
+  const { schema } = await getSchemaExt(model, 'edit', merge({ args: [req] }, { params }, options))
   if (schema.disabled.includes('update')) return await reply.view(templateDisabled, { action: 'edit' })
   // req.query.attachment = true
   opts.fields = schema.view.fields

package/lib/crud/export-handler.js

@@ -9,7 +9,7 @@ async function exportHandler ({ req, reply, model, params = {}, templateDisabled
   const { buildUrl } = this.app.waibuMpa
   const { pushDownload } = getPlugin('sumba')
   model = pascalCase(model ?? req.params.model)
-  const { schema } = await getSchemaExt(model, 'add', merge({}, { params }, options))
+  const { schema } = await getSchemaExt(model, 'add', merge({ args: [req] }, { params }, options))
   if (schema.disabled.includes('find')) return await reply.view(templateDisabled, { action: 'list' })
   const data = await prepCrud.call(getPlugin('waibuDb'), { model, req, reply, args: ['model'] })
   data.opts = omit(data.opts, ['req', 'reply'])

package/lib/crud/list-handler.js

@@ -6,7 +6,7 @@ async function listHandler ({ req, reply, model, template, params = {}, addOnsHa
 
   const opts = merge({}, { count: true, refs: '*' }, options.modelOpts)
   model = pascalCase(model ?? req.params.model)
-  const { schema } = await getSchemaExt(model, 'list', merge({}, { params }, options))
+  const { schema } = await getSchemaExt(model, 'list', merge({ args: [req] }, { params }, options))
   if (schema.disabled.includes('find')) return await reply.view(templateDisabled, { action: 'list' })
   for (const key of ['sort', 'limit', 'fields']) {
     const sessKey = `wdb${model}${upperFirst(key)}`

package/lib/method/count-record.js

@@ -4,7 +4,7 @@ async function countRecord ({ model, req, reply, options = {}, transaction } = {
   const { model: mdl, opts, filter } = await prepCrud.call(this, { model, req, reply, options, args: ['model'], transaction })
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     return await mdl.countRecord(filter, opts)
   }
 

package/lib/method/create-aggregate.js

@@ -9,7 +9,7 @@ async function createAggregate ({ model, req, reply, options = {}, transaction }
   params.aggregates = params.aggregates ?? ['count']
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     return await mdl.createAggregate(filter, params, opts)
   }
 

package/lib/method/create-histogram.js

@@ -9,7 +9,7 @@ async function createHistogram ({ model, req, reply, options = {}, transaction }
   params.aggregates = params.aggregates ?? ['count']
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     return await mdl.createHistogram(filter, params, opts)
   }
 

package/lib/method/find-all-record.js

@@ -4,7 +4,7 @@ async function findAllRecord ({ model, req, reply, options = {}, transaction } =
   const { model: mdl, opts, filter, attachment, stats, mimeType } = await prepCrud.call(this, { model, req, reply, options, args: ['model'], transaction })
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     const ret = await mdl.findAllRecord(filter, opts)
     if (attachment) {
       for (const d of ret.data) {

package/lib/method/find-one-record.js

@@ -4,7 +4,7 @@ async function findOneRecord ({ model, req, reply, options = {}, transaction } =
   const { model: mdl, opts, filter, attachment, stats, mimeType } = await prepCrud.call(this, { model, req, reply, options, args: ['model'], transaction })
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     const ret = await mdl.findOneRecord(filter, opts)
     if (attachment) ret.data._attachment = await mdl.findAttachment(ret.data.id, { stats, mimeType })
     return ret

package/lib/method/find-record.js

@@ -4,7 +4,7 @@ async function findRecord ({ model, req, reply, options = {}, transaction } = {}
   const { model: mdl, opts, filter, attachment, stats, mimeType } = await prepCrud.call(this, { model, req, reply, options, args: ['model'], transaction })
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     const ret = await mdl.findRecord(filter, opts)
     if (attachment) {
       for (const d of ret.data) {

package/lib/method/get-record.js

@@ -5,7 +5,7 @@ async function getRecord ({ model, req, reply, id, options = {}, transaction } =
   const me = this
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     const data = await getOneRecord.call(me, mdl, recId, filter, opts)
     if (attachment) data._attachment = await mdl.findAttachment(data.id, { stats, mimeType })
     return data

package/lib/method/get-schema-ext.js

@@ -165,6 +165,9 @@ const handler = {
   },
   edit: async function (schema, ext, options) {
     applyLayout.call(this, 'edit', schema, ext, options)
+  },
+  delete: async function (schema, ext, options) {
+    applyLayout.call(this, 'delete', schema, ext, options)
   }
 }
 

package/lib/method/remove-record.js

@@ -5,7 +5,7 @@ async function removeRecord ({ model, req, reply, id, options = {}, transaction
   const me = this
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     const resp = await getOneRecord.call(me, mdl, recId, filter, opts)
     opts._data = resp.data
     return await mdl.removeRecord(recId, opts)

package/lib/method/update-record.js

@@ -5,7 +5,7 @@ async function updateRecord ({ model, req, reply, id, body, options = {}, transa
   const me = this
 
   async function handler (trx) {
-    opts.trx = opts.trx || trx
+    if (opts.trx === true) opts.trx = trx
     const resp = await getOneRecord.call(me, mdl, recId, filter, opts)
     opts._data = resp.data
     const ret = await mdl.updateRecord(recId, input, opts)

package/lib/util.js

@@ -14,7 +14,6 @@ export async function prepCrud ({ model, body, id, req, reply, transaction, opti
   if (opts.suppressError === true) opts.suppressError = actions
   else if (isString(opts.suppressError)) opts.suppressError = [opts.suppressError]
   else opts.suppressError = opts.suppressError ?? []
-  opts.trx = options.trx
   const params = this.getParams(req, ...args)
   for (const k of ['count', 'fields']) {
     opts[k] = opts[k] ?? params[k]
@@ -23,7 +22,7 @@ export async function prepCrud ({ model, body, id, req, reply, transaction, opti
   opts.dataOnly = opts.dataOnly ?? false
   opts.req = req
   opts.reply = reply
-  opts.trx = opts.trx || transaction
+  opts.trx = opts.trx ?? options.trx ?? transaction ?? true
 
   let { attachment, stats, mimeType } = opts
   attachment = attachment ?? req.query.attachment

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "waibu-db",
-  "version": "2.12.1",
+  "version": "2.12.4",
   "description": "DB Helper",
   "main": "index.js",
   "scripts": {

package/wiki/CHANGES.md

@@ -1,9 +1,14 @@
 # Changes
 
+## 2026-03-30
+
+- [2.12.3] Bug fix in transaction supports
+- [2.12.4] Bug fix in ```options.limit```, removed due to use ```findAllRecords()```
+
 ## 2026-03-27
 
 - [2.12.1] Bug fix in all ```view.formatter``` & ```view.valueFormatter```
-- [2.12.1] Bug fix in ```wdb-form``` & ```wdb-table``` widgets, now correctly use value from formatter if provided
+- [2.12.2] Bug fix in ```wdb-form``` & ```wdb-table``` widgets, now correctly use value from formatter if provided
 
 ## 2026-03-26