search

AY2324S2-CS2103-F09-2 / tp

https://ay2324s2-cs2103-f09-2.github.io/tp/
Other
1 stars 5 forks source link

[PE-D][Tester E] Hobbies field should be implemented with an upper bound. #187

Open nus-se-script opened 7 months ago

nus-se-script commented 7 months ago

Description

Screenshot 2024-04-05 at 16.37.01.png

Edited Patient: 21; PreferredName: 123; PatientHospitalId: 34567898765432456789876543; FoodPreferences: 12; FamilyConditions: FAM; Hobbies: nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6nRl9ZxCg4UG8V0RINQNdpBulKGlPG3hSMmDN5Bdv2K7ySVg9z6; Tags: [tags]

I was able to add this 3050 char-long String as hobbies, which should not be the case for hobbies.

Fix

Create an upper count for the number of chars one can input as the hobbies field.

Labels: severity.Low type.FeatureFlaw original: c-wenlong/ped#4

SherisseTJW commented 7 months ago

image

Invalid report;

Also no actual problems occurred as a result, so is more of a feature suggestion