Skip to content
Snippets Groups Projects

Fix mismatching player sorting between server and client

Merged terencehill requested to merge terencehill/player_sorting into master
  • Fix mismatching player sorting between server (XonStats scoreboard) and client (game scoreboard)

  • tweak player sorting priority a little bit

  • Ignore elo and fps sorting players in the server (client scoreboard already does)

Extra:

  • Don't initialize the teamkill score field in games without teams so it no longer appears in the complete layout (scoreboard_columns_set all)
Edited by terencehill

Merge request reports

Merge train pipeline #668143870 passed

Merge train pipeline passed for 9db18420

Approved by

Merged by terencehillterencehill 2 years ago (Oct 16, 2022 9:28pm UTC)

Merge details

  • Changes merged into master with 72b4ff94.
  • Deleted the source branch.
  • Auto-merge enabled

Pipeline #668144715 passed

Pipeline passed for 72b4ff94 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • terencehill added 1 commit

    added 1 commit

    • 7299b6f0 - Make keepaway registration order consistent. It doesn't change player sorting...

    Compare with previous version

  • terencehill changed the description

    changed the description

  • Differences between current and new player sorting priority

    current (client) current (server) new (client and server)
    primary primary primary
    secondary secondary secondary
    - race_fastest race_laps
    - race_laps race_time
    - race_time race_fastest
    - assault_objectives assault_objectives
    - ctf_drops ctf_caps
    - ctf_captime ctf_fckills
    - ctf_caps ctf_returns
    - ctf_returns ctf_drops
    - ctf_fckills ctf_pickups
    - ctf_pickups ctf_captime
    - dom_ticks dom_takes
    - dom_takes dom_ticks
    - freezetag_revivals freezetag_revivals
    - keepaway_carrierkills keepaway_bctime
    - keepaway_bctime keepaway_carrierkills
    - keepaway_pickups keepaway_pickups
    - kh_losses kh_caps
    - kh_destroys kh_kckills
    - kh_pushes kh_losses
    - kh_kckills kh_destroys
    - kh_caps kh_pushes
    - kh_pickups kh_pickups
    - lms_lives lms_rank
    - lms_rank lms_lives
    - nexball_faults nexball_goals
    - nexball_goals nexball_faults
    - ons_caps ons_caps
    - ons_takes ons_takes
    - fps -
    - elo -
    - teamkills score
    kills suicides kills
    deaths deaths deaths
    - kills teamkills
    suicides dmgtaken suicides
    dmg dmg dmg
    dmgtaken score dmgtaken

    NOTES:

    • current server's sorting priority is the inverse score registration order (from common/scores.qh), however in this table I rearranged gamemode score groups (without altering sorting priority since gamemode scores are independent of each other) so they are next to server's new sorting priority and it's easier to see the changes I did in the new sorting priority.
    • some gamemode scores are primary and secondary (race_laps, race_time, assault_objectives, keepaway_bctime, lms_lives, lms_rank, kh_caps, ctf_caps, nexball_goals, nexball_faults, ons_caps) and can be ignored
    • 'score' usually is primary and can be ignored
  • Developer

    Nice work fixing this inconsistency, everything seems to be functional in my tests. Approving.

  • Mario approved this merge request

    approved this merge request

  • LegendaryGuard approved this merge request

    approved this merge request

  • terencehill added 464 commits

    added 464 commits

    Compare with previous version

  • terencehill added 1 commit

    added 1 commit

    Compare with previous version

  • terencehill started a merge train

    started a merge train

  • merged

  • terencehill mentioned in commit 72b4ff94

    mentioned in commit 72b4ff94

Please register or sign in to reply
Loading