Ibid

Merge lp:~stefanor/ibid/empty-gdefine-719851-0.1 into lp:~ibid-core/ibid/old-release-0.1-1.6

  1. empty-gdefine-719851-0.1
  2. Merge into old-release-0.1-1.6
Proposed by Stefano Rivera
Status: Merged
Approved by: Stefano Rivera
Approved revision: 975
Merged at revision: 981
Proposed branch: lp:~stefanor/ibid/empty-gdefine-719851-0.1
Merge into: lp:~ibid-core/ibid/old-release-0.1-1.6
Diff against target: 13 lines (+2/-1)
1 file modified
ibid/plugins/google.py (+2/-1)
To merge this branch: bzr merge lp:~stefanor/ibid/empty-gdefine-719851-0.1
Reviewer Review Type Date Requested Status
Max Rabkin Approve
Jonathan Hitchcock Approve
marcog (community) Approve
Review via email: mp+49928@code.launchpad.net

Commit message

Filter out empty definitions in gdefine

To post a comment you must log in.
Revision history for this message
marcog (marco-gallotta) :
review: Approve
Revision history for this message
Jonathan Hitchcock (vhata) :
review: Approve
Revision history for this message
Max Rabkin (max-rabkin) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ibid/plugins/google.py'
2--- ibid/plugins/google.py 2011-01-19 19:21:58 +0000
3+++ ibid/plugins/google.py 2011-02-16 08:06:42 +0000
4@@ -130,7 +130,8 @@
5
6 definitions = []
7 for li in tree.findall('.//li'):
8- definitions.append(li.text)
9+ if li.text:
10+ definitions.append(li.text)
11
12 if definitions:
13 event.addresponse(u' :: '.join(definitions))

Subscribers

People subscribed via source and target branches