Sokomine reports: c h really needs to be short ... #4167

Open
opened 2023-04-11 23:39:44 +00:00 by yourland-report · 4 comments

Sokomine reports a bug:

c h really needs to be shorter. it scrolls far too much and doesn't present the required information in a really helpful way....

Player position:

{
	y = 3.5,
	z = -6187.1962890625,
	x = -4662.5849609375
}

Player look:

{
	y = -0.046234738081694,
	z = 0.33377888798714,
	x = -0.94151681661606
}

Player information:

{
	formspec_version = 6,
	max_rtt = 10.960000038147,
	major = 5,
	version_string = "5.8.0-dev-4a742be73-dirty",
	minor = 8,
	state = "Active",
	ip_version = 6,
	min_rtt = 0.01799999922514,
	avg_rtt = 0.018999999389052,
	min_jitter = 0,
	max_jitter = 10.855999946594,
	avg_jitter = 0,
	connection_uptime = 9402,
	serialization_version = 29,
	patch = 0,
	protocol_version = 41,
	lang_code = ""
}

Player meta:

{
	fields = {
		["3d_armor_inventory"] = "return {\"\", \"3d_armor:helmet_crystal 1 40\", \"shields:shield_rainbow 1 16\", \"3d_armor:chestplate_crystal 1 40\", \"3d_armor:boots_crystal 1 40\", \"3d_armor:leggings_crystal 1 40\"}",
		partychat = "main",
		hud_state = "on",
		["ocean_build.last_warning"] = "1.65032e+09",
		["ocean_build.ocean_built"] = "5",
		["stamina:level"] = "15",
		["stamina:poisoned"] = "no",
		repellant = "0",
		["stamina:exhaustion"] = "26",
		yl_commons_player_created = "1617905120",
		yl_commons_player_joined = "1681247006",
		["ethereal:fly_timer"] = "-99",
		xp = "2006208",
		died = "42",
		["signslib:pos"] = "(2247,-164,1417)",
		punch_count = "223452",
		yl_commons_thankyou = "372",
		arenalib_infobox_arenaID = "0",
		played_time = "12847380",
		digged_nodes = "1787757",
		placed_nodes = "303732",
		["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"water_life:croc_bag\"}",
		inflicted_damage = "4636320",
		crafted = "542580",
		yl_church = "return {[\"last_death_portal\"] = 1681162650, [\"last_death\"] = {[\"x\"] = 2221, [\"z\"] = 1645, [\"y\"] = -139}, [\"last_heal\"] = 1673909657}",
		jointime = "1617905120",
		bitten = "0"
	}
}

Log identifier


[MOD] yl_report log identifier = H84bD1cbDg0rTsrKo87A4pln6qPX3UZq

Profiler save:

profile-20230411T233944.json_prettyEE

Status:

# Server: version: 5.6.1-yl | game: Minetest Game | uptime: 2h 46min 57s | max lag: 0.515s | clients (25/52): AliasAlreadyTaken, AspireMint, Bailiff, bubbafrog, Chache, copper248, CorvidCrusader, Dirac, Elias, flux, HorusDamocles, jackofthebean000, JeCel, Lupercus, mahou, MineWorlds, Naan, niceride, rewired_X, Scathach, Service, shanish3, sixer, Sokomine, Therottenpotato

Teleport command:

/teleport xyz -4663 4 -6187

Compass command:

/give_compass Construction H84bD1cbDg0rTsrKo87A4pln6qPX3UZq D2691E -4663 4 -6187
Sokomine reports a bug: > c h really needs to be shorter. it scrolls far too much and doesn't present the required information in a really helpful way.... Player position: ``` { y = 3.5, z = -6187.1962890625, x = -4662.5849609375 } ``` Player look: ``` { y = -0.046234738081694, z = 0.33377888798714, x = -0.94151681661606 } ``` Player information: ``` { formspec_version = 6, max_rtt = 10.960000038147, major = 5, version_string = "5.8.0-dev-4a742be73-dirty", minor = 8, state = "Active", ip_version = 6, min_rtt = 0.01799999922514, avg_rtt = 0.018999999389052, min_jitter = 0, max_jitter = 10.855999946594, avg_jitter = 0, connection_uptime = 9402, serialization_version = 29, patch = 0, protocol_version = 41, lang_code = "" } ``` Player meta: ``` { fields = { ["3d_armor_inventory"] = "return {\"\", \"3d_armor:helmet_crystal 1 40\", \"shields:shield_rainbow 1 16\", \"3d_armor:chestplate_crystal 1 40\", \"3d_armor:boots_crystal 1 40\", \"3d_armor:leggings_crystal 1 40\"}", partychat = "main", hud_state = "on", ["ocean_build.last_warning"] = "1.65032e+09", ["ocean_build.ocean_built"] = "5", ["stamina:level"] = "15", ["stamina:poisoned"] = "no", repellant = "0", ["stamina:exhaustion"] = "26", yl_commons_player_created = "1617905120", yl_commons_player_joined = "1681247006", ["ethereal:fly_timer"] = "-99", xp = "2006208", died = "42", ["signslib:pos"] = "(2247,-164,1417)", punch_count = "223452", yl_commons_thankyou = "372", arenalib_infobox_arenaID = "0", played_time = "12847380", digged_nodes = "1787757", placed_nodes = "303732", ["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"water_life:croc_bag\"}", inflicted_damage = "4636320", crafted = "542580", yl_church = "return {[\"last_death_portal\"] = 1681162650, [\"last_death\"] = {[\"x\"] = 2221, [\"z\"] = 1645, [\"y\"] = -139}, [\"last_heal\"] = 1673909657}", jointime = "1617905120", bitten = "0" } } ``` Log identifier ``` [MOD] yl_report log identifier = H84bD1cbDg0rTsrKo87A4pln6qPX3UZq ``` Profiler save: ``` profile-20230411T233944.json_prettyEE ``` Status: ``` # Server: version: 5.6.1-yl | game: Minetest Game | uptime: 2h 46min 57s | max lag: 0.515s | clients (25/52): AliasAlreadyTaken, AspireMint, Bailiff, bubbafrog, Chache, copper248, CorvidCrusader, Dirac, Elias, flux, HorusDamocles, jackofthebean000, JeCel, Lupercus, mahou, MineWorlds, Naan, niceride, rewired_X, Scathach, Service, shanish3, sixer, Sokomine, Therottenpotato ``` Teleport command: ``` /teleport xyz -4663 4 -6187 ``` Compass command: ``` /give_compass Construction H84bD1cbDg0rTsrKo87A4pln6qPX3UZq D2691E -4663 4 -6187 ```
AliasAlreadyTaken was assigned by yourland-report 2023-04-11 23:39:44 +00:00
AliasAlreadyTaken added the
1. kind/enhancement
1. kind/documentation
labels 2023-04-11 23:49:06 +00:00

What exactly could be shorter and what information is needed?

What exactly could be shorter and what information is needed?
AliasAlreadyTaken added the
4. step/question
label 2023-04-11 23:50:45 +00:00

smartchat command "/c h" text flood

smartchat command "/c h" text flood
Member

Example of a single command (there are about 16!):

Name: join
Description: Join or change a channel to <channel>.
Usage: /c join <channel>
Parameter: <channel>
Shortcut: /c j <channel>
--------------

Wanted:

/c j|join <channel> Join or change a channel to <channel>.

Six lines would thus turn into one without loss of information. The result would be this (still pretty long...):

/c h|help <>|<command> Shows this help or help for the given command of Smart-Chat.
    
/c j|join <channel>    Join or change a channel to <channel>.
/c l|leave             Leaves a channel.
/c li|list <name>      List all player in the channel.
/c i|invite <player>   Invites a <player> to your Channel.
/c k|kick <player>     Kicks a player from the channel.
/c w|where <name>      Shows which channel <name> is on.
/c m|move <player> <channel> Moves a player to <channel> or to self.

/c t|toggle            Turns the permanent public Chat on or off.    
/c tp|talk2public <message>  Sends a message to the public channel.
/c a|all <message>     Sends a message on all.

/c c|channels          Lists all channels on the Server.
/c st|status           Writes Information about the Chat.
/c rc|reconnect        Starts the counter for automatic reconnect.    
/c mc|mark_channel     Marks a channel permanent.
/c uc|unmark_channel   Unmarks a channel as permanent.

Come to think of that, it's still far too long. Sepearate it into two outputs - the above one behind "/c fh|full_help Show all commands." And behind this extended help there ought to be where, move, all, channels, status, reconnect, mark_channel and unmark_channel command. Normal help ought to be limited to join, leave, list, invite, kick, toggle, talk2public, full_help.

Example of a single command (there are about 16!): ``` Name: join Description: Join or change a channel to <channel>. Usage: /c join <channel> Parameter: <channel> Shortcut: /c j <channel> -------------- ``` Wanted: `/c j|join <channel> Join or change a channel to <channel>.` Six lines would thus turn into one without loss of information. The result would be this (still pretty long...): ``` /c h|help <>|<command> Shows this help or help for the given command of Smart-Chat. /c j|join <channel> Join or change a channel to <channel>. /c l|leave Leaves a channel. /c li|list <name> List all player in the channel. /c i|invite <player> Invites a <player> to your Channel. /c k|kick <player> Kicks a player from the channel. /c w|where <name> Shows which channel <name> is on. /c m|move <player> <channel> Moves a player to <channel> or to self. /c t|toggle Turns the permanent public Chat on or off. /c tp|talk2public <message> Sends a message to the public channel. /c a|all <message> Sends a message on all. /c c|channels Lists all channels on the Server. /c st|status Writes Information about the Chat. /c rc|reconnect Starts the counter for automatic reconnect. /c mc|mark_channel Marks a channel permanent. /c uc|unmark_channel Unmarks a channel as permanent. ``` Come to think of that, it's still far too long. Sepearate it into two outputs - the above one behind "/c fh|full_help Show all commands." And behind this extended help there ought to be where, move, all, channels, status, reconnect, mark_channel and unmark_channel command. Normal help ought to be limited to join, leave, list, invite, kick, toggle, talk2public, full_help.
AliasAlreadyTaken added
3. source/mod upstream
and removed
4. step/question
labels 2023-04-12 15:12:16 +00:00
Member

maybe it'd be even better if /c help even put up a formspec like /help does

maybe it'd be even better if `/c help` even put up a formspec like `/help` does
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: your-land/bugtracker#4167
No description provided.