PhpGedcom\Writer\Head\Char

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	convert()	0	20	3

<?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
 * @license         MIT
 *
 * @link            http://github.com/mrkrstphr/php-gedcom
 */

namespace PhpGedcom\Writer\Head;

class Char
{
    /**
     * @param \PhpGedcom\Record\Head\Char $char
     * @param string                      $format
     * @param int                         $level
     *
     * @return string
     */
    public static function convert(\PhpGedcom\Record\Head\Char &$char, $level)
    {
        $output = '';
        // char
        $_char = $char->getChar();

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

        if ($_char) {
            $output .= $level.' CHAR '.$_char."\n";

Are you sure $_char of type PhpGedcom\Record\Head\Char|mixed can be used in concatenation?

        } else {
            return $output;
        }

        // level up
        $level++;
        // VERS
        $vers = $char->getVersion();

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

        if ($vers) {
            $output .= $level.' VERS '.$vers."\n";

Are you sure $vers of type PhpGedcom\Record\Head\Char|mixed can be used in concatenation?

        }

        return $output;
    }
}