Set command does not update current borrows.

[sreesubbash]

After borrowing a book,

I set load period to 1 day but the due date for currently borrowed books does not update.

While it can be argued that due dates should not be updated. Maximum renew count is more pressing.

Here I have updated maximum renew count to 2

But because the book was borrowed before making this max renew count to 2,

The app still thinks animal farm cannot be renewed. I believe it is using the past value of renew count of 1.
This incorrect behaviour as renew counts are library policy and should be updated globally.

[nus-pe-bot]

Team's Response

Cannot be reproduced. Tested and works. It is very likely that the tester tried to renew the same book more than once in the same serve session, which is not allowed as stated in the UG. i.e., in the same serve mode session, the tester likely renewed that book, then set mr to 2, and tried renewing again. As intended, this does not work, which is why the error says there are no books that still can be renewed currently as that book was just renewed in the same session. If the user is done from serve mode and serves that borrower again to renew, it works fine.

Items for the Tester to Verify

❓ Issue response

Team chose [response.CannotReproduce]

• I disagree

Reason for disagreement: [replace this with your reason]

---

❓ Issue severity

Team chose [severity.Medium].
Originally [severity.High].

• I disagree

Reason for disagreement: [replace this with your reason]

---