wyt-sonia/ped

ped's Issues

Long customer name cannot fully display

When adding a customer with a long name, his/her name cannot be displayed completely.
Maybe set the max-length of the name will be helpful.

This issue can be reproduced by this command:

• addc n/12345678901234567890123456789012345678901234567890 p/111 e/ww.@ww a/ADDRESS t/12

Long tag cause customer information invisible

When adding a customer with a long tag:

1. The tag won’t be fully displayed
2. The Phone, address and email information disappear as well

This can be reproduced using command:

• addc n/testLongTag p/11111111 e/ee@www a/110 t/1234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890

Issue of using multiple number of the same terms in command

For some commands, as editc and addc, the app does not block the command with multiple number of the same terms.

Thus, for example, when adding customer with multiple e/, n/ and p/ the app will only take the last term.

To reproduce the issue, can use the command shows in pictures:

Typo in error message

Here I think it is also a typo

Feedback box not refresh after 'editt'

The feedback box not refresh but the date time of the target transaction will update when updating dt of a transaction.
To reprodece it can use the cammand in picture.

Can plotsales with fure time

Also for plotsales, user can use date in future, I think it is not a valid logic here.

To reproduce : plotsales 2 sd/2022-03-20 10:00 ed/2022-03-30 10:00

Cannot search for “AP” using findp

The product list is as:

But using 'findp Ap' cannot get expected outcome:

Product list reordered unexpectedly after executing 'editp'

The product list reordered unexpectedly when use:

• editp 1 pr/00110 cp/00100 s/001999 q/00101

I recorded a vedio for this issue:

• 1st is Apple Watch

• 1st is Apple pencil
  

The log file generated is still named as address book

The log file generated is still address book version, which might make users confused.

This can be reproduced by downloading the jar file and launching it.



Long email cannot fully display

When adding a customer with long email, his/her emial will not display completely.

This issue can be repoduced by:

• addc n/testc p/12345678 e/ww1234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890@ss a/ADDRESS t/12

Typo in UG

Under chapter 3.22, 3.23 of the UG

Here you say: The start date must be after or equal to end date

I think it is a typo.

Can update product with pr/00110 cp/00100 s/001999 q/00101

The product price and cost price can be updated with number start with 0.

To reproduce it can use:

• pr/00110 cp/00100 s/001999 q/00101

Also this issue can let app stores 2 products that are actually the same.

Can edit the date time of a transaction to a time in future

Can edit the date time of a transaction to a time in future, which I think is not a valid logic here.

To reproduce, can use the command in the picture.

Findc command behaves differently from the UG

According to your UG, the findc should perfom OR search.
“Customers matching at least one keyword will be returned (i.e. OR search).

1. While when I try to search using "findc n/test11 p/111", even there is a customer named test11, the system displays nothing.
2. Same thing happens when I mix the search terms, like "findc p/222 e/12.@11" etc

Long phone number cannot fully display

When adding a customer with a long phone number, his/her phone number will not be displayed completely.
Also, the max-length of a valid phone number is 15 digits, can check from here: https://www.cm.com/blog/how-to-format-international-telephone-numbers/

Thus, maybe it is better to set the constrain to limit the length as well.

This issue can bre repoduced by:

• addc n/testc p/123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890 e/ww@ss a/ADDRESS t/12