OldSound\RabbitMqBundle\RabbitMq\Validator\XmlValidator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	17
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getContentType()	0	2	1
A	validate()	0	5	2
A	setSchema()	0	2	1

<?php

namespace OldSound\RabbitMqBundle\RabbitMq\Validator;

use DOMDocument;

class XmlValidator implements ValidatorInterface
{
    private $schema = null;

    public function setSchema($schema, $additionalProperties = array()) {
        $this->schema = $schema;
    }

    public function validate($msg)
    {
        $xml = new DOMDocument();
        $xml->load($msg);
        return $xml->schemaValidate($this->schema) == true ? null : "XML schema validation failed.";

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

    }

    public function getContentType() {
        return "application/xml";
    }
}