Search doesn’t display enough commits per page

Bjoern A. Zeeb pointed out a problem today. On the following URL, there should be more commits per page. Only three are shown (at the time of writing).

http://www.freshports.org/search.php?stype=committer&method=exact&query=jb&num=100&orderby=category&orderbyupdown=asc&search=Search&include_src_commits=include_src_commits

The page is showing 100 results per page. This should be 100 commits, but the result set is showing only 100 files. The SQL query has the LIMIT on the wrong level. It is on the file level. It should be on the commit level.

Compare the output to this query, which does not demonstrate the problem:

http://www.freshports.org/search.php?stype=tree&method=prefix&query=%2Fports%2Faccessibility%2Fat-poke&num=100&orderby=category&orderbyupdown=asc&search=Search

The main difference between the two search results: the former (with incorrect paging) searches the /src/ tree; the latter searches the /ports/ tree.

For now, I’ll post this as documentation and fix it later. :)

Fixed. Here is what it used to look like.