FamilyTree365\LaravelGedcom\Utils\Importer\Sour\Data::read()

Complexity

Conditions	7
Paths	6

Size

Total Lines	25
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function read($conn, \Gedcom\Record\Sour\Data $data, $group = '', $group_id = 0)
    {
        $date = $data->getDate();
        $agnc = $data->getAgnc();
        $text = $data->getText();

        // store Data of sources
        $key = ['group'=>$group, 'gid'=>$group_id, 'date'=>$date, 'text'=>$text, 'agnc'=>$agnc];
        $_data = ['group'=>$group, 'gid'=>$group_id, 'date'=>$date, 'text'=>$text, 'agnc'=>$agnc];
        $record = SourceData::on($conn)->updateOrCreate($key, $_data);

        $_group = 'sourcedata';
        $_gid = $record->id;
        // \Gedcom\Record\Sour\Data\Even array
        $even = $data->getEven();
        if ($even && count($even) > 0) {

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

            foreach ($even as $item) {
                Even::read($conn, $item, $_group, $_gid);
            }
        }
        // \Gedcom\Record\NoteRef array
        $note = $data->getNote();
        if ($note && count($note) > 0) {
            foreach ($note as $item) {
                NoteRef::read($conn, $item, $_group, $_gid);
            }