PhpGedcom\Writer\Subm

Complexity

Total Complexity

21

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
F	convert()	0	85	21

<?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;

/**
 *
 */
class Subm
{
    /**
     * @param \PhpGedcom\Record\Subm $note
     * @param int $level
     * @return string
     */
    public static function convert(\PhpGedcom\Record\Subm &$subm)
    {
        $level = 0;
        $output = "";
        $_subm = $subm->getSubn();

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

        if(empty($_subm)){
            return $output;
        }else{
            $output.=$level." ".$_subm." SUBM "."\n";

Are you sure $_subm of type PhpGedcom\Record\Subm|mixed can be used in concatenation?

        }
        // level up
        $level++;

        // NAME
        $name = $subm->getName();
        if(!empty($name)){
            $output.=$level." NAME ".$name."\n";
        }
        // $chan
        $chan = $subm->getChan();
        if($chan){

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

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

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

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

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

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

        // $rfn
        $rfn = $subm->getRfn();
        if(!empty($rfn)){
            $output.=$level." RFN ".$rfn."\n";
        }

        // $lang = array()
        $langs = $subm->getLang();
        if(!empty($langs) && count($langs) > 0){
            foreach($langs as $item){
                if($item){
                    $_convert = $level." LANG ".$item."\n";
                    $output.=$_convert;
                }
            }
        }

        // $phon = array()
        $phon = $subm->getLang();
        if(!empty($phon) && count($phon) > 0){
            foreach($phon as $item){
                if($item){
                    $_convert = \PhpGedcom\Writer\Phon::convert($item, $level);
                    $output.= $_convert;
                }
            }
        }

        // $obje = array()
        $obje = $subm->getObje();
        if(!empty($obje) && count($obje) > 0){
            foreach($obje as $item){
                $_convert = \PhpGedcom\Writer\ObjeRef::convert($item, $level);
                $output.=$_convert;
            }
        }

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