tau-bar / pe

0 stars 0 forks source link

Medicine name can only be 20 characters long. #9

Open tau-bar opened 2 years ago

tau-bar commented 2 years ago

Expected:

• Expected Behaviour: For me to be able to store medicine names with more than 20 characters.

Actual:

• Actual Behaviour: I cannot store a long medcine name exceeding 20 characters.

• Why that severity? Low, as it is rare that medicines have such a long name. However, in the case that they do, this will cause a minor inconvenience.

Steps to reproduce:

  1. Add a long medicine name like :apmt pa 1 n/WWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWWW v/400 ml d/2 times a week

Screenshots:

Screenshot 2021-11-12 at 4.49.14 PM.png

nus-pe-bot commented 2 years ago

Team's Response

Not a bug, the limit was imposed to prevent abuse.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

Response

I disagree.

Rationale

It is uncommon for medicine to have long names exceeding 20 characters. However, there are medicines whose names are actually above 20 characters (eg. ALCLOMETASONE DIPROPIONATE). The screenshot below shows me trying to use the feature with a long medicine name, but unable to do so due to the limit imposed by the developer team. I would not classify this as abuse, this is an actual medicine name, this is a topical medication used to treat eczema.

As stated in the module website, overzealous input blocking is considered a Feature Flaw (shown below). The inability to add more than 20 characters might potentially annoy the user:

Screenshot 2021-11-17 at 12.57.44 PM.png

Evidence

Proof that this is an actual medicine:
Screenshot 2021-11-17 at 12.51.47 PM.png

Unable to add medicines with long names:
Screenshot 2021-11-17 at 12.49.44 PM.png