From 54a3fb3b12a1f789d529c4686bedccb5c541c34e Mon Sep 17 00:00:00 2001 From: rijkvanzanten Date: Wed, 13 Jan 2021 17:10:38 -0500 Subject: [PATCH 1/6] Default to primary key for related values display Fixes #3470 --- app/src/displays/related-values/index.ts | 6 ++++-- .../displays/related-values/related-values.vue | 18 +++++++++++++----- 2 files changed, 17 insertions(+), 7 deletions(-) diff --git a/app/src/displays/related-values/index.ts b/app/src/displays/related-values/index.ts index 99c24dfa7e..f421f220f6 100644 --- a/app/src/displays/related-values/index.ts +++ b/app/src/displays/related-values/index.ts @@ -20,13 +20,15 @@ export default defineDisplay(({ i18n }) => ({ options: options, types: ['alias', 'string', 'uuid', 'integer', 'bigInteger', 'json'], groups: ['m2m', 'm2o', 'o2m'], - fields: (options: Options, { field, collection }) => { + fields: (options: Options | null, { field, collection }) => { const relatedCollection = getRelatedCollection(collection, field); const { primaryKeyField } = useCollection(ref(relatedCollection as string)); if (!relatedCollection) return []; - const fields = adjustFieldsForDisplays(getFieldsFromTemplate(options.template), relatedCollection); + const fields = options?.template + ? adjustFieldsForDisplays(getFieldsFromTemplate(options.template), relatedCollection) + : []; if (fields.includes(primaryKeyField.value.field) === false) { fields.push(primaryKeyField.value.field); diff --git a/app/src/displays/related-values/related-values.vue b/app/src/displays/related-values/related-values.vue index 2b262bcfaa..b8994ad526 100644 --- a/app/src/displays/related-values/related-values.vue +++ b/app/src/displays/related-values/related-values.vue @@ -1,6 +1,10 @@