Kevin
cee4a2e0eb
Ergebnis des Senior-Reviews über dev/2026-07-22 (Konzept 22.07., Feedback 24.07.,
Plan 19.08.). Rechnerisch war nichts zu korrigieren: Payline-Punkte und Provision
des Prognose-Blocks (N4) stimmen exakt mit der Provisions-Engine überein
(54.718,5 P. bzw. 3.772,88 EUR gegengerechnet an User 454). Behoben sind
Darstellungs- und Konsistenzfehler, die der Beraterin als falsche Zahlen
aufgefallen wären:
- "+N neu" schrumpfte rückwirkend, der Bestand daneben nicht: newAboCount() nutzte
weiter activeAbo(), während aboStockCount() auf das Monatsende zurückrechnet.
Beide teilen sich jetzt aboStockQuery(); die Jahres-Wachstumstabelle ebenso.
- Level-Kachel widersprach sich ("Zum nächsten Level Diamant fehlen X" über
"Karriere-Level: Diamant"): Text auf die Monats-Qualifikation umformuliert
(de/en/es/fr), Überschrift "Qualifikation in diesem Monat", Karriere-Level nur
noch bei Abweichung.
- Firstlines-Kachel zählte alle direkten Partner, "Teampartner gesamt" daneben nur
aktive (F1): Kachel nutzt jetzt isActiveConsultant(), die Detailseite listet
weiter alle und weist beide Zahlen aus.
- Neupartnerzahlen vergangener Monate sanken auf dem Home-Dashboard, sobald
Jahresgebühren ausliefen: neue User::newPartnerReferenceDate() (laufender Monat =
heute, abgeschlossener Monat = Monatsende). Dabei die drei Inline-Kopien der
Neupartner-Regel auf scopeNewPartnerIn()/isNewPartnerIn() zusammengeführt.
- Top-3-Karte ließ den Betrachter aus, obwohl "ich gehört in die Liste" in der
Juli-Runde für alle anderen Ranglisten umgesetzt wurde: zählt jetzt mit, inkl.
is_self und "ich"-Badge.
- Neues Profilfoto zeigte bis zu 6 h ein totes Bild: die URL steckt nicht mehr im
Cache, sondern entsteht beim Lesen (withProfileImages()).
- Test-Regression seit der Juli-Runde: Unit/Services/BackofficeDashboardServiceTest
fiel über die fehlende Spalte status_points (kam mit nonPaylinePoints() dazu).
- Kleinigkeiten: ProfileImageController gab die komplette Exception als
Flash-Message aus (jetzt Log + Sprachkey), BackofficeDrilldownService formatierte
Datumsangaben hart auf d.m.Y statt über Util::formatDateDB().
Tests: neuer HomeMonthlyStatsTest (buildMonthlyStats war ungetestet) plus Tests für
Stichtags-Zählung und Foto-Cache. Fünf Tests kodierten das alte Verhalten
(Betrachter zählt nicht mit, abgelaufener Account zählt nie) und wurden auf die
neue Semantik umgestellt, jeweils mit Gegenprobe. Suite: 861 grün (vorher 855),
89 rot — ausschließlich die in docs/2026-06-11-test-suite-vorbestehende-fehler.md
dokumentierten Altlasten.
Offen und bewusst nicht enthalten:
- Deploy-Schritt: backoffice:store-statistics-snapshots --force (vorher dumpen).
Kein vorhandener Snapshot enthält recruiter_ranking, Mai/Juni zusätzlich weder
non_payline_points noch die Führungskräfte-"gesamt"-Spalten; der Cron überspringt
vorhandene Snapshots ohne --force. Achtung: die Neuberechnung ist nicht
verlustfrei.
- TreeHtmlRendererTest/TreeCalcBotIntegrationTest: die Abo-Badges machten den
Renderer datenbankabhängig; die Dateien waren schon vorher rot und gehören in die
separate Test-Aufräumaufgabe.
- deleteImage bleibt ein GET-Route (gleiches Muster wie das Shop-Bild).
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
63 lines
3.8 KiB
PHP
63 lines
3.8 KiB
PHP
{{-- "Die besten Ergebnisse des Monats" (N6, teamintern): drei Top-3-Ranglisten
|
|
aus TopResultsService via $monthlyStats['top_results']. Karte erscheint nur,
|
|
wenn mindestens eine Liste Einträge hat. --}}
|
|
@php
|
|
$topResults = $monthlyStats['top_results'] ?? [];
|
|
$topColumns = [
|
|
['key' => 'top_volume', 'title' => __('home.top_results_volume'), 'unit' => __('home.top_results_unit_points'), 'decimals' => 0],
|
|
['key' => 'top_recruiters', 'title' => __('home.top_results_recruiters'), 'unit' => __('home.top_results_unit_partners'), 'decimals' => 0],
|
|
['key' => 'top_new_abos', 'title' => __('home.top_results_new_abos'), 'unit' => __('home.top_results_unit_abos'), 'decimals' => 0],
|
|
];
|
|
$hasTopResults = collect($topColumns)->contains(fn($column) => !empty($topResults[$column['key']]));
|
|
@endphp
|
|
|
|
@if (!empty($monthlyStats) && $hasTopResults)
|
|
<div class="d-flex col-xl-12 align-items-stretch">
|
|
<div class="card w-100 mb-4">
|
|
<h5 class="card-header">
|
|
<i class="ion ion-md-trophy mr-2"></i>{{ __('home.top_results_title') }} -
|
|
{{ HTMLHelper::getMonth($monthlyStats['month']) }} {{ $monthlyStats['year'] }}
|
|
</h5>
|
|
<div class="card-body">
|
|
<div class="row">
|
|
@foreach ($topColumns as $column)
|
|
<div class="col-md-4 mb-3 {{ !$loop->last ? 'border-md-right' : '' }}">
|
|
<h6 class="text-muted text-uppercase font-weight-bold small mb-3">{{ $column['title'] }}
|
|
</h6>
|
|
@forelse ($topResults[$column['key']] ?? [] as $row)
|
|
<div class="d-flex align-items-center mb-3">
|
|
<div class="position-relative" style="flex: 0 0 auto;">
|
|
@include('components._avatar', [
|
|
'imageUrl' => $row['profile_image_url'],
|
|
'firstName' => $row['first_name'],
|
|
'lastName' => $row['last_name'],
|
|
'size' => 48,
|
|
])
|
|
@if ($loop->first)
|
|
<i class="ion ion-md-trophy text-warning position-absolute"
|
|
style="right: -4px; bottom: -4px; font-size: 1rem;"
|
|
title="{{ $column['title'] }}"></i>
|
|
@endif
|
|
</div>
|
|
<div class="ml-3">
|
|
<div class="font-weight-bold">
|
|
{{ $row['first_name'] }} {{ $row['last_name'] }}
|
|
@if (!empty($row['is_self']))
|
|
<span class="badge badge-pill badge-info ml-1">{{ __('home.top_results_self') }}</span>
|
|
@endif
|
|
</div>
|
|
<div class="text-muted small">
|
|
{{ number_format((float) $row['value'], $column['decimals'], ',', '.') }}
|
|
{{ $column['unit'] }}</div>
|
|
</div>
|
|
</div>
|
|
@empty
|
|
<div class="text-muted small">{{ __('home.top_results_empty') }}</div>
|
|
@endforelse
|
|
</div>
|
|
@endforeach
|
|
</div>
|
|
</div>
|
|
</div>
|
|
</div>
|
|
@endif
|