Use a separate VehicleType when drawing APCW-type images

[DCoder DCoder]

Here's an idea. Instead of fussing with turret support, barrel support, SHP support and whatever else is needed for APCW logic, I can instead add a flag like WaterClass=APCW and draw [APCW] instead of [APC] when over water. Kind of like CMIN gets drawn as UnloadingClass=CMON. This would save me a lot of coding time, and should provide enough customisation.

Caveat: You'd have to actually add APCW to [VehicleTypes].

Vote below.

[Bug Importer (bug-importer)]

nice and simple solution.

[EVA-251 (eva-251)]

Optimal solution. One of the things that bugged me about TS's simple APCW system is that the APC lost no speed on water.

Changing vehicletype means you can make it move slower and use a different engine sound when in the water, as well as a new set of VOs if you have such clips available.

[DCoder DCoder (dcoder1337)]

Err, UnloadingClass (CMON/SCHD) is a graphical-only change, it doesn't change the unit's actual properties... making it actually change those on the fly is quite complicated and not in my plans for this specific issue.

[Bug Importer (bug-importer)]

Code related to this issue has just been checked in!
Author: DCoder
Location: trunk, r711
Revision comment:
Fixed issue #293 -
Fixed issue #513 -
Fixed issue #1030 - [Vehicle]WaterImage=APCW to draw an [APCW] image. Rather limited for now, the limitations might be lifted in the future:
- [APCW] has to be declared as a [VehicleType], similar to UnloadingClass.
- This is a graphical change only! - changes to other properties such as Speed, Locomotor, or whatever will not take effect.
- Cannot switch between SHP and Voxel images.
- Cannot change the weapons or their FLHs between states (falls under "other properties" really).
This deprecates the old WaterVoxel=yes logic... might have to bring it back if demand exists.
Related to issue 293,513,1030 .
SVN: http://svn.renegadeprojects.com/Ares/711

[lt albrecht (lt-albrecht)]

Wait so this means these units can have turrets and stuff now?

[WoRmINaToR (worminator)]

I may be mistaken, but it seems to me that this has been fixed, tested, and confirmed working.

If there's no more work to be done on this feature we can close this.