mrcurry

1. Just cause you don't hear about it doesn't mean nothing is happening. Speaking about the countermeasures would make it easier for an attacker, and keep posting "we're working on it" and you'll look even more foolish. 2. Well is there any urgency? While we are customers, at the end of the day it's just a game and any attacker(s) will eventually tire and/or run out of resources or BIS will figure out their infrastructure. The attackers want anger, outrage and attention, it's the currency in whatever scheme they're playing at. All I'm going to give them is patience. This the perfect moment to pickup one of those 0-hours-played games in your steam library.

From a quick glance: private ["...", "_TotalSpawned", "_MaxAiTeams", "...", "", ""]; means your _TotalSpawned is local to your script and you are using it like a number, the name also implies a number. Numbers are passed by value which means the value of _TotalSpawned will never change unless you update it inside your loop.

CfgFunctions are always available too all, preinit functions will be called automatically on all machines. You should avoid execution of code that may or may not exist on the other machine. Only execute that which you know exists As a side note: Just so you are aware, defining the final function variables on your own like that means you do not get all the features of the functions library, like compileFinal or pre-preinit definition. Here's a not so organised example how to get full functionality.

Broadly speaking you can either use a shared data container like @pierremgi suggested, ( publicVariable or setVariable with broadcast: true ), he explained it so I won't. The other option is to implement server query and client response functions and use remoteExec to handle the comms. It's more complex but encapsulates the code well. To do this split the procedure into 3 parts: fnc_server_query_destination - Runs only on server. Does prep and remoteExecs function 2. on the client. fnc_client_select_destination_and_respond - Here the client takes over. This runs only on client with interface. This is where you use the previous trick with the "MapSingleClick" eventhandler to fetch the result. Then the client remoteExecs function 3. back onto the server. fnc_server_handle_response - The server takes over again and applies the changes i.e. makes the helo move and what not. The remoteExec calls in this solution need to be sent only between the machines involved, do not use 0 as target. That will not work as is, no. remoteExec is fire-and-forget so cannot return a value. I just want to clarify that there are two parts to the problem we're talking about: Getting the selected position from the eventhandler to the calling script on client's machine. Getting the selected position from the client to the server. 1. can be solved with the hashmap-trick or using setVariable on the player 2. can be solved with the server-client response pattern or using publicVariable or setVariable on the player with broadcast: true There are no right answers to which approach to take, choose whichever you feel you can pull off. If you need more examples just say which method you'd prefer.

The command has local effect so run it only when leader player == player.

So you're trying to pass data to/from the separate scopes while avoiding global vars? Use local variables in your "controller" script (the one which adds the eventhandler), pass the required values as the "arguments" parameter of addMissionEventhandler to have them available in the eventhandler's scope as _thisArgs If you need to return a value from the eventhandler you can try this, I'm 90% sure it maybe works... Edit: Tested it, it definitely works. In your controller script: _data = createHashMap; addMissionEventhandler [ "MapSingleClick", { _thisArgs params ["_returnData"]; _returnData set ["some key", "some value"]; //Return whatever you need }, [ _data] //Important ]; waitUntil { count _data > 0 }; systemChat "map clicked!"; Here's a complete example with comments showing a simple teleport feature demonstrating the technique where the event handler returns the position to the calling script appropriately. Ofc working with MP you need to think about the locality of your operations; server vs client, who does what, what is the command requirements (LA / LE / GA / GE). No code formatting cause lousy options on phone Edited on PC.

While this is true it is a simplified truth and I think it's worth highlighting for future readers that triggers are actually a special kind of object. thisTrigger refers to the actual trigger object which in turn holds all info like area, rotation, activation type and conditions, accessible through the triggerX commands.

Nice! Then you really don't need to worry about performance of compile Vs execVM at such an early stage, just get it working. 🙂 That said you can do rapid prototyping with CfgFunctions too. If you are still using VSCode you can setup a user snippet to quickly generate the CfgFunctions entry for use in a prototype mission, when you're done testing the resulting function file is ready to be dropped straight into a larger project with no changes. Tomato tomato and whatever floats the boat

Could I just take this moment to strongly recommend using the Functions Library to define your scripts into proper Functions: Arma 3 Functions Library Automatic compilation before anything runs. Your code will always be available as a missionNamespace variable TAG_fnc_functionName. And yes the code only compiles once. It's a different workflow than just creating sqf files, but very easy to work with once you get it set up and can really help keeping things neat and organised.

Nice! I'll just drop this link here: https://community.bistudio.com/wiki/getRoadInfo

I'm surprised you see any noticeable effect at all cause this line should be throwing/logging a script error. Unary (single argument) commands like "group" take precedence. With implict brackets added your code would look like so: _group = (group _this) select 1; but what you want is: _group = group (_this select 1);

@RobustcolorFrom the biki: So short answer is yes you need to add it to all pcs to cover all possible shooters

Neat! You know, you could just say that it's the users responsibility to verify that the trigger exists before it's defined like that... 😛 But incase you don't wanna go that route I'd go with requesting the variable name of the trigger or target as a string instead and then at the start of your loop where you validate the list read from missionNamespace to get the reference and update the input array with the correct value. Any code after won't know the difference. Example: Assume _inputArray contains the list to validate. { if( _x isEqualType "" ) then { if( isNil _x ) then { // It's not a valid variable name, either throw an error or handle it as regular string } else { // It's a valid variable, get the value private _val = missionNamespace getVariable _x; // Here you could do further validation that _val contains an expected value i.e. it's an Object. _x = _val; // set local ref for further validation in the loop _inputArray set [_forEachIndex, _val]; // set array ref for use later in the script }; }; // Handle _x == Object and _x == Number after } forEach _inputArray;

Now we're getting somewhere. The error message is quite clear. It's not your condition checking that's the problem. It errors on this line: [BLUFOR, [CSWR_spwnsBLU], CSWR_group_BLU_light, _form_BLU_2, _be_SAFE, [_move_ANY], [this_trigger_doesnt_exist]] call THY_fnc_CSWR_add_group; cause when you "set the trigger" you build an array like this: [this_trigger_doesnt_exist] This essentially does two operations in order: Read value from variable "this_trigger_doesnt_exist" Create an array and insert said reference in the first index. In step 1 if the variable is nil (undefined) you will be reading the value from an undefined location in memory. In essence you are asking "What's in the box?" and the game goes "What box?" and throws the error. You simply cannot do that directly. So what to do? That depends on what your requirement is. Do you know what the trigger will be named? Great. Just use getVariable to retrieve the value with a reasonable default e.g. objNull. Use isNull to verify the quality of the input. Is the trigger input by the user? Have the user input a string instead/as well and parse it, getVariable is your friend here too. Is the value set by another script? Prior to creating the array verify that the expected variable is actually defined using isNil "variable_name".

This is why describing the whole scenario from the get go is useful. There are many ways to skin a programmable cat. 🙂 Hmm, isNil "_x" should work... Try just checking if the value is undefined using: isNil { _x } as the condition. Just curly brackets and no stringification needed. If that fails I'm gonna have to boot up the game for some tests. If you're parsing "user input" to your function you can use param to retrieve the value instead like so: _x = _spwnDelayMethods param [_forEachIndex, objNull]; This way _x will fall back to objNull if its undefined. Put that at the start of your loop and you can just check isNull _x and do comparisons with isEqualX commands etc. Nice and clean.