search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Possible to add duplicate classes if only case differs #5685

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

Steps to reproduce bug:

  1. type addClass 4e2

Expected output: error saying that class with this name already exists in the address book

Actual output:

image.png

Reason for bug report : Although the UG mentions that this command is case sensitive, checking for duplicates should not be trivial. Just by changing the case of 1 letter, it allows the user to add the class 4e2 when 4E2 actually exists in the address book.

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

Wilfredwongkc commented 2 years ago

Team's Response

This bug is a duplicate because it is still referring to the case sensitivity validation in class names (which is actually a feature decided by our team). We have made this case-sensitive for a reason and that is because when teachers want to export the information for admin work, the office might be particular about the case sensitivity.

Duplicate status (if any):

Duplicate of #5728