untrusted_connection::run() - Code Metrics - Inspection of "Merge pull request #69 from senky/ad-code-analysis" - phpbb-extensions/ad-management - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f9651b...451a88 )

by Matt

created 2017-08-09 15:46 UTC

untrusted_connection::run() A

↳ Parent: untrusted_connection

Complexity

Conditions	3
Paths	2

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	13
ccs	8
cts	8
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	7
nc	2
nop	1
crap	3

<?php
/**
 *
 * Advertisement management. An extension for the phpBB Forum Software package.
 *
 * @copyright (c) 2017 phpBB Limited <https://www.phpbb.com>
 * @license GNU General Public License, version 2 (GPL-2.0)
 *
 */

namespace phpbb\ads\analyser\test;

class untrusted_connection implements test_interface
{
	/** @var \phpbb\request\request */
	protected $request;

	/**
	 * Construct an ad code analysis manager object
	 *
	 * @param \phpbb\request\request $request Request object
	 */
	public function __construct(\phpbb\request\request $request)
	{
		$this->request = $request;
	}

	/**
	 * {@inheritDoc}
	 *
	 * Untrusted connection test.
	 * When board runs on HTTPS and ad tries to load a file from
	 * HTTP source, browser throws a warning. We should prevent that.
	 */
	public function run($ad_code)
	{
		$is_https = $this->request->server('HTTPS', false);
		if ($is_https && preg_match('/http[^s]/', $ad_code))
		{
			return array(
				'severity'	=> 'warning',
				'message'	=> 'UNSECURE_CONNECTION',
			);
		}

		return false;
	}
}


1		<?php
2		/**
3		*
4		* Advertisement management. An extension for the phpBB Forum Software package.
5		*
6		* @copyright (c) 2017 phpBB Limited <https://www.phpbb.com>
7		* @license GNU General Public License, version 2 (GPL-2.0)
8		*
9		*/
10
11		namespace phpbb\ads\analyser\test;
12
13		class untrusted_connection implements test_interface
14		{
15		/** @var \phpbb\request\request */
16		protected $request;
17
18		/**
19		* Construct an ad code analysis manager object
20		*
21		* @param \phpbb\request\request $request Request object
22		*/
23	13	public function __construct(\phpbb\request\request $request)
24		{
25	13	$this->request = $request;
26	13	}
27
28		/**
29		* {@inheritDoc}
30		*
31		* Untrusted connection test.
32		* When board runs on HTTPS and ad tries to load a file from
33		* HTTP source, browser throws a warning. We should prevent that.
34		*/
35	13	public function run($ad_code)
36		{
37	13	$is_https = $this->request->server('HTTPS', false);
38	13	if ($is_https && preg_match('/http[^s]/', $ad_code))
39	13	{
40		return array(
41	2	'severity' => 'warning',
42	2	'message' => 'UNSECURE_CONNECTION',
43	2	);
44		}
45
46	11	return false;
47		}
48		}
49

phpbb-extensions / ad-management

Push — master ( f9651b...451a88 )

untrusted_connection::run() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like