Skip to content

Fix group injection fro printers - #672

Open
Lainow wants to merge 2 commits into
mainfrom
fix-groupinjection-for-printer
Open

Lainow wants to merge 2 commits into
mainfrom
fix-groupinjection-for-printer

Conversation

@Lainow

@Lainow Lainow commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

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 !46346
  • Here is a brief description of what this PR does

The “Group” field did not appear in the printer mapping because its search option did not have a defined linkfield, which made it non-injectable. Added $tab[71][‘linkfield’] = ‘groups_id_normal’;, as had already been done for computers.

Screenshots (if appropriate):

@Lainow Lainow self-assigned this Sep 16, 2026
@Lainow
Lainow requested review from MyvTsv and RomainLvr September 16, 2026 12:16
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.

1 participant