Skip to content

fix(SO): search options not working with default values in multi-select#980

Merged
Rom1-B merged 2 commits into
pluginsGLPI:mainfrom
MyvTsv:ticket38487
Aug 12, 2025
Merged

fix(SO): search options not working with default values in multi-select#980
Rom1-B merged 2 commits into
pluginsGLPI:mainfrom
MyvTsv:ticket38487

Conversation

@MyvTsv
Copy link
Copy Markdown
Contributor

@MyvTsv MyvTsv commented Jul 17, 2025

  • 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 !38487

When a multiple drop-down list field, defined via the Fields plugin, has a default value, this is displayed in the interface when a new item is created (itemtype), but is not saved in the database.

This leads to inconsistent behavior: when GLPI filters search for this value, items supposed to have this default value are not found, as it is not actually saved.

Screenshots (if appropriate):

image image

@MyvTsv MyvTsv self-assigned this Jul 17, 2025
@MyvTsv MyvTsv requested review from Rom1-B and stonebuzz and removed request for stonebuzz July 17, 2025 09:40
@MyvTsv MyvTsv marked this pull request as ready for review July 17, 2025 09:40
@stonebuzz
Copy link
Copy Markdown
Contributor

@MyvTsv can you valide with customer?

Copy link
Copy Markdown
Contributor

@Rom1-B Rom1-B left a comment

Choose a reason for hiding this comment

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

Seems OK
Waiting customer validation.

@MyvTsv
Copy link
Copy Markdown
Contributor Author

MyvTsv commented Jul 30, 2025

It's good for the customer

@MyvTsv MyvTsv requested a review from Rom1-B July 30, 2025 13:08
Copy link
Copy Markdown
Contributor

@Rom1-B Rom1-B left a comment

Choose a reason for hiding this comment

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

Please rebase to solve conflicts

@Rom1-B Rom1-B merged commit 25b63bd into pluginsGLPI:main Aug 12, 2025
3 checks passed
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.

3 participants