search

hanchingyong / pe

0 stars 0 forks source link

No case-sensitivity check and Tissue item does not exist in current suppliers inventory. #5

Open hanchingyong opened 4 years ago

hanchingyong commented 4 years ago

image.png

How to replicate: buy 2 g/tissue q/9

tissue are bought, which do not update Tissue. The original list of items also do not have suppliers that sell Tissue thus it is impossible to restock on Tissue.

nus-pe-bot commented 4 years ago

Team's Response

Case sensitive nature of Good Names is intentional and the expected behaviour of inventory manager. This point is mentioned in the user guide:

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: In this case, I believe that this should be changed to a Feature Flaw, of medium severity. According to the 2103 PE guidelines, severity.Medium : A flaw that causes occasional inconvenience to some users but they can continue to use the product.

In this case, this is a flaw to users, as the find function clearly allows for searching of goods in a case non-sensitive manner. Thus, it would be a simple and elegant solution to include the purchasing of goods using this case non-sensitive method. In no supermarket would Tissue be different from tissue, or even tIsSuE. This is a feature flaw as it does not streamline the user experience, and causes inconvenience for them should they forget to follow the correct capitalisation.