Fix. Settings. ct_prev_referer cookie breaks page caching.

[svfcode]

task https://app.doboard.com/1/task/54234

[Copilot]

Pull request overview

This PR adjusts how CleanTalk’s “previous referrer” and cookie-test probe cookies are set/read to avoid PHP-driven Set-Cookie updates that can disrupt page caching, and bumps the extension version to 5.8.1.

Changes:

• Set ct_prev_referer from JS (document.referrer) instead of PHP, and read it from the unprefixed cookie name in spam checks.
• Change ct_cookies_test payload and set an explicit lifetime (7 days).
• Bump extension version from 5.8.0 to 5.8.1 in composer metadata and ACP module info.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
cleantalk/antispam/styles/all/template/cleantalk.js	Adds JS-managed ct_prev_referer cookie on pageview.
cleantalk/antispam/model/main_model.php	Reads ct_prev_referer without phpBB prefix; revises cookie-test probe cookie behavior/lifetime.
cleantalk/antispam/composer.json	Version bump to 5.8.1.
cleantalk/antispam/acp/main_info.php	ACP module version bump to 5.8.1.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Suppressed comments (1)

cleantalk/antispam/model/main_model.php:308

• Early-return validation for an existing ct_cookies_test cookie only checks check_value. If a client sends a malformed cookie that has the correct check_value but is missing cookies_names (or it’s not an array), set_cookie() will skip rewriting it, and test_cookie() will later do foreach ($cookie_test['cookies_names'] ...), triggering warnings / incorrect behavior. Validate the cookie structure (including cookies_names being an array) before returning early.

        if ( $this->request->is_set($cookie_key, \phpbb\request\request_interface::COOKIE) ) {
            $existing = json_decode(htmlspecialchars_decode($this->request->variable($cookie_key, '', false, \phpbb\request\request_interface::COOKIE)), true);
            if ( is_array($existing) && isset($existing['check_value']) && $existing['check_value'] === $expected ) {
                return;
            }