FilesWebDavClient::getFile() - Code Metrics - Inspection of "dep: include phpunit/phpunit for scrutinizer build" - grommunio/grommunio-web - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Branch — master (340a00)

by Mike

created 2024-05-12 00:15 UTC

FilesWebDavClient::getFile() D

↳ Parent: Project

Complexity

Conditions	18
Paths	58

Size

Total Lines	81
Code Lines	50

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
cc	18
eloc	50
c	1
b	1
f	0
nc	58
nop	3
dl	0
loc	81
rs	4.8666

How to fix Long Method Complexity

<?php
/**
 * File plugin webdav client.
 * Overrides the request function to add support for downloading really large files with less memory.
 */

namespace Files\Backend\Webdav\sabredav;

include __DIR__ . "/vendor/autoload.php";

class FilesWebDavClient extends \Sabre\DAV\Client {
	public function __construct(array $settings) {
		if (isset($settings['userName'])) {
			$this->userName = $settings['userName'];
		}
		if (isset($settings['password'])) {
			$this->password = $settings['password'];
		}
		parent::__construct($settings);
	}

	/**
	 * Performs an actual HTTP request, and returns the result.
	 *
	 * If the specified url is relative, it will be expanded based on the base
	 * url.
	 *
	 * The returned array contains 3 keys:
	 *   * body - the response body
	 *   * httpCode - a HTTP code (200, 404, etc)
	 *   * headers - a list of response http headers. The header names have
	 *     been lowercased.
	 *
	 * @param string $url
	 * @param string $dstpath
	 * @param array  $headers
	 *
	 * @return array
	 */
	public function getFile($url, $dstpath, $headers = []) {
		if (empty($url)) {
			throw new \Sabre\DAV\Exception('Empty path');
		}
		$url = $this->getAbsoluteUrl($url);
		$file_handle = fopen($dstpath, "w");

		if (!$file_handle) {
			throw new \Sabre\DAV\Exception('[CURL] Error writing to temporary file! (' . $dstpath . ')');
		}

		// straight up curl instead of sabredav here, sabredav put's the entire get result in memory
		$curl = curl_init($url);

		if ($this->verifyPeer !== null) {
			curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, $this->verifyPeer);
		}
		if ($this->trustedCertificates) {
			curl_setopt($curl, CURLOPT_CAINFO, $this->trustedCertificates);
		}

		curl_setopt($curl, CURLOPT_USERPWD, $this->userName . ":" . $this->password);
		curl_setopt($curl, CURLOPT_FILE, $file_handle);
		curl_setopt($curl, CURLOPT_FOLLOWLOCATION, true);
		curl_setopt($curl, CURLOPT_PROTOCOLS, CURLPROTO_HTTP | CURLPROTO_HTTPS);
		curl_setopt($curl, CURLOPT_REDIR_PROTOCOLS, CURLPROTO_HTTP | CURLPROTO_HTTPS);

		curl_exec($curl);

		$statusCode = curl_getinfo($curl, CURLINFO_HTTP_CODE);

		curl_close($curl);

		$response = [
			'statusCode' => $statusCode,
		];

		if ($response['statusCode'] >= 400) {
			switch ($response['statusCode']) {
				case 400 :
					throw new \Sabre\DAV\Exception\BadRequest('Bad request');

				case 401 :
					throw new \Sabre\DAV\Exception\NotAuthenticated('Not authenticated');

				case 402 :
					throw new \Sabre\DAV\Exception\PaymentRequired('Payment required');

				case 403 :
					throw new \Sabre\DAV\Exception\Forbidden('Forbidden');

				case 404:
					throw new \Sabre\DAV\Exception\NotFound('Resource not found.');

				case 405 :
					throw new \Sabre\DAV\Exception\MethodNotAllowed('Method not allowed');

				case 409 :
					throw new \Sabre\DAV\Exception\Conflict('Conflict');

				case 412 :
					throw new \Sabre\DAV\Exception\PreconditionFailed('Precondition failed');

				case 416 :
					throw new \Sabre\DAV\Exception\RequestedRangeNotSatisfiable('Requested Range Not Satisfiable');

				case 500 :
					throw new \Sabre\DAV\Exception('Internal server error');

				case 501 :
					throw new \Sabre\DAV\Exception\NotImplemented('Not Implemented');

				case 507 :
					throw new \Sabre\DAV\Exception\InsufficientStorage('Insufficient storage');

				default:
					throw new \Sabre\DAV\Exception('HTTP error response. (errorcode ' . $response['statusCode'] . ')');
			}
		}

		return $response;
	}

	public function uploadChunkedFile($destination, $resource) {
		return $this->request("PUT", $destination, $resource);
	}
}


1			<?php
2			/**
3			* File plugin webdav client.
4			* Overrides the request function to add support for downloading really large files with less memory.
5			*/
6
7			namespace Files\Backend\Webdav\sabredav;
8
9			include __DIR__ . "/vendor/autoload.php";
10
11			class FilesWebDavClient extends \Sabre\DAV\Client {
12			public function __construct(array $settings) {
13			if (isset($settings['userName'])) {
14			$this->userName = $settings['userName'];
15			}
16			if (isset($settings['password'])) {
17			$this->password = $settings['password'];
18			}
19			parent::__construct($settings);
20			}
21
22			/**
23			* Performs an actual HTTP request, and returns the result.
24			*
25			* If the specified url is relative, it will be expanded based on the base
26			* url.
27			*
28			* The returned array contains 3 keys:
29			* * body - the response body
30			* * httpCode - a HTTP code (200, 404, etc)
31			* * headers - a list of response http headers. The header names have
32			* been lowercased.
33			*
34			* @param string $url
35			* @param string $dstpath
36			* @param array $headers
37			*
38			* @return array
39			*/
40			public function getFile($url, $dstpath, $headers = []) {
41			if (empty($url)) {
42			throw new \Sabre\DAV\Exception('Empty path');
43			}
44			$url = $this->getAbsoluteUrl($url);
45			$file_handle = fopen($dstpath, "w");
46
47			if (!$file_handle) {
48			throw new \Sabre\DAV\Exception('[CURL] Error writing to temporary file! (' . $dstpath . ')');
49			}
50
51			// straight up curl instead of sabredav here, sabredav put's the entire get result in memory
52			$curl = curl_init($url);
53
54			if ($this->verifyPeer !== null) {
55			curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, $this->verifyPeer);
56			}
57			if ($this->trustedCertificates) {
58			curl_setopt($curl, CURLOPT_CAINFO, $this->trustedCertificates);
59			}
60
61			curl_setopt($curl, CURLOPT_USERPWD, $this->userName . ":" . $this->password);
62			curl_setopt($curl, CURLOPT_FILE, $file_handle);
63			curl_setopt($curl, CURLOPT_FOLLOWLOCATION, true);
64			curl_setopt($curl, CURLOPT_PROTOCOLS, CURLPROTO_HTTP \| CURLPROTO_HTTPS);
65			curl_setopt($curl, CURLOPT_REDIR_PROTOCOLS, CURLPROTO_HTTP \| CURLPROTO_HTTPS);
66
67			curl_exec($curl);
68
69			$statusCode = curl_getinfo($curl, CURLINFO_HTTP_CODE);
70
71			curl_close($curl);
72
73			$response = [
74			'statusCode' => $statusCode,
75			];
76
77			if ($response['statusCode'] >= 400) {
78			switch ($response['statusCode']) {
79			case 400 :
80			throw new \Sabre\DAV\Exception\BadRequest('Bad request');
81
82			case 401 :
83			throw new \Sabre\DAV\Exception\NotAuthenticated('Not authenticated');
84
85			case 402 :
86			throw new \Sabre\DAV\Exception\PaymentRequired('Payment required');
87
88			case 403 :
89			throw new \Sabre\DAV\Exception\Forbidden('Forbidden');
90
91			case 404:
92			throw new \Sabre\DAV\Exception\NotFound('Resource not found.');
93
94			case 405 :
95			throw new \Sabre\DAV\Exception\MethodNotAllowed('Method not allowed');
96
97			case 409 :
98			throw new \Sabre\DAV\Exception\Conflict('Conflict');
99
100			case 412 :
101			throw new \Sabre\DAV\Exception\PreconditionFailed('Precondition failed');
102
103			case 416 :
104			throw new \Sabre\DAV\Exception\RequestedRangeNotSatisfiable('Requested Range Not Satisfiable');
105
106			case 500 :
107			throw new \Sabre\DAV\Exception('Internal server error');
108
109			case 501 :
110			throw new \Sabre\DAV\Exception\NotImplemented('Not Implemented');
111
112			case 507 :
113			throw new \Sabre\DAV\Exception\InsufficientStorage('Insufficient storage');
114
115			default:
116			throw new \Sabre\DAV\Exception('HTTP error response. (errorcode ' . $response['statusCode'] . ')');
117			}
118			}
119
120			return $response;
121			}
122
123			public function uploadChunkedFile($destination, $resource) {
124			return $this->request("PUT", $destination, $resource);
125			}
126			}
127

grommunio / grommunio-web

Branch — master (340a00)

FilesWebDavClient::getFile() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like