unistra / python-glpi-api

Python module for interacting with GLPI using the API.
GNU General Public License v3.0
18 stars 10 forks source link

adding add_sub_items and update_sub_items #21

Closed thel1988 closed 7 months ago

thel1988 commented 10 months ago

Hi there

As someone really happy about this package, i would like to hear if it would be possible to implement the following functions: add_sub_items and update_sub_items

I had the need to have both of these functions in a project where we add entries to the glpi installation from another system.

I changed in my local installation, but i'm not sure if you want to do any of the following:

I basically copied the get_sub_item function and changed the names: eg: def add_sub_items(self, itemtype, item_id, sub_itemtype, *items):

And changed the method to either post or put, depending on if it is a update request or create.

I know it is not official on the api documentation from glpi, but it seems to work.

Either way let me know what you think.

fmenabe commented 10 months ago

Hi,

For what I understand, theses API calls work even though they are not documented. As long as it just that, it is ok to add them to the library.

I'll prefer a pull request, less work for me and you get the credit ;-). Just know I'm not working at the time for Unistra and I may need to find someone to get me the permissions or merge the PR.

And for coherence, del_sub_items should also be implemented.

thel1988 commented 7 months ago

Hi any news on the pull request? Is it a permission issue? :) Pull issue have been made in oct: https://github.com/unistra/python-glpi-api/pull/22

fmenabe commented 7 months ago

Hi @thel1988, sorry but it took me some time to ask for the permissions and it take a few more time to get them. I only get back access a few days ago but didn't have much time to look back at the remaining issues on this repo.

I'll merge the PR and release a new version on Pypi.