Class_Method::__construct() - Code Metrics - Inspection of "Move declarations into its own namespace" - Automattic/jetpack - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — try/code-signature-diff ( 5d24fe...2d24ed )

unknown

created 2019-06-24 15:48 UTC

Class_Method::__construct() A

↳ Parent: Class_Method

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	7
Ratio	100 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	5
dl	7
loc	7
rs	10
c	0
b	0
f	0

<?php

namespace Automattic\Jetpack\Analyzer\Declarations;

/**
 * We only log public class methods, whether they are static, and their parameters
 */
class Class_Method extends Declaration {
	public $class_name;
	public $name;
	public $params;
	public $static;

	function __construct( $path, $line, $class_name, $name, $static ) {
		$this->class_name = $class_name;
		$this->name = $name;
		$this->params = array();
		$this->static = $static;
		parent::__construct( $path, $line );
	}

	// TODO: parse "default" into comparable string form?
	function add_param( $name, $default, $type, $byRef, $variadic ) {
		$this->params[] = (object) compact( 'name', 'default', 'type', 'byRef', 'variadic' );
	}

	function to_csv_array() {
		return array(
			$this->type(),
			$this->path,
			$this->line,
			$this->class_name,
			$this->name,
			$this->static,
			json_encode( $this->params )
		);
	}

	function type() {
		return 'method';
	}

	function display_name() {
		$sep = $this->static ? '::' : '->';
		return $this->class_name . $sep . $this->name . '(' . implode( ', ', array_map( function( $param ) { return '$' . $param->name; }, $this->params ) ) . ')';
	}
}

1		<?php
2
3		namespace Automattic\Jetpack\Analyzer\Declarations;
4
5		/**
6		* We only log public class methods, whether they are static, and their parameters
7		*/
8		class Class_Method extends Declaration {
9		public $class_name;
10		public $name;
11		public $params;
12		public $static;
13
14	View Code Duplication	function __construct( $path, $line, $class_name, $name, $static ) {
15		$this->class_name = $class_name;
16		$this->name = $name;
17		$this->params = array();
18		$this->static = $static;
19		parent::__construct( $path, $line );
20		}
21
22		// TODO: parse "default" into comparable string form?
23		function add_param( $name, $default, $type, $byRef, $variadic ) {
24		$this->params[] = (object) compact( 'name', 'default', 'type', 'byRef', 'variadic' );
25		}
26
27	View Code Duplication	function to_csv_array() {
28		return array(
29		$this->type(),
30		$this->path,
31		$this->line,
32		$this->class_name,
33		$this->name,
34		$this->static,
35		json_encode( $this->params )
36		);
37		}
38
39		function type() {
40		return 'method';
41		}
42
43		function display_name() {
44		$sep = $this->static ? '::' : '->';
45		return $this->class_name . $sep . $this->name . '(' . implode( ', ', array_map( function( $param ) { return '$' . $param->name; }, $this->params ) ) . ')';
46		}
47		}

Automattic / jetpack

Push — try/code-signature-diff ( 5d24fe...2d24ed )

Class_Method::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like