Ibid

Merge lp:~stefanor/ibid/factoid-search-722655 into lp:~ibid-core/ibid/old-trunk-1.6

  1. factoid-search-722655
  2. Merge into old-trunk-1.6
Proposed by Stefano Rivera
Status: Merged
Approved by: Stefano Rivera
Approved revision: 1011
Merged at revision: 1014
Proposed branch: lp:~stefanor/ibid/factoid-search-722655
Merge into: lp:~ibid-core/ibid/old-trunk-1.6
Diff against target: 12 lines (+1/-1)
1 file modified
ibid/plugins/factoid.py (+1/-1)
To merge this branch: bzr merge lp:~stefanor/ibid/factoid-search-722655
Reviewer Review Type Date Requested Status
Jonathan Hitchcock Approve
Max Rabkin Approve
marcog (community) Approve
Review via email: mp+50617@code.launchpad.net

Commit message

Use bounded_matches when returning search results

Description of the change

Regression from factoid-2

To post a comment you must log in.
lp:~stefanor/ibid/factoid-search-722655 updated
1011. By Stefano Rivera

Use bounded_matches when returning search results

Revision history for this message
marcog (marco-gallotta) :
review: Approve
Revision history for this message
Max Rabkin (max-rabkin) :
review: Approve
Revision history for this message
Jonathan Hitchcock (vhata) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ibid/plugins/factoid.py'
2--- ibid/plugins/factoid.py 2011-02-20 18:04:24 +0000
3+++ ibid/plugins/factoid.py 2011-02-21 15:39:57 +0000
4@@ -463,7 +463,7 @@
5 if bounded_matches:
6 event.addresponse(u'; '.join(
7 u'%s [%s]' % (fname.name, len(factoid.values))
8- for factoid, fname in matches))
9+ for factoid, fname in bounded_matches))
10 elif len(matches):
11 event.addresponse(u"I could only find %(number)d things that matched '%(pattern)s'", {
12 u'number': len(matches),

Subscribers

People subscribed via source and target branches