naranghardik16 / pe

0 stars 0 forks source link

Command result showing wrong data (address) #4

Open naranghardik16 opened 3 years ago

naranghardik16 commented 3 years ago

When I tried to edit the details of person by changing the phone number, the output in the command result box was something like this (Actual):

Edited Person: David Li; Phone: 76787678; Email: 76787678; Address: 76787678; Tags: [family]

The output expected was:

Edited Person: David Li; Phone: 76787678; Email: lidavid@example.com; Address: Blk 436 Serangoon Gardens Street 26, #16-43; Tags: [family]

Screenshot 2021-11-12 at 4.40.51 PM.png

nus-pe-bot commented 3 years ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Command Result showing wrong data (Email)

The command entered is:

add -n John Doe -p +659875432 -e johnd@example.com

In the command result box the output that should be shown should be (Expected):

New person added: John Doe; Phone: +659875432; Email: johnd@example.com

The output that is actually shown in the command result box is (Actual):

New person added: John Doe; Phone: +659875432; Email: +659875432

Screenshot 2021-11-12 at 4.23.35 PM.png

The correct email is not shown in the command result box instead the phone number is displayed.

This bug is valid for other commands as well. When I tried delete 7, the output was (actual):

Deleted Person: John Doe; Phone: +659875432; Email: +659875432

The expected output is:

Deleted Person: John Doe; Phone: +659875432; Email: johnd@example.com

The email is however correctly shown in the contact list. The issue lies in the result being displayed in the command result box.


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

Their Response to the 'Original' Bug

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

Accepted. It was a mistake on our part since we introduced this bug as we were fixing another one...

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]