[ videodb-Bugs-1520726 ] Cover URL isn't working

Bugs item #1520726, was opened at 2006-07-11 13:07
Message generated for change (Comment added) made by bxmas13
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=586362&aid=1520726&group_id=88349

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: engines
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Sree (sjallipa)
Assigned to: Nobody/Anonymous (nobody)
Summary: Cover URL isn't working

Initial Comment:
When I try to download Cover URL for some of the movies
and do a internet search (using the magnifying glass)
it returns no results almost every time (even for those
that actually VideoDB found before), not sure what the
problem is.
Thanks
Sree

----------------------------------------------------------------------

Comment By: Brian C. (bxmas13)
Date: 2006-09-10 21:32

Message:
Logged In: YES 
user_id=1595144

OK last post!  Someone already did this fix.  It is in the 
CVS portion of this site.  Just click on the "Download" at 
the top of this page:

http://videodb.cvs.sourceforge.net/videodb/videodb/engines/
google.php?view=log

----------------------------------------------------------------------

Comment By: Brian C. (bxmas13)
Date: 2006-09-10 21:26

Message:
Logged In: YES 
user_id=1595144

Sorry, updated with no line spaces:  


if(preg_match_all('#/imgres\?imgurl=(.+?)&.+?<img src=
(.+?)\swidth=(\d+).+?height=(\d+)#', $resp['data'], $data, 
PREG_SET_ORDER)){
foreach ($data as $row) {
$info = array();
$info['title'] = $row[3].'x'.$row[4]; // Width x height
$info['imgsmall'] = $googleServer.$row[2]; // small 
thumbnail url
$info['coverurl'] = $row[1]; // resulting target url
$result[] = $info;
        }
    }


----------------------------------------------------------------------

Comment By: Brian C. (bxmas13)
Date: 2006-09-10 21:19

Message:
Logged In: YES 
user_id=1595144

I have it working somewhat.  Here is what I did:

Modified /engines/google.php

FIND: 
    if (preg_match_all('#/imgres\?imgurl=(.+?)&.+?<img src=
(.+?)\swidth=(\d+).+?height=(\d+)#', $resp['data'], $data, 
PREG_SET_ORDER))
    {
        foreach ($data as $row) 
        {
            $info = array();
            $info['title']      = $row[3].'x'.$row
[4];      // width x height
            $info['imgsmall']   = $googleServer.$row
[2];    // small thumbnail url
            $info['coverurl']   = $row
[1];                  // resulting target url
            $result[]           = $info;
        }
    }


REPLACE WITH:
        if (preg_match_all('#,"http:
(.+?)","(.+?)","(.+?)"#', $resp['data'], $data, 
PREG_SET_ORDER)){
                foreach ($data as $row) 
        {
            $info = array();
            $info['title']      = $row[2].'x'.$row
[3];      // width x height
            $info['imgsmall']   = "http:".$row[1];    // 
small thumbnail url
            $info['coverurl']   = "http:".$row
[1];                  // resulting target url
            $result[]           = $info;
        }
    }


The code isn't 100% perfect, but it works!  Comments/Fixes 
appreciated.

-Brian




----------------------------------------------------------------------

Comment By: bard_h (bard_h)
Date: 2006-07-11 18:34

Message:
Logged In: YES 
user_id=1554022

Google have changed the layout of image result pages. The
image URL is now hidden in JavaScript functions. 

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=586362&aid=1520726&group_id=88349

-------------------------------------------------------------------------
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