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

Updated to latest version... #40

Closed MikeFalcor closed 8 years ago

MikeFalcor commented 8 years ago

So I recently updated PlexEmail to the latest version and now I am getting errors when I try to run it:

Traceback (most recent call last): File "plexEmail.py", line 1037, in <module> imageInfo = processImage(hash, imageInfo['thumb'], 'show', 0, 0) File "plexEmail.py", line 343, in processImage imgName = thumb[thumb.index('_') + 1:len(thumb)] ValueError: substring not found

I was able to modify previous errors away, but this one I cannot get my head around. Any idea what could be causing this?

BurntFrost commented 8 years ago

Pull request 29 should fix this

https://github.com/jakewaldron/PlexEmail/pull/29

On Fri, Jul 29, 2016, 5:42 PM Mike notifications@github.com wrote:

So I recently updated PlexEmail to the latest version and now I am getting errors when I try to run it:

Traceback (most recent call last): File "plexEmail.py", line 1037, in imageInfo = processImage(hash, imageInfo['thumb'], 'show', 0, 0) File "plexEmail.py", line 343, in processImage imgName = thumb[thumb.index('_') + 1:len(thumb)] ValueError: substring not found

I was able to modify previous errors away, but this one I cannot get my head around. Any idea what could be causing this?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jakewaldron/PlexEmail/issues/40, or mute the thread https://github.com/notifications/unsubscribe-auth/AAktmF98xPQcXEwZhA5qjpg5JfTFipUAks5qap4HgaJpZM4JYrlU .

Thanks, Steve

MikeFalcor commented 8 years ago

Alrighty. So, I switched over to the dev build and manually added the info from pull #29.

Here is the tail from the logging (nice job incorporating this, by the way!): 2016-07-29 22:37:46,976 - INFO:processImage: Setting image paths to cloudinary 2016-07-29 22:37:46,980 - INFO:processImage: begin 2016-07-29 22:37:46,980 - INFO:processImage: imageHash = 42be2fb215fac6654b5b0a799de8811713a4d4f8 - thumb = https://thetvdb.com/banners/posters/305288-4.jpg - mediaType = show - seasonIndex = 0 - episodeIndex = 0 2016-07-29 22:37:46,981 - ERROR:Logging an uncaught exception Traceback (most recent call last): File "plexEmail.py", line 1120, in <module> imageInfo = processImage(hash, imageInfo['thumb'], 'show', 0, 0) File "plexEmail.py", line 379, in processImage imgName = thumb[thumb.index('_') + 1:len(thumb)] ValueError: substring not found

Bear in mind I have edited the HTML portion of plexEmail.py, so my lines are a little off.

It would appear as though the error is the same even after adding all of the pulls. Any ideas?

MikeFalcor commented 8 years ago

Doh! In the Dev version, you need to apply the pull bugfix about 5 times as opposed to only once in the master. Testing again...

MikeFalcor commented 8 years ago

OR...I could actually pay attention to the pull and take note that it only applied to one specific section of code...

Don't mind me. Added the pull correctly this time to the master version. New error: title += movies[movie]['original_title'] + ' AKA ' TypeError: unsupported operand type(s) for +: 'NoneType' and 'str'

Yay?

MikeFalcor commented 8 years ago

Actually...I will close this issue to keep things neat. Will create new issue for new error. Thanks!