flux reports: this might already be a reques ... #5569

New Issue

yourland-report · 2023-11-29T06:11:59Z

yourland-report commented

2023-11-29 06:11:59 +00:00

flux reports a bug:

this might already be a request, but we should enable "ender chest" mode on mailboxes, so that players can receive mail on any of their mailboxes - and not have to check a specific box

Player position:

{
	z = 1094.0300292969,
	x = 1981.33203125,
	y = 12.875
}

Player look:

{
	z = 0.23668603599072,
	x = 0.96579569578171,
	y = 0.10591697692871
}

Player information:

{
	avg_jitter = 0,
	connection_uptime = 42318,
	serialization_version = 29,
	patch = 0,
	minor = 8,
	lang_code = "",
	state = "Active",
	version_string = "5.8.0-dev-841a1a788",
	major = 5,
	min_jitter = 0,
	avg_rtt = 0.1870000064373,
	min_rtt = 0.17200000584126,
	protocol_version = 42,
	ip_version = 6,
	formspec_version = 7,
	max_rtt = 13.152000427246,
	max_jitter = 11.335000038147
}

Player meta:

{
	fields = {
		punch_count = "318819",
		played_time = "43174814",
		yl_church = "return {[\"last_death\"] = {[\"y\"] = 52, [\"z\"] = 214, [\"x\"] = 2810}, [\"last_death_portal\"] = 1692114993, [\"last_heal\"] = 1615504208}",
		digged_nodes = "1716636",
		placed_nodes = "481338",
		died = "101",
		["petz:lycanthropy"] = "1",
		["3d_armor_inventory"] = "return {\"3d_armor:helmet_rainbow 1 9320\", \"3d_armor:leggings_rainbow 1 9320\", \"shields:shield_rainbow 1 9320\", \"3d_armor:chestplate_rainbow 1 9320\", \"nether_mobs:dragon_boots 1 23240\", \"\"}",
		["petz:werewolf_vignette_id"] = "20",
		hud_state = "on",
		crafted = "210472",
		arenalib_infobox_arenaID = "0",
		repellant = "0",
		["signslib:pos"] = "(1290,65,550)",
		inflicted_damage = "6303432",
		hotbar_size = "16",
		["petz:werewolf"] = "1",
		yl_commons_thankyou = "349",
		jointime = "1615351173",
		bitten = "0",
		yl_commons_player_joined = "1701196018",
		["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}",
		xp = "2313238",
		["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}",
		["petz:pre_werewolf_animation"] = "return {[\"textures\"] = {\"player.flux.0.png\", \"blank.png\", \"3d_armor_trans.png^3d_armor_helmet_rainbow.png^3d_armor_leggings_rainbow.png^shields_shield_rainbow.png^3d_armor_chestplate_rainbow.png^nether_mobs_dragon_boots.png\", \"3d_armor_trans.png\"}, [\"animation\"] = \"stand\", [\"animation_speed\"] = 30, [\"model\"] = \"skinsdb_3d_armor_character_5.b3d\"}",
		["stamina:level"] = "6",
		["petz:werewolf_clan_idx"] = "1",
		["stamina:poisoned"] = "no",
		partychat = "party",
		["stamina:exhaustion"] = "52.5",
		yl_commons_player_created = "1615351173"
	}
}

Log identifier


[MOD] yl_report log identifier = xDVafsZdFwCotEUq7rBsEsBizeD0VURy

Profiler save:

profile-20231129T061159.json_prettyEE

Status:

# Server: version: 5.7.0-yl-thx-tmm | game: Minetest Game | uptime: 2d 19min 53s | max lag: 0.22s | clients (14/52): AliasAlreadyTaken, Bailiff, bancha, daydream, dodi, flux, jackofthebean000, JohannesPaulHoffman, Penelopee, rewired_X1, rheo, Service, taonza12, the_chosen_one

Teleport command:

/teleport xyz 1981 13 1094

Compass command:

/give_compass Construction xDVafsZdFwCotEUq7rBsEsBizeD0VURy D2691E 1981 13 1094

flux reports a bug: > this might already be a request, but we should enable "ender chest" mode on mailboxes, so that players can receive mail on any of their mailboxes - and not have to check a specific box Player position: ``` { z = 1094.0300292969, x = 1981.33203125, y = 12.875 } ``` Player look: ``` { z = 0.23668603599072, x = 0.96579569578171, y = 0.10591697692871 } ``` Player information: ``` { avg_jitter = 0, connection_uptime = 42318, serialization_version = 29, patch = 0, minor = 8, lang_code = "", state = "Active", version_string = "5.8.0-dev-841a1a788", major = 5, min_jitter = 0, avg_rtt = 0.1870000064373, min_rtt = 0.17200000584126, protocol_version = 42, ip_version = 6, formspec_version = 7, max_rtt = 13.152000427246, max_jitter = 11.335000038147 } ``` Player meta: ``` { fields = { punch_count = "318819", played_time = "43174814", yl_church = "return {[\"last_death\"] = {[\"y\"] = 52, [\"z\"] = 214, [\"x\"] = 2810}, [\"last_death_portal\"] = 1692114993, [\"last_heal\"] = 1615504208}", digged_nodes = "1716636", placed_nodes = "481338", died = "101", ["petz:lycanthropy"] = "1", ["3d_armor_inventory"] = "return {\"3d_armor:helmet_rainbow 1 9320\", \"3d_armor:leggings_rainbow 1 9320\", \"shields:shield_rainbow 1 9320\", \"3d_armor:chestplate_rainbow 1 9320\", \"nether_mobs:dragon_boots 1 23240\", \"\"}", ["petz:werewolf_vignette_id"] = "20", hud_state = "on", crafted = "210472", arenalib_infobox_arenaID = "0", repellant = "0", ["signslib:pos"] = "(1290,65,550)", inflicted_damage = "6303432", hotbar_size = "16", ["petz:werewolf"] = "1", yl_commons_thankyou = "349", jointime = "1615351173", bitten = "0", yl_commons_player_joined = "1701196018", ["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}", xp = "2313238", ["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}", ["petz:pre_werewolf_animation"] = "return {[\"textures\"] = {\"player.flux.0.png\", \"blank.png\", \"3d_armor_trans.png^3d_armor_helmet_rainbow.png^3d_armor_leggings_rainbow.png^shields_shield_rainbow.png^3d_armor_chestplate_rainbow.png^nether_mobs_dragon_boots.png\", \"3d_armor_trans.png\"}, [\"animation\"] = \"stand\", [\"animation_speed\"] = 30, [\"model\"] = \"skinsdb_3d_armor_character_5.b3d\"}", ["stamina:level"] = "6", ["petz:werewolf_clan_idx"] = "1", ["stamina:poisoned"] = "no", partychat = "party", ["stamina:exhaustion"] = "52.5", yl_commons_player_created = "1615351173" } } ``` Log identifier ``` [MOD] yl_report log identifier = xDVafsZdFwCotEUq7rBsEsBizeD0VURy ``` Profiler save: ``` profile-20231129T061159.json_prettyEE ``` Status: ``` # Server: version: 5.7.0-yl-thx-tmm | game: Minetest Game | uptime: 2d 19min 53s | max lag: 0.22s | clients (14/52): AliasAlreadyTaken, Bailiff, bancha, daydream, dodi, flux, jackofthebean000, JohannesPaulHoffman, Penelopee, rewired_X1, rheo, Service, taonza12, the_chosen_one ``` Teleport command: ``` /teleport xyz 1981 13 1094 ``` Compass command: ``` /give_compass Construction xDVafsZdFwCotEUq7rBsEsBizeD0VURy D2691E 1981 13 1094 ```

AliasAlreadyTaken was assigned by yourland-report

2023-11-29 06:11:59 +00:00

AliasAlreadyTaken added the

1. kind/balancing

label 2023-11-29 11:52:33 +00:00

AliasAlreadyTaken commented

2023-11-29 11:55:34 +00:00

Does such a mode already exist or would we have to make it?
That would require migration, to have all mailbox contents gathered in one inv?
What are the technical implications?
Does that not require a detached inventory, stored in the inventory database instead of the mapblocks?
People might abuse their own mailbox as ender chest storage

1. Does such a mode already exist or would we have to make it? 2. That would require migration, to have all mailbox contents gathered in one inv? 3. What are the technical implications? 4. Does that not require a detached inventory, stored in the inventory database instead of the mapblocks? 5. People might abuse their own mailbox as ender chest storage

AliasAlreadyTaken added the

4. step/question

label 2023-11-29 11:55:39 +00:00

flux commented

2023-11-30 23:12:39 +00:00

Does such a mode already exist or would we have to make it?

it exists, but not as part of our mailbox fork. i'd have to patch it in.

That would require migration, to have all mailbox contents gathered in one inv?

no, each mailbox can still have its own inventory, but you can toggle a switch to make it instead interact w/ a per-player detached inventory. a player would have to manually go to all their mailboxes and toggle the switch to enable that mode, and they can make sure to take anything out of the mailbox at that time.

What are the technical implications?

Does that not require a detached inventory, stored in the inventory database instead of the mapblocks?

the detached inventories would be stored in mod storage (not a player inventory - currently that'd require the player to be logged in). the storage and performance implications are negligible.

People might abuse their own mailbox as ender chest storage

perhaps, but they'd have to use an alt to put things into the mailbox, making it impractical to use it like that while out in the field. if you think that's still a problem, i could add a timer where the mailbox has to be at least an hour old before you can enable ender-chest mode.

> 1. Does such a mode already exist or would we have to make it? [it exists](https://github.com/BlockySurvival/mailbox), but not as part of our mailbox fork. i'd have to patch it in. > 2. That would require migration, to have all mailbox contents gathered in one inv? no, each mailbox can still have its own inventory, but you can toggle a switch to make it instead interact w/ a per-player detached inventory. a player would have to manually go to all their mailboxes and toggle the switch to enable that mode, and they can make sure to take anything out of the mailbox at that time. > 3. What are the technical implications? > 4. Does that not require a detached inventory, stored in the inventory database instead of the mapblocks? the detached inventories would be stored in mod storage (not a player inventory - currently that'd require the player to be logged in). the storage and performance implications are negligible. > 5. People might abuse their own mailbox as ender chest storage perhaps, but they'd have to use an alt to put things *into* the mailbox, making it impractical to use it like that while out in the field. if you think that's still a problem, i could add a timer where the mailbox has to be at least an hour old before you can enable ender-chest mode.

flux added this to the flux's TODO list project 2024-02-04 18:51:03 +00:00

flux added the

4. step/want approval

label 2024-02-04 18:51:11 +00:00

flux referenced this issue

2024-02-14 02:15:37 +00:00

Ravise reports: Text Custom labels for mailbox ... #6284

flux commented

2024-02-14 02:19:44 +00:00

People might abuse their own mailbox as ender chest storage

i just realized this could be abused a bit - if someone pre-loads their mailbox w/ a bunch of stuff using an alt, they'll have an extra inventory they could take stuff out of when they're out in the field. but they won't be able to put anything back into it. given that players can already set up death portals and use alts, which make for a far more convenient exploit, i don't think this is likely to be meaningfully abused.

> 5. People might abuse their own mailbox as ender chest storage i just realized this could be abused a bit - if someone pre-loads their mailbox w/ a bunch of stuff using an alt, they'll have an extra inventory they could take stuff out of when they're out in the field. but they won't be able to put anything back into it. given that players can already set up death portals and use alts, which make for a far more convenient exploit, i don't think this is likely to be meaningfully abused.

Sign in to join this conversation.