Gedcom\Writer\Head\Sour\Data

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	convert()	0	26	4

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

namespace Gedcom\Writer\Head\Sour;

class Data
{
    /**
     * @param string $format
     * @param int    $level
     *
     * @return string
     */
    public static function convert(\Gedcom\Record\Head\Sour\Data &$data, $level)
    {
        $output = '';
        $_data = $data->getData();

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

        if ($_data) {
            $output .= $level.' DATA '.$_data."\n";

Are you sure $_data of type Gedcom\Record\Head\Sour\Data|mixed can be used in concatenation?

        } else {
            return $output;
        }

        // level up
        $level++;

        // DATE
        $date = $corp->getDate();

The variable $corp seems to be never defined.

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

        // COPR
        $corp = $corp->getCorp();
        if ($corp) {
            $output .= $level.' COPR '.$corp."\n";
        }

        return $output;
    }
}