Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix(Overrides): load overrides from related item if container is 'tab' type #743

Merged
merged 3 commits into from
Feb 21, 2024

Conversation

stonebuzz
Copy link
Contributor

@stonebuzz stonebuzz commented Dec 15, 2023

Container with 'tab' type can load override

it does not contain the fields of the parent object in $data

This PR fix this by loading overrides with related item fields instead of $data

Fix : #739

Add explanatory text

image

Fix : #738

@stonebuzz stonebuzz added the bug label Dec 15, 2023
@stonebuzz stonebuzz self-assigned this Dec 15, 2023
inc/container.class.php Outdated Show resolved Hide resolved
@cedric-anne cedric-anne added this to the 1.21.7 milestone Dec 21, 2023
@cedric-anne cedric-anne dismissed their stale review February 21, 2024 08:09

Changes were made.

@cedric-anne cedric-anne requested a review from Rom1-B February 21, 2024 08:09
@stonebuzz stonebuzz merged commit 3453f4f into pluginsGLPI:main Feb 21, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mandtory field - Override by status Add an explanatory text in the header tab
4 participants