PhpGedcom\Writer\RepoRef::convert()

Complexity

Conditions	8
Paths	5

Size

Total Lines	31
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function convert(\PhpGedcom\Record\RepoRef &$reporef, $level)
    {
        $output = '';
        $_repo = $reporef->getRepo();

The method getRepo() does not exist on PhpGedcom\Record\RepoRef. Since you implemented __call, consider adding a @method annotation.

        if (empty($_sour)) {

The variable $_sour seems to never exist and therefore empty should always be true.

            return $output;
        } else {
            $output .= $level.' REPO '.$_repo."\n";

Are you sure $_repo of type PhpGedcom\Record\RepoRef|mixed|null can be used in concatenation?

        }
        // level up
        $level++;

        // Note array
        $note = $reporef->getNote();

The method getNote() does not exist on PhpGedcom\Record\RepoRef. Since you implemented __call, consider adding a @method annotation.

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

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

            foreach ($note as $item) {
                $_convert = \PhpGedcom\Writer\NoteRef::convert($item, $level);
                $output .= $_convert;
            }
        }

        // _caln array
        $_caln = $reporef->getCaln();

The method getCaln() does not exist on PhpGedcom\Record\RepoRef. Since you implemented __call, consider adding a @method annotation.

        if (!empty($_caln) && count($_caln) > 0) {
            foreach ($_caln as $item) {
                $_convert = \PhpGedcom\Writer\Caln::convert($item, $level);
                $output .= $_convert;
            }
        }

        return $output;
    }