Merge lp:~lifanxi/calibre/douban-metadata-plugin into lp:calibre

Proposed by Li Fanxi
Status: Merged
Merged at revision: 9313
Proposed branch: lp:~lifanxi/calibre/douban-metadata-plugin
Merge into: lp:calibre
Diff against target: 14 lines (+4/-1)
1 file modified
src/calibre/ebooks/metadata/sources/douban.py (+4/-1)
To merge this branch: bzr merge lp:~lifanxi/calibre/douban-metadata-plugin
Reviewer Review Type Date Requested Status
Kovid Goyal Pending
Review via email: mp+62341@code.launchpad.net

Description of the change

Bug fix for Douban metadata plugin. Error in constructing the API request URL when adding the API key to the URL.

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
=== modified file 'src/calibre/ebooks/metadata/sources/douban.py'
--- src/calibre/ebooks/metadata/sources/douban.py 2011-05-12 23:12:59 +0000
+++ src/calibre/ebooks/metadata/sources/douban.py 2011-05-25 16:37:39 +0000
@@ -211,7 +211,10 @@
211 'q': q,211 'q': q,
212 })212 })
213 if self.DOUBAN_API_KEY and self.DOUBAN_API_KEY != '':213 if self.DOUBAN_API_KEY and self.DOUBAN_API_KEY != '':
214 url = url + "?apikey=" + self.DOUBAN_API_KEY214 if t == "isbn" or t == "subject":
215 url = url + "?apikey=" + self.DOUBAN_API_KEY
216 else:
217 url = url + "&apikey=" + self.DOUBAN_API_KEY
215 return url218 return url
216 # }}}219 # }}}
217220

Subscribers

People subscribed via source and target branches