Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Images not displaying correctly on home/wanted page #52

Closed
Enjoy14 opened this issue Aug 31, 2013 · 8 comments
Closed

Images not displaying correctly on home/wanted page #52

Enjoy14 opened this issue Aug 31, 2013 · 8 comments

Comments

@Enjoy14
Copy link

Enjoy14 commented Aug 31, 2013

When I do a search for movies, poster pictures show up correctly, but when I transfer them to the wanted page the poster shows as a broken link (see attached pics)

missing pic

After clicking on the missing pic:

clicking on missing pic

@Enjoy14 Enjoy14 closed this as completed Aug 31, 2013
@Enjoy14 Enjoy14 reopened this Aug 31, 2013
@lad1337
Copy link
Owner

lad1337 commented Aug 31, 2013

On the wanted page the images are loaded from themoviedb.org when added to XDM (wanted page) they are downloaded and saved in the "image" folder in the XDM data dir

  • do you use webroot ?
  • is the image file on the harddrive ? (the image folder is most likely in the XDM install dir. or if using the mac app at ~/Library/Application Support/XDM/)
  • check the url of the missing image url, how does it look ?
  • when adding a movie it should tell in the log what image it is downloading

@Enjoy14
Copy link
Author

Enjoy14 commented Sep 1, 2013

I am not using webroot.
Yes, the file is saved to the hard drive. In my case it is located at C:\XDM\images\de.lad1337.movies
Here is the url for the missing file:
http://localhost:8085/images%5Cde.lad1337.movies%5CThe%20Book%20of%20Eli%20%282010%29%20%2811%29%20poster_image.jpeg

Here is the image download log (seems to be working just fine):

INFO 2013-08-31 23:35:43.068000
Init search on Movies (de.lad1337.movies) for the book of eli
DEBUG 2013-08-31 23:35:45.518000
init paint on given root Movies(de.lad1337.movies)[10] the book of eli
DEBUG 2013-08-31 23:35:56.070000
Downloading image: poster_image http://d3gtl9l2a4fn1j.cloudfront.net/t/p/w185/qL3FnEug9DyBcaBXVb0oT3DJMJu.jpg
DEBUG 2013-08-31 23:35:56.367000
Running searchElement with args: (,) and kwargs: {}
DEBUG 2013-08-31 23:35:56.373000
Movie(de.lad1337.movies)[11] The Book of Eli (2010) is released at 2010-01-14 00:00:00 but we are supposed to search 2 days ahead

I'm not sure how to get the url to correctly find the image.

Here is what I get when the url is put directly into my browser:

404 Not Found

The path '/images\de.lad1337.movies\Harry Potter and the Deathly Hallows Part 1 (2010) (21) poster_image.jpeg' was not found.

Traceback (most recent call last):
File "C:\XDM\rootLibs\cherrypy_cprequest.py", line 656, in respond
response.body = self.handler()
File "C:\XDM\rootLibs\cherrypy\lib\encoding.py", line 188, in call
self.body = self.oldhandler(_args, *_kwargs)
File "C:\XDM\rootLibs\cherrypy_cperror.py", line 386, in call
raise self
NotFound: (404, "The path '/images\de.lad1337.movies\Harry Potter and the Deathly Hallows Part 1 (2010) (21) poster_image.jpeg' was not found.")

@lad1337
Copy link
Owner

lad1337 commented Sep 1, 2013

Ha i might know hat the error is .... The URL for the Image is composer by the Python path Module,,,, wich is wrong because on Windows the path delimiter are \ not /

Either you are the first person on windows with XDM or the others had browsers that just conformed
Anyway thisis a bug and I can fix it but right now I am not on my computer ...

@Enjoy14
Copy link
Author

Enjoy14 commented Sep 1, 2013

FWIW, I'm getting the same broken link in IE10.0.9200.16660 and Chrome Version 29.0.1547.62 m

@lad1337
Copy link
Owner

lad1337 commented Sep 3, 2013

would you mind trying the develop branch
it should be fixed with f8c7a36

@Enjoy14
Copy link
Author

Enjoy14 commented Sep 3, 2013

switched to the develop branch and it worked. Nice job!

pics working

@lad1337
Copy link
Owner

lad1337 commented Sep 3, 2013

okay cool i'll merge it with the master in the next days till then will leave this issue open (so you will be notified when its in the master) ^^

@lad1337
Copy link
Owner

lad1337 commented Sep 19, 2013

merged

@lad1337 lad1337 closed this as completed Sep 19, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants