File FilesPlugin.php has 303 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/**
 * @author Björn Schießle <bjoern@schiessle.org>
 * @author Joas Schilling <coding@schilljs.com>
 * @author Lukas Reschke <lukas@statuscode.ch>

Method handleGetProperties has 79 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function handleGetProperties(PropFind $propFind, \Sabre\DAV\INode $node) {
        $httpRequest = $this->server->httpRequest;

        if ($node instanceof \OCA\DAV\Connector\Sabre\Node) {
            if (!$node->getFileInfo()->isReadable()) {

Function httpGet has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    public function httpGet(RequestInterface $request, ResponseInterface $response) {
        // Only handle valid files
        $node = $this->tree->getNodeForPath($request->getPath());
        if (!($node instanceof IFile)) {
            return;

• Read up
  Read up

Method httpGet has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function httpGet(RequestInterface $request, ResponseInterface $response) {
        // Only handle valid files
        $node = $this->tree->getNodeForPath($request->getPath());
        if (!($node instanceof IFile)) {
            return;

Method initialize has 31 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function initialize(\Sabre\DAV\Server $server) {
        $server->xml->namespaceMap[self::NS_OWNCLOUD] = 'oc';
        $server->protectedProperties[] = self::FILEID_PROPERTYNAME;
        $server->protectedProperties[] = self::INTERNAL_FILEID_PROPERTYNAME;
        $server->protectedProperties[] = self::PERMISSIONS_PROPERTYNAME;

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        Tree $tree,
        IConfig $config,
        IRequest $request,
        $isPublic = false,
        $downloadAttachment = true

Function sendFileIdHeader has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

    public function sendFileIdHeader($filePath, \Sabre\DAV\INode $node = null) {
        // chunked upload handling
        if (\OC_FileChunking::isWebdavChunk()) {
            list($path, $name) = \Sabre\Uri\split($filePath);
            $info = \OC_FileChunking::decodeName($name);

• Read up
  Read up

Avoid too many return statements within this method.

Open

                    return null;

Avoid too many return statements within this method.

Open

                return new ChecksumList($checksum);

Avoid too many return statements within this method.

Open

                return \OC::$server->getURLGenerator()
                    ->linkToRouteAbsolute('files.viewcontroller.showFile', ['fileId' => $node->getInternalFileId()]);

Avoid too many return statements within this method.

Open

                    return null;

Avoid too many return statements within this method.

Open

                return $node->getETag();

Avoid too many return statements within this method.

Open

                return $displayName;

Avoid too many return statements within this method.

Open

                return $node->getSize();

Avoid too many return statements within this method.

Open

                    return false;

Avoid too many return statements within this method.

Open

                    return false;

Avoid too many return statements within this method.

Open

                return $owner->getUID();

Avoid too many return statements within this method.

Open

                return $this->config->getSystemValue('data-fingerprint', '');

Avoid too many return statements within this method.

Open

                return false;

Avoid too many return statements within this method.

Open

                        return $directDownloadUrl['url'];

Avoid too many return statements within this method.

Open

                return true;

Avoid too many return statements within this method.

Open

            return false;

Avoid too many return statements within this method.

Open

                return $node->getSize();

Function handleGetProperties has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function handleGetProperties(PropFind $propFind, \Sabre\DAV\INode $node) {
        $httpRequest = $this->server->httpRequest;

        if ($node instanceof \OCA\DAV\Connector\Sabre\Node) {
            if (!$node->getFileInfo()->isReadable()) {

• Read up
  Read up

Function checkPropFind has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function checkPropFind($request) {
        $path = $request->getPath();
        $node = $this->server->tree->getNodeForPath($path);

        if ($node instanceof Node) {

• Read up
  Read up

Function checkMove has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function checkMove($source, $destination) {
        $sourceNode = $this->tree->getNodeForPath($source);
        if (!$sourceNode instanceof Node) {
            return;
        }

• Read up
  Read up