Merge pull request #283 from woocommerce/add/plugin-check

Add the WordPress Plugin Check Action
woocommerce · Feb 7, 2025 · e285bc4 · e285bc4
2 parents 59a8f45 + 686045a
commit e285bc4
Show file tree

Hide file tree

Showing 3 changed files with 46 additions and 3 deletions.
diff --git a/.github/workflows/plugin-check.yml b/.github/workflows/plugin-check.yml
@@ -0,0 +1,43 @@
+name: 'WordPress Plugin Checks'
+on:
+  push:
+    branches:
+    - trunk
+    - develop
+
+  pull_request:
+    branches:
+      - trunk
+      - develop
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v4
+
+    - name: Run plugin check
+      uses: wordpress/plugin-check-action@v1
+      with:
+        exclude-checks: |
+          plugin_readme
+          plugin_updater
+          file_type
+          trademarks
+          direct_db_queries
+          performant_wp_query_params
+          enqueued_scripts_in_footer
+          plugin_header_text_domain
+        
+        exclude-directories: |
+          .github
+          node_modules
+          vendor
+          tests
+          bin
+          build
+          assets
+
+        ignore-codes: |
+          plugin_header_no_license
diff --git a/includes/class-wc-gateway-payfast-privacy.php b/includes/class-wc-gateway-payfast-privacy.php
@@ -250,7 +250,7 @@ protected function maybe_handle_subscription( $order ) {
 				array(
 					sprintf(
 						/* translators: %d: Order ID */
-						esc_html__( 'Order ID %d contains an active Subscription' ),
+						esc_html__( 'Order ID %d contains an active Subscription', 'woocommerce-gateway-payfast' ),
 						$order->get_id()
 					),
 				),

diff --git a/includes/class-wc-gateway-payfast.php b/includes/class-wc-gateway-payfast.php
@@ -1752,7 +1752,7 @@ public function display_order_fee( $order_id ) {
 
 		<tr>
 			<td class="label payfast-fee">
-				<?php echo wc_help_tip( __( 'This represents the fee Payfast collects for the transaction.', 'woocommerce-gateway-payfast' ) ); ?>
+				<?php echo wc_help_tip( esc_html__( 'This represents the fee Payfast collects for the transaction.', 'woocommerce-gateway-payfast' ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>
 				<?php esc_html_e( 'Payfast Fee:', 'woocommerce-gateway-payfast' ); ?>
 			</td>
 			<td width="1%"></td>
@@ -1782,7 +1782,7 @@ public function display_order_net( $order_id ) {
 
 		<tr>
 			<td class="label payfast-net">
-				<?php echo wc_help_tip( __( 'This represents the net total that was credited to your Payfast account.', 'woocommerce-gateway-payfast' ) ); ?>
+				<?php echo wc_help_tip( esc_html__( 'This represents the net total that was credited to your Payfast account.', 'woocommerce-gateway-payfast' ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped ?>
 				<?php esc_html_e( 'Amount Net:', 'woocommerce-gateway-payfast' ); ?>
 			</td>
 			<td width="1%"></td>