search

nshian / nus-cs2113-pe

0 stars 0 forks source link

Application does not handle trailing invalid characters in module code #8

Open nshian opened 1 year ago

nshian commented 1 year ago

The user input of "m/CS1231??" technically fulfils the criteria of MODULE_CODE containing both number and alphabet (since there is a presence of "1231" and "CS") and is hence a valid input. However, the application does not handle the case of an invalid trailing character like "?" in the module code and does not trim this from the module code.

image.png

soc-pe-bot commented 1 year ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Module code can in wrong format

Screenshot 2022-11-11 at 17.17.18.png

module code start with / is added into system

[original: nus-cs2113-AY2223S1/pe-interim#881] [original labels: severity.High type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

special characters should not be accepted.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]