@ditojs/admin 3.1.0 → 3.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ditojs/admin",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "type": "module",
   "description": "Dito.js Admin is a schema based admin interface for Dito.js Server, featuring auto-generated views and forms and built with Vue.js",
   "repository": "https://github.com/ditojs/dito/tree/main/packages/admin",
@@ -41,8 +41,8 @@
     "not ie_mob > 0"
   ],
   "dependencies": {
-    "@ditojs/ui": "^3.1.0",
-    "@ditojs/utils": "^3.1.0"
+    "@ditojs/ui": "^3.1.1",
+    "@ditojs/utils": "^3.1.1"
   },
   "peerDependencies": {
     "@kyvg/vue3-notification": "^3.4.2",
@@ -87,7 +87,7 @@
     "vue-upload-component": "^3.2.2"
   },
   "devDependencies": {
-    "@ditojs/build": "^3.1.0",
+    "@ditojs/build": "^3.1.1",
     "@kyvg/vue3-notification": "^3.4.2",
     "@tiptap/core": "^3.31.4",
     "@tiptap/extension-blockquote": "^3.31.4",
@@ -131,5 +131,5 @@
     "vue-router": "^5.3.1",
     "vue-upload-component": "^3.2.2"
   },
-  "gitHead": "97314265c5cbac6a248d7350f43fea8ea92b995f"
+  "gitHead": "acce85bb38d817aab18f9d5d03b3aa9dff00c3f9"
 }

package/src/mixins/ResourceMixin.js

@@ -54,8 +54,10 @@ export default {
       // `isTransient()` to only return `!this.providesData`.
       const form = this.formComponent
       return (
-        !this.providesData &&
-        !this.linksToView ||
+        (
+          !this.providesData &&
+          !this.linksToView
+        ) ||
         form && (
           form.isTransient ||
           form.isCreating

package/src/types/DitoTypeTreeList.vue

@@ -87,9 +87,10 @@ export default DitoTypeComponent.register(
                 type: Boolean,
                 default: false,
                 schema
-              }) ||
-              schema.children &&
-              hasEditableForms(schema.children)
+              }) || (
+                schema.children &&
+                hasEditableForms(schema.children)
+              )
             )
           )
         }

package/src/utils/FormModel.js

@@ -9,7 +9,7 @@ import {
 } from '@ditojs/utils'
 import DitoContext from '../DitoContext.js'
 import { DataSchemaResolver } from './DataSchemaResolver.js'
-import { isNested, isSourceWithResource } from './schema/structure.js'
+import { isNested } from './schema/structure.js'
 import { isEmptySchema } from './schema/lookup.js'
 import {
   getSchemaValue,
@@ -295,7 +295,7 @@ export class FormModel {
       initializeData(schema, data, this.component, {
         dataPath: this.dataPath,
         rootData: this.rootData,
-        shouldProcess: entry => !isSourceWithResource(entry.schema),
+        shouldSkipSourcesWithResource: true,
         // The model writes the values of components with `compute()`,
         // including their defaults:
         shouldSetDefaultsOfComponentsWithCompute: false
@@ -318,9 +318,8 @@ export class FormModel {
       processSchemaData(schema, data, {
         dataPath: this.dataPath,
         componentPath: this.componentPath,
-        shouldProcess: entry => (
-          !isSourceWithResource(entry.schema) && this.isEntryShown(entry)
-        ),
+        shouldProcess: entry => this.isEntryShown(entry),
+        shouldSkipSourcesWithResource: true,
         before: entry => {
           if (hasComputedValueSource(entry.schema)) {
             computedValueEntries.set(entry.componentPath, entry)
@@ -585,12 +584,13 @@ function takeOverChangedValues(target, before, after) {
     return target
   }
   const isSameShape = (
-    isPlainObject(before) && isPlainObject(after) && isPlainObject(target) ||
-    isArray(before) &&
-    isArray(after) &&
-    isArray(target) &&
-    before.length === after.length &&
-    after.length === target.length
+    isPlainObject(before) && isPlainObject(after) && isPlainObject(target) || (
+      isArray(before) &&
+      isArray(after) &&
+      isArray(target) &&
+      before.length === after.length &&
+      after.length === target.length
+    )
   )
   if (!isSameShape) {
     return clone(after)

package/src/utils/schema/data.js

@@ -16,6 +16,7 @@ import {
   someNestedSchemaComponent,
   hasNestedSchemaComponents,
   isPanelWithOwnData,
+  isSourceWithResource,
   isNested
 } from './structure.js'
 import {
@@ -129,9 +130,10 @@ 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) ||
-    !!getTypeOptions(schema)?.treatNullAsMissing?.(getContext(context)) &&
-    data[name] === null
+    !(name in data) || (
+      !!getTypeOptions(schema)?.treatNullAsMissing?.(getContext(context)) &&
+      data[name] === null
+    )
   )
 }
 
@@ -178,6 +180,7 @@ export function initializeData(schema, data = {}, component, {
   dataPath = null,
   rootData = data,
   shouldProcess,
+  shouldSkipSourcesWithResource = false,
   // Whether to also set the defaults of components with `compute()`. New data
   // starts with all defaults, which `compute()` can rely on. In data that was
   // loaded, `compute()` may derive missing values instead, and they fall back
@@ -214,6 +217,7 @@ export function initializeData(schema, data = {}, component, {
     dataPath,
     before,
     shouldProcess,
+    shouldSkipSourcesWithResource,
     options
   })
 }
@@ -429,6 +433,11 @@ export function processData(schema, sourceSchema, data, dataPath, {
  * them contain the `componentPath` of the component, tab or panel, continuing
  * `componentPath` like `DitoMixin.componentPath` does.
  *
+ * With `shouldSkipSourcesWithResource`, sources with their own resource are
+ * skipped along with their items, which they load and save through their
+ * resource, but not their panels, which display the data that contains them,
+ * see `DitoContainer.panelEntries`.
+ *
  * 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
@@ -444,9 +453,16 @@ export function processSchemaData(schema, data, {
   before = null,
   after = null,
   shouldProcess = () => true,
+  shouldSkipSourcesWithResource = false,
   options
 }) {
-  const walkOptions = { before, after, shouldProcess, options }
+  const walkOptions = {
+    before,
+    after,
+    shouldProcess,
+    shouldSkipSourcesWithResource,
+    options
+  }
   const getDataPath = (dataPath, token) =>
     dataPath != null
       ? appendDataPath(dataPath, token)
@@ -472,6 +488,13 @@ export function processSchemaData(schema, data, {
         if (!shouldProcess(entry)) {
           continue
         }
+        if (
+          shouldSkipSourcesWithResource &&
+          isSourceWithResource(componentSchema)
+        ) {
+          processTabOrPanelSchemas(componentSchema.panels, entry.componentPath)
+          continue
+        }
         if (isPanelWithOwnData(componentSchema)) {
           // The components of panels with their own data don't edit `data`.
           continue

package/src/utils/schema/validation.js

@@ -2,7 +2,6 @@ import DitoContext from '../../DitoContext.js'
 import * as validators from '../../validators/index.js'
 import { isFunction, isRegExp, asArray, normalizeDataPath } from '@ditojs/utils'
 import { getTypeOptions } from './types.js'
-import { isSourceWithResource } from './structure.js'
 import {
   getSchemaValue,
   shouldRenderSchema,
@@ -86,10 +85,8 @@ export function getDataValidationErrors(schema, data, {
       rootData
     })
 
-  const shouldProcess = entry => (
-    !isSourceWithResource(entry.schema) &&
+  const shouldProcess = entry =>
     shouldRenderSchema(entry.schema, createContext(entry))
-  )
 
   const before = entry => {
     const { schema, data, name, dataPath } = entry
@@ -112,6 +109,7 @@ export function getDataValidationErrors(schema, data, {
   processSchemaData(schema, data, {
     dataPath,
     shouldProcess,
+    shouldSkipSourcesWithResource: true,
     before,
     options: { component, rootData }
   })