gitmedia/mivita
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Audit 18.09.: Zahlungsdatum nur bei Zahlung, Neupartner-IDs im Snapshot, Pausenende folgt dem Termin

Browse source 
F1: partner_paid_at wird beim Rechnungskauf (paymentStatusPaidAction mit
paid=false) nicht mehr gesetzt, sondern erst bei der Rechnungszahlung; der
Backfill nimmt das Aktivierungsdatum nur noch ohne jede
Registrierungsbestellung (offene Rechnung bleibt leer).
F2: Snapshots speichern je Linie new_partner_ids, die Neupartner-Liste zeigt
genau diese; aeltere Snapshots ohne IDs rechnen mit der damaligen Regel
(Registrierungsmonat, scopeNewPartnerInByActiveDate), damit Kachel und Liste
uebereinstimmen. Keine Payload-Versionsaenderung, kein Neuaufbau noetig.
F3: Liefertag-/Termin-Aenderung waehrend einer Pause zieht paused_until mit;
eine Ausfuehrung loescht auch den paused_by_*-Grund.
F4: expiredCount der Firstlines-Seite (Controller, View, Sprachen) zum bereits
committeten Test nachgezogen.
F5: Backfills schreiben ohne updated_at (toBase()->update, timestamps=false).
Kuendigungsbestaetigung abo_cancel_confirm wird im Selfservice serverseitig
verlangt. AboStatusExplainer laedt die Historie je Abo und Aktion nur einmal.

Stellungnahme und Deploy-Folgen im Audit-Dokument, Regressionstests je Befund.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
Kevin 2026-09-18 08:59:45 +00:00
parent d5ad1ba50b
commit 4899e3a890
27 changed files with 560 additions and 21 deletions
Download patch file Download diff file Expand all files Collapse all files

48
tests/Feature/PartnerPaidAtTest.php
View file

@ -106,7 +106,55 @@ describe('partner_paid_at: erste Kontozahlung', function () {
});
});
describe('partner_paid_at: Rechnungskauf (Audit 18.09., F1)', function () {
it('setzt den Wert erst bei der Zahlung der Rechnung, nicht bei der offenen Rechnung', function () {
Carbon::setTestNow(Carbon::parse('2026-08-31 18:00:00'));
$user = ppaMakeUser();
$product = Product::forceCreate([
'name' => 'Starterpaket', 'title' => 'Starterpaket', 'price' => 89.0, 'tax' => 19.0, 'active' => true,
'pos' => 1, 'weight' => 0, 'points' => 0, 'slug' => 'starter-'.uniqid(), 'action' => [0],
]);
$order = ppaRegistrationOrder($user, false);
ShoppingOrderItem::forceCreate(['shopping_order_id' => $order->id, 'product_id' => $product->id, 'qty' => 1, 'price' => 89.0]);
// Rechnungskauf: Konto wird vorab freigeschaltet, bezahlt ist noch nichts.
Payment::paymentStatusPaidAction($order->fresh(), false);
$user->refresh();
expect($user->payment_account)->not->toBeNull()
->and($user->partner_paid_at)->toBeNull()
->and($user->isNewPartnerIn(8, 2026))->toBeFalse();
// Zahlungseingang im September: jetzt zaehlt er, und zwar im September.
Carbon::setTestNow(Carbon::parse('2026-09-01 09:00:00'));
Payment::paymentStatusPaidAction($order->fresh(), true);
$user->refresh();
expect($user->partner_paid_at?->toDateString())->toBe('2026-09-01')
->and($user->isNewPartnerIn(9, 2026))->toBeTrue()
->and($user->isNewPartnerIn(8, 2026))->toBeFalse();
});
});
describe('partner_paid_at: Rueckfuellung fuer bestehende Partner', function () {
it('laesst eine offene Rechnung ohne bezahlte Bestellung leer statt das Aktivierungsdatum zu nehmen (Audit F1)', function () {
$user = ppaMakeUser(['payment_account' => '2027-08-28 00:00:00']);
ppaRegistrationOrder($user, false);
expect(UserUtil::backfillPartnerPaidAt())->toBe(0)
->and($user->fresh()->partner_paid_at)->toBeNull();
});
it('laesst updated_at beim Rueckfuellen unangetastet (Audit F5)', function () {
$user = ppaMakeUser(['payment_account' => '2027-08-28 00:00:00']);
User::whereKey($user->id)->toBase()->update(['updated_at' => '2026-05-05 05:05:05']);
UserUtil::backfillPartnerPaidAt();
expect($user->fresh()->partner_paid_at)->not->toBeNull()
->and($user->fresh()->updated_at->format('Y-m-d H:i:s'))->toBe('2026-05-05 05:05:05');
});
it('nimmt die Punktebuchung der ersten bezahlten Registrierungsbestellung', function () {
$user = ppaMakeUser(['payment_account' => '2027-09-01 06:49:35']);
$order = ppaRegistrationOrder($user);