New. Integration. Integration with Email Subscribers#809
Conversation
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (64.28%) 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 #809 +/- ##
============================================
+ Coverage 26.25% 26.35% +0.09%
- Complexity 5632 5647 +15
============================================
Files 265 266 +1
Lines 24160 24184 +24
============================================
+ Hits 6344 6373 +29
+ Misses 17816 17811 -5 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds a dedicated CleanTalk integration for the Email Subscribers by Icegram Express plugin, moving handling from the generic AJAX hook into the integrations-by-hook framework.
Changes:
- Introduces
Cleantalk\Antispam\Integrations\EmailSubscribersto collect subscription POST data and return a standardized block JSON payload. - Registers the new integration in
inc/cleantalk-integrations-by-hook.phpfor thees_add_subscriberAJAX action. - Removes the legacy
es_add_subscriberspecial-case fromct_ajax_hook().
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
lib/Cleantalk/Antispam/Integrations/EmailSubscribers.php |
New integration implementation: data extraction + AJAX block response. |
inc/cleantalk-pluggable.php |
Adds a skip rule for es_add_subscriber when Email Subscribers is active. |
inc/cleantalk-integrations-by-hook.php |
Registers EmailSubscribers integration on es_add_subscriber (AJAX). |
inc/cleantalk-ajax.php |
Removes legacy hardcoded handling for es_add_subscriber. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Email Subscribers by Icegram Express - skip subscribe action | ||
| if ( | ||
| apbct_is_plugin_active('email-subscribers/email-subscribers.php') && | ||
| Post::equal('action', 'es_add_subscriber') | ||
| ) { | ||
| return 'Email Subscribers by Icegram Express - skip subscribe action'; | ||
| } |
https://app.doboard.com/1/task/49806