Refactor code to remove WEP_FLAG_SPECIALATTACK checks if WEP_FLAG_HIDDEN is already checked.

This refactoring is now possible thanks to the changes of !1344, where all WEP_FLAG_SPECIALATTACK checks have been paired with WEP_FLAG_HIDDEN checks

To make sure WEP_FLAG_HIDDEN is always set on special attacks, weapon initialization code triggers a fatal error if WEP_FLAG_SPECIALATTACK is set but WEP_FLAG_HIDDEN is not.

Remove useless vaporizer fields that I forgot to remove in !1282

It amends be66a429a

Merge branch 'drjaska/survival-sendflags' into 'master'

Optimise sending survival statuses

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

Optimise sending survival statuses

Merge branch 'Mario/ignore_cleanup' into 'master'

Move ignore functions out of the header to avoid potential inclusion errors

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

Move ignore functions out of the header to avoid potential inclusion errors

Fix #2890 "Several Effect settings are grayed out in current Xonotic git / autobuild"

Fixed by applying the same compatibility workaround used for settings in Settings - Video window

Merge branch 'bones_was_here/new_toys' into 'master'

Classify arc and shockwave as "new toys", allow "new toys" to spawn if placed by the mapper

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

Allow weapons classified as "new toys" to spawn if placed by the mapper

See https://xonotic.org/balancevotes/24/

Changes the pipeline hash because bots may now use these weapons due to
`g_weaponarena all` in serverbench.cfg.

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

Classify arc and shockwave as "new toys"

These are very unfinished, some of the other "new toys" weapons are far
more polished. As with the others, they can be allowed to spawn as
placed with g_new_toys 1; g_new_toys_autoreplace 0.

Fixes shockwave being listed in nt_IsNewToy() but not having
WEP_FLAG_MUTATORBLOCKED.

Fixes item spawning and g_weaponarena all and all_available not
respecting WEP_FLAG_HIDDEN (g_warmup_allguns 1 did respect it).

Changes the pipeline hash because apparently bots have been using a
WEP_FLAG_HIDDEN weapon due to g_weaponarena all in serverbench.cfg.

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

Merge branch 'Spike29/ca_ft_teamspawnpoints' into 'master'

Use team spawn points (if any) by default in Clan Arena and Freeze Tag

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

Use team spawn points (if any) by default in Clan Arena, applies vote 23 of the Balance Council.

Use team spawn points (if any) by default in Freeze Tag, applies vote 22 of the Balance Council.

Merge branch 'k9er/fix-2907' into 'master'

Remove queued cts-related kills when crossing finish line multiple times

Closes #2907

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

Remove queued cts-related kills when crossing finish line multiple times

Transifex autosync

Fix #2843 "Backwards compatibility strings for scoreboard cvars no longer needed"

Merge branch 'Mario/spiderbot_no_sprint' into 'master'

Remove sprint option from the Spiderbot vehicle

Closes #2403

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

Merge branch 'Mario/observepoints' into 'master'

Make observers view from predefined locations if available

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

Make observers view from predefined locations if available

Merge branch 'k9er/strafehud-changes' into 'master'

Strafehud fixes and additions related to W-turning, side-strafing, and angle accuracy

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

Strafehud fixes and additions related to W-turning, side-strafing, and angle accuracy

Merge branch 'des/quickmenu_keep' into 'master'

Implement KEEP_OPEN flag for quickmenu, and make holding CTRL coherent with it

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

Implement KEEP_OPEN flag for quickmenu, and make holding CTRL coherent with it

Merge branch 'Mario/infinite_ammo_start' into 'master'

Don't force all ammo values to 999 when spawning with infinite ammo

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

Don't force all ammo values to 999 when spawning with infinite ammo

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

Remove sprint option from the Spiderbot vehicle

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