OC\Files\Storage\File

Complexity

Total Complexity

5

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getContent()	0	3	1
A	putContent()	0	3	1
A	fopen()	0	3	1
A	delete()	0	3	1
A	hash()	0	3	1

<?php
/**
 * @author Jörn Friedrich Dreyer <jfd@butonic.de>
 *
 * @copyright Copyright (c) 2018, ownCloud GmbH
 * @license AGPL-3.0
 *
 * This code is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License, version 3,
 * as published by the Free Software Foundation.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License, version 3,
 * along with this program.  If not, see <http://www.gnu.org/licenses/>
 *
 */

namespace OC\Files\Storage;

use OCP\Files\File as FilesFile;

class File extends Node implements FilesFile {

	/**
	 * @inheritdoc
	 */
	public function getContent() {
		return $this->storage->file_get_contents($this->path);

The expression $this->storage->file_get_contents($this->path); of type string|false adds false to the return on line 32 which is incompatible with the return type declared by the interface OCP\Files\File::getContent of type string. It seems like you forgot to handle an error condition.

	}

	/**
	 * @inheritdoc
	 */
	public function putContent($data) {
		$this->storage->file_put_contents($this->path, $data);
	}

	/**
	 * @inheritdoc
	 */
	public function fopen($mode) {
		return $this->storage->fopen($this->path, $mode);

The expression $this->storage->fopen($this->path, $mode); of type resource|false adds false to the return on line 46 which is incompatible with the return type declared by the interface OCP\Files\File::fopen of type resource. It seems like you forgot to handle an error condition.

	}

	/**
	 * @inheritdoc
	 */
	public function delete() {
		$this->storage->unlink($this->path);
	}

	/**
	 * @inheritdoc
	 */
	public function hash($type, $raw = false) {
		return $this->storage->hash($type, $this->path, $raw);

The expression $this->storage->hash($type, $this->path, $raw); of type string|false adds false to the return on line 60 which is incompatible with the return type declared by the interface OCP\Files\File::hash of type string. It seems like you forgot to handle an error condition.

	}
}