implementing the API request to delete tags

[urschrei]

implementing the API request to delete tags

[urschrei]

Released to PyPI as v1.1.16

The signature is the same as add_tags(); you can pass an arbitrary (up to 50) number of tags, or use a list with the splat operator:
delete_tags('tag_a', 'tag_b', 'tag_c')
delete_tags(*[tag_list])

[urschrei]

Could you test this by uninstalling pyzotero, then doing pip install https://github.com/urschrei/pyzotero/archive/dev.zip?
That should give you a delete_tags(['tag a', 'tag b', 'tag c'])(up to 50 tags) method. I'll merge it into master and cut a release if it's working correctly.

[Cutuchiqueno]

Hi,

thanks for this quick implementation! I tested it and it works perfect. Tags are completely removed from the tag library and not only from items so that versions of the same tag with different case can be added later on.

Just for curiosity and security reasons I also tested if tags are removed by substring match and fortunately they are not. Curiously enough the string that matches the tag which should be removed is case sensitive like it sould be in the Zotero API. I really wonder why the same does not comply with the way Tags are handled inside the Zotero GUI. So everything is great, thx.

The only small issue which might be a question of consistency within pyzotero is that the add_tags method add tags as a list of n parameters whereas the new del_tags method takes one parameter that is a list of n tags. I do not know if you want to change that but I wanted to mention it at least.

Thx

[urschrei]

Hmm I'll have a think about the consistency issue.