Fix syncRotation ignored to AttachPlayerObjectToObject

[SimoSbara]

This PR references openmultiplayer/open.mp/1163.

[AmyrAhmady]

You should not move function position from its own place to another
Adding a parameter already breaks ABI compatibility, but moving the position would break it for every other function in this table as well
Please fix it by moving it back to its own original position

[ksenonadv]

Could you please keep the original method in place with the same number of args it had before and just add another one at the end of the table with that extra param?

[SimoSbara]

Hi, like that?

struct IPlayerObject : public IBaseObject
{
	/// Attach the object to a player
	virtual void attachToPlayer(IPlayer& player, Vector3 offset, Vector3 rotation) = 0;

	/// Attach the object to another player object
	virtual void attachToObject(IPlayerObject& object, Vector3 offset, Vector3 rotation, bool syncRotation) = 0;
};

[ksenonadv]

Something like:

struct IPlayerObject : public IBaseObject
{
	// Attach the object to another object (rotation is always synced)
	virtual void attachToObject(IPlayerObject& object, Vector3 offset, Vector3 rotation) = 0;

	/// Attach the object to a player
	virtual void attachToPlayer(IPlayer& player, Vector3 offset, Vector3 rotation) = 0;

	/// Attach the object to another object (with rotation sync option)
	virtual void attachToObject(IPlayerObject& object, Vector3 offset, Vector3 rotation, bool syncRotation) = 0;
};