Add seat parameter to setPedEnterVehicle

[MohabCodeX]

This PR adds an optional seat parameter to setPedEnterVehicle.

The implementation reuses the existing seat validation logic from warpPedIntoVehicle.
When a seat is specified, it validates that:

• The seat number doesn't exceed the vehicle's maximum passenger count
• The vehicle model supports passengers (some models return 255 for invalid passenger counts)

Syntax:

bool setPedEnterVehicle ( ped thePed [, vehicle theVehicle = nil, bool passenger = false, int seat = nil ] )

Optional Arguments:

• theVehicle: The vehicle to enter. If not specified, the ped will enter the nearest vehicle
• passenger: If set to true and no seat is specified, the ped will enter as a passenger
• seat: The seat number to enter (0-8). If specified, this overrides the passenger parameter

Note

The seat parameter takes priority over the passenger parameter.

Examples:

-- Enter driver seat
setPedEnterVehicle(ped, vehicle, false)

-- Enter as passenger (auto-determines seat)
setPedEnterVehicle(ped, vehicle, true)

-- Enter specific seats
setPedEnterVehicle(ped, vehicle, nil, 0)  -- Driver seat
setPedEnterVehicle(ped, vehicle, nil, 1)  -- Front passenger
setPedEnterVehicle(ped, vehicle, nil, 2)  

2025-11-23.15-13-33.2.mp4

Testing

hit.zip

[qaisjp]

This is really cool. Great pull request description too.

What do you think about just letting an int be provided as the third argument? I think this would be cleaner and remove the need for a dummy/unused argument.

You should be able to use a std::variant to achieve this, as demonstrated here:

mtasa-blue/Server/mods/deathmatch/logic/luadefs/CLuaColShapeDefs.cpp

Lines 825 to 843 in 3d02b86

	bool CLuaColShapeDefs::SetColPolygonHeight(CColShape* shape, std::variant<bool, float> floor, std::variant<bool, float> ceil)
	{
	if (shape->GetShapeType() != COLSHAPE_POLYGON)
	{
	throw std::invalid_argument("Shape must be a polygon");
	}
	auto* polygon = static_cast<CColPolygon*>(shape);
	float lowest = std::holds_alternative<bool>(floor) ? std::numeric_limits<float>::lowest() : std::get<float>(floor);
	float highest = std::holds_alternative<bool>(ceil) ? std::numeric_limits<float>::max() : std::get<float>(ceil);
	if (lowest > highest)
	{
	std::swap(lowest, highest);
	}
	return CStaticFunctionDefinitions::SetColPolygonHeight(polygon, lowest, highest);
	}

[qaisjp]

If this is copied from the warpPedIntoVehicle method, then I think it'd make sense to extract to a function that can be used by both. E.g. IsValidSeat

[qaisjp]

Great job, thanks for contributing this.

[qaisjp]

I wonder why this is a special case not accounted for by GetMaxPassengerCount but I won't argue with it since it was there before 😆

[qaisjp]

I'm moderately confident in this change, so I'm going to let this merge into 1.7 despite the feature freeze.

[qaisjp]

Thanks for sending this patch @MohabCodeX, you're churning out lots of great code!