gedcom icon indicating copy to clipboard operation
gedcom copied to clipboard

`Gedcom::Individual::relationship`: Add step-relationships

Open tukusejssirs opened this issue 5 years ago • 2 comments

In Gedcom::Individual::relationship, you missed step-brother and step-sister. Difference between half-sibling and step-sibling is that half-siblings have one common parent, where step-siblings none.

I could create a PR with the following code added:

if(($self->mother() ne $other->mother()) && ($self->father() ne $other->father())) {
  return ($sex eq 'M') ? 'step-brother' : 'step-sister';
}

but when is the following code used?

return $self->relationship_down($other) || $self->relationship_up($other);

tukusejssirs avatar Oct 17 '19 13:10 tukusejssirs

This can only be done if step-mother and/or step-father relationships are worked out. Which currently they are not.

nigelhorne avatar Oct 18 '19 19:10 nigelhorne

I’ll look at it out tomorrow (CEST).

tukusejssirs avatar Oct 18 '19 20:10 tukusejssirs