Pagemark::getContent() - Code Metrics - ryanwinchester/pagemark - Measure and Improve Code Quality continuously with Scrutinizer

Pagemark::getContent() B
last analyzed 2016-04-29 16:25 UTC

↳ Parent: Pagemark

Complexity

Conditions	8
Paths	16

Size

Total Lines	55
Code Lines	39

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	55
rs	7.4033
cc	8
eloc	39
nc	16
nop	2

How to fix Long Method

<?php

namespace Pagemark;

use Illuminate\Filesystem\Filesystem;
use Pagemark\Contracts\Parseable;

class Pagemark
{
    /**
     * @var Filesystem
     */
    private $filesystem;
    /**
     * @var Parseable
     */
    private $parser;

    /**
     * TODO: Not liking the filesystem dependency, so look at other options
     *
     * @param Filesystem $filesystem
     * @param Parseable $parser
     */
    public function __construct(Filesystem $filesystem, Parseable $parser)
    {
        $this->filesystem = $filesystem;
        $this->parser = $parser;
    }

    /**
     * Return a new pagemark instance.
     *
     * @param Parseable $parser
     * @return static
     */
    public static function create(Parseable $parser = null)
    {
        $parser = $parser ?: new Parser(new \Parsedown());

        return new static(new Filesystem(), $parser);
    }

    /**
     * Parse a post into an array of content elements
     * without needing to instantiate the class first.
     *
     * @param string $basePath
     * @param string $post
     * @return mixed
     */
    public static function parse($basePath, $post)
    {
        $pagemark = static::create();

        return $pagemark->getContent($basePath, $post);
    }

    /**
     * Parse a post into an array of content elements.
     *
     * TODO: Refactor
     *
     * @param string $basePath
     * @param string $post
     * @return array
     * @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
     */
    public function getContent($basePath, $post)
    {
        $breadcrumbs = $this->makeBreadcrumbsFromPost($post);
        $title = $this->makeTitleFromBreadcrumbs($breadcrumbs);

        $postPath = $basePath .($post? '/'. $post : '');
        $isDir = false;
        $index = [];

        if ($this->filesystem->isDirectory($postPath)) {
            $isDir = true;
            $index['subcategories'] = $this->filesystem->directories($postPath);
            $index['files'] = $this->filesystem->files($postPath);
            $postPath .= '/index';
        }

        $file = $postPath.'.md';
        if ($this->filesystem->exists($file)) {
            $post = $this->parser->parse($this->filesystem->get($file));
        } else {
            $post = '';
        }

        if ($isDir) {
            foreach ($index['subcategories'] as $i => $item) {
                $item = str_replace($basePath, '', $item);
                $paths = explode('/', $item);
                $name = array_pop($paths);
                $index['subcategories'][$i] = [
                    'href' => $item,
                    'name' => $this->deslugify($name),
                ];
            }
            foreach ($index['files'] as $i => $item) {
                $item = str_replace($basePath, '', $item);
                $item = str_replace('.md', '', $item);
                $paths = explode('/', $item);
                $name = array_pop($paths);
                $index['files'][$i] = [
                    'href' => $item,
                    'name' => $this->deslugify($name),
                ];
                if ($index['files'][$i]['name'] === 'index') {
                    unset($index['files'][$i]);
                }
            }
        }

        return [
            'title'       => $title,
            'breadcrumbs' => $breadcrumbs,
            'index'       => $index,
            'post'        => $post,
        ];
    }

    /**
     * Get the Post or Category title from the breadcrumbs array.
     *
     * @param $breadcrumbs
     * @return mixed
     */
    private function makeTitleFromBreadcrumbs($breadcrumbs)
    {
        $lastCrumb = current(array_slice($breadcrumbs, -1));

        return $lastCrumb['name'];
    }

    /**
     * Make an array of breadcrumb items.
     *
     * @param string $post
     * @return array
     */
    private function makeBreadcrumbsFromPost($post)
    {
        $crumbs = explode('/', $post);
        $crumbs = empty($crumbs[0]) ? [] : $crumbs;

        $breadcrumbs = [];

        foreach ($crumbs as $i => $crumb) {
            $breadcrumbs[$i] = [
                'href' => '/'.$this->slugify(implode('/', array_slice($crumbs, 0, $i+1))),
                'name' => $this->deslugify($crumbs[$i]),
            ];
        }

        return $breadcrumbs;
    }

    /**
     * Take a string with spaces and convert the spaces to the specified separator.
     *
     * @param string $string
     * @param string $separator
     * @return mixed
     */
    private function slugify($string, $separator = '-')
    {
        return str_replace(' ', $separator, $string);
    }

    /**
     * Take a string with dash or underscore word separators and convert the to spaces.
     *
     * @param array|string $slugged
     * @return array|string
     */
    private function deslugify($slugged)
    {
        if (is_array($slugged)) {
            return array_map('static::deslugify', $slugged);
        }

        return str_replace(['-', '_'], ' ', $slugged);
    }
}


1			<?php
2
3			namespace Pagemark;
4
5			use Illuminate\Filesystem\Filesystem;
6			use Pagemark\Contracts\Parseable;
7
8			class Pagemark
9			{
10			/**
11			* @var Filesystem
12			*/
13			private $filesystem;
14			/**
15			* @var Parseable
16			*/
17			private $parser;
18
19			/**
20			* TODO: Not liking the filesystem dependency, so look at other options
21			*
22			* @param Filesystem $filesystem
23			* @param Parseable $parser
24			*/
25			public function __construct(Filesystem $filesystem, Parseable $parser)
26			{
27			$this->filesystem = $filesystem;
28			$this->parser = $parser;
29			}
30
31			/**
32			* Return a new pagemark instance.
33			*
34			* @param Parseable $parser
35			* @return static
36			*/
37			public static function create(Parseable $parser = null)
38			{
39			$parser = $parser ?: new Parser(new \Parsedown());
40
41			return new static(new Filesystem(), $parser);
42			}
43
44			/**
45			* Parse a post into an array of content elements
46			* without needing to instantiate the class first.
47			*
48			* @param string $basePath
49			* @param string $post
50			* @return mixed
51			*/
52			public static function parse($basePath, $post)
53			{
54			$pagemark = static::create();
55
56			return $pagemark->getContent($basePath, $post);
57			}
58
59			/**
60			* Parse a post into an array of content elements.
61			*
62			* TODO: Refactor
63			*
64			* @param string $basePath
65			* @param string $post
66			* @return array
67			* @throws \Illuminate\Contracts\Filesystem\FileNotFoundException
68			*/
69			public function getContent($basePath, $post)
70			{
71			$breadcrumbs = $this->makeBreadcrumbsFromPost($post);
72			$title = $this->makeTitleFromBreadcrumbs($breadcrumbs);
73
74			$postPath = $basePath .($post? '/'. $post : '');
75			$isDir = false;
76			$index = [];
77
78			if ($this->filesystem->isDirectory($postPath)) {
79			$isDir = true;
80			$index['subcategories'] = $this->filesystem->directories($postPath);
81			$index['files'] = $this->filesystem->files($postPath);
82			$postPath .= '/index';
83			}
84
85			$file = $postPath.'.md';
86			if ($this->filesystem->exists($file)) {
87			$post = $this->parser->parse($this->filesystem->get($file));
88			} else {
89			$post = '';
90			}
91
92			if ($isDir) {
93			foreach ($index['subcategories'] as $i => $item) {
94			$item = str_replace($basePath, '', $item);
95			$paths = explode('/', $item);
96			$name = array_pop($paths);
97			$index['subcategories'][$i] = [
98			'href' => $item,
99			'name' => $this->deslugify($name),
100			];
101			}
102			foreach ($index['files'] as $i => $item) {
103			$item = str_replace($basePath, '', $item);
104			$item = str_replace('.md', '', $item);
105			$paths = explode('/', $item);
106			$name = array_pop($paths);
107			$index['files'][$i] = [
108			'href' => $item,
109			'name' => $this->deslugify($name),
110			];
111			if ($index['files'][$i]['name'] === 'index') {
112			unset($index['files'][$i]);
113			}
114			}
115			}
116
117			return [
118			'title' => $title,
119			'breadcrumbs' => $breadcrumbs,
120			'index' => $index,
121			'post' => $post,
122			];
123			}
124
125			/**
126			* Get the Post or Category title from the breadcrumbs array.
127			*
128			* @param $breadcrumbs
129			* @return mixed
130			*/
131			private function makeTitleFromBreadcrumbs($breadcrumbs)
132			{
133			$lastCrumb = current(array_slice($breadcrumbs, -1));
134
135			return $lastCrumb['name'];
136			}
137
138			/**
139			* Make an array of breadcrumb items.
140			*
141			* @param string $post
142			* @return array
143			*/
144			private function makeBreadcrumbsFromPost($post)
145			{
146			$crumbs = explode('/', $post);
147			$crumbs = empty($crumbs[0]) ? [] : $crumbs;
148
149			$breadcrumbs = [];
150
151			foreach ($crumbs as $i => $crumb) {
152			$breadcrumbs[$i] = [
153			'href' => '/'.$this->slugify(implode('/', array_slice($crumbs, 0, $i+1))),
154			'name' => $this->deslugify($crumbs[$i]),
155			];
156			}
157
158			return $breadcrumbs;
159			}
160
161			/**
162			* Take a string with spaces and convert the spaces to the specified separator.
163			*
164			* @param string $string
165			* @param string $separator
166			* @return mixed
167			*/
168			private function slugify($string, $separator = '-')
169			{
170			return str_replace(' ', $separator, $string);
171			}
172
173			/**
174			* Take a string with dash or underscore word separators and convert the to spaces.
175			*
176			* @param array\|string $slugged
177			* @return array\|string
178			*/
179			private function deslugify($slugged)
180			{
181			if (is_array($slugged)) {
182			return array_map('static::deslugify', $slugged);
183			}
184
185			return str_replace(['-', '_'], ' ', $slugged);
186			}
187			}
188

ryanwinchester / pagemark

Pagemark::getContent() B last analyzed 2016-04-29 16:25 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

Pagemark::getContent() B
last analyzed 2016-04-29 16:25 UTC