Skip to content
Snippets Groups Projects
Commit b00b4d9c authored by Sigurd's avatar Sigurd
Browse files

Extracted duplicated code

parent 18123180
No related branches found
No related tags found
No related merge requests found
Pipeline #171912 passed
......@@ -26,7 +26,6 @@ class MuscleGroup {
}
getMuscleGroupType = () => {
console.log(this.type, "SWIOEFIWEUFH")
return this.type;
}
}
......@@ -49,13 +48,17 @@ function handleCancelButtonDuringEdit() {
if (oldFormData.has("muscleGroup")) form.muscleGroup.value = oldFormData.get("muscleGroup");
if (oldFormData.has("unit")) form.unit.value = oldFormData.get("unit");
deleteFormData();
}
function deleteFormData() {
oldFormData.delete("name");
oldFormData.delete("description");
oldFormData.delete("duration");
oldFormData.delete("calories");
oldFormData.delete("muscleGroup");
oldFormData.delete("unit");
}
function handleCancelButtonDuringCreate() {
......@@ -160,8 +163,6 @@ async function updateExercise(id) {
return;
}
muscleGroupSelector.setAttribute("disabled", "")
// duplicate code from handleCancelButtonDuringEdit
// you should refactor this
setReadOnly(true, "#form-exercise");
okButton.className += " hide";
deleteButton.className += " hide";
......@@ -170,12 +171,7 @@ async function updateExercise(id) {
cancelButton.removeEventListener("click", handleCancelButtonDuringEdit);
oldFormData.delete("name");
oldFormData.delete("description");
oldFormData.delete("duration");
oldFormData.delete("calories");
oldFormData.delete("muscleGroup");
oldFormData.delete("unit");
deleteFormData();
}
window.addEventListener("DOMContentLoaded", async () => {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment