AbstractDocAboutMenuPage::getContentToPrint() - Code Metrics - Inspection of "Sanitize filename" - GraphQLAPI/graphql-api-for-wp - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 90d9fc...295a38 )

by Leonardo

created 2020-09-21 15:29 UTC

AbstractDocAboutMenuPage::getContentToPrint() A

↳ Parent: AbstractDocAboutMenuPage

Complexity

Conditions	2
Paths	2

Size

Total Lines	21
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	16
c	1
b	0
f	0
nc	2
nop	0
dl	0
loc	21
rs	9.7333

<?php

declare(strict_types=1);

namespace GraphQLAPI\GraphQLAPI\Admin\MenuPages;

use InvalidArgumentException;
use GraphQLAPI\GraphQLAPI\General\RequestParams;
use GraphQLAPI\GraphQLAPI\Facades\ContentProcessors\MarkdownContentParserFacade;

/**
 * Open documentation within the About page
 */
abstract class AbstractDocAboutMenuPage extends AbstractDocsMenuPage
{
    protected function openInModalWindow(): bool
    {
        return true;
    }

    /**
     * Enable "/" in the filename
     *
     * @param string[] $specialChars
     * @return string[]
     */
    public function enableSpecialCharsForSanitization(array $specialChars): array
    {
        return array_diff(
            $specialChars,
            [
                '/',
            ]
        );
    }

    protected function getContentToPrint(): string
    {
        // Enable "/" in the filename
        add_filter(
            'sanitize_file_name_chars',
            [$this, 'enableSpecialCharsForSanitization']
        );
        $doc = \sanitize_file_name($_REQUEST[RequestParams::DOC]) . '.md';
        remove_filter(
            'sanitize_file_name_chars',
            [$this, 'enableSpecialCharsForSanitization']
        );
        $markdownContentParser = MarkdownContentParserFacade::getInstance();
        try {
            return $markdownContentParser->getContent($doc);
        } catch (InvalidArgumentException $e) {
            return sprintf(
                '<p>%s</p>',
                sprintf(
                    \__('Page \'%s\' does not exist', 'graphql-api'),
                    $doc
                )
            );
        }
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace GraphQLAPI\GraphQLAPI\Admin\MenuPages;
6
7			use InvalidArgumentException;
8			use GraphQLAPI\GraphQLAPI\General\RequestParams;
9			use GraphQLAPI\GraphQLAPI\Facades\ContentProcessors\MarkdownContentParserFacade;
10
11			/**
12			* Open documentation within the About page
13			*/
14			abstract class AbstractDocAboutMenuPage extends AbstractDocsMenuPage
15			{
16			protected function openInModalWindow(): bool
17			{
18			return true;
19			}
20
21			/**
22			* Enable "/" in the filename
23			*
24			* @param string[] $specialChars
25			* @return string[]
26			*/
27			public function enableSpecialCharsForSanitization(array $specialChars): array
28			{
29			return array_diff(
30			$specialChars,
31			[
32			'/',
33			]
34			);
35			}
36
37			protected function getContentToPrint(): string
38			{
39			// Enable "/" in the filename
40			add_filter(
41			'sanitize_file_name_chars',
42			[$this, 'enableSpecialCharsForSanitization']
43			);
44			$doc = \sanitize_file_name($_REQUEST[RequestParams::DOC]) . '.md';
45			remove_filter(
46			'sanitize_file_name_chars',
47			[$this, 'enableSpecialCharsForSanitization']
48			);
49			$markdownContentParser = MarkdownContentParserFacade::getInstance();
50			try {
51			return $markdownContentParser->getContent($doc);
52			} catch (InvalidArgumentException $e) {
53			return sprintf(
54			'<p>%s</p>',
55			sprintf(
56			\__('Page \'%s\' does not exist', 'graphql-api'),
57			$doc
58			)
59			);
60			}
61			}
62			}
63

GraphQLAPI / graphql-api-for-wp

GitHub Access Token became invalid

Push — master ( 90d9fc...295a38 )

AbstractDocAboutMenuPage::getContentToPrint() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like