search

nus-cs2103-AY2021S1 / forum

20 stars 2 forks source link

Can the user mess with the preference/json/config file and claim it as a bug? #423

Open GabrielSimbingyang opened 3 years ago

GabrielSimbingyang commented 3 years ago

Hi,

I have received several bugs reports where the user directly modified the data file to hold invalid data then reported the issue as high severity bugs )=

Regards, Gabriel Sim

damithc commented 3 years ago

As per https://nus-cs2103-ay2021s1.github.io/website/admin/tp-constraints.html#constraint-human-editable-file, there is some expectation of users being able to modify the data file. You can still argue that that ability is not supported yet, but the software is still expected to fail gracefully if the file doesn't contain valid data i.e., should not crash

GabrielSimbingyang commented 3 years ago

Hi Prof,

Must the data edits be done in a reasonable manner?

The user modified the application width and height of the application to an unreasonable size (1.0E7) and then claimed my team's application does not appear on the screen and hence cannot be resized.

Regards, Gabriel

damithc commented 3 years ago

Must the data edits be done in a reasonable manner?

The user modified the application width and height of the application to an unreasonable size (1.0E7) and then claimed my team's application does not appear on the screen and hence cannot be resized.

This feature is meant for advance users. So, they are supposed to know what they are doing. It's fine as long as the user can recover by reverting the change to the file and restarting the application.

GabrielSimbingyang commented 3 years ago

Thanks prof @damithc !