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

imdbphp 2 no longer pulling all information #89

Open
underslung opened this issue Mar 4, 2016 · 8 comments
Open

imdbphp 2 no longer pulling all information #89

underslung opened this issue Mar 4, 2016 · 8 comments

Comments

@underslung
Copy link

I noticed recent additions weren't pulling images so I went to check for an imdbphp update and found it's been restructured. It's easy to move from v2 to v3, though. Download and extract it into ./lib/site and symlink imdbphp2 to it, then change this in imdbphp.class.php:

//include_once("./lib/site/imdbphp2/imdb.class.php");
//include_once("./lib/site/imdbphp2/imdbsearch.class.php");
include_once("./lib/site/imdbphp2/bootstrap.php");

Then at line 118 change imdb to \Imdb\Title :

                    $imdb = new \Imdb\Title($search_attributes_r['imdb_id']);
@pellcorp
Copy link
Owner

pellcorp commented Mar 7, 2016

Can you create a pull request with this change. I can then merge from ui

@underslung
Copy link
Author

Yeah, I will when I look at it a bit more, this isn't really a complete solution, just got me up and running. How relevant is the imdb.config.php file these days? It also references an old class and is just getting ignored. I edited Config.php in the new imdbphp directly.

@pellcorp
Copy link
Owner

pellcorp commented Mar 9, 2016

If its ignored it sounds fine to delete it.

@pellcorp
Copy link
Owner

Did you get anywhere with this I never saw an mr

@underslung
Copy link
Author

underslung commented Oct 18, 2017 via email

@pellcorp
Copy link
Owner

If we can avoid soft links and just copy imdbphp into the lib for. Opendb is windows compatible so no soft link

@pellcorp
Copy link
Owner

Did you make progress with this

@pellcorp
Copy link
Owner

You could clone this repo
Then copy your changes over top of that checked out workspace and that should give you most of the changes required for a Mr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants