AlexanderTanJunAn / pe

0 stars 0 forks source link

Program says ID already exists #3

Open AlexanderTanJunAn opened 3 years ago

AlexanderTanJunAn commented 3 years ago

Screenshot 2021-04-16 at 4.51.03 PM.png

nus-pe-bot commented 3 years ago

Team's Response

Cannot be reproduced. After checking in 10 people as shown in the screenshot, the command checkin i/132D n/Test p/12345678 does not show any error message.

image.png

However, we suspect that it is caused by the clear command mentioned in #676.

The 'Original' Bug

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

Cannot checkin after clearing

Screenshot 2021-04-16 at 4.58.49 PM.png

Cannot checkin after Clearing, based on UG, it seems to mean clear everything, but program still keeps track of the past id's, which might be confusing.


[original: nus-cs2113-AY2021S2/pe-interim#676] [original labels: severity.Medium type.FunctionalityBug]

Their Response to the 'Original' Bug

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

clear command only clears tracking list and not the log file as stated in the FAQ section of the UG:

image.png

Therefore, checking in checkin i/001D n/Tom p/12345678 will lead to an error as the person with the same ID with a different name exists in the log file. However, we recognise that we were not clear about the clear command in the UG. Therefore we accept it as a Low severity of documentation bug.

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.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

Team chose [severity.Low] Originally [severity.High]

Reason for disagreement: [replace this with your explanation]