search

AY1920S1-CS2103T-T10-1 / main

JARVIS
https://ay1920s1-cs2103t-t10-1.github.io/main/
MIT License
0 stars 8 forks source link

Undo and redo fails if the user intends to undo/redo tasks after adding finance data #289

Closed nus-pe-bot closed 5 years ago

nus-pe-bot commented 5 years ago

Steps to reproduce:

  1. Add a task add-task t/todo des/Test1
  2. Perform this command add-paid d/Test2 a/400.
  3. Delete task Test1.
  4. Attempt to perform an undo for Test1.

[original: weihaw08/ped#11]

marcfyk commented 5 years ago

Undo was failing due to variable shadowing on command execution. Careless mistake of shadowing the field responsible for remembering the task that was deleted.

Bug has been fixed, the task deletion's undo operation works correctly now.