sarjinius / pe

0 stars 0 forks source link

Looking for 'Tan Keng Yan' returns many profs despite it being unique #6

Open sarjinius opened 2 months ago

sarjinius commented 2 months ago


video:https://raw.githubusercontent.com/sarjinius/pe/main/files/653d7ae0-024e-4ce5-bc6f-38b30c7cf98b.mov

I am looking for Colin Tan Keng Yan, and I input 'prof -n tan keng tan'. As far as I know, he is the only prof containing that name. However, multiple names show up containing one of 'Tan', 'Keng', or 'Yan'.

nus-se-bot commented 2 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Add professors: add in many unrelated professors even if I specify the exact full name

When I try to add a professor with his full name, a lot of other professors will also be added in. It i Example: When I enter prof -n ben leong wing lup, the expected result should be that only the certain prof will be added in. However, totally 11 professors are added in. This is quite inconvenient, because I have to manually delete all the other professors added accidentally. Screenshot 2024-04-19 at 4.29.08 PM.png


[original: nus-cs2103-AY2324S2/pe-interim#5548] [original labels: type.FunctionalityBug severity.Medium]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Accepted - this is a major known issue that causes inconvinience to the user.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [x] I disagree **Reason for disagreement:** The name is non-null and unique as required in the UG, and it states it adds a specific professor (which I assume would be the only professor matching the name). So it does not work as specified, implying it is a functionality bug.
## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]