Cause is EventFact instead of String

[michelesalvador]

I can't understand why EventFact.caus is of class EventFact instead of String.

In GEDCOM 5.5 Cause is defined as

EVENT_STRUCTURE
n  <EVENT_TAG>          {1:1}
...
+1 CAUS <CAUSE_OF_DEATH>    {0:1}

And in GEDCOM 5.5.1

EVENT_DETAIL
...
n CAUS <CAUSE_OF_EVENT>     {0:1}

Both <CAUSE_OF_DEATH> and <CAUSE_OF_EVENT> are simple values of {Size=1:90}. No trace of another event nested inside.

Should EventFact.caus be of class String?

[michelesalvador]

There is a comment in EventFact that suggests the reason why caus tag is of class EventFact:

so it can have source citations

It's a noble intent, but in my opinion it's not GEDCOM standard.

[stoicflame]

Agreed.

I'm happy to merge a pull request.

[michelesalvador]

Opened pull request #12.

[stoicflame]

Closed with #12.