From e11bd7aa0f88072d679b902a40fd3c38f2b558fe Mon Sep 17 00:00:00 2001 From: Paul Rastoin <45004772+prastoin@users.noreply.github.com> Date: Fri, 20 Jun 2025 17:58:31 +0200 Subject: [PATCH] `getLabelIdentifierFieldValue` should always return string (#12772) ## Introduction For a custom object if the selected identifier field metadata is an number type than it wouldn't get be converted to a string #closes https://github.com/twentyhq/twenty/issues/12717 ## Concerns Kinda the same than for https://github.com/twentyhq/twenty/pull/12728 Here ObjectRecord unknown fields are typed as any, we might wanna do a poc in order to migrate to `unknown` usage ```ts import { BaseObjectRecord } from '@/object-record/types/BaseObjectRecord'; export type ObjectRecord = Record & BaseObjectRecord; ``` --- .../object-metadata/utils/getLabelIdentifierFieldValue.ts | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/packages/twenty-front/src/modules/object-metadata/utils/getLabelIdentifierFieldValue.ts b/packages/twenty-front/src/modules/object-metadata/utils/getLabelIdentifierFieldValue.ts index 2f9254e95..1bac55144 100644 --- a/packages/twenty-front/src/modules/object-metadata/utils/getLabelIdentifierFieldValue.ts +++ b/packages/twenty-front/src/modules/object-metadata/utils/getLabelIdentifierFieldValue.ts @@ -13,12 +13,13 @@ export const getLabelIdentifierFieldValue = ( return record.id; } + const recordIdentifierValue = record[labelIdentifierFieldMetadataItem.name]; if ( objectNameSingular === CoreObjectNameSingular.WorkspaceMember || labelIdentifierFieldMetadataItem.type === FieldMetadataType.FULL_NAME ) { - return `${record[labelIdentifierFieldMetadataItem.name]?.firstName ?? ''} ${record[labelIdentifierFieldMetadataItem.name]?.lastName ?? ''}`; + return `${recordIdentifierValue?.firstName ?? ''} ${recordIdentifierValue?.lastName ?? ''}`; } - return record[labelIdentifierFieldMetadataItem.name] ?? ''; + return isDefined(recordIdentifierValue) ? `${recordIdentifierValue}` : ''; };