diff options
| author | Fabian Mastenbroek <mail.fabianm@gmail.com> | 2022-09-14 14:12:41 +0200 |
|---|---|---|
| committer | Fabian Mastenbroek <mail.fabianm@gmail.com> | 2022-09-20 16:07:05 +0200 |
| commit | 7da6543fb9c58736c2fa8c000b25290be4f78de4 (patch) | |
| tree | 27d30112f99efa695087bb29686afc262a3ae57e /opendc-web/opendc-web-ui | |
| parent | 419297128ce2ab2c8085c81fafdb843c1135661d (diff) | |
fix(web/ui): Fix duplication of topology
This change addresses an issue where a new topology did not correctly
clone an existing topology.
Diffstat (limited to 'opendc-web/opendc-web-ui')
| -rw-r--r-- | opendc-web/opendc-web-ui/src/components/projects/NewTopologyModal.js | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/opendc-web/opendc-web-ui/src/components/projects/NewTopologyModal.js b/opendc-web/opendc-web-ui/src/components/projects/NewTopologyModal.js index be4256e3..780ec034 100644 --- a/opendc-web/opendc-web-ui/src/components/projects/NewTopologyModal.js +++ b/opendc-web/opendc-web-ui/src/components/projects/NewTopologyModal.js @@ -57,9 +57,10 @@ const NewTopologyModal = ({ projectId, isOpen, onSubmit: onSubmitUpstream, onCan setErrors({ name: true }) return false } else { - const candidate = topologies.find((topology) => topology.id === originTopology) || { projectId, rooms: [] } + const candidate = topologies.find((topology) => topology.id === originTopology) || { rooms: [] } const topology = produce(candidate, (draft) => { - delete draft.id + delete draft.project + draft.projectId = projectId draft.name = name }) onSubmitUpstream(topology) @@ -87,7 +88,12 @@ const NewTopologyModal = ({ projectId, isOpen, onSubmit: onSubmitUpstream, onCan <TextInput id="name" name="name" type="text" isRequired ref={nameInput} /> </FormGroup> <FormGroup label="Topology to duplicate" fieldId="origin" isRequired> - <FormSelect id="origin" name="origin" value={originTopology} onChange={setOriginTopology}> + <FormSelect + id="origin" + name="origin" + value={originTopology} + onChange={(v) => setOriginTopology(+v)} + > <FormSelectOption value={-1} key={-1} label="None - start from scratch" /> {topologies.map((topology) => ( <FormSelectOption value={topology.id} key={topology.id} label={topology.name} /> |
