dobo 1.1.15 → 1.1.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dobo",
-  "version": "1.1.15",
+  "version": "1.1.17",
   "description": "Database ORM/ODM for Bajo Framework",
   "main": "index.js",
   "scripts": {

package/plugin/factory.js

@@ -5,6 +5,7 @@ import collectSchemas from '../lib/collect-schemas.js'
 import memDbStart from '../lib/mem-db/start.js'
 import memDbInstantiate from '../lib/mem-db/instantiate.js'
 import nql from '@tryghost/nql'
+import path from 'path'
 
 async function factory (pkgName) {
   const me = this
@@ -360,6 +361,34 @@ async function factory (pkgName) {
       if (fields.length === 0) return all
       return map(all, item => pick(item, fields))
     }
+
+    listAttachments = async ({ model, id = '*', field = '*', file = '*' } = {}, { uriEncoded = true } = {}) => {
+      const { map, kebabCase } = this.lib._
+      const { importPkg, getPluginDataDir, pascalCase } = this.app.bajo
+      const mime = await importPkg('waibu:mime')
+      const { fastGlob } = this.lib
+      const root = `${getPluginDataDir('dobo')}/attachment`
+      model = pascalCase(model)
+      let pattern = `${root}/${model}/${id}/${field}/${file}`
+      if (uriEncoded) pattern = pattern.split('/').map(p => decodeURI(p)).join('/')
+      return map(await fastGlob(pattern), f => {
+        const mimeType = mime.getType(path.extname(f)) ?? ''
+        const fullPath = f.replace(root, '')
+        const row = {
+          file: f,
+          fileName: path.basename(fullPath),
+          fullPath,
+          mimeType,
+          params: { model, id, field, file }
+        }
+        if (this.app.waibuMpa) {
+          const { routePath } = this.app.waibu
+          const [, _model, _id, _field, _file] = fullPath.split('/')
+          row.url = routePath(`dobo:/attachment/${kebabCase(_model)}/${_id}/${_field}/${_file}`)
+        }
+        return row
+      })
+    }
   }
 }
 

package/plugin/method/attachment/copy-uploaded.js

@@ -20,8 +20,9 @@ async function copyUploaded (name, id, options = {}) {
     if (parts.length === 0) continue
     field = setField ?? field
     const file = setFile ?? parts.join('@')
-    const opts = { source: f, field, file, mimeType, stats, req }
+    const opts = { source: f, field, file, mimeType, stats, req, silent: true }
     const rec = await this.attachmentCreate(name, id, opts)
+    if (!rec) continue
     delete rec.dir
     result.push(rec)
     if (setField || setFile) break

package/plugin/method/attachment/create.js

@@ -2,9 +2,11 @@ import mergeAttachmentInfo from '../../../lib/merge-attachment-info.js'
 
 async function create (name, id, options = {}) {
   const { fs } = this.lib
+  const { isEmpty } = this.lib._
   name = this.attachmentPreCheck(name)
   if (!name) return
-  const { source, field, file } = options
+  const { source, field = 'file', file } = options
+  if (isEmpty(file)) return
   if (!source) throw this.error('isMissing%s', this.print.write('field.source'))
   const baseDir = await this.attachmentGetPath(name, id, field, file, { dirOnly: true })
   const { fullPath, stats, mimeType, req } = options
@@ -20,7 +22,7 @@ async function create (name, id, options = {}) {
     file
   }
   await mergeAttachmentInfo.call(this, rec, dest, { mimeType, fullPath, stats })
-  if (req && req.flash) req.flash('notify', req.t('attachmentUploaded'))
+  if (!options.silent && req && req.flash) req.flash('notify', req.t('attachmentUploaded'))
   return rec
 }
 

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

@@ -16,6 +16,7 @@ async function findOne (name, filter = {}, opts = {}) {
   options.dataOnly = false
   await this.modelExists(name, true)
   filter.limit = 1
+  filter.page = 1
   const { handler, schema, driver } = await resolveMethod.call(this, name, 'record-find', options)
   if (!schema.cacheable) noCache = true
   filter.query = await this.buildQuery({ filter, schema, options }) ?? {}
@@ -34,6 +35,7 @@ async function findOne (name, filter = {}, opts = {}) {
     }
   }
   filter.limit = 1
+  filter.page = 1
   const record = options.record ?? (await handler.call(this.app[driver.ns], { schema, filter, options }))
   delete options.record
   record.data = record.data[0]

package/waibuMpa/route/attachment/@model/@id/@field/@file.js

@@ -1,25 +1,34 @@
 import path from 'path'
 
 async function attachment (req, reply) {
-  const { isString, isEmpty } = this.lib._
-  const { importPkg, getPluginDataDir, pascalCase } = this.app.bajo
+  const { isString, isEmpty, find } = this.lib._
+  const { pascalCase } = this.app.bajo
   const { routePath } = this.app.waibu
-  const mime = await importPkg('waibu:mime')
-  const { fs, fastGlob } = this.lib
-  let file = `${getPluginDataDir('dobo')}/attachment/${pascalCase(req.params.model)}/${req.params.id}/${req.params.field}/${req.params.file}`
-  if (path.basename(file) === '_first') {
-    const files = await fastGlob(`${path.dirname(file)}/*`)
-    if (files.length > 0) file = files[0]
+  const { fs } = this.lib
+  const items = await this.listAttachments({
+    model: req.params.model,
+    id: req.params.id,
+    field: req.params.field,
+    file: '*'
+  })
+  let item = req.params.file === '_first' ? items[0] : undefined
+  if (!item) {
+    item = find(items, i => {
+      const [, model, id, field, file] = i.fullPath.split('/')
+      return model === pascalCase(req.params.model) &&
+        id === decodeURI(req.params.id) &&
+        field === decodeURI(req.params.field) &&
+        file === decodeURI(req.params.file)
+    })
   }
-  const mimeType = mime.getType(path.extname(file)) ?? ''
-  if (!fs.existsSync(file)) {
+  if (!item) {
     if (!req.query.notfound) throw this.error('_notFound', { noContent: true })
-    const [, ext] = mimeType.split('/')
-    const replacer = isString(req.query.notfound) ? req.query.notfound : `waibuStatic.asset:/not-found.${ext ?? 'png'}`
+    const ext = path.extname(req.params.file)
+    const replacer = isString(req.query.notfound) ? req.query.notfound : `waibuStatic.asset:/not-found${isEmpty(ext) ? '.png' : ext}`
     return reply.redirectTo(routePath(replacer))
   }
-  if (!isEmpty(mimeType)) reply.header('Content-Type', mimeType)
-  const stream = fs.createReadStream(file)
+  if (!isEmpty(item.mimeType)) reply.header('Content-Type', item.mimeType)
+  const stream = fs.createReadStream(item.file)
   reply.send(stream)
   return reply
 }