woocommerce/woocommerce-paypal-payments
2
0
Fork 0
mirror of https://github.com/woocommerce/woocommerce-paypal-payments.git synced 2025-09-01 07:02:48 +08:00
Code Issues Projects Packages Wiki Activity Actions

Fix Psalm errors

Browse source
This commit is contained in:
Daniel Dudzic 2024-10-18 10:44:02 +02:00
parent 407f526064
commit 763d4a6482
No known key found for this signature in database
GPG key ID: 31B40D33E3465483
1 changed files with 25 additions and 24 deletions
Download patch file Download diff file Expand all files Collapse all files

49
modules/ppcp-axo/services.php
View file

@ -22,14 +22,14 @@ use WooCommerce\PayPalCommerce\WcGateway\Helper\DCCGatewayConfiguration;
return array( return array(
// If AXO can be configured. // If AXO can be configured.
'axo.eligible' => static function ( ContainerInterface $container ): bool { 'axo.eligible' => static function ( ContainerInterface $container ): bool {
$apm_applies = $container->get( 'axo.helpers.apm-applies' ); $apm_applies = $container->get( 'axo.helpers.apm-applies' );
assert( $apm_applies instanceof ApmApplies ); assert( $apm_applies instanceof ApmApplies );
return $apm_applies->for_country_currency(); return $apm_applies->for_country_currency();
}, },
'axo.helpers.apm-applies' => static function ( ContainerInterface $container ) : ApmApplies { 'axo.helpers.apm-applies' => static function ( ContainerInterface $container ) : ApmApplies {
return new ApmApplies( return new ApmApplies(
$container->get( 'axo.supported-country-currency-matrix' ), $container->get( 'axo.supported-country-currency-matrix' ),
$container->get( 'api.shop.currency.getter' ), $container->get( 'api.shop.currency.getter' ),
@ -37,16 +37,16 @@ return array(
); );
}, },
'axo.helpers.settings-notice-generator' => static function ( ContainerInterface $container ) : SettingsNoticeGenerator { 'axo.helpers.settings-notice-generator' => static function ( ContainerInterface $container ) : SettingsNoticeGenerator {
return new SettingsNoticeGenerator( $container->get( 'axo.fastlane-incompatible-plugin-names' ) ); return new SettingsNoticeGenerator( $container->get( 'axo.fastlane-incompatible-plugin-names' ) );
}, },
// If AXO is configured and onboarded. // If AXO is configured and onboarded.
'axo.available' => static function ( ContainerInterface $container ): bool { 'axo.available' => static function ( ContainerInterface $container ): bool {
return true; return true;
}, },
'axo.url' => static function ( ContainerInterface $container ): string { 'axo.url' => static function ( ContainerInterface $container ): string {
$path = realpath( __FILE__ ); $path = realpath( __FILE__ );
if ( false === $path ) { if ( false === $path ) {
return ''; return '';
@ -57,7 +57,7 @@ return array(
); );
}, },
'axo.manager' => static function ( ContainerInterface $container ): AxoManager { 'axo.manager' => static function ( ContainerInterface $container ): AxoManager {
return new AxoManager( return new AxoManager(
$container->get( 'axo.url' ), $container->get( 'axo.url' ),
$container->get( 'ppcp.asset-version' ), $container->get( 'ppcp.asset-version' ),
@ -72,7 +72,7 @@ return array(
); );
}, },
'axo.gateway' => static function ( ContainerInterface $container ): AxoGateway { 'axo.gateway' => static function ( ContainerInterface $container ): AxoGateway {
return new AxoGateway( return new AxoGateway(
$container->get( 'wcgateway.settings.render' ), $container->get( 'wcgateway.settings.render' ),
$container->get( 'wcgateway.settings' ), $container->get( 'wcgateway.settings' ),
@ -93,7 +93,7 @@ return array(
/** /**
* The matrix which countries and currency combinations can be used for AXO. * The matrix which countries and currency combinations can be used for AXO.
*/ */
'axo.supported-country-currency-matrix' => static function ( ContainerInterface $container ) : array { 'axo.supported-country-currency-matrix' => static function ( ContainerInterface $container ) : array {
/** /**
* Returns which countries and currency combinations can be used for AXO. * Returns which countries and currency combinations can be used for AXO.
*/ */
@ -112,7 +112,7 @@ return array(
); );
}, },
'axo.settings-conflict-notice' => static function ( ContainerInterface $container ) : string { 'axo.settings-conflict-notice' => static function ( ContainerInterface $container ) : string {
$settings_notice_generator = $container->get( 'axo.helpers.settings-notice-generator' ); $settings_notice_generator = $container->get( 'axo.helpers.settings-notice-generator' );
assert( $settings_notice_generator instanceof SettingsNoticeGenerator ); assert( $settings_notice_generator instanceof SettingsNoticeGenerator );
@ -122,21 +122,21 @@ return array(
return $settings_notice_generator->generate_settings_conflict_notice( $settings ); return $settings_notice_generator->generate_settings_conflict_notice( $settings );
}, },
'axo.checkout-config-notice' => static function ( ContainerInterface $container ) : string { 'axo.checkout-config-notice' => static function ( ContainerInterface $container ) : string {
$settings_notice_generator = $container->get( 'axo.helpers.settings-notice-generator' ); $settings_notice_generator = $container->get( 'axo.helpers.settings-notice-generator' );
assert( $settings_notice_generator instanceof SettingsNoticeGenerator ); assert( $settings_notice_generator instanceof SettingsNoticeGenerator );
return $settings_notice_generator->generate_checkout_notice(); return $settings_notice_generator->generate_checkout_notice();
}, },
'axo.incompatible-plugins-notice' => static function ( ContainerInterface $container ) : string { 'axo.incompatible-plugins-notice' => static function ( ContainerInterface $container ) : string {
$settings_notice_generator = $container->get( 'axo.helpers.settings-notice-generator' ); $settings_notice_generator = $container->get( 'axo.helpers.settings-notice-generator' );
assert( $settings_notice_generator instanceof SettingsNoticeGenerator ); assert( $settings_notice_generator instanceof SettingsNoticeGenerator );
return $settings_notice_generator->generate_incompatible_plugins_notice(); return $settings_notice_generator->generate_incompatible_plugins_notice();
}, },
'axo.smart-button-location-notice' => static function ( ContainerInterface $container ) : string { 'axo.smart-button-location-notice' => static function ( ContainerInterface $container ) : string {
$dcc_configuration = $container->get( 'wcgateway.configuration.dcc' ); $dcc_configuration = $container->get( 'wcgateway.configuration.dcc' );
assert( $dcc_configuration instanceof DCCGatewayConfiguration ); assert( $dcc_configuration instanceof DCCGatewayConfiguration );
@ -164,7 +164,7 @@ return array(
return '<div class="ppcp-notice ppcp-notice-warning"><p>' . $notice_content . '</p></div>'; return '<div class="ppcp-notice ppcp-notice-warning"><p>' . $notice_content . '</p></div>';
}, },
'axo.endpoint.frontend-logger' => static function ( ContainerInterface $container ): FrontendLoggerEndpoint { 'axo.endpoint.frontend-logger' => static function ( ContainerInterface $container ): FrontendLoggerEndpoint {
return new FrontendLoggerEndpoint( return new FrontendLoggerEndpoint(
$container->get( 'button.request-data' ), $container->get( 'button.request-data' ),
$container->get( 'woocommerce.logger.woocommerce' ) $container->get( 'woocommerce.logger.woocommerce' )
@ -176,7 +176,7 @@ return array(
* *
* @returns array<array{name: string, is_active: bool}> * @returns array<array{name: string, is_active: bool}>
*/ */
'axo.fastlane-incompatible-plugins' => static function () : array { 'axo.fastlane-incompatible-plugins' => static function () : array {
/** /**
* Filters the list of Fastlane incompatible plugins. * Filters the list of Fastlane incompatible plugins.
*/ */
@ -231,7 +231,7 @@ return array(
); );
}, },
'axo.fastlane-incompatible-plugin-names' => static function ( ContainerInterface $container ) : array { 'axo.fastlane-incompatible-plugin-names' => static function ( ContainerInterface $container ) : array {
$incompatible_plugins = $container->get( 'axo.fastlane-incompatible-plugins' ); $incompatible_plugins = $container->get( 'axo.fastlane-incompatible-plugins' );
$active_plugins_list = array_filter( $active_plugins_list = array_filter(
@ -253,10 +253,10 @@ return array(
); );
}, },
'axo.shipping-wc-enabled-locations' => static function ( ContainerInterface $container ): array { 'axo.shipping-wc-enabled-locations' => static function ( ContainerInterface $container ): array {
$default_zone = new \WC_Shipping_Zone( 0 ); $default_zone = new \WC_Shipping_Zone( 0 );
$is_method_enabled = fn( $method) => $method->enabled === 'yes'; $is_method_enabled = fn( \WC_Shipping_Method $method): bool => $method->enabled === 'yes';
$is_default_zone_enabled = ! empty( $is_default_zone_enabled = ! empty(
array_filter( array_filter(
@ -271,20 +271,21 @@ return array(
$shipping_zones = \WC_Shipping_Zones::get_zones(); $shipping_zones = \WC_Shipping_Zones::get_zones();
$get_zone_locations = fn($zone) => $get_zone_locations = fn( \WC_Shipping_Zone $zone): array =>
!empty(array_filter($zone->get_shipping_methods(), $is_method_enabled)) ! empty( array_filter( $zone->get_shipping_methods(), $is_method_enabled ) )
? array_map( ? array_map(
fn($location) => $location->code, fn( object $location): string => $location->code,
$zone->get_zone_locations() $zone->get_zone_locations()
) )
: []; : array();
$enabled_locations = array_unique( $enabled_locations = array_unique(
array_merge( array_merge(
...array_map( ...array_map(
$get_zone_locations, $get_zone_locations,
array_map( array_map(
fn( $zone) => $zone instanceof \WC_Shipping_Zone ? $zone : new \WC_Shipping_Zone( $zone['id'] ), fn( $zone): \WC_Shipping_Zone =>
$zone instanceof \WC_Shipping_Zone ? $zone : new \WC_Shipping_Zone( $zone['id'] ),
$shipping_zones $shipping_zones
) )
) )