Pronamic\WordPress\Pay\Gateways\IDealAdvancedV3\XML\DirectoryParser::parse()

Complexity

Conditions	3
Paths	4

Size

Total Lines	18
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

0

<?php
	public static function parse( SimpleXMLElement $xml ) {
		$directory = new Directory();

		// Date.
		$timestamp = Security::filter( $xml->directoryDateTimestamp );

		if ( null !== $timestamp ) {
			$directory->set_date( new DateTime( $timestamp ) );
		}

		// Country.
		foreach ( $xml->Country as $element ) {
			$country = CountryParser::parse( $element );

It seems like $element can also be of type null; however, parameter $xml of Pronamic\WordPress\Pay\Gateways\IDealAdvancedV3\XML\CountryParser::parse() does only seem to accept SimpleXMLElement, maybe add an additional type check?

			$directory->add_country( $country );
		}

		return $directory;
	}