Skip to content

fix(MassiveAction): dropdown not displaying when dropdown is shared across multiple fields#981

Merged
Rom1-B merged 2 commits into
pluginsGLPI:mainfrom
MyvTsv:ticket38333
Aug 13, 2025
Merged

fix(MassiveAction): dropdown not displaying when dropdown is shared across multiple fields#981
Rom1-B merged 2 commits into
pluginsGLPI:mainfrom
MyvTsv:ticket38333

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 !38333

Problem:
When a Fields plugin dropdown is shared between several fields and a user performs a bulk action on one of these fields, the dropdown for selecting the new value is not displayed.

Impact:
Users cannot perform mass actions on fields using a shared Fields plugin dropdown, rendering this functionality unusable.

Solution:
This PR corrects the problem of dropdown display during mass actions when the dropdown is shared between several Fields plugin fields.

Screenshots (if appropriate):

image

@MyvTsv MyvTsv self-assigned this Jul 17, 2025
@MyvTsv MyvTsv requested review from Rom1-B and stonebuzz July 17, 2025 12:50
@MyvTsv MyvTsv marked this pull request as ready for review July 17, 2025 12:50
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.

Can you get the client to validate it?

@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:07
@Rom1-B
Copy link
Copy Markdown
Contributor

Rom1-B commented Jul 30, 2025

Please rebase to solve conflicts

@Rom1-B Rom1-B merged commit 5832807 into pluginsGLPI:main Aug 13, 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.

2 participants