SiteSummaryExtension::updateColumns() - Code Metrics - Inspection of "API Upgrade to SilverStripe 4 compatibility" - bringyourownideas/silverstripe-composer-security-checker - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#37)

unknown

created 2018-06-06 22:58 UTC

SiteSummaryExtension::updateColumns() A

↳ Parent: SiteSummaryExtension

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	4
nc	1
nop	1
dl	0
loc	7
rs	9.4285
c	0
b	0
f	0

<?php

namespace BringYourOwnIdeas\SecurityChecker\Extensions;

use SilverStripe\View\Requirements;
use SilverStripe\Core\Extension;

class SiteSummaryExtension extends Extension
{
    /**
     * Update screen bound report columns to remove the text (csv) column
     * listing SecurityAlert Identifier numbers, and include the view assets to render appropriately
     *
     * @param array $columns Report display columns
     */
    public function updateColumns(&$columns)
    {
        Requirements::css('bringyourownideas/silverstripe-composer-security-checker: client/dist/css/securityalerts.css');
        Requirements::javascript(
            'bringyourownideas/silverstripe-composer-security-checker: client/dist/javascript/summaryalerts.js'
        );
        unset($columns['listSecurityAlertIdentifiers']);
    }

    /**
     * Update the Package's screen bound summary info with little badges to indicate
     * security alerts are present for this package
     *
     * @param array $alerts a list of alerts to display
     */
    public function updateAlerts(&$alerts)
    {
        $securityWarnings = $this->owner->sourceRecords()->filter('SecurityAlerts.ID:GreaterThan', 0);

        if ($securityWarnings->exists()) {
            $alerts['SecurityAlerts'] = $securityWarnings->renderWith('SecurityAlertSummary');
        }
    }
}


1			<?php
2
3			namespace BringYourOwnIdeas\SecurityChecker\Extensions;
4
5			use SilverStripe\View\Requirements;
6			use SilverStripe\Core\Extension;
7
8			class SiteSummaryExtension extends Extension
9			{
10			/**
11			* Update screen bound report columns to remove the text (csv) column
12			* listing SecurityAlert Identifier numbers, and include the view assets to render appropriately
13			*
14			* @param array $columns Report display columns
15			*/
16			public function updateColumns(&$columns)
17			{
18			Requirements::css('bringyourownideas/silverstripe-composer-security-checker: client/dist/css/securityalerts.css');
19			Requirements::javascript(
20			'bringyourownideas/silverstripe-composer-security-checker: client/dist/javascript/summaryalerts.js'
21			);
22			unset($columns['listSecurityAlertIdentifiers']);
23			}
24
25			/**
26			* Update the Package's screen bound summary info with little badges to indicate
27			* security alerts are present for this package
28			*
29			* @param array $alerts a list of alerts to display
30			*/
31			public function updateAlerts(&$alerts)
32			{
33			$securityWarnings = $this->owner->sourceRecords()->filter('SecurityAlerts.ID:GreaterThan', 0);
34
35			if ($securityWarnings->exists()) {
36			$alerts['SecurityAlerts'] = $securityWarnings->renderWith('SecurityAlertSummary');
37			}
38			}
39			}
40

bringyourownideas / silverstripe-composer-security-checker

Pull Request — master (#37)

SiteSummaryExtension::updateColumns() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like