Gedcom\Writer\Repo::convert()

Complexity

Conditions	15
Paths	129

Size

Total Lines	70
Code Lines	39

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public static function convert(\Gedcom\Record\Repo &$repo)
    {
        $level = 0;
        $output = '';
        $_repo = $repo->getRepo();
        if ($_repo) {
            $output .= $level.' '.$_repo." REPO\n";
        } else {
            return $output;
        }

        // level up
        $level++;

        //NAME
        $name = $repo->getName();
        if ($name) {
            $output .= $level.' NAME '.$name."\n";
        }

        // ADDR
        $addr = $repo->getAddr();
        if ($addr) {

$addr is of type Gedcom\Record\Addr, thus it always evaluated to true.

            $_convert = \Gedcom\Writer\Addr::convert($addr, $level);
            $output .= $_convert;
        }

        // PHON
        $phon = $repo->getPhon();
        if ($phon) {

The expression $phon of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $_convert = \Gedcom\Writer\Phon::convert($phon, $level);

$phon of type array is incompatible with the type string expected by parameter $phon of Gedcom\Writer\Phon::convert().

            $output .= $_convert;
        }

        // NOTE array
        $note = $repo->getNote();
        if ($note && count($note) > 0) {

The expression $note of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

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

        // REFN
        $refn = $repo->getRefn();
        if (!empty($refn) && count($refn) > 0) {
            foreach ($refn as $item) {
                if ($item) {
                    $_convert = \Gedcom\Writer\Refn::convert($item, $level);
                    $output .= $_convert;
                }
            }
        }

        // CHAN
        $chan = $repo->getChan();
        if ($chan) {

$chan is of type Gedcom\Record\Chan, thus it always evaluated to true.

            $_convert = \Gedcom\Writer\Chan::convert($chan, $level);
            $output .= $_convert;
        }

        // RIN
        $rin = $repo->getRin();
        if ($rin) {
            $output .= $level.' RIN '.$rin."\n";
        }

        return $output;
    }