Merge lp:~tomek3d/calibre/store into lp:~user-none/calibre/store

Proposed by Tomasz Długosz
Status: Merged
Merged at revision: 8558
Proposed branch: lp:~tomek3d/calibre/store
Merge into: lp:~user-none/calibre/store
Diff against target: 54 lines (+11/-15)
1 file modified
src/calibre/gui2/store/stores/virtualo_plugin.py (+11/-15)
To merge this branch: bzr merge lp:~tomek3d/calibre/store
Reviewer Review Type Date Requested Status
John Schember Pending
Review via email: mp+109524@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/calibre/gui2/store/stores/virtualo_plugin.py'
2--- src/calibre/gui2/store/stores/virtualo_plugin.py 2012-01-19 18:47:19 +0000
3+++ src/calibre/gui2/store/stores/virtualo_plugin.py 2012-06-10 14:02:24 +0000
4@@ -40,39 +40,35 @@
5 url = 'http://virtualo.pl/?q=' + urllib.quote(query) + '&f=format_id:4,6,3'
6
7 br = browser()
8- drm_pattern = re.compile("ADE")
9+ no_drm_pattern = re.compile("Znak wodny")
10
11 counter = max_results
12 with closing(br.open(url, timeout=timeout)) as f:
13 doc = html.fromstring(f.read())
14- for data in doc.xpath('//div[@id="product_list"]/div/div[@class="column"]'):
15+ for data in doc.xpath('//div[@id="content"]//div[@class="list_box list_box_border"]'):
16 if counter <= 0:
17 break
18
19- id = ''.join(data.xpath('.//table/tr[1]/td[1]/a/@href'))
20+ id = ''.join(data.xpath('.//div[@class="list_middle_left"]//a/@href'))
21 if not id:
22 continue
23
24 price = ''.join(data.xpath('.//span[@class="price"]/text() | .//span[@class="price abbr"]/text()'))
25- cover_url = ''.join(data.xpath('.//table/tr[1]/td[1]/a/img/@src'))
26- title = ''.join(data.xpath('.//div[@class="title"]/a/text()'))
27- title = re.sub(r'\ WM', '', title)
28- author = ', '.join(data.xpath('.//div[@class="authors"]/a/text()'))
29- formats = ', '.join(data.xpath('.//span[@class="format"]/a/text()'))
30- formats = re.sub(r'(, )?ONLINE(, )?', '', formats)
31- drm = drm_pattern.search(formats)
32- formats = re.sub(r'(, )?ADE(, )?', '', formats)
33- formats = re.sub(r'\ WM', '', formats)
34+ cover_url = ''.join(data.xpath('.//div[@class="list_middle_left"]//a/img/@src'))
35+ title = ''.join(data.xpath('.//div[@class="list_title list_text_left"]/a/text()'))
36+ author = ', '.join(data.xpath('.//div[@class="list_authors list_text_left"]/a/text()'))
37+ formats = [ form.split('_')[-1].replace('.png', '') for form in data.xpath('.//div[@style="width:55%;float:left;text-align:left;height:18px;"]//img/@src')]
38+ nodrm = no_drm_pattern.search(''.join(data.xpath('.//div[@style="width:45%;float:right;text-align:right;height:18px;"]/div/div/text()')))
39
40 counter -= 1
41
42 s = SearchResult()
43 s.cover_url = cover_url.split('.jpg')[0] + '.jpg'
44- s.title = title.strip() + ' ' + formats
45+ s.title = title.strip()
46 s.author = author.strip()
47 s.price = price + ' zł'
48 s.detail_item = 'http://virtualo.pl' + id.strip().split('http://')[0]
49- s.formats = formats.upper().strip()
50- s.drm = SearchResult.DRM_LOCKED if drm else SearchResult.DRM_UNLOCKED
51+ s.formats = ', '.join(formats).upper()
52+ s.drm = SearchResult.DRM_UNLOCKED if nodrm else SearchResult.DRM_UNKNOWN
53
54 yield s

Subscribers

People subscribed via source and target branches

to all changes: