On Feb 19, Hadley Rich <[EMAIL PROTECTED]> wrote:

> This looks like it is because on the search results page they have
> added a link to the latest episode in the same li tag and the code
> grabs the title but uses last link that was seen in the li to get
> the id.

Exactly; I wasn't aware of this new behavior of the IMDb's search system.
I'll add a test to the test-suite for this case; 'mobile' is not
affected by the problem.

> My quick hack to fix it was to add a check to line 198 of
> searchMovieParser.py to check that the link doesn't end with a / (it
> looks like the latest episode link is the only one which has this).

Can work, but I prefer to stop collecting movieIDs after the first one
is got; it's easily done adding (this fix is in the CVS):
  if self._current_imdbID: return

as the first line of the 'start_a' method.

PS: you can wander a bit around the IMDbPY's code; if you'll still be
interested in doing some development on it (working on something listed
in the TODO.txt file or on IMDbPY-based programs or on something else,
doesn't matter) I will be glad to add you to the developers list,
granting CVS access.
Some work-power is always welcome! :-)


Thank you very much (again!)
-- 
Davide Alberani <[EMAIL PROTECTED]> [PGP KeyID: 0x465BFD47]
http://erlug.linux.it/~da/

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Imdbpy-devel mailing list
Imdbpy-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/imdbpy-devel

Reply via email to