CreatesNewPageSourceFile::fileName() - Code Metrics - Inspection of "Merge pull request #1261 from hydephp/remove-depre..." - hydephp/develop - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 721b09...b6aafe )

by Caen

created 2023-03-12 21:11 UTC

CreatesNewPageSourceFile::fileName() A

↳ Parent: CreatesNewPageSourceFile

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	3
c	0
b	0
f	0
nc	2
nop	1
dl	0
loc	10
rs	10

<?php

declare(strict_types=1);

namespace Hyde\Framework\Actions;

use Hyde\Hyde;
use Hyde\Pages\BladePage;
use Illuminate\Support\Str;
use Hyde\Pages\MarkdownPage;
use Hyde\Pages\DocumentationPage;
use Hyde\Framework\Exceptions\FileConflictException;
use Hyde\Framework\Concerns\InteractsWithDirectories;
use Hyde\Framework\Exceptions\UnsupportedPageTypeException;
use function file_put_contents;
use function file_exists;
use function basename;
use function in_array;
use function unslash;
use function rtrim;

/**
 * Scaffold a new Markdown, Blade, or documentation page.
 *
 * @see \Hyde\Framework\Testing\Feature\Actions\CreatesNewPageSourceFileTest
 */
class CreatesNewPageSourceFile
{
    use InteractsWithDirectories;

    /** @var class-string<\Hyde\Pages\Concerns\HydePage> */

    protected string $pageClass;

    protected string $title;
    protected string $filename;
    protected string $outputPath;
    protected string $subDir = '';
    protected bool $force;

    public function __construct(string $title, string $pageClass = MarkdownPage::class, bool $force = false)
    {
        $this->validateType($pageClass);
        $this->pageClass = $pageClass;

        $this->title = $this->parseTitle($title);
        $this->filename = $this->fileName($title);
        $this->force = $force;

        $this->outputPath = $this->makeOutputPath($pageClass);
    }

    public function save(): string
    {
        $this->failIfFileCannotBeSaved($this->outputPath);

        match ($this->pageClass) {
            BladePage::class => $this->createBladeFile(),
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            MarkdownPage::class => $this->createMarkdownFile(),
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
            DocumentationPage::class => $this->createDocumentationFile(),
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
        };

        return $this->outputPath;
    }

    protected function parseTitle(string $title): string
    {
        return Str::afterLast($title, '/');
    }

    protected function fileName(string $title): string
    {
        // If title contains a slash, it's a subdirectory
        if (str_contains($title, '/')) {
            // So we normalize the subdirectory name
            $this->subDir = $this->normalizeSubdirectory($title);
        }

        // And return a slug made from just the title without the subdirectory
        return Str::slug(basename($title));
    }

    protected function normalizeSubdirectory(string $title): string
    {
        return unslash('/'.rtrim(Str::beforeLast($title, '/').'/', '/\\'));
    }

    /** @param class-string<\Hyde\Pages\Concerns\HydePage> $pageClass */

    protected function makeOutputPath(string $pageClass): string
    {
        return Hyde::path($pageClass::sourcePath($this->formatIdentifier()));
    }

    protected function createBladeFile(): void
    {
        $this->needsParentDirectory($this->outputPath);

        file_put_contents($this->outputPath, Hyde::normalizeNewlines(<<<BLADE
            @extends('hyde::layouts.app')
            @section('content')
            @php(\$title = "$this->title")
            
            <main class="mx-auto max-w-7xl py-16 px-8">
                <h1 class="text-center text-3xl font-bold">$this->title</h1>
            </main>
            
            @endsection

            BLADE
        ));
    }

    protected function createMarkdownFile(): void
    {
        (new MarkdownPage($this->formatIdentifier(), ['title' => $this->title], "# $this->title"))->save();
    }

    protected function createDocumentationFile(): void
    {
        (new DocumentationPage($this->formatIdentifier(), [], "# $this->title"))->save();
    }

    protected function formatIdentifier(): string
    {
        return "$this->subDir/$this->filename";
    }

    protected function validateType(string $pageClass): void
    {
        if (! in_array($pageClass, [MarkdownPage::class, BladePage::class, DocumentationPage::class])) {
            throw new UnsupportedPageTypeException('The page type must be either "markdown", "blade", or "documentation"');
        }
    }

    protected function failIfFileCannotBeSaved(string $path): void
    {
        if ($this->force !== true && file_exists($path)) {
            throw new FileConflictException($path);
        }
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Hyde\Framework\Actions;
6
7			use Hyde\Hyde;
8			use Hyde\Pages\BladePage;
9			use Illuminate\Support\Str;
10			use Hyde\Pages\MarkdownPage;
11			use Hyde\Pages\DocumentationPage;
12			use Hyde\Framework\Exceptions\FileConflictException;
13			use Hyde\Framework\Concerns\InteractsWithDirectories;
14			use Hyde\Framework\Exceptions\UnsupportedPageTypeException;
15			use function file_put_contents;
16			use function file_exists;
17			use function basename;
18			use function in_array;
19			use function unslash;
20			use function rtrim;
21
22			/**
23			* Scaffold a new Markdown, Blade, or documentation page.
24			*
25			* @see \Hyde\Framework\Testing\Feature\Actions\CreatesNewPageSourceFileTest
26			*/
27			class CreatesNewPageSourceFile
28			{
29			use InteractsWithDirectories;
30
31			/** @var class-string<\Hyde\Pages\Concerns\HydePage> */
			0 ignored issues – show Documentation Bug introduced 2023-02-21 09:29 UTC by Report Bug Copy Issue Report The doc comment `class-string<\Hyde\Pages\Concerns\HydePage>` at position `0` could not be parsed: Unknown type name 'class-string' at position 0 in class-string<\Hyde\Pages\Concerns\HydePage>. Loading history...
32			protected string $pageClass;
33
34			protected string $title;
35			protected string $filename;
36			protected string $outputPath;
37			protected string $subDir = '';
38			protected bool $force;
39
40			public function __construct(string $title, string $pageClass = MarkdownPage::class, bool $force = false)
41			{
42			$this->validateType($pageClass);
43			$this->pageClass = $pageClass;
44
45			$this->title = $this->parseTitle($title);
46			$this->filename = $this->fileName($title);
47			$this->force = $force;
48
49			$this->outputPath = $this->makeOutputPath($pageClass);
50			}
51
52			public function save(): string
53			{
54			$this->failIfFileCannotBeSaved($this->outputPath);
55
56			match ($this->pageClass) {
57			BladePage::class => $this->createBladeFile(),
			0 ignored issues – show Bug introduced 2022-11-03 18:26 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->createBladeFile()` targeting `Hyde\Framework\Actions\C...File::createBladeFile()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
58			MarkdownPage::class => $this->createMarkdownFile(),
			0 ignored issues – show Bug introduced 2022-11-03 18:26 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->createMarkdownFile()` targeting `Hyde\Framework\Actions\C...e::createMarkdownFile()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
59			DocumentationPage::class => $this->createDocumentationFile(),
			0 ignored issues – show Bug introduced 2022-11-03 18:26 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->createDocumentationFile()` targeting `Hyde\Framework\Actions\C...eateDocumentationFile()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
60			};
61
62			return $this->outputPath;
63			}
64
65			protected function parseTitle(string $title): string
66			{
67			return Str::afterLast($title, '/');
68			}
69
70			protected function fileName(string $title): string
71			{
72			// If title contains a slash, it's a subdirectory
73			if (str_contains($title, '/')) {
74			// So we normalize the subdirectory name
75			$this->subDir = $this->normalizeSubdirectory($title);
76			}
77
78			// And return a slug made from just the title without the subdirectory
79			return Str::slug(basename($title));
80			}
81
82			protected function normalizeSubdirectory(string $title): string
83			{
84			return unslash('/'.rtrim(Str::beforeLast($title, '/').'/', '/\\'));
85			}
86
87			/** @param class-string<\Hyde\Pages\Concerns\HydePage> $pageClass */
			0 ignored issues – show Documentation Bug introduced 2022-11-09 20:30 UTC by Report Bug Copy Issue Report The doc comment `class-string<\Hyde\Pages\Concerns\HydePage>` at position `0` could not be parsed: Unknown type name 'class-string' at position 0 in class-string<\Hyde\Pages\Concerns\HydePage>. Loading history...
88			protected function makeOutputPath(string $pageClass): string
89			{
90			return Hyde::path($pageClass::sourcePath($this->formatIdentifier()));
91			}
92
93			protected function createBladeFile(): void
94			{
95			$this->needsParentDirectory($this->outputPath);
96
97			file_put_contents($this->outputPath, Hyde::normalizeNewlines(<<<BLADE
98			@extends('hyde::layouts.app')
99			@section('content')
100			@php(\$title = "$this->title")
101
102			<main class="mx-auto max-w-7xl py-16 px-8">
103			<h1 class="text-center text-3xl font-bold">$this->title</h1>
104			</main>
105
106			@endsection
107
108			BLADE
109			));
110			}
111
112			protected function createMarkdownFile(): void
113			{
114			(new MarkdownPage($this->formatIdentifier(), ['title' => $this->title], "# $this->title"))->save();
115			}
116
117			protected function createDocumentationFile(): void
118			{
119			(new DocumentationPage($this->formatIdentifier(), [], "# $this->title"))->save();
120			}
121
122			protected function formatIdentifier(): string
123			{
124			return "$this->subDir/$this->filename";
125			}
126
127			protected function validateType(string $pageClass): void
128			{
129			if (! in_array($pageClass, [MarkdownPage::class, BladePage::class, DocumentationPage::class])) {
130			throw new UnsupportedPageTypeException('The page type must be either "markdown", "blade", or "documentation"');
131			}
132			}
133
134			protected function failIfFileCannotBeSaved(string $path): void
135			{
136			if ($this->force !== true && file_exists($path)) {
137			throw new FileConflictException($path);
138			}
139			}
140			}
141

hydephp / develop

Push — master ( 721b09...b6aafe )

CreatesNewPageSourceFile::fileName() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like