Skip to content

Drop not injectable fields before single linkfield workaround#260

Merged
cedric-anne merged 2 commits intopluginsGLPI:developfrom
cconard96:fix/drop_notinjectable
Mar 17, 2022
Merged

Drop not injectable fields before single linkfield workaround#260
cedric-anne merged 2 commits intopluginsGLPI:developfrom
cconard96:fix/drop_notinjectable

Conversation

@cconard96
Copy link
Contributor

Since glpi-project/glpi#8816, there was a second search option with the same "users_id" link field. Then in the workaround for #121, the correct User option gets removed and the Appliance User option remains. This remaining option isn't injectable though and is dropped later anyways. It makes sense to drop all non-injectable options before the workaround.

fixes #259
fixes #243
fixes #233

@cedric-anne cedric-anne changed the base branch from master to develop March 1, 2022 13:21
@cconard96
Copy link
Contributor Author

Last commit addresses issue where the plugin tried searching for the entities_id search option which isn't injectable and is now not present in the results but is statically set internally.

@cedric-anne cedric-anne merged commit ae0907a into pluginsGLPI:develop Mar 17, 2022
@cedric-anne cedric-anne modified the milestones: 2.9.1, 2.10.0 Mar 24, 2022
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.

'user' field missing from 'computers' table Cannot map computers.user field User field unavailable after GLPI 9.5.5 update

2 participants