News:

Use the "Forum Search"
It may help you to find anything in the forum ;).

[Abuse] Player can unlock the other player's schedule button by switching the wa

Started by Ranran, May 26, 2021, 01:18:14 PM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Ranran

Reproducibility: 100%

Reproduction procedure:
(1) Open the Player A's line management dialog.
(2) Keep it open and switch to Player B (at this point the button on that dialog will be locked once)
(3) Switch the waytype tab of Player A's Line management dialog
(4) The Edit Line button will be unlocked, so use it to open the schedule edit dialog.
(5) The opened dialog is Player A's Line, but Player B can edit it freely. (´・ω・`)
(6) Withdraw all button is also available
(7) Retire all Player A's convoys  :P


You can also change the line name.  ;) This is another issue and you can change it at any time without having to go through the steps above.


This was brought about by a merge from the standard or it has been lurking for a long time. And it was still reproducible as of r9274. I don't know how it is now in standard. The related dialogs are currently undergoing major changes. I would appreciate it if someone could confirm it.

This is caused by calling update_lineinfo(). The button is then unlocked. There are some other processes to unlock the button.

I threw a pull request to fix this. please confirm.
https://github.com/jamespetts/simutrans-extended/pull/383

prissi

I wonder why this works. In standard this just brings up the password dialog. The same should happen with extended, at least if this is done properly via the tools.

Ranran

I think password entry is only relevant for locked players. It works fine.
I think the current specification is that we can't change the schedule of other players. So even if player want to edit the schedule of an unlocked player, you still need to switch players once.
The schedule list dialog also behaves like that in the initial state, but the above operation causes inconsistent behavior.
For example, the convoy dialog also disables the schedule button for other players, whether locked or not.

Ronin1996

I noticed that when they updated a while back, they gave public player more... power if you will. Did you change public player to player 2? That might be why.

prissi


jamespetts

Download Simutrans-Extended.

Want to help with development? See here for things to do for coding, and here for information on how to make graphics/objects.

Follow Simutrans-Extended on Facebook.