Skip to content

Commit

Permalink
Explicit issues order in API. Fixes tests for mysql. Again :)
Browse files Browse the repository at this point in the history
Signed-off-by: Dmitriy Zaporozhets <[email protected]>
  • Loading branch information
dzaporozhets committed Sep 4, 2014
1 parent 678f3dd commit 66516da
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion lib/api/issues.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ def filter_issues_state(issues, state = nil)
case state
when 'opened' then issues.opened
when 'closed' then issues.closed
else issues.order('id DESC')
else issues
end
end

Expand All @@ -35,6 +35,7 @@ def filter_issues_labels(issues, labels)
issues = current_user.issues
issues = filter_issues_state(issues, params[:state]) unless params[:state].nil?
issues = filter_issues_labels(issues, params[:labels]) unless params[:labels].nil?
issues = issues.order('issues.id DESC')

present paginate(issues), with: Entities::Issue
end
Expand All @@ -60,6 +61,7 @@ def filter_issues_labels(issues, labels)
issues = user_project.issues
issues = filter_issues_state(issues, params[:state]) unless params[:state].nil?
issues = filter_issues_labels(issues, params[:labels]) unless params[:labels].nil?
issues = issues.order('issues.id DESC')

present paginate(issues), with: Entities::Issue
end
Expand Down

0 comments on commit 66516da

Please sign in to comment.