Diff options
Diffstat (limited to '')
-rw-r--r--source/ui-empty.c21
1 file changed, 12 insertions, 9 deletions
diff --git a/source/ui-empty.c b/source/ui-empty.c
index deefc21..76b2697 100644
--- a/source/ui-empty.c
+++ b/source/ui-empty.c
@@ -11,8 +11,17 @@
#include "ui-empty.h"
#include "ui-shared.h"
+static int clone_urls_printed;
+
static void print_clone_url(const char *url)
{
+ // cgit_add_clone_urls may call back no times at all, so the table
+ // waits for a first url rather than being opened ahead of the walk.
+ if (clone_urls_printed++ == 0) {
+ html("<table class='list clone' aria-label='Clone urls'>\n<tbody class='clone'>\n");
+ html("<tr class='nohover'><th class='left' scope='col'>Clone</th></tr>\n");
+ }
+
cgit_print_clone_row(url, 0);
}
@@ -23,14 +32,8 @@ static void print_clone_url(const char *url)
void cgit_print_empty_repo(void)
{
cgit_print_error("Repository seems to be empty");
-
- // cgit_add_clone_urls draws on these two alone, so with neither set
- // the table below would be a Clone heading with no rows.
- if (!ctx.repo->clone_url && !ctx.cfg.clone_prefix)
- return;
-
- html("<table class='list clone' aria-label='Clone urls'>\n<tbody class='clone'>\n");
- html("<tr class='nohover'><th class='left' scope='col'>Clone</th></tr>\n");
+ clone_urls_printed = 0;
cgit_add_clone_urls(print_clone_url);
- html("</tbody>\n</table>\n");
+ if (clone_urls_printed)
+ html("</tbody>\n</table>\n");
}