Skip to content

Conversation

ccailly
Copy link

@ccailly ccailly commented Oct 6, 2025

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

Implement Tag question type

Screenshots (if appropriate):

Form editor

image

Form rendering

image

@ccailly ccailly force-pushed the feature/implement-tag-question-type branch 3 times, most recently from 59a5b9a to 1494937 Compare October 6, 2025 09:50
@ccailly ccailly force-pushed the feature/implement-tag-question-type branch from 1494937 to 6a7f41b Compare October 6, 2025 12:31
@ccailly ccailly force-pushed the feature/implement-tag-question-type branch from 6a7f41b to 18e58e4 Compare October 6, 2025 12:32
use Glpi\Form\QuestionType\AbstractQuestionType;
use Glpi\Form\QuestionType\QuestionTypeCategoryInterface;

class PluginTagQuestionType extends AbstractQuestionType implements FormQuestionDataConverterInterface
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
class PluginTagQuestionType extends AbstractQuestionType implements FormQuestionDataConverterInterface
final class PluginTagQuestionType extends AbstractQuestionType implements FormQuestionDataConverterInterface


if (isset($_POST['add'])) {
$item = new PluginTagTagItem();
class PluginTagQuestionTypeCategory implements QuestionTypeCategoryInterface
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
class PluginTagQuestionTypeCategory implements QuestionTypeCategoryInterface
final class PluginTagQuestionTypeCategory implements QuestionTypeCategoryInterface

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.

4 participants