aaronxujiachen / pe

0 stars 0 forks source link

ViewBudget feature should be made more secure #6

Open aaronxujiachen opened 10 months ago

aaronxujiachen commented 10 months ago

Same problem as the viewFinancialStatement feature that I mentioned previously.

Screenshot 2023-11-17 170342.png

nus-se-script commented 10 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]

ViewFinancialStatement feature should be made more secure

According to the UG, the correct format of viewing all financal statements should be "viewFinancialStatement".

Screenshot 2023-11-17 165848.png

However, it seems like the command does not consider the mixing of uppercase and lowercase.

Screenshot 2023-11-17 165823.png


[original: nus-cs2113-AY2324S1/pe-interim#63] [original labels: severity.Medium type.FeatureFlaw]

Their Response to the 'Original' Bug

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

On the contrary, it was designed as such on purpose, so that casing was not a problem. In fact, this was explicitly mentioned on the CS2113 website.

image.png

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 response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]