Patch: sort results by time and enumerate ranks

[konrad2002]

Summary by CodeRabbit

• Bug Fixes
  • Age group rankings for race events are now properly calculated based on competitor results, with correct handling of finish times and special cases.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7ec631a6-e0c7-4c15-95fd-361e785f5d50

📥 Commits

Reviewing files that changed from the base of the PR and between e86590a and f1cefb4.

📒 Files selected for processing (1)

• src/app/content/events/components/event-view/event-view.component.ts

---

📝 Walkthrough

Walkthrough

The fetchResultStarts() method's age-group rank reassignment logic for result-mode was re-enabled. Previously commented-out code now iterates through starts, assigns rank 0 for disqualified or missing-result entries, and applies ranking based on result times while handling ties via skipped-rank accounting.

Changes

Cohort / File(s)	Summary
Rank Reassignment Logic Restoration src/app/content/events/components/event-view/event-view.component.ts	Re-enabled age-group rank reassignment in fetchResultStarts(). Code now iterates through starts, sets rank to 0 for disqualified/missing-result entries, assigns sequential ranks based on result times, and accounts for ties via skippedRanks counter advancement.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Ranks were sleeping, oh what a shame,
Now they awaken to join the game,
Ties are tied up, and zeros assigned,
The race results finally aligned!
Hopping through logic, all ranks in place, ✨

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch develop

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}