search

AY1920S1-CS2103T-W11-2 / main

SplitWiser - Epic debt tracking
https://ay1920s1-cs2103t-w11-2.github.io/main/
MIT License
0 stars 5 forks source link

Modify delete for deletion of contact, activity, expense based on context #196

Closed JohnNzj closed 4 years ago

liakify commented 4 years ago

If I do:

  1. expense p/you p/bob e/10
  2. settle p/bob p/you e/1
  3. delete 1 (i.e. delete the initial expense)
  4. close and open the app

the fact that you now owe bob $1 is lost

In the mean time can you poke around and see if you can spot any other funny edge cases either with the normal deleting or some re-serialising inconsistency

Aulud commented 4 years ago

If I do:

  1. expense p/you p/bob e/10
  2. settle p/bob p/you e/1
  3. delete 1 (i.e. delete the initial expense)
  4. close and open the app

the fact that you now owe bob $1 is lost

In the mean time can you poke around and see if you can spot any other funny edge cases either with the normal deleting or some re-serialising inconsistency

@liakify I've tried for half an hour and can't seem to find another sequence of commands and re-opening/closing the app apart from this that leads to odd behaviour. Do you have an inkling which of the stages (Logic, Model, Storage or UI) is causing this bug?