[lewismichaelbbc]

Code:

CMD:buyveh(playerid, params[])
{
	if(GetPlayerVehicles(playerid) >= MAX_PLAYER_VEHICLES)
	{
		ShowErrorDialog(playerid, "You can't buy any more vehicles! Max: " #MAX_PLAYER_VEHICLES );
		return 1;
	}
	new vehicleid;
	new id = GetVehicleID(vehicleid);
	if(GetPlayerMoney(playerid) < VehicleValue[id])
	{
		ShowErrorDialog(playerid, "You don't have enough money to buy this vehicle!");
		return 1;
	}
	new freeid = GetFreeVehicleID();
	if(!freeid)
	{
		ShowErrorDialog(playerid, "Vehicle dealership is out of stock!");
		return 1;
	}
	GivePlayerMoney(playerid, -VehicleValue[id]);
	new dealerid = strval(VehicleOwner[id]);
	VehicleCreated[freeid] = VEHICLE_PLAYER;
	VehicleModel[freeid] = VehicleModel[id];
	VehiclePos[freeid] = DealershipPos[dealerid];
	VehicleColor[freeid] = VehicleColor[id];
	VehicleInterior[freeid] = VehicleInterior[id];
	VehicleWorld[freeid] = VehicleWorld[id];
	VehicleValue[freeid] = VehicleValue[id];
	GetPlayerName(playerid, VehicleOwner[freeid], sizeof(VehicleOwner[]));
	VehicleNumberPlate[freeid] = DEFAULT_NUMBER_PLATE;
	for(new d=0; d < sizeof(VehicleTrunk[]); d++)
	{
		VehicleTrunk[freeid][d][0] = 0;
		VehicleTrunk[freeid][d][1] = 0;
	}
	for(new d=0; d < sizeof(VehicleMods[]); d++)
	{
		VehicleMods[freeid][d] = 0;
	}
	VehiclePaintjob[freeid] = 255;
	VehicleLock[freeid] = 0;
	VehicleAlarm[freeid] = 0;
	UpdateVehicle(freeid, 0);
	SaveVehicle(freeid);
	new msg[128];
	format(msg, sizeof(msg), "You have bought this vehicle for $%d", VehicleValue[id]);
	SendClientMessage(playerid, COLOR_WHITE, msg);
	return 1;
}

I converted this from being in a dialog box to a command. However, when I type /buyveh and the price of the vehicle is $50, it will still only take $0... and the car doesnt spawn.

Anyone know why?