Skip to content

Commit

Permalink
Fix duplicate issues when searching.
Browse files Browse the repository at this point in the history
Don't search for open and closed as the legacy request doesn't work.
This is just a temporal solution, I've added a TODO to fix the request
ASAP.

Fixes pockethub#495
  • Loading branch information
jonan committed Dec 18, 2014
1 parent 20186e6 commit c6bb458
Showing 1 changed file with 6 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -98,10 +98,12 @@ public List<SearchIssue> loadData() throws Exception {
if (repository == null)
return Collections.emptyList();
List<SearchIssue> matches = new ArrayList<SearchIssue>();
matches.addAll(service.searchIssues(repository, STATE_OPEN,
query));
matches.addAll(service.searchIssues(repository, STATE_CLOSED,
query));
/** TODO
* This request is using a legacy API that is not working properly
* it needs to be fixed
*/
matches.addAll(service.searchIssues(repository, STATE_OPEN, query));
//matches.addAll(service.searchIssues(repository, STATE_CLOSED, query));
Collections.sort(matches, SearchIssueListFragment.this);
return matches;
}
Expand Down

0 comments on commit c6bb458

Please sign in to comment.