Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transferring more money than you have #834

Closed
DorpsGek opened this issue Jun 6, 2007 · 2 comments
Closed

Transferring more money than you have #834

DorpsGek opened this issue Jun 6, 2007 · 2 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

DorpsGek commented Jun 6, 2007

WJB opened the ticket and wrote:

When transferring money to other players:

Time 1: You have 22,47 mln, you want to send 22,50 mln
Time 2: Timeout, nothing happens
Time 3: You try again to send 22,50 , now an extra train arrived and you do have enough.
Time 4: The system now sends 22,47, the amount you had on timestamp 1.

Conclusion: I sended my money twice to another player!

Added a screenshot. Note that the console messages say i sended 22,47 + 22,50 mln, while my Financial record only displays the 22,50. The other user did receive both payments

Attachments

Reported version: 0.5.2
Operating system: Windows


This issue was imported from FlySpray: https://bugs.openttd.org/task/834
@DorpsGek
Copy link
Member Author

Rubidium wrote:

It's technically impossible to actually send the money twice. The first one succeeded and the second one failed. The other user did never receive both payments, because that would've meant that it would be substracted twice from your account to.

The only thing that could've happened is the fact that it "said" that it transfered money twice when it only sent (or rather performed) the action once. This is due to the fact that when that text is printed you haven't actually performed the operation yet.


This comment was imported from FlySpray: https://bugs.openttd.org/task/834#comment1323

@DorpsGek
Copy link
Member Author

Rubidium closed the ticket.

Reason for closing: Fixed

In r10087; will make it's way into 0.5.3


This comment was imported from FlySpray: https://bugs.openttd.org/task/834

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Development

No branches or pull requests

1 participant