Code

gitweb: Make git_search_* subroutines render whole pages
authorJakub Narebski <jnareb@gmail.com>
Wed, 22 Jun 2011 15:28:55 +0000 (17:28 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 22 Jun 2011 18:53:37 +0000 (11:53 -0700)
commit1ae05be4aa448163a381e131396fd52dcf6f83eb
tree057bbb3e9f6e026edcfaa74b62d7922e4508eb54
parent882541b87d8f7bedca824228a8ad14aed69c904e
gitweb: Make git_search_* subroutines render whole pages

Move git_header_html() and git_footer_html() invocation from git_search()
to individual git_search_* subroutines.

While at it, reorganize search-related code a bit, moving invoking of git
commands before any output is generated.

This has the following advantages:

  * gitweb now shows an error page if there was unknown search type
    (evaluate_and_validate_params checks only that it looks sanely);
    remember that we shouldn't call die_error after any output.

  * git_search_message is now safe agains die_error in parse_commits
    (though this is very unlikely).

  * gitweb now can check errors while invoking git commands and show
    error page (again, quite unlikely).

Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
gitweb/gitweb.perl