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.

join: display translated notifications for relevant failures

teamplay: display translated notifications for typical failures or state changes

Fixes some uses of NOTIF_ONE where it should have been NOTIF_ONE_ONLY.

join queue: UI improvements and refactoring

Fixes duplicate and redundant chatcon notifications.

Displays relevant centreprint each time the team selection GUI or +jump
are used to try to join (even if the player is already queued) for
better noob friendliness.

Always notifies the player when their team selection conflicts with that
of a queued player.

Notifies queued players to get their attention when they join as it may
take some time before another player triggers the join.

Includes the team the player actually got assigned to in the relevant
centreprint (for the cases where they had no preference, or someone else
chose their preferred team first).

Prevents SetPlayerTeam() and queuePlayer() when the player definitely
can't play (version mismatch, locked teams), fixing misleading
notifications.
Allows players to join the queue even when g_maxplayers blocks them from
joining the match, so they can tag in if someone leaves.

Fixes bug with > 2 teams where if a player with a lower entity number
queues for any available team, a player with a higher entity number
and a specific team preference could be assigned to the same team (by
adding a second pass in Join() ).

Fixes lack of "now playing" notification when a bot joins a gametype
without teams.

This required some refactoring:

Calls joinAllowed() from ClientCommand_selectteam(), passing the
team_index through to queuePlayer().  This means calling queuePlayer
from SetPlayerTeam() is no longer needed.
Uses a different logic order in joinAllowed() when the queue is enabled.

Moves the ForbidSpawn MUTATOR hook from joinAllowed() to
PutPlayerInServer(). It seemed to be in the wrong place anyway: the
player is only blocked from spawning, not from joining the match.

No longer sets the .team field when adding a player to the queue
(they're not actually on the team yet), instead calls SetPlayerTeam()
from Join() when actually spawning the queued player.

Uses the .team_selected field to pass selection conflicts to Join(), it
can't be done with a parameter because `join` and `selectteam` are
separate commands.

Moves the queue conflict detection from Player_SetTeamIndexChecked() to
queuePlayer().

Reduces TeamBalance_JoinBestTeam() callsites.

Fixes some uses of NOTIF_ONE where it should have been NOTIF_ONE_ONLY.

join queue: fix 2 bugs that could result in too many players on a team

Join(): when a client queued for red, then a blue player left the team,
then a client (re)joined blue, the client queued for red was joined too
(stacking red team). Fixed by only joining queued players in Join() when
teams are currently balanced (the unbalanced cases are handled by
TeamBalance_QueuedPlayersTagIn()).

ClientKill_Now_TeamChange(): when all queued client(s) chose specific
team(s) and the last client (which skips the queue and triggers the
joins) chose autoselect from the team selection UI, it was possible they
were assigned the same team as one of the queued clients.  Fixed by
deferring the autoselect to Join() when needed/possible.

join queue: fix HUD infomessage, improve consistency and clarity of notifications

The state of having no team preference (deferred autoselect) was
implemented in 624a242e556387367d088dc2cde847989c0fd321 but it wasn't
correctly declared and networked and displayed in the HUD info panel (it
said you're queued to join the Neutral team).

join queue: disable in singleplayer, improve description

Fixes https://gitlab.com/xonotic/xonotic-data.pk3dir/-/issues/2905

Enable sv_teamnagger in CA, remove HideTeamNagger mutator hook

sv_teamnagger seems to work fine in CA and is already disabled in
gametypes without teams.