feat(onesync/natives): add GetVehicleHornType

[ahcenezdh]

Goal of this PR

Adding a server side getter to the client side native START_VEHICLE_HORN

How is this PR achieving the goal

by reading the HornTypeHash bit in the CVehicleAppearance data node.

This PR applies to the following area(s)

FXServer/FiveM

Successfully tested on

Game builds: 3095

Platforms: Window

Was tested by using StartVehicleHorn client side, then getting the hornType value server side.

-- Client side
RegisterCommand("setHorn", function()
  local veh = GetVehiclePedIsIn(PlayerPedId(), false)
  StartVehicleHorn(veh, 1000000, GetHashKey("HELDDOWN"), false)
end)

-- Server Side
RegisterCommand("hornType", function(source, args)
    local playerPed = GetPlayerPed(source)
    local veh = GetVehiclePedIsIn(playerPed, false)
   if GetVehicleHornType(veh) == GetHashKey("HELDDOWN") then
        print("yes")
   end
end)

To test it, use this snippet, use the setHorn command, then the hornType command

Checklist

• Code compiles and has been tested successfully.
• Code explains itself well and/or is documented.
• My commit message explains what the changes do and what they are for.
• No extra compilation warnings are added by these changes.

[ahcenezdh]

Should be good @AvarianKnight, thanks

[FabianTerhorst]

Can you update your commit messages? Otherwise we can do that as well.

[ahcenezdh]

Can you update your commit messages? Otherwise we can do that as well.

Should be good

[FabianTerhorst]

Can you update your commit messages? Otherwise we can do that as well.

Should be good

Thanks 👍

[FabianTerhorst]

the master branch merge was not required, because there was no conflict. Now you need to get that commit await.

[ahcenezdh]

the master branch merge was not required, because there was no conflict. Now you need to get that commit await.

Should be good but looks like the CI didn't succeed to Validate my PR

[FabianTerhorst]

The ci isn't happy, because you didnt excluded redm from the code that only works with fivem.

#ifdef STATE_FIVE

around the native handler.

and add

game: gta5

the the native declaration.

[ahcenezdh]

The ci isn't happy, because you didnt excluded redm from the code that only works with fivem.

#ifdef STATE_FIVE

around the native handler.

and add

game: gta5

the the native declaration.

Looking at ServerGame_Scripting only cross native are using the #idef STATE_FIVE

[FabianTerhorst]

Looks good now 👍