Ensured action is string to prevent fatal

[girishpanchal30]

All Submissions:

• Have you followed the Optimole Contributing guideline?
• Does your code follow the WordPress' coding standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

Improves AJAX request detection to avoid potential errors when the action parameter isn't a string. It also adds comprehensive unit tests to ensure correct behavior for various action values, including arrays and missing keys.

Closes #1147

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you successfully ran tests with your changes locally?

[pirate-bot]

Plugin build for 34bc4ec is ready 🛎️!

• Download build
• Download dev

[Copilot]

🟢 Approval recommended

The targeted guard resolves the reported TypeError while preserving existing behavior with comprehensive regression coverage.

Pull request overview

Prevents fatal errors when malformed AJAX action parameters are arrays.

Changes:

• Guards strpos() with an is_string() check.
• Adds coverage for array, nested-array, missing, excluded, and normal actions.

File summaries

File	Description
inc/manager.php	Safely validates AJAX action type.
tests/test-ajax-request-detection.php	Tests malformed and valid action scenarios.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Balanced (auto)

Note

Copilot is running an experiment and ran this review at Balanced.

---

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

[pirate-bot]

Approved: The guard prevents malformed AJAX actions from crashing while preserving existing string behavior.

PHP 8.3 execution verified every asserted action shape and the security review found no changed trust-boundary risk.

Validation details

• PHP 8.3 executed the extracted method. Arrays returned true, wpmdb returned false, and other supported inputs returned true.
• The base method threw the reported strpos() TypeError for an array.
• Source inspection found no authorization, capability, nonce, database, file, or outbound-access effect.
• PHP 8.3 syntax checks passed for both changed files.

Untested areas

• The WordPress PHPUnit class was not run because its runtime and Composer dependencies were absent.

🤖 Automated review · run code-review-agent_6aa7f3b9992878.18996665.

---

🤖 Review agent — review posted ✅

• Review: APPROVED · 0 findings
• Head: 34bc4ec
• Duration: 4 min 07 s

Run code-review-agent_6aa7f3b9992878.18996665 · trail