search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Lack of standardisation between indices and days #478

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

01 and 02 can be used for indices but not for days - since they are both looking for positive integers, it makes sense to decide whether or not to allow or disallow this behaviour for both parameters.

[original: nus-cs2103-AY2122S1/pe-interim#492] [original labels: severity.Low type.FunctionalityBug]

Moley456 commented 2 years ago

Team's Response

Thanks for the feedback!
Since no screenshots were provided, the issue here is quite unclear, but if we were to make a guess, the main concern here is about the difference in the behaviour of the INDICES and DAY(S) parameters right?
To address your concern, the team refers you to the Command Syntax section in our User Guide. DAY(S) only accepts the integers 1 - 7, while INDICES can be any positive integer, which includes the possibility of integers with more than 1 digit. Therefore, this is not a bug, but rather a deliberate decision by the team.

Duplicate status (if any):

--