search

AY1920S2-CS2103T-W12-1 / main

Delino - Managing Delivery Tasks
https://ay1920s2-cs2103t-w12-1.github.io/main/
MIT License
0 stars 6 forks source link

[PE-D] Delivered Command with does not deliver the order of the given `INDEX` #287

Closed nus-pe-bot closed 4 years ago

nus-pe-bot commented 4 years ago

Delivered command with the -r flag does not deliver the return order of the specified INDEX.

Update: I realised that this bug does not only occur with the delivered command. Other commands such as edit face the same issue as well.

How to reproduce the bug:

  1. From a clean state of the app with default populated data, run delivered -r 1

image.png

  1. The result will be correct.

image.png

  1. Then, run delivered -r 6.

image.png

  1. The result is correct but the index 6 return order will jump to index 3 instead

image.png

  1. Now, I try to delete the new index 6 return order using delivered -r 6.

image.png

  1. The result will save order's delivered when the displayed index 6 return order is not delivered.

image.png

Labels: severity.High type.FunctionalityBug original: zenatrick/ped#1

JeremyLoh commented 4 years ago

Hi @zenatrick, Thanks for the bug report! It was due to an incorrect implementation of our default sorting. This has been resolved in 676145a