Pronamic\WordPress\Pay\Gateways\IDealAdvancedV3\XML\CountryParser::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 ) {
		$country = new Country();

		// Name.
		$name = Security::filter( $xml->countryNames );

		if ( null !== $name ) {
			$country->set_name( $name );
		}

		// Issuers.
		foreach ( $xml->Issuer as $element ) {
			$issuer = IssuerParser::parse( $element );

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

			$country->add_issuer( $issuer );
		}

		return $country;
	}