Remove legacy Quake bbox expansion: physics

Remove legacy Quake bbox expansion: bmodel entities

Removes duplication of the box + '1 1 1' and boxesoverlap code.
This is handled in WarpZoneLib_ExactTrigger_Touch().

Disable sv_legacy_bbox_expand

This gives QC full control of bboxes
and makes SVQC and CSQC behaviour consistent.

Groups all the sv_gameplayfix cvars together in xonotic-server.cfg.

Adds a note about sv_gameplayfix_droptofloorstartsolid (cvar for unused
engine feature).

Rewrite the QC port of droptofloor()

This uses DP_QC_NUDGEOUTOFSOLID to achieve better results (especially
noticeable with sv_legacy_bbox_expand 0) in less CPU time.

Documents the design.
Removes legacy code paths.
Adds warnings for badly placed items.
Implements Quake-compliant behaviour for each BSP format.

See also: https://gitlab.com/xonotic/darkplaces/-/merge_requests/144

Removes hacky workaround for #2774 and thus depends on !1245

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

items: Apply vertical offset on Q3 maps to match Q3 bbox floor height

The lack of this caused high-placed items to be stuck in a "ceiling"
brush in a few cases (problematic for droptofloor). It also meant
suspended items were too high, causing cases where a jump pad sent the
player slightly under the item, with nothing they could do to reach it.

Merge branch 'terencehill/respawn_ghosts_keepcolors_enabled' into 'master'

Display respawn ghost with player's color instead of white

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

Fix .gitattributes

A later line overrides an earlier line.

Use C syntax highlighting for QC files on gitlab

Fix wrong respawn ghost colors with cl_forceuniqueplayercolors 1

Display respawn ghost with player's color instead of white

Merge branch 'bones_was_here/sv_playerthink_optimise' into 'master'

Move some code that only needs to run once per frame out of PlayerP*Think()

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

Merge branch 'terencehill/intermission_restart_fix' into 'master'

Fix restart applying different gametype if called after gametype vote

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

Merge branch 'drjaska/teleport-minspeed' into 'master'

Mininum velocity for teleporter exits

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

Mininum velocity for teleporter exits

Fix string length checks in world.qc and add VM_TEMPSTRING_MAXSIZE

Merge branch 'Spike29/balance_council_vote19' into 'master'

Apply the balance council vote 19

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

Apply the balance council vote 19

Merge branch 'bones_was_here/maplist_fixes' into 'master'

Map list fixes

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

Fix GUI map voting with g_maplist_shuffle <= 0 && g_maplist_selectrandom 0

MaplistMethod_Iterate() returned the same map every time because
Map_Current wasn't updated.

Documents g_maplist_shuffle -1.

g_maplist: refactor initialisation

Reduces code duplication.

Checks/initialises once when adding maps to the voting screen, instead
of doing it again for each map added.

Always uses gametype-specific g_maplist generation, instead of sometimes
including maps from other gametypes (depending which code path was used).

Updates some declarations and removes an unused global.

g_maplist: update with `gametype` command, as well as GUI gametype voting

This fixes the need to manually set g_maplist to \"\" after running the
gametype command on a votable server.

This is consistent with GUI gametype voting (sv_vote_gametype 1)
behaviour and shares its code to reduce duplication.

mapinfo: fix and rename noautomaplist flag to donotwant

This flag was ignored in the code paths used on most servers, and its
name and behaviour were not ideal for its intended use, see
7d48e976c8a9b3f80350b2ad41d9feec2797cf76 description which seems clear
that it's for allowing a certain kind of map to be available on the
server but not be included in the usual voting pool.

This updated implementation omits the flagged maps from GUI voting and
prevents them being selected by other GotoNextMap()/GetNextMap() code
paths, unless there aren't enough viable maps in which case they
can still be used as fallbacks.

The flagged maps are always listed in CLI and may be nominated with
`suggestmap` or vcalled with `gotomap` or `nextmap`.

Q3 compat: implement a 1ms fallback traveltime for all movers

This fixes the doors failing to open on q3df map sdc-uk_08 which uses
trigger_multiple -> func_button.speed = -1 -> func_door
Buttons trigger targets at the end of their movement and that one didn't
move at all.

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

Merge branch 'terencehill/registry_check' into 'master'

Fix REGISTRY_CHECK

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

Merge branch 'drjaska/ca_prevent_stalemate_2' into 'master'

Separated `g_ca_prevent_stalemate`'s survivor count and health checking.

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

Separated `g_ca_prevent_stalemate`'s survivor count and health checking.

Remove a few dangling semicolons

Fix REGISTRY_CHECK not checking all elements of a huge registry like the Notifications registry

Code is optimized to have basically the same performance as before while adding string length checks.

Merge MR 'func_door and func_plat fixes and Q3 compatibility'

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

Merge branch 'drjaska/spawnfunc-item_buff_name' into 'master'

restore item_buff_speed and item_buff_invisibility spawnfuncs

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

Transifex autosync

playerlist.qc: remove some useless operations in 2 draw_Text calls; while at it clean up code

Fix #2893 "cl_zoomfactor 1 results in division by zero"; also simplify the current_viewzoom check (almost_equals performs redundant checks in this case)

Transifex autosync

Transifex autosync

Transifex autosync

Merge branch 'drjaska/hashtest' into 'master'

Create an usable tool for local hash testing

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

Create an usable tool for local hash testing

Transifex autosync

Fix and make  indentation consistent in the Overkill weapon files

Use cvar_set

Freeze Tag: fix players getting 1 point for a teamkill (they get -1 points now) and not getting -1 points if killed by a map entity (e.g. triggerhurt)

hud_postprocessing: disable the postprocess shader when not in use.

Works best with upcoming r_skipblend.

Fix some damagetext accumulation edge cases

Fixes cl_damagetext_accumulate_alpha_rel 1 and
cl_damagetext_accumulate_lifetime 0 not being able to opt
out of DT accumulation

Merge branch 'drjaska/damagetext' into 'master'

DamageText code quality changes and new DT lifetime and accumulation lifetime cvars

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

DamageText code quality changes and new DT lifetime and accumulation lifetime cvars

Update pipeline for DP engine upgrade

The FIXME was fixed (or at least worked around) in DP's Makefile, see
-mno-avx and its comments.

Transifex autosync

Merge branch 'terencehill/playerstats_update' into 'master'

Minor updates to playerstats

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

Transifex autosync

Transifex autosync

Apply voted game type on map switch, not on game type vote end so that the "restart" command, if called, properly restarts the map applying the current game type

Don't send match stats if game ends during warmup (xonstats doesn't register them anyway)

Send total number of rounds played to xonstats

Transifex autosync

Transifex autosync

Merge branch 'drjaska/shotorigincleanup' into 'master'

Allow for negative z offsets for shot origin

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

Transifex autosync

cleanup and a few comments

Transifex autosync

Transifex autosync

Transifex autosync

Transifex autosync

Transifex autosync

Don't allow negative values for the y component of the shot origin offset because the gun can't be aligned to the left by default. Thanks to this change it's possible to restore precision of the y and z components back to 0.25

Allow negative values for the shot origin offset vector. This change reduces precision of the y and z components from 0.25 to 0.5

Merge branch 'Juhu/velocity_pads_cleanup2' into 'master'

Move trigger_push_velocity last_pushed code out of StartFrame()

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

move trigger_push_velocity's last_pushed check, StartFrame() should not run jump pad physics code

Merge branch 'Juhu/target_print' into 'master'

q3compat: add support for the target_print entity

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

Merge branch 'drjaska/damage-center-player-origin' into 'master'

Added an inactive gameplay cvar for using the own shot origin or centers of other players' bboxes for force direction calculations

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

Added an inactive gameplay cvar for using the own shot origin or centers of other players' bboxes for force direction calculations

Merge branch 'drjaska/damagetext' into 'master'

Refactor damagetext to use a new linked list instead of proximity search and fix 2D and 3D damagetexts not accumulating

Closes #2861

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

Refactor damagetext to use a new linked list instead of proximity search and fix 2D and 3D damagetexts not accumulating

Merge branch 'Juhu/velocity_pads_cleanup' into 'master'

Move trigger_push_velocity last_pushed code out of execute_next_frame()

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

Move trigger_push_velocity last_pushed code out of execute_next_frame()

Merge branch 'bones_was_here/sv_autopause_intermission' into 'master'

sv_autopause: don't pause during intermission, unpause on endmatch command

Closes #2880

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

Transifex autosync

Transifex autosync

sv_autopause: don't pause during intermission, unpause on endmatch command

Transifex autosync

g_maxplayers -1: when rounding to a team multiple, go down at the midpoint

It seems a 5 players max DM map is more likely to suit 2v2 TDM/FT than 3v3.
It will still round up to 6 if there's 3 teams.

Also catches misconfigurations with g_maxplayers -1 and no teams.

Changes to more meaningful var names.

Explicitly enable sv_gameplayfix_unstickplayers

Xonotic's QC physics still depend on this cvar, but the feature is
deprecated in current DP.

Merge branch 'terencehill/load_fonts_once' into 'master'

Fix #2878 "Client startup executes font-xolonium.cfg twice"

Closes #2878

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

Fix #2878 "Client startup executes font-xolonium.cfg twice"

Fix sticky electro orbs and mines being unhittable on DP master

The moving projectiles are configured correctly but when they touch a
surface they're deleted and replaced with a sticky projectile.
Those weren't configured to be hittable directly, and weren't linked to
the area grid so findradius couldn't find them for splash damage.

Support custom func_door and func_plat sounds on Q3A maps

This adds FindFileInMapPack(), a very simple version of
_MapInfo_FindArenaFile().

func_plat: Use Q3 defaults for .dmg .speed and .lip on Q3 maps

func_plat: don't spawn a "start moving" trigger when .targetname is set on Q3 maps

func_plat: fix being stuck in the extended position when .targetname is set on Q3 maps

func_door: merge door_use() and door_fire()

Unnecessary tiny func is tiny and unnecessary (and suboptimal for perf).

func_door: implement Q3 CRUSHER spawnflag

func_door: implement Q3 team-based door linking using fullspawndata

func_door: fix bug where doors blocked by players never reversed direction

This bug was inherited from Nexuiz.  It worked properly in Quake which
didn't have this condition, and in Quake 3.

func_door: use Q3 default .dmg of 2 on Q3 maps

func_door: update documentation to reflect default .dmg of 0

IMO 0 is the best default, because if it were > 0 then mappers would
have no way to configure a door that doesn't deal any damage when blocked.

The implemented default has been 0 since (at least) Nexuiz 2.5.2 so
changing it now would not be ideal for compatibility.

The comment about the default of 2 was inherited from Quake.

func_door: fix .dmg field having no effect

The player's .takedamage == DAMAGE_AIM not DAMAGE_YES

func_door: update DOOR_START_OPEN documentation

func_door: use Q3 default of 2s .wait time on Q3 maps

func_door: fix .angle values of 315 and 135 degrees being inverted

This bug was inherited from Quake and is fixed by updating to
the same maths used in Quake 2 and 3.

func_door: fix erratic behaviour of spawned trigger field

This was most noticeable with door .wait <= 1 but would eventually occur
at higher wait values if players hang around in the trigger field.

Also the door(s) usually did not stay open for the full .wait time after
the player left the trigger field, as some time had passed since the
.door_finished timestamp was set.

To make this reliable AND efficient requires updating the state of the
door(s), instead of maintaining a separate state in the trigger field entity.