Introduce Player Debug UI + fixes for debug stuffs

[Deewarz]

Main

• Introduce Player Debug UI

Also

• Cleanup dev features
• EntityBrowser UI, Vehicle Debug UI, Debug window can now be closed
• Improve UX for pos, dir, rot debug sliders
• Import sdk/entities/human/c_human_script.h in sdk/entities/c_human_2.h (cleanup files which import both)
• Remove sdk/entities/c_human_2.h import when sdk/entities/c_player_2.h is already imported
• Remove automatic weapon giving when human is created or setup
• Fix C_HumanScript::GetInvulnerabilityByScript
• Documentation for C_PlayerTeleportModule::TeleportPlayer known parameters
• Documentation for C_Human2::IsInCoverFull
• Documentation for C_HumanWeaponController::DoWeaponSelectByItemId

Known issues

• SetScale needs a single float but GetScale returns a Vector3 => Override JS player/vehicle GetScale/SetScale methods to only use scalar float #49
• We can't kill a player => Research: trigger player death #52
• SDK Player.addWeapon doesn't works => Fix Player.addWeapon JS built-in #51
• SDK Player.setPosition doesn't works in the event playerConnected => SDK::GetPlayerTeleportModule()->TeleportPlayer fails to teleport local player #45
• SDK chat args[0] is the command name when there is no args => Fix incorrect handling of arguments in chatCommand event #50
• SDK sdk.World.createVehicle returns an object even when the vehicle doesn't exists

I will create specific tickets for these issues.

[zpl-zak]

• SetScale needs a single float but GetScale returns a Vector3

SetScale needs to remain Vector3 as it's defined by the Framework itself, MafiaMP specifically can just rely on either X,Y,Z value to uniformly scale an object or use its own field in netcode for that.

• SDK Player.setPosition doesn't works in the event playerConnected

See #45

• SDK sdk.World.createVehicle returns an object even when the vehicle doesn't exist

server is not aware of supported vehicles. Back in LH-MP we would have a whitelist of supported model names, but I do not think it's appropriate these days, let's consider it a user error if modder tries to spawn a non-existent vehicle, because from server's POV nothing forbids it to do so.

[Segfaultd]

I've requested a few changes, otherwise the whole idea looks good.

We should avoid introducing new ways-of-doing in term of codestyle, and also avoid breaking code consistency.

[Deewarz]

@zpl-zak

• SDK sdk.World.createVehicle returns an object even when the vehicle doesn't exist

server is not aware of supported vehicles. Back in LH-MP we would have a whitelist of supported model names, but I do not think it's appropriate these days, let's consider it a user error if modder tries to spawn a non-existent vehicle, because from server's POV nothing forbids it to do so.

Couldn't we just detect if the server created the vehicle or not? not necessarily a validation of the model in itself.

[zpl-zak]

@zpl-zak

• SDK sdk.World.createVehicle returns an object even when the vehicle doesn't exist

server is not aware of supported vehicles. Back in LH-MP we would have a whitelist of supported model names, but I do not think it's appropriate these days, let's consider it a user error if modder tries to spawn a non-existent vehicle, because from server's POV nothing forbids it to do so.

Couldn't we just detect if the server created the vehicle or not? not necessarily a validation of the model in itself.

but server always creates the vehicle, it will always succeed to do so.