Opened 3 years ago
Last modified 3 years ago
#6133 new enhancement
Clean Identity.js.
Reported by: | Freagarach | Owned by: | |
---|---|---|---|
Priority: | Should Have | Milestone: | Backlog |
Component: | Simulation | Keywords: | |
Cc: | Patch: |
Description
"<element name='Lang' a:help='Unit language for voices.'>"
seems like it belongs in the sound component, although one may argue that it is closely related to some other properties stored inIdentity.js
."<element name='Rank'>"
looks like it should be refactored out using a more extensible promotion system/component."<element name='Formations'
sounds like it should be inUnitAI
(or even a seperateFormationMember
component)."<element name='RequiredTechnology' a:help='Optional name of a technology which must be researched before the entity can be produced.'>"
smells like an extendedEntityLimits
,[Training|Build]-Restrictions
orCost
component of some sorts.
Change History (2)
comment:1 by , 3 years ago
comment:2 by , 3 years ago
Disagree for the former one. The language a unit speaks is definitely parts of one's identity :)
Note:
See TracTickets
for help on using tickets.
(One could also argue for
Controllable
andDeletable
to not belong in Identity.)