Team's Response

No details provided by team.

The 'Original' Bug

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

Unable to edit student ID

The "edit student" command of the app does not provide a functionality to edit student ID. Thus if the user made a typo in student ID and later discovered it, the user must delete the student entirely, and add a new student with the correct ID. This forces the user to re-type all student information including name, email and tele handle, even if they are all correct in the original student.

Suggestion: I understand that ID is used to identify a student, but the problem is this implementation does not allow the editing of ID itself. Perhaps you can identify students by their index number listed in the mod. In this way, identification is not dependent on any editable data.

"edit student" command does not provide functionality to edit student ID, according to UG

---

_{[original: nus-cs2103-AY2122S1/pe-interim#5185] [original labels: type.FeatureFlaw severity.Medium]}

Their Response to the 'Original' Bug

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

Student ID is being used as an identifier hence it is crucial in keeping the identity of the student separate. Student ID is meant to be uneditable.

Items for the Tester to Verify

❓ Issue duplicate status

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

• I disagree

Reason for disagreement: My bug is not a duplicate of the other bug as these 2 bugs must be fixed independently. In my case, the issue is about the edit student command accepting unnecessarily long and complicated parameters (student ID) when just a simple list index would suffice. The student ID being uneditable has no relation to my reported bug.

❓ Issue response

Team chose [response.Rejected]

• I disagree

Reason for disagreement: Unlogical that student ID is a parameter when a list index would suffice. The student ID is numerical and long, thus prone to typos So, in this case, the team should instead use the list index for their commands.
And also, the team mentioned that "students with the same IDs but are in different classes are different identities", there's a very huge contradiction here since the students with the same IDs are treated as different when the team mentioned in the UG that "Student ID is being used as an identifier hence it is crucial in keeping the identity of the student separate"?

I feel that this is quite a huge design flaw and since most of their commands use the student ID as a parameter, it would affect the user-friendliness of the app if not corrected.

❓ Issue severity

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

• I disagree

Reason for disagreement: [replace this with your explanation]

from pe.