github/directus
1
0
Fork 0
mirror of https://github.com/directus/directus.git synced 2026-01-23 13:37:58 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix: use nullish coalescing operator for setting nullable and defaultValue properties in the getSchema function instead of the ||(or) logical operator which always causes assigning true to the nullable property and ignores 0 and false default values (#5063)

Browse Source
This commit is contained in:
Pavlo Savenko
2021-04-14 23:18:39 +02:00
committed by GitHub
parent 8c214841bd
commit b160ae733b
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
api/src/utils/get-schema.ts
View File

@@ -89,8 +89,8 @@ export async function getSchema(options?: {
sortField: collectionMeta?.sort_field || null,
fields: mapValues(schemaOverview[collection].columns, (column) => ({
field: column.column_name,
defaultValue: getDefaultValue(column) || null,
nullable: column.is_nullable || true,
defaultValue: getDefaultValue(column) ?? null,
nullable: column.is_nullable ?? true,
type: getLocalType(column) || 'alias',
precision: column.numeric_precision || null,
scale: column.numeric_scale || null,
@@ -121,8 +121,8 @@ export async function getSchema(options?: {
result.collections[field.collection].fields[field.field] = {
field: field.field,
defaultValue: existing?.defaultValue || null,
nullable: existing?.nullable || true,
defaultValue: existing?.defaultValue ?? null,
nullable: existing?.nullable ?? true,
type: existing
? getLocalType(schemaOverview[field.collection].columns[field.field], {
special: field.special ? toArray(field.special) : [],