liberu-genealogy / php-gedcom

Gedcom reading and writing for PHP 8.3
https://www.liberu.co.uk
MIT License
74 stars 41 forks source link

Sweep: bug duplicate issue #48

Closed curtisdelicata closed 2 months ago

curtisdelicata commented 3 months ago

Details

The DEST Tag is an optional part of the HEAD tag. The DEST tag itself has no value, only sub-tags. THE DEST tag is very much like the SOUR tag, the mandatory part of the HEAD tag.

The error is thrown in \vendor\liberu-genealogy\php-gedcom\src\Parser\Head.php in line 62:

case 'DEST':
    $head->setDest(trim((string) $record[2]));
    break;

Handling the DEST tag should be like handling the SOUR tag.

case 'DEST':
    $dest= \Gedcom\Parser\Head\Dest::parse($parser);
    $head->setDest($dest);
    break;

The files for handling the DEST sub-tags and their class files are to be implemented.

sweep-ai[bot] commented 3 months ago
Sweeping

0%
💎 Sweep Pro: You have unlimited Sweep issues

Actions


[!TIP] To recreate the pull request, edit the issue title or description.