Socket::__construct() - Code Metrics - Inspection of "Throw exception in constructor. Add connection tes..." - owncloud/files_antivirus - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — closes-181 ( fce50e...02104e )

by Victor

created 2017-10-24 17:13 UTC

Socket::__construct() A

↳ Parent: Socket

Complexity

Conditions	2
Paths	2

Size

Total Lines	16
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	0
cts	14
cp	0
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	11
nc	2
nop	2
crap	6

<?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\Scanner;

use OCA\Files_Antivirus\AppConfig;
use OCP\ILogger;

class Socket extends Daemon {

	public function __construct(AppConfig $config, ILogger $logger) {
		parent::__construct($config, $logger);
		$this->writeHandle = stream_socket_client(
			'unix://' . $this->appConfig->getAvSocket(), $errorCode, $errorMessage, 5
		);
		if (!$this->getWriteHandle()) {
			throw new InitException(
				sprintf(
					'Could not connect to socket "%s": %s (code %d)',
					$this->appConfig->getAvSocket(),
					$errorMessage,
					$errorCode
				)
			);
		}
	}

	public function initScanner(){
		parent::initScanner();

		if (@fwrite($this->getWriteHandle(), "nINSTREAM\n") === false) {
			throw new InitException(
				sprintf(
					'Writing to socket "%s" failed',
					$this->appConfig->getAvSocket()
				)
			);
		}
	}
}


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
10			namespace OCA\Files_Antivirus\Scanner;
11
12			use OCA\Files_Antivirus\AppConfig;
13			use OCP\ILogger;
14
15			class Socket extends Daemon {
16
17			public function __construct(AppConfig $config, ILogger $logger) {
18			parent::__construct($config, $logger);
19			$this->writeHandle = stream_socket_client(
20			'unix://' . $this->appConfig->getAvSocket(), $errorCode, $errorMessage, 5
21			);
22			if (!$this->getWriteHandle()) {
23			throw new InitException(
24			sprintf(
25			'Could not connect to socket "%s": %s (code %d)',
26			$this->appConfig->getAvSocket(),
27			$errorMessage,
28			$errorCode
29			)
30			);
31			}
32			}
33
34			public function initScanner(){
35			parent::initScanner();
36
37			if (@fwrite($this->getWriteHandle(), "nINSTREAM\n") === false) {
38			throw new InitException(
39			sprintf(
40			'Writing to socket "%s" failed',
41			$this->appConfig->getAvSocket()
42			)
43			);
44			}
45			}
46			}
47

owncloud / files_antivirus

Push — closes-181 ( fce50e...02104e )

Socket::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like