Darkarche3 / pe

0 stars 0 forks source link

Can use class parameters multiple times #5

Open Darkarche3 opened 2 months ago

Darkarche3 commented 2 months ago

image.png

When the class parameter was used multiple times, the edit command still worked. It took the input from the last class parameter to edit the details of the person of index 00001. Good job on preventing this for the other parameters. You need to include it for this one as well.

nus-pe-script commented 2 months ago

Team's Response

Reproduction

Successfully reproduced. Accepted.

Reasons for changing severity status

We consider it to be unusual for a teacher to specify more than 1 class parameter for a student (since each student can only belong in 1 form class) and thus is unlikely to affect normal operations of the application. Furthermore, the application does not break down when there is an additional class field provided. Will be corrected in the future.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]