Gedcom\Writer\Indi\Asso::convert()

Complexity

Conditions	9
Paths	9

Size

Total Lines	36
Code Lines	20

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function convert(\Gedcom\Record\Indi\Asso &$asso, $level = 0)
    {
        $output = '';
        // _indi
        $_indi = $asso->getIndi();

The method getIndi() does not exist on Gedcom\Record\Indi\Asso. Since you implemented __call, consider adding a @method annotation.

        if (empty($_indi)) {
            return $output;
        }
        $output .= $level.' ASSO '.$_indi."\n";

Are you sure $_indi of type Gedcom\Record\Indi\Asso|mixed can be used in concatenation?

        // level up
        $level++;

        // RELA
        $rela = $asso->getRela();

The method getRela() does not exist on Gedcom\Record\Indi\Asso. Since you implemented __call, consider adding a @method annotation.

        if (!empty($rela)) {
            $output .= $level.' RELA '.$rela."\n";

Are you sure $rela of type Gedcom\Record\Indi\Asso|mixed can be used in concatenation?

        }
        // sour
        $sour = $asso->getSour();

The method getSour() does not exist on Gedcom\Record\Indi\Asso. Since you implemented __call, consider adding a @method annotation.

        if (!empty($sour) && count($sour) > 0) {

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

            foreach ($sour as $item) {
                $_convert = \Gedcom\Writer\SourRef::convert($item, $level);
                $output .= $_convert;
            }
        }

        // note
        $note = $asso->getSour();
        if (!empty($note) && count($note) > 0) {
            foreach ($note as $item) {
                $_convert = \Gedcom\Writer\NoteRef::convert($item, $level);
                $output .= $_convert;
            }
        }

        return $output;
    }