search

bookieio / Bookie

Python based delicious.com replacement
GNU Affero General Public License v3.0
633 stars 138 forks source link

/bmarks/search/:terms not respecting count parameter #450

Closed dyladan closed 10 years ago

dyladan commented 10 years ago

https://bmark.us/api/v1//bmarks/search/stuff&count=2 is returning 10 bookmarks. Like #449 it appears that it is using the parameter as a part of the search term as shown by the JSON returned "phrase": "stuff&count=2"

dyladan commented 10 years ago

WOW i just realized what was causing this. I was using & instead of ?

dyladan commented 10 years ago

ok so there is still an issue here

https://bmark.us/api/v1/bmarks/search/coroutines returns 5 results

https://bmark.us/api/v1/bmarks/search/coroutines?count=2 returns 2 results

https://bmark.us/api/v1/dyladan/bmarks/search/coroutines returns 1 result

https://bmark.us/api/v1/dyladan/bmarks/search/coroutines?count=1 returns 0 results

dyladan commented 10 years ago

this appears related to #449

mitechie commented 10 years ago

Hmm, so the bug here with the count is corrected, but the number of results seems shy for your bookmarks. I'm closing this bug but looking into matching up the number of fulltext indexed bookmarks with the number of total bookmarks.