FamilyTree365\LaravelGedcom\Utils\Exporter\Indi\Name

Complexity

Total Complexity

11

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	read()	0	71	11

<?php

namespace FamilyTree365\LaravelGedcom\Utils\Exporter\Indi;

use FamilyTree365\LaravelGedcom\Models\PersonName;

class Name
{
    /**
     * \Gedcom\Record\Indi\Asso $asso
     * String $group
     * Integer $group_id.
     */
    public static function read($conn, \Gedcom\Record\Indi\Name $item, $group = '', $group_id = 0)
    {
        $name = $item->getName();
        $type = $item->getType();
        $npfx = $item->getNpfx();
        $givn = $item->getGivn();
        $nick = $item->getNick();
        $spfx = $item->getSpfx();
        $surn = $item->getSurn();
        $nsfx = $item->getNsfx();

        // store asso
        $key = [
            'group'=> $group,
            'gid'  => $group_id,
            'type' => $type,
            'name' => $name,
            'npfx' => $npfx,
            'givn' => $givn,
            'nick' => $nick,
            'spfx' => $spfx,
            'surn' => $surn,
            'nsfx' => $nsfx,
        ];
        $data = [
            'group'=> $group,
            'gid'  => $group_id,
            'type' => $type,
            'name' => $name,
            'npfx' => $npfx,
            'givn' => $givn,
            'nick' => $nick,
            'spfx' => $spfx,
            'surn' => $surn,
            'nsfx' => $nsfx,
        ];

        $record = PersonName::on($conn)->updateOrCreate($key, $data);

        $_group = 'indi_name';
        $_gid = $record->id;
        // store Note
        $note = $item->getNote();
        if ($note && count($note) > 0) {

It seems like $note can also be of type Gedcom\Record\Indi\Name; however, parameter $value of count() does only seem to accept Countable|array, maybe add an additional type check?

            foreach ($note as $_item) {
                if ($_item) {
                    \FamilyTree365\LaravelGedcom\Utils\Importer\NoteRef::read($conn, $_item, $_group, $_gid);
                }
            }
        }

        // store sourref
        $sour = $item->getSour();
        if ($sour && count($sour) > 0) {
            foreach ($sour as $_item) {
                if ($_item) {
                    \FamilyTree365\LaravelGedcom\Utils\Importer\SourRef::read($conn, $_item, $_group, $_gid);
                }
            }
        }

        // store fone
        $fone = $item->getFone();
        if ($fone != null) {
            \FamilyTree365\LaravelGedcom\Utils\Importer\Indi\Name\Fone::read($conn, $fone, $_group, $_gid);

It seems like $fone can also be of type Gedcom\Record\Indi\Name; however, parameter $item of FamilyTree365\LaravelGedcom\Utils\Importer\Indi\Name\Fone::read() does only seem to accept Gedcom\Record\Indi\Name\Fone, maybe add an additional type check?

        }

        // store romn
        $romn = $item->getRomn();
        if ($romn != null) {
            \FamilyTree365\LaravelGedcom\Utils\Importer\Indi\Name\Romn::read($conn, $romn, $_group, $_gid);

It seems like $romn can also be of type Gedcom\Record\Indi\Name; however, parameter $item of FamilyTree365\LaravelGedcom\Utils\Importer\Indi\Name\Romn::read() does only seem to accept Gedcom\Record\Indi\Name\Romn, maybe add an additional type check?

        }
    }
}