familytree365 / php-gedcom

src/Writer/SourRef.php

<?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 Gedcom\Writer;

class SourRef
{
    /**
     * @param int $level
     *
     * @return string
     */
    public static function convert(\Gedcom\Record\SourRef &$sour, $level)
    {
        $output = '';
        $_sour = $sour->getSour();

The method getSour() does not exist on Gedcom\Record\SourRef. Since you implemented __call, consider adding a @method annotation.

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

Are you sure $_sour of type Gedcom\Record\SourRef|mixed can be used in concatenation?

        }
        $level++;
        // protected $_text    = null;
        $_text = $sour->getText();

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

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

Are you sure $_text of type Gedcom\Record\SourRef|mixed can be used in concatenation?

        }
        // protected $_note    = array();
        $note = $sour->getNote();

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

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

It seems like $note can also be of type Gedcom\Record\SourRef; however, parameter $value of count() does only seem to accept Countable|array, maybe add an additional type check?

            foreach ($note as $item) {
                $_convert = \Gedcom\Writer\NoteRef::convert($item, $level);
                $output .= $_convert;
            }
        }
        // protected $_data    = null;
        $_data = $sour->getData();

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

        if ($_data) {
            $_convert = \Gedcom\Writer\Sour\Data::convert($_data, $level);

It seems like $_data can also be of type Gedcom\Record\SourRef; however, parameter $data of Gedcom\Writer\Sour\Data::convert() does only seem to accept Gedcom\Record\Sour\Data, maybe add an additional type check?

            $output .= $_convert;
        }
        // protected $_page setPage
        $_page = $sour->getPage();

The method getPage() does not exist on Gedcom\Record\SourRef. Since you implemented __call, consider adding a @method annotation.

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

Are you sure $_page of type Gedcom\Record\SourRef|mixed can be used in concatenation?

        }
        // protected $_even    = null;
        $_even = $sour->getData();
        if ($_even) {
            $_convert = \Gedcom\Writer\SourRef\Even::convert($_even, $level);

It seems like $_even can also be of type Gedcom\Record\SourRef; however, parameter $even of Gedcom\Writer\SourRef\Even::convert() does only seem to accept Gedcom\Record\SourRef\Even, maybe add an additional type check?

            $output .= $_convert;
        }
        // protected $_quay
        $_quay = $sour->getQuay();

The method getQuay() does not exist on Gedcom\Record\SourRef. Since you implemented __call, consider adding a @method annotation.

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

Are you sure $_quay of type Gedcom\Record\SourRef|mixed can be used in concatenation?

        }
        // protected $_obje    = array();
        // This is not defined in parser.
        return $output;
    }
}