PhpGedcom\Parser\Plac\Fone

Complexity

Total Complexity

5

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	parse()	0	39	5

<?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 PhpGedcom\Parser\Plac;

class Fone extends \PhpGedcom\Parser\Component
{
    public static function parse(\PhpGedcom\Parser $parser)
    {
        $record = $parser->getCurrentLineRecord();
        $depth = (int) $record[0];
        if (isset($record[2])) {
            $_fone = trim($record[2]);
        } else {
            $parser->skipToNextLevel($depth);

            return null;
        }

        $fone = new \PhpGedcom\Record\Plac\Fone();
        $fone->setPlac($_fone);

The method setPlac() does not exist on PhpGedcom\Record\Plac\Fone. Since you implemented __call, consider adding a @method annotation.

        $parser->forward();

        while (!$parser->eof()) {
            $record = $parser->getCurrentLineRecord();
            $currentDepth = (int) $record[0];
            $recordType = strtoupper(trim($record[1]));

            if ($currentDepth <= $depth) {
                $parser->back();
                break;
            }

            switch ($recordType) {
                case 'TYPE':
                    $fone->setType(trim($record[2]));

The method setType() does not exist on PhpGedcom\Record\Plac\Fone. Since you implemented __call, consider adding a @method annotation.

                    break;
                default:
                    $parser->logUnhandledRecord(get_class().' @ '.__LINE__);
            }

            $parser->forward();
        }

        return $fone;
    }
}