Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#1808 closed defect (fixed)

[PATCH] unable to un-garrison some units

Reported by: mimo Owned by: leper
Priority: Should Have Milestone: Alpha 13
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description

to reproduce the problem, start the peloponnese map and garrison your units in the cc. The button to un-garrison the athenian marines is greyed and inactive. The patch attached fixes this bug.

Attachments (1)

garrisonBug.patch​ (799 bytes ) - added by mimo 11 years ago.

Download all attachments as: .zip

Change History (5)

by mimo, 11 years ago

Attachment: garrisonBug.patch​ added

comment:1 by historic_bruno, 11 years ago

Component: Core engine → UI & Simulation
Keywords: patch review added
Milestone: Backlog → Alpha 13

Please follow the guidelines at SubmittingPatches, especially adding "patch review" keywords to the ticket, so it shows up in search queries.

comment:2 by mimo, 11 years ago

I suppose #1672 has the same origin and would be fixed by this patch.

Last edited 11 years ago by historic_bruno (previous) (diff)

comment:3 by leper, 11 years ago

Owner: set to leper
Resolution: → fixed
Status: new → closed

In 13072:

Allow ungarrisoning units even if they cannot be built due to tech requirements. Patch by mimo. Fixes #1808, #1672.
Fix batch training tooltip if we have more units than the entity limit allows for.

comment:4 by leper, 11 years ago

Keywords: review removed

Thanks for the patch.

Note: See TracTickets for help on using tickets.