Skip to content

Commit 5286a2c

Browse files
#1499 | Use utils to replace preferredname check in ProposalTableOfficer
1 parent 0c63a0b commit 5286a2c

1 file changed

Lines changed: 5 additions & 16 deletions

File tree

apps/frontend/src/components/proposal/ProposalTableOfficer.tsx

Lines changed: 5 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ import {
7575
} from 'utils/helperFunctions';
7676
import { tableIcons } from 'utils/materialIcons';
7777
import useDataApiWithFeedback from 'utils/useDataApiWithFeedback';
78-
import { getFullUserName } from 'utils/user';
78+
import { getFullUserName, getPreferredName } from 'utils/user';
7979
import withConfirm, { WithConfirmType } from 'utils/withConfirm';
8080

8181
import CallSelectModalOnProposalsClone from './CallSelectModalOnProposalClone';
@@ -127,21 +127,10 @@ let columns: Column<ProposalViewData>[] = [
127127
field: 'principalInvestigator',
128128
sorting: false,
129129
emptyValue: '-',
130-
render: (proposalView) => {
131-
if (
132-
proposalView.principalInvestigator?.lastname &&
133-
proposalView.principalInvestigator?.preferredname
134-
) {
135-
return `${proposalView.principalInvestigator.lastname}, ${proposalView.principalInvestigator.preferredname}`;
136-
} else if (
137-
proposalView.principalInvestigator?.lastname &&
138-
proposalView.principalInvestigator?.firstname
139-
) {
140-
return `${proposalView.principalInvestigator.lastname}, ${proposalView.principalInvestigator.firstname}`;
141-
}
142-
143-
return '';
144-
},
130+
render: (proposalView) =>
131+
proposalView.principalInvestigator?.lastname
132+
? `${proposalView.principalInvestigator.lastname}, ${getPreferredName(proposalView.principalInvestigator)}`
133+
: '',
145134
},
146135
{
147136
title: 'PI Email',

0 commit comments

Comments
 (0)