PhpGedcom\Writer\Sour\Data

Complexity

Total Complexity

10

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	convert()	0	44	10

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

class Data
{
    /**
     * @param \PhpGedcom\Record\Sour\Data $data
     * @param int                         $level
     *
     * @return string
     */
    public static function convert(\PhpGedcom\Record\Sour\Data &$data, $level = 0)
    {
        $output = '';

The assignment to $output is dead and can be removed.

        $output = $level." DATA\n";
        $level++;

        // $_date;
        $date = $data->getDate();

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

        if (!empty($date)) {
            $output .= $level.' DATE '.$date."\n";

Are you sure $date of type PhpGedcom\Record\Sour\Data|mixed can be used in concatenation?

        }

        // $_agnc AGNC
        $_agnc = $data->getAgnc();

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

        if (!empty($_agnc)) {
            $output .= $level.' AGNC '.$_agnc."\n";

Are you sure $_agnc of type PhpGedcom\Record\Sour\Data|mixed can be used in concatenation?

        }

        // $_text
        $_text = $data->getText();

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

        if (!empty($_text)) {
            $output .= $level.' TEXT '.$_text."\n";

Are you sure $_text of type PhpGedcom\Record\Sour\Data|mixed can be used in concatenation?

        }

        // $_note
        $note = $data->getNote();

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

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

It seems like $note can also be of type PhpGedcom\Record\Sour\Data; 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;
            }
        }

        // $_even
        $_even = $data->getEven();

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

        if ($_even && count($_even) > 0) {
            foreach ($_even as $item) {
                $_convert = \PhpGedcom\Writer\Sour\Data\Even::convert($item, $level);
                $output .= $_convert;
            }
        }

        return $output;
    }
}