nus-se-script
commented
2 years ago Team's Response
No details provided by team.
The 'Original' Bug
[The team marked this bug as a duplicate of the following bug]
Removing module doesn't delete mapping completely
Originally, uni 75 was mapped to mod 1. After removing mod 1, search map has deleted the record. But the "list /suni" command which lists the universities and module mappings in your list hasn't removed the mapping yet
[original: nus-cs2113-AY2122S1/pe-interim#829] [original labels: severity.High type.FunctionalityBug]
Their Response to the 'Original' Bug
[This is the team's response to the above 'original' bug]
As per the user guide, removing a module does not remove the mappings associated with it, in order to aid the user. This is as intended.
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)
- [x] I disagree
Reason for disagreement: This was stated as a feature flaw, not a functionality bug. Hence, the nature of the issue here is different from the 'original' bug and is hence not a duplicate. The point of the issue was that it brings inconvenience to the user as it doesn't make sense to have removed modules to be part of module mappings.
:question: Issue response
Team chose [response.Rejected]
- [x] I disagree
Reason for disagreement: This was stated as a feature flaw, not a functionality bug. Hence, the bug cannot be rejected just because it works as stated in the UG.
:question: Issue type
Team chose [type.FunctionalityBug]
Originally [type.FeatureFlaw]
- [x] I disagree
Reason for disagreement: It was included in the bug report that "This is a feature flaw, as removed modules shouldn't exist anymore in mappings.". It is not a functionality bug as I know that it works as stated in the UG. However, this brings inconvenience to the user as it doesn't make sense to have removed modules still in mappings.
:question: Issue severity
Team chose [severity.VeryLow]
Originally [severity.Medium]
- [x] I disagree
Reason for disagreement: 
This is definitely not a cosmetic issue, and hence the severity cannot be very low. It was listed as a medium feature flaw as users will use the delete module and mapping functions quite frequently, and not having an automatic delete mapping function will cause occasional inconvenience to the user.
When a module is removed from a user's modules, existing mapping aren't automatically removed. This is a feature flaw, as removed modules shouldn't exist anymore in mappings.