EditorController::editor() - Code Metrics - Inspection of "Update translations" - rudolfcms/rudolf - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( aa3e09...85e896 )

by Mikołaj

created 2017-08-15 19:48 UTC

EditorController::editor() B

↳ Parent: EditorController

Complexity

Conditions	4
Paths	6

Size

Total Lines	36
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	22
nc	6
nop	1
dl	0
loc	36
rs	8.5806
c	0
b	0
f	0

<?php

namespace Rudolf\Modules\Appearance\One\Admin;

use Rudolf\Component\Alerts\Alert;
use Rudolf\Component\Alerts\AlertsCollection;
use Rudolf\Framework\Controller\AdminController;

class EditorController extends AdminController
{
    /**
     * Editor.
     *
     * @param string $file Filename in base64, default templates/_header.html.php
     *
     * @return void
     */
    public function editor($file)
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

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

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if ('/admin/appearance/editor' === $file) {
            $this->redirectTo('/admin/appearance/editor/file/dGVtcGxhdGVzL19oZWFkLmh0bWwucGhw');
        }

        $model = new EditorModel();

        $filesList['templates'] = $model->getFilesListByPath('templates');
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

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

    // do something with $myArray
}
        $filesList['css'] = $model->getFilesListByPath('css');
        $filesList['js'] = $model->getFilesListByPath('js');

        $filename = base64_decode($file);

        // if data was send
        if (isset($_POST['save'])) {
            if ($model->saveFile($filename, $_POST['content'])) {
0   == false // true
0   == null  // true
123 == false // false
123 == null  // false

// It is often better to use strict comparison
0 === false // false
0 === null  // false
                AlertsCollection::add(new Alert(
                    'success',
                    'Pomyślnie zapisano plik szablonu '.$filename.'.'
                ));
            } else {
                AlertsCollection::add(new Alert(
                    'error',
                    'Coś poszło nie tak podczas zapisywania pliku '.$filename.'. '.
                    'Sprawdź prawa dostępu'
                ));
            }

            $this->redirectTo('../../editor');
        }

        $view = new EditorView();
        $view->editor($filesList, $model->getFileInfo($filename));
        $view->render('admin');
    }
}


1			<?php
2
3			namespace Rudolf\Modules\Appearance\One\Admin;
4
5			use Rudolf\Component\Alerts\Alert;
6			use Rudolf\Component\Alerts\AlertsCollection;
7			use Rudolf\Framework\Controller\AdminController;
8
9			class EditorController extends AdminController
10			{
11			/**
12			* Editor.
13			*
14			* @param string $file Filename in base64, default templates/_header.html.php
15			*
16			* @return void
17			*/
18			public function editor($file)
			0 ignored issues – show Coding Style introduced 2017-08-15 19:52 UTC by Report Bug Copy Issue Report `editor` uses the super-global variable `$_POST` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
19			{
20			if ('/admin/appearance/editor' === $file) {
21			$this->redirectTo('/admin/appearance/editor/file/dGVtcGxhdGVzL19oZWFkLmh0bWwucGhw');
22			}
23
24			$model = new EditorModel();
25
26			$filesList['templates'] = $model->getFilesListByPath('templates');
			0 ignored issues – show Coding Style Comprehensibility introduced 2017-08-15 19:52 UTC by Report Bug Copy Issue Report `$filesList` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$filesList = 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...
27			$filesList['css'] = $model->getFilesListByPath('css');
28			$filesList['js'] = $model->getFilesListByPath('js');
29
30			$filename = base64_decode($file);
31
32			// if data was send
33			if (isset($_POST['save'])) {
34			if ($model->saveFile($filename, $_POST['content'])) {
			0 ignored issues – show Bug Best Practice introduced 2017-08-15 19:52 UTC by Report Bug Copy Issue Report The expression `$model->saveFile($filename, $_POST['content'])` of type `false\|integer` is loosely compared to `true`; this is ambiguous if the integer can be zero. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `integer` values, zero is a special case, in particular the following results might be unexpected: 0 == false // true 0 == null // true 123 == false // false 123 == null // false // It is often better to use strict comparison 0 === false // false 0 === null // false Loading history...
35			AlertsCollection::add(new Alert(
36			'success',
37			'Pomyślnie zapisano plik szablonu '.$filename.'.'
38			));
39			} else {
40			AlertsCollection::add(new Alert(
41			'error',
42			'Coś poszło nie tak podczas zapisywania pliku '.$filename.'. '.
43			'Sprawdź prawa dostępu'
44			));
45			}
46
47			$this->redirectTo('../../editor');
48			}
49
50			$view = new EditorView();
51			$view->editor($filesList, $model->getFileInfo($filename));
52			$view->render('admin');
53			}
54			}
55

rudolfcms / rudolf

Push — master ( aa3e09...85e896 )

EditorController::editor() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like