Fix bug where upon editing topset it would change exercise to Squat (1st in list)

This commit is contained in:
Peter Stockings
2022-12-04 17:22:24 +11:00
parent 4ff578f8b6
commit 947015f2a4
2 changed files with 4 additions and 4 deletions

2
app.py
View File

@@ -173,7 +173,7 @@ def get_topset(person_id, workout_id, topset_id):
def get_topset_edit_form(person_id, workout_id, topset_id): def get_topset_edit_form(person_id, workout_id, topset_id):
exercises = db.get_exercises() exercises = db.get_exercises()
topset = db.get_topset(person_id, workout_id, topset_id) topset = db.get_topset(person_id, workout_id, topset_id)
return render_template('partials/topset.html', person_id=person_id, workout_id=workout_id, topset_id=topset_id, exercises=exercises, repetitions=topset['Repetitions'], weight=topset['Weight'], is_edit=True) return render_template('partials/topset.html', person_id=person_id, workout_id=workout_id, topset_id=topset_id, exercises=exercises, exercise_name=topset['ExerciseName'], repetitions=topset['Repetitions'], weight=topset['Weight'], is_edit=True)
@ app.route("/person/<int:person_id>/workout/<int:workout_id>/topset", methods=['POST']) @ app.route("/person/<int:person_id>/workout/<int:workout_id>/topset", methods=['POST'])

View File

@@ -7,9 +7,9 @@
class="block appearance-none w-full bg-gray-200 border border-gray-200 text-gray-700 py-3 px-4 pr-8 rounded leading-tight focus:outline-none focus:bg-white focus:border-gray-500" class="block appearance-none w-full bg-gray-200 border border-gray-200 text-gray-700 py-3 px-4 pr-8 rounded leading-tight focus:outline-none focus:bg-white focus:border-gray-500"
name="exercise_id"> name="exercise_id">
{% for exercise in exercises|default([], true) %} {% for exercise in exercises|default([], true) %}
<option value="{{ exercise['ExerciseId'] }}" {% if exercise['ExerciseId']==exercise_id %} selected {% endif <option value="{{ exercise['ExerciseId'] }}" {% if exercise['Name']==exercise_name %} selected {% endif %}>
%}>{{ {{exercise['Name']}}
exercise['Name']}}</option> </option>
{% endfor %} {% endfor %}
</select> </select>
{% endif %} {% endif %}