Skip to content

feat(Parameters): Add boolean element type#63

Open
Hipska wants to merge 1 commit intoCombodo:masterfrom
Super-Visions:feat/parameters/boolean
Open

feat(Parameters): Add boolean element type#63
Hipska wants to merge 1 commit intoCombodo:masterfrom
Super-Visions:feat/parameters/boolean

Conversation

@Hipska
Copy link
Copy Markdown
Collaborator

@Hipska Hipska commented Dec 8, 2025

Base information

Question Answer
Related to a SourceForge thread / Another PR / Combodo ticket? N/A
Type of change? Enhancement

Objective

Add enhanced support for boolean parameters which can now contain 1, 0, true, false, on, off, yes, no and return the boolean representation to the collector.

Proposed solution

Use integrated FILTER_VALIDATE_BOOL.

No Unit test added, since there's no existing test for the modified method.

Checklist before requesting a review

  • I have performed a self-review of my code, and that it's compliant with Combodo's guidelines
  • I have tested all changes I made on an iTop instance
  • I have added a unit test, otherwise I have explained why I couldn't
  • I have made sure the PR is clear and detailled enough so anyone can understand the real purpose without digging in the code

@github-project-automation github-project-automation Bot moved this to First review needed in Combodo PRs dashboard Dec 8, 2025
@jf-cbd jf-cbd moved this from First review needed to Pending review in Combodo PRs dashboard Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending review

Development

Successfully merging this pull request may close these issues.

3 participants