Ibid

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

  1. empty-gdefine-719851
  2. Merge into old-trunk-1.6
Proposed by Stefano Rivera
Status: Merged
Approved by: Stefano Rivera
Approved revision: 1004
Merged at revision: 1010
Proposed branch: lp:~stefanor/ibid/empty-gdefine-719851
Merge into: lp:~ibid-core/ibid/old-trunk-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
Reviewer Review Type Date Requested Status
Max Rabkin Approve
Jonathan Hitchcock Approve
marcog (community) Approve
Review via email: mp+49927@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-17 17:46:59 +0000
3+++ ibid/plugins/google.py 2011-02-16 08:06:22 +0000
4@@ -133,7 +133,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