[7418] Forcehttps control

[jvega190]

craftercms/craftercms#7418

Summary by CodeRabbit

• New Features
  • Enabled Force HTTPS control support with proper rendering and boolean value handling in forms.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7bafec1 and 4270219.

📒 Files selected for processing (3)

• ui/app/src/components/ContentTypeManagement/descriptors/controls/forceHttps.ts
• ui/app/src/components/FormsEngine/lib/controlMap.ts
• ui/app/src/components/FormsEngine/lib/valueRetrievers.ts

---

Walkthrough

Added an "id" field to the forceHttps descriptor, mapped the forcehttps control to a lazy-loaded Checkbox component in the control map, and configured boolean value parsing for the forcehttps field using booleanFieldExtractor in the value retriever lookup.

Changes

Cohort / File(s)	Summary
ForceHttps Control Configuration ui/app/src/components/ContentTypeManagement/descriptors/controls/forceHttps.ts, ui/app/src/components/FormsEngine/lib/controlMap.ts, ui/app/src/components/FormsEngine/lib/valueRetrievers.ts	Added id field definition to forceHttps descriptor, mapped forcehttps control to lazy-loaded Checkbox component, and updated value retriever to use booleanFieldExtractor for boolean field parsing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• [7418] Use XML values on fields types in control descriptors #4688: Modifies the forceHttps control descriptor field type literals (readonly from 'checkbox' to 'boolean').
• [7418] Type Builder #4451: Updates the forceHttps descriptor file with refactoring, typings, and internationalization changes.

Suggested reviewers

• rart
• sumerjabri

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description only contains a ticket reference URL without explaining the changes, objectives, or implementation details.	Add detailed explanation of changes made: field descriptor additions, control map updates, and value retriever mappings, along with the rationale for these changes.
Title check	❓ Inconclusive	The title references the ticket number but lacks specificity about what was implemented for the forceHttps control.	Clarify the title to describe the specific implementation, such as 'Add forceHttps control field descriptor and UI component integration'.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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.}