Statistik: Neupartner zaehlen im Monat der Zahlung, nicht der Registrierung

Wer Ende August registriert und am 1. September bezahlt hat, tauchte in keiner
Neupartner-Zaehlung auf: Der August-Snapshot wurde um 04:45 eingefroren, da
fehlte noch das Abrechnungskonto; im September lag active_date im falschen
Monat. Betroffen sind 2026 acht Partner, auch im Recruiting ihrer Sponsoren.

Kundenentscheidung 16.09.2026: der Zeitpunkt der Zahlung ist massgeblich.

Neue Spalte users.partner_paid_at, gesetzt bei der ersten Kontozahlung
(Payment::paymentStatusPaidAction) und bei manueller Freischaltung durch den
Admin — nur, solange sie leer ist; Verlaengerung und Wiederherstellung aendern
sie nicht. Bestehende Partner werden per Migration aus der ersten bezahlten
Registrierungsbestellung rueckgefuellt (Zeitstempel der Punktebuchung, ersatzweise
Bestelldatum, ohne Bestellung das Aktivierungsdatum). Bewusst nicht
payment_account minus ein Jahr: das ergaebe bei Verlaengerern den Jahrestag der
Erstzahlung.

User::scopeNewPartnerIn() und isNewPartnerIn() lesen jetzt partner_paid_at —
damit Kachel, Recruiter-Ranking, Top 3 und Neupartner-Liste gleichermassen.
Die Liste und der CSV-Export zeigen "Bezahlt am" neben "Registriert am".

Unveraendert: die Teamstruktur haengt weiter an active_date (auch im
Provisionsbaum). Wer ist neu dazugekommen und wer gehoert zur Linie sind zwei
verschiedene Fragen.

Die eingefrorenen Monate Januar bis August enthalten sieben dieser Partner
nicht; ob sie per --force neu gebaut werden, ist eine Entscheidung — siehe
dev/2026-07-22/neupartner-zahlungsmonat.md.

Mit drin: die handgebaute SQLite-Tabelle in BackofficeDashboardServiceTest
kannte status_reason aus d6efe56 noch nicht.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

app/Http/Controllers/AdminUserController.php

@@ -162,6 +162,9 @@ class AdminUserController extends Controller
                 $payment_account = Carbon::parse(str_replace('- ', '', $data['payment_account']));
                 $user->payment_account = $payment_account;
                 if ($payment_account > Carbon::now()) {
+                    // Manuelle Freischaltung ohne Bestellung: der Partner zaehlt ab jetzt
+                    // als bezahlt (nur beim ersten Mal, siehe markPartnerPaid).
+                    UserUtil::markPartnerPaid($user);
                     if ($user->active === 0) {
                         $user->active = true;
                         UserUtil::reactiveUserResetChilds($user->id, 'on save-account AdminUserController');
@@ -299,7 +302,7 @@ class AdminUserController extends Controller
             ->addColumn('active', function (User $user) {
                 if ($user->trashed()) {
                     $date = Carbon::parse(now())->addDays(10)->format('d.m.Y H:i');
-                    $email = str_replace('delete-', '', $user->email);
+                    $email = UserUtil::stripDeletePrefix($user->email);
                     $link = '<a href="#" data-toggle="modal" data-target="#modals-restore" data-id="'.$user->id.'" data-email="'.$email.'" data-active="'.$user->active.'" data-payment_account="'.$date.'">';
 
                     return $link.'<span class="badge badge-pill badge-info"><i class="fa fa-undo"></i> Account reaktivieren</span></a>';

app/Http/Controllers/User/BackofficeStatisticsController.php

@@ -288,7 +288,7 @@ class BackofficeStatisticsController extends Controller
         }
 
         if ($metric === 'new_partners') {
-            return ['Name', 'E-Mail', 'Telefon', 'Karriere-Level', 'Sponsor', 'Punkte seiner Kunden', 'Gesamt inkl. Eigenbestellung', 'Aktiv seit', 'Account gueltig bis'];
+            return ['Name', 'E-Mail', 'Telefon', 'Karriere-Level', 'Sponsor', 'Punkte seiner Kunden', 'Gesamt inkl. Eigenbestellung', 'Bezahlt am', 'Registriert am', 'Account gueltig bis'];
         }
 
         return ['Name', 'E-Mail', 'Karriere-Level', 'Aktiv seit', 'Account gueltig bis', 'Account Status'];
@@ -348,6 +348,7 @@ class BackofficeStatisticsController extends Controller
                 $row['sponsor'] ?? '',
                 $row['customer_points'] ?? 0,
                 $row['total_points'] ?? 0,
+                $row['paid_at'] ?? '',
                 $row['active_date'] ?? '',
                 $row['payment_account'] ?? '',
             ];
@@ -385,10 +386,10 @@ class BackofficeStatisticsController extends Controller
         }
 
         if ($metric === 'new_partners') {
-            // Neun Werte fuer neun Spalten: die Telefonspalte kam nachtraeglich dazu,
-            // wodurch die Summen um eine Spalte verrutscht waren (Kundenpunkte
-            // standen unter "Sponsor").
-            return ['Summe', $summary['count'].' Eintraege', '', '', '', $summary['customer_points'], $summary['total_points'], '', ''];
+            // Zehn Werte fuer zehn Spalten — die Breitenpruefung im Test faengt ein
+            // Verrutschen der Summen ab, wie es beim Nachruesten der Telefonspalte
+            // passiert war (Kundenpunkte standen unter "Sponsor").
+            return ['Summe', $summary['count'].' Eintraege', '', '', '', $summary['customer_points'], $summary['total_points'], '', '', ''];
         }
 
         return ['Summe', $summary['count'].' Eintraege', '', '', '', ''];

app/Services/Backoffice/BackofficeDrilldownService.php

@@ -155,7 +155,7 @@ class BackofficeDrilldownService
             ->with(['account', 'user_level'])
             ->whereIn('id', $userIds)
             ->newPartnerIn($month, $year)
-            ->orderBy('active_date')
+            ->orderBy('partner_paid_at')
             ->get();
 
         $sponsors = User::query()
@@ -179,6 +179,9 @@ class BackofficeDrilldownService
                 'sponsor' => $user->m_sponsor && $sponsors->has((int) $user->m_sponsor) ? $this->userName($sponsors->get((int) $user->m_sponsor)) : '-',
                 'customer_points' => (float) ($monthlyPoints->get((int) $user->id)['external_points'] ?? 0.0),
                 'total_points' => (float) ($monthlyPoints->get((int) $user->id)['total_points'] ?? 0.0),
+                // Massgeblich fuer die Zugehoerigkeit zum Monat (User::scopeNewPartnerIn()).
+                'paid_at' => $this->formatDate($user->partner_paid_at),
+                'paid_at_sort' => $this->sortDate($user->partner_paid_at),
                 'active_date' => $this->formatDate($user->active_date),
                 'active_date_sort' => $this->sortDate($user->active_date),
                 'payment_account' => $this->formatDate($user->payment_account),

app/Services/Payment.php

@@ -225,6 +225,9 @@ class Payment
                                 $user->payment_order_id = $shopping_order_item->product->id;
                                 $user->payment_account = $date;
                                 $user->wizard = 100;
+                                // Erste Kontozahlung = Zeitpunkt, ab dem der Partner in der
+                                // Neupartner-Statistik zaehlt. Verlaengerungen aendern ihn nicht.
+                                UserUtil::markPartnerPaid($user);
                                 // only date is > now and acount is deactive.
                                 if ($date > \Carbon::now()) {
                                     if ($user->active === 0) {

app/Services/UserUtil.php

@@ -2,11 +2,14 @@
 
 namespace App\Services;
 
+use App\Models\ShoppingOrder;
 use App\Models\ShoppingUser;
 use App\Models\ShoppingUserMemberLog;
 use App\Models\UserCleanUpLog;
+use App\Models\UserSalesVolume;
 use App\Models\UserShop;
 use App\User;
+use Carbon\Carbon;
 
 class UserUtil
 {
@@ -166,6 +169,27 @@ class UserUtil
         $user->save();
     }
 
+    /**
+     * Praefix fuer pseudonymisierte E-Mail-Adressen geloeschter Benutzer.
+     * Die User-ID gehoert zwingend dazu: ohne sie kollidiert die zweite
+     * Loeschung derselben Adresse mit dem Unique-Index auf users.email.
+     */
+    public static function buildDeletedEmail(User $user): string
+    {
+        $prefix = 'delete-'.$user->id.'-';
+
+        return $prefix.mb_substr(self::stripDeletePrefix((string) $user->email), 0, 255 - mb_strlen($prefix));
+    }
+
+    /**
+     * Entfernt das Loesch-Praefix inklusive der optionalen User-ID.
+     * Beruecksichtigt bewusst auch das alte Format ohne ID.
+     */
+    public static function stripDeletePrefix(?string $email): string
+    {
+        return (string) preg_replace('/^delete-(\d+-)?/', '', (string) $email);
+    }
+
     public static function deleteUser(User $user, $complete = false)
     {
         // shop wird gelöscht
@@ -177,7 +201,7 @@ class UserUtil
         }
 
         // user soll nicht komplett gelöscht werden
-        $user->email = 'delete-'.$user->email;
+        $user->email = self::buildDeletedEmail($user);
         // password wird gelöscht
         $user->password = 'delete'.time();
         $user->confirmed = 0;
@@ -194,7 +218,7 @@ class UserUtil
         $user->pre_deleted_at = now();
         // user soll komplett gelöscht werden
         if ($complete) {
-            $user->email = 'delete-'.time().'-'.rand(1000, 9999);
+            $user->email = 'delete-'.$user->id.'-'.time().'-'.rand(1000, 9999);
             if ($user->account) {
                 $user->account->delete();
             }
@@ -208,7 +232,7 @@ class UserUtil
 
     public static function checkEmailExists($user)
     {
-        $email = str_replace('delete-', '', $user->email);
+        $email = self::stripDeletePrefix($user->email);
 
         $user = User::where('email', $email)->first();
         if ($user) {
@@ -226,7 +250,7 @@ class UserUtil
             $user->pre_sponsor = null;
         }
 
-        $user->email = str_replace('delete-', '', $user->email);
+        $user->email = self::stripDeletePrefix($user->email);
         $user->confirmed = 1;
         $user->confirmation_date = now();
         $user->confirmation_code = null;
@@ -251,6 +275,89 @@ class UserUtil
         }
     }
 
+    /**
+     * Haelt fest, wann der Partner zum bezahlten Partner wurde — nur beim ersten
+     * Mal. Verlaengerungen und Wiederherstellungen lassen den Wert stehen, sonst
+     * wuerde ein alter Partner in der Neupartner-Statistik wieder auftauchen.
+     *
+     * @return bool true, wenn der Wert gerade gesetzt wurde
+     */
+    public static function markPartnerPaid(User $user, ?Carbon $paidAt = null): bool
+    {
+        if ($user->partner_paid_at !== null) {
+            return false;
+        }
+
+        $user->partner_paid_at = $paidAt ?? now();
+
+        return true;
+    }
+
+    /**
+     * Rueckfuellung von `users.partner_paid_at` fuer bestehende Partner, nur wo
+     * die Spalte leer ist — mehrfach ausfuehrbar.
+     *
+     * Quelle in dieser Reihenfolge:
+     *  1. die Punktebuchung der ersten bezahlten Registrierungsbestellung
+     *     (`payment_for = 1`) — sie entsteht im Moment der Zahlung;
+     *  2. ersatzweise das Aenderungsdatum dieser Bestellung;
+     *  3. ohne Registrierungsbestellung (Konto vom Admin freigeschaltet) das
+     *     Aktivierungsdatum — einen Zahlungszeitpunkt gibt es dort nicht.
+     *
+     * Absichtlich NICHT `payment_account` minus ein Jahr: bei Verlaengerungen
+     * wird das Konto vom alten Ablaufdatum aus verlaengert, das Ergebnis waere der
+     * Jahrestag der Erstzahlung und jeder Verlaengerer ein „Neupartner".
+     *
+     * @return int Anzahl gesetzter Zeilen
+     */
+    public static function backfillPartnerPaidAt(): int
+    {
+        $set = 0;
+
+        User::withTrashed()
+            ->whereNull('partner_paid_at')
+            ->whereNotNull('payment_account')
+            ->orderBy('id')
+            ->chunkById(200, function ($users) use (&$set): void {
+                foreach ($users as $user) {
+                    $paidAt = self::firstRegistrationPaidAt((int) $user->id)
+                        ?? ($user->active_date ? Carbon::parse($user->active_date) : null);
+
+                    if ($paidAt === null) {
+                        continue;
+                    }
+
+                    $user->partner_paid_at = $paidAt;
+                    $user->saveQuietly();
+                    $set++;
+                }
+            });
+
+        return $set;
+    }
+
+    private static function firstRegistrationPaidAt(int $userId): ?Carbon
+    {
+        $order = ShoppingOrder::query()
+            ->where('auth_user_id', $userId)
+            ->where('payment_for', 1)
+            ->where('paid', 1)
+            ->orderBy('id')
+            ->first(['id', 'updated_at']);
+
+        if ($order === null) {
+            return null;
+        }
+
+        $booking = UserSalesVolume::query()
+            ->where('user_id', $userId)
+            ->where('shopping_order_id', $order->id)
+            ->orderBy('id')
+            ->value('created_at');
+
+        return Carbon::parse($booking ?? $order->updated_at);
+    }
+
     public static function reactiveUserResetChilds($user_id, $info = '')
     {
 

app/User.php

@@ -153,6 +153,7 @@ use Laravel\Passport\Token;
  * @method static \Illuminate\Database\Eloquent\Builder|User wherePreSponsor($value)
  *
  * @property \Illuminate\Support\Carbon|null $pre_deleted_at
+ * @property \Illuminate\Support\Carbon|null $partner_paid_at
  *
  * @method static \Illuminate\Database\Eloquent\Builder<static>|User wherePreDeletedAt($value)
  *
@@ -185,7 +186,7 @@ class User extends Authenticatable
         'settings' => 'array',
         'payment_methods' => 'array',
         'pre_deleted_at' => 'datetime',
-
+        'partner_paid_at' => 'datetime',
     ];
 
     /**
@@ -248,7 +249,16 @@ class User extends Authenticatable
     /**
      * Neupartner-Definition der Statistik (eine Regel für KPI, Recruiter-Ranking
      * und Top-3-Karte): Mitgliedsebene gesetzt, Abrechnungskonto zum Stichtag
-     * gültig ({@see newPartnerReferenceDate()}), im Monat aktiv geworden.
+     * gültig ({@see newPartnerReferenceDate()}), **im Monat bezahlt**.
+     *
+     * Massgeblich ist `partner_paid_at`, nicht `active_date` (Kundenentscheidung
+     * 16.09.2026): Wer Ende August registriert und am 1. September bezahlt,
+     * ist ein September-Neupartner. Mit `active_date` fiel dieser Fall durch —
+     * im August noch ohne Abrechnungskonto, im September mit dem falschen
+     * Aktivierungsmonat — und fehlte damit in jeder Zählung, auch im Recruiting.
+     *
+     * Die Teamstruktur (wer gehört zum Monatsende zur Linie) hängt weiterhin an
+     * `active_date`; das ist eine andere Frage als „wer ist neu dazugekommen".
      *
      * @param  Builder  $query
      * @return Builder
@@ -261,7 +271,7 @@ class User extends Authenticatable
         return $query->whereNotNull('m_level')
             ->whereNotNull('payment_account')
             ->where('payment_account', '>', self::newPartnerReferenceDate($month, $year))
-            ->whereBetween('active_date', [$startDate, $endDate]);
+            ->whereBetween('partner_paid_at', [$startDate, $endDate]);
     }
 
     /**
@@ -271,7 +281,7 @@ class User extends Authenticatable
      */
     public function isNewPartnerIn(int $month, int $year): bool
     {
-        if ($this->m_level === null || $this->active_date === null || $this->payment_account === null) {
+        if ($this->m_level === null || $this->partner_paid_at === null || $this->payment_account === null) {
             return false;
         }
 
@@ -281,7 +291,7 @@ class User extends Authenticatable
 
         $startDate = Carbon::create($year, $month, 1)->startOfMonth();
 
-        return Carbon::parse($this->active_date)->betweenIncluded($startDate, $startDate->copy()->endOfMonth());
+        return Carbon::parse($this->partner_paid_at)->betweenIncluded($startDate, $startDate->copy()->endOfMonth());
     }
 
     public function user_pre_sponsor()