Skip to content

Feat split tree dropdown question#22

Open
Lainow wants to merge 9 commits intomainfrom
feat-split-tree-dropdown-question
Open

Feat split tree dropdown question#22
Lainow wants to merge 9 commits intomainfrom
feat-split-tree-dropdown-question

Conversation

@Lainow
Copy link

@Lainow Lainow commented Feb 12, 2026

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !41364
  • Here is a brief description of what this PR does
    Added a new question type called Tree Cascade Dropdown, allowing the names of ITIL Locations and Categories to be separated into several dropdowns. (As the splitcat plugin did).

Screenshots (if appropriate):

Before
image

After
image

@Lainow Lainow self-assigned this Feb 12, 2026
Copy link
Contributor

@ccailly ccailly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried it and it works pretty well.
However, I think you forgot to handle the advanced configuration of the extended question type.
If I choose an item other than the default root as the root, it seems to break.

Image Image Image

By the way, it would probably be more relevant to extend QuestionTypeItemDropdown rather than QuestionTypeItem. Certain specific behaviors are required for CommonTreeDropdown (this case, for example: glpi-project/glpi#19967).
The QuestionTypeItemDropdown class probably cannot be extended at present; a fix would need to be made to the core to remove the final attribute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants