Commit 5ff54e5b authored by Mahmoud Aglan's avatar Mahmoud Aglan

docs(subscriptions): mark orphan root cause as fixed in architecture map

Co-Authored-By: 's avatarClaude Opus 4.6 <noreply@anthropic.com>
parent 39cb5800
...@@ -415,7 +415,7 @@ php cli.php subscriptions:validate # Verify data integrity ...@@ -415,7 +415,7 @@ php cli.php subscriptions:validate # Verify data integrity
- 66 members, 53 spouses, 86-89 children, 1-11 temporary — all matching active dependents - 66 members, 53 spouses, 86-89 children, 1-11 temporary — all matching active dependents
- Validation: ALL CLEAN - Validation: ALL CLEAN
**Known limitation**: `RetroactiveMembershipService` (line 714) has `$sub['person_id'] ?? $memberId` fallback that creates orphaned records. The migration cleans these up. **Root cause fixed (2026-07-04)**: `RetroactiveMembershipService` previously fell back to `$memberId` for unmapped dependents — now resolves by name from the dependent table, or skips the row if unresolvable. Migration still cleans up historical orphans.
--- ---
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment