Logout::preProcess() - Code Metrics - Inspection of "allow actions to be initialized without an action..." - splitbrain/dokuwiki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — actionrefactor ( 6e4bf0 )

by Andreas

created 2017-03-31 10:54 UTC

Logout::preProcess() A

↳ Parent: Logout

Complexity

Conditions	2
Paths	2

Size

Total Lines	17
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace dokuwiki\Action;

use dokuwiki\Action\Exception\ActionException;

/**
 * Class Logout
 *
 * Log out a user
 *
 * @package dokuwiki\Action
 */
class Logout extends AbstractUserAction {

    /** @inheritdoc */
    function minimumPermission() {

        return AUTH_NONE;
    }

    /** @inheritdoc */
    public function preProcess() {
        global $ID;
        global $INPUT;

        // when logging out during an edit session, unlock the page
        $lockedby = checklock($ID);
        if($lockedby == $INPUT->server->str('REMOTE_USER')) {
            unlock($ID);
        }

        // do the logout stuff and redirect to login
        auth_logoff();
        send_redirect(wl($ID, array('do' => 'login')));

        // should never be reached
        throw new ActionException('login');
    }

}


1			<?php
2
3			namespace dokuwiki\Action;
4
5			use dokuwiki\Action\Exception\ActionException;
6
7			/**
8			* Class Logout
9			*
10			* Log out a user
11			*
12			* @package dokuwiki\Action
13			*/
14			class Logout extends AbstractUserAction {
15
16			/** @inheritdoc */
17			function minimumPermission() {
			0 ignored issues – show Best Practice introduced 2017-03-31 10:58 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
18			return AUTH_NONE;
19			}
20
21			/** @inheritdoc */
22			public function preProcess() {
23			global $ID;
24			global $INPUT;
25
26			// when logging out during an edit session, unlock the page
27			$lockedby = checklock($ID);
28			if($lockedby == $INPUT->server->str('REMOTE_USER')) {
29			unlock($ID);
30			}
31
32			// do the logout stuff and redirect to login
33			auth_logoff();
34			send_redirect(wl($ID, array('do' => 'login')));
35
36			// should never be reached
37			throw new ActionException('login');
38			}
39
40			}
41

splitbrain / dokuwiki

Push — actionrefactor ( 6e4bf0 )

Logout::preProcess() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like