woocommerce/woocommerce-paypal-payments
2
0
Fork 0
mirror of https://github.com/woocommerce/woocommerce-paypal-payments.git synced 2025-09-04 08:47:23 +08:00
Code Issues Projects Packages Wiki Activity Actions 7

Refactor styles loading on GooglePay and ApplePay

Browse source 
Remove unnecessary console.log on GooglePay and ApplePay
This commit is contained in:
Pedro Silva 2023-10-19 18:44:44 +01:00
parent 567a7a2065
commit b11a539c24
No known key found for this signature in database
GPG key ID: E2EE20C0669D24B3
6 changed files with 56 additions and 21 deletions
Download patch file Download diff file Expand all files Collapse all files

1
modules/ppcp-applepay/resources/js/boot.js
View file

@ -20,7 +20,6 @@ import ApplepayManager from "./ApplepayManager";
(typeof (buttonConfig) === 'undefined') ||
(typeof (ppcpConfig) === 'undefined')
) {
console.error('PayPal button could not be configured.');
return;
}
const isMiniCart = ppcpConfig.mini_cart_buttons_enabled;

12
modules/ppcp-applepay/src/ApplepayModule.php
View file

@ -136,10 +136,18 @@ class ApplepayModule implements ModuleInterface {
assert( $button instanceof ApplePayButton );
$smart_button = $c->get( 'button.smart-button' );
assert( $smart_button instanceof SmartButtonInterface );
$page_has_block = has_block( 'woocommerce/checkout' ) || has_block( 'woocommerce/cart' );
if ( $smart_button->should_load_ppcp_script() || $page_has_block ) {
if ( $smart_button->should_load_ppcp_script() ) {
$button->enqueue();
}
if ( has_block( 'woocommerce/checkout' ) || has_block( 'woocommerce/cart' ) ) {
/**
* Should add this to the ButtonInterface.
*
* @psalm-suppress UndefinedInterfaceMethod
*/
$button->enqueue_styles();
}
}
);
add_action(

25
modules/ppcp-applepay/src/Assets/ApplePayButton.php
View file

@ -1017,13 +1017,7 @@ class ApplePayButton implements ButtonInterface {
);
wp_enqueue_script( 'wc-ppcp-applepay' );
wp_register_style(
'wc-ppcp-applepay',
untrailingslashit( $this->module_url ) . '/assets/css/styles.css',
array(),
$this->version
);
wp_enqueue_style( 'wc-ppcp-applepay' );
$this->enqueue_styles();
wp_localize_script(
'wc-ppcp-applepay',
@ -1038,6 +1032,23 @@ class ApplePayButton implements ButtonInterface {
);
}
/**
* Enqueues styles.
*/
public function enqueue_styles(): void {
if ( ! $this->is_enabled() ) {
return;
}
wp_register_style(
'wc-ppcp-applepay',
untrailingslashit( $this->module_url ) . '/assets/css/styles.css',
array(),
$this->version
);
wp_enqueue_style( 'wc-ppcp-applepay' );
}
/**
* Returns the script data.
*

1
modules/ppcp-googlepay/resources/js/GooglepayButton.js
View file

@ -189,7 +189,6 @@ class GooglepayButton {
callback(el);
} else if (timeElapsed > timeout) {
clearInterval(interval);
console.error('Waiting for wrapper timed out.', selector);
}
}, delay);
}

23
modules/ppcp-googlepay/src/Assets/Button.php
View file

@ -363,6 +363,23 @@ class Button implements ButtonInterface {
);
wp_enqueue_script( 'wc-ppcp-googlepay' );
$this->enqueue_styles();
wp_localize_script(
'wc-ppcp-googlepay',
'wc_ppcp_googlepay',
$this->script_data()
);
}
/**
* Enqueues styles.
*/
public function enqueue_styles(): void {
if ( ! $this->is_enabled() ) {
return;
}
wp_register_style(
'wc-ppcp-googlepay',
untrailingslashit( $this->module_url ) . '/assets/css/styles.css',
@ -370,12 +387,6 @@ class Button implements ButtonInterface {
$this->version
);
wp_enqueue_style( 'wc-ppcp-googlepay' );
wp_localize_script(
'wc-ppcp-googlepay',
'wc_ppcp_googlepay',
$this->script_data()
);
}
/**

15
modules/ppcp-googlepay/src/GooglepayModule.php
View file

@ -89,11 +89,18 @@ class GooglepayModule implements ModuleInterface {
static function () use ( $c, $button ) {
$smart_button = $c->get( 'button.smart-button' );
assert( $smart_button instanceof SmartButtonInterface );
$page_has_block = has_block( 'woocommerce/checkout' ) || has_block( 'woocommerce/cart' );
if ( ! $smart_button->should_load_ppcp_script() && ! $page_has_block ) {
return;
if ( $smart_button->should_load_ppcp_script() ) {
$button->enqueue();
}
if ( has_block( 'woocommerce/checkout' ) || has_block( 'woocommerce/cart' ) ) {
/**
* Should add this to the ButtonInterface.
*
* @psalm-suppress UndefinedInterfaceMethod
*/
$button->enqueue_styles();
}
$button->enqueue();
}
);