Session::getAttributes() - Code Metrics - Inspection of "Merge pull request #350 from gyselroth/v2.6" - gyselroth/balloon - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b97427...e235cc )

by Raffael

created 2019-06-25 09:06 UTC

Session::getAttributes() A

↳ Parent: Session

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	42
ccs	0
cts	35
cp	0
rs	9.248
c	0
b	0
f	0
cc	1
nc	1
nop	0
crap	2

<?php

declare(strict_types=1);

/**
 * balloon
 *
 * @copyright   Copryright (c) 2012-2019 gyselroth GmbH (https://gyselroth.com)
 * @license     GPL-3.0 https://opensource.org/licenses/GPL-3.0
 */

namespace Balloon\App\Wopi\Session;

use Balloon\Filesystem\Node\AbstractNode;
use Balloon\Filesystem\Node\File;
use Balloon\Server\User;

class Session implements SessionInterface
{
    /**
     * Session.
     *
     * @var array
     */
    protected $session = [];

    /**
     * File.
     *
     * @var File
     */
    protected $file;

    /**
     * User.
     *
     * @var User
     */
    protected $user;

    /**
     * Session.
     */
    public function __construct(File $file, User $user, array $session)
    {
        $this->file = $file;
        $this->user = $user;
        $this->session = $session;
    }

    /**
     * {@inheritdoc}
     */
    public function getUser(): User
    {
        return $this->user;
    }

    /**
     * {@inheritdoc}
     */
    public function getFile(): File
    {
        return $this->file;
    }

    /**
     * {@inheritdoc}
     */
    public function getAccessTokenTTl(): int
    {
        return $this->session['ttl']->toDateTime()->format('U') * 1000;
    }

    /**
     * {@inheritdoc}
     */
    public function getAccessToken(): string
    {
        return $this->session['token'];
    }

    /**
     * Get wopi url.
     */
    public function getWopiUrl(): string
    {
        return $this->session['client'].'/api/v2/office/wopi/files/'.$this->file->getId().'?access_token='.$this->getAccessToken().'&access_token_ttl='.$this->getAccessTokenTTl();
    }

    /**
     * {@inheritdoc}
     */
    public function getAttributes(): array
    {
        $attrs = $this->file->getAttributes(['name', 'version']);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $attributes = [
            'AllowExternalMarketplace' => false,
            'BaseFileName' => $this->file->getName(),

            'DisablePrint' => false,
            'DisableTranslation' => false,
            //'DownloadUrl' => null,
            //'FileSharingUrl' => null,
            //'FileUrl' => null,
            'FileVersionPostMessage' => true,
           // 'FileSharingPostMessage' => null,
            'PostMessageOrigin' => $this->session['client'],
            'OwnerId' => (string) $this->file->getOwner(),
            'ReadOnly' => $this->file->isReadonly(),
            'RestrictedWebViewOnly' => false,
            //'SHA256' => null,
            //'SignoutUrl' => null,
            'Size' => $this->file->getSize(),
            'SupportsCobalt' => false,
            'SupportsFolders' => true,
            'SupportsLocks' => true,
            'SupportsGetLock' => true,
            'SupportsExtendedLockLength' => true,
            'SupportsUserInfo' => false,
            'SupportsDeleteFile' => true,
            'SupportsUpdate' => true,
            'SupportsRename' => true,
            'FileNameMaxLength' => AbstractNode::MAX_NAME_LENGTH,
            'UserCanAttend' => false,
            'UserCanNotWriteRelative' => false,
            'UserCanPresent' => false,
            'UserCanRename' => true,
            'UserCanWrite' => $this->file->mayWrite(),
            'UserFriendlyName' => $this->user->getUsername(),
            'UserId' => (string) $this->user->getId(),
            'Version' => (string) $this->file->getVersion(),
        ];

        return $attributes;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/**
6			* balloon
7			*
8			* @copyright Copryright (c) 2012-2019 gyselroth GmbH (https://gyselroth.com)
9			* @license GPL-3.0 https://opensource.org/licenses/GPL-3.0
10			*/
11
12			namespace Balloon\App\Wopi\Session;
13
14			use Balloon\Filesystem\Node\AbstractNode;
15			use Balloon\Filesystem\Node\File;
16			use Balloon\Server\User;
17
18			class Session implements SessionInterface
19			{
20			/**
21			* Session.
22			*
23			* @var array
24			*/
25			protected $session = [];
26
27			/**
28			* File.
29			*
30			* @var File
31			*/
32			protected $file;
33
34			/**
35			* User.
36			*
37			* @var User
38			*/
39			protected $user;
40
41			/**
42			* Session.
43			*/
44			public function __construct(File $file, User $user, array $session)
45			{
46			$this->file = $file;
47			$this->user = $user;
48			$this->session = $session;
49			}
50
51			/**
52			* {@inheritdoc}
53			*/
54			public function getUser(): User
55			{
56			return $this->user;
57			}
58
59			/**
60			* {@inheritdoc}
61			*/
62			public function getFile(): File
63			{
64			return $this->file;
65			}
66
67			/**
68			* {@inheritdoc}
69			*/
70			public function getAccessTokenTTl(): int
71			{
72			return $this->session['ttl']->toDateTime()->format('U') * 1000;
73			}
74
75			/**
76			* {@inheritdoc}
77			*/
78			public function getAccessToken(): string
79			{
80			return $this->session['token'];
81			}
82
83			/**
84			* Get wopi url.
85			*/
86			public function getWopiUrl(): string
87			{
88			return $this->session['client'].'/api/v2/office/wopi/files/'.$this->file->getId().'?access_token='.$this->getAccessToken().'&access_token_ttl='.$this->getAccessTokenTTl();
89			}
90
91			/**
92			* {@inheritdoc}
93			*/
94			public function getAttributes(): array
95			{
96			$attrs = $this->file->getAttributes(['name', 'version']);
			0 ignored issues – show Unused Code introduced 2019-06-25 09:32 UTC by Report Bug Copy Issue Report `$attrs` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
97			$attributes = [
98			'AllowExternalMarketplace' => false,
99			'BaseFileName' => $this->file->getName(),
			0 ignored issues – show Bug introduced 2019-06-25 09:37 UTC by Report Bug Copy Issue Report Consider using `$this->file->name`. There is an issue with `getName()` and APC-enabled PHP versions. Loading history...
100			'DisablePrint' => false,
101			'DisableTranslation' => false,
102			//'DownloadUrl' => null,
103			//'FileSharingUrl' => null,
104			//'FileUrl' => null,
105			'FileVersionPostMessage' => true,
106			// 'FileSharingPostMessage' => null,
107			'PostMessageOrigin' => $this->session['client'],
108			'OwnerId' => (string) $this->file->getOwner(),
109			'ReadOnly' => $this->file->isReadonly(),
110			'RestrictedWebViewOnly' => false,
111			//'SHA256' => null,
112			//'SignoutUrl' => null,
113			'Size' => $this->file->getSize(),
114			'SupportsCobalt' => false,
115			'SupportsFolders' => true,
116			'SupportsLocks' => true,
117			'SupportsGetLock' => true,
118			'SupportsExtendedLockLength' => true,
119			'SupportsUserInfo' => false,
120			'SupportsDeleteFile' => true,
121			'SupportsUpdate' => true,
122			'SupportsRename' => true,
123			'FileNameMaxLength' => AbstractNode::MAX_NAME_LENGTH,
124			'UserCanAttend' => false,
125			'UserCanNotWriteRelative' => false,
126			'UserCanPresent' => false,
127			'UserCanRename' => true,
128			'UserCanWrite' => $this->file->mayWrite(),
129			'UserFriendlyName' => $this->user->getUsername(),
130			'UserId' => (string) $this->user->getId(),
131			'Version' => (string) $this->file->getVersion(),
132			];
133
134			return $attributes;
135			}
136			}
137

gyselroth / balloon

Push — master ( b97427...e235cc )

Session::getAttributes() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like