Transphporm\Formatter\Nl2br

Complexity

Total Complexity

7

Size/Duplication

Total Lines	35
Duplicated Lines	28.57 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	nl2br()	0	17	3
A	getContent()	10	10	3
A	getClonedElement()	0	4	1

<?php
namespace Transphporm\Formatter;
class Nl2br {
    public function nl2br($var) {

Please use __construct() instead of a PHP4-style constructor that is named after the class.

        $parts = explode("\n", $var);
        $doc = new \DomDocument();
        $root = $doc->createElement('root');
        $doc->appendChild($root);

        foreach ($parts as $key => $part) {
            $new = $doc->createTextNode($part);
            $doc->documentElement->appendChild($new);
            if ($key !== count($parts)-1) {
                $br = $doc->createElement('br');
                $doc->documentElement->appendChild($br);
            }
        }

        return $this->getContent($doc);
    }

    private function getContent($document) {

This method seems to be duplicated in your project.

		$newNode = $document->documentElement;
		$result = [];
		if ($newNode->tagName === 'root') {
			foreach ($newNode->childNodes as $node) {
				$result[] = $this->getClonedElement($node);
			}
		}
		return $result;
	}

	private function getClonedElement($node) {
		$clone = $node->cloneNode(true);
		return $clone;
	}
}