Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Default category preference not saved for future torrents #12311

Open
ghost opened this issue Mar 28, 2020 · 7 comments
Open

Default category preference not saved for future torrents #12311

ghost opened this issue Mar 28, 2020 · 7 comments
Assignees
Labels
GUI GUI-related issues/changes

Comments

@ghost
Copy link

ghost commented Mar 28, 2020

Please provide the following information

qBittorrent version and Operating System

V4.2.2 macOS 10.14.6

If on linux, libtorrent-rasterbar and Qt version

(type here)

What is the problem

When setting the default category to "Movie" in the torrent dialog window and selecting "save as default" the option is not saved for future torrents (after choosing not to display the torrent dialog in future torrents in preferences). Choosing to show the torrent dialog again shows that the default is still selected as "Movie" but new torrents don't have that category.

What is the expected behavior

After setting "Movie" as the default category and choosing not to view torrent dialog when adding a torrent that all torrents are tagged with "Movie" for their category

Steps to reproduce

  1. In Downloads Prefs check "display torrent content and some options"
  2. add a torrent
  3. when the torrent dialog appears select Category "Movies"
  4. check the box below for "set as default category"
  5. Click "OK"
  6. Open Downloads Preferences and deselect "display torrent content and some options"
  7. Click "Apply"

Extra info(if any)

Thank you

@glassez
Copy link
Member

glassez commented Mar 28, 2020

The option that you use in "Add Torrent" dialog affects "Add Torrent" dialog itself only. qBittorrent has no global "Default category" option.

@ghost
Copy link
Author

ghost commented Mar 29, 2020

I understand now. It wasn't clear. Other settings appear to be global in that same dialog box.

I suppose I should create a "feature" request?

It seems counterproductive in conjunction with the option to avoid the dialog window which is arguably there to streamline the download experience. I use a script that looks for a "category" when post processing torrents. A category helps this script decide if it's a movie or tvshow.

@FranciscoPombal
Copy link
Member

I agree with @jprokos, the option should probably be renamed to "remember category selection for future added torrents" or equivalent.

The way it is now, it makes it seem like future added torrents will be assigned that category by default.

@jprokos For now, don't create a new "feature request" issue, let's continue the discussion and hear more opinions first. @glassez @Chocobo1

@glassez
Copy link
Member

glassez commented Mar 29, 2020

the option should probably be renamed to "remember category selection for future added torrents" or equivalent.

"Remember category selection" is what it really does so it should be renamed accordingly.

@FranciscoPombal
Copy link
Member

FranciscoPombal commented Mar 29, 2020

@glassez

"Remember category selection" is what it really does.

Yes, but that's not obvious from its current description/name.

@ghost
Copy link
Author

ghost commented Mar 29, 2020

@FranciscoPombal Renaming would avoid confusion. @glassez was suggesting that very thing

...it should be renamed accordingly.

A global default — this feature would be helpful. Where would that global preference live?? I don't know what would be consistent with the current user interface.

@FranciscoPombal
Copy link
Member

@glassez

"Remember category selection" is what it really does so it should be renamed accordingly

I was recently thinking about this. I don't think renaming the checkbox to "Remember category selection" will be of any help - the users will have the expectation that the remembered category will actually be applied in the future. Why not remove the checkbox altogether, and always remember the last used category in the dialog? This will prevent the expectation that the category is set as default even when not using the dialog. Then, later, a proper "default category" option that controls the default assigned category (regardless of whether the "add torrent dialog" is skipped) can be implemented in the settings.

One question - if we do decide to remove it, do I have to delete the corresponding entry from the TS files?

@FranciscoPombal FranciscoPombal added the GUI GUI-related issues/changes label Jul 3, 2020
@FranciscoPombal FranciscoPombal self-assigned this Jul 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI GUI-related issues/changes
Projects
None yet
Development

No branches or pull requests

2 participants