FixedQuotation::quoteHtml() - Code Metrics - Inspection of "create reply message" - genkgo/mail - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#50)

by Frederik

created 2018-04-02 20:05 UTC

FixedQuotation::quoteHtml() B

↳ Parent: FixedQuotation

Complexity

Conditions	7
Paths	10

Size

Total Lines	55
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	55
rs	7.8235
c	0
b	0
f	0
cc	7
eloc	37
nc	10
nop	3

How to fix Long Method

<?php
declare(strict_types=1);

namespace Genkgo\Mail\Quotation;

use Genkgo\Mail\Address;
use Genkgo\Mail\AlternativeText;
use Genkgo\Mail\MessageBodyCollection;
use Genkgo\Mail\MessageInterface;
use Genkgo\Mail\QuotationInterface;

final class FixedQuotation implements QuotationInterface
{
    /**
     * @var string
     */
    private $headerText;

    /**
     * @param string $headerText
     */
    public function __construct(string $headerText = '%s (%s):')
    {
        $this->headerText = $headerText;
    }

    /**
     * @param MessageBodyCollection $body
     * @param MessageInterface $originalMessage
     * @return MessageBodyCollection
     * @throws \DOMException
     */
    public function quote(MessageBodyCollection $body, MessageInterface $originalMessage): MessageBodyCollection
    {
        $originalBody = MessageBodyCollection::extract($originalMessage);
        $dateString = 'unknown';
        foreach ($originalMessage->getHeader('Date') as $header) {
            try {
                $date = new \DateTimeImmutable($header->getValue()->getRaw());
                $dateString = \IntlDateFormatter::create(
                    \Locale::getDefault(),
                    \IntlDateFormatter::MEDIUM,
                    \IntlDateFormatter::MEDIUM
                )->format($date);
            } catch (\Exception $e) {

            }
        }

        $fromString = '';
        foreach ($originalMessage->getHeader('From') as $header) {
            $from = Address::fromString($header->getValue()->getRaw());
            $fromString = $from->getName() ? $from->getName() : (string)$from->getAddress();
        }

        $headerText = \sprintf($this->headerText, $fromString, $dateString);

        return $body
            ->withHtmlAndNoGeneratedAlternativeText(
                $this->quoteHtml(
                    $body->getHtml(),
                    $originalBody->getHtml(),
                    $headerText
                )
            )
            ->withAlternativeText(
                $this->quoteText(
                    $body->getText(),
                    $originalBody->getText(),
                    $headerText
                )
            );
    }

    /**
     * @param string $newHtml
     * @param string $originalHtml
     * @param string $headerText
     * @return string
     * @throws \DOMException
     */
    private function quoteHtml(string $newHtml, string $originalHtml, string $headerText): string
    {
        $originalHtml = \trim($originalHtml);
        if ($originalHtml === '') {
            return '';
        }

        $document = new \DOMDocument();
        $document->substituteEntities = false;
        $document->resolveExternals = false;

        $result = @$document->loadHTML($originalHtml);
        if ($result === false) {
            throw new \DOMException('Incorrect HTML');
        }

        $query = new \DOMXPath($document);
        $removeItems = $query->query('//head|//script|//body/@style|//html/@style', $document->documentElement);
        /** @var \DOMElement $removeItem */
        foreach ($removeItems as $removeItem) {
            $parent = $removeItem->parentNode;
            $parent->removeChild($removeItem);
        }

        $body = $document->getElementsByTagName('body');
        $quote = $document->createElement('blockquote');
        $quote->setAttribute('type', 'cite');

        if ($body->length === 0) {
            $quote->appendChild($document->removeChild($document->documentElement));
        } else {
            $root = $body->item(0);
            while ($root->childNodes->length !== 0) {
                $quote->appendChild($root->childNodes->item(0));
            }
        }

        $newDocument = new \DOMDocument();
        $newDocument->substituteEntities = false;
        $newDocument->resolveExternals = false;
        $result = @$newDocument->loadHTML($newHtml, LIBXML_HTML_NOIMPLIED | LIBXML_HTML_NODEFDTD);
        if ($result === false) {
            throw new \DOMException('Incorrect HTML');
        }

        $quotedNode = $newDocument->importNode($quote, true);
        $newBody = $this->prepareBody($newDocument);
        $newBody->appendChild($quotedNode);

        $header = $newDocument->createElement('p');
        $header->textContent = $headerText;

        $quotedNode->parentNode->insertBefore($header, $quotedNode);
        return \trim($newDocument->saveHTML());
    }

    /**
     * @param \DOMDocument $document
     * @return \DOMElement
     */
    private function prepareBody(\DOMDocument $document): \DOMElement
    {
        $bodyList = $document->getElementsByTagName('body');
        if ($bodyList->length === 0) {
            $html = $document->createElement('html');
            $body = $document->createElement('body');
            $html->appendChild($body);
            $body->appendChild($document->documentElement);
            $document->removeChild($document->documentElement);
            $document->appendChild($html);
            return $body;
        }

        $body = $bodyList->item(0);

        $queryHtml = new \DOMXPath($document);
        $htmlTags = $queryHtml->query('//html');
        if ($htmlTags->length > 0) {
            $html = $htmlTags->item(0);
            $html->appendChild($body);
            $document->removeChild($document->documentElement);
            $document->appendChild($html);
            return $body;
        }

        $html = $document->createElement('html');
        $html->appendChild($body);
        $document->removeChild($document->documentElement);
        $document->appendChild($html);
        return $body;
    }

    /**
     * @param AlternativeText $newText
     * @param AlternativeText $originalText
     * @param string $headerText
     * @return AlternativeText
     */
    private function quoteText(
        AlternativeText $newText,
        AlternativeText $originalText,
        string $headerText
    ): AlternativeText {
        return new AlternativeText(
            \sprintf(
                "%s\n\n%s\n>%s",
                (string)$newText,
                $headerText,
                \str_replace("\n", "\n>", $originalText->getRaw())
            )
        );
    }
}


1			<?php
2			declare(strict_types=1);
3
4			namespace Genkgo\Mail\Quotation;
5
6			use Genkgo\Mail\Address;
7			use Genkgo\Mail\AlternativeText;
8			use Genkgo\Mail\MessageBodyCollection;
9			use Genkgo\Mail\MessageInterface;
10			use Genkgo\Mail\QuotationInterface;
11
12			final class FixedQuotation implements QuotationInterface
13			{
14			/**
15			* @var string
16			*/
17			private $headerText;
18
19			/**
20			* @param string $headerText
21			*/
22			public function __construct(string $headerText = '%s (%s):')
23			{
24			$this->headerText = $headerText;
25			}
26
27			/**
28			* @param MessageBodyCollection $body
29			* @param MessageInterface $originalMessage
30			* @return MessageBodyCollection
31			* @throws \DOMException
32			*/
33			public function quote(MessageBodyCollection $body, MessageInterface $originalMessage): MessageBodyCollection
34			{
35			$originalBody = MessageBodyCollection::extract($originalMessage);
36			$dateString = 'unknown';
37			foreach ($originalMessage->getHeader('Date') as $header) {
38			try {
39			$date = new \DateTimeImmutable($header->getValue()->getRaw());
40			$dateString = \IntlDateFormatter::create(
41			\Locale::getDefault(),
42			\IntlDateFormatter::MEDIUM,
43			\IntlDateFormatter::MEDIUM
44			)->format($date);
45			} catch (\Exception $e) {
			0 ignored issues – show Coding Style Comprehensibility introduced 2018-03-09 17:15 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
46			}
47			}
48
49			$fromString = '';
50			foreach ($originalMessage->getHeader('From') as $header) {
51			$from = Address::fromString($header->getValue()->getRaw());
52			$fromString = $from->getName() ? $from->getName() : (string)$from->getAddress();
53			}
54
55			$headerText = \sprintf($this->headerText, $fromString, $dateString);
56
57			return $body
58			->withHtmlAndNoGeneratedAlternativeText(
59			$this->quoteHtml(
60			$body->getHtml(),
61			$originalBody->getHtml(),
62			$headerText
63			)
64			)
65			->withAlternativeText(
66			$this->quoteText(
67			$body->getText(),
68			$originalBody->getText(),
69			$headerText
70			)
71			);
72			}
73
74			/**
75			* @param string $newHtml
76			* @param string $originalHtml
77			* @param string $headerText
78			* @return string
79			* @throws \DOMException
80			*/
81			private function quoteHtml(string $newHtml, string $originalHtml, string $headerText): string
82			{
83			$originalHtml = \trim($originalHtml);
84			if ($originalHtml === '') {
85			return '';
86			}
87
88			$document = new \DOMDocument();
89			$document->substituteEntities = false;
90			$document->resolveExternals = false;
91
92			$result = @$document->loadHTML($originalHtml);
93			if ($result === false) {
94			throw new \DOMException('Incorrect HTML');
95			}
96
97			$query = new \DOMXPath($document);
98			$removeItems = $query->query('//head\|//script\|//body/@style\|//html/@style', $document->documentElement);
99			/** @var \DOMElement $removeItem */
100			foreach ($removeItems as $removeItem) {
101			$parent = $removeItem->parentNode;
102			$parent->removeChild($removeItem);
103			}
104
105			$body = $document->getElementsByTagName('body');
106			$quote = $document->createElement('blockquote');
107			$quote->setAttribute('type', 'cite');
108
109			if ($body->length === 0) {
110			$quote->appendChild($document->removeChild($document->documentElement));
111			} else {
112			$root = $body->item(0);
113			while ($root->childNodes->length !== 0) {
114			$quote->appendChild($root->childNodes->item(0));
115			}
116			}
117
118			$newDocument = new \DOMDocument();
119			$newDocument->substituteEntities = false;
120			$newDocument->resolveExternals = false;
121			$result = @$newDocument->loadHTML($newHtml, LIBXML_HTML_NOIMPLIED \| LIBXML_HTML_NODEFDTD);
122			if ($result === false) {
123			throw new \DOMException('Incorrect HTML');
124			}
125
126			$quotedNode = $newDocument->importNode($quote, true);
127			$newBody = $this->prepareBody($newDocument);
128			$newBody->appendChild($quotedNode);
129
130			$header = $newDocument->createElement('p');
131			$header->textContent = $headerText;
132
133			$quotedNode->parentNode->insertBefore($header, $quotedNode);
134			return \trim($newDocument->saveHTML());
135			}
136
137			/**
138			* @param \DOMDocument $document
139			* @return \DOMElement
140			*/
141			private function prepareBody(\DOMDocument $document): \DOMElement
142			{
143			$bodyList = $document->getElementsByTagName('body');
144			if ($bodyList->length === 0) {
145			$html = $document->createElement('html');
146			$body = $document->createElement('body');
147			$html->appendChild($body);
148			$body->appendChild($document->documentElement);
149			$document->removeChild($document->documentElement);
150			$document->appendChild($html);
151			return $body;
152			}
153
154			$body = $bodyList->item(0);
155
156			$queryHtml = new \DOMXPath($document);
157			$htmlTags = $queryHtml->query('//html');
158			if ($htmlTags->length > 0) {
159			$html = $htmlTags->item(0);
160			$html->appendChild($body);
161			$document->removeChild($document->documentElement);
162			$document->appendChild($html);
163			return $body;
164			}
165
166			$html = $document->createElement('html');
167			$html->appendChild($body);
168			$document->removeChild($document->documentElement);
169			$document->appendChild($html);
170			return $body;
171			}
172
173			/**
174			* @param AlternativeText $newText
175			* @param AlternativeText $originalText
176			* @param string $headerText
177			* @return AlternativeText
178			*/
179			private function quoteText(
180			AlternativeText $newText,
181			AlternativeText $originalText,
182			string $headerText
183			): AlternativeText {
184			return new AlternativeText(
185			\sprintf(
186			"%s\n\n%s\n>%s",
187			(string)$newText,
188			$headerText,
189			\str_replace("\n", "\n>", $originalText->getRaw())
190			)
191			);
192			}
193			}
194

genkgo / mail

Pull Request — master (#50)

FixedQuotation::quoteHtml() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like