-rw-r--r-- | cgit.c | 1 | ||||
-rw-r--r-- | ui-shared.c | 7 |
2 files changed, 6 insertions, 2 deletions
@@ -272,49 +272,48 @@ static int prepare_repo_cmd(struct cgit_context *ctx) | |||
272 | return 1; | 272 | return 1; |
273 | } | 273 | } |
274 | 274 | ||
275 | if (get_sha1(ctx->qry.head, sha1)) { | 275 | if (get_sha1(ctx->qry.head, sha1)) { |
276 | tmp = xstrdup(ctx->qry.head); | 276 | tmp = xstrdup(ctx->qry.head); |
277 | ctx->qry.head = ctx->repo->defbranch; | 277 | ctx->qry.head = ctx->repo->defbranch; |
278 | cgit_print_http_headers(ctx); | 278 | cgit_print_http_headers(ctx); |
279 | cgit_print_docstart(ctx); | 279 | cgit_print_docstart(ctx); |
280 | cgit_print_pageheader(ctx); | 280 | cgit_print_pageheader(ctx); |
281 | cgit_print_error(fmt("Invalid branch: %s", tmp)); | 281 | cgit_print_error(fmt("Invalid branch: %s", tmp)); |
282 | cgit_print_docend(); | 282 | cgit_print_docend(); |
283 | return 1; | 283 | return 1; |
284 | } | 284 | } |
285 | return 0; | 285 | return 0; |
286 | } | 286 | } |
287 | 287 | ||
288 | static void process_request(void *cbdata) | 288 | static void process_request(void *cbdata) |
289 | { | 289 | { |
290 | struct cgit_context *ctx = cbdata; | 290 | struct cgit_context *ctx = cbdata; |
291 | struct cgit_cmd *cmd; | 291 | struct cgit_cmd *cmd; |
292 | 292 | ||
293 | cmd = cgit_get_cmd(ctx); | 293 | cmd = cgit_get_cmd(ctx); |
294 | if (!cmd) { | 294 | if (!cmd) { |
295 | ctx->page.title = "cgit error"; | 295 | ctx->page.title = "cgit error"; |
296 | ctx->repo = NULL; | ||
297 | cgit_print_http_headers(ctx); | 296 | cgit_print_http_headers(ctx); |
298 | cgit_print_docstart(ctx); | 297 | cgit_print_docstart(ctx); |
299 | cgit_print_pageheader(ctx); | 298 | cgit_print_pageheader(ctx); |
300 | cgit_print_error("Invalid request"); | 299 | cgit_print_error("Invalid request"); |
301 | cgit_print_docend(); | 300 | cgit_print_docend(); |
302 | return; | 301 | return; |
303 | } | 302 | } |
304 | 303 | ||
305 | if (cmd->want_repo && !ctx->repo) { | 304 | if (cmd->want_repo && !ctx->repo) { |
306 | cgit_print_http_headers(ctx); | 305 | cgit_print_http_headers(ctx); |
307 | cgit_print_docstart(ctx); | 306 | cgit_print_docstart(ctx); |
308 | cgit_print_pageheader(ctx); | 307 | cgit_print_pageheader(ctx); |
309 | cgit_print_error(fmt("No repository selected")); | 308 | cgit_print_error(fmt("No repository selected")); |
310 | cgit_print_docend(); | 309 | cgit_print_docend(); |
311 | return; | 310 | return; |
312 | } | 311 | } |
313 | 312 | ||
314 | if (ctx->repo && prepare_repo_cmd(ctx)) | 313 | if (ctx->repo && prepare_repo_cmd(ctx)) |
315 | return; | 314 | return; |
316 | 315 | ||
317 | if (cmd->want_layout) { | 316 | if (cmd->want_layout) { |
318 | cgit_print_http_headers(ctx); | 317 | cgit_print_http_headers(ctx); |
319 | cgit_print_docstart(ctx); | 318 | cgit_print_docstart(ctx); |
320 | cgit_print_pageheader(ctx); | 319 | cgit_print_pageheader(ctx); |
diff --git a/ui-shared.c b/ui-shared.c index 95dfeb4..fba1ba6 100644 --- a/ui-shared.c +++ b/ui-shared.c | |||
@@ -566,57 +566,62 @@ void add_hidden_formfields(int incl_head, int incl_search, char *page) | |||
566 | if (ctx.qry.path) | 566 | if (ctx.qry.path) |
567 | url = fmt("%s/%s", url, ctx.qry.path); | 567 | url = fmt("%s/%s", url, ctx.qry.path); |
568 | html_hidden("url", url); | 568 | html_hidden("url", url); |
569 | } | 569 | } |
570 | 570 | ||
571 | if (incl_head && ctx.qry.head && ctx.repo->defbranch && | 571 | if (incl_head && ctx.qry.head && ctx.repo->defbranch && |
572 | strcmp(ctx.qry.head, ctx.repo->defbranch)) | 572 | strcmp(ctx.qry.head, ctx.repo->defbranch)) |
573 | html_hidden("h", ctx.qry.head); | 573 | html_hidden("h", ctx.qry.head); |
574 | 574 | ||
575 | if (ctx.qry.sha1) | 575 | if (ctx.qry.sha1) |
576 | html_hidden("id", ctx.qry.sha1); | 576 | html_hidden("id", ctx.qry.sha1); |
577 | if (ctx.qry.sha2) | 577 | if (ctx.qry.sha2) |
578 | html_hidden("id2", ctx.qry.sha2); | 578 | html_hidden("id2", ctx.qry.sha2); |
579 | if (ctx.qry.showmsg) | 579 | if (ctx.qry.showmsg) |
580 | html_hidden("showmsg", "1"); | 580 | html_hidden("showmsg", "1"); |
581 | 581 | ||
582 | if (incl_search) { | 582 | if (incl_search) { |
583 | if (ctx.qry.grep) | 583 | if (ctx.qry.grep) |
584 | html_hidden("qt", ctx.qry.grep); | 584 | html_hidden("qt", ctx.qry.grep); |
585 | if (ctx.qry.search) | 585 | if (ctx.qry.search) |
586 | html_hidden("q", ctx.qry.search); | 586 | html_hidden("q", ctx.qry.search); |
587 | } | 587 | } |
588 | } | 588 | } |
589 | 589 | ||
590 | const char *fallback_cmd = "repolist"; | ||
591 | |||
590 | char *hc(struct cgit_cmd *cmd, const char *page) | 592 | char *hc(struct cgit_cmd *cmd, const char *page) |
591 | { | 593 | { |
592 | return (strcmp(cmd->name, page) ? NULL : "active"); | 594 | return (strcmp(cmd ? cmd->name : fallback_cmd, page) ? NULL : "active"); |
593 | } | 595 | } |
594 | 596 | ||
595 | void cgit_print_pageheader(struct cgit_context *ctx) | 597 | void cgit_print_pageheader(struct cgit_context *ctx) |
596 | { | 598 | { |
597 | struct cgit_cmd *cmd = cgit_get_cmd(ctx); | 599 | struct cgit_cmd *cmd = cgit_get_cmd(ctx); |
598 | 600 | ||
601 | if (!cmd && ctx->repo) | ||
602 | fallback_cmd = "summary"; | ||
603 | |||
599 | html("<table id='header'>\n"); | 604 | html("<table id='header'>\n"); |
600 | html("<tr>\n"); | 605 | html("<tr>\n"); |
601 | html("<td class='logo' rowspan='2'><a href='"); | 606 | html("<td class='logo' rowspan='2'><a href='"); |
602 | if (ctx->cfg.logo_link) | 607 | if (ctx->cfg.logo_link) |
603 | html_attr(ctx->cfg.logo_link); | 608 | html_attr(ctx->cfg.logo_link); |
604 | else | 609 | else |
605 | html_attr(cgit_rooturl()); | 610 | html_attr(cgit_rooturl()); |
606 | html("'><img src='"); | 611 | html("'><img src='"); |
607 | html_attr(ctx->cfg.logo); | 612 | html_attr(ctx->cfg.logo); |
608 | html("' alt='cgit logo'/></a></td>\n"); | 613 | html("' alt='cgit logo'/></a></td>\n"); |
609 | 614 | ||
610 | html("<td class='main'>"); | 615 | html("<td class='main'>"); |
611 | if (ctx->repo) { | 616 | if (ctx->repo) { |
612 | cgit_index_link("index", NULL, NULL, NULL, 0); | 617 | cgit_index_link("index", NULL, NULL, NULL, 0); |
613 | html(" : "); | 618 | html(" : "); |
614 | cgit_summary_link(ctx->repo->name, ctx->repo->name, NULL, NULL); | 619 | cgit_summary_link(ctx->repo->name, ctx->repo->name, NULL, NULL); |
615 | html("</td><td class='form'>"); | 620 | html("</td><td class='form'>"); |
616 | html("<form method='get' action=''>\n"); | 621 | html("<form method='get' action=''>\n"); |
617 | add_hidden_formfields(0, 1, ctx->qry.page); | 622 | add_hidden_formfields(0, 1, ctx->qry.page); |
618 | html("<select name='h' onchange='this.form.submit();'>\n"); | 623 | html("<select name='h' onchange='this.form.submit();'>\n"); |
619 | for_each_branch_ref(print_branch_option, ctx->qry.head); | 624 | for_each_branch_ref(print_branch_option, ctx->qry.head); |
620 | html("</select> "); | 625 | html("</select> "); |
621 | html("<input type='submit' name='' value='switch'/>"); | 626 | html("<input type='submit' name='' value='switch'/>"); |
622 | html("</form>"); | 627 | html("</form>"); |