search

nus-cs2103-AY2122S2 / pe-dev-response

0 stars 0 forks source link

Duplicate classes can be added #589

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Duplicate classes can be added if the input has different capitalization.

How to reproduce: 1) Input add_class tn/T04 v/LT13 d/Monday tm/13:00 wk/13 and ENTER 2) Check that this class has been successfully added 3) Input add_class tn/t04 v/LT13 d/Monday tm/13:00 wk/13 and ENTER

Expected Result: The second input should not be successfully added as a class since there is already a duplicate that exists.

Actual Result: Both classes added exists and displays in the app.

Logically, this would not be allowed as they represent the same tutorial slot (on Luminus, you are able to see that all tutorials are capitalized)

image.png

[original: nus-cs2103-AY2122S2/pe-interim#813] [original labels: type.FunctionalityBug severity.Medium]

seanlaiys commented 2 years ago

Team's Response

As per our latest milestone, it is stated in the User Guide that all parameters are case-sensitive. Hence, classes with similar names but different cases will be treated differently.

Duplicate status (if any):

--