Room system now uses CharacterData instead of Entity

server/rooms/makefile

@@ -1,5 +1,5 @@
 #config
-INCLUDES+=. ../entities ../monsters ../server_utilities ../waypoints ../../common/map ../../common/utilities
+INCLUDES+=. ../characters ../entities ../monsters ../server_utilities ../waypoints ../../common/gameplay ../../common/map ../../common/utilities
 LIBS+=
 CXXFLAGS+=-std=c++11 $(addprefix -I,$(INCLUDES))
 

server/rooms/room_data.cpp

@@ -49,6 +49,6 @@ WaypointManager* RoomData::GetWaypointMgr() {
 	return &waypointMgr;
 }
 
-std::list<Entity*>* RoomData::GetEntityList() {
+std::list<CharacterData*>* RoomData::GetCharacterList() {
-	return &entityList;
+	return &characterList;
 }

server/rooms/room_data.hpp

@@ -22,7 +22,7 @@
 #ifndef ROOMDATA_HPP_
 #define ROOMDATA_HPP_
 
-#include "entity.hpp"
+#include "character_data.hpp"
 #include "monster_manager.hpp"
 #include "region_pager_lua.hpp"
 #include "waypoint_manager.hpp"
@@ -47,7 +47,7 @@ public:
 	RegionPagerLua* GetPager();
 	MonsterManager* GetMonsterMgr();
 	WaypointManager* GetWaypointMgr();
-	std::list<Entity*>* GetEntityList();
+	std::list<CharacterData*>* GetCharacterList();
 
 	//TODO: triggers for unload, save, per-second, player enter, player exit, etc.
 
@@ -61,7 +61,7 @@ private:
 	RegionPagerLua pager;
 	MonsterManager monsterMgr;
 	WaypointManager waypointMgr;
-	std::list<Entity*> entityList;
+	std::list<CharacterData*> characterList;
 };
 
 #endif

server/rooms/room_manager.cpp

@@ -60,34 +60,34 @@ void RoomManager::UnloadIf(std::function<bool(std::pair<const int, RoomData cons
 	}
 }
 
-void RoomManager::PushEntity(Entity* entity) {
+void RoomManager::PushCharacter(CharacterData* character) {
-	if (!entity) {
+	if (!character) {
-		throw(std::runtime_error("Failed to push a null entity to a room"));
+		throw(std::runtime_error("Failed to push a null character to a room"));
 	}
 
-	RoomData* room = Get(entity->GetRoomIndex());
+	RoomData* room = Get(character->GetRoomIndex());
 
 	if (!room) {
-		throw(std::runtime_error("Failed to push an entity to a non-existant room"));
+		throw(std::runtime_error("Failed to push an character to a non-existant room"));
 	}
 
-	room->entityList.push_back(entity);
+	room->characterList.push_back(character);
 }
 
-void RoomManager::PopEntity(Entity const* entity) {
+void RoomManager::PopCharacter(CharacterData const* character) {
-	//NOTE: to pop an entity from a room, the entity must first exist
+	//NOTE: to pop an character from a room, the character must first exist
-	if (!entity) {
+	if (!character) {
-		throw(std::runtime_error("Failed to pop a null entity to a room"));
+		throw(std::runtime_error("Failed to pop a null character to a room"));
 	}
 
-	RoomData* room = Get(entity->GetRoomIndex());
+	RoomData* room = Get(character->GetRoomIndex());
 
 	if (!room) {
-		throw(std::runtime_error("Failed to pop an entity to a non-existant room"));
+		throw(std::runtime_error("Failed to pop an character to a non-existant room"));
 	}
 
-	room->entityList.remove_if([entity](Entity* ptr) {
+	room->characterList.remove_if([character](CharacterData* ptr) {
-		return entity == ptr;
+		return character == ptr;
 	});
 }
 

server/rooms/room_manager.hpp

@@ -22,7 +22,7 @@
 #ifndef ROOMMANAGER_HPP_
 #define ROOMMANAGER_HPP_
 
-#include "entity.hpp"
+#include "character_data.hpp"
 #include "room_data.hpp"
 #include "singleton.hpp"
 
@@ -40,8 +40,8 @@ public:
 	void UnloadAll();
 	void UnloadIf(std::function<bool(std::pair<const int, RoomData const&>)> fn);
 
-	void PushEntity(Entity* entity);
+	void PushCharacter(CharacterData* character);
-	void PopEntity(Entity const* entity);
+	void PopCharacter(CharacterData const* character);
 
 	//accessors and mutators
 	RoomData* Get(int uid);

server/server_character_methods.cpp

@@ -46,7 +46,7 @@ void ServerApplication::HandleCharacterCreate(CharacterPacket* const argPacket)
 	}
 
 	//push to the rooms
-	roomMgr.PushEntity(characterMgr.Get(characterIndex));
+	roomMgr.PushCharacter(characterMgr.Get(characterIndex));
 
 	//pump this character to all clients
 	CharacterPacket newPacket;
@@ -90,7 +90,7 @@ void ServerApplication::HandleCharacterDelete(CharacterPacket* const argPacket)
 	}
 
 	//pop from the rooms
-	roomMgr.PopEntity(characterMgr.Get(characterIndex));
+	roomMgr.PopCharacter(characterMgr.Get(characterIndex));
 
 	//delete the character
 	characterMgr.Delete(characterIndex);
@@ -126,7 +126,7 @@ void ServerApplication::HandleCharacterLoad(CharacterPacket* const argPacket) {
 	}
 
 	//push to the rooms
-	roomMgr.PushEntity(characterMgr.Get(characterIndex));
+	roomMgr.PushCharacter(characterMgr.Get(characterIndex));
 
 	//pump this character to all clients
 	CharacterPacket newPacket;
@@ -159,7 +159,7 @@ void ServerApplication::HandleCharacterUnload(CharacterPacket* const argPacket)
 	}
 
 	//pop from the rooms
-	roomMgr.PopEntity(characterData);
+	roomMgr.PopCharacter(characterData);
 
 	//unload the character
 	characterMgr.Unload(argPacket->characterIndex);
@@ -203,7 +203,7 @@ void ServerApplication::HandleCharacterSetRoom(CharacterPacket* const argPacket)
 	}
 
 	//pop from the old room
-	roomMgr.PopEntity(characterData);
+	roomMgr.PopCharacter(characterData);
 
 	//set the character's room, zero it's origin, zero it's motion
 	characterData->SetRoomIndex(argPacket->roomIndex);
@@ -211,7 +211,7 @@ void ServerApplication::HandleCharacterSetRoom(CharacterPacket* const argPacket)
 	characterData->SetMotion({0, 0});
 
 	//push to the new room
-	roomMgr.PushEntity(characterData);
+	roomMgr.PushCharacter(characterData);
 
 	//update the clients
 	CharacterPacket newPacket;

server/server_methods.cpp

@@ -150,7 +150,7 @@ void ServerApplication::FullCharacterUnload(int index) {
 		}
 
 		//pop from the rooms
-		roomMgr.PopEntity(&character.second);
+		roomMgr.PopCharacter(&character.second);
 
 		//pump character unload
 		CharacterPacket newPacket;