PhpGedcom\Writer\Addr

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	convert()	0	20	2

<?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
 * @package         php-gedcom 
 * @license         MIT
 * @link            http://github.com/mrkrstphr/php-gedcom
 */

namespace PhpGedcom\Writer;

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

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

The constant PhpGedcom\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";
        }

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

The property adr1 is declared protected in PhpGedcom\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";
        
        return $output;
    }
}