NativeServer::listShares() - Code Metrics - Inspection of "return non escaped path from exceptions" - icewind1991/SMB - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 328c46...6889d8 )

by Robin

created 2016-08-26 22:40 UTC

NativeServer::listShares() A

↳ Parent: NativeServer

Complexity

Conditions	3
Paths	3

Size

Total Lines	12
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	3

Importance

Changes	3
Bugs	0	Features	1

Metric	Value
c	3
b	0
f	1
dl	0
loc	12
ccs	11
cts	11
cp	1
rs	9.4285
cc	3
eloc	9
nc	3
nop	0
crap	3

<?php
/**
 * Copyright (c) 2014 Robin Appelman <[email protected]>
 * This file is licensed under the Licensed under the MIT license:
 * http://opensource.org/licenses/MIT
 */

namespace Icewind\SMB;

class NativeServer extends Server {
	/**
	 * @var \Icewind\SMB\NativeState
	 */
	protected $state;

	/**
	 * @param string $host
	 * @param string $user
	 * @param string $password
	 */
	public function __construct($host, $user, $password) {
		parent::__construct($host, $user, $password);
		$this->state = new NativeState();
	}

	protected function connect() {
		$this->state->init($this->getWorkgroup(), $this->getUser(), $this->getPassword());
	}

	/**
	 * @return \Icewind\SMB\IShare[]
	 * @throws \Icewind\SMB\Exception\AuthenticationException
	 * @throws \Icewind\SMB\Exception\InvalidHostException
	 */
	public function listShares() {
		$this->connect();
		$shares = array();
		$dh = $this->state->opendir('smb://' . $this->getHost());
		while ($share = $this->state->readdir($dh)) {
			if ($share['type'] === 'file share') {
				$shares[] = $this->getShare($share['name']);
			}
		}
		$this->state->closedir($dh);
		return $shares;
	}

	/**
	 * @param string $name
	 * @return \Icewind\SMB\IShare
	 */
	public function getShare($name) {
		return new NativeShare($this, $name);
	}
}


1		<?php
2		/**
3		* Copyright (c) 2014 Robin Appelman <[email protected]>
4		* This file is licensed under the Licensed under the MIT license:
5		* http://opensource.org/licenses/MIT
6		*/
7
8		namespace Icewind\SMB;
9
10		class NativeServer extends Server {
11		/**
12		* @var \Icewind\SMB\NativeState
13		*/
14		protected $state;
15
16		/**
17		* @param string $host
18		* @param string $user
19		* @param string $password
20		*/
21	506	public function __construct($host, $user, $password) {
22	506	parent::__construct($host, $user, $password);
23	506	$this->state = new NativeState();
24	506	}
25
26	2	protected function connect() {
27	2	$this->state->init($this->getWorkgroup(), $this->getUser(), $this->getPassword());
28	2	}
29
30		/**
31		* @return \Icewind\SMB\IShare[]
32		* @throws \Icewind\SMB\Exception\AuthenticationException
33		* @throws \Icewind\SMB\Exception\InvalidHostException
34		*/
35	2	public function listShares() {
36	2	$this->connect();
37	2	$shares = array();
38	2	$dh = $this->state->opendir('smb://' . $this->getHost());
39	2	while ($share = $this->state->readdir($dh)) {
40	2	if ($share['type'] === 'file share') {
41	2	$shares[] = $this->getShare($share['name']);
42	2	}
43	2	}
44	2	$this->state->closedir($dh);
45	2	return $shares;
46		}
47
48		/**
49		* @param string $name
50		* @return \Icewind\SMB\IShare
51		*/
52	506	public function getShare($name) {
53	506	return new NativeShare($this, $name);
54		}
55		}
56

icewind1991 / SMB

Push — master ( 328c46...6889d8 )

NativeServer::listShares() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like