WaterState=

Bug #895948 reported by Black Temple Gaurdian
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ares
New
Wishlist
Unassigned

Bug Description

Basically, the string for WaterState= would be another unit, which determines the unit's attributes.

For example, if I want my SEAL to swim underwater (cloaked) I'd have to make him permenantly cloaked. However, with this tag;
[GHOST]
WaterState=GHOSTW
Speed=4

[GHOSTW]
Cloak(or whatever it was)=yes
Speed=6

The SEAL would only cloak when in water. Additionally, it'd be faster in water than on land. Hope this is as clear as it is in my mind.

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

Not a duplicate, merely related. That other issue only deals with VehicleTypes and graphical changes, not functional ones.

That said, don't expect the functional changes to happen here either.

Revision history for this message
Renegade (renegade) wrote :

According to yourself, in the commit message
[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.
your current implementation was basically exactly this, you just limited it to only use the image.

WaterImage= is a flag taking a VehicleType to change the current vehicle's properties when it's in water.
The proposed WaterState= is a flag taking a VehicleType to change the current vehicle's properties when it's in water.

The only difference is the fact that the current implementation is limited in the attributes that can be overridden, while this request asks for full control.

I don't see how this isn't a duplicate - an extension of the already existing solution to #1030 should completely fulfill this request.

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

Funny, I don't remember GHOST being a technosexual. :)

Revision history for this message
Renegade (renegade) wrote :

Hence why he needs the extension...he's lacking the drive.

Edit: Let's approach this from the other side - are you proposing that, in a hypothetical case where this were implemented, we should have both WaterState=, taking a TechnoType that modifies all attributes of the current unit, and WaterImage=, taking a VehicleType that modifies select attributes of the current unit?

There is a very clear duplication there, imo.

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

Meh, I said I wasn't planning on making the functional changes work here either.

My point was that the other issue needs to be extended to all TechnoTypes before it can be considered a duplicate of this.

Revision history for this message
Renegade (renegade) wrote :

And my point is that this request is inherently a duplicate of the other issue, since its functionality is a superset of the other issue's.

No matter which point one subscribes to, the more important question is: Is anyone going to code this?
I'm not.

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

Something I just realised - you can already set up swim sequences independently of ground ones, so without the functional changes this is completely useless.

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

Other bug subscribers

Related blueprints

Remote bug watches

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