Skip to content
This repository has been archived by the owner on Apr 6, 2024. It is now read-only.

[PE-D][Tester E] pay command after find causes card to disappear #180

Closed
nus-se-script opened this issue Nov 3, 2023 · 3 comments · Fixed by #217
Closed

[PE-D][Tester E] pay command after find causes card to disappear #180

nus-se-script opened this issue Nov 3, 2023 · 3 comments · Fixed by #217
Assignees
Labels
triaged.Bug [External] Valid bug reports
Milestone

Comments

@nus-se-script
Copy link

When Alex owes 95 in this case, inputting a find command causes Alex to disappear.

image.png

image.png


Labels: type.FunctionalityBug severity.High
original: quzhetao01/ped#6

@kokrui
Copy link

kokrui commented Nov 6, 2023

I'm not sure what's going on here but it looks pretty scary and I'd like to fix it, so I've asked the reporter here quzhetao01/ped#6 (comment)

@conradsoon
Copy link

conradsoon commented Nov 6, 2023

@kokrui similar behaviour replicated but possibly not a bug

find bal/20 finds people whose balances are above $20.00.
Running pay/owe accordingly to cause their balance to drop below $20.00 causes their card to disappear (because it doesn't hit the criterion of balances being above $20.00 anymore. But arguably this is expected behaviour.

Unsure why this is the case actually, I thought it shouldn't affect LastShownList.

Good catch regardless @quzhetao01, thank you.

@kokrui kokrui self-assigned this Nov 8, 2023
@kokrui kokrui added this to the v1.4 milestone Nov 8, 2023
@kokrui kokrui added the triaged.Bug [External] Valid bug reports label Nov 8, 2023
@quzhetao01
Copy link

Hi just a friendly tip! Maybe you might want to specify this behaviour into your UG so it won't confuse anyone. Hopefully you find this helpful!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
triaged.Bug [External] Valid bug reports
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants