flux reports: /status and /privs should be s ... #2469

New Issue

yourland-report · 2022-08-17T00:03:55Z

yourland-report commented

2022-08-17 00:03:55 +00:00

flux reports a bug:

/status and /privs should be sorted

Player position:

{
	z = 572.19903564453,
	x = 1310.5920410156,
	y = 62.5
}

Player look:

{
	z = -0.72398978471756,
	x = 0.25339555740356,
	y = -0.64158350229263
}

Player information:

{
	min_jitter = 0,
	max_jitter = 0.77100002765656,
	avg_jitter = 0.0029999911785126,
	connection_uptime = 6279,
	serialization_version = 29,
	patch = 0,
	state = "Active",
	lang_code = "",
	protocol_version = 40,
	formspec_version = 6,
	major = 5,
	minor = 6,
	version_string = "5.6.0",
	ip_version = 6,
	min_rtt = 0.15999999642372,
	max_rtt = 1.4349999427795,
	avg_rtt = 0.17000000178814
}

Player meta:

{
	fields = {
		digged_nodes = "1090598",
		placed_nodes = "353296",
		died = "68",
		crafted = "39554",
		yl_commons_thankyou = "145",
		xp = "1281858",
		hud_state = "on",
		jointime = "1615351173",
		bitten = "0",
		["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}",
		["petz:lycanthropy"] = "1",
		["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}",
		yl_commons_player_created = "1615351173",
		inflicted_damage = "1539542",
		["petz:werewolf_vignette_id"] = "20",
		["petz:werewolf"] = "0",
		["signslib:pos"] = "(1273,69,540)",
		["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 700\", \"3d_armor:leggings_nether 1 1400\", \"3d_armor:helmet_nether 1 700\", \"nether_mobs:dragon_shield 1 700\", \"3d_armor:chestplate_nether 1 1400\", \"\"}",
		repellant = "0",
		["petz:werewolf_clan_idx"] = "1",
		arenalib_infobox_arenaID = "0",
		["stamina:poisoned"] = "no",
		["stamina:level"] = "4",
		["stamina:exhaustion"] = "136.5",
		punch_count = "92148",
		yl_church = "return {[\"last_death_portal\"] = 1660095612, [\"last_death\"] = {[\"y\"] = 15, [\"z\"] = 1874, [\"x\"] = 2417}, [\"last_heal\"] = 1615504208}",
		partychat = "party",
		yl_commons_player_joined = "1660688374",
		played_time = "21843334"
	}
}

Log identifier


[MOD] yl_report log identifier = WwhejCVVPODdCjEZiIkNMcWqMyrgEw4j

Profiler save:

profile-20220817T000355.json_prettyEE

Status:

# Server: version: 5.5.1-yl | game: Minetest Game | uptime: 1h 52min 8s | max lag: 2.3s | clients: Bailiff, Service, Lupercus, perilforeverfire, mattattack, Zizis, pitman, Cody3, LeetPeet, AliasAlreadyTaken, flux, Kiyoko, JeCel, Wushkin, daydream, the_chosen_one, demaro, Hulda

Teleport command:

/teleport xyz 1311 63 572

Compass command:

/give_compass Construction WwhejCVVPODdCjEZiIkNMcWqMyrgEw4j D2691E 1311 63 572

flux reports a bug: > /status and /privs should be sorted Player position: ``` { z = 572.19903564453, x = 1310.5920410156, y = 62.5 } ``` Player look: ``` { z = -0.72398978471756, x = 0.25339555740356, y = -0.64158350229263 } ``` Player information: ``` { min_jitter = 0, max_jitter = 0.77100002765656, avg_jitter = 0.0029999911785126, connection_uptime = 6279, serialization_version = 29, patch = 0, state = "Active", lang_code = "", protocol_version = 40, formspec_version = 6, major = 5, minor = 6, version_string = "5.6.0", ip_version = 6, min_rtt = 0.15999999642372, max_rtt = 1.4349999427795, avg_rtt = 0.17000000178814 } ``` Player meta: ``` { fields = { digged_nodes = "1090598", placed_nodes = "353296", died = "68", crafted = "39554", yl_commons_thankyou = "145", xp = "1281858", hud_state = "on", jointime = "1615351173", bitten = "0", ["unified_inventory:bags"] = "return {\"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\", \"water_life:croc_bag\"}", ["petz:lycanthropy"] = "1", ["petz:old_override_table"] = "return {[\"new_move\"] = true, [\"sneak\"] = true, [\"sneak_glitch\"] = false, [\"jump\"] = 1.5, [\"speed\"] = 2, [\"gravity\"] = 1}", yl_commons_player_created = "1615351173", inflicted_damage = "1539542", ["petz:werewolf_vignette_id"] = "20", ["petz:werewolf"] = "0", ["signslib:pos"] = "(1273,69,540)", ["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 700\", \"3d_armor:leggings_nether 1 1400\", \"3d_armor:helmet_nether 1 700\", \"nether_mobs:dragon_shield 1 700\", \"3d_armor:chestplate_nether 1 1400\", \"\"}", repellant = "0", ["petz:werewolf_clan_idx"] = "1", arenalib_infobox_arenaID = "0", ["stamina:poisoned"] = "no", ["stamina:level"] = "4", ["stamina:exhaustion"] = "136.5", punch_count = "92148", yl_church = "return {[\"last_death_portal\"] = 1660095612, [\"last_death\"] = {[\"y\"] = 15, [\"z\"] = 1874, [\"x\"] = 2417}, [\"last_heal\"] = 1615504208}", partychat = "party", yl_commons_player_joined = "1660688374", played_time = "21843334" } } ``` Log identifier ``` [MOD] yl_report log identifier = WwhejCVVPODdCjEZiIkNMcWqMyrgEw4j ``` Profiler save: ``` profile-20220817T000355.json_prettyEE ``` Status: ``` # Server: version: 5.5.1-yl | game: Minetest Game | uptime: 1h 52min 8s | max lag: 2.3s | clients: Bailiff, Service, Lupercus, perilforeverfire, mattattack, Zizis, pitman, Cody3, LeetPeet, AliasAlreadyTaken, flux, Kiyoko, JeCel, Wushkin, daydream, the_chosen_one, demaro, Hulda ``` Teleport command: ``` /teleport xyz 1311 63 572 ``` Compass command: ``` /give_compass Construction WwhejCVVPODdCjEZiIkNMcWqMyrgEw4j D2691E 1311 63 572 ```

AliasAlreadyTaken was assigned by yourland-report

2022-08-17 00:03:55 +00:00

AliasAlreadyTaken added the

1. kind/enhancement

3. source/engine

labels 2022-08-17 05:46:21 +00:00

flux commented

2022-08-17 15:43:16 +00:00

this is easy to do via an override

AliasAlreadyTaken commented

2022-08-17 17:04:51 +00:00

Yes, and upstream might also benefit.

Maybe we should also "redesign" our commands. I'm not happy with the /help command leading not to some help, but to the commands. But that's another issue: your-land/administration#162

Yes, and upstream might also benefit. Maybe we should also "redesign" our commands. I'm not happy with the /help command leading not to some help, but to the commands. But that's another issue: https://gitea.your-land.de/your-land/administration/issues/162

flux added the

4. step/ready to QA test

label 2022-08-31 23:13:19 +00:00

flux commented

2022-09-01 00:30:51 +00:00

implemented in 1a686b62e4

upstream would benefit, but

the contents of /status are generated in the c++ code by default, and i don't want to deal w/ that
i can do a lot more fun things w/ /privs that my be controversial upstream, e.g. highlighting privs that are non-default or admin-default:

implemented in https://gitea.your-land.de/your-land/yl_commons/commit/1a686b62e43ab45eeaa18610e7f0b1ba36c04897 upstream would benefit, but 1. the contents of `/status` are generated in the c++ code by default, and i don't want to deal w/ that 2. i can do a lot more fun things w/ `/privs` that my be controversial upstream, e.g. highlighting privs that are non-default or admin-default: ![](https://gitea.your-land.de/attachments/bbdfc08a-6f41-4fbb-8f45-df01886794c4)

Screenshot from 2022-08-31 17-30-27.png

2.5 MiB

flux commented

2022-10-07 21:25:03 +00:00

this became https://github.com/fluxionary/minetest-builtin_overrides

flux added

5. result/fixed

and removed

4. step/ready to QA test

labels 2022-11-13 22:16:29 +00:00

AliasAlreadyTaken was unassigned by flux

2022-11-13 22:16:44 +00:00