[InventoryState] Replaced by 'show_inventory' function in init.lua.

client/include/gui/PlayerInventoryWidget.hpp

@@ -18,7 +18,7 @@
 
 class PlayerInventoryWidget : public Widget {
 	public:
-		PlayerInventoryWidget(ClientCommandHandler &client, Inventory &playerInventory, Widget *parent = nullptr);
+		PlayerInventoryWidget(ClientCommandHandler &client, MouseItemWidget &mouseItemWidget, Inventory &playerInventory, Widget *parent = nullptr);
 
 		void onEvent(const SDL_Event &event) override;
 
@@ -29,6 +29,8 @@ class PlayerInventoryWidget : public Widget {
 
 		ClientCommandHandler &m_client;
 
+		MouseItemWidget &m_mouseItemWidget;
+
 		gk::Image m_background;
 
 		Inventory m_craftingInventory{2, 2};
@@ -37,8 +39,6 @@ class PlayerInventoryWidget : public Widget {
 		Inventory &m_playerInventory;
 		InventoryWidget m_playerInventoryWidget{m_client, this};
 		InventoryWidget m_hotbarInventoryWidget{m_client, this};
-
-		MouseItemWidget m_mouseItemWidget{this};
 };
 
 #endif // PLAYERINVENTORYWIDGET_HPP_

client/include/network/ClientCommandHandler.hpp

@@ -35,6 +35,7 @@ class ClientCommandHandler {
 		void sendPlayerPosUpdate();
 		void sendPlayerDigBlock(const glm::vec4 &selectedBlock);
 		void sendPlayerPlaceBlock(s32 x, s32 y, s32 z, u32 block);
+		void sendPlayerInventoryRequest();
 		void sendBlockActivated(const glm::vec4 &selectedBlock);
 		void sendBlockInvUpdate(Inventory &inventory);
 		void sendChunkRequest(s32 chunkX, s32 chunkY, s32 chunkZ);

client/source/gui/PlayerInventoryWidget.cpp

@@ -13,8 +13,8 @@
  */
 #include "PlayerInventoryWidget.hpp"
 
-PlayerInventoryWidget::PlayerInventoryWidget(ClientCommandHandler &client, Inventory &playerInventory, Widget *parent)
-	: Widget(176, 166, parent), m_client(client), m_playerInventory(playerInventory)
+PlayerInventoryWidget::PlayerInventoryWidget(ClientCommandHandler &client, MouseItemWidget &mouseItemWidget, Inventory &playerInventory, Widget *parent)
+	: Widget(176, 166, parent), m_client(client), m_mouseItemWidget(mouseItemWidget), m_playerInventory(playerInventory)
 {
 	m_background.load("texture-inventory");
 	m_background.setClipRect(0, 0, 176, 166);
@@ -26,6 +26,7 @@ PlayerInventoryWidget::PlayerInventoryWidget(ClientCommandHandler &client, Inven
 	m_hotbarInventoryWidget.setPosition(7, 141, 0);
 
 	m_craftingWidget.craftingInventoryWidget().setPosition(97, 17, 0);
+	m_craftingWidget.craftingInventoryWidget().init(m_craftingInventory, 0, 4);
 	m_craftingWidget.craftingResultInventoryWidget().setPosition(153, 27, 0);
 }
 
@@ -34,8 +35,6 @@ void PlayerInventoryWidget::onEvent(const SDL_Event &event) {
 
 	m_playerInventoryWidget.onMouseEvent(event, m_mouseItemWidget);
 	m_hotbarInventoryWidget.onMouseEvent(event, m_mouseItemWidget);
-
-	m_mouseItemWidget.onEvent(event);
 }
 
 void PlayerInventoryWidget::update() {
@@ -59,7 +58,5 @@ void PlayerInventoryWidget::draw(gk::RenderTarget &target, gk::RenderStates stat
 
 	target.draw(m_playerInventoryWidget, states);
 	target.draw(m_hotbarInventoryWidget, states);
-
-	target.draw(m_mouseItemWidget, states);
 }
 

client/source/network/ClientCommandHandler.cpp

@@ -56,6 +56,12 @@ void ClientCommandHandler::sendPlayerPlaceBlock(s32 x, s32 y, s32 z, u32 block)
 	m_client.send(packet);
 }
 
+void ClientCommandHandler::sendPlayerInventoryRequest() {
+	sf::Packet packet;
+	packet << Network::Command::PlayerInventory;
+	m_client.send(packet);
+}
+
 void ClientCommandHandler::sendBlockActivated(const glm::vec4 &selectedBlock) {
 	sf::Packet packet;
 	packet << Network::Command::BlockActivated

client/source/states/GameState.cpp

@@ -25,7 +25,6 @@
 
 #include "GameKey.hpp"
 #include "GameState.hpp"
-#include "InventoryState.hpp"
 #include "LuaGUIState.hpp"
 #include "PauseMenuState.hpp"
 #include "PlayerInventoryWidget.hpp"
@@ -108,8 +107,7 @@ void GameState::update() {
 				m_player.processInputs();
 
 				if (gk::GamePad::isKeyPressedOnce(GameKey::Inventory)) {
-					auto &inventoryState = m_stateStack->push<InventoryState>(this);
-					inventoryState.setupWidget<PlayerInventoryWidget>(m_clientCommandHandler, m_player.inventory());
+					m_clientCommandHandler.sendPlayerInventoryRequest();
 				}
 			}
 

client/source/states/LuaGUIState.cpp

@@ -24,8 +24,10 @@
 #include "FurnaceWidget.hpp"
 #include "InventoryWidget.hpp"
 #include "LuaGUIState.hpp"
+#include "LuaWidget.hpp"
 #include "Network.hpp"
 #include "Player.hpp"
+#include "PlayerInventoryWidget.hpp"
 #include "TextButton.hpp"
 
 LuaGUIState::LuaGUIState(ClientCommandHandler &client, ClientPlayer &player, ClientWorld &world, sf::Packet &packet, gk::ApplicationState *parent)
@@ -85,7 +87,8 @@ void LuaGUIState::update() {
 			currentItemWidget = it.currentItemWidget();
 	}
 
-	m_mouseItemWidget.update(currentItemWidget);
+	if (m_widgets.size() != 1) // FIXME
+		m_mouseItemWidget.update(currentItemWidget);
 }
 
 void LuaGUIState::draw(gk::RenderTarget &target, gk::RenderStates states) const {
@@ -117,7 +120,7 @@ void LuaGUIState::loadGUI(ClientPlayer &player, ClientWorld &world, sf::Packet &
 	std::string name;
 	float x, y;
 	packet >> type >> name >> x >> y;
-	if (type == 0) {
+	if (type == LuaWidget::Image) {
 		std::string texture;
 		gk::FloatRect clipRect;
 		packet >> texture >> clipRect.x >> clipRect.y >> clipRect.width >> clipRect.height;
@@ -127,7 +130,7 @@ void LuaGUIState::loadGUI(ClientPlayer &player, ClientWorld &world, sf::Packet &
 		image->setClipRect(clipRect.x, clipRect.y, clipRect.width, clipRect.height);
 		m_drawables.emplace_back(image);
 	}
-	else if (type == 1) {
+	else if (type == LuaWidget::TextButton) {
 		std::string text;
 		packet >> text;
 
@@ -137,7 +140,7 @@ void LuaGUIState::loadGUI(ClientPlayer &player, ClientWorld &world, sf::Packet &
 		button->setText(text);
 		m_widgets.emplace_back(button);
 	}
-	else if (type == 2) {
+	else if (type == LuaWidget::InventoryWidget) {
 		std::string playerName, inventory;
 		float width, height;
 		u16 offset, count;
@@ -149,7 +152,7 @@ void LuaGUIState::loadGUI(ClientPlayer &player, ClientWorld &world, sf::Packet &
 		inventoryWidget.setPosition(x, y);
 		inventoryWidget.init(player.inventory(), offset, count);
 	}
-	else if (type == 3) {
+	else if (type == LuaWidget::CraftingWidget) {
 		gk::Vector3i block;
 		u16 offset, size;
 		packet >> block.x >> block.y >> block.z >> offset >> size;
@@ -165,7 +168,7 @@ void LuaGUIState::loadGUI(ClientPlayer &player, ClientWorld &world, sf::Packet &
 			DEBUG("ERROR: No inventory found at", block.x, block.y, block.z);
 		}
 	}
-	else if (type == 4) {
+	else if (type == LuaWidget::FurnaceWidget) {
 		gk::Vector3i block;
 		packet >> block.x >> block.y >> block.z;
 		BlockData *data = world.getBlockData(block.x, block.y, block.z);
@@ -178,5 +181,10 @@ void LuaGUIState::loadGUI(ClientPlayer &player, ClientWorld &world, sf::Packet &
 			DEBUG("ERROR: No inventory found at", block.x, block.y, block.z);
 		}
 	}
+	else if (type == LuaWidget::PlayerInventoryWidget) {
+		auto *widget = new PlayerInventoryWidget(m_client, m_mouseItemWidget, player.inventory(), &m_mainWidget);
+		widget->setPosition(x, y);
+		m_widgets.emplace_back(widget);
+	}
 }
 

common/include/core/LuaWidget.hpp

@@ -0,0 +1,30 @@
+/*
+ * =====================================================================================
+ *
+ *       Filename:  LuaWidget.hpp
+ *
+ *    Description:
+ *
+ *        Created:  08/02/2020 02:32:09
+ *
+ *         Author:  Quentin Bazin, <[email protected]>
+ *
+ * =====================================================================================
+ */
+#ifndef LUAWIDGET_HPP_
+#define LUAWIDGET_HPP_
+
+#include <gk/core/IntTypes.hpp>
+
+namespace LuaWidget {
+	enum : u8 {
+		Image           = 0,
+		TextButton      = 1,
+		InventoryWidget = 2,
+		CraftingWidget  = 3,
+		FurnaceWidget   = 4,
+		PlayerInventoryWidget = 5,
+	};
+}
+
+#endif // LUAWIDGET_HPP_

common/include/network/Network.hpp

@@ -37,6 +37,7 @@ namespace Network {
 		PlayerInvUpdate,   // <TCP> [NetworkCommand][u16 client id][[std::string item][u16 amount][u8 x, y]...] (both) [FIXME]
 		PlayerPosUpdate,   // <TCP> [NetworkCommand][u16 client id][s32 x, y, z] (both) // FIXME
 		PlayerSpawn,       // <TCP> [NetworkCommand][u16 client id][s32 x, y, z]            (from Server only)
+		PlayerInventory,   // <TCP> [NetworkCommand]                         (from Client only)
 
 		// Block commands
 		BlockUpdate,       // <TCP> [NetworkCommand][s32 x, y, z][u32 block] (from Server only)

common/source/network/Network.cpp

@@ -34,6 +34,7 @@ std::string Network::commandToString(Network::Command command) {
 		{Network::Command::PlayerInvUpdate,  "PlayerInvUpdate"},
 		{Network::Command::PlayerPosUpdate,  "PlayerPosUpdate"},
 		{Network::Command::PlayerSpawn,      "PlayerSpawn"},
+		{Network::Command::PlayerInventory,  "PlayerInventory"},
 
 		{Network::Command::BlockUpdate,      "BlockUpdate"},
 		{Network::Command::BlockActivated,   "BlockActivated"},

mods/default/init.lua

@@ -46,3 +46,20 @@ function init(player)
 	player_inv:add_stack("default:coal", 64);
 end
 
+function show_inventory(client)
+	local gui = LuaGUI.new()
+
+	-- FIXME: Replace this by gui:set_size() and gui:set_centered()
+	local gui_pos = {
+		x = SCREEN_WIDTH / GUI_SCALE / 2.0 - 176 / 2.0,
+		y = SCREEN_HEIGHT / GUI_SCALE / 2.0 - 166 / 2.0
+	}
+
+	gui:player_inventory {
+		name = "inventory",
+		pos = gui_pos,
+	}
+
+	gui:show(client)
+end
+

server/include/lua/LuaGUI.hpp

@@ -20,19 +20,12 @@
 #include "ServerInfo.hpp"
 
 struct LuaGUIData {
-	enum {
-		Image,
-		TextButton,
-		InventoryWidget,
-		CraftingWidget,
-		FurnaceWidget,
-	};
-
 	std::list<LuaWidgetDef::Image> imageList;
 	std::list<LuaWidgetDef::TextButton> textButtonList;
 	std::list<LuaWidgetDef::InventoryWidget> inventoryWidgetList;
 	std::list<LuaWidgetDef::CraftingWidget> craftingWidgetList;
 	std::list<LuaWidgetDef::FurnaceWidget> furnaceWidgetList;
+	std::list<LuaWidgetDef::Widget> playerInventoryWidgetList;
 };
 
 // This class is meant to be used ONLY in Lua
@@ -43,6 +36,7 @@ class LuaGUI {
 		void addInventoryWidget(const sol::table &table);
 		void addCraftingWidget(const sol::table &table);
 		void addFurnaceWidget(const sol::table &table);
+		void addPlayerInventoryWidget(const sol::table &table);
 
 		void show(Client &client);