Ryan/Reactive-Resume
1
0
Fork 0
mirror of https://github.com/AmruthPillai/Reactive-Resume.git synced 2025-11-18 18:51:32 +10:00
Code Issues Packages Projects Releases Wiki Activity

Fix issue when a section is duplicated/cloned, ID is null

Browse Source
This commit is contained in:
Amruth Pillai
2023-07-27 18:35:19 +02:00
parent 9c76999945
commit f9a11092a6
1 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
client/templates/sectionMap.tsx
View File

@ -44,11 +44,11 @@ const sectionMap = (Section: React.FC<SectionProps>): Record<string, JSX.Element
references: <Section key="references" path="sections.references" titlePath="name" subtitlePath="relationship" />,
});
export const getSectionById = (id: string, Section: React.FC<SectionProps>): JSX.Element => {
// Check if section id is a custom section (an uuid)
if (validate(id)) {
return <Section key={id} path={`sections.${id}`} />;
}
export const getSectionById = (id: string, Section: React.FC<SectionProps>): JSX.Element | null => {
if (!id) return null;
// Check if section id is a custom section (is a valid uuid)
if (validate(id)) return <Section key={id} path={`sections.${id}`} />;
// Check if section id is a predefined seciton in config
const predefinedSection = get(sectionMap(Section), id);
@ -57,9 +57,11 @@ export const getSectionById = (id: string, Section: React.FC<SectionProps>): JSX
return predefinedSection;
}
// Other ways section should be a cloned section
const section = find(sectionMap(Section), (element, key) => id.includes(key));
return React.cloneElement(section!, { path: `sections.${id}` });
// Otherwise, section must be a cloned section
const section = find(sectionMap(Section), (_element, key) => id.includes(key));
if (section) return React.cloneElement(section, { path: `sections.${id}` });
return null;
};
export default sectionMap;