WantedPagesCLI::setup() - Code Metrics - Inspection of "updated composer dependencies" - splitbrain/dokuwiki - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — master ( b2c9cd...43d3f0 )

by Andreas

created 2018-04-13 06:38 UTC

WantedPagesCLI::setup() B

↳ Parent: WantedPagesCLI

Complexity

Conditions	1
Paths	1

Size

Total Lines	24
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	17
nc	1
nop	1
dl	0
loc	24
rs	8.9713
c	0
b	0
f	0

#!/usr/bin/php
<?php

use splitbrain\phpcli\CLI;
use splitbrain\phpcli\Options;

if(!defined('DOKU_INC')) define('DOKU_INC', realpath(dirname(__FILE__) . '/../') . '/');
define('NOSESSION', 1);
require_once(DOKU_INC . 'inc/init.php');

/**
 * Find wanted pages
 */
class WantedPagesCLI extends CLI {

    const DIR_CONTINUE = 1;
    const DIR_NS = 2;
    const DIR_PAGE = 3;

    private $skip = false;
    private $sort = 'wanted';

    private $result = array();

    /**
     * Register options and arguments on the given $options object
     *
     * @param Options $options
     * @return void
     */
    protected function setup(Options $options) {
        $options->setHelp(
            'Outputs a list of wanted pages (pages that do not exist yet) and their origin pages ' .
            ' (the pages that are linkin to these missing pages).'
        );
        $options->registerArgument(
            'namespace',
            'The namespace to lookup. Defaults to root namespace',
            false
        );

        $options->registerOption(
            'sort',
            'Sort by wanted or origin page',
            's',
            '(wanted|origin)'
        );

        $options->registerOption(
            'skip',
            'Do not show the second dimension',
            'k'
        );
    }

    /**
     * Your main program
     *
     * Arguments and options have been parsed when this is run
     *
     * @param Options $options
     * @return void
     */
    protected function main(Options $options) {
        $args = $options->getArgs();
        if($args) {

            $startdir = dirname(wikiFN($args[0] . ':xxx'));
        } else {
            $startdir = dirname(wikiFN('xxx'));
        }

        $this->skip = $options->getOpt('skip');
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        $this->sort = $options->getOpt('sort');
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}

        $this->info("searching $startdir");

        foreach($this->get_pages($startdir) as $page) {
            $this->internal_links($page);
        }
        ksort($this->result);
        foreach($this->result as $main => $subs) {
            if($this->skip) {
                print "$main\n";
            } else {
                $subs = array_unique($subs);
                sort($subs);
                foreach($subs as $sub) {
                    printf("%-40s %s\n", $main, $sub);
                }
            }
        }
    }

    /**
     * Determine directions of the search loop
     *
     * @param string $entry
     * @param string $basepath
     * @return int
     */
    protected function dir_filter($entry, $basepath) {
        if($entry == '.' || $entry == '..') {
            return WantedPagesCLI::DIR_CONTINUE;
        }
        if(is_dir($basepath . '/' . $entry)) {
            if(strpos($entry, '_') === 0) {
                return WantedPagesCLI::DIR_CONTINUE;
            }
            return WantedPagesCLI::DIR_NS;
        }
        if(preg_match('/\.txt$/', $entry)) {
            return WantedPagesCLI::DIR_PAGE;
        }
        return WantedPagesCLI::DIR_CONTINUE;
    }

    /**
     * Collects recursively the pages in a namespace
     *
     * @param string $dir
     * @return array
     * @throws DokuCLI_Exception
     */
    protected function get_pages($dir) {
        static $trunclen = null;
        if(!$trunclen) {
            global $conf;
            $trunclen = strlen($conf['datadir'] . ':');
        }

        if(!is_dir($dir)) {
            throw new DokuCLI_Exception("Unable to read directory $dir");
        }

        $pages = array();
        $dh = opendir($dir);
        while(false !== ($entry = readdir($dh))) {
            $status = $this->dir_filter($entry, $dir);
            if($status == WantedPagesCLI::DIR_CONTINUE) {
                continue;
            } else if($status == WantedPagesCLI::DIR_NS) {
                $pages = array_merge($pages, $this->get_pages($dir . '/' . $entry));
            } else {
                $page = array(
                    'id' => pathID(substr($dir . '/' . $entry, $trunclen)),
                    'file' => $dir . '/' . $entry,
                );
                $pages[] = $page;
            }
        }
        closedir($dh);
        return $pages;
    }

    /**
     * Parse instructions and add the non-existing links to the result array
     *
     * @param array $page array with page id and file path
     */
    function internal_links($page) {

        global $conf;
        $instructions = p_get_instructions(file_get_contents($page['file']));
        $cns = getNS($page['id']);
        $exists = false;
        $pid = $page['id'];
        foreach($instructions as $ins) {
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
            if($ins[0] == 'internallink' || ($conf['camelcase'] && $ins[0] == 'camelcaselink')) {
                $mid = $ins[1][0];
                resolve_pageid($cns, $mid, $exists);
<?php

function getDate($date)
{
    if ($date !== null) {
        return new DateTime($date);
    }

    return false;
}
                if(!$exists) {
                    list($mid) = explode('#', $mid); //record pages without hashes

                    if($this->sort == 'origin') {
                        $this->result[$pid][] = $mid;
                    } else {
                        $this->result[$mid][] = $pid;
                    }
                }
            }
        }
    }
}

// Main
$cli = new WantedPagesCLI();
$cli->run();


1			#!/usr/bin/php
2			<?php
3
4			use splitbrain\phpcli\CLI;
5			use splitbrain\phpcli\Options;
6
7			if(!defined('DOKU_INC')) define('DOKU_INC', realpath(dirname(__FILE__) . '/../') . '/');
8			define('NOSESSION', 1);
9			require_once(DOKU_INC . 'inc/init.php');
10
11			/**
12			* Find wanted pages
13			*/
14			class WantedPagesCLI extends CLI {
15
16			const DIR_CONTINUE = 1;
17			const DIR_NS = 2;
18			const DIR_PAGE = 3;
19
20			private $skip = false;
21			private $sort = 'wanted';
22
23			private $result = array();
24
25			/**
26			* Register options and arguments on the given $options object
27			*
28			* @param Options $options
29			* @return void
30			*/
31			protected function setup(Options $options) {
32			$options->setHelp(
33			'Outputs a list of wanted pages (pages that do not exist yet) and their origin pages ' .
34			' (the pages that are linkin to these missing pages).'
35			);
36			$options->registerArgument(
37			'namespace',
38			'The namespace to lookup. Defaults to root namespace',
39			false
40			);
41
42			$options->registerOption(
43			'sort',
44			'Sort by wanted or origin page',
45			's',
46			'(wanted\|origin)'
47			);
48
49			$options->registerOption(
50			'skip',
51			'Do not show the second dimension',
52			'k'
53			);
54			}
55
56			/**
57			* Your main program
58			*
59			* Arguments and options have been parsed when this is run
60			*
61			* @param Options $options
62			* @return void
63			*/
64			protected function main(Options $options) {
65			$args = $options->getArgs();
66			if($args) {
			0 ignored issues – show Bug Best Practice introduced 2017-11-10 16:53 UTC by Report Bug Copy Issue Report The expression `$args` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
67			$startdir = dirname(wikiFN($args[0] . ':xxx'));
68			} else {
69			$startdir = dirname(wikiFN('xxx'));
70			}
71
72			$this->skip = $options->getOpt('skip');
			0 ignored issues – show Documentation Bug introduced 2018-04-13 06:51 UTC by Report Bug Copy Issue Report It seems like `$options->getOpt('skip')` can also be of type `string` or `array<integer,string>`. However, the property `$skip` is declared as type `boolean`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
73			$this->sort = $options->getOpt('sort');
			0 ignored issues – show Documentation Bug introduced 2018-04-13 06:51 UTC by Report Bug Copy Issue Report It seems like `$options->getOpt('sort')` can also be of type `boolean` or `array<integer,string>`. However, the property `$sort` is declared as type `string`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
74
75			$this->info("searching $startdir");
76
77			foreach($this->get_pages($startdir) as $page) {
78			$this->internal_links($page);
79			}
80			ksort($this->result);
81			foreach($this->result as $main => $subs) {
82			if($this->skip) {
83			print "$main\n";
84			} else {
85			$subs = array_unique($subs);
86			sort($subs);
87			foreach($subs as $sub) {
88			printf("%-40s %s\n", $main, $sub);
89			}
90			}
91			}
92			}
93
94			/**
95			* Determine directions of the search loop
96			*
97			* @param string $entry
98			* @param string $basepath
99			* @return int
100			*/
101			protected function dir_filter($entry, $basepath) {
102			if($entry == '.' \|\| $entry == '..') {
103			return WantedPagesCLI::DIR_CONTINUE;
104			}
105			if(is_dir($basepath . '/' . $entry)) {
106			if(strpos($entry, '_') === 0) {
107			return WantedPagesCLI::DIR_CONTINUE;
108			}
109			return WantedPagesCLI::DIR_NS;
110			}
111			if(preg_match('/\.txt$/', $entry)) {
112			return WantedPagesCLI::DIR_PAGE;
113			}
114			return WantedPagesCLI::DIR_CONTINUE;
115			}
116
117			/**
118			* Collects recursively the pages in a namespace
119			*
120			* @param string $dir
121			* @return array
122			* @throws DokuCLI_Exception
123			*/
124			protected function get_pages($dir) {
125			static $trunclen = null;
126			if(!$trunclen) {
127			global $conf;
128			$trunclen = strlen($conf['datadir'] . ':');
129			}
130
131			if(!is_dir($dir)) {
132			throw new DokuCLI_Exception("Unable to read directory $dir");
133			}
134
135			$pages = array();
136			$dh = opendir($dir);
137			while(false !== ($entry = readdir($dh))) {
138			$status = $this->dir_filter($entry, $dir);
139			if($status == WantedPagesCLI::DIR_CONTINUE) {
140			continue;
141			} else if($status == WantedPagesCLI::DIR_NS) {
142			$pages = array_merge($pages, $this->get_pages($dir . '/' . $entry));
143			} else {
144			$page = array(
145			'id' => pathID(substr($dir . '/' . $entry, $trunclen)),
146			'file' => $dir . '/' . $entry,
147			);
148			$pages[] = $page;
149			}
150			}
151			closedir($dh);
152			return $pages;
153			}
154
155			/**
156			* Parse instructions and add the non-existing links to the result array
157			*
158			* @param array $page array with page id and file path
159			*/
160			function internal_links($page) {
			0 ignored issues – show Best Practice introduced 2015-11-25 07:06 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
161			global $conf;
162			$instructions = p_get_instructions(file_get_contents($page['file']));
163			$cns = getNS($page['id']);
164			$exists = false;
165			$pid = $page['id'];
166			foreach($instructions as $ins) {
			0 ignored issues – show Bug introduced 2015-11-25 07:06 UTC by Report Bug Copy Issue Report The expression `$instructions` of type `array\|false` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
167			if($ins[0] == 'internallink' \|\| ($conf['camelcase'] && $ins[0] == 'camelcaselink')) {
168			$mid = $ins[1][0];
169			resolve_pageid($cns, $mid, $exists);
			0 ignored issues – show Security Bug introduced 2016-02-04 19:12 UTC by Report Bug Copy Issue Report It seems like `$cns` defined by `getNS($page['id'])` on line `163` can also be of type `false`; however, `resolve_pageid()` does only seem to accept `string`, did you maybe forget to handle an error condition? This check looks for type mismatches where the missing type is `false`. This is usually indicative of an error condtion. Consider the follow example <?php function getDate($date) { if ($date !== null) { return new DateTime($date); } return false; } This function either returns a new `DateTime` object or false, if there was an error. This is a typical pattern in PHP programming to show that an error has occurred without raising an exception. The calling code should check for this returned `false` before passing on the value to another function or method that may not be able to handle a `false`. Loading history...
170			if(!$exists) {
171			list($mid) = explode('#', $mid); //record pages without hashes
172
173			if($this->sort == 'origin') {
174			$this->result[$pid][] = $mid;
175			} else {
176			$this->result[$mid][] = $pid;
177			}
178			}
179			}
180			}
181			}
182			}
183
184			// Main
185			$cli = new WantedPagesCLI();
186			$cli->run();
187

splitbrain / dokuwiki

Push — master ( b2c9cd...43d3f0 )

WantedPagesCLI::setup() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like