Adding additional Armor types

Bug #894966 reported by Millennium
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ares
Fix Released
Wishlist
DCoder DCoder

Bug Description

Would it be possible to have a feature to allow the creation of additional armor types? You could, for example, have a list:

[ArmorTypes]
1=none
2=flak
3=plate
4=light
5=medium
6=heavy
7=wood
8=steel
9=concrete
10=special1
11=special2
; this is where the fun starts...
12=ExampleNewArmor1
13=ExampleNewArmor2
14=...
etc

Now, on a warhead, the percentages listed in the "Verses" flag would be applied to the different warheads by checking their numbers on the list. For example:

[ExampleWarhead]
...
Verses=100%,90%,80%,100%,150%,120%,40%,30%,20%,100%,200%,175%,20%
...

The last two percentages in the "Verses" list on the warhead would be applied to the armors listed as 12 and 13 in the ArmorTypes list...

That'd be cool!

Revision history for this message
Millennium (kotaka) wrote :

Whoops! "none" should of course be item number 0 on the list, to keep it all ordered...

Revision history for this message
DCoder DCoder (dcoder1337) wrote :

Implemented as this:

[ArmorTypes]
;No need to redeclare the original types.
NewType = default_armor_type_to_take_verses_from

[Warhead]
Versus.NewType = float
;protip - Versus.OriginalType also works and overrides what is set in the old Verses value. Going to remove the original Verses parser too, since it causes IEs with wrong count of values.

Revision history for this message
Renegade (renegade) wrote :

Re-opened as per
291 is screwey and needs fixing, it has an odd problem with the versus (does not read, discards or forgets anything after the decimal point when calculating the damage to deal -i.e not allowing use of the weapon when the verses values say it'd deal 0.51 damage, which should be rounded up to 1)
http://forums.renegadeprojects.com/showthread.php?tid=1375&pid=14713#pid14713

Revision history for this message
Bug Importer (bug-importer) wrote :

Code related to this issue has just been checked in; DCoder wrote in revision 230:
Fixing issue 643 (SellUnit IE), fixing issue 291 ([ArmorTypes] defaults), fixing issue 639 (multiple loose PKTs), fixing issue 344 (Custom paradrop speed). Importing minidump, refactoring Insignia, removing useless logging entries. Updated manual!
Related to issue 643,291,639,344 .
SVN: http://svn.renegadeprojects.com/listing.php?repname=Ares&path=%2Ftrunk%2F&rev=230&sc=1

Revision history for this message
DCoder DCoder (dcoder1337) wrote :

Closing these issues - implemented a fair while ago, tested during implementation. If any bugs do crop up, be sure to thank the lazy testers for doing their job so well.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.