Xeno69 / Domination

Domination - A MP mission for Arma 3
85 stars 36 forks source link

airdrop fighting vehicles and some bug fixes #607

Closed longtimegamer closed 10 months ago

longtimegamer commented 11 months ago

I added some vehicles to the airdrop if d_with_airdrop == 2

image

longtimegamer commented 11 months ago

I am still testing this. Playing SOG maps gives the players a message airdrop not available. Probably related to this pull request.

longtimegamer commented 10 months ago

Ok, I fixed the problem. This is ready to merge.

longtimegamer commented 10 months ago

I also added two factions using the existing definitions for CUP Russian Armed Forces and CUP Takistani Armed Forces.

longtimegamer commented 10 months ago

I also added OPFOR CUP Russian Armed Forces Winter and INDFOR ION PMC and ION PMC Arctic.

longtimegamer commented 10 months ago

ok I think that is all, this pull request is getting long

Xeno69 commented 10 months ago

I'm sorry but I won't merge this.

  1. You replace simple boolean checks which cost virtually no CPU cycles with case sensivtive string checks that are quite CPU heavy (laser).
  2. ptakeweapon does checks even if ranked is disabled (if weapons are missing then parts of the classes have to be added like it is done for other mods)
  3. d_cas_plane_ai can also be an array so the check has to stay

There are probably some more issues.

longtimegamer commented 10 months ago
  1. Sure, easy fix
  2. You should look again. This bug has broken Domination for years.
  3. Sure, easy fix.

"probably more issues" - This is not helpful.

Xeno69 commented 10 months ago

I know that it isn't helpfull but as I have no A3 on my drives anymore I can only check here on GitHub.

But for example, why do you pv these variables?

publicVariable "d_sm_ammotrucktype_own_side";
publicVariable "d_sm_apc_own_side";
publicVariable "d_sm_HunterGMG_own_side";
publicVariable "d_sm_tank_own_side";
longtimegamer commented 10 months ago

The code comment explains why. It only runs once.

longtimegamer commented 10 months ago

I can't work with "probably more issues." I will leave you alone.

Xeno69 commented 10 months ago

I'm merging this now, please fix the issues I've mentioned.

Xeno69 commented 10 months ago

And you don't need to pv those, only when d_with_airdrop == 2