Annotate changelog bug being closed.

[youtubedl] / youtube_dl / extractor / googlesearch.py

diff --git a/youtube_dl/extractor/googlesearch.py b/youtube_dl/extractor/googlesearch.py
index 21c240e512b6c544efa616d95fcb1b0a76865a8a..498304cb2bd9b605d44e67291a2f38bf4481a6f8 100644 (file)
--- a/youtube_dl/extractor/googlesearch.py
+++ b/youtube_dl/extractor/googlesearch.py
@@ -1,39 +1,59 @@
+from __future__ import unicode_literals
+

 import itertools
 import re
 
 from .common import SearchInfoExtractor
 import itertools
 import re
 
 from .common import SearchInfoExtractor

-from ..utils import (
+from ..compat import (

     compat_urllib_parse,
 )
 
 
 class GoogleSearchIE(SearchInfoExtractor):
     compat_urllib_parse,
 )
 
 
 class GoogleSearchIE(SearchInfoExtractor):

-    """Information Extractor for Google Video search queries."""
-    _MORE_PAGES_INDICATOR = r'id="pnnext" class="pn"'
+    IE_DESC = 'Google Video search'

     _MAX_RESULTS = 1000
     _MAX_RESULTS = 1000

-    IE_NAME = u'video.google:search'
+    IE_NAME = 'video.google:search'

     _SEARCH_KEY = 'gvsearch'
     _SEARCH_KEY = 'gvsearch'

+    _TEST = {
+        'url': 'gvsearch15:python language',
+        'info_dict': {
+            'id': 'python language',
+            'title': 'python language',
+        },
+        'playlist_count': 15,
+    }

 
     def _get_n_results(self, query, n):
         """Get a specified number of results for a query"""
 
 
     def _get_n_results(self, query, n):
         """Get a specified number of results for a query"""
 

+        entries = []

         res = {
             '_type': 'playlist',
             'id': query,
         res = {
             '_type': 'playlist',
             'id': query,

-            'entries': []
+            'title': query,

         }
 
         }
 

-        for pagenum in itertools.count(1):
-            result_url = u'http://www.google.com/search?tbm=vid&q=%s&start=%s&hl=en' % (compat_urllib_parse.quote_plus(query), pagenum*10)
-            webpage = self._download_webpage(result_url, u'gvsearch:' + query,
-                                             note='Downloading result page ' + str(pagenum))
+        for pagenum in itertools.count():
+            result_url = (
+                'http://www.google.com/search?tbm=vid&q=%s&start=%s&hl=en'
+                % (compat_urllib_parse.quote_plus(query), pagenum * 10))
+
+            webpage = self._download_webpage(
+                result_url, 'gvsearch:' + query,
+                note='Downloading result page ' + str(pagenum + 1))
+
+            for hit_idx, mobj in enumerate(re.finditer(
+                    r'<h3 class="r"><a href="([^"]+)"', webpage)):
+
+                # Skip playlists
+                if not re.search(r'id="vidthumb%d"' % (hit_idx + 1), webpage):
+                    continue

 
 

-            for mobj in re.finditer(r'<h3 class="r"><a href="([^"]+)"', webpage):
-                e = {
+                entries.append({

                     '_type': 'url',
                     'url': mobj.group(1)
                     '_type': 'url',
                     'url': mobj.group(1)

-                }
-                res['entries'].append(e)
+                })

 
 

-            if (pagenum * 10 > n) or not re.search(self._MORE_PAGES_INDICATOR, webpage):
+            if (len(entries) >= n) or not re.search(r'id="pnnext"', webpage):
+                res['entries'] = entries[:n]

                 return res
                 return res