search

nus-cs2103-AY2122S1 / pe-dev-response

0 stars 0 forks source link

Error message doesn't match the cause of the error #5821

Open nus-se-bot opened 2 years ago

nus-se-bot commented 2 years ago

addpolicy n/Life Policy p/1000 c/10 4 cl/1 e/2100-06-13 t/AIA

The error message displayed for the above command is shown below:

Commission percentage should be a number from 0 to 100 representing the percentage; Num of payments with commission should be non-negative and not larger than the num of payments in the payment structure and 2147483647

The error message could be clearer about the missing [PMTS_PER_YR] parameter

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

yatbun commented 2 years ago

Team's Response

Actually, this error was because the number of payments with commission is larger than the number of payments in the payment structure which was mentioned in the error message. It was provided that there were 4 payments with commission yet there is only one payment in the payment structure, as defined in your command.

In addition, this was also warned in the use guide explicitly. image

Duplicate status (if any):

--