nus-cs2103-AY2324S1 / pe-dev-response

0 stars 0 forks source link

Duplicate entries due to failure to trim extra internal spaces #1382

Open nus-se-script opened 11 months ago

nus-se-script commented 11 months ago

image.png

To replicate:

Enter addstaff n/The Rock p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 t/friends t/owesMoney

Then enter addstaff n/The Rock p/98765432 e/johnd@example.com a/311, Clementi Ave 2, #02-25 t/friends t/owesMoney

Expected behavior: Duplicate entry detected

Actual Behavior: 2 separate entries are created

The Rock vs The Rock (with an extra space in between) are intuitively the same – the extra space is due to a typo which is an extremely common occurrence for an application driven by CLI commands. Since this is not a rare bug, I assigned it severity.Medium


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

cxo05 commented 11 months ago

Team's Response

image.png

Duplicate status (if any):

Duplicate of #1139