flux reports: /clearinv should only destroy ... #4092

New Issue

yourland-report · 2023-03-31T02:12:53Z

yourland-report commented

2023-03-31 02:12:53 +00:00

flux reports a bug:

/clearinv should only destroy the contents of the player's main inventory, not the crafting grid (or any other inventories it erases)

Player position:

{
	z = 513.08703613281,
	x = 1284.1140136719,
	y = 77.5
}

Player look:

{
	z = -0.012089962139726,
	x = 0.87672734260559,
	y = 0.48083570599556
}

Player information:

{
	lang_code = "",
	protocol_version = 41,
	min_rtt = 0.16400000452995,
	avg_rtt = 0.17299999296665,
	min_jitter = 0,
	max_jitter = 4.1669998168945,
	formspec_version = 6,
	connection_uptime = 98442,
	serialization_version = 29,
	patch = 0,
	major = 5,
	version_string = "5.7.0-dev-491ee1c08-dirty",
	minor = 7,
	max_rtt = 4.3410000801086,
	state = "Active",
	avg_jitter = 0.0020000040531158,
	ip_version = 6
}

Player meta:

{
	fields = {
		punch_count = "208189",
		inflicted_damage = "3969954",
		yl_commons_thankyou = "229",
		["petz:werewolf_vignette_id"] = "19",
		["signslib:pos"] = "(1268,49,608)",
		arenalib_infobox_arenaID = "0",
		["stamina:exhaustion"] = "46.5",
		jointime = "1615351173",
		["petz:werewolf"] = "0",
		partychat = "party",
		["petz:lycanthropy"] = "1",
		yl_church = "return {[\"last_heal\"] = 1615504208, [\"last_death\"] = {[\"y\"] = 47, [\"z\"] = 429, [\"x\"] = 2733}, [\"last_death_portal\"] = 1678914871}",
		["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}",
		bitten = "0",
		played_time = "33183188",
		repellant = "0",
		["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 8460\", \"3d_armor:leggings_rainbow 1 3604\", \"shields:shield_rainbow 1 3604\", \"3d_armor:chestplate_rainbow 1 3604\", \"3d_armor:helmet_rainbow 1 3604\", \"\"}",
		died = "86",
		digged_nodes = "1469842",
		crafted = "112527",
		["petz:werewolf_clan_idx"] = "1",
		["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}",
		yl_commons_player_joined = "1680130362",
		["stamina:poisoned"] = "no",
		placed_nodes = "427981",
		["stamina:level"] = "19",
		hud_state = "on",
		yl_commons_player_created = "1615351173",
		xp = "1857024"
	}
}

Log identifier


[MOD] yl_report log identifier = 4cy3y5WrczFwhuscwBlzQMPs7Ugg6y1T

Profiler save:

profile-20230331T021253.json_prettyEE

Status:

# Server: version: 5.6.1-yl | game: Minetest Game | uptime: 1d 4h 11min 25s | max lag: 0.576s | clients (20/52): AKN4, AliasAlreadyTaken, Aliza, answer2, Bailiff, daydream, drgn, Ernesto, flux, HorusDamocles, Lupercus, mahou, niceride, noface, rheo, Service, shanish3, sixer, Sokomine, STEALTH

Teleport command:

/teleport xyz 1284 78 513

Compass command:

/give_compass Construction 4cy3y5WrczFwhuscwBlzQMPs7Ugg6y1T D2691E 1284 78 513

flux reports a bug: > /clearinv should only destroy the contents of the player's main inventory, not the crafting grid (or any other inventories it erases) Player position: ``` { z = 513.08703613281, x = 1284.1140136719, y = 77.5 } ``` Player look: ``` { z = -0.012089962139726, x = 0.87672734260559, y = 0.48083570599556 } ``` Player information: ``` { lang_code = "", protocol_version = 41, min_rtt = 0.16400000452995, avg_rtt = 0.17299999296665, min_jitter = 0, max_jitter = 4.1669998168945, formspec_version = 6, connection_uptime = 98442, serialization_version = 29, patch = 0, major = 5, version_string = "5.7.0-dev-491ee1c08-dirty", minor = 7, max_rtt = 4.3410000801086, state = "Active", avg_jitter = 0.0020000040531158, ip_version = 6 } ``` Player meta: ``` { fields = { punch_count = "208189", inflicted_damage = "3969954", yl_commons_thankyou = "229", ["petz:werewolf_vignette_id"] = "19", ["signslib:pos"] = "(1268,49,608)", arenalib_infobox_arenaID = "0", ["stamina:exhaustion"] = "46.5", jointime = "1615351173", ["petz:werewolf"] = "0", partychat = "party", ["petz:lycanthropy"] = "1", yl_church = "return {[\"last_heal\"] = 1615504208, [\"last_death\"] = {[\"y\"] = 47, [\"z\"] = 429, [\"x\"] = 2733}, [\"last_death_portal\"] = 1678914871}", ["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}", bitten = "0", played_time = "33183188", repellant = "0", ["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 8460\", \"3d_armor:leggings_rainbow 1 3604\", \"shields:shield_rainbow 1 3604\", \"3d_armor:chestplate_rainbow 1 3604\", \"3d_armor:helmet_rainbow 1 3604\", \"\"}", died = "86", digged_nodes = "1469842", crafted = "112527", ["petz:werewolf_clan_idx"] = "1", ["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}", yl_commons_player_joined = "1680130362", ["stamina:poisoned"] = "no", placed_nodes = "427981", ["stamina:level"] = "19", hud_state = "on", yl_commons_player_created = "1615351173", xp = "1857024" } } ``` Log identifier ``` [MOD] yl_report log identifier = 4cy3y5WrczFwhuscwBlzQMPs7Ugg6y1T ``` Profiler save: ``` profile-20230331T021253.json_prettyEE ``` Status: ``` # Server: version: 5.6.1-yl | game: Minetest Game | uptime: 1d 4h 11min 25s | max lag: 0.576s | clients (20/52): AKN4, AliasAlreadyTaken, Aliza, answer2, Bailiff, daydream, drgn, Ernesto, flux, HorusDamocles, Lupercus, mahou, niceride, noface, rheo, Service, shanish3, sixer, Sokomine, STEALTH ``` Teleport command: ``` /teleport xyz 1284 78 513 ``` Compass command: ``` /give_compass Construction 4cy3y5WrczFwhuscwBlzQMPs7Ugg6y1T D2691E 1284 78 513 ```

AliasAlreadyTaken was assigned by yourland-report

2023-03-31 02:12:53 +00:00

AliasAlreadyTaken added the

1. kind/enhancement

label 2023-03-31 06:34:25 +00:00

AliasAlreadyTaken commented

2023-03-31 08:56:37 +00:00

We should rename the command to "/clearinv_dangerous"

We could have it have parameters which inventory to clear, with default to main, but accepting a list truncated by a space?

We should add logging which items got cleared.

We should rename the command to "/clearinv_dangerous" We could have it have parameters which inventory to clear, with default to main, but accepting a list truncated by a space? We should add logging which items got cleared.

👍 1

flux referenced this issue

2023-04-01 02:17:15 +00:00

flux reports: clearinv shouldn't be somethin ... #4093

daydream commented

2023-04-02 00:52:38 +00:00

Silly to give this command to non-creative - will just cause you more work. But dumping on ground is a GREAT alternative!

flux added this to the flux's TODO list project 2023-04-02 15:19:43 +00:00

Sign in to join this conversation.