WatchController::post() - Code Metrics - Inspection of "Slim upgrade to v3" - perftools/xhgui - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#333)

by Elan

created 2020-10-14 12:25 UTC

WatchController::post() A

↳ Parent: Project

Complexity

Conditions	3
Paths	4

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.8666
c	0
b	0
f	0
cc	3
nc	4
nop	1

<?php

namespace XHGui\Controller;

use Slim\App;
use Slim\Http\Request;
use Slim\Slim as App;

use XHGui\AbstractController;
use XHGui\Searcher\SearcherInterface;

class WatchController extends AbstractController
{
    /**
     * @var SearcherInterface
     */
    protected $searcher;

    public function __construct(App $app, SearcherInterface $searcher)
    {
        parent::__construct($app);
        $this->searcher = $searcher;
    }

    public function get()
    {
        $watched = $this->searcher->getAllWatches();

        $this->_template = 'watch/list.twig';
        $this->set(['watched' => $watched]);
    }

    public function post(Request $request)
    {
        $saved = false;
        foreach ((array)$request->post('watch') as $data) {
            $saved = true;
            $this->searcher->saveWatch($data);
        }
        if ($saved) {
            $this->app->flash('success', 'Watch functions updated.');
        }
        $this->app->redirect($this->app->urlFor('watch.list'));
    }
}


1			<?php
2
3			namespace XHGui\Controller;
4
5			use Slim\App;
6			use Slim\Http\Request;
7			use Slim\Slim as App;
			0 ignored issues – show Bug introduced 2020-10-14 12:27 UTC by Report Bug Copy Issue Report This code did not parse for me. Apparently, there is an error somewhere around this line: Cannot use Slim\Slim as App because the name is already in use Loading history...
8			use XHGui\AbstractController;
9			use XHGui\Searcher\SearcherInterface;
10
11			class WatchController extends AbstractController
12			{
13			/**
14			* @var SearcherInterface
15			*/
16			protected $searcher;
17
18			public function __construct(App $app, SearcherInterface $searcher)
19			{
20			parent::__construct($app);
21			$this->searcher = $searcher;
22			}
23
24			public function get()
25			{
26			$watched = $this->searcher->getAllWatches();
27
28			$this->_template = 'watch/list.twig';
29			$this->set(['watched' => $watched]);
30			}
31
32			public function post(Request $request)
33			{
34			$saved = false;
35			foreach ((array)$request->post('watch') as $data) {
36			$saved = true;
37			$this->searcher->saveWatch($data);
38			}
39			if ($saved) {
40			$this->app->flash('success', 'Watch functions updated.');
41			}
42			$this->app->redirect($this->app->urlFor('watch.list'));
43			}
44			}
45

perftools / xhgui

Pull Request — master (#333)

WatchController::post() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like