[GameState|Key] Fixed registry access for singleplayer games.

Author: Unarelith

source/client/states/GameState.cpp

@@ -121,7 +121,7 @@ void GameState::onEvent(const sf::Event &event) {
 			gk::Mouse::setCursorVisible(false);
 		}
 		else if (event.type == sf::Event::KeyPressed) {
-			for (auto &key : m_registry.keys()) {
+			for (auto &key : Registry::getInstance().keys()) {
 				if (event.key.code == key.keycode()) {
 					m_clientCommandHandler.sendKeyPressed(key.id());
 				}
@@ -152,7 +152,7 @@ void GameState::update() {
 	}
 
 	if (!m_areModKeysLoaded) {
-		for (auto &it : m_registry.keys()) {
+		for (auto &it : Registry::getInstance().keys()) {
 			m_keyboardHandler->addKey(it.id(), it.name(), it.keycode(), it.stringID(), &it);
 		}
 

source/common/world/Key.hpp

@@ -45,14 +45,8 @@ class Key : public ISerializable {
 		Key(u16 id, const std::string &stringID, const std::string &name)
 			: m_id(id), m_stringID(stringID), m_name(name) {}
 
-		void serialize(sf::Packet &packet) const override {
-			packet << m_id << m_stringID << m_name << m_defaultKey;
-		}
-
-		void deserialize(sf::Packet &packet) override {
-			packet >> m_id >> m_stringID >> m_name >> m_defaultKey;
-			m_keycode = gk::KeyboardUtils::getKeyFromName(m_defaultKey);
-		}
+		void serialize(sf::Packet &packet) const override { packet << m_id << m_stringID << m_name << m_defaultKey; }
+		void deserialize(sf::Packet &packet) override { packet >> m_id >> m_stringID >> m_name >> m_defaultKey; }
 
 		u16 id() const { return m_id; }
 
@@ -64,7 +58,12 @@ class Key : public ISerializable {
 		void setKeycode(sf::Keyboard::Key keycode) { m_keycode = keycode; if (m_parent) m_parent->m_keycode = keycode; }
 
 		const std::string &defaultKey() const { return m_defaultKey; }
-		void setDefaultKey(const std::string &defaultKey) { m_defaultKey = defaultKey; }
+		void setDefaultKey(const std::string &defaultKey) {
+			m_defaultKey = defaultKey;
+
+			if (m_keycode == sf::Keyboard::Unknown)
+				m_keycode = gk::KeyboardUtils::getKeyFromName(m_defaultKey);
+		}
 
 		const sol::unsafe_function &callback() const { return m_callback; }
 		void setCallback(const sol::unsafe_function &callback) { m_callback = callback; }