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.

Increase size of sv_teamnagger message, also use bold

Implements https://gitlab.com/xonotic/xonotic-data.pk3dir/-/issues/2702

join queue: fix removal of AFK players

Was broken in 433e811eb85f24adf86b8a81679a5c9a6c273337

Improves documentation.

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

Don't end warmup when teams are unbalanced (WRT sv_teamnagger)

Allows spectators to see the unbalanced teams warning, because they
could fix it by joining the smaller team.

Aborts countdown if teams become unbalanced, for the case of someone
joining this can be avoided by enabling g_balance_teams_queue.

Updates the chatcon notification for countdown abort when the player
count drops below the minimum.

Fix and extend networking of sv_teamnagger

It wasn't enabled reliably because the message could be received before
CSQC was ready for it.
Adds support for specifying a team size difference, mainly for use with
g_balance_teams_queue.

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

Allow g_balance_teams_prevent_imbalance without g_balance_teams

I disagree with the comment because g_balance_teams is a misnomer: all
it does is change whether the team selection UI appears when pressing
+jump. Selecting a team manually can improve balance when players know
each other's skills, and the fact that's in use doesn't automatically
mean that people should be allowed to stack mid-match.

Added a condition to only enforce during the match or countdown, because
it's easier to reorganise teams during warmup when players can directly
switch to their team, instead of one of them needing to spec before the
other can swap.

Merge branch 'bones_was_here/ticrate' into 'master'

Support only perfect ticrates: 32hz 64hz 128hz, raise default to 64hz

Closes #2860

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

Turrets: always run at server ticrate

Monsters: always run at server ticrate

Support only perfect ticrates: 32hz 64hz 128hz, raise default to 64hz

At these ticrates there's no floating point error accumulation.

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

Merge branch 'Mario/client_command_registry' into 'master'

Migrate client commands to a registry

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

Migrates client->server commands (such as `say` and `clientversion`) to a registry, matching the other command handlers.

Also reword some client commands to be less clunky and more consistent

Merge branch 'LegendaryGuard/optimize_ok_player_models' into 'master'

Optimize Overkill player models

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

Optimize Overkill player models

Transifex autosync

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

Make jetpacks respect g_powerups and add settings to disable them individually

Merge branch 'Mario/instagib_explicit_powerups' into 'master'

Fix Jetpacks and other undefined powerup items spawning in InstaGib

Closes #2916

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

Fix Jetpacks and other undefined powerup items spawning in InstaGib

Fix cmd flood control blocking cmds after only a few cmds at startup

Merge branch 'Mario/join_floodcontrol_fix' into 'master'

Add a mutator hook to handle flood control on client commands

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

Add a mutator hook to handle flood control on client commands

Merge branch 'Mario/warpzone_div0_fix' into 'master'

Prevent division by zero when warpzones teleport a stationary player

Closes #2924

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

Merge branch 'Mario/maplist_improvements' into 'master'

Refactor map list code to use buffers

Closes #2648

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

Refactor map list code to use buffers

Merge branch 'Mario/player_death_size_fix' into 'master'

Scale player hitbox size on death to fix potential crashes

Closes #2920

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