modifying the number of times a book has been borrowed

[okg00dbye]

No details provided.

the current implementation replaces the value, will a implementation that increments the current value instead be abit more useful? e.g currently the librarian will need to find the book and note down the current times value, +1 to it and then replace that value, but if its increment maybe they can just do times 1 t/{incrementValue} without needing to know its current value first

[nus-se-bot]

Team's Response

Thanks for reporting this issue:) I do believe this is a wonderful suggestion for future improvement. However, this issue does not affect the usage of this command, and since our product is mainly targeting at librarian for managing the library, they would usually add the times for a number of more than 1. In other words, every time they update the system, the addition in number is usually more than 1. Hence, the current design is more or less sufficient for our target users. With regret, we decide to reject this bug report.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

• [x] I disagree

Reason for disagreement: Hi, thank you for your explanation :-) To clarify, I think I was unclear in my bug report in the sense that it does not always have to be +1 (e.g times 1 t/{incrementValue} will add a value of {incrementValue}, and {incrementValue} may even be negative to allow more flexibility). With this clarification, this implementation will still be more optimized/efficient than the current implementation, as it will definitely reduce the number of mental steps the librarian has to do each time they want to update the value. Sorry for the lack of clarity in my report and hope this explanation can be reconsidered as a small feature flaw

---

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

• [ ] I disagree

Reason for disagreement: [replace this with your explanation]

---