Skip to content

Commit 6d82475

Browse files
committed
[StudyGuidance] Correctly set initial year when adding starting year to group
1 parent bdcdd43 commit 6d82475

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

services/frontend/src/components/StudyGuidanceGroups/StudyGuidanceGroupOverview.tsx

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,9 @@ const EditTagModal = ({
6969
const [formErrors, setFormErrors] = useState({})
7070

7171
useEffect(() => {
72-
setFormValues({ [tagName]: initialState ?? '' })
72+
// Magic number: previous year (if initialState not set)
73+
const initialYear = formValues[tagName] === '' ? new Date().getFullYear() - 1 : Number(formValues[tagName])
74+
setFormValues({ [tagName]: initialState ?? (tagName === 'year' ? initialYear : '') })
7375
setFormErrors({})
7476
}, [group, tagName, open])
7577

@@ -130,7 +132,7 @@ const EditTagModal = ({
130132
label="Select degree programme"
131133
name={tagName}
132134
onChange={event => handleChange(tagName, event.target.value)}
133-
value={formValues[tagName] ?? undefined}
135+
value={formValues[tagName]}
134136
>
135137
{selectFieldItems?.map(({ key, value, description, text }) => (
136138
<MenuItem key={key} sx={{ justifyContent: 'space-between' }} value={value}>
@@ -142,11 +144,7 @@ const EditTagModal = ({
142144
</FormControl>
143145
) : (
144146
<Box sx={{ display: 'flex', justifyContent: 'center' }}>
145-
<EnrollmentDateSelector
146-
setYear={value => handleChange(tagName, value)}
147-
// Magic number: previous year (if none set)
148-
year={formValues[tagName] === '' ? new Date().getFullYear() - 1 : Number(formValues[tagName])}
149-
/>
147+
<EnrollmentDateSelector setYear={value => handleChange(tagName, value)} year={formValues[tagName]} />
150148
</Box>
151149
)}
152150
</Box>

0 commit comments

Comments
 (0)