Edit::setIcon() - Code Metrics - Inspection of "feat(Form): add parameter to omit security token i..." - splitbrain/dokuwiki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — betterCoreSearch ( 385027...7fa270 )

by Michael

created 2018-03-27 18:29 UTC

Edit::setIcon() A

↳ Parent: Edit

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	9
nc	2
nop	0
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0

<?php

namespace dokuwiki\Menu\Item;

/**
 * Class Edit
 *
 * Most complex item. Shows the edit button but mutates to show, draft and create based on
 * current state.
 */
class Edit extends AbstractItem {

    /** @inheritdoc */
    public function __construct() {
        global $ACT;
        global $INFO;
        global $REV;

        parent::__construct();

        if($ACT === 'show') {
            $this->method = 'post';
            if($INFO['writable']) {
                $this->accesskey = 'e';
                if(!empty($INFO['draft'])) {
                    $this->type = 'draft';
                    $this->params['do'] = 'draft';
                } else {
                    $this->params['rev'] = $REV;
                    if(!$INFO['exists']) {
                        $this->type = 'create';
                    }
                }
            } else {
                if(!actionOK($this->type)) throw new \RuntimeException("action disabled: source");
                $params['rev'] = $REV;
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
                $this->type = 'source';
                $this->accesskey = 'v';
            }
        } else {
            $this->params = array('do' => '');
            $this->type = 'show';
            $this->accesskey = 'v';
        }

        $this->setIcon();
    }

    /**
     * change the icon according to what type the edit button has
     */
    protected function setIcon() {
        $icons = array(
            'edit' => '01-edit_pencil.svg',
            'create' => '02-create_pencil.svg',
            'draft' => '03-draft_android-studio.svg',
            'show' => '04-show_file-document.svg',
            'source' => '05-source_file-xml.svg',
        );
        if(isset($icons[$this->type])) {
            $this->svg = DOKU_INC . 'lib/images/menu/' . $icons[$this->type];
        }
    }

}


1			<?php
2
3			namespace dokuwiki\Menu\Item;
4
5			/**
6			* Class Edit
7			*
8			* Most complex item. Shows the edit button but mutates to show, draft and create based on
9			* current state.
10			*/
11			class Edit extends AbstractItem {
12
13			/** @inheritdoc */
14			public function __construct() {
15			global $ACT;
16			global $INFO;
17			global $REV;
18
19			parent::__construct();
20
21			if($ACT === 'show') {
22			$this->method = 'post';
23			if($INFO['writable']) {
24			$this->accesskey = 'e';
25			if(!empty($INFO['draft'])) {
26			$this->type = 'draft';
27			$this->params['do'] = 'draft';
28			} else {
29			$this->params['rev'] = $REV;
30			if(!$INFO['exists']) {
31			$this->type = 'create';
32			}
33			}
34			} else {
35			if(!actionOK($this->type)) throw new \RuntimeException("action disabled: source");
36			$params['rev'] = $REV;
			0 ignored issues – show Coding Style Comprehensibility introduced 2017-05-19 15:09 UTC by Report Bug Copy Issue Report `$params` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$params = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
37			$this->type = 'source';
38			$this->accesskey = 'v';
39			}
40			} else {
41			$this->params = array('do' => '');
42			$this->type = 'show';
43			$this->accesskey = 'v';
44			}
45
46			$this->setIcon();
47			}
48
49			/**
50			* change the icon according to what type the edit button has
51			*/
52			protected function setIcon() {
53			$icons = array(
54			'edit' => '01-edit_pencil.svg',
55			'create' => '02-create_pencil.svg',
56			'draft' => '03-draft_android-studio.svg',
57			'show' => '04-show_file-document.svg',
58			'source' => '05-source_file-xml.svg',
59			);
60			if(isset($icons[$this->type])) {
61			$this->svg = DOKU_INC . 'lib/images/menu/' . $icons[$this->type];
62			}
63			}
64
65			}
66

splitbrain / dokuwiki

Push — betterCoreSearch ( 385027...7fa270 )

Edit::setIcon() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like