l3uddz / plex_dupefinder

Find and delete duplicate files in Plex
GNU General Public License v3.0
303 stars 54 forks source link

unexpected entry causes ValueError: invalid literal for int() with base 10: #41

Open ackerthehacker2 opened 4 years ago

ackerthehacker2 commented 4 years ago

Describe the bug in keep_item, it is expecting a "s", "b" or a number. If you type ss by mistake, you will get Choose item to keep (0 or s = skip | 1 or b = best): ss Traceback (most recent call last): File "./plex_dupefinder.py", line 413, in if (keep_item.lower() != 's') and (keep_item.lower() == 'b' or 0 < int(keep_item) <= len(media_items)): ValueError: invalid literal for int() with base 10: 'ss'

To Reproduce

  1. Start plex_dupefinder
  2. Get to an entry
  3. Enter "ss" for example

Expected behavior Either reask for input (it should validate that the input is correct) and prompt (ideally) or skip with unexpected

Screenshots Choose item to keep (0 or s = skip | 1 or b = best): ss Traceback (most recent call last): File "./plex_dupefinder.py", line 413, in if (keep_item.lower() != 's') and (keep_item.lower() == 'b' or 0 < int(keep_item) <= len(media_items)): ValueError: invalid literal for int() with base 10: 'ss'

Logs Link to debug or trace log files.

You can enable debug mode by adding --loglevel=DEBUG to the run command.

System Information

900d4a0

Additional context Add any other context about the problem here.