Allow / in label title #1279

Closed
opened 2026-02-21 15:34:32 -05:00 by deekerman · 3 comments
Owner

Originally created by @ghost on GitHub (Mar 5, 2014).

This is very simple and yet very handy feature.
Suggestion: allow / character in label names everywhere.
Rationale: this is simple and yet very handy modification would allow users to categorize their downloads into subfolders. When "Append label to torrent save path" option is checked character / being filesystem separator would put files labeled as "Label/Subfolder" into ../download_path/Label/Subfolder allowing more fine-grained categorization.

This actually works even now, but there are few minor details:

  1. "../download_path/Label Subfolder" is still created but left empty.
  2. Adding label in main window with character / will complain and wont allow doing it.
Originally created by @ghost on GitHub (Mar 5, 2014). This is very simple and yet very handy feature. Suggestion: allow / character in label names everywhere. Rationale: this is simple and yet very handy modification would allow users to categorize their downloads into subfolders. When "Append label to torrent save path" option is checked character / being filesystem separator would put files labeled as "Label/Subfolder" into ../download_path/Label/Subfolder allowing more fine-grained categorization. This actually works even now, but there are few minor details: 1. "../download_path/Label Subfolder" is still created but left empty. 2. Adding label in main window with character / will complain and wont allow doing it.
Author
Owner

@sledgehammer999 commented on GitHub (Mar 7, 2014):

Read here why this isn't possible atm:

  1. http://qbforums.shiki.hu/index.php/topic,2259.msg9756.html
  2. http://qbforums.shiki.hu/index.php/topic,2259.msg9755.html#msg9755
@sledgehammer999 commented on GitHub (Mar 7, 2014): Read here why this isn't possible atm: 1. http://qbforums.shiki.hu/index.php/topic,2259.msg9756.html 2. http://qbforums.shiki.hu/index.php/topic,2259.msg9755.html#msg9755
Author
Owner

@ghost commented on GitHub (Mar 8, 2014):

I suppose you meant this:
3. Not allowing the "\" character is because that would create additional folders that qbt isn't designed to handle yet.
Excuse my ignorance but is it not as simple as appending label to download path, and then just doing QDir::mkpath() to ensure it is created? For good measure \ and / can be replaced with native OS path separator be it needed. Sounds like simple fix really.

@ghost commented on GitHub (Mar 8, 2014): I suppose you meant this: `3. Not allowing the "\" character is because that would create additional folders that qbt isn't designed to handle yet.` Excuse my ignorance but is it not as simple as appending label to download path, and then just doing QDir::mkpath() to ensure it is created? For good measure \ and / can be replaced with native OS path separator be it needed. Sounds like simple fix really.
Author
Owner

@sledgehammer999 commented on GitHub (Mar 8, 2014):

IIRC qbt does other things with labels internally and assumes that one label == one dir.
Look in the bugstracker everything tagged under "labels". There are 2-3 big wishlist items. Once they get implemented, they will make your suggestion obsolete.

@sledgehammer999 commented on GitHub (Mar 8, 2014): IIRC qbt does other things with labels internally and assumes that one label == one dir. Look in the bugstracker everything tagged under "labels". There are 2-3 big wishlist items. Once they get implemented, they will make your suggestion obsolete.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/qBittorrent#1279
No description provided.