search

nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Able to make a conflicting sorting comparator #4183

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

image.png

Even though the error message here states that fields must be in all upper or all lowercase:

image.png

Note that the userguide doesn't contain much info on this makesort too:

image.png

[original: nus-cs2103-AY1920S1/pe-interim#4205]

seanlowjk commented 4 years ago

Team's Response

Team's Response

Thank you for testing our application.

For the second photo, it seems that you are not sure how makesort works.

However, it was stated in the user guide that tag is an invalid field, with reference to the photo from our user guide below. As a result, the subsequent error message is thrown.

image

Not only that, my team feels that this issue should not be of high severity. It is not a flaw that affects most users and causes major problems for users. i.e., makes the product almost unusable for most users.

Hence, we would have to give it low severity, where it is a flaw that is unlikely to affect normal operations of the product. Appears only in very rare situations and causes a minor inconvenience only. Users will understand from the User Guide that you are not allowed to have tag as a FIELD.

As a result of the reasons given, we would have to reject this issue. Thanks for your bug report!

Duplicate status (if any):

Based on your first photo, you have mentioned it already in #4185 As a result, you can click on the link to see the comments given by my team.

Duplicate status (if any):

--