Issues in AbstractIntegration.php - Failed Issues - Inspection of "Integration supports webhook." - wp-pay-gateways/ideal-basic - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Push — develop ( a7c8c9...652c37 )

by Remco

created 2019-05-31 10:25 UTC

src/AbstractIntegration.php (1 issue)

Labels

Severity

Major 1

<?php

namespace Pronamic\WordPress\Pay\Gateways\IDealBasic;

use Pronamic\WordPress\Pay\Gateways\IDeal\AbstractIntegration as IDeal_AbstractIntegration;

/**
 * Title: Abstract integration
 * Description:
 * Copyright: 2005-2019 Pronamic
 * Company: Pronamic
 *
 * @author  Remco Tolsma
 * @version 2.0.0
 * @since   1.0.0
 */
abstract class AbstractIntegration extends IDeal_AbstractIntegration {
	public function __construct() {
		// Actions.
		$function = array( __NAMESPACE__ . '\Listener', 'listen' );

		if ( ! has_action( 'wp_loaded', $function ) ) {
			add_action( 'wp_loaded', $function );
		}
	}

	public function get_config_factory_class() {
		return __NAMESPACE__ . '\ConfigFactory';
	}

	public function get_settings_class() {
		return array(
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
			'Pronamic\WordPress\Pay\Gateways\IDeal\Settings',
			__NAMESPACE__ . '\Settings',
		);
	}

	/**
	 * Get required settings for this integration.
	 *
	 * @link https://github.com/wp-premium/gravityforms/blob/1.9.16/includes/fields/class-gf-field-multiselect.php#L21-L42
	 * @since 1.1.3
	 * @return array
	 */
	public function get_settings() {
		$settings = parent::get_settings();

		$settings[] = 'ideal-basic';

		return $settings;
	}
}


1			<?php
2
3			namespace Pronamic\WordPress\Pay\Gateways\IDealBasic;
4
5			use Pronamic\WordPress\Pay\Gateways\IDeal\AbstractIntegration as IDeal_AbstractIntegration;
6
7			/**
8			* Title: Abstract integration
9			* Description:
10			* Copyright: 2005-2019 Pronamic
11			* Company: Pronamic
12			*
13			* @author Remco Tolsma
14			* @version 2.0.0
15			* @since 1.0.0
16			*/
17			abstract class AbstractIntegration extends IDeal_AbstractIntegration {
18			public function __construct() {
19			// Actions.
20			$function = array( __NAMESPACE__ . '\Listener', 'listen' );
21
22			if ( ! has_action( 'wp_loaded', $function ) ) {
23			add_action( 'wp_loaded', $function );
24			}
25			}
26
27			public function get_config_factory_class() {
28			return __NAMESPACE__ . '\ConfigFactory';
29			}
30
31			public function get_settings_class() {
32			return array(
			0 ignored issues – show Bug Best Practice introduced 2019-05-28 13:15 UTC by Report Bug Copy Issue Report Show Similar Issues like this The expression `return array('Pronamic\W...ESPACE__ . '\Settings')` returns the type `array<integer,string>` which is incompatible with the return type mandated by `Pronamic\WordPress\Pay\G...e::get_settings_class()` of `string`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
33			'Pronamic\WordPress\Pay\Gateways\IDeal\Settings',
34			__NAMESPACE__ . '\Settings',
35			);
36			}
37
38			/**
39			* Get required settings for this integration.
40			*
41			* @link https://github.com/wp-premium/gravityforms/blob/1.9.16/includes/fields/class-gf-field-multiselect.php#L21-L42
42			* @since 1.1.3
43			* @return array
44			*/
45			public function get_settings() {
46			$settings = parent::get_settings();
47
48			$settings[] = 'ideal-basic';
49
50			return $settings;
51			}
52			}
53

wp-pay-gateways / ideal-basic

Push — develop ( a7c8c9...652c37 )

src/AbstractIntegration.php (1 issue)

Labels

Severity

Introduced By

Duplication Side-by-Side

Filter issues like