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 Silier)

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 fatherbushido, 8 years ago

Description: modified (diff)

comment:2 by fatherbushido, 8 years ago

Description: modified (diff)

comment:3 by Imarok, 5 years ago

Component: UI & SimulationSimulation

Move tickets to Simulation as UI & Simulation got some sub components.

comment:4 by Freagarach, 4 years ago

Description: modified (diff)
Keywords: simple added

Mainly searching and renaming, doesn't seem too hard?

comment:5 by Silier, 4 years ago

Milestone: BacklogWork In Progress
Patch: Phab:D2948

comment:6 by Silier, 4 years ago

Description: modified (diff)

fix irc link

comment:7 by Freagarach, 3 years ago

Milestone: Work In ProgressAlpha 24

comment:8 by Freagarach, 3 years ago

Resolution: fixed
Status: newclosed

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.