All user check#70
Open
alexander-b-clean wants to merge 2 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new extension setting to moderate all registered users’ posts/topics (not only “newly registered” users), wiring it through migration, ACP UI, language packs, and the runtime moderation decision.
Changes:
- Introduces
cleantalk_antispam_allusersconfig via a newrelease_5_7_8migration (default off). - Exposes the new toggle in the ACP settings page and persists it in the ACP module.
- Updates the main event listener to enable moderation for any registered user when the toggle is enabled, plus adds translations for all shipped locales.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| cleantalk/antispam/migrations/release_5_7_8.php | Adds migration to create the new cleantalk_antispam_allusers config key (default 0). |
| cleantalk/antispam/event/main_listener.php | Applies the new config option in the moderation decision for registered users. |
| cleantalk/antispam/acp/main_module.php | Saves the new ACP setting and assigns it to the template. |
| cleantalk/antispam/adm/style/settings_body.html | Adds ACP UI radios for enabling/disabling “all users” moderation. |
| cleantalk/antispam/language/en/common.php | Adds English label/description for the new setting. |
| cleantalk/antispam/language/ru/common.php | Adds Russian label/description for the new setting. |
| cleantalk/antispam/language/it/common.php | Adds Italian label/description for the new setting. |
| cleantalk/antispam/language/fr/common.php | Adds French label/description for the new setting. |
| cleantalk/antispam/language/es/common.php | Adds Spanish label/description for the new setting. |
| cleantalk/antispam/language/de/common.php | Adds German label/description for the new setting. |
| cleantalk/antispam/language/ar/common.php | Adds Arabic label/description for the new setting. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
AntonV1211
approved these changes
Jul 23, 2026
Member
|
Rename pull request according to standards. |
alexandergull
approved these changes
Jul 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://app.doboard.com/1/task/50628