familytree365 / php-gedcom

src/Writer/Addr.php

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

class Addr
{
    /**
     * @param string $format
     * @param int    $level
     *
     * @return string
     */
    public static function convert(\Gedcom\Record\Addr &$addr, $format = self::GEDCOM55, $level = 1)

The parameter $format is not used and could be removed.

The constant Gedcom\Writer\Addr::GEDCOM55 was not found. Maybe you did not declare it correctly or list all dependencies?

    {
        $addrs = explode("\n", $addr->getAddr());

        $output = "{$level} ADDR ".$addrs[0]."\n";

        array_shift($addrs);

        foreach ($addrs as $cont) {
            $output .= ($level + 1).' CONT '.$cont."\n";
        }

        return $output.(($level + 1).' ADR1 '.$addr->adr1."\n".

The property adr1 is declared protected in Gedcom\Record\Addr and cannot be accessed from this context.

            ($level + 1).' ADR2 '.$addr->getAdr2()."\n".
            ($level + 1).' CITY '.$addr->getCity()."\n".
            ($level + 1).' STAE '.$addr->getStae()."\n".
            ($level + 1).' POST '.$addr->getPost()."\n".
            ($level + 1).' CTRY '.$addr->getCtry()."\n");
    }
}