Opened 8 years ago
Closed 3 years ago
#4081 closed task (fixed)
Move PopulationBonus outside of Cost component
Reported by: | fatherbushido | Owned by: | |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 24 |
Component: | Simulation | Keywords: | simple |
Cc: | Patch: | Phab:D2948 |
Description (last modified by )
As noticed by elexis and discussed with sanderd17 and itms, PopulationBonus is not really a cost, so it should be move elsewhere. (It was yet discussed and suggest in http://irclogs.wildfiregames.com/2012-04/2012-04-21-QuakeNet-%230ad-dev.log. We can do a small component like PopulationBonus or perhaps something like PopulationHolder (itms suggestion).
Change History (8)
comment:1 by , 8 years ago
Description: | modified (diff) |
---|
comment:2 by , 8 years ago
Description: | modified (diff) |
---|
comment:3 by , 5 years ago
Component: | UI & Simulation → Simulation |
---|
comment:4 by , 4 years ago
Description: | modified (diff) |
---|---|
Keywords: | simple added |
Mainly searching and renaming, doesn't seem too hard?
comment:5 by , 4 years ago
Milestone: | Backlog → Work In Progress |
---|---|
Patch: | → Phab:D2948 |
comment:7 by , 3 years ago
Milestone: | Work In Progress → Alpha 24 |
---|
comment:8 by , 3 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
In r24394: Move PopulationBonus from cmpCost to new cmpPopulation.
Since PopBonus is not a cost.
Patch by: @lonehawk
Differential Revision: D2948
Comments by: @Angen, @Nescio, @wraitii
Closes #4081
Note:
See TracTickets
for help on using tickets.
Move tickets to
Simulation
asUI & Simulation
got some sub components.