search

nus-cs2103-AY2324S2 / pe-dev-response

0 stars 0 forks source link

The assignment name is case insensitive, so new assignment is created when trying to mark the assignment with different case #5561

Open nus-pe-bot opened 1 month ago

nus-pe-bot commented 1 month ago

image.png

  1. Make sure that the person has Assignment1.
  2. mark 1 /t assignment1 /ts cg The Assignment1 should be marked because we don't usually distinguish them as separate assignment in real world. However, new one called "assingment1 is re-created. This might confuse user.

[original: nus-cs2103-AY2324S2/pe-interim#5538] [original labels: severity.Medium type.FeatureFlaw]

wang-xinrong commented 4 weeks ago

Team's Response

Quoting the text book

"Overzealous input validation: It is better to warn rather than to block when inputs are not compliant with the expected format, unless accepting such inputs can hinder the operations of the software. Allowing such flexibility can in turn allow the user to use the software in ways you didn't even anticipate while overzealous rejection of inputs can annoy the user."

While the suggested change does bring a higher level of convenience, it is conflicting with the current set of constraints we set for the tags. Currently, we are allowing the users to create tags with the same name but different capitalisation. Following this, making the mark command case-insensitive will lead to ambiguous situations where the app does not know specifically which tag to mark.

Duplicate status (if any):

--