@ditojs/admin 3.0.0 → 3.1.0

package/src/utils/FormModel.js

@@ -12,7 +12,9 @@ import { DataSchemaResolver } from './DataSchemaResolver.js'
 import { isNested, isSourceWithResource } from './schema/structure.js'
 import { isEmptySchema } from './schema/lookup.js'
 import {
-  shouldRenderSchema,
+  getSchemaValue,
+  shouldSetDefaultValue,
+  getDefaultValue,
   initializeData,
   computeValue,
   hasValueFromDataSchema,
@@ -23,9 +25,10 @@ import {
 // FormModel holds the state of the data edited by a form, view or dialog that
 // is derived from schema and data, independently of what is rendered:
 //
-// - Missing values are set to their defaults when the data is set up, except
-//   the values of components with `compute()`, which fall back to their
-//   defaults when `compute()` doesn't return a value, unlike in new data, see
+// - Missing values are set to their defaults when the data is set up, and when
+//   they go missing later, e.g. in items that code adds, except the values of
+//   components with `compute()`, which fall back to their defaults when
+//   `compute()` doesn't return a value, unlike in new data, see
 //   `initializeData()`.
 // - The items of lists with `orderKey` are numbered when the data is set up,
 //   which doesn't make the data dirty.
@@ -94,12 +97,19 @@ export class FormModel {
       // The entries are read through a computed property, so that the watchers
       // of the entries can check synchronously whether their entry is still
       // current, see `createComputedValueScope()`.
-      this.computedValueEntries = computed(() => this.getComputedValueEntries())
+      this.dataEntries = computed(() => this.getDataEntries())
       watch(
-        () => this.computedValueEntries.value,
+        () => this.dataEntries.value.computedValueEntries,
         entries => this.updateComputedValueRecords(entries),
         { immediate: true }
       )
+      // Values that go missing after the data was set up, e.g. in items that
+      // code adds, get their defaults too:
+      watch(
+        () => this.dataEntries.value.entriesWithMissingValues,
+        entries => this.setDefaultValues(entries),
+        { immediate: true }
+      )
       if (getSourceSchema) {
         // Data that is set up, e.g. loaded, saved or applied, isn't dirty:
         watch(
@@ -293,32 +303,79 @@ export class FormModel {
     }
   }
 
-  // Returns the entries of `processSchemaData()` for all components with
-  // computed values whose `if` doesn't evaluate to `false`, by component path.
-  // Called by a computed property, so that it runs again when the data
-  // structure changes, e.g. when list items are added or removed, or when `if`
-  // conditions change.
-  getComputedValueEntries() {
+  // Walks the data and returns the entries of `processSchemaData()` of the
+  // components whose `if` doesn't evaluate to `false`: the ones with computed
+  // values, by component path, and the ones whose values are missing. Called
+  // by a computed property, so that it runs again when the data structure
+  // changes, e.g. when list items are added or removed, values go missing, or
+  // `if` conditions change.
+  getDataEntries() {
     const schema = this.getSchema()
     const data = this.getData()
-    const entries = new Map()
+    const computedValueEntries = new Map()
+    const entriesWithMissingValues = []
     if (data && !isEmptySchema(schema)) {
       processSchemaData(schema, data, {
         dataPath: this.dataPath,
         componentPath: this.componentPath,
         shouldProcess: entry => (
-          !isSourceWithResource(entry.schema) &&
-          shouldRenderSchema(entry.schema, this.createEntryContext(entry))
+          !isSourceWithResource(entry.schema) && this.isEntryShown(entry)
         ),
         before: entry => {
           if (hasComputedValueSource(entry.schema)) {
-            entries.set(entry.componentPath, entry)
+            computedValueEntries.set(entry.componentPath, entry)
+          }
+          if (this.shouldSetDefaultValue(entry)) {
+            entriesWithMissingValues.push(entry)
           }
         },
         options: { component: this.component, rootData: this.rootData }
       })
     }
-    return entries
+    return { computedValueEntries, entriesWithMissingValues }
+  }
+
+  // Returns whether the `if` of the entry's component doesn't evaluate to
+  // `false`. Unlike `shouldRenderSchema()`, it doesn't evaluate the components
+  // of sections and tabs, as the walk of the data visits them anyway, see
+  // `getDataEntries()`, which leads to the same entries.
+  isEntryShown(entry) {
+    return (
+      entry.schema.if === undefined ||
+      getSchemaValue('if', {
+        type: Boolean,
+        schema: entry.schema,
+        context: this.createEntryContext(entry),
+        default: true
+      })
+    )
+  }
+
+  // Returns whether the value of the entry is missing and needs its default.
+  // The model writes the values of components with `compute()`, including
+  // their defaults, see `computeValue()`.
+  shouldSetDefaultValue(entry) {
+    const { schema, data, name } = entry
+    return shouldSetDefaultValue(
+      schema,
+      data,
+      name,
+      () => this.createEntryContext(entry),
+      { shouldSetDefaultsOfComponentsWithCompute: false }
+    )
+  }
+
+  // Sets the defaults of the values of the entries, unless they were set in
+  // the meantime.
+  setDefaultValues(entries) {
+    for (const entry of entries) {
+      if (this.shouldSetDefaultValue(entry)) {
+        const { schema, data, name } = entry
+        data[name] = getDefaultValue(schema, () =>
+          this.createEntryContext(entry)
+        )
+      }
+    }
   }
 
   // Keeps the records of the components that are still present with the same
@@ -395,9 +452,10 @@ export class FormModel {
   }
 
   // Returns whether the entry is still one of the current entries, with the
-  // same schema and data, see `getComputedValueEntries()`.
+  // same schema and data, see `getDataEntries()`.
   isEntryCurrent({ componentPath, schema, data }) {
-    const currentEntry = this.computedValueEntries.value.get(componentPath)
+    const currentEntry =
+      this.dataEntries.value.computedValueEntries.get(componentPath)
     return currentEntry?.schema === schema && currentEntry.data === data
   }
 
@@ -466,7 +524,7 @@ export class FormModel {
     const tokens = isNested(schema)
       ? parseDataPath(dataPath)
       : [...parseDataPath(dataPath), name]
-    return tokens.slice(parseDataPath(this.dataPath).length)
+    return tokens.slice(parseDataPath(this.dataPath).length).join('/')
   }
 
   // Returns the context for `if`, data schemas and options, with the component
@@ -497,8 +555,9 @@ function takeOverValuesAtDataPaths(
   }
   const result = clone(processedData)
   for (const dataPath of dataPaths) {
-    const parentDataPath = dataPath.slice(0, -1)
-    const key = dataPath.at(-1)
+    const tokens = parseDataPath(dataPath)
+    const parentDataPath = tokens.slice(0, -1)
+    const key = tokens.at(-1)
     const parent = getValueAtDataPath(result, parentDataPath, () => null)
     if (parent && typeof parent === 'object') {
       const value = getValueAtDataPath(

package/src/utils/request.js

@@ -0,0 +1,70 @@
+import { isAbsoluteUrl } from '@ditojs/utils'
+import { formatQuery } from './route.js'
+
+export class RequestError extends Error {
+  constructor(response) {
+    super(
+      `Request failed with status code: ${response.status} (${
+        response.statusText
+      })`
+    )
+    this.response = response
+  }
+}
+
+export async function request(api, {
+  url,
+  method = 'get',
+  query = null,
+  headers = null,
+  data = null,
+  signal = null,
+  timeout = api.timeout ?? null
+}) {
+  const isApiUrl = api.isApiUrl(url)
+  // Abort requests that take longer than the timeout, if there is one, so that
+  // nothing waits for them forever, e.g. submitting for pending loads:
+  const signals = [signal, timeout && AbortSignal.timeout(timeout)]
+  signal = AbortSignal.any(signals.filter(Boolean))
+
+  const response = await fetch(api.getApiUrl({ url, query }), {
+    method: method.toUpperCase(),
+    ...(data && { body: JSON.stringify(data) }),
+    headers: {
+      ...(isApiUrl && api.headers),
+      ...headers
+    },
+    credentials:
+      isApiUrl && api.cors?.credentials
+        ? 'include'
+        : 'same-origin',
+    signal
+  })
+
+  if (response.headers.get('Content-Type')?.includes('application/json')) {
+    response.data = await response.json()
+  }
+
+  if (!response.ok) {
+    throw new RequestError(response)
+  }
+  return response
+}
+
+export function isApiUrl(api, url) {
+  return !isAbsoluteUrl(url) || url.startsWith(api.url)
+}
+
+export function getApiUrl(api, { url, query }) {
+  if (!url.startsWith(api.url) && !isAbsoluteUrl(url)) {
+    url = combineUrls(api.url, url)
+  }
+  // Support optional query parameters, to be are added to the URL.
+  const search = formatQuery(query)
+  return search ? `${url}${url.includes('?') ? '&' : '?'}${search}` : url
+}
+
+function combineUrls(baseUrl, relativeUrl) {
+  // Use same approach as axios `combineURLs()` to join baseUrl & relativeUrl:
+  return `${baseUrl.replace(/\/+$/, '')}/${relativeUrl.replace(/^\/+/, '')}`
+}

package/src/utils/schema/data.js

@@ -126,10 +126,12 @@ export function shouldIgnoreMissingValue(schema, context) {
 // is missing in `data`: if there's no value, or if the value is `null` and
 // the type treats that as missing, e.g. nested sections.
 export function isMissingValue(schema, data, name, context) {
+  // Only read the value if needed, so that callers that are tracked, e.g. the
+  // walk of the form model's data, don't depend on all values.
   return (
     !(name in data) ||
-    data[name] === null &&
-    !!getTypeOptions(schema)?.treatNullAsMissing?.(getContext(context))
+    !!getTypeOptions(schema)?.treatNullAsMissing?.(getContext(context)) &&
+    data[name] === null
   )
 }
 
@@ -150,6 +152,22 @@ export function getMultipleValue(schema) {
   return schema.multiple ?? !!getTypeOptions(schema)?.defaultMultiple
 }
 
+/**
+ * Returns whether the value of the component described by `schema` and `name`
+ * is missing in `data` and needs its default, see `initializeData()`.
+ * `shouldSetDefaultsOfComponentsWithCompute` decides for components with
+ * `compute()`, which may derive their missing values instead.
+ */
+export function shouldSetDefaultValue(schema, data, name, context, {
+  shouldSetDefaultsOfComponentsWithCompute = true
+} = {}) {
+  return (
+    (shouldSetDefaultsOfComponentsWithCompute || !schema.compute) &&
+    isMissingValue(schema, data, name, context) &&
+    !shouldIgnoreMissingValue(schema, context)
+  )
+}
+
 /**
  * Initializes `data` for the components of `schema`: Sets missing values to
  * their defaults, so they can be correctly watched for changes, and numbers
@@ -177,11 +195,14 @@ export function initializeData(schema, data = {}, component, {
         dataPath,
         rootData
       })
-    if (
-      (shouldSetDefaultsOfComponentsWithCompute || !schema.compute) &&
-      isMissingValue(schema, data, name, context) &&
-      !shouldIgnoreMissingValue(schema, context)
-    ) {
+    const shouldSetDefault = shouldSetDefaultValue(
+      schema,
+      data,
+      name,
+      context,
+      { shouldSetDefaultsOfComponentsWithCompute }
+    )
+    if (shouldSetDefault) {
       data[name] = getDefaultValue(schema, context)
     }
     if (hasItemsNumberedByOrderKey(schema) && isArray(data[name])) {
@@ -203,8 +224,9 @@ export function initializeData(schema, data = {}, component, {
  * value in `data`, or its default if it's missing. Never writes into `data`:
  * Computed values, including the defaults of components with `compute()`, are
  * written by `FormModel`, and other defaults when the data is set up, see
- * `initializeData()`. If provided, `getOptions()` returns the
- * options for `context.options`, called only when they're read.
+ * `initializeData()`, or when they go missing later, see `FormModel`. If
+ * provided, `getOptions()` returns the options for `context.options`, called
+ * only when they're read.
  */
 export function computeValue(schema, data, name, dataPath, {
   component = null,
@@ -406,11 +428,19 @@ export function processData(schema, sourceSchema, data, dataPath, {
  * returns `false` are skipped along with their content. The entries passed to
  * them contain the `componentPath` of the component, tab or panel, continuing
  * `componentPath` like `DitoMixin.componentPath` does.
+ *
+ * The primitive values of lists with `wrapPrimitives` are walked wrapped in
+ * objects under the `wrapPrimitives` key, like the admin edits them, see
+ * `SourceMixin`. `wrappedPrimitiveName` is that key while walking such an
+ * object, and its component keeps the data path of the item, where the admin
+ * maps errors of the value to. Values written into the objects don't reach
+ * the data, e.g. defaults, as the primitive values themselves are the data.
  */
 export function processSchemaData(schema, data, {
   dataPath = null,
   componentPath = '',
   processedData = null,
+  wrappedPrimitiveName = null,
   before = null,
   after = null,
   shouldProcess = () => true,
@@ -426,9 +456,11 @@ export function processSchemaData(schema, data, {
     if (components) {
       for (const [name, componentSchema] of Object.entries(components)) {
         const isNestedComponent = isNested(componentSchema)
-        const componentDataPath = isNestedComponent
-          ? getDataPath(dataPath, name)
-          : dataPath
+        const isWrappedPrimitive = name === wrappedPrimitiveName
+        const componentDataPath =
+          isNestedComponent && !isWrappedPrimitive
+            ? getDataPath(dataPath, name)
+            : dataPath
         const entry = {
           schema: componentSchema,
           data,
@@ -453,12 +485,12 @@ export function processSchemaData(schema, data, {
             processedData
           })
         } else {
+          const { wrapPrimitives } = componentSchema
           const processItem = (item, index = null) => {
-            if (!isObject(item)) {
-              // Items that aren't objects can't hold component values, e.g.
-              // the unwrapped values of lists with `wrapPrimitives` when
-              // walking data that isn't processed.
-              return item
+            // Data that isn't processed holds the primitive values themselves,
+            // processed data holds them wrapped already, see `processData()`.
+            if (wrapPrimitives && !isObject(item)) {
+              item = { [wrapPrimitives]: item }
             }
             const itemDataPath =
               index !== null
@@ -489,7 +521,8 @@ export function processSchemaData(schema, data, {
                 ...walkOptions,
                 dataPath: itemDataPath,
                 componentPath: itemComponentPath,
-                processedData: processedItem
+                processedData: processedItem,
+                wrappedPrimitiveName: wrapPrimitives ?? null
               })
             } else {
               // Items without a matching form, e.g. of an unknown type, are
@@ -500,29 +533,38 @@ export function processSchemaData(schema, data, {
 
           before?.(entry)
 
-          let value = processedData ? processedData[name] : data[name]
-          if (value != null && hasFormSchema(componentSchema)) {
-            // Recursively process data on nested form items.
-            if (isArray(value)) {
-              // Optimization: No need to collect values if we're not cloning!
-              value = processedData
-                ? value.map(processItem)
-                : value.forEach(processItem)
-            } else {
-              value = processItem(value)
-            }
-            if (processedData) {
-              processedData[name] = value
+          // Only read the values of sources with forms, so that callers that
+          // are tracked, e.g. the walk of the form model's data, don't depend
+          // on all values.
+          if (hasFormSchema(componentSchema)) {
+            let value = processedData ? processedData[name] : data[name]
+            if (value != null) {
+              // Recursively process data on nested form items.
+              if (isArray(value)) {
+                // Optimization: No need to collect values if not cloning!
+                value = processedData
+                  ? value.map(processItem)
+                  : value.forEach(processItem)
+              } else {
+                value = processItem(value)
+              }
+              if (processedData) {
+                processedData[name] = value
+              }
             }
           }
 
           after?.(entry)
+          // The panels of nested components display the data that contains
+          // the components, like the ones of unnested components, see
+          // `DitoContainer.panelEntries`.
+          processTabOrPanelSchemas(componentSchema.panels, entry.componentPath)
         }
       }
     }
   }
 
-  const processTabOrPanelSchemas = schemas => {
+  const processTabOrPanelSchemas = (schemas, parentComponentPath) => {
     // Tabs and panels add their names to component paths, not to data paths.
     for (const [name, tabOrPanelSchema] of Object.entries(schemas || {})) {
       const entry = {
@@ -530,7 +572,7 @@ export function processSchemaData(schema, data, {
         data,
         name: null,
         dataPath,
-        componentPath: appendDataPath(componentPath, name),
+        componentPath: appendDataPath(parentComponentPath, name),
         processedData
       }
       // The components of panels with their own data don't edit `data`.
@@ -541,8 +583,8 @@ export function processSchemaData(schema, data, {
   }
 
   processComponents(schema.components, componentPath)
-  processTabOrPanelSchemas(schema.tabs)
-  processTabOrPanelSchemas(schema.panels)
+  processTabOrPanelSchemas(schema.tabs, componentPath)
+  processTabOrPanelSchemas(schema.panels, componentPath)
 
   return processedData || data
 }

package/src/utils/schema/lookup.js

@@ -176,27 +176,10 @@ export function getButtonSchemas(buttons) {
   )
 }
 
-export function getPanelEntry(
-  schema,
-  dataPath = null,
-  componentPath = '',
-  tabComponent = null
-) {
-  return schema
-    ? {
-        schema,
-        // If the panel provides its own name, append it to the paths. This is
-        // used e.g. for $filters panels.
-        dataPath:
-          dataPath != null && schema.name
-            ? appendDataPath(dataPath, schema.name)
-            : dataPath,
-        componentPath: schema.name
-          ? appendDataPath(componentPath, schema.name)
-          : componentPath,
-        tabComponent
-      }
-    : null
+// Returns the entry of the panel described by `schema`, which `DitoPanels`
+// displays at `dataPath` and `componentPath`.
+function getPanelEntry(schema, dataPath, componentPath, tabComponent) {
+  return schema ? { schema, dataPath, componentPath, tabComponent } : null
 }
 
 export function getPanelEntries(
@@ -224,8 +207,9 @@ export function getPanelEntries(
 
 // Returns the entries of the panel that the type of the component described by
 // `schema` provides, and of the panels in `schema.panels`. The type's panel is
-// addressed relative to `dataPath` and its component path,
-// `dataPathComponentPath`, while the panels in `schema.panels` continue
+// addressed by its name, e.g. `$filters`, relative to `dataPath` and its
+// component path, `dataPathComponentPath`, while the panels in `schema.panels`
+// are addressed by their keys, see `getPanelEntries()`, continuing
 // `schemaComponentPath`, the component path of `schema` itself, which also
 // contains the names of unnested components.
 export function getAllPanelEntries(
@@ -247,8 +231,8 @@ export function getAllPanelEntries(
     ? [
         getPanelEntry(
           panelSchema,
-          dataPath,
-          dataPathComponentPath,
+          appendDataPath(dataPath, panelSchema.name),
+          appendDataPath(dataPathComponentPath, panelSchema.name),
           tabComponent
         )
       ]

package/types/index.d.ts

@@ -52,6 +52,12 @@ export type RequestMethod = <T>(options: {
   headers?: Record<string, string> | null
   /** Abort signal to cancel the request. */
   signal?: AbortSignal | null
+  /**
+   * Milliseconds after which the request is aborted.
+   *
+   * @defaultValue {@link ApiConfig.timeout}
+   */
+  timeout?: number | null
 }) => Promise<RequestMethodResponse<T>>
 
 /**
@@ -170,6 +176,14 @@ export interface ApiConfig {
          */
         durationFactor: number
       }
+  /**
+   * Milliseconds after which requests are aborted, so that
+   * nothing waits for them forever, e.g. submitting forms for
+   * pending loads of options. Requests can set their own.
+   *
+   * @defaultValue `null`, no timeout
+   */
+  timeout?: number | null
   /**
    * CORS settings applied to API requests (where
    * {@link ApiConfig.isApiUrl} returns `true`).
@@ -330,6 +344,11 @@ export interface BaseSchema<$Item>
    * when the field isn't rendered, unless its `if`
    * evaluates to `false`. If the callback returns
    * `undefined`, the current value is preserved.
+   *
+   * As it doesn't depend on the field's component, the
+   * context's component properties, e.g. `component`
+   * and `schemaComponent`, refer to the form, view or
+   * dialog that holds the data, not to the field.
    */
   compute?: ItemAccessor<$Item>
   /**
@@ -797,6 +816,12 @@ export type SchemaOptions<$Item, $Option = any> =
        * receives the `DitoContext` and returns an inner
        * function that is called to fetch the actual data,
        * enabling reactive dependency tracking.
+       *
+       * The options are loaded independently of the
+       * field's component, also for computes, so the
+       * context's component properties, e.g. `component`
+       * and `schemaComponent`, refer to the form, view or
+       * dialog that holds the data, not to the field.
        */
       data?: OrItemAccessor<
         $Item,