jakewaldron / PlexEmail

This script aggregates all new TV, movie and music releases for the past configured time then optionally writes to your web directory and sends out an email.
206 stars 32 forks source link

GUID detection from config file #69

Closed pengc99 closed 7 years ago

pengc99 commented 7 years ago

I'm not a Python programmer, but I don't know if this is correct. On my setup, I don't use DLNA so I need to set the GUID of the server in the config file.

However, after some debugging, I found that the plexEmail script always tries to hit the DLNA server for the GUID and the value I put in the config file.

if (config['filter_include_plex_web_link']):
  if (config['plex_web_server_guid'] == ''):
    plexWebLink = 'http://plex.tv/web/app#!/server/' + config['plex_web_server_guid'] + '/details/%2Flibrary%2Fmetadata%2F'

Perhaps that comparison should be: if (config['plex_web_server_guid'] != ''):

jakewaldron commented 7 years ago

Good catch. Looks like this is broken (guess not too many people use this feature). Will fix it for a future release.

jakewaldron commented 7 years ago

Fixed in dev branch

jakewaldron commented 7 years ago

Dev branch has now been merged into master. Closing.