kempo-inventory 0.5.7 → 0.5.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kempo-inventory",
-  "version": "0.5.7",
+  "version": "0.5.8",
   "description": "A basic inventory management system for kempo cms",
   "license": "MIT",
   "author": "Dustin Poissant <dustinpoissant@gmail.com>",

package/public/components/FieldsManager.js

@@ -22,7 +22,7 @@ import { getFields, createField, updateField, deleteField, getCategories } from
 */
 const TYPE_LABELS = {
   text: 'Text',
-  url: 'Web address (link)',
+  url: 'URL',
   longtext: 'Long text',
   number: 'Number',
   boolean: 'Yes / No',

package/public-items/[id]/index.page.html

@@ -113,7 +113,7 @@
 								: f.type === 'item' && linked[item.fields[f.key]]
 								? html`<a href="/items/${item.fields[f.key]}">${displayValue(f, item.fields[f.key], context)}</a>`
 								: f.type === 'url'
-									? html`<a href=${item.fields[f.key]} rel="noopener">${item.fields[f.key]}</a>`
+									? html`<a href=${item.fields[f.key]} target="_blank" rel="noopener">${item.fields[f.key]}</a>`
 									: f.type === 'longtext'
 									? html`<span style="white-space: pre-wrap;">${item.fields[f.key]}</span>`
 									: displayValue(f, item.fields[f.key], context)}</dd>`)}

package/server/utils/fieldTypes.js

@@ -24,7 +24,7 @@ export const MAX_IMAGE_EDGE = 8000;
   else, such as text to a number, could orphan existing values, so it is refused: delete the field
   and make a new one instead.
 */
-export const CONVERSIONS = { select: ['text'], text: ['longtext'] };
+export const CONVERSIONS = { select: ['text'], text: ['longtext', 'url'] };
 
 export const canConvert = (from, to) => from === to || (CONVERSIONS[from] ?? []).includes(to);
 

package/server/utils/fields.js

@@ -3,7 +3,7 @@ import { eq, and, asc, sql } from 'drizzle-orm';
 import crypto from 'crypto';
 import { kempoInventoryField, kempoInventoryItem, kempoInventoryCategory } from '../db/schema.js';
 import { categoryKey, tidyCategory, MAX_CATEGORY } from './categoryLogic.js';
-import { normalizeFieldDefinition, OPTIONAL_TYPES, canConvert } from './fieldTypes.js';
+import { normalizeFieldDefinition, OPTIONAL_TYPES, canConvert, coerceValue } from './fieldTypes.js';
 import { mediaAvailable } from './media.js';
 import { EVENTS, guard, notify } from './hooks.js';
 
@@ -11,6 +11,22 @@ const newId = () => crypto.randomBytes(8).toString('hex');
 
 const isUniqueViolation = error => (error?.code || error?.cause?.code) === '23505';
 
+/*
+  The values items hold for a key, rewritten as full web addresses where they are one. Anything else is left
+  as it is (it still shows as text, and is not made into a link).
+*/
+const addressesOf = async (key, label) => {
+  const rows = await db.select({ id: kempoInventoryItem.id, data: kempoInventoryItem.data }).from(kempoInventoryItem)
+    .where(sql`${kempoInventoryItem.data}->>${key} IS NOT NULL`);
+  const rewrites = [];
+  for(const row of rows){
+    const value = row.data[key];
+    if(value === '' || value === null) continue;
+    const [invalid, address] = coerceValue({ type: 'url', label }, value);
+    if(!invalid && address && address !== value) rewrites.push({ id: row.id, data: { ...row.data, [key]: address } });
+  }
+  return rewrites;
+};
 /*
   Properties an extension's own field accepts on update, and what anyone else (the admin UI) may
   adjust on a field an extension owns. Key and ownership never change after creation, the scope
@@ -160,6 +176,9 @@ export const updateField = async (key, changes = {}, { owner = null, category =
   }
   if(!Object.keys(updates).length) return [{ code: 400, msg: 'No changes provided' }, null];
 
+  /* Text to a URL: values that are web addresses are kept as their full address; anything else stays text. */
+  const rewrites = targetType === 'url' && existing.type !== 'url' ? await addressesOf(key, existing.label) : [];
+
   const refused = await guard(EVENTS.fieldBeforeUpdate, { field: existing, changes: updates, actor: owner ?? '' });
   if(refused) return [refused, null];
 
@@ -172,6 +191,9 @@ export const updateField = async (key, changes = {}, { owner = null, category =
           .values({ id: newId(), key: targetScope, name: targetName, image: null, created: now, updated: now })
           .onConflictDoNothing();
       }
+      for(const rewrite of rewrites){
+        await tx.update(kempoInventoryItem).set({ data: rewrite.data }).where(eq(kempoInventoryItem.id, rewrite.id));
+      }
       const [updated] = await tx.update(kempoInventoryField).set(updates)
         .where(and(eq(kempoInventoryField.key, key), eq(kempoInventoryField.category, existing.category))).returning();
       return updated;