PhpGedcom\Writer\Indi\Famc

Complexity

Total Complexity

6

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	convert()	0	28	6

<?php
/**
 * php-gedcom
 *
 * php-gedcom is a library for parsing, manipulating, importing and exporting
 * GEDCOM 5.5 files in PHP 5.3+.
 *
 * @author          Xiang Ming <wenqiangliu344@gmail.com>
 * @copyright       Copyright (c) 2010-2013, Xiang Ming
 * @package         php-gedcom 
 * @license         MIT
 * @link            http://github.com/mrkrstphr/php-gedcom
 */

namespace PhpGedcom\Writer\Indi;

/**
 *
 */
class Famc
{
    /**
     * @param \PhpGedcom\Record\Indi\Famc $attr
     * @param int $level
     * @return string
     */
    public static function convert(\PhpGedcom\Record\Indi\Famc &$famc, $level = 0)
    {
        $output = '';
        // NAME
        $_famc = $famc->getFams();

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

        if(empty($_fams)){

The variable $_fams does not exist. Did you maybe mean $_famc?

            return $output;
        }
        $output.= $level." FAMC @".$_famc."@\n";

Are you sure $_famc of type PhpGedcom\Record\Indi\Famc|mixed|null can be used in concatenation?

        // level up
        $level++;
        
        // PEDI
        $pedi = $famc->getPedi();

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

        if(!empty($pedi)){
            $output.=$level." PEDI ".$pedi."\n";

Are you sure $pedi of type PhpGedcom\Record\Indi\Famc|mixed can be used in concatenation?

        }
        
        // note
        $note = $famc->getSour();

The method getSour() does not exist on PhpGedcom\Record\Indi\Famc. 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\Indi\Famc; 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;
            }
        }

        return $output;
    }
}