CSM: Bugfixes to camera:get_pos() and camera:get_fov()

closes #9857

2 files changed, 44 insertions, 31 deletions

diff --git a/src/script/lua_api/l_camera.cpp b/src/script/lua_api/l_camera.cpp
index 9c1470284..bfa60be67 100644
--- a/src/script/lua_api/l_camera.cpp
+++ b/src/script/lua_api/l_camera.cpp
@@ -51,6 +51,7 @@ void LuaCamera::create(lua_State *L, Camera *m)
 	lua_setfield(L, objectstable, "camera");
 }
 
+// set_camera_mode(self, mode)
 int LuaCamera::l_set_camera_mode(lua_State *L)
 {
 	Camera *camera = getobject(L, 1);
@@ -67,17 +68,19 @@ int LuaCamera::l_set_camera_mode(lua_State *L)
 	return 0;
 }
 
+// get_camera_mode(self)
 int LuaCamera::l_get_camera_mode(lua_State *L)
 {
 	Camera *camera = getobject(L, 1);
 	if (!camera)
 		return 0;
 
-	lua_pushnumber(L, (int)camera->getCameraMode());
+	lua_pushinteger(L, (int)camera->getCameraMode());
 
 	return 1;
 }
 
+// get_fov(self)
 int LuaCamera::l_get_fov(lua_State *L)
 {
 	Camera *camera = getobject(L, 1);
@@ -85,9 +88,9 @@ int LuaCamera::l_get_fov(lua_State *L)
 		return 0;
 
 	lua_newtable(L);
-	lua_pushnumber(L, camera->getFovX() * core::DEGTORAD);
+	lua_pushnumber(L, camera->getFovX() * core::RADTODEG);
 	lua_setfield(L, -2, "x");
-	lua_pushnumber(L, camera->getFovY() * core::DEGTORAD);
+	lua_pushnumber(L, camera->getFovY() * core::RADTODEG);
 	lua_setfield(L, -2, "y");
 	lua_pushnumber(L, camera->getCameraNode()->getFOV() * core::RADTODEG);
 	lua_setfield(L, -2, "actual");
@@ -96,16 +99,18 @@ int LuaCamera::l_get_fov(lua_State *L)
 	return 1;
 }
 
+// get_pos(self)
 int LuaCamera::l_get_pos(lua_State *L)
 {
 	Camera *camera = getobject(L, 1);
 	if (!camera)
 		return 0;
 
-	push_v3f(L, camera->getPosition());
+	push_v3f(L, camera->getPosition() / BS);
 	return 1;
 }
 
+// get_offset(self)
 int LuaCamera::l_get_offset(lua_State *L)
 {
 	LocalPlayer *player = getClient(L)->getEnv().getLocalPlayer();
@@ -115,38 +120,40 @@ int LuaCamera::l_get_offset(lua_State *L)
 	return 1;
 }
 
+// get_look_dir(self)
 int LuaCamera::l_get_look_dir(lua_State *L)
 {
-	LocalPlayer *player = getClient(L)->getEnv().getLocalPlayer();
-	sanity_check(player);
-
-	float pitch = -1.0 * player->getPitch() * core::DEGTORAD;
-	float yaw = (player->getYaw() + 90.) * core::DEGTORAD;
-	v3f v(std::cos(pitch) * std::cos(yaw), std::sin(pitch),
-			std::cos(pitch) * std::sin(yaw));
+	Camera *camera = getobject(L, 1);
+	if (!camera)
+		return 0;
 
-	push_v3f(L, v);
+	push_v3f(L, camera->getDirection());
 	return 1;
 }
 
+// get_look_horizontal(self)
+// FIXME: wouldn't localplayer be a better place for this?
 int LuaCamera::l_get_look_horizontal(lua_State *L)
 {
 	LocalPlayer *player = getClient(L)->getEnv().getLocalPlayer();
 	sanity_check(player);
 
-	lua_pushnumber(L, (player->getYaw() + 90.) * core::DEGTORAD);
+	lua_pushnumber(L, (player->getYaw() + 90.f) * core::DEGTORAD);
 	return 1;
 }
 
+// get_look_vertical(self)
+// FIXME: wouldn't localplayer be a better place for this?
 int LuaCamera::l_get_look_vertical(lua_State *L)
 {
 	LocalPlayer *player = getClient(L)->getEnv().getLocalPlayer();
 	sanity_check(player);
 
-	lua_pushnumber(L, -1.0 * player->getPitch() * core::DEGTORAD);
+	lua_pushnumber(L, -1.0f * player->getPitch() * core::DEGTORAD);
 	return 1;
 }
 
+// get_aspect_ratio(self)
 int LuaCamera::l_get_aspect_ratio(lua_State *L)
 {
 	Camera *camera = getobject(L, 1);
@@ -215,13 +222,19 @@ void LuaCamera::Register(lua_State *L)
 	lua_pop(L, 1);
 }
 
+// clang-format off
 const char LuaCamera::className[] = "Camera";
-const luaL_Reg LuaCamera::methods[] = {luamethod(LuaCamera, set_camera_mode),
-		luamethod(LuaCamera, get_camera_mode), luamethod(LuaCamera, get_fov),
-		luamethod(LuaCamera, get_pos), luamethod(LuaCamera, get_offset),
-		luamethod(LuaCamera, get_look_dir),
-		luamethod(LuaCamera, get_look_vertical),
-		luamethod(LuaCamera, get_look_horizontal),
-		luamethod(LuaCamera, get_aspect_ratio),
-
-		{0, 0}};
+const luaL_Reg LuaCamera::methods[] = {
+	luamethod(LuaCamera, set_camera_mode),
+	luamethod(LuaCamera, get_camera_mode),
+	luamethod(LuaCamera, get_fov),
+	luamethod(LuaCamera, get_pos),
+	luamethod(LuaCamera, get_offset),
+	luamethod(LuaCamera, get_look_dir),
+	luamethod(LuaCamera, get_look_vertical),
+	luamethod(LuaCamera, get_look_horizontal),
+	luamethod(LuaCamera, get_aspect_ratio),
+
+	{0, 0}
+};
+// clang-format on
diff --git a/src/script/lua_api/l_env.cpp b/src/script/lua_api/l_env.cpp
index cabca124d..b8a8a5ce1 100644
--- a/src/script/lua_api/l_env.cpp
+++ b/src/script/lua_api/l_env.cpp
@@ -780,8 +780,8 @@ int ModApiEnvMod::l_find_node_near(lua_State *L)
 
 #ifndef SERVER
 	// Client API limitations
-	if (getClient(L))
-		radius = getClient(L)->CSMClampRadius(pos, radius);
+	if (Client *client = getClient(L))
+		radius = client->CSMClampRadius(pos, radius);
 #endif
 
 	for (int d = start_radius; d <= radius; d++) {
@@ -811,9 +811,9 @@ int ModApiEnvMod::l_find_nodes_in_area(lua_State *L)
 	const NodeDefManager *ndef = env->getGameDef()->ndef();
 
 #ifndef SERVER
-	if (getClient(L)) {
-		minp = getClient(L)->CSMClampPos(minp);
-		maxp = getClient(L)->CSMClampPos(maxp);
+	if (Client *client = getClient(L)) {
+		minp = client->CSMClampPos(minp);
+		maxp = client->CSMClampPos(maxp);
 	}
 #endif
 
@@ -887,9 +887,9 @@ int ModApiEnvMod::l_find_nodes_in_area_under_air(lua_State *L)
 	const NodeDefManager *ndef = env->getGameDef()->ndef();
 
 #ifndef SERVER
-	if (getClient(L)) {
-		minp = getClient(L)->CSMClampPos(minp);
-		maxp = getClient(L)->CSMClampPos(maxp);
+	if (Client *client = getClient(L)) {
+		minp = client->CSMClampPos(minp);
+		maxp = client->CSMClampPos(maxp);
 	}
 #endif