DocumentController::index() - Code Metrics - Inspection of "Add support for adding canonical webroot (#157)" - nextcloud/richdocuments - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fec3a7...a08c0b )

by Andras

created 2018-01-31 12:25 UTC

DocumentController::index() A

↳ Parent: DocumentController

Complexity

Conditions	4
Paths	24

Size

Total Lines	54
Code Lines	43

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	54
ccs	0
cts	50
cp	0
rs	9.0306
cc	4
eloc	43
nc	24
nop	1
crap	20

How to fix Long Method

<?php
/**
 * ownCloud - Richdocuments App
 *
 * @author Victor Dubiniuk
 * @copyright 2014 Victor Dubiniuk [email protected]
 *
 * This file is licensed under the Affero General Public License version 3 or
 * later.
 */

namespace OCA\Richdocuments\Controller;

use OCA\Richdocuments\TokenManager;
use OCA\Richdocuments\WOPI\Parser;
use \OCP\AppFramework\Controller;
use OCP\AppFramework\Http\JSONResponse;
use OCP\Files\Folder;
use OCP\Files\IRootFolder;
use OCP\Files\Node;
use \OCP\IRequest;
use \OCP\IConfig;
use \OCP\IL10N;
use \OCP\ILogger;
use \OCP\AppFramework\Http\ContentSecurityPolicy;
use \OCP\AppFramework\Http\TemplateResponse;
use \OCA\Richdocuments\AppConfig;
use \OCA\Richdocuments\Helper;
use \OC\Files\View;
use OCP\ISession;
use OCP\Share\IManager;

class DocumentController extends Controller {
	/** @var string */
	private $uid;
	/** @var IL10N */
	private $l10n;
	/** @var IConfig */
	private $settings;
	/** @var AppConfig */
	private $appConfig;
	/** @var ILogger */
	private $logger;
	/** @var Parser */
	private $wopiParser;
	/** @var IManager */
	private $shareManager;
	/** @var TokenManager */
	private $tokenManager;
	/** @var ISession */
	private $session;
	/** @var IRootFolder */
	private $rootFolder;

	const ODT_TEMPLATE_PATH = '/assets/odttemplate.odt';

	/**
	 * @param string $appName
	 * @param IRequest $request
	 * @param IConfig $settings
	 * @param AppConfig $appConfig
	 * @param IL10N $l10n
	 * @param Parser $wopiParser
	 * @param IManager $shareManager
	 * @param TokenManager $tokenManager
	 * @param IRootFolder $rootFolder
	 * @param ISession $session
	 * @param string $UserId
	 */
	public function __construct($appName,
								IRequest $request,
								IConfig $settings,
								AppConfig $appConfig,
								IL10N $l10n,
								Parser $wopiParser,
								IManager $shareManager,
								TokenManager $tokenManager,
								IRootFolder $rootFolder,
								ISession $session,
								$UserId,
								ILogger $logger) {
		parent::__construct($appName, $request);
		$this->uid = $UserId;
		$this->l10n = $l10n;
		$this->settings = $settings;
		$this->appConfig = $appConfig;
		$this->wopiParser = $wopiParser;
		$this->shareManager = $shareManager;
		$this->tokenManager = $tokenManager;
		$this->rootFolder = $rootFolder;
		$this->session = $session;
		$this->logger = $logger;
	}

	/**
	 * @PublicPage
	 * @NoCSRFRequired
	 *
	 * Returns the access_token and urlsrc for WOPI access for given $fileId
	 * Requests is accepted only when a secret_token is provided set by admin in
	 * settings page
	 *
	 * @param string $fileId
	 * @return access_token, urlsrc

	 */
	public function extAppGetData($fileId) {
		$secretToken = $this->request->getParam('secret_token');
		$apps = array_filter(explode(',', $this->appConfig->getAppValue('external_apps')));
		foreach($apps as $app) {
			if ($app !== '') {
				if ($secretToken === $app) {
					$appName = explode(':', $app);
					\OC::$server->getLogger()->debug('External app "{extApp}" authenticated; issuing access token for fileId {fileId}', [
						'app' => $this->appName,
						'extApp' => $appName[0],
						'fileId' => $fileId
					]);
					try {
						$folder = $this->rootFolder->getUserFolder($this->uid);
						$item = $folder->getById($fileId)[0];
						if(!($item instanceof Node)) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
							throw new \Exception();
						}
						list($urlSrc, $token) = $this->tokenManager->getToken($item->getId());
						return array(
							'status' => 'success',
							'urlsrc' => $urlSrc,
							'token' => $token
						);
					} catch (\Exception $e) {
						$this->logger->logException($e, ['app'=>'richdocuments']);
						$params = [
							'remoteAddr' => $this->request->getRemoteAddress(),
							'requestID' => $this->request->getId(),
							'debugMode' => $this->settings->getSystemValue('debug'),
							'errorClass' => get_class($e),
							'errorCode' => $e->getCode(),
							'errorMsg' => $e->getMessage(),
							'file' => $e->getFile(),
							'line' => $e->getLine(),
							'trace' => $e->getTraceAsString()
						];
						return new TemplateResponse('core', 'exception', $params, 'guest');
					}
				}
			}

			return array(
				'status' => 'error',
				'message' => 'Permission denied'
			);
		}
	}

	/**
	 * @NoAdminRequired
	 *
	 * @param string $fileId
	 * @return TemplateResponse
	 */
	public function index($fileId) {
		try {
			$folder = $this->rootFolder->getUserFolder($this->uid);
			$item = $folder->getById($fileId)[0];
			if(!($item instanceof Node)) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
				throw new \Exception();
			}
			list($urlSrc, $token) = $this->tokenManager->getToken($item->getId());
			$params = [
				'permissions' => $item->getPermissions(),
				'title' => $item->getName(),
				'fileId' => $item->getId() . '_' . $this->settings->getSystemValue('instanceid'),
				'token' => $token,
				'urlsrc' => $urlSrc,
				'path' => $folder->getRelativePath($item->getPath()),
				'instanceId' => $this->settings->getSystemValue('instanceid'),
				'canonical_webroot' => $this->appConfig->getAppValue('canonical_webroot'),
			];

			$encryptionManager = \OC::$server->getEncryptionManager();
			if ($encryptionManager->isEnabled())
			{
				// Update the current file to be accessible with system public shared key
				$owner = $item->getOwner()->getUID();
				$absPath = '/' . $owner . '/' .  $item->getInternalPath();
				$accessList = \OC::$server->getEncryptionFilesHelper()->getAccessList($absPath);
				$accessList['public'] = true;
				$encryptionManager->getEncryptionModule()->update($absPath, $owner, $accessList);
			}

			$response = new TemplateResponse('richdocuments', 'documents', $params, 'empty');
			$policy = new ContentSecurityPolicy();
			$policy->addAllowedFrameDomain($this->appConfig->getAppValue('wopi_url'));
			$policy->allowInlineScript(true);
			$response->setContentSecurityPolicy($policy);
			return $response;
		} catch (\Exception $e) {
			$this->logger->logException($e, ['app'=>'richdocuments']);
			$params = [
				'remoteAddr' => $this->request->getRemoteAddress(),
				'requestID' => $this->request->getId(),
				'debugMode' => $this->settings->getSystemValue('debug'),
				'errorClass' => get_class($e),
				'errorCode' => $e->getCode(),
				'errorMsg' => $e->getMessage(),
				'file' => $e->getFile(),
				'line' => $e->getLine(),
				'trace' => $e->getTraceAsString()
			];
			return new TemplateResponse('core', 'exception', $params, 'guest');
		}

		return new TemplateResponse('core', '403', [], 'guest');
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
	}

	/**
	 * @PublicPage
	 *
	 * @param string $shareToken
	 * @param string $fileName
	 * @return TemplateResponse
	 * @throws \Exception
	 */
	public function publicPage($shareToken, $fileName, $fileId) {
		try {
			$share = $this->shareManager->getShareByToken($shareToken);
			// not authenticated ?
			if($share->getPassword()){
				if (!$this->session->exists('public_link_authenticated')
					|| $this->session->get('public_link_authenticated') !== (string)$share->getId()
				) {
					throw new \Exception('Invalid password');
				}
			}

			$node = $share->getNode();
			if($node instanceof Folder) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
				$item = $node->getById($fileId)[0];
			} else {
				$item = $node;
			}
			if ($item instanceof Node) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
				list($urlSrc, $token) = $this->tokenManager->getToken($item->getId(), $shareToken);
				$params = [
					'permissions' => $share->getPermissions(),
					'title' => $item->getName(),
					'fileId' => $item->getId() . '_' . $this->settings->getSystemValue('instanceid'),
					'token' => $token,
					'urlsrc' => $urlSrc,
					'path' => '/',
					'instanceId' => $this->settings->getSystemValue('instanceid'),
				];

				$response = new TemplateResponse('richdocuments', 'documents', $params, 'empty');
				$policy = new ContentSecurityPolicy();
				$policy->addAllowedFrameDomain($this->appConfig->getAppValue('wopi_url'));
				$policy->allowInlineScript(true);
				$response->setContentSecurityPolicy($policy);
				return $response;
			}
		} catch (\Exception $e) {
			$this->logger->logException($e, ['app'=>'richdocuments']);
			$params = [
				'remoteAddr' => $this->request->getRemoteAddress(),
				'requestID' => $this->request->getId(),
				'debugMode' => $this->settings->getSystemValue('debug'),
				'errorClass' => get_class($e),
				'errorCode' => $e->getCode(),
				'errorMsg' => $e->getMessage(),
				'file' => $e->getFile(),
				'line' => $e->getLine(),
				'trace' => $e->getTraceAsString()
			];
			return new TemplateResponse('core', 'exception', $params, 'guest');
		}

		return new TemplateResponse('core', '403', [], 'guest');
	}

	/**
	 * @NoAdminRequired
	 *
	 * @param string $mimetype
	 * @param string $filename
	 * @param string $dir
	 * @return JSONResponse
	 */
	public function create($mimetype,
						   $filename,
						   $dir){

		$view = new View('/' . $this->uid . '/files');
		if (!$dir){
			$dir = '/';
		}

		$basename = $this->l10n->t('New Document.odt');
		switch ($mimetype) {
			case 'application/vnd.oasis.opendocument.spreadsheet':
				$basename = $this->l10n->t('New Spreadsheet.ods');
				break;
			case 'application/vnd.oasis.opendocument.presentation':
				$basename = $this->l10n->t('New Presentation.odp');
				break;
			case 'application/vnd.openxmlformats-officedocument.wordprocessingml.document':
				$basename = $this->l10n->t('New Document.docx');
				break;
			case 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet':
				$basename = $this->l10n->t('New Spreadsheet.xlsx');
				break;
			case 'application/vnd.openxmlformats-officedocument.presentationml.presentation':
				$basename = $this->l10n->t('New Presentation.pptx');
				break;
			default:
				// to be safe
				$mimetype = 'application/vnd.oasis.opendocument.text';
				break;
		}

		if (!$filename){
			$path = Helper::getNewFileName($view, $dir . '/' . $basename);
		} else {
			$path = $dir . '/' . $filename;
		}

		$content = '';
		if (class_exists('\OC\Files\Type\TemplateManager')){
			$manager = \OC_Helper::getFileTemplateManager();
			$content = $manager->getTemplate($mimetype);
		}

		if (!$content){
			$content = file_get_contents(dirname(__DIR__) . self::ODT_TEMPLATE_PATH);
		}

		if ($content && $view->file_put_contents($path, $content)) {
			$info = $view->getFileInfo($path);
			$ret = $this->wopiParser->getUrlSrc($mimetype);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
			$response =  array(
				'status' => 'success',
				'data' => \OCA\Files\Helper::formatFileInfo($info)
			);
		} else {
			$response =  array(
				'status' => 'error',
				'message' => (string) $this->l10n->t('Can\'t create document')
			);
		}

		return $response;
	}
}


Push — master ( fec3a7...a08c0b )

DocumentController::index() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1			<?php
2			/**
3			* ownCloud - Richdocuments App
4			*
5			* @author Victor Dubiniuk
6			* @copyright 2014 Victor Dubiniuk [email protected]
7			*
8			* This file is licensed under the Affero General Public License version 3 or
9			* later.
10			*/
11
12			namespace OCA\Richdocuments\Controller;
13
14			use OCA\Richdocuments\TokenManager;
15			use OCA\Richdocuments\WOPI\Parser;
16			use \OCP\AppFramework\Controller;
17			use OCP\AppFramework\Http\JSONResponse;
18			use OCP\Files\Folder;
19			use OCP\Files\IRootFolder;
20			use OCP\Files\Node;
21			use \OCP\IRequest;
22			use \OCP\IConfig;
23			use \OCP\IL10N;
24			use \OCP\ILogger;
25			use \OCP\AppFramework\Http\ContentSecurityPolicy;
26			use \OCP\AppFramework\Http\TemplateResponse;
27			use \OCA\Richdocuments\AppConfig;
28			use \OCA\Richdocuments\Helper;
29			use \OC\Files\View;
30			use OCP\ISession;
31			use OCP\Share\IManager;
32
33			class DocumentController extends Controller {
34			/** @var string */
35			private $uid;
36			/** @var IL10N */
37			private $l10n;
38			/** @var IConfig */
39			private $settings;
40			/** @var AppConfig */
41			private $appConfig;
42			/** @var ILogger */
43			private $logger;
44			/** @var Parser */
45			private $wopiParser;
46			/** @var IManager */
47			private $shareManager;
48			/** @var TokenManager */
49			private $tokenManager;
50			/** @var ISession */
51			private $session;
52			/** @var IRootFolder */
53			private $rootFolder;
54
55			const ODT_TEMPLATE_PATH = '/assets/odttemplate.odt';
56
57			/**
58			* @param string $appName
59			* @param IRequest $request
60			* @param IConfig $settings
61			* @param AppConfig $appConfig
62			* @param IL10N $l10n
63			* @param Parser $wopiParser
64			* @param IManager $shareManager
65			* @param TokenManager $tokenManager
66			* @param IRootFolder $rootFolder
67			* @param ISession $session
68			* @param string $UserId
69			*/
70			public function __construct($appName,
71			IRequest $request,
72			IConfig $settings,
73			AppConfig $appConfig,
74			IL10N $l10n,
75			Parser $wopiParser,
76			IManager $shareManager,
77			TokenManager $tokenManager,
78			IRootFolder $rootFolder,
79			ISession $session,
80			$UserId,
81			ILogger $logger) {
82			parent::__construct($appName, $request);
83			$this->uid = $UserId;
84			$this->l10n = $l10n;
85			$this->settings = $settings;
86			$this->appConfig = $appConfig;
87			$this->wopiParser = $wopiParser;
88			$this->shareManager = $shareManager;
89			$this->tokenManager = $tokenManager;
90			$this->rootFolder = $rootFolder;
91			$this->session = $session;
92			$this->logger = $logger;
93			}
94
95			/**
96			* @PublicPage
97			* @NoCSRFRequired
98			*
99			* Returns the access_token and urlsrc for WOPI access for given $fileId
100			* Requests is accepted only when a secret_token is provided set by admin in
101			* settings page
102			*
103			* @param string $fileId
104			* @return access_token, urlsrc
			0 ignored issues – show Documentation introduced 2017-09-26 07:24 UTC by Report Bug Copy Issue Report The doc-type `access_token,` could not be parsed: Expected "\|" or "end of type", but got "," at position 12. (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
105			*/
106			public function extAppGetData($fileId) {
107			$secretToken = $this->request->getParam('secret_token');
108			$apps = array_filter(explode(',', $this->appConfig->getAppValue('external_apps')));
109			foreach($apps as $app) {
110			if ($app !== '') {
111			if ($secretToken === $app) {
112			$appName = explode(':', $app);
113			\OC::$server->getLogger()->debug('External app "{extApp}" authenticated; issuing access token for fileId {fileId}', [
114			'app' => $this->appName,
115			'extApp' => $appName[0],
116			'fileId' => $fileId
117			]);
118			try {
119			$folder = $this->rootFolder->getUserFolder($this->uid);
120			$item = $folder->getById($fileId)[0];
121			if(!($item instanceof Node)) {
			0 ignored issues – show Bug introduced 2016-12-06 23:59 UTC by Report Bug Copy Issue Report The class `OCP\Files\Node` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
122			throw new \Exception();
123			}
124			list($urlSrc, $token) = $this->tokenManager->getToken($item->getId());
125			return array(
126			'status' => 'success',
127			'urlsrc' => $urlSrc,
128			'token' => $token
129			);
130			} catch (\Exception $e) {
131			$this->logger->logException($e, ['app'=>'richdocuments']);
132			$params = [
133			'remoteAddr' => $this->request->getRemoteAddress(),
134			'requestID' => $this->request->getId(),
135			'debugMode' => $this->settings->getSystemValue('debug'),
136			'errorClass' => get_class($e),
137			'errorCode' => $e->getCode(),
138			'errorMsg' => $e->getMessage(),
139			'file' => $e->getFile(),
140			'line' => $e->getLine(),
141			'trace' => $e->getTraceAsString()
142			];
143			return new TemplateResponse('core', 'exception', $params, 'guest');
144			}
145			}
146			}
147
148			return array(
149			'status' => 'error',
150			'message' => 'Permission denied'
151			);
152			}
153			}
154
155			/**
156			* @NoAdminRequired
157			*
158			* @param string $fileId
159			* @return TemplateResponse
160			*/
161			public function index($fileId) {
162			try {
163			$folder = $this->rootFolder->getUserFolder($this->uid);
164			$item = $folder->getById($fileId)[0];
165			if(!($item instanceof Node)) {
			0 ignored issues – show Bug introduced 2017-09-26 07:24 UTC by Report Bug Copy Issue Report The class `OCP\Files\Node` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
166			throw new \Exception();
167			}
168			list($urlSrc, $token) = $this->tokenManager->getToken($item->getId());
169			$params = [
170			'permissions' => $item->getPermissions(),
171			'title' => $item->getName(),
172			'fileId' => $item->getId() . '_' . $this->settings->getSystemValue('instanceid'),
173			'token' => $token,
174			'urlsrc' => $urlSrc,
175			'path' => $folder->getRelativePath($item->getPath()),
176			'instanceId' => $this->settings->getSystemValue('instanceid'),
177			'canonical_webroot' => $this->appConfig->getAppValue('canonical_webroot'),
178			];
179
180			$encryptionManager = \OC::$server->getEncryptionManager();
181			if ($encryptionManager->isEnabled())
182			{
183			// Update the current file to be accessible with system public shared key
184			$owner = $item->getOwner()->getUID();
185			$absPath = '/' . $owner . '/' . $item->getInternalPath();
186			$accessList = \OC::$server->getEncryptionFilesHelper()->getAccessList($absPath);
187			$accessList['public'] = true;
188			$encryptionManager->getEncryptionModule()->update($absPath, $owner, $accessList);
189			}
190
191			$response = new TemplateResponse('richdocuments', 'documents', $params, 'empty');
192			$policy = new ContentSecurityPolicy();
193			$policy->addAllowedFrameDomain($this->appConfig->getAppValue('wopi_url'));
194			$policy->allowInlineScript(true);
195			$response->setContentSecurityPolicy($policy);
196			return $response;
197			} catch (\Exception $e) {
198			$this->logger->logException($e, ['app'=>'richdocuments']);
199			$params = [
200			'remoteAddr' => $this->request->getRemoteAddress(),
201			'requestID' => $this->request->getId(),
202			'debugMode' => $this->settings->getSystemValue('debug'),
203			'errorClass' => get_class($e),
204			'errorCode' => $e->getCode(),
205			'errorMsg' => $e->getMessage(),
206			'file' => $e->getFile(),
207			'line' => $e->getLine(),
208			'trace' => $e->getTraceAsString()
209			];
210			return new TemplateResponse('core', 'exception', $params, 'guest');
211			}
212
213			return new TemplateResponse('core', '403', [], 'guest');
			0 ignored issues – show Unused Code introduced 2017-04-08 15:38 UTC by Report Bug Copy Issue Report `return new \OCP\AppFrame...03', array(), 'guest');` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
214			}
215
216			/**
217			* @PublicPage
218			*
219			* @param string $shareToken
220			* @param string $fileName
221			* @return TemplateResponse
222			* @throws \Exception
223			*/
224			public function publicPage($shareToken, $fileName, $fileId) {
225			try {
226			$share = $this->shareManager->getShareByToken($shareToken);
227			// not authenticated ?
228			if($share->getPassword()){
229			if (!$this->session->exists('public_link_authenticated')
230			\|\| $this->session->get('public_link_authenticated') !== (string)$share->getId()
231			) {
232			throw new \Exception('Invalid password');
233			}
234			}
235
236			$node = $share->getNode();
237			if($node instanceof Folder) {
			0 ignored issues – show Bug introduced 2016-12-06 23:59 UTC by Report Bug Copy Issue Report The class `OCP\Files\Folder` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
238			$item = $node->getById($fileId)[0];
239			} else {
240			$item = $node;
241			}
242			if ($item instanceof Node) {
			0 ignored issues – show Bug introduced 2016-12-06 23:59 UTC by Report Bug Copy Issue Report The class `OCP\Files\Node` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
243			list($urlSrc, $token) = $this->tokenManager->getToken($item->getId(), $shareToken);
244			$params = [
245			'permissions' => $share->getPermissions(),
246			'title' => $item->getName(),
247			'fileId' => $item->getId() . '_' . $this->settings->getSystemValue('instanceid'),
248			'token' => $token,
249			'urlsrc' => $urlSrc,
250			'path' => '/',
251			'instanceId' => $this->settings->getSystemValue('instanceid'),
252			];
253
254			$response = new TemplateResponse('richdocuments', 'documents', $params, 'empty');
255			$policy = new ContentSecurityPolicy();
256			$policy->addAllowedFrameDomain($this->appConfig->getAppValue('wopi_url'));
257			$policy->allowInlineScript(true);
258			$response->setContentSecurityPolicy($policy);
259			return $response;
260			}
261			} catch (\Exception $e) {
262			$this->logger->logException($e, ['app'=>'richdocuments']);
263			$params = [
264			'remoteAddr' => $this->request->getRemoteAddress(),
265			'requestID' => $this->request->getId(),
266			'debugMode' => $this->settings->getSystemValue('debug'),
267			'errorClass' => get_class($e),
268			'errorCode' => $e->getCode(),
269			'errorMsg' => $e->getMessage(),
270			'file' => $e->getFile(),
271			'line' => $e->getLine(),
272			'trace' => $e->getTraceAsString()
273			];
274			return new TemplateResponse('core', 'exception', $params, 'guest');
275			}
276
277			return new TemplateResponse('core', '403', [], 'guest');
278			}
279
280			/**
281			* @NoAdminRequired
282			*
283			* @param string $mimetype
284			* @param string $filename
285			* @param string $dir
286			* @return JSONResponse
287			*/
288			public function create($mimetype,
289			$filename,
290			$dir){
291
292			$view = new View('/' . $this->uid . '/files');
293			if (!$dir){
294			$dir = '/';
295			}
296
297			$basename = $this->l10n->t('New Document.odt');
298			switch ($mimetype) {
299			case 'application/vnd.oasis.opendocument.spreadsheet':
300			$basename = $this->l10n->t('New Spreadsheet.ods');
301			break;
302			case 'application/vnd.oasis.opendocument.presentation':
303			$basename = $this->l10n->t('New Presentation.odp');
304			break;
305			case 'application/vnd.openxmlformats-officedocument.wordprocessingml.document':
306			$basename = $this->l10n->t('New Document.docx');
307			break;
308			case 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet':
309			$basename = $this->l10n->t('New Spreadsheet.xlsx');
310			break;
311			case 'application/vnd.openxmlformats-officedocument.presentationml.presentation':
312			$basename = $this->l10n->t('New Presentation.pptx');
313			break;
314			default:
315			// to be safe
316			$mimetype = 'application/vnd.oasis.opendocument.text';
317			break;
318			}
319
320			if (!$filename){
321			$path = Helper::getNewFileName($view, $dir . '/' . $basename);
322			} else {
323			$path = $dir . '/' . $filename;
324			}
325
326			$content = '';
327			if (class_exists('\OC\Files\Type\TemplateManager')){
328			$manager = \OC_Helper::getFileTemplateManager();
329			$content = $manager->getTemplate($mimetype);
330			}
331
332			if (!$content){
333			$content = file_get_contents(dirname(__DIR__) . self::ODT_TEMPLATE_PATH);
334			}
335
336			if ($content && $view->file_put_contents($path, $content)) {
337			$info = $view->getFileInfo($path);
338			$ret = $this->wopiParser->getUrlSrc($mimetype);
			0 ignored issues – show Unused Code introduced 2017-04-05 10:44 UTC by Report Bug Copy Issue Report `$ret` 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...
339			$response = array(
340			'status' => 'success',
341			'data' => \OCA\Files\Helper::formatFileInfo($info)
342			);
343			} else {
344			$response = array(
345			'status' => 'error',
346			'message' => (string) $this->l10n->t('Can\'t create document')
347			);
348			}
349
350			return $response;
351			}
352			}
353

nextcloud / richdocuments

Push — master ( fec3a7...a08c0b )

DocumentController::index() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method

Long Method

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like