familytree365 / php-gedcom

src/Writer/Fam/Slgs.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\Fam;

class Slgs
{
    /**
     * @param int $level
     *
     * @return string
     */
    public static function convert(\Gedcom\Record\Fam\Slgs &$slgs, $level)
    {
        $output = '';
        $output .= $level." SLGS \n";

        // Level up
        $level++;

        // $STAT;
        $stat = $slgs->getStat();

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

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

Are you sure $stat of type Gedcom\Record\Fam\Slgs|mixed can be used in concatenation?

        }

        // $date;
        $date = $slgs->getDate();

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

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

Are you sure $date of type Gedcom\Record\Fam\Slgs|mixed can be used in concatenation?

        }

        // PLAC
        $plac = $slgs->getPlac();

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

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

Are you sure $plac of type Gedcom\Record\Fam\Slgs|mixed can be used in concatenation?

        }

        // $TEMP;
        $temp = $slgs->getTemp();

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

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

Are you sure $temp of type Gedcom\Record\Fam\Slgs|mixed can be used in concatenation?

        }

        // $sour = array();
        $sour = $slgs->getSour();

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

        if (!empty($sour) && count($sour) > 0) {

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

            foreach ($sour as $item) {
                $_convert = \Gedcom\Writer\SourRef::convert($item, $level);
                $output .= $_convert;
            }
        }
        // $note = array();
        $note = $slgs->getNote();

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

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

        return $output;
    }
}