SitemapController::indexAction() - Code Metrics - alexander-torosh/yona-cms - Measure and Improve Code Quality continuously with Scrutinizer

SitemapController::indexAction() A
last analyzed 2019-11-25 19:20 UTC

↳ Parent: SitemapController

Complexity

Conditions	4
Paths	4

Size

Total Lines

Duplication

Lines	17
Ratio	62.96 %

Importance

Changes

Metric	Value
dl	17
loc	27
rs	9.488
c	0
b	0
f	0
cc	4
nc	4
nop	0

<?php

namespace Seo\Controller;

use Application\Mvc\Controller;
use Seo\Form\SitemapForm;

class SitemapController extends Controller
{

    private $sitemapFilePath;

    public function initialize()
    {
        $this->setAdminEnvironment();
        $this->helper->activeMenu()->setActive('seo-sitemap');
        $this->sitemapFilePath = ROOT . '/sitemap.xml';
        $this->view->languages_disabled = true;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
    }

    public function indexAction()
    {
        $form = new SitemapForm();

        if ($this->request->isPost()) {
            if ($form->isValid()) {
                $sitemap = $this->request->getPost('sitemap');
                $result = file_put_contents($this->sitemapFilePath, $sitemap);
                if ($result) {
                    $this->flash->success('File sitemap.xml has been saved');
                    $this->redirect($this->url->get() . 'seo/sitemap');
                } else {
                    $this->flash->error('Error! The sitemap.xml file is not updated. Check the write permissions to the file.');
                }
            } else {
                $this->flashErrors($form);
            }
        } else {
            $sitemap = file_get_contents($this->sitemapFilePath);
            $form->get('sitemap')->setDefault($sitemap);

        }

        $title = 'Editing sitemap.xml';
        $this->helper->title($title);
        $this->view->title = $title;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
        $this->view->form = $form;
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
    }

}


1		<?php
2
3		namespace Seo\Controller;
4
5		use Application\Mvc\Controller;
6		use Seo\Form\SitemapForm;
7
8		class SitemapController extends Controller
9		{
10
11		private $sitemapFilePath;
12
13	View Code Duplication	public function initialize()
14		{
15		$this->setAdminEnvironment();
16		$this->helper->activeMenu()->setActive('seo-sitemap');
17		$this->sitemapFilePath = ROOT . '/sitemap.xml';
18		$this->view->languages_disabled = true;
		0 ignored issues – show Bug introduced 2016-09-12 07:35 UTC by Report Bug Copy Issue Report Accessing `languages_disabled` on the interface `Phalcon\Mvc\ViewInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
19		}
20
21		public function indexAction()
22		{
23		$form = new SitemapForm();
24
25	View Code Duplication	if ($this->request->isPost()) {
26		if ($form->isValid()) {
27		$sitemap = $this->request->getPost('sitemap');
28		$result = file_put_contents($this->sitemapFilePath, $sitemap);
29		if ($result) {
30		$this->flash->success('File sitemap.xml has been saved');
31		$this->redirect($this->url->get() . 'seo/sitemap');
32		} else {
33		$this->flash->error('Error! The sitemap.xml file is not updated. Check the write permissions to the file.');
34		}
35		} else {
36		$this->flashErrors($form);
37		}
38		} else {
39		$sitemap = file_get_contents($this->sitemapFilePath);
40		$form->get('sitemap')->setDefault($sitemap);
		0 ignored issues – show Bug introduced 2018-03-22 08:36 UTC by Report Bug Copy Issue Report The method `setDefault` cannot be called on `$form->get('sitemap')` (of type `null`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
41		}
42
43		$title = 'Editing sitemap.xml';
44		$this->helper->title($title);
45		$this->view->title = $title;
		0 ignored issues – show Bug introduced 2016-09-12 07:35 UTC by Report Bug Copy Issue Report Accessing `title` on the interface `Phalcon\Mvc\ViewInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
46		$this->view->form = $form;
		0 ignored issues – show Bug introduced 2016-09-12 07:35 UTC by Report Bug Copy Issue Report Accessing `form` on the interface `Phalcon\Mvc\ViewInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
47		}
48
49		}
50

alexander-torosh / yona-cms

SitemapController::indexAction() A last analyzed 2019-11-25 19:20 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

SitemapController::indexAction() A
last analyzed 2019-11-25 19:20 UTC