ControllerAccountLogout::index() - Code Metrics - Nickbur/Sunrise-CMS - Measure and Improve Code Quality continuously with Scrutinizer

ControllerAccountLogout::index() A
last analyzed 2019-09-02 14:26 UTC

↳ Parent: ControllerAccountLogout

Complexity

Conditions	2
Paths	2

Size

Total Lines	55
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	34
c	0
b	0
f	0
nc	2
nop	0
dl	0
loc	55
rs	9.376

How to fix Long Method

<?php

/* 	Divine CMS - Open source CMS for widespread use.
    Copyright (c) 2019 Mykola Burakov ([email protected])

    See SOURCE.txt for other and additional information.

    This file is part of Divine CMS.

    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
    the Free Software Foundation, either version 3 of the License, or
    (at your option) any later version.

    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
    GNU General Public License for more details.

    You should have received a copy of the GNU General Public License
    along with this program. If not, see <http://www.gnu.org/licenses/>. */

class ControllerAccountLogout extends \Divine\Engine\Core\Controller
namespace YourVendor;

class YourClass { }
{
    public function index()

    {
        if ($this->customer->isLogged()) {
            $this->customer->logout();

            unset($this->session->data['shipping_address']);
            unset($this->session->data['shipping_method']);
            unset($this->session->data['shipping_methods']);
            unset($this->session->data['payment_address']);
            unset($this->session->data['payment_method']);
            unset($this->session->data['payment_methods']);
            unset($this->session->data['comment']);
            unset($this->session->data['order_id']);
            unset($this->session->data['reward']);

            $this->response->redirect($this->url->link('account/logout', '', true));
        }

        $this->load->language('account/logout');

        $this->document->setTitle($this->language->get('heading_title'));

        $data['breadcrumbs'] = array();


        $data['breadcrumbs'][] = array(
            'text' => $this->language->get('text_home'),
            'href' => $this->url->link('common/home')
        );

        $data['breadcrumbs'][] = array(
            'text' => $this->language->get('text_account'),
            'href' => $this->url->link('account/account', '', true)
        );

        $data['breadcrumbs'][] = array(
            'text' => $this->language->get('text_logout'),
            'href' => $this->url->link('account/logout', '', true)
        );

        $data['heading_title'] = $this->language->get('heading_title');

        $data['text_message'] = $this->language->get('text_message');

        $data['button_continue'] = $this->language->get('button_continue');

        $data['continue'] = $this->url->link('common/home');

        $data['column'] = $this->load->controller('common/column');
        
        $data['content_top'] = $this->load->controller('common/content_top');
        $data['content_bottom'] = $this->load->controller('common/content_bottom');
        $data['footer'] = $this->load->controller('common/footer');
        $data['header'] = $this->load->controller('common/header');

        $this->response->setOutput($this->load->view('common/success', $data));
    }
}


1			<?php
2
3			/* Divine CMS - Open source CMS for widespread use.
4			Copyright (c) 2019 Mykola Burakov ([email protected])
5
6			See SOURCE.txt for other and additional information.
7
8			This file is part of Divine CMS.
9
10			This program is free software: you can redistribute it and/or modify
11			it under the terms of the GNU General Public License as published by
12			the Free Software Foundation, either version 3 of the License, or
13			(at your option) any later version.
14
15			This program is distributed in the hope that it will be useful,
16			but WITHOUT ANY WARRANTY; without even the implied warranty of
17			MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
18			GNU General Public License for more details.
19
20			You should have received a copy of the GNU General Public License
21			along with this program. If not, see <http://www.gnu.org/licenses/>. */
22
23			class ControllerAccountLogout extends \Divine\Engine\Core\Controller
			0 ignored issues – show Coding Style Compatibility introduced 2019-07-29 15:05 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
24			{
25			public function index()
			0 ignored issues – show Coding Style introduced 2019-07-29 15:05 UTC by Report Bug Copy Issue Report Expected 2 blank lines before function; 0 found Loading history...
26			{
27			if ($this->customer->isLogged()) {
28			$this->customer->logout();
29
30			unset($this->session->data['shipping_address']);
31			unset($this->session->data['shipping_method']);
32			unset($this->session->data['shipping_methods']);
33			unset($this->session->data['payment_address']);
34			unset($this->session->data['payment_method']);
35			unset($this->session->data['payment_methods']);
36			unset($this->session->data['comment']);
37			unset($this->session->data['order_id']);
38			unset($this->session->data['reward']);
39
40			$this->response->redirect($this->url->link('account/logout', '', true));
41			}
42
43			$this->load->language('account/logout');
44
45			$this->document->setTitle($this->language->get('heading_title'));
46
47			$data['breadcrumbs'] = array();
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-07-29 15:05 UTC by Report Bug Copy Issue Report `$data` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$data = array();` before regardless. Loading history...
48
49			$data['breadcrumbs'][] = array(
50			'text' => $this->language->get('text_home'),
51			'href' => $this->url->link('common/home')
52			);
53
54			$data['breadcrumbs'][] = array(
55			'text' => $this->language->get('text_account'),
56			'href' => $this->url->link('account/account', '', true)
57			);
58
59			$data['breadcrumbs'][] = array(
60			'text' => $this->language->get('text_logout'),
61			'href' => $this->url->link('account/logout', '', true)
62			);
63
64			$data['heading_title'] = $this->language->get('heading_title');
65
66			$data['text_message'] = $this->language->get('text_message');
67
68			$data['button_continue'] = $this->language->get('button_continue');
69
70			$data['continue'] = $this->url->link('common/home');
71
72			$data['column'] = $this->load->controller('common/column');
73
74			$data['content_top'] = $this->load->controller('common/content_top');
75			$data['content_bottom'] = $this->load->controller('common/content_bottom');
76			$data['footer'] = $this->load->controller('common/footer');
77			$data['header'] = $this->load->controller('common/header');
78
79			$this->response->setOutput($this->load->view('common/success', $data));
80			}
81			}
82

Nickbur / Sunrise-CMS

ControllerAccountLogout::index() A last analyzed 2019-09-02 14:26 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

ControllerAccountLogout::index() A
last analyzed 2019-09-02 14:26 UTC