MakePotCommand - Code Metrics - Inspection of "Merge tag '5.5.4' into develop" - pronamic/wp-pronamic-ideal - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — develop ( a66df3...4f548c )

by Reüel

created 2019-01-24 13:41 UTC

MakePotCommand A

↳ Parent: anonymous()

Complexity

Total Complexity

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Test Coverage

Coverage

Importance

Changes

Metric	Value
dl	0
loc	32
ccs	0
cts	27
cp	0
c	0
b	0
f	0
rs	10
wmc	1

<?php


namespace Pronamic\WordPress\Pay;

/*
 * The `pronamic i18n make-pot` command requires the `i18n make-pot` command.
 *
 * @link https://make.wordpress.org/cli/2017/05/03/managing-command-dependencies/
 */
\WP_CLI::add_hook( 'after_add_command:i18n make-pot', function () {
filter:
    dependency_paths: ["lib/*"]
	class MakePotCommand extends \WP_CLI\I18n\MakePotCommand {
filter:
    dependency_paths: ["lib/*"]
		public function __construct() {

			parent::__construct();


			// https://github.com/wp-cli/i18n-command/blob/v2.0.1/src/MakePotCommand.php#L36-L44

			$this->exclude = array_diff(

				$this->exclude,

				array(
					'vendor',
				)
			);

			$this->exclude = array_merge(

				$this->exclude,

				array(
					'build',
					'deploy',
					'documentation',
					'etc',
					'repositories',
					'wordpress',
					'wp-content',
				)
			);

			$this->include = array(

				'admin',
				'includes',
				'templates',
				'vendor',
				'views',
				'*.php',
			);
		}
	}

	// https://github.com/wp-cli/i18n-command/blob/v2.0.1/i18n-command.php

	\WP_CLI::add_command( 'pronamic i18n make-pot', '\Pronamic\WordPress\Pay\MakePotCommand' );
	// wp pronamic i18n make-pot . languages/pronamic_ideal.pot --slug="pronamic-ideal"

} );
someFunctionCall(
    $firstArgument,
    $secondArgument,
    $thirdArgument
); // Closing parenthesis on a new line.


1			<?php
			0 ignored issues – show Coding Style introduced 2018-05-09 11:52 UTC by Report Bug Copy Issue Report This file is missing a doc comment. Loading history...
2
3			namespace Pronamic\WordPress\Pay;
4
5			/*
6			* The `pronamic i18n make-pot` command requires the `i18n make-pot` command.
7			*
8			* @link https://make.wordpress.org/cli/2017/05/03/managing-command-dependencies/
9			*/
10			\WP_CLI::add_hook( 'after_add_command:i18n make-pot', function () {
			0 ignored issues – show Bug introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report The type `WP_CLI` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history... Coding Style introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report The opening parenthesis of a multi-line function call should be the last content on the line. Loading history... Coding Style introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report For multi-line function calls, each argument should be on a separate line. For a function calls that spawns multiple lines, the coding style suggests to split arguments to separate lines like this: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); Loading history...
11			class MakePotCommand extends \WP_CLI\I18n\MakePotCommand {
			1 ignored issue – show Bug introduced 2018-09-21 13:16 UTC by Report Bug Copy Issue Report The type `WP_CLI\I18n\MakePotCommand` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history... Coding Style Documentation introduced 2018-09-21 13:16 UTC by Report Bug Copy Issue Report Missing class doc comment Loading history...
12			public function __construct() {
			0 ignored issues – show Coding Style Documentation introduced 2018-09-21 13:16 UTC by Report Bug Copy Issue Report Missing function doc comment Loading history...
13			parent::__construct();
			0 ignored issues – show introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions could not use "parent::" in PHP 5.3 or earlier Loading history...
14
15			// https://github.com/wp-cli/i18n-command/blob/v2.0.1/src/MakePotCommand.php#L36-L44
			0 ignored issues – show Coding Style introduced 2018-09-21 13:16 UTC by Report Bug Copy Issue Report Inline comments must end in full-stops, exclamation marks, or question marks Loading history...
16			$this->exclude = array_diff(
			1 ignored issue – show Bug Best Practice introduced 2018-12-18 09:13 UTC by Report Bug Copy Issue Report The property `exclude` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history... introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions only have access to $this if used within a class or when bound to an object using bindTo(). Please verify. Loading history... introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions did not have access to $this in PHP 5.3 or earlier Loading history...
17			$this->exclude,
			0 ignored issues – show introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions only have access to $this if used within a class or when bound to an object using bindTo(). Please verify. Loading history... introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions did not have access to $this in PHP 5.3 or earlier Loading history...
18			array(
19			'vendor',
20			)
21			);
22
23			$this->exclude = array_merge(
			0 ignored issues – show introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions only have access to $this if used within a class or when bound to an object using bindTo(). Please verify. Loading history... introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions did not have access to $this in PHP 5.3 or earlier Loading history...
24			$this->exclude,
			0 ignored issues – show introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions only have access to $this if used within a class or when bound to an object using bindTo(). Please verify. Loading history... introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions did not have access to $this in PHP 5.3 or earlier Loading history...
25			array(
26			'build',
27			'deploy',
28			'documentation',
29			'etc',
30			'repositories',
31			'wordpress',
32			'wp-content',
33			)
34			);
35
36			$this->include = array(
			1 ignored issue – show Bug Best Practice introduced 2018-09-21 13:16 UTC by Report Bug Copy Issue Report The property `include` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history... introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions only have access to $this if used within a class or when bound to an object using bindTo(). Please verify. Loading history... introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report Closures / anonymous functions did not have access to $this in PHP 5.3 or earlier Loading history...
37			'admin',
38			'includes',
39			'templates',
40			'vendor',
41			'views',
42			'*.php',
43			);
44			}
45			}
46
47			// https://github.com/wp-cli/i18n-command/blob/v2.0.1/i18n-command.php
			0 ignored issues – show Coding Style introduced 2018-09-21 13:16 UTC by Report Bug Copy Issue Report Inline comments must end in full-stops, exclamation marks, or question marks Loading history...
48			\WP_CLI::add_command( 'pronamic i18n make-pot', '\Pronamic\WordPress\Pay\MakePotCommand' );
49			// wp pronamic i18n make-pot . languages/pronamic_ideal.pot --slug="pronamic-ideal"
			0 ignored issues – show Coding Style introduced 2018-09-21 13:16 UTC by Report Bug Copy Issue Report Inline comments must end in full-stops, exclamation marks, or question marks Loading history...
50			} );
			0 ignored issues – show Coding Style introduced 2019-01-22 14:02 UTC by Report Bug Copy Issue Report For multi-line function calls, the closing parenthesis should be on a new line. If a function call spawns multiple lines, the coding standard suggests to move the closing parenthesis to a new line: someFunctionCall( $firstArgument, $secondArgument, $thirdArgument ); // Closing parenthesis on a new line. Loading history...
51

pronamic / wp-pronamic-ideal

Push — develop ( a66df3...4f548c )

MakePotCommand A

Complexity

Size/Duplication

Test Coverage

Importance

Duplication Side-by-Side

Filter issues like