Enforce phpcs useless variable Slevomat rule and php cs fixer return_assignment rule

[Crabcyborg]

No description provided.

[coderabbitai]

Walkthrough

This PR removes intermediate local variables across multiple files, replacing them with direct return statements. The changes affect controller, helper, and model classes, including FrmFormsController, FrmEntriesListHelper, FrmFormsListHelper, FrmStylesHelper, and FrmAntiSpam, maintaining identical functionality throughout.

Changes

Cohort / File(s)	Summary
Variable elimination and direct returns classes/controllers/FrmFormsController.php, classes/helpers/FrmEntriesListHelper.php, classes/helpers/FrmFormsListHelper.php, classes/helpers/FrmStylesHelper.php, classes/models/FrmAntiSpam.php	Simplifies functions by removing intermediate variables ($message, $hex, $form_token_string) and returning computed values directly via return statements. No functional changes; purely refactoring for code clarity.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Fix some phpcs warnings #2676: Both PRs edit FrmFormsListHelper::single_row with similar local-variable and direct-return refactoring patterns.
• New rector rule for useless variables #2665: Applies the same code-level refactoring pattern (removing intermediate variables and switching to direct returns) across the same set of files and functions.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No description was provided by the author, making it impossible to evaluate whether the description relates to the changeset.	Add a description explaining the purpose of enforcing the PHPCS useless variable rule and its benefits.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Title check	✅ Passed	The title accurately describes the main objective of the pull request: enforcing phpcs useless variable and php cs fixer return_assignment rules across multiple files.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}