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

Issue with tmdb_writer and TMDB photo #118

Closed
bigdork opened this issue Nov 16, 2020 · 3 comments · Fixed by #115
Closed

Issue with tmdb_writer and TMDB photo #118

bigdork opened this issue Nov 16, 2020 · 3 comments · Fixed by #115

Comments

@bigdork
Copy link

bigdork commented Nov 16, 2020

Crash when TMDB missing photo, maybe others than tmdb_writer but I've not encountered.
I'll be uploading images to TMDB for the writers I'm having trouble with, but FYI:

  Writer Dean Koontz:
    sort_title: -5 Writer Dean Koontz
    tmdb_writer: 29923
| Updating collection: Writer Dean Koontz...
| Sync Mode: append
Traceback (most recent call last):
  File "/opt/scripts/Plex-Auto-Collections/app/plex_auto_collections.py", line 986, in <module>
    update_from_config(config_path, plex, True, args.no_meta, args.no_images)
  File "/opt/scripts/Plex-Auto-Collections/app/plex_auto_collections.py", line 408, in update_from_config
    details["poster"] = ["url", tmdb_get_metadata(config_path, id, "profile_path"), method_name]
  File "/opt/scripts/Plex-Auto-Collections/app/imdb_tools.py", line 507, in tmdb_get_metadata
    return tmdb_url_prefix + person.details(id).profile_path
TypeError: must be str, not NoneType
@meisnate12
Copy link

nice catch ill add in something to check for this and let the user know no photo was found on tmdb

@bigdork
Copy link
Author

bigdork commented Nov 16, 2020

I uploaded photos but still crashing, probably need to wait a little bit.
If it's not that, these are the people I added that are crashing the script:

  Writer Dean Koontz:
    tmdb_writer: 29923
  Writer James Patterson:
    tmdb_writer: 21022
  Writer John Grisham:
    tmdb_writer: 11060
  Writer Tom Clancy:
    tmdb_writer: 38568

@meisnate12
Copy link

it probably hasn't updated on their end I'd give it 24 hours

meisnate12 added a commit to meisnate12/Plex-Auto-Collections-Replacement-Plex-Meta-Manager that referenced this issue Nov 16, 2020
meisnate12 added a commit to meisnate12/Plex-Auto-Collections-Replacement-Plex-Meta-Manager that referenced this issue Nov 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants