Skip to content

Fix/46642/groups values in pdf - #95

Open
jdurand-teclib wants to merge 4 commits into
11.0/bugfixesfrom
fix/46642/groups-values-in-pdf
Open

jdurand-teclib wants to merge 4 commits into
11.0/bugfixesfrom
fix/46642/groups-values-in-pdf

Conversation

@jdurand-teclib

Copy link
Copy Markdown
  • 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 !46642
  • Here is a brief description of what this PR does
    • Fix groups values display in PDF (groups_id & groups_id_tech) for every concerned assets/items. groups values were treated as single id where it is actually a list of ids.
    • Applied striptag to groups names
    • Added unit tests

Screenshots

Testing asset:
image

Before:
image

After:
image

@jdurand-teclib
jdurand-teclib force-pushed the fix/46642/groups-values-in-pdf branch from 465949d to 23773b8 Compare October 1, 2026 13:19
@jdurand-teclib jdurand-teclib self-assigned this Oct 5, 2026

@Mary-Clb Mary-Clb left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I looks like the same problem as #63 (group fields loaded as arrays of IDs for assignable assets). Yours is more complete (more asset classes covered), so I'll close #63 in favor of this one.

@jdurand-teclib
jdurand-teclib changed the base branch from main to 11.0/bugfixes October 7, 2026 13:15
@jdurand-teclib

Copy link
Copy Markdown
Author

I gave the patch to the customers, I'm waiting for their feedback before merging

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