Skip to content

New. Integration. Integration with Pagelayer - #896

Open
AntonV1211 wants to merge 5 commits into
fixfrom
fix_rmv_srv_fields_av
Open

AntonV1211 wants to merge 5 commits into
fixfrom
fix_rmv_srv_fields_av

Conversation

@AntonV1211

Copy link
Copy Markdown
Contributor

@codecov

codecov Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.95238% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.98%. Comparing base (5ad7fb0) to head (e00ae42).

Files with missing lines Patch % Lines
lib/Cleantalk/Antispam/Integrations/Pagelayer.php 82.35% 3 Missing ⚠️
inc/cleantalk-ajax.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                fix     #896      +/-   ##
============================================
+ Coverage     30.59%   30.98%   +0.38%     
- Complexity     6639     6644       +5     
============================================
  Files           292      293       +1     
  Lines         26493    26516      +23     
============================================
+ Hits           8105     8215     +110     
+ Misses        18388    18301      -87     

☔ 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.

Comment thread inc/cleantalk-ajax.php Outdated
@AntonV1211 AntonV1211 changed the title Fix. Code. Deleting service fields after checking for spam New. Integration. Integration with Pagelayer Sep 25, 2026

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.

Copilot review overview

🟡 Changes recommended

Unresolved request-guard, production bundle, and Pagelayer service-field cleanup issues remain.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity · 1 Medium severity

Open (2)
What changed in this PR

Adds Pagelayer contact-form spam protection, synchronous submission handling, request detection, and service-field cleanup.

Changes:

  • Adds Pagelayer integration and hook registration.
  • Updates source and generated JavaScript bundles.
  • Adds Pagelayer request filtering and AJAX cleanup.
File Summary
lib/​Cleantalk/​Antispam/​Integrations/​Pagelayer.php Pagelayer checking and blocking logic
js/​src/​public-1-main.js Synchronous Pagelayer submit handling
js/​prebuild/​apbct-public-bundle.js Updated default prebuild bundle
js/​prebuild/​apbct-public-bundle_int-protection.js Updated internal-protection bundle
js/​prebuild/​apbct-public-bundle_int-protection_gathering.js Updated internal gathering bundle
js/​prebuild/​apbct-public-bundle_gathering.js Updated gathering bundle
js/​prebuild/​apbct-public-bundle_full-protection.js Updated full-protection bundle
js/​prebuild/​apbct-public-bundle_full-protection_gathering.js Updated full gathering bundle
js/​prebuild/​apbct-public-bundle_ext-protection.js Updated external-protection bundle
js/​prebuild/​apbct-public-bundle_ext-protection_gathering.js Updated external gathering bundle
js/​apbct-public-bundle.min.js Updated default minified bundle
inc/​cleantalk-pluggable.php Pagelayer request detection
inc/​cleantalk-integrations-by-hook.php Pagelayer hook registration
inc/​cleantalk-ajax.php Service-field cleanup after checks

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

Comment thread inc/cleantalk-pluggable.php
Comment thread lib/Cleantalk/Antispam/Integrations/Pagelayer.php
@svedge
svedge requested review from svedge and removed request for svfcode September 28, 2026 03:37

This branch has not been deployed

No deployments
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.

4 participants