search

nus-cs2113-AY2425S1 / pe-dev-response

0 stars 0 forks source link

when there is -m in item name, the delete command cannot work #274

Open nus-pe-bot opened 2 weeks ago

nus-pe-bot commented 2 weeks ago

when there is -m in item name, the delete command cannot work

step tp reproduce:

edit json with name containing -m use deletecommand, it wont work

expect: successfully delete actual: didn't work

image.png

image.png

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

averageandyyy commented 2 weeks ago

Team's Response

This limitation has already been mentioned in the UG and is expected behaviour, specifically that if users change the names of fields to a valid flag, they will not be able to access or alter the data in the program. The UG has also mentioned the steps that have to be taken to rectify this issue. Users are advised to change names containing valid flags to "--m" if they want to add it into the record. We have also lowered the severity to low as this does not affect most users (e.g. only advanced users that decide to edit the catalog.json file and want to use a name with a valid flag will encounter this issue). Addressing this bug would require us to re-examine our parser design in its entirety.

Duplicate status (if any):

--