Skip to content

fix(migration): ensure GenericObject type dropdowns are migrated#1199

Open
MyvTsv wants to merge 6 commits into
pluginsGLPI:mainfrom
MyvTsv:ticket42658
Open

fix(migration): ensure GenericObject type dropdowns are migrated#1199
MyvTsv wants to merge 6 commits into
pluginsGLPI:mainfrom
MyvTsv:ticket42658

Conversation

@MyvTsv

@MyvTsv MyvTsv commented May 20, 2026

Copy link
Copy Markdown
Contributor
  • 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 #43878
  • Here is a brief description of what this PR does
    The dropdown menus from an old GenericObject asset were not migrated correctly and caused a Twig error when the Fields block was displayed
image

Screenshots (if appropriate):

Old GenericObject type dropdown fields:
image

image

@MyvTsv MyvTsv self-assigned this May 20, 2026
@MyvTsv MyvTsv requested review from Rom1-B and stonebuzz May 20, 2026 15:27

@Rom1-B Rom1-B left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please fix CI

@MyvTsv MyvTsv requested a review from Rom1-B May 21, 2026 09:18
@stonebuzz

Copy link
Copy Markdown
Contributor

Could you please have the cutsomer validatation ?

Comment thread inc/field.class.php Outdated
Comment thread inc/field.class.php Outdated
Comment thread inc/migration.class.php Outdated
MyvTsv and others added 3 commits May 22, 2026 12:28
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
Co-authored-by: Romain B. <8530352+Rom1-B@users.noreply.github.com>
@MyvTsv MyvTsv requested a review from Rom1-B May 22, 2026 14:46
@Herafia Herafia requested a review from Rom1-B June 10, 2026 09:20
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