diff --git a/src/base/bittorrent/session.cpp b/src/base/bittorrent/session.cpp index b081f98d9..bd804bddf 100644 --- a/src/base/bittorrent/session.cpp +++ b/src/base/bittorrent/session.cpp @@ -857,17 +857,13 @@ bool Session::hasTag(const QString &tag) const bool Session::addTag(const QString &tag) { - if (!isValidTag(tag)) + if (!isValidTag(tag) || hasTag(tag)) return false; - if (!hasTag(tag)) - { - m_tags.insert(tag); - m_storedTags = m_tags.values(); - emit tagAdded(tag); - return true; - } - return false; + m_tags.insert(tag); + m_storedTags = m_tags.values(); + emit tagAdded(tag); + return true; } bool Session::removeTag(const QString &tag) @@ -2083,7 +2079,6 @@ LoadTorrentParams Session::initLoadTorrentParams(const AddTorrentParams &addTorr LoadTorrentParams loadTorrentParams; loadTorrentParams.name = addTorrentParams.name; - loadTorrentParams.tags = addTorrentParams.tags; loadTorrentParams.firstLastPiecePriority = addTorrentParams.firstLastPiecePriority; loadTorrentParams.hasSeedStatus = addTorrentParams.skipChecking; // do not react on 'torrent_finished_alert' when skipping loadTorrentParams.contentLayout = addTorrentParams.contentLayout.value_or(torrentContentLayout()); @@ -2108,6 +2103,12 @@ LoadTorrentParams Session::initLoadTorrentParams(const AddTorrentParams &addTorr else loadTorrentParams.category = addTorrentParams.category; + for (const QString &tag : addTorrentParams.tags) + { + if (hasTag(tag) || addTag(tag)) + loadTorrentParams.tags.insert(tag); + } + return loadTorrentParams; }