#4016 closed defect (fixed)
[PATCH] Merge duplicate theatron templates
Reported by: | elexis | Owned by: | elexis |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 21 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description (last modified by )
The templates for the theatron are duplicated and should just use a parent template.
Code duplication results in one of the copies not being updated when changing things (and has already happened to that and many other files, for example ConquestCritical
wasn't updated in theb_theatron
).
This might be the case with other templates too and should be checked.
Attachments (3)
Change History (8)
by , 8 years ago
Attachment: | theatron.diff added |
---|
by , 8 years ago
Attachment: | theatron.2.diff added |
---|
by , 8 years ago
Attachment: | theatron.3.diff added |
---|
nuke the theb_theatron template as it is used nowhere
comment:1 by , 8 years ago
Keywords: | patch review added |
---|---|
Summary: | Merge duplicate theatron templates → [PATCH] Merge duplicate theatron templates |
comment:3 by , 8 years ago
Keywords: | simple review removed |
---|---|
Milestone: | Backlog → Alpha 21 |
Thanks, keep it up!
comment:5 by , 8 years ago
Description: | modified (diff) |
---|
Note:
See TracTickets
for help on using tickets.
or this one with the same sound for the theb one too