Skip to content

Build Order Fixes - #11771

Open
armele wants to merge 6 commits into
ldtteam:version/mainfrom
armele:builder_order
Open

Build Order Fixes#11771
armele wants to merge 6 commits into
ldtteam:version/mainfrom
armele:builder_order

Conversation

@armele

@armele armele commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Fix town hall prioritization of build orders and display builder queue in priority order.

Checklist

  • I have read and accept the Contributing Guidelines
  • I have tested and confirmed my changes are working on the most recent commit of this pull request
  • I have used AI in the creation of this pull request

AI was used to analyze the current logic for better understanding, and to review my code changes.

Related issues

Closes #11388

Changes proposed in this pull request

  • Work-order priority now controls Builder selection, with claimed orders processed before unclaimed orders and work-order ID breaking "ties".
  • The Builder hut shows work in priority order (with all claimed work ahead of any work available to be assigned when in manual mode).
  • The current work order border is highlighted in green for a visual signal of the current job.

Testing

  • Assigned multiple builds to a builder, then reordered the priority in the town hall. Observed that the builder moved on to the reordered "next priority" rather than picking the building in the sequence the builds were requested.
  • Verified builder hut ordering matched priority order.
  • Verified "available" work orders were present when the builder was in manual mode, and appeared after all assigned/claimed work orders.
  • Verified priority persisted through a restart

Review please

Fix town hall prioritization of build orders and display builder queue in priority order.

@Raycoms Raycoms left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some comments

Comment thread src/main/java/com/minecolonies/core/colony/workorders/WorkManager.java Outdated
Comment thread src/main/java/com/minecolonies/core/colony/workorders/WorkManager.java Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

work order priority doesn't work

2 participants