RequestHelper::getUploadSize() - Code Metrics - Inspection of "Merge pull request #196 from owncloud/skip-by-size" - owncloud/files_antivirus - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9b82db...1365a9 )

by Victor

created 2017-11-16 15:08 UTC

RequestHelper::getUploadSize() C

↳ Parent: RequestHelper

Complexity

Conditions	7
Paths	4

Size

Total Lines	24
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	13.125

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	24
ccs	6
cts	12
cp	0.5
rs	6.7272
c	1
b	0
f	0
cc	7
eloc	14
nc	4
nop	1
crap	13.125

<?php
/**
 * Copyright (c) 2017 Viktar Dubiniuk <[email protected]>
 * This file is licensed under the Affero General Public License version 3 or
 * later.
 * See the COPYING-README file.
 */

namespace OCA\Files_Antivirus;

use \OCP\IRequest;

class RequestHelper {

	/** @var  IRequest */
	private $request;

	public function __construct(IRequest $request) {
		$this->request = $request;
	}

	/**
	 * Get current upload size
	 * returns null for chunks and when there is no upload
	 *
	 * @param string $path
	 * @return int|null
	 */
	public function getUploadSize($path) {
		$uploadSize = null;

		$requestMethod = $this->request->getMethod();
		$isRemoteScript = $this->isScriptName('remote.php');
		// Are we uploading anything?
		if (in_array($requestMethod, ['MOVE', 'PUT']) && $isRemoteScript) {
			// v1 && v2 Chunks are not scanned
			if (
				\OC_FileChunking::isWebdavChunk()
				|| ($requestMethod === 'PUT' &&  strpos($path, 'uploads/') === 0)
			) {
				return null;
			}

			if ($requestMethod === 'PUT') {
				$uploadSize = (int)$this->request->getHeader('CONTENT_LENGTH');
			} else {
				$uploadSize = (int)$this->request->getHeader('OC_TOTAL_LENGTH');
			}
		}

		return $uploadSize;
	}

	/**
	 *
	 * @param string $string
	 * @return bool
	 */
	public function isScriptName($string) {
		$pattern = sprintf('|/%s|', preg_quote($string));
		return preg_match($pattern, $this->request->getScriptName()) === 1;
	}
}


1		<?php
2		/**
3		* Copyright (c) 2017 Viktar Dubiniuk <[email protected]>
4		* This file is licensed under the Affero General Public License version 3 or
5		* later.
6		* See the COPYING-README file.
7		*/
8
9		namespace OCA\Files_Antivirus;
10
11		use \OCP\IRequest;
12
13		class RequestHelper {
14
15		/** @var IRequest */
16		private $request;
17
18	2	public function __construct(IRequest $request) {
19	2	$this->request = $request;
20	2	}
21
22		/**
23		* Get current upload size
24		* returns null for chunks and when there is no upload
25		*
26		* @param string $path
27		* @return int\|null
28		*/
29	2	public function getUploadSize($path) {
30	2	$uploadSize = null;
31
32	2	$requestMethod = $this->request->getMethod();
33	2	$isRemoteScript = $this->isScriptName('remote.php');
34		// Are we uploading anything?
35	2	if (in_array($requestMethod, ['MOVE', 'PUT']) && $isRemoteScript) {
36		// v1 && v2 Chunks are not scanned
37		if (
38		\OC_FileChunking::isWebdavChunk()
39		\|\| ($requestMethod === 'PUT' && strpos($path, 'uploads/') === 0)
40		) {
41		return null;
42		}
43
44		if ($requestMethod === 'PUT') {
45		$uploadSize = (int)$this->request->getHeader('CONTENT_LENGTH');
46		} else {
47		$uploadSize = (int)$this->request->getHeader('OC_TOTAL_LENGTH');
48		}
49		}
50
51	2	return $uploadSize;
52		}
53
54		/**
55		*
56		* @param string $string
57		* @return bool
58		*/
59	2	public function isScriptName($string) {
60	2	$pattern = sprintf('\|/%s\|', preg_quote($string));
61	2	return preg_match($pattern, $this->request->getScriptName()) === 1;
62		}
63		}
64

owncloud / files_antivirus

Push — master ( 9b82db...1365a9 )

RequestHelper::getUploadSize() C

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like