jesec / flood

A modern web UI for various torrent clients with a Node.js backend and React frontend.

https://flood.js.org

GNU General Public License v3.0

2.24k stars 135 forks source link

Closed LXGaming closed 3 months ago

LXGaming commented 3 months ago

Description

Flood is attempting to remove the tag undefined when it's suppose to be removing all tags.

Coalesce undefined to a blank string.

[ ] Breaking change (changes that break backward compatibility of public API or CLI - semver MAJOR)
[ ] New feature (non-breaking change which adds functionality - semver MINOR)
[x] Bug fix (non-breaking change which fixes an issue - semver PATCH)

trim21 commented 3 months ago

so {tags: ''} will remove all tags?

LXGaming commented 3 months ago

Sorry I should have mentioned that in the PR, Yes sending {tags: ''} will remove all tags, this behavior is documented https://github.com/qbittorrent/qBittorrent/wiki/WebUI-API-(qBittorrent-4.1)#remove-torrent-tags

tags is the list of tags you want to remove from passed torrents. Empty list removes all tags from relevant torrents.