Formspecs: Introduce formspec_version to mods

This commit is contained in:
SmallJoker 2019-09-10 21:18:42 +02:00
parent b0baa698a4
commit 9acd36bf99
11 changed files with 54 additions and 10 deletions

@ -13,6 +13,7 @@ core.features = {
object_use_texture_alpha = true, object_use_texture_alpha = true,
object_independent_selectionbox = true, object_independent_selectionbox = true,
httpfetch_binary_data = true, httpfetch_binary_data = true,
formspec_version_element = true,
} }
function core.has_feature(arg) function core.has_feature(arg)

@ -1959,6 +1959,15 @@ Examples
Elements Elements
-------- --------
### `formspec_version[<version>]`
* Set the formspec version to a certain number. If not specified,
version 1 is assumed.
* Must be specified before `size` element.
* Clients older than this version can neither show newer elements nor display
elements with new arguments correctly.
* Available since feature `formspec_version_element`.
### `size[<W>,<H>,<fixed_size>]` ### `size[<W>,<H>,<fixed_size>]`
* Define the size of the menu in inventory slots * Define the size of the menu in inventory slots
@ -1995,6 +2004,7 @@ Elements
### `real_coordinates[<bool>]` ### `real_coordinates[<bool>]`
* INFORMATION: Enable it automatically using `formspec_version` version 2 or newer.
* When set to true, all following formspec elements will use the new coordinate system. * When set to true, all following formspec elements will use the new coordinate system.
* If used immediately after `size`, `position`, `anchor`, and `no_prepend` elements * If used immediately after `size`, `position`, `anchor`, and `no_prepend` elements
(if present), the form size will use the new coordinate system. (if present), the form size will use the new coordinate system.
@ -2114,6 +2124,7 @@ Elements
image shall be sized 8 times 16px times 4 times 16px image shall be sized 8 times 16px times 4 times 16px
* If `auto_clip` is `true`, the background is clipped to the formspec size * If `auto_clip` is `true`, the background is clipped to the formspec size
(`x` and `y` are used as offset values, `w` and `h` are ignored) (`x` and `y` are used as offset values, `w` and `h` are ignored)
* Available since formspec version 2
### `pwdfield[<X>,<Y>;<W>,<H>;<name>;<label>]` ### `pwdfield[<X>,<Y>;<W>,<H>;<name>;<label>]`
@ -3788,6 +3799,8 @@ Utilities
-- Specifies whether binary data can be uploaded or downloaded using -- Specifies whether binary data can be uploaded or downloaded using
-- the HTTP API (5.1.0) -- the HTTP API (5.1.0)
httpfetch_binary_data = true, httpfetch_binary_data = true,
-- Whether formspec_version[<version>] may be used (5.1.0)
formspec_version_element = true,
} }
* `minetest.has_feature(arg)`: returns `boolean, missing_features` * `minetest.has_feature(arg)`: returns `boolean, missing_features`
@ -3807,6 +3820,7 @@ Utilities
avg_jitter = 0.03, -- average packet time jitter avg_jitter = 0.03, -- average packet time jitter
connection_uptime = 200, -- seconds since client connected connection_uptime = 200, -- seconds since client connected
protocol_version = 32, -- protocol version used by client protocol_version = 32, -- protocol version used by client
formspec_version = 2, -- supported formspec version
-- following information is available on debug build only!!! -- following information is available on debug build only!!!
-- DO NOT USE IN MODS -- DO NOT USE IN MODS
--ser_vers = 26, -- serialization version used by client --ser_vers = 26, -- serialization version used by client

@ -1222,12 +1222,13 @@ void Client::sendRespawn()
void Client::sendReady() void Client::sendReady()
{ {
NetworkPacket pkt(TOSERVER_CLIENT_READY, NetworkPacket pkt(TOSERVER_CLIENT_READY,
1 + 1 + 1 + 1 + 2 + sizeof(char) * strlen(g_version_hash)); 1 + 1 + 1 + 1 + 2 + sizeof(char) * strlen(g_version_hash) + 2);
pkt << (u8) VERSION_MAJOR << (u8) VERSION_MINOR << (u8) VERSION_PATCH pkt << (u8) VERSION_MAJOR << (u8) VERSION_MINOR << (u8) VERSION_PATCH
<< (u8) 0 << (u16) strlen(g_version_hash); << (u8) 0 << (u16) strlen(g_version_hash);
pkt.putRawString(g_version_hash, (u16) strlen(g_version_hash)); pkt.putRawString(g_version_hash, (u16) strlen(g_version_hash));
pkt << (u16)FORMSPEC_API_VERSION;
Send(&pkt); Send(&pkt);
} }

@ -2167,6 +2167,9 @@ void GUIFormSpecMenu::parseElement(parserData* data, const std::string &element)
if (element.empty()) if (element.empty())
return; return;
if (parseVersionDirect(element))
return;
std::vector<std::string> parts = split(element,'['); std::vector<std::string> parts = split(element,'[');
// ugly workaround to keep compatibility // ugly workaround to keep compatibility
@ -2503,7 +2506,7 @@ void GUIFormSpecMenu::regenerateGui(v2u32 screensize)
} }
/* Copy of the "real_coordinates" element for after the form size. */ /* Copy of the "real_coordinates" element for after the form size. */
mydata.real_coordinates = false; mydata.real_coordinates = m_formspec_version >= 2;
for (; i < elements.size(); i++) { for (; i < elements.size(); i++) {
std::vector<std::string> parts = split(elements[i], '['); std::vector<std::string> parts = split(elements[i], '[');
std::string name = trim(parts[0]); std::string name = trim(parts[0]);
@ -2648,10 +2651,14 @@ void GUIFormSpecMenu::regenerateGui(v2u32 screensize)
if (enable_prepends) { if (enable_prepends) {
// Backup the coordinates so that prepends can use the coordinates of choice. // Backup the coordinates so that prepends can use the coordinates of choice.
bool rc_backup = mydata.real_coordinates; bool rc_backup = mydata.real_coordinates;
bool version_backup = m_formspec_version;
mydata.real_coordinates = false; // Old coordinates by default. mydata.real_coordinates = false; // Old coordinates by default.
std::vector<std::string> prepend_elements = split(m_formspec_prepend, ']'); std::vector<std::string> prepend_elements = split(m_formspec_prepend, ']');
for (const auto &element : prepend_elements) for (const auto &element : prepend_elements)
parseElement(&mydata, element); parseElement(&mydata, element);
m_formspec_version = version_backup;
mydata.real_coordinates = rc_backup; // Restore coordinates mydata.real_coordinates = rc_backup; // Restore coordinates
} }

@ -472,7 +472,7 @@ protected:
private: private:
IFormSource *m_form_src; IFormSource *m_form_src;
TextDest *m_text_dst; TextDest *m_text_dst;
u32 m_formspec_version = 0; u16 m_formspec_version = 1;
std::string m_focused_element = ""; std::string m_focused_element = "";
JoystickController *m_joystick; JoystickController *m_joystick;
@ -591,7 +591,7 @@ public:
void setForm(const std::string &formspec) void setForm(const std::string &formspec)
{ {
m_formspec = FORMSPEC_VERSION_STRING + formspec; m_formspec = formspec;
} }
const std::string &getForm() const const std::string &getForm() const

@ -198,6 +198,7 @@ with this program; if not, write to the Free Software Foundation, Inc.,
PROTOCOL VERSION 38: PROTOCOL VERSION 38:
Incremental inventory sending mode Incremental inventory sending mode
Unknown inventory serialization fields no longer throw an error Unknown inventory serialization fields no longer throw an error
Mod-specific formspec version
*/ */
#define LATEST_PROTOCOL_VERSION 38 #define LATEST_PROTOCOL_VERSION 38
@ -219,7 +220,16 @@ with this program; if not, write to the Free Software Foundation, Inc.,
#define PASSWORD_SIZE 28 // Maximum password length. Allows for #define PASSWORD_SIZE 28 // Maximum password length. Allows for
// base64-encoded SHA-1 (27+\0). // base64-encoded SHA-1 (27+\0).
#define FORMSPEC_API_VERSION 1 /*
Changes by FORMSPEC_API_VERSION:
FORMSPEC VERSION 1:
(too much)
FORMSPEC VERSION 2:
Forced real coordinates
background[]: 9-slice scaling parameters
*/
#define FORMSPEC_API_VERSION 2
#define FORMSPEC_VERSION_STRING "formspec_version[" TOSTRING(FORMSPEC_API_VERSION) "]" #define FORMSPEC_VERSION_STRING "formspec_version[" TOSTRING(FORMSPEC_API_VERSION) "]"
#define TEXTURENAME_ALLOWED_CHARS "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789_.-" #define TEXTURENAME_ALLOWED_CHARS "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789_.-"

@ -386,6 +386,9 @@ void Server::handleCommand_ClientReady(NetworkPacket* pkt)
peer_id, major_ver, minor_ver, patch_ver, peer_id, major_ver, minor_ver, patch_ver,
full_ver); full_ver);
if (pkt->getRemainingBytes() >= 2)
*pkt >> playersao->getPlayer()->formspec_version;
const std::vector<std::string> &players = m_clients.getPlayerNames(); const std::vector<std::string> &players = m_clients.getPlayerNames();
NetworkPacket list_pkt(TOCLIENT_UPDATE_PLAYER_LIST, 0, peer_id); NetworkPacket list_pkt(TOCLIENT_UPDATE_PLAYER_LIST, 0, peer_id);
list_pkt << (u8) PLAYER_LIST_INIT << (u16) players.size(); list_pkt << (u8) PLAYER_LIST_INIT << (u16) players.size();

@ -130,6 +130,9 @@ public:
u16 protocol_version = 0; u16 protocol_version = 0;
// v1 for clients older than 5.1.0-dev
u16 formspec_version = 1;
session_t getPeerId() const { return m_peer_id; } session_t getPeerId() const { return m_peer_id; }
void setPeerId(session_t peer_id) { m_peer_id = peer_id; } void setPeerId(session_t peer_id) { m_peer_id = peer_id; }

@ -233,6 +233,10 @@ int ModApiServer::l_get_player_information(lua_State *L)
lua_pushnumber(L, prot_vers); lua_pushnumber(L, prot_vers);
lua_settable(L, table); lua_settable(L, table);
lua_pushstring(L, "formspec_version");
lua_pushnumber(L, player->formspec_version);
lua_settable(L, table);
#ifndef NDEBUG #ifndef NDEBUG
lua_pushstring(L,"serialization_version"); lua_pushstring(L,"serialization_version");
lua_pushnumber(L, ser_vers); lua_pushnumber(L, ser_vers);

@ -1568,7 +1568,7 @@ void Server::SendChatMessage(session_t peer_id, const ChatMessage &message)
void Server::SendShowFormspecMessage(session_t peer_id, const std::string &formspec, void Server::SendShowFormspecMessage(session_t peer_id, const std::string &formspec,
const std::string &formname) const std::string &formname)
{ {
NetworkPacket pkt(TOCLIENT_SHOW_FORMSPEC, 0 , peer_id); NetworkPacket pkt(TOCLIENT_SHOW_FORMSPEC, 0, peer_id);
if (formspec.empty()){ if (formspec.empty()){
//the client should close the formspec //the client should close the formspec
//but make sure there wasn't another one open in meantime //but make sure there wasn't another one open in meantime
@ -1579,7 +1579,7 @@ void Server::SendShowFormspecMessage(session_t peer_id, const std::string &forms
pkt.putLongString(""); pkt.putLongString("");
} else { } else {
m_formspec_state_data[peer_id] = formname; m_formspec_state_data[peer_id] = formname;
pkt.putLongString(FORMSPEC_VERSION_STRING + formspec); pkt.putLongString(formspec);
} }
pkt << formname; pkt << formname;
@ -1908,7 +1908,8 @@ void Server::SendPlayerInventoryFormspec(session_t peer_id)
return; return;
NetworkPacket pkt(TOCLIENT_INVENTORY_FORMSPEC, 0, peer_id); NetworkPacket pkt(TOCLIENT_INVENTORY_FORMSPEC, 0, peer_id);
pkt.putLongString(FORMSPEC_VERSION_STRING + player->inventory_formspec); pkt.putLongString(player->inventory_formspec);
Send(&pkt); Send(&pkt);
} }
@ -1920,7 +1921,7 @@ void Server::SendPlayerFormspecPrepend(session_t peer_id)
return; return;
NetworkPacket pkt(TOCLIENT_FORMSPEC_PREPEND, 0, peer_id); NetworkPacket pkt(TOCLIENT_FORMSPEC_PREPEND, 0, peer_id);
pkt << FORMSPEC_VERSION_STRING + player->formspec_prepend; pkt << player->formspec_prepend;
Send(&pkt); Send(&pkt);
} }