Skip to content

Commit f4a57b5

Browse files
purfectliteraturecysjonathan
authored andcommitted
refactor(GroupCreationForm): remove react-emitter-factory
1 parent 082190e commit f4a57b5

File tree

2 files changed

+9
-12
lines changed

2 files changed

+9
-12
lines changed

client/app/bundles/course/group/forms/GroupCreationForm.jsx

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import { useEffect, useMemo } from 'react';
2-
import useEmitterFactory from 'react-emitter-factory';
32
import { Controller, useForm } from 'react-hook-form';
43
import { defineMessages, FormattedMessage } from 'react-intl';
54
import { connect } from 'react-redux';
@@ -113,7 +112,9 @@ const getConflictingNames = (name, numToCreate, existingGroups) => {
113112
};
114113

115114
const GroupCreationForm = (props) => {
116-
const { dispatch, existingGroups, initialValues, onSubmit } = props;
115+
const { dispatch, existingGroups, initialValues, onSubmit, onDirtyChange } =
116+
props;
117+
117118
const {
118119
control,
119120
handleSubmit,
@@ -126,13 +127,10 @@ const GroupCreationForm = (props) => {
126127
mode: 'onChange',
127128
resolver: yupResolver(validationSchema),
128129
});
129-
useEmitterFactory(
130-
props,
131-
{
132-
isDirty,
133-
},
134-
[isDirty],
135-
);
130+
131+
useEffect(() => {
132+
onDirtyChange?.(isDirty);
133+
}, [isDirty]);
136134

137135
const name = watch('name');
138136
const numToCreate = Number.parseInt(watch('num_to_create'), 10);
@@ -294,6 +292,7 @@ GroupCreationForm.propTypes = {
294292
dispatch: PropTypes.func.isRequired,
295293
initialValues: PropTypes.object.isRequired,
296294
onSubmit: PropTypes.func.isRequired,
295+
onDirtyChange: PropTypes.func,
297296
};
298297

299298
export default connect(({ groups }) => ({

client/app/bundles/course/group/pages/GroupShow/GroupManager/GroupManager.jsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -321,16 +321,14 @@ const GroupManager = ({
321321
skipConfirmation={!isDirty}
322322
>
323323
<GroupCreationForm
324-
emitsVia={(nameDescriptionForm) =>
325-
setIsDirty(nameDescriptionForm.isDirty)
326-
}
327324
existingGroups={groups}
328325
initialValues={{
329326
name: '',
330327
description: '',
331328
num_to_create: 0,
332329
is_single: true,
333330
}}
331+
onDirtyChange={setIsDirty}
334332
onSubmit={onCreateFormSubmit}
335333
/>
336334
</GroupFormDialog>

0 commit comments

Comments
 (0)