search

a1021492980 / pe

0 stars 0 forks source link

Can get the description of items not in the list #7

Open a1021492980 opened 2 years ago

a1021492980 commented 2 years ago

image.png

Clearly, from the list there are only item with index 1 and 4, but when I try to get the description of index 2 and index 3, it actually provides some descriptions. Maybe there are some issues related to the list feature, causing it not showing the complete data.

nus-pe-bot commented 2 years ago

Team's Response

This is in relation to arrow keys on MacOS. Severity set to very low as it is extreme behaviour.

The 'Original' Bug

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

The search result is not readable, all massed together

image.png

I am using MacOS.

First, I added a item named '^[[A^[[A'

Then search for it using 'search n/^[[A^[[A'

The mass occurs

[original: nus-cs2113-AY2122S2/pe-interim#596] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

Our team has tried to replicate the issue. This issue occurs when you use the Up arrow key on MacOS which produces ^[[A on the terminal. This messes up the system in the following commands. We believe this is considered extreme behaviour which would be hard to work against.

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: According to the definition of duplicates provided in PE instruction, this is not a duplicate. Since they can be fixed independently as they are 2 different command. They are also not exactly the same issue.

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As mentioned in the lecture, it is the developers' responsibility to ensure the software is working on different system. And this issue can be easily caught by pre-define the '^[[A' and throw error messages when user input contains it.

:question: Issue severity

Team chose [severity.Low] Originally [severity.Medium]

Reason for disagreement: This causes the indexes to be inconsistent, users are unable to see certain range of indexes, so it severity should be at least medium.