#1349 closed task (fixed)
[PATCH] change instances of "Citizen" class to "Worker"
| Reported by: | gudo | Owned by: | michael |
|---|---|---|---|
| Priority: | Nice to Have | Milestone: | Alpha 10 |
| Component: | UI & Simulation | Keywords: | patch, simple |
| Cc: | Patch: |
Description
Currently, the plough tech doesn't improve the gather rate for citizensoliders. Either the tooltip should be updated to reflect this, or the tech should be expanded to affect citizensoliders.
The attached patch takes the later approach
Attachments (2)
Change History (8)
comment:1 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:2 by , 12 years ago
| Milestone: | Backlog → Alpha 10 |
|---|
comment:3 by , 12 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Instead of creating yet another class, I think a better solution would be to add the "Worker" to template_unit_infantry and change plough tech to use that. The naming is better, it reduces class spread, and it would work for slave units.
Feel free to re-close if the current solution is good enough. (r11620 and r11619)
by , 12 years ago
| Attachment: | plough.json.patch added |
|---|
Changes plough to use "Worker", alters classes on Women and Infantry
comment:4 by , 12 years ago
| Summary: | [PATCH] plough tech doesn't improve citizen solider farming rate → [PATCH] change instances of "Citizen" class to "Worker" |
|---|---|
| Type: | enhancement → task |
comment:6 by , 8 years ago
| Keywords: | review removed |
|---|
Note:
See TracTickets
for help on using tickets.
Fixed. Added new "Citizen" class for this purpose and edited the templates and technology to suit.