Update RaRecord type & fix some other types

djhi · djhi · commit 8b916b36199d · 2023-05-11T17:25:13.000+02:00
diff --git a/examples/crm/src/deals/DealShow.tsx b/examples/crm/src/deals/DealShow.tsx
@@ -15,6 +15,7 @@ import { CompanyAvatar } from '../companies/CompanyAvatar';
 import { NotesIterator } from '../notes';
 import { ContactList } from './ContactList';
 import { stageNames } from './stages';
+import { Deal } from '../types';
 
 export const DealShow = ({ open, id }: { open: boolean; id?: string }) => {
     const redirect = useRedirect();
@@ -48,7 +49,7 @@ export const DealShow = ({ open, id }: { open: boolean; id?: string }) => {
 };
 
 const DealShowContent = () => {
-    const record = useRecordContext();
+    const record = useRecordContext<Deal>();
     if (!record) return null;
     return (
         <div>
diff --git a/examples/crm/src/notes/NewNote.tsx b/examples/crm/src/notes/NewNote.tsx
@@ -13,6 +13,7 @@ import {
 import { Box, TextField as TextInput, Button } from '@mui/material';
 
 import { StatusSelector } from './StatusSelector';
+import { Contact, Deal } from '../types';
 
 export const NewNote = ({
     showStatus,
@@ -22,7 +23,7 @@ export const NewNote = ({
     reference: 'contacts' | 'deals';
 }) => {
     const resource = useResourceContext();
-    const record = useRecordContext();
+    const record = useRecordContext<Deal | Contact>();
     const { refetch } = useListContext();
     const [text, setText] = useState('');
     const [status, setStatus] = useState(record && record.status);
diff --git a/examples/crm/src/notes/Note.tsx b/examples/crm/src/notes/Note.tsx
@@ -22,6 +22,7 @@ import EditIcon from '@mui/icons-material/Edit';
 import TrashIcon from '@mui/icons-material/Delete';
 
 import { Status } from '../misc/Status';
+import { Contact, Deal } from '../types';
 
 export const Note = ({
     showStatus,
@@ -37,7 +38,7 @@ export const Note = ({
     const [isEditing, setEditing] = useState(false);
     const [noteText, setNoteText] = useState(note.text);
     const resource = useResourceContext();
-    const record = useRecordContext();
+    const record = useRecordContext<Deal | Contact>();
     const notify = useNotify();
 
     const [update, { isLoading }] = useUpdate();
diff --git a/examples/demo/src/dashboard/PendingReviews.tsx b/examples/demo/src/dashboard/PendingReviews.tsx
@@ -71,16 +71,20 @@ const PendingReviews = () => {
                                 reference="customers"
                                 link={false}
                             >
-                                <FunctionField
-                                    render={(customer: Customer) => (
-                                        <Avatar
-                                            src={`${customer.avatar}?size=32x32`}
-                                            sx={{
-                                                bgcolor: 'background.paper',
-                                            }}
-                                            alt={`${customer.first_name} ${customer.last_name}`}
-                                        />
-                                    )}
+                                <FunctionField<Customer>
+                                    render={customer =>
+                                        customer ? (
+                                            <Avatar
+                                                src={`${customer.avatar}?size=32x32`}
+                                                sx={{
+                                                    bgcolor: 'background.paper',
+                                                }}
+                                                alt={`${customer.first_name} ${customer.last_name}`}
+                                            />
+                                        ) : (
+                                            ''
+                                        )
+                                    }
                                 />
                             </ReferenceField>
                         </ListItemAvatar>
diff --git a/examples/demo/src/reviews/ReviewItem.tsx b/examples/demo/src/reviews/ReviewItem.tsx
@@ -53,8 +53,8 @@ export const ReviewItem = () => {
                                 reference="customers"
                                 link={false}
                             >
-                                <FunctionField
-                                    render={(record?: Customer) =>
+                                <FunctionField<Customer>
+                                    render={record =>
                                         record
                                             ? `${record.first_name} ${record.last_name}`
                                             : ''
diff --git a/examples/demo/src/visitors/Aside.tsx b/examples/demo/src/visitors/Aside.tsx
@@ -200,7 +200,7 @@ const mixOrdersAndReviews = (
 };
 
 const Order = () => {
-    const record = useRecordContext();
+    const record = useRecordContext<OrderRecord>();
     const translate = useTranslate();
     if (!record) return null;
     return (
diff --git a/packages/ra-core/src/controller/record/useRecordContext.ts b/packages/ra-core/src/controller/record/useRecordContext.ts
@@ -1,5 +1,4 @@
 import { useContext } from 'react';
-import { RaRecord } from '../../types';
 import { RecordContext } from './RecordContext';
 
 /**
@@ -31,7 +30,7 @@ import { RecordContext } from './RecordContext';
  * @returns {RaRecord} A record object
  */
 export const useRecordContext = <
-    RecordType extends RaRecord | Omit<RaRecord, 'id'> = RaRecord
+    RecordType extends Record<string, unknown> = Record<string, unknown>
 >(
     props?: UseRecordContextParams<RecordType>
 ): RecordType | undefined => {
@@ -43,7 +42,7 @@ export const useRecordContext = <
 };
 
 export interface UseRecordContextParams<
-    RecordType extends RaRecord | Omit<RaRecord, 'id'> = RaRecord
+    RecordType extends Record<string, unknown> = Record<string, unknown>
 > {
     record?: RecordType;
     [key: string]: any;
diff --git a/packages/ra-core/src/dataProvider/useGetMany.ts b/packages/ra-core/src/dataProvider/useGetMany.ts
@@ -113,5 +113,5 @@ export const useGetMany = <RecordType extends RaRecord = any>(
 };
 
 export type UseGetManyHookValue<
-    RecordType extends RaRecord = any
+    RecordType extends RaRecord = RaRecord
 > = UseQueryResult<RecordType[], Error>;
diff --git a/packages/ra-core/src/dataProvider/useGetRecordId.ts b/packages/ra-core/src/dataProvider/useGetRecordId.ts
@@ -1,6 +1,6 @@
 import { useParams } from 'react-router-dom';
 import { useRecordContext } from '../controller';
-import { Identifier } from '../types';
+import { Identifier, RaRecord } from '../types';
 
 /**
  * Helper hook to get the current `recordId`.
@@ -15,7 +15,7 @@ import { Identifier } from '../types';
  * const recordId = useGetRecordId();
  */
 export function useGetRecordId(recordId?: Identifier): Identifier {
-    const contextRecord = useRecordContext();
+    const contextRecord = useRecordContext<RaRecord>();
     const { id: routeId } = useParams<'id'>();
     const actualRecordId = recordId ?? contextRecord?.id ?? routeId;
     if (actualRecordId == null)
diff --git a/packages/ra-core/src/routing/types.ts b/packages/ra-core/src/routing/types.ts
@@ -1,8 +1,11 @@
 import { RaRecord } from '../types';
 
-export type LinkToFunctionType = (
-    record: RaRecord,
+export type LinkToFunctionType<RecordType extends RaRecord = RaRecord> = (
+    record: RecordType,
     reference: string
 ) => string;
 
-export type LinkToType = string | false | LinkToFunctionType;
+export type LinkToType<RecordType extends RaRecord = RaRecord> =
+    | string
+    | false
+    | LinkToFunctionType<RecordType>;
diff --git a/packages/ra-core/src/types.ts b/packages/ra-core/src/types.ts
@@ -9,9 +9,9 @@ import { AuthActionType } from './auth/types';
 
 export type Identifier = string | number;
 
-export interface RaRecord {
-    id: Identifier;
-    [key: string]: any;
+export interface RaRecord<IdentifierType extends Identifier = Identifier>
+    extends Record<string, any> {
+    id: IdentifierType;
 }
 
 export interface SortPayload {
diff --git a/packages/ra-ui-materialui/src/field/FunctionField.tsx b/packages/ra-ui-materialui/src/field/FunctionField.tsx
@@ -18,7 +18,9 @@ import { PublicFieldProps, InjectedFieldProps, fieldPropTypes } from './types';
  * />
  */
 
-export const FunctionField = <RecordType extends unknown = any>(
+export const FunctionField = <
+    RecordType extends Record<string, unknown> = Record<string, unknown>
+>(
     props: FunctionFieldProps<RecordType>
 ) => {
     const { className, source = '', render, ...rest } = props;
diff --git a/packages/ra-ui-materialui/src/field/ReferenceField.tsx b/packages/ra-ui-materialui/src/field/ReferenceField.tsx
@@ -112,7 +112,7 @@ export interface ReferenceFieldProps<RecordType extends RaRecord = any>
     resource?: string;
     source: string;
     translateChoice?: Function | boolean;
-    link?: LinkToType;
+    link?: LinkToType<RecordType>;
     sx?: SxProps;
 }
 
diff --git a/packages/ra-ui-materialui/src/field/ReferenceOneField.tsx b/packages/ra-ui-materialui/src/field/ReferenceOneField.tsx
@@ -103,7 +103,7 @@ export interface ReferenceOneFieldProps<RecordType extends RaRecord = any>
     target: string;
     sort?: SortPayload;
     filter?: any;
-    link?: LinkToType;
+    link?: LinkToType<RecordType>;
     queryOptions?: UseQueryOptions<{
         data: RecordType[];
         total: number;
