OKW32 / pe

0 stars 0 forks source link

Editing name of workout to an already existing name #3

Open OKW32 opened 4 months ago

OKW32 commented 4 months ago

Editing of workout does not consider the case when user changes the workout name to an already existing workout. Causing lost of data, making the editing function not work as expected.

image.png

nus-se-script commented 4 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Rename workout

If i have added a workout "pull up day" and another "push up day" and i decide to rename "push up day" to "pull up day" which is the same as my first workout name, it should warn me that this name exists and not change it and give me only one output. What if the use has forgotten that this name exists already?

image.png


[original: nus-cs2113-AY2324S2/pe-interim#1873] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** This should not be a low level bug as it causes a significant enough inconvenience when data is loss when user accidentally changes the name to a pre-existing workout's name. There should have been error handling to prevent such accidents from happening and hence, this should be of medium severity level. ![image.png](https://raw.githubusercontent.com/OKW32/pe/main/files/20786c58-0978-4667-a9e1-dc78e74c3b23.png)