nus-cs2103-AY2122S2 / pe-dev-response

0 stars 0 forks source link

When adding event/person, it is allowed to use company' name with different cases(upper/lower), this fails to remove the event/person associated to the company when the company is deleted #2565

Open nus-pe-bot opened 2 years ago

nus-pe-bot commented 2 years ago

Reproduce the bug:

  1. `addc n/ByteDance t/bank e/bytedance@protonmail.com p/1234567 a/31 Race Card Road

    02-03 t/tech`

  2. adde n/Interview c/Bytedance d/2022-04-02 ti/14:00 l/Zoom this command is valid and being executed, note that even though "ByteDance" is being typed as "Bytedance", and there is no company called "Bytedance" in the current list of companies
  3. listc
  4. delete 10 where 10 is the index of "ByteDance" during testing

Expected outcome:

  1. Company "ByteDance" is deleted
  2. Event "Interview " which is associated to "ByteDance" is deleted

Actual outcome:

  1. Company "ByteDance" is deleted
  2. Event "Interview " which is associated to "ByteDance" is not being deleted, because in it the company is called "Bytedance"

image.png Note that the details in the screenshot is different from the one in bug report, however it serves the same purpose.


[original: nus-cs2103-AY2122S2/pe-interim#2602] [original labels: type.FunctionalityBug severity.High]

albertsutz commented 2 years ago

Team's Response

Reason for Changing to Medium: We accept that the bug is present. However, the application can still be used (it does not cause major problem for the users). It will cause some convenience (deleting the persons and events related to the deleted companies one by one). However, the application will still be useable.

Duplicate status (if any):

--