Skip to content

when last member of a family is deleted then delete that family #5341

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
BertKoor wants to merge 1 commit into
base: main
Choose a base branch
from
+9 −0
Open

when last member of a family is deleted then delete that family #5341

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions app/Http/RequestHandlers/DeleteRecord.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,15 @@ public function handle(ServerRequestInterface $request): ResponseInterface
/* I18N: %s are names of records, such as sources, repositories or individuals */
FlashMessages::addMessage(I18N::translate('The link from “%1$s” to “%2$s” has been deleted.', sprintf('<a href="%1$s" class="alert-link">%2$s</a>', e($relict->url()), $relict->fullName()), $linker->fullName()));
}
} else if (
// If we have removed the last member from a family
$linker instanceof Family &&
preg_match_all('/\n1 (HUSB|WIFE|CHIL) @(' . Gedcom::REGEX_XREF . ')@/', $new_gedcom, $match) === 0
) {
// Delete the family
/* I18N: %s is the name of a family group, e.g. “Husband name + Wife name” */
FlashMessages::addMessage(I18N::translate('The family “%s” has been deleted because the last member has been deleted.', $linker->fullName()));
$linker->deleteRecord();
} else {
// Remove links from $linker to $record
/* I18N: %s are names of records, such as sources, repositories or individuals */
Expand Down
Loading