Add minetest.close_formspec() to close formspecs from the server side

[orwell96]

new implementation of #4675
When an empty formspec string is supplied to show_formspec, and the formname matches the one of the formspec currently open on the client, it will close.
when supplying empty formspec and empty formname, it will close the client's formspec regardless of the open form's name.

For more info, see #4675
Has been documented in init.lua

Has been tested, test code:

minetest.register_node("test:disapp", {
	description = "Disappears",
	tiles = {"default_dirt.png"},
	groups = {crumbly = 3, soil = 1},
	on_construct=function(pos)
		minetest.get_meta(pos):set_string("formspec", "size[2,2]")
	end,
})
minetest.register_node("test:set", {
	description = "Sets form empty",
	tiles = {"default_dirt.png"},
	groups = {crumbly = 3, soil = 1},
	on_construct=function(pos)
		minetest.get_meta(pos):set_string("formspec", "size[2,2]")
	end,
})
minetest.register_node("test:form", {
	description = "show_formspec and close",
	tiles = {"default_dirt.png"},
	groups = {crumbly = 3, soil = 1},
	on_construct=function(pos)
		minetest.show_formspec("moflo", "test", "size[2,4]")
	end,
})
minetest.register_node("test:form2", {
	description = "show_formspec and close with different formname",
	tiles = {"default_dirt.png"},
	groups = {crumbly = 3, soil = 1},
	on_construct=function(pos)
		minetest.show_formspec("moflo", "test2", "size[2,4]")
	end,
})
minetest.register_node("test:form3", {
	description = "show_formspec and close wit formname empty",
	tiles = {"default_dirt.png"},
	groups = {crumbly = 3, soil = 1},
	on_construct=function(pos)
		minetest.show_formspec("moflo", "test2", "size[4,2]")
	end,
})
minetest.register_abm({
	nodenames = {"test:disapp"},
	interval = 10, -- Operation interval in seconds
	chance = 1, -- Chance of trigger per-node per-interval is 1.0 / this
	action = function(pos, node, active_object_count, active_object_count_wider) minetest.set_node(pos, {name="air"}) end,
})
minetest.register_abm({
	nodenames = {"test:form"},
	interval = 10, -- Operation interval in seconds
	chance = 1, -- Chance of trigger per-node per-interval is 1.0 / this
	action = function(pos, node, active_object_count, active_object_count_wider) minetest.set_node(pos, {name="air"}) minetest.close_formspec("moflo", "test") end,
})
minetest.register_abm({
	nodenames = {"test:form2"},
	interval = 10, -- Operation interval in seconds
	chance = 1, -- Chance of trigger per-node per-interval is 1.0 / this
	action = function(pos, node, active_object_count, active_object_count_wider) minetest.set_node(pos, {name="air"}) minetest.close_formspec("moflo", "testblablubb") end,
})
minetest.register_abm({
	nodenames = {"test:form3"},
	interval = 10, -- Operation interval in seconds
	chance = 1, -- Chance of trigger per-node per-interval is 1.0 / this
	action = function(pos, node, active_object_count, active_object_count_wider) minetest.set_node(pos, {name="air"}) minetest.close_formspec("moflo", "") end,
})
minetest.register_abm({
	nodenames = {"test:set"},
	interval = 10, -- Operation interval in seconds
	chance = 1, -- Chance of trigger per-node per-interval is 1.0 / this
	action = function(pos, node, active_object_count, active_object_count_wider) minetest.get_meta(pos):set_string("formspec", "") end,
})

[PureTryOut]

Next time put your code between "code tags":

code here 

That will show up as

code here

It will make it easier readable.

[orwell96]

Ah, 3 of them works...
Here you are!

[est31]

👍 looks good

[raymoo]

Does this behave well with the death screen formspec?

[orwell96]

the death screen formspec is not covered by this, only metadata and show_formspec

[Zeno-]

👍