Refactor spec_forfeit to avoid networking spec_forfeit_ask and _menu_forfeit_dialog_available cvars

Simplify spec_forfeit (at the cost of sending another cvar to the server)

Fix spec_forfeit alias

Add a "Never ask again" checkbox to the Forfeit dialog

Fix spectate command help message and improve Forfeit dialog message

Use a new alias spec_ask_forfeit to ask for forfeit confirmation

Add code to automatically update existing spec binds to spec_ask_forfeit, executed by the menu only once

Restore an empty line

Restore BACKSPACE to change spectator camera mode, make right mouse button switch to previous player

Forfeit dialog: implement backward compatibility: no forfeit confirmation for old menus

Also fix wrong command name sent by the server

Change key to change spectator camera mode from Backspace (drop weapon) to Right mouse button (secondary fire)

Right mouse button used to allow switching from spectator to observer but it's now redundant since F3 can do it

Ask player if they want to forfeit on F3 press while playing. Allow F3 to switch from spectator to observer too

The F3 key now consistently works while playing and while spectating another player

Merge branch 'terencehill/spectator_camera_stuff' into 'master'

Spectator camera: allow switching from 3rd to 1st person camera by setting chase_active to 0

Closes #2873

See merge request xonotic/xonotic-data.pk3dir!1355

pipeline: update hash for stormkeep changes, prevent potential flakiness

I have isolated the cause of the hash change to
https://gitlab.com/xonotic/xonotic-maps.pk3dir/-/merge_requests/191
(only change needed to restore the previous hash is using stormkeep from
the previous autobuild 20240906_2).

DP uses nonblocking stdout on Linux by default, in theory this could
cause occasional pipeline failures that can't be reproduced.

Transifex autosync

Spectator camera: move code to a dedicated function

Spectator camera: allow switching from 3rd to 1st person camera by setting chase_active to 0

Transifex autosync

Mostly whitespace weapons cleanup chores

Fix indenting in qcsrc/lib/stats.qh

Transifex autosync

Remove ancient untouched legacy svn files

Transifex autosync

Merge branch 'drjaska/nextmap' into 'master'

Show nextmap on HUD and rewrite nextmap from cvar to cmd

See merge request xonotic/xonotic-data.pk3dir!1341

Show nextmap on HUD and rewrite nextmap from cvar to cmd

Merge branch 'drjaska/shotgun-animation-time' into 'master'

Opt-in cvar to block shotgun melee directly after a shot

See merge request xonotic/xonotic-data.pk3dir!1173

Opt-in cvar to block shotgun melee directly after a shot

Merge branch 'des/print-race-splits-v2' into 'master'

Add optional split times in infomessage hud, and print_cptimes command to display the stored splits

See merge request xonotic/xonotic-data.pk3dir!1330

Add optional split times in infomessage hud, and print_cptimes command to display the stored splits

Merge branch 'terencehill/status_effect_fix' into 'master'

Fix some minor issues

See merge request xonotic/xonotic-data.pk3dir!1228

Fix `give no superweapons` giving default superweapons time

It happened because GiveStatusEffect removed superweapons status effect and this code block added it back.

Simplify buff effect time calculation

Don't allow StatusEffects_gettime to return a time in the past

By not doing so the effect would be still active in the last frame even if end time has passed.
It prevents wrong usage of the returned value, for example in Fire_AddDamage where mintime can be set to a negative value and used to apply a negative damage to the player
Removed now redundant max operations at StatusEffects_gettime call sites

Don't apply status effects ending the moment they are applied. It fixes spawnshield applied for a moment even if g_spawnshieldtime is 0

This change required a refactoring of the give command code, which now no longer needs changing internal fields of the status effects to update them

Merge branch 'des/cts_pbhook' into 'master'

Add cl_hook_race_pb to run on client when they set a new personal best time

See merge request xonotic/xonotic-data.pk3dir!1328

Add cl_hook_race_pb to run on client when they set a new personal best time

Transifex autosync

Merge branch 'Mario/menu_developer_options' into 'master'

Clean up the use of developer in the menu and remove other weird settings

See merge request xonotic/xonotic-data.pk3dir!1323

Clean up the use of developer in the menu and remove other weird settings

Merge branch 'bones_was_here/Q1BSP_items_droptofloor' into 'master'

Q1BSP: improve droptofloor compatibility with Quake progs

See merge request xonotic/xonotic-data.pk3dir!1327

Merge branch 'Mario/resource_item_cleanup' into 'master'

Clean up ammo displaying to not require direct item references

See merge request xonotic/xonotic-data.pk3dir!1326

Clean up ammo displaying to not require direct item references

Refactor write_recordmarker to reduce engine function calls and while at it indent code with TABs

Merge branch 'des/cmd_cleancps' into 'master'

Race/CTS: Add clear_bestcptimes command to clear all your best checkpoint times

See merge request xonotic/xonotic-data.pk3dir!1332

Race/CTS: Add clear_bestcptimes command to clear all your best checkpoint times

Transifex autosync

Merge branch 'des/race_dont_save_pb_on_demoplay' into 'master'

Don't save local record if we are just watching a demo

See merge request xonotic/xonotic-data.pk3dir!1340

Merge branch 'Spike29/campaign_french' into 'master'

Add a French translation to the Xonotic campaign

See merge request xonotic/xonotic-data.pk3dir!1338

Transifex autosync

Transifex autosync

Merge branch 'Mario/jetpack_powerup_settings' into 'master'

Make Jetpacks respect g_powerups and add settings to disable them individually

See merge request xonotic/xonotic-data.pk3dir!1321

Don't save local record if we are just watching a demo.

Merge branch 'terencehill/translatable_strings_cleanup' into 'master'

Clean up some translatable strings

See merge request xonotic/xonotic-data.pk3dir!1333

Merge branch 'drjaska/race_CheckName' into 'master'

Fix race_CheckName by actually using the given argument

Closes #2554 and #2123

See merge request xonotic/xonotic-data.pk3dir!1339

Transifex autosync

Fix race_CheckName by actually using the given arg

// Check if the given name already exists in race rankings?
// In that case, where? (otherwise return 0)
int race_CheckName(string net_name)

Merge branch 'terencehill/blaster_attack_shared' into 'master'

Make weapons using blaster as secondary attack load settings from blaster

See merge request xonotic/xonotic-data.pk3dir!1282

Add a French translation to the Xonotic campaign

Merge branch 'terencehill/antispam_stuff' into 'master'

Make cmd and chat anti spam systems work during timeouts

See merge request xonotic/xonotic-data.pk3dir!1334

Transifex autosync

Shownames: restore an early alpha check that was made exclusive to CTS by mistake in 0aa7cb88f

Transifex autosync

Hud editor: "Title at %s"  -->  "Title"

Transifex autosync

HUD editor: fix Chat, Vote and StrafeHud panels being visible while the settings menu of another panel is open

Transifex autosync

Merge branch 'des/race_honor_tracking' into 'master'

Don't save race times if player doesn't want tracking, even if we have uid2name stored

Closes #2806

See merge request xonotic/xonotic-data.pk3dir!1336

Don't save race times if player doesn't want tracking, even if we have uuid2name stored.

Transifex autosync

Merge branch 'terencehill/ctf_fixes' into 'master'

Misc CTF fixes

See merge request xonotic/xonotic-data.pk3dir!1331

Transifex autosync

Fix electro not leaving decals on all walls

Remove dead code after changes of the previous commit

Remove optional blaster secondary attack (copy of primary attack with slightly different settings)

Weapons using blaster as secondary attack now load settings from blaster primary

Make so that weapons using blaster as secondary attack load settings directly from blaster instead of loading their own blaster settings

It implements Balance Council vote 18 "Keep secondary/offhand blaster default settings (including vaporizer, ok*) the same as vanilla blaster defaults"

Simplify REGISTER_WEAPON, now possible thanks to the new WEP_CVAR* macros

Switch to using the WEP_CVAR* macros I added in the previous commit as they are more versatile

Transifex autosync

Transifex autosync

Fix weapons with blaster secondary attack improperly using damage and force settings of the blaster itself

Reduce code duplication in chat anti spam system

Make cmd and chat anti spam systems work during timeouts and make them work properly at different game speed

Chat anti spam code now works with flood time (.(flood_field)) initialized to 0, matching cmd anti spam code

Remove cruft of an outdated system against cmd name spam (not functional because .nickspamtime is never set).

cmd name is already protected by spam by the sv_clientcommand_antispam system anyway

Also remove unused floodcontrol_voice* fields

Make Domination capture message easier to translate (only the second part of the message is translatable). Also update notification argument descriptions

Replace notification color codes only in custom string arguments.

No message uses them in basic string arguments, except for player names where they shouldn't be replaced.

Also remove a pointless CCR (color code replace) call for the race_diff argument

Remove trailing space and " %s^BG" used for connecting different sentences from translatable messages

Rename a variable in notif_arg_item_wepammo

CTF: fix fire effect not stopped when flag returns to base

This bug was due to flags being improperly set as damageable by contents while standing at their own bases

CTF: Fix flag returning to base on first hit if g_ctf_flag_return_time is 0 and g_ctf_flag_return_damage is enabled

Also default g_ctf_flag_return_time to 0 if negative

CTF: fix waypoint healthbar not shown if g_ctf_flag_return_damage is enabled and g_ctf_flag_health is 0

CTF: g_ctf_flag_health: default to 0 if negative, allow reading it during a match and improve cvar description

Merge branch 'drjaska/serverlist-modified' into 'master'

Made servers modified via cvar changes appear in normal servers

See merge request xonotic/xonotic-data.pk3dir!1233

Made servers modified via cvar changes appear in normal servers

Q1BSP: improve droptofloor compatibility with Quake progs

Signed-off-by: bones_was_here <bones_was_here@xonotic.au>

Merge branch 'terencehill/floodcontrol_refactoring' into 'master'

Refactor flood control code to work with .cmd_floodtime initialized to 0

See merge request xonotic/xonotic-data.pk3dir!1325

HUD: don't draw panels needing player data before it's received

Fixes
"client VM warning: VM_CL_getstati: index(-1) is >=MAX_CL_STATS(256) or <0"
often appearing when connecting, with my hud_panel_ammo config.

It could also have been fixed by checking for (ammoType == 0) as well as
(ammoType == RES_NONE) but a many-cases fix is better than a
special-case fix.

Merge branch 'Mario/show_all_button' into 'master'

Add a menu option to show hidden gametypes

See merge request xonotic/xonotic-data.pk3dir!1322

Add a show all checkbox to the Create menu to list previously hidden gametypes

Also remove now unused mutator and weapon arena list label

Make some recently added messages easier to translate

Merge branch 'bones_was_here/teamplay5' into 'master'

selectteam, join, spec cmds: notifications, server admin features, fixes

Closes #2905 and #2702

See merge request xonotic/xonotic-data.pk3dir!1301

Improve sv_spectate 0 notifications and description, refactor nospectators command

The blockSpectators global is redundant.
See https://gitlab.com/xonotic/xonotic-data.pk3dir/-/merge_requests/1289

Players are now given the grace period and SPECTATE_WARNING when
sv_spectate is disabled during a game, as well as when `nospectators` is
voted.

Centreprints are added and displayed when they don't conflict with join
restriction centreprints.