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

Auto-order sticks in busy state #337

Open
riverwanderer opened this issue Dec 4, 2024 · 3 comments
Open

Auto-order sticks in busy state #337

riverwanderer opened this issue Dec 4, 2024 · 3 comments
Assignees
Labels
beta Report from a Beta release bug Something isn't working Order Assist Testing Fix needs more testing
Milestone

Comments

@riverwanderer
Copy link
Owner

Module V5.1.0a1100
Vassal V3.7.15
Scenario 602 Austerlitz - 2 December 1805 (Davout’s Counter Attack)

beta10 602A auto order busy stuck.zip

@riverwanderer riverwanderer added bug Something isn't working beta Report from a Beta release Order Assist labels Dec 4, 2024
@riverwanderer riverwanderer added this to the 5.1.0 milestone Dec 4, 2024
@riverwanderer riverwanderer self-assigned this Dec 4, 2024
@riverwanderer
Copy link
Owner Author

Possibly relates to how Vassal implements the the screen refresh aspect of Sleep - see also #332 (Orders not displayed for Passive side until first unit is ordered). Theory - Screen refresh operates for local side when a blank window is displayed but this is not synchronised, and if not further screen changes occur, the other player(s) don't check a screen refresh.

@riverwanderer riverwanderer added the Testing Fix needs more testing label Jan 3, 2025
@riverwanderer
Copy link
Owner Author

Attempted fix by explicit reset of badOrders (maybe fixing spurious Stop display?) in p/t Orders Display. In a12.007. Tested with 3.7.15 and 3.8 (refresh PR).

@riverwanderer
Copy link
Owner Author

First attempt does not fix the issue, so isWaiting becomes a local property to prevent wait sign showing at all on other players' screens. Fix in a12.013.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta Report from a Beta release bug Something isn't working Order Assist Testing Fix needs more testing
Projects
Status: No status
Development

No branches or pull requests

1 participant