Skip to content

Udp.Code.WooCommerce UI - #867

Open
alexander-b-clean wants to merge 12 commits into
devfrom
woo-commerce-update.ab-2
Open

Udp.Code.WooCommerce UI#867
alexander-b-clean wants to merge 12 commits into
devfrom
woo-commerce-update.ab-2

Conversation

@alexander-b-clean

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the WooCommerce spam-orders experience by integrating the plugin’s “blocked orders” table into the native WooCommerce Orders admin screen (HPOS), aligning columns/styles with WooCommerce UI, and adjusting checkout handling to support storing/redirecting blocked checkouts.

Changes:

  • Reworks the spam orders list table UI (new columns, totals, status views integration, pagination/sorting tweaks).
  • Hooks into the WooCommerce HPOS orders screen to add “Spam”/“On hold” status links and to replace the renderer when viewing Spam.
  • Adds a helper for counting stored spam orders and updates admin/settings links to point at the WooCommerce orders screen spam view.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
lib/Cleantalk/ApbctWP/WcSpamOrdersListTable.php Refactors table UI/columns, adds views/status/total calculation, updates delete URL building, and introduces page notices rendering.
lib/Cleantalk/ApbctWP/WcSpamOrdersFunctions.php Adds a DB-count helper for stored blocked orders.
lib/Cleantalk/ApbctWP/State.php Changes the default for storing blocked orders.
lib/Cleantalk/Antispam/IntegrationsByClass/Woocommerce.php Integrates spam orders into HPOS orders screen; adjusts blocked-checkout behavior and adds view/link logic.
inc/cleantalk-settings.php Updates settings-page action link target to the WooCommerce orders spam status view.
inc/cleantalk-admin.php Updates admin bar link target to the WooCommerce orders spam status view.
css/src/cleantalk-admin.css Adds styling to make the spam orders table match WooCommerce orders list UI.
css/cleantalk-admin.min.css Minified CSS update reflecting the new admin styling.
Files not reviewed (1)
  • css/cleantalk-admin.min.css: Generated file
Suppressed comments (1)

lib/Cleantalk/Antispam/IntegrationsByClass/Woocommerce.php:239

  • Same issue as the AJAX checkout path: returning early here skips the spam check for Store API checkouts when data__wc_store_blocked_orders is off, even though the setting is documented as “store blocked orders” (not “disable protection”).
        if ( ! $apbct->settings['data__wc_store_blocked_orders'] ) {
            // The checkout is left to WooCommerce as is: no check, no blocked order to store.
            return;
        }

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread lib/Cleantalk/ApbctWP/WcSpamOrdersListTable.php Outdated
Comment thread lib/Cleantalk/Antispam/IntegrationsByClass/Woocommerce.php
Comment thread lib/Cleantalk/Antispam/IntegrationsByClass/Woocommerce.php
Comment thread lib/Cleantalk/ApbctWP/WcSpamOrdersListTable.php
Comment thread lib/Cleantalk/ApbctWP/State.php
datorik and others added 5 commits August 24, 2026 13:46
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 263 lines in your changes missing coverage. Please review.
✅ Project coverage is 28.98%. Comparing base (59767a2) to head (9601aa2).
⚠️ Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
lib/Cleantalk/ApbctWP/WcSpamOrdersListTable.php 0.00% 150 Missing ⚠️
...ntalk/Antispam/IntegrationsByClass/Woocommerce.php 0.00% 108 Missing ⚠️
lib/Cleantalk/ApbctWP/WcSpamOrdersFunctions.php 0.00% 5 Missing ⚠️

❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (70.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev     #867      +/-   ##
============================================
- Coverage     29.20%   28.98%   -0.22%     
- Complexity     6443     6531      +88     
============================================
  Files           287      287              
  Lines         26064    26249     +185     
============================================
- Hits           7611     7608       -3     
- Misses        18453    18641     +188     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AntonV1211
AntonV1211 requested review from alexandergull and removed request for AntonV1211 August 24, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants