isaaceng7 / pe

0 stars 0 forks source link

Clear feature does not clear logs #5

Open isaaceng7 opened 4 months ago

isaaceng7 commented 4 months ago

Clear feature does not clear logs.

image.png

nus-pe-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]

Clear does not clear logs

Description: In the User Guide, the section about the command clear states that it deletes all the logs entered throughout the usage time period. However, after clearing and re-entering all the ata about heigh, weight, etc., running list still shows previously entered logs.

Steps to Reproduce:

  1. Enter some logs
  2. Type clear and go through the process
  3. Type list

Actual Result: The old logged data still appears.

Edited to add: When opening data.txt, it appears that the old data was not being overwritten, but instead appended and might be why. See the second screenshot below.

Screenshot:

image.png

image.png


[original: nus-cs2113-AY2324S2/pe-interim#814] [original labels: type.FeatureFlaw severity.High]

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 type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** This is not a FeatureFlaw but a FunctionalityBug. The functionality does not work as specified, therefore FunctionalityBug ![image.png](https://raw.githubusercontent.com/isaaceng7/pe/main/files/f70fa6a8-37d1-4015-b51f-df75ef9d94ae.png)
## :question: Issue severity Team chose [`severity.High`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]