Skip to content

Fix massive action add multiple dropdown#1004

Merged
Rom1-B merged 3 commits into
pluginsGLPI:mainfrom
Lainow:fix-ma-add-multiplke-dropdown
Sep 15, 2025
Merged

Fix massive action add multiple dropdown#1004
Rom1-B merged 3 commits into
pluginsGLPI:mainfrom
Lainow:fix-ma-add-multiplke-dropdown

Conversation

@Lainow
Copy link
Copy Markdown
Contributor

@Lainow Lainow commented Sep 1, 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

  • It fixes !38850
  • Here is a brief description of what this PR does
    Correction of the bulk action to add value to multiple drop-down fields, specifically the ‘Add’ option, which now compares the values in the table rather than the keys in order to prevent duplicates.

Screenshots (if appropriate):

@Lainow Lainow requested a review from stonebuzz September 1, 2025 09:23
@Lainow Lainow self-assigned this Sep 1, 2025
@Lainow Lainow changed the title Fix ma add multiplke dropdown Fix massive action add multiplke dropdown Sep 1, 2025
@Lainow Lainow changed the title Fix massive action add multiplke dropdown Fix massive action add multiple dropdown Sep 1, 2025
@stonebuzz
Copy link
Copy Markdown
Contributor

Waiting validation with customers

@stonebuzz
Copy link
Copy Markdown
Contributor

@Lainow

can you rebase, it is ok from customer ?

@stonebuzz stonebuzz requested a review from Rom1-B September 15, 2025 07:23
@Rom1-B Rom1-B merged commit af827b8 into pluginsGLPI:main Sep 15, 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