#3590 closed enhancement (fixed)
[PATCH] Undeletable entities
Reported by: | leper | Owned by: | leper |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 20 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description
For some maps having undeletable (and indestructible (which can be done by having a trigger using cmpArmour.SetInvulnerability()
already) entities can enable some nice gameplay features.
Examples would be trade posts on the Silk Road map niektb is working on, or mercenary camps.
Below patch takes care of this, selecting entities and pressing del still opens up the delete dialog, but no deletion takes place.
Attachments (2)
Change History (6)
by , 8 years ago
Attachment: | undeletable.diff added |
---|
comment:1 by , 8 years ago
comment:3 by , 8 years ago
Keywords: | review removed |
---|
by , 8 years ago
Attachment: | t3590_fix_tests_v1.patch added |
---|
Note:
See TracTickets
for help on using tickets.
Any reason for not doing as for mirages when pressing the del key, i.e. returning a tooltip explaining why you can't delete it and a different icon in the getinfo function, and adding a return in the execute function ? Apart from that, looks fine.