index-functions.php ➔ printLogOff() - Code Metrics - Inspection of "CIL-175, CIL-615, CIL-616, CIL-620 - Bootstrap CSS..." - cilogon/service - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1a0f38...3a0846 )

by Terrence

created 2019-12-09 21:32 UTC

index-functions.php ➔ printLogOff() B

↳ Parent: Project

Complexity

Conditions	5
Paths	16

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
nc	16
nop	0
dl	0
loc	85
rs	8.0161
c	0
b	0
f	0

How to fix Long Method

<?php

/**
 * This file contains functions called by index-site.php. The index-site.php
 * file should include this file with the following statement at the top:
 *
 * require_once __DIR__ . '/index-functions.php';
 */

use CILogon\Service\Util;
use CILogon\Service\Content;
use CILogon\Service\Loggit;

/**
 * printLogonPage
 *
 * This function prints out the HTML for the main cilogon.org page.
 * Explanatory text is shown as well as a button to log in to an IdP
 * and get rerouted to the Shibboleth protected service script, or the
 * OpenID script.
 *
 * @param bool $clearcookies True if the Shibboleth cookies and session
 *        variables  should be cleared out before displaying the page.
 *        Defaults to false.
 */
function printLogonPage($clearcookies = false)
/**
 * @ignore
 */
function getUser() {

}

function getUser($id, $realm) {

}
{
    if ($clearcookies) {
        Util::removeShibCookies();
        Util::unsetAllUserSessionVars();
        Util::getSkin()->init(true);  // Clear cilogon_skin var; check for forced skin
    }

    $log = new Loggit();
    $log->info('Welcome page hit.');

    Util::setSessionVar('stage', 'logon'); // For Show/Hide Help button clicks

    Content::printHeader('Welcome To The CILogon Service');
    Content::printWAYF();
    Content::printFooter();
}

/**
 * printMainPage
 *
 * This function prints out the HTML for the main page where the user
 * can download a certificate.
 */
function printMainPage()
/**
 * @ignore
 */
function getUser() {

}

function getUser($id, $realm) {

}
{
    $log = new Loggit();
    $log->info('Get And Use Certificate page hit.');

    Util::setSessionVar('stage', 'main'); // For Show/Hide Help button clicks
    // CIL-626 Allow browser 'reload page' by adding CSRF to the PHP session
    Util::setSessionVar('submit', 'Proceed');
    Util::getCsrf()->setTheSession();

    Content::printHeader('Get Your Certificate');
    Content::printGetCertificate();

    Content::printCertInfo();

    Content::printUserAttributes();

    Content::printIdPMetadata();

    Content::printLogOff();

    Content::printFooter();
}


1			<?php
2
3			/**
4			* This file contains functions called by index-site.php. The index-site.php
5			* file should include this file with the following statement at the top:
6			*
7			* require_once __DIR__ . '/index-functions.php';
8			*/
9
10			use CILogon\Service\Util;
11			use CILogon\Service\Content;
12			use CILogon\Service\Loggit;
13
14			/**
15			* printLogonPage
16			*
17			* This function prints out the HTML for the main cilogon.org page.
18			* Explanatory text is shown as well as a button to log in to an IdP
19			* and get rerouted to the Shibboleth protected service script, or the
20			* OpenID script.
21			*
22			* @param bool $clearcookies True if the Shibboleth cookies and session
23			* variables should be cleared out before displaying the page.
24			* Defaults to false.
25			*/
26			function printLogonPage($clearcookies = false)
			0 ignored issues – show Best Practice introduced 2019-11-18 22:50 UTC by Report Bug Copy Issue Report The function `printLogonPage()` has been defined more than once; this definition is ignored, only the first definition in `authorize/index-functions.php` (L22-54) is considered. This check looks for functions that have already been defined in other files. Some Codebases, like WordPress, make a practice of defining functions multiple times. This may lead to problems with the detection of function parameters and types. If you really need to do this, you can mark the duplicate definition with the `@ignore` annotation. /** * @ignore */ function getUser() { } function getUser($id, $realm) { } See also the PhpDoc documentation for @ignore. Loading history...
27			{
28			if ($clearcookies) {
29			Util::removeShibCookies();
30			Util::unsetAllUserSessionVars();
31			Util::getSkin()->init(true); // Clear cilogon_skin var; check for forced skin
32			}
33
34			$log = new Loggit();
35			$log->info('Welcome page hit.');
36
37			Util::setSessionVar('stage', 'logon'); // For Show/Hide Help button clicks
38
39			Content::printHeader('Welcome To The CILogon Service');
40			Content::printWAYF();
41			Content::printFooter();
42			}
43
44			/**
45			* printMainPage
46			*
47			* This function prints out the HTML for the main page where the user
48			* can download a certificate.
49			*/
50			function printMainPage()
			0 ignored issues – show Best Practice introduced 2019-11-18 22:50 UTC by Report Bug Copy Issue Report The function `printMainPage()` has been defined more than once; this definition is ignored, only the first definition in `authorize/index-functions.php` (L128-224) is considered. This check looks for functions that have already been defined in other files. Some Codebases, like WordPress, make a practice of defining functions multiple times. This may lead to problems with the detection of function parameters and types. If you really need to do this, you can mark the duplicate definition with the `@ignore` annotation. /** * @ignore */ function getUser() { } function getUser($id, $realm) { } See also the PhpDoc documentation for @ignore. Loading history...
51			{
52			$log = new Loggit();
53			$log->info('Get And Use Certificate page hit.');
54
55			Util::setSessionVar('stage', 'main'); // For Show/Hide Help button clicks
56			// CIL-626 Allow browser 'reload page' by adding CSRF to the PHP session
57			Util::setSessionVar('submit', 'Proceed');
58			Util::getCsrf()->setTheSession();
59
60			Content::printHeader('Get Your Certificate');
61			Content::printGetCertificate();
			0 ignored issues – show Bug introduced 2019-12-09 21:44 UTC by Report Bug Copy Issue Report The method `printGetCertificate()` does not seem to exist on `object<CILogon\Service\Content>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
62			Content::printCertInfo();
			0 ignored issues – show Bug introduced 2019-12-09 21:44 UTC by Report Bug Copy Issue Report The method `printCertInfo()` does not seem to exist on `object<CILogon\Service\Content>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
63			Content::printUserAttributes();
			0 ignored issues – show Bug introduced 2019-11-21 22:10 UTC by Report Bug Copy Issue Report The method `printUserAttributes()` does not seem to exist on `object<CILogon\Service\Content>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
64			Content::printIdPMetadata();
			0 ignored issues – show Bug introduced 2019-11-21 22:10 UTC by Report Bug Copy Issue Report The method `printIdPMetadata()` does not seem to exist on `object<CILogon\Service\Content>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
65			Content::printLogOff();
			0 ignored issues – show Bug introduced 2019-12-09 21:44 UTC by Report Bug Copy Issue Report The method `printLogOff()` does not seem to exist on `object<CILogon\Service\Content>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
66			Content::printFooter();
67			}
68

cilogon / service

Push — master ( 1a0f38...3a0846 )

index-functions.php ➔ printLogOff() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like