ShareDraftContentControllerExtension::getShareDraftLinkAction() - Code Metrics - Inspection of "Merge pull request #84 from creative-commoners/doc..." - silverstripe/silverstripe-sharedraftcontent - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f3d1e6...12f788 )

by Robbie

created 2018-06-25 02:37 UTC

getShareDraftLinkAction() A

↳ Parent: ShareDraftContentControllerExtension

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
nc	2
nop	0
dl	0
loc	6
rs	10
c	0
b	0
f	0

<?php

namespace SilverStripe\ShareDraftContent\Extensions;

use SilverStripe\Core\Extension;
use SilverStripe\Security\Security;

class ShareDraftContentControllerExtension extends Extension
{
    /**
     * @var array
     */
    private static $allowed_actions = array(

        'MakeShareDraftLink',
    );

    /**
     * @return mixed
     */
    public function MakeShareDraftLink()
    {
        if ($member = Security::getCurrentUser()) {
            if ($this->owner->hasMethod('CurrentPage') && $this->owner->CurrentPage()->canEdit($member)) {
                return $this->owner->CurrentPage()->ShareTokenLink();
            } elseif ($this->owner->hasMethod('canEdit') && $this->owner->canEdit($member)) {
                return $this->owner->ShareTokenLink();
            }
        }

        return Security::permissionFailure();
    }

    /**
     * @return string
     */
    public function getShareDraftLinkAction()
    {
        if ($this->owner->config()->get('url_segment')) {
            return $this->owner->Link('MakeShareDraftLink');
        }
        return '';
    }
}


1			<?php
2
3			namespace SilverStripe\ShareDraftContent\Extensions;
4
5			use SilverStripe\Core\Extension;
6			use SilverStripe\Security\Security;
7
8			class ShareDraftContentControllerExtension extends Extension
9			{
10			/**
11			* @var array
12			*/
13			private static $allowed_actions = array(
			0 ignored issues – show introduced 2017-12-21 20:39 UTC by Report Bug Copy Issue Report The private property `$allowed_actions` is not used, and could be removed. Loading history...
14			'MakeShareDraftLink',
15			);
16
17			/**
18			* @return mixed
19			*/
20			public function MakeShareDraftLink()
21			{
22			if ($member = Security::getCurrentUser()) {
23			if ($this->owner->hasMethod('CurrentPage') && $this->owner->CurrentPage()->canEdit($member)) {
24			return $this->owner->CurrentPage()->ShareTokenLink();
25			} elseif ($this->owner->hasMethod('canEdit') && $this->owner->canEdit($member)) {
26			return $this->owner->ShareTokenLink();
27			}
28			}
29
30			return Security::permissionFailure();
31			}
32
33			/**
34			* @return string
35			*/
36			public function getShareDraftLinkAction()
37			{
38			if ($this->owner->config()->get('url_segment')) {
39			return $this->owner->Link('MakeShareDraftLink');
40			}
41			return '';
42			}
43			}
44

silverstripe / silverstripe-sharedraftcontent

Push — master ( f3d1e6...12f788 )

getShareDraftLinkAction() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like