soc-se-bot
commented
2 years ago Team's Response
According to CS2103T docs:
We do not believe this issue is in scope since rectifying it is less important (based on the value/effort considerations) than the work that has been done already (users can simply replace italian special characters with alternative letters from the alphabet).
In addition, users can use the application as per normal with no errors/issues/crashes/etc.
Items for the Tester to Verify
:question: Issue response
Team chose [response.NotInScope]
- [x] I disagree
Reason for disagreement: Thank you to the developer team for taking the time to review the bug report.
However, I would have to disagree with the team's response. First of all, as per the screenshot from the CS2103T website:
First of all, the team did not state explicitly that accented letters are not allowed. It only says that ingredient names should only contain letters. That is for the first point. Next, for the second point, the error shown is too generic and not suitable at all, where it just says "Names should contain valid alpha-numeric characters". However, the user guide did not even specify what is considered valid, as per the error message. The user guide and the error message did not explicitly state that accented letters such as è are not allowed, which are considered letters as well.
Clearly, the team did not satisfy any of the two bullet points to convert this bug as "NotinScope".
Moreover, from the module website, for the PE-dry run it has been stated before:
Disallowing names that include accented letters such as è would be unreasonable because as an app that revolves around recipes, how can cuisines such as Italian cuisines not be accepted as it is?
Moreover, allowing the conversion of the bug to not in scope is not acceptable as it allows the team to convert all naming bugs to be not in scope, which is an exploit and an attempt to game the system. This sends the wrong message to all the teams in the module, giving them an excuse to say that bugs such as accepting special letters or characters are not considered bugs. This is even more important given that other teams have received similar bug reports as such as well. Accepting this bug as not in scope is not fair to the other teams as well, given that there should be no special treatment to any team.
Lastly, I have assigned the severity as low because I do understand that this bug could potentially affect users who are in love with exotic recipes such as Italian cuisines.
I do think that this is a fair bug report and hope that the team accepts the bug with grace.
Steps to reproduce: add -n lasagnè -d 15 -ss 1 -i mushroom 50 grams | oil 20 grams -s Cut mushroom | stir fry mushroom
expected: Added recipe
actual: error message
Some Italian recipes may contain special letters such as è. But this is not covered in the application.