nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Add class bug - unable to add certain types of class names #2162

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

image.png

Steps to reproduce:

Key in the following command:

addc c/ GEC1036/GET1022

Some modules may have names like the "GEC1036/GET1022" due to changes in curriculum (General education module requirements). Reasonably, I would not want to confuse myself by saving 2 classes with different names that represent the same thing, so I might want to save the module like so. However, in general I do not have a good way of representing a class in this case (since other special characters and spaces are not allowed either).


[original: nus-cs2103-AY2223S1/pe-interim#1818] [original labels: type.FeatureFlaw severity.Low]

Berted commented 2 years ago

Team's Response

Good spot, this is a valid point.

However, we find that the effort to fix this is not worth the effort required to revamp the whole parsing system that was inherited from AB3, perhaps this can be resolved in a future iteration with a better parsing system.

Duplicate status (if any):

--