plex_dupefinder icon indicating copy to clipboard operation
plex_dupefinder copied to clipboard

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

Open ackerthehacker2 opened this issue 5 years ago • 0 comments

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

  • Plex DupeFinder Version: 900d4a0
  • Operating System: MacOS 10.15.5 Python 3.8.2 (v3.8.2:7b3ab5921f, Feb 24 2020, 17:52:18)

900d4a0

Additional context Add any other context about the problem here.

ackerthehacker2 avatar May 09 '20 23:05 ackerthehacker2